hdt: fix format string warnings

Fix format string warnings spit by the compiler.

- Sebastian

com32: remove unused variables

Silence compiler warnings by removing unused variables.

- Sebastian

bcopyxx: when going to 16-bit PM, might as well do it right

When entering 16-bit PM after shuffle and boot, we might as well
do so sanely.  Specifically, set up the data segments so that they
match the code segment, generating a 16-bit "tiny" model environment.

This makes it a lot saner to bootstrap a proper PM environment from
there if that is what the user intends.  For the presumably more
common case of RM entry, it won't do any harm, and it's only a handful
of additional instructions.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'syslinux-3.7x'

rosh: don't compile rosh.lnx by default

Don't compile rosh.lnx by default.  It causes unnecessary dependencies
on the host system.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: document HDT PCI fix

hdt: Fixing latency indentation

Impact: Improving display

Removing useless space

pci: Using for_each_pci_func3

Impact: cleaning code

Using for_each_pci_func3 instead of for_each_pci_func + pci_mkaddr

pci: add pci_for_each_func3() which produces an address

Add pci_for_each_func3() iterator, which produces an address in
addition to the device pointer.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

PCI: Fixing freeze when detecting many PCI devices

Impact: Prevent hard freeze when having many PCI devices

A user reported that calling gather_additional_pci_config() when many
PCI devices exists on a host could lead to a complete freeze.
This patch remove some useless code and use for_each_pci_func() instead.
This prevent useless memory allocation.

NEWS: add the latest stuff...

Document additional changes since 3.74.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

gpxe: Don't use "lret $2" to return from an interrupt

Using "lret $2" to return from an interrupt causes interrupts to be
disabled in the calling program, since the INT instruction will have
disabled interrupts.  Instead, patch CF on the stack and use iret to
return.

Interestingly, the original PC BIOS had this bug in at least one
place.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

menu: modify MENU SAVE to be a cascading default

Change MENU SAVE to be a cascading default that can be reverted with
MENU NOSAVE.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

doc: fix stray parenthesis in syslinux.txt

Remove stay parenthesis in syslinux.txt

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pci: add pci_for_each_func3() which produces an address

Add pci_for_each_func3() iterator, which produces an address in
addition to the device pointer.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

doc: clarify CS.base restriction for KVM

Clarify that the CS.base restriction is a prescription of
compatibility, and not something that the ABI handles transparently.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

doc: update copyright in syslinux.txt

Update and clarify copyright in syslinux.txt.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'syslinux-3.7x'

NEWS: the extlinux(1) is from Brian Pellin, not Daniel Baumann.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

NEWS: document MEMDISK fixes, updates.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Adding manpage from Brian Pellin <bpellin@gmail.com> for extlinux.

Correcting typo in extlinux documentation.

MEMDISK: traceability for the geometry

Print the source of the geometry determination.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

MEMDISK: fix MBR geometry extraction

Fix the MBR geometry extraction; there was an off-by-2 error in the
MBR signature offset.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'syslinux-3.7x'

Conflicts:
core/bootsect.inc
version

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

NEWS, version: bump version to 3.75, document keeppxe fix

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxelinux: fix the handling of KeepPXE

KeepPXE was broken in 3.74, because we tried to use both bits 0 and 1,
but failed to actually test both bits.  Instead, just use bit 0, but
move the place where we clear the bits.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core/bcopyxx.inc: save a few bytes

Muck with the GDT patching logic to save a few bytes.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Fix infinite loop in syslinux_memmap_find()

We can't use continue; since we're not using a for loop; we have to
advance to the next list entry...

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge branch 'timeout-fix'

pxelinux: fix the new timeout code

Fix the new timeout code; in particular:

- on receiving an ERROR packet, we fake an immediate timeout;
  this has to be done differently now.
- fix missing decrement of timeout counter.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

com32: use xfer_buf_seg as the bounce buffer

We cannot realistically realign comboot_seg without breaking com16
modules (which have the DOS-derived assumption that they own the rest
of low memory.)  However, we can use xfer_buf_seg as the com32 bounce
buffer, which allows us to retain the benefit of a 64K-aligned buffer.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

doc: add note about KVM restriction on CS.base

Add note about KVM restrictions on CS.base before entering real mode.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: don't misalign xfer_buf_seg

xfer_buf_seg is used for bulk transfers, and we really don't want
to break it up in order to avoid 64K DMA boundaries.

Note: the same is true for the com32 bounce buffer, which is
real_mode_seg.  However, there are hard assumptions in the code that
real_mode_seg is the last segment; those need to be cleaned up, or we
need to verify that xfer_buf_seg can be used as the com32 bounce
buffer.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Something slightly saner for DummyTSS

DummyTSS should hopefully never be actually used.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

GDT: remove unused DS16_4G and SS16 descriptors

We no longer use the DS16_4G and SS16 descriptors; we now use 32-bit
protected mode for all "big" work.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

shuffle_rm.c: minor style cleanup

Try to make the code just a little easier to read.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

shuffler: work around KVM problem with the new shuffler

KVM uses V86 mode to simulate real mode.  This causes problems with
the new shuffler.  This changes the shuffler handover to be in
16-bit protected mode instead, and requires the stub to do the actual
entry to real mode.  For the KVM hack to work, all segments must have:

(seg.base & 0xfff0000f) == 0 && seg.limit == 0xffff

As a result, we have to make sure the real-mode entry stub is
paragraph-aligned, lest we violate the first criterion.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: move vgafontbuf out of the zero segment

vgafontbuf is one of the largest (bss) structures in the zero segment,
together with the trackbuf (which we realistically can't move.)
Create a new "auxilliary segment" for fixed-sized data which needs to
be in the low megabyte, but not necessarily in the zero segment.

This pushes up the low memory usage, but only by 8K.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core: add alignz macro (analogous to "alignb") for zero-pad align

Add a new macro "alignz" to align with zero bytes; use to align inside
a data section (i.e. a progbits section which doesn't contain code.)

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core/macros.inc: iterative definition of asciidec

A cleaner iterative defintion of the asciidec macro.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

bootrm: allow entering with IF=1; smarter stub location

Change the stub location algorithm to prefer low memory above
0x800; above 0x510 if that is unavailable.  Also add the ability
to invoke the real-mode code with IF=1 if so is desired.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxelinux: use a table-driven backoff

Use a table-driven backoff so we can provide a backoff which doesn't
increase as sharply (by integral powers of two) as before.  Rather
than worrying about doing fractional multiplication, simply provide a
backoff table.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxelinux: be more aggressive about retransmitting ACKs

Some PXE stacks drop packets on transmit on a regular basis.  Avoid
severe slowdowns by being much more aggressive about ACK
retransmissions.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Makefile: add missing MBR variants to BOBJECTS

Add missing variants to BOBJECTS.

pxelinux: be more paranoid about saving/restoring flags

Save and restore flags around writechr and the PXE stack call, just in
case the underlying stack does something weird.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'syslinux-3.74'

gfxboot: parse DEFAULT keywork in syslinux config file

The gfxboot COM module currently ignores the DEFAULT entry set in
syslinux config file and always default to 0 instead. This patch parses
DEFAULT entries in the config file and set the default entry
accordingly.

Signed-off-by: Christophe Fergeau <cfergeau@mandriva.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Call the next version 3.80

The new shuffler machinery is a major enough change that it's probably
a good idea to bump the version number to 3.80.

Merge branch 'new-shuffler'

hdt: clear memtest_label

Clear memtest_label in init_hardware().

com32: the keyboard map size is in CX not in DX

Impact: fixes kbdmap.c32

The keyboard map size is passed in CX, not in DX.  Fix typo in
libcom32.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

gfxboot module: fix copyright notice

Include copyright notices of reused code.

NEWS: document gfxboot.com

Gfxboot COMBOOT module

A COMBOOT module to interface to Steffen Winterfeldt's Gfxboot
graphical front end (http://gfxboot.sourceforge.net/).

Merge commit 'hdt/master'

Add back "install: all" to the com32 devel directories

"make install" in the com32 directory currently installs the devel
environment, which should be build using the host toolchain... so make
it so.

Makefiles: try to make "make install" do the right thing

One more attempt at getting semi-sanity to "make install".
Realistically, the install/netinstall/install-all/... needs to be
pushed into the com32 Makefiles, but this is at least a quick hack.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Implement MENU SAVE; fix COM32 setadv function

Implement MENU SAVE, which allows the menu system to retain the
previous selection from one boot to another.  In the process, fix the
syslinux_setadv() function.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: unbreak "make depend", fix make spotless

cmdline.inc: Unbreak "make depend"
Makefile: fix "make spotless"

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: document cmdline fix

ipappend: clean up ipappend code; add ipappend to plain cmdline

ipappend information is needed for the plain cmdline too, since it may
be used by linux.c32.  While we're at it, clean up the handling of
ipappend lines -- we can rather obviously use a loop for them.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

hdt: Don't display unsupported modes

Impact: Prevent from displaying 0x0 modes

Some vesa bioses reports 0x0 modes, no need to display them

Provide a shared MIME file for the LSS16 image format

Add a MIME file for the LSS16 image format.  Currently only included
in source, as it is somewhat unclear what the proper way to package
this thing is.

syslinux.ld: fix comment syntax

Fix build failure due to incorrect comment syntax in syslinux.ld.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memscan: adjust lower limit; saner handling of insane DOS memory sizes

Adjust the lower limit of DOS memory to 0x510; with the last known
BIOS byte in use at 0x500 (and only then for Print Screen) this makes
sense.

If the DOS memory size is clearly insane, fall back to using the EBDA
address.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge branch 'master' into new-shuffler

memscan: protect 0-0x7c00 until the new shuffler is merged

Mark the memory range 0-0x7c00 unavailable until the new shuffler code
is ready, at which point we can go for a tighter bound.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Makefile: install new MBR variants; NEWS: document

Make the new MBR variants installables, and add to the NEWS file.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memdisk: fix comment syntax in memdisk.ld

Fix build failure due to incorrect comment syntax in memdisk.ld

Merge branch 'ctrl-mbr'

mbr: generate normal versions, force-80 versions, and ctrl-80 versions

Produce MBR variants with normal (DL) behavior, force to drive 80
behavior, and force to 80 on Ctrl pressed behavior.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Update copyright notices

Update copyright notices; add Intel copyright notices where
appropriate.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Update copyright notices

Update copyright notices; add Intel copyright notices where appropriate.

Merge branch 'master' into new-shuffler

Conflicts:
com32/lib/syslinux/memmap.c

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Update copyright notices; add Intel notices where appropriate

Update copyright notices.  Per agreement with my new employer, Intel
Corporation, add Intel copyright notices where appropriate.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'hdt/master'

linux.c32: don't relocate the real-mode code *above* 0x90000

Do not relocate the real-mode code above address 0x90000... that would
not be prudent.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

linux.c32: make load_linux() support relocation

In the case where the preferred kernel locations are not available,
make load_linux() smart enough to be able to relocate the kernel if at
all possible.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

com32: add _t to scan_memory_callback

Change scan_memory_callback to scan_memory_callback_t

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

com32: merge all memory map discovery to one file

Merge the memory map discovery for malloc and the memory map discovery
for memmap into one file that scans memory and invokes a callback.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

hdt: Adding exit menu in menu mode

Impact: Users can now exit hdt :)

It's now possible to exit HDT properly. Thx to fcrozat report.

hdt: Fixing aligment in about menu

Impact: cosmetic

Fixing contact in about mode

hdt: Fixing author name

Impact: cosmetic

Fixing author name

NEWS: document new com32 memory management

com32: fix the argument order

Impact: bug fix

Fix the order of arguments to a com32 program.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

com32: make memory beyond the core HighMem available to malloc

Impact: should deal with memory holes more gracefully

Right now, if we find a memory hole, we simply don't use the memory
beyond that point.  This makes it possible for com32 modules to
use that memory.  After this, we should be able to add relocation
support to allow loading when the memory at 1 MB is unavailable.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: fix typo

shuffler: zero-extend the proper register for shuffle list length

The shuffle list length needs to be passed into the shuffler in ECX,
but when using the shuffler internally it is a 16-bit number, which
needs to be zero-extended.  Zero-extend the proper register, please!

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

shuffler: use lss for real-mode return in simple_pm_call

Instead of doing weird stuff with BP and the stack, just do an atomic
reload of SS:ESP by using the LSS instruction.

shuffler: minor cleanups in bcopyxx.inc

Minor cleanups: calling convention, documentation, etc.

shuffler: reload EDI, ESI, ECX before calling shuffle_and_boot_raw

comapi_cleanup can clobber registers, so reload EDI, ESI, ECX before
calling shuffle_and_boot_raw.

shuffler: fix infinite loop in syslinux_memmap_find()

Fix an infinite loop in syslinux_memmap_find().

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

chain.c32: remove misleading comment

Remove a never-implemented, misleading comment

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

shuffle: use SMT_RESERVED for the forbidden region

If we're going to mark memory unusable, it's cleaner to mark it
SMT_RESERVED.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

chain.c32: remove loadbase < 0x7c00 hack

Remove the special hack for loadbase < 0x7c00.  We no longer need it,
since the shuffler will relocate appropriately by magic.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

shuffler: make the new shuffler actually work

Make the new shuffler actually work.  This includes changing
rllpack.inc to run in 32-bit mode (since simple_pm_call now switches
to 32-bit mode) and changing the new shuffler interface to move the
shuffle list before actually doing any work.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

shuffler: comboot interface to the new shuffler

Impact: new feature

COMBOOT interface to the new shuffler; resurrect comapi_chainboot as
that functionality is still potentially relevant.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Banner: "and contributors" is just too long; try "et al"

Impact: aestetics

Change "and contributors" to "et al" so that the message actually fits
on one line.  This is unfortunate, but what can you do...

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

shuffer: make the new shuffler not pollute unrelated memory

Keep the shuffler from polluting memory outside its own "safe area".
This means being more clever about the relocation code, but it should
make it a lot easier to use for our own uses.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>