mboot: set up a stack even though the spec doesn't require it

Set up a stack for the Multiboot OS even though it is not required by
spec.

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

First attempt at a rewritten mboot module

First attempt at rewriting the mboot module to use the Syslinux
shuffle APIs.

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

Merge branch 'syslinux-3.7x'

ISOLINUX: set directory length correctly

Since searchdir() uses getfssec() to read the directory it needs to set
file_bytesleft. Else it loops forever if you have a large directory (and/or
rock ridge on).

pxelinux: fix minor bugs

Fix minor bugs reported by pfranz73@tiscali.it.

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

shuffle_rm: clean up the trampoline generator with macros

Define macros for common constructs to clean up the rm trampoline
generator.

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

shuffle_rm: set up SS:ESP and segs as quickly as possible

For extra paranoia's sake, set up SS:ESP immediately after the
real-mode switch, and then set all the segment registers.

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

hdt: fix conflict

Impact: making build working again

A forgoten conflict

Merge branch 'master' of git://git./boot/syslinux/syslinux into mainline

Conflicts:
com32/hdt/hdt-cli-pci.c
com32/hdt/hdt-cli-vesa.c
com32/hdt/hdt-common.c
com32/hdt/hdt-menu-dmi.c

hdt: 0.3.1

Impact: new release

New release

Merge commit 'mouraf/for-erwan' into for-erwan

Conflicts:

com32/hdt/hdt-cli-hdt.c

a20: advance A20Test by a large, prime number

Rather than simply incrementing A20Test, advance it by a large prime
number every time.  The reason is to minimize false aliasing as
quickly as possible.

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

a20: remove DO_WBINVD configurable

We haven't enabled DO_WBINVD for a very long time.  A20 is implemented
on the inside of the L1 cache for 486+ (and 386 didn't have WBINVD),
so an I/O delay is the only thing we can rely on anyway.

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

a20: inline io_delay

The io_delay operation is only two instructions, might as well inline
it.

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

a20: A20Test is now a dword

A20Test is now a dword; this makes it less likely that we'll have a
case of false aliasing resulting in the io_delay path being taken.

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

a20: try to avoid io_delay if A20 is already enabled

We have at least the possibility to avoid io_delay if A20 is already
enabled.  Thus, give it a try.  Furthermore, when calling enable_a20,
always try the zero-work case first, since we may have been enabled by
a previous call.  This should improve performance of the rm/pm
ping-pong.

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

Global whitespace cleanup.

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

core: remove references to MDSLINUX

MDSLINUX (Multiple Disk Syslinux) will never happen, kill references
to it.

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

core: merge the startup code for disk-based derivatives

Merge the startup code for disk-based derivatives (currently SYSLINUX,
EXTLINUX) into a single file.  There is probably still additional
shared code that should be merged, but this is a good start.

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

hdt: removing useless more_printf

Impact: improving user experience

hdt>show modes was using more_printf. This is pretty useless and
annoying for users.

hdt: removing useless more_printf

Impact: none

more_printf isn't needed for such display

hdt: more_printf was eating a line

Impact: Visual. Some lines were missing at display

The more_printf call was missing some lines to display

hdt: MAC Address should only be displayed on the PXE Device

Impact: MAC address were shown on invalid devices

Prior to that commit, the mac address of the PXE booted network card was
displayed on every pci devices.... weird.

hdt: fixing typo

Impact: fixing compilation errors

Wrong call :(

hdt: Cleaning dmi chassis asset tag output

Impact: Visual

Some users reported this dmi/chassis/asset tag could reports many
spaces.

hdt: removing multiple spaces

Impact: Improving visual experience

Some hardware reported strings contains multiple spaces.
This isn't very nice for managing display. The del_multiple_spaces()
function is removing multiple spaces.

hdt: moving contact adress to the mailing list

Impact: none

Mailing list is the best place to have user inputs

hdt: Add VPD menu item

Add an item to display VPD information, if detected.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: Add vpd mode (CLI)

Add the vpd mode to dump the vpd structure, if found.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

Merge commit 'erwan/master' into for-erwan

Conflicts:
com32/hdt/hdt-cli-hdt.c
com32/hdt/hdt-cli.c

hdt: Get rid of nb_modules in struct cli_module_descr

End list of modules commands by NULL instead of managing manually
the number in nb_modules. This should limit the number of bugs when adding new
features.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

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>

hdt: Get rid of MAX_MODES

End the list of modes by NULL instead of managing manually the length.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: removing useless clear_screen

Impact: Improve user experience

Prevent clear screen when not necessary

hdt: Improving more_printf & clear_screen management

Impact: greatly improve user experience

This commit adds more coherency in more_printf & clear_screen usage

hdt: Fixing wrong printf calls

Impact: Removing memory corruption

Some printf called were badly setup

hdt: Adding reset_more_printf()

Impact: first step toward more_printf() rationalization

reset_more_printf() reset the counter used by more_printf to count
lines.

hdt: menu should be 80x25

Impact: increasing menu size to improve display

80x24 was a mistake, it have to be 80x25

Merge branch 'master' of git://git./boot/syslinux/syslinux into erwan

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.

pci: Using for_each_pci_func3

Impact: cleaning code

Using for_each_pci_func3 instead of for_each_pci_func + pci_mkaddr

Merge branch 'master' of git://git./boot/syslinux/syslinux

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>

hdt: Fixing latency indentation

Impact: Improving display

Removing useless space

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.

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>

hdt: Shortening extension when using ISOLINUX

Impact: When using ISOLINUX, we have to shorten the extension

Shortening extension when using ISOLINUX.
i.e: modules.pcimap is convertd to modules.pci

hdt: Opening console earlier

Impact: Earlier printf are now possible

Prior to that commit, it was impossible to printf during paramaters
& syslinux checks

hdt: Detect syslinux earlier

Impact: Internal changes

Some parameters might be dependant on syslinux's version

hdt: 0.3.0

Impact: This commit ends the merge with pam's branch

CLI is improved and manage completion
Thx to pierre for that great work

Merge branch 'for-erwan' of ssh://erwan@terminus.zytor.com/pub/git/users/pam/hdt-pierre into pam

Conflicts:
com32/hdt/hdt-cli.c

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>

hdt: DEL must be handle as BACKSPACE

Impact: Better conformity to standards

To conform with serial lines, DEL must be handled as BACKSPACE
Using Ctrl+D to delete a char
Taken from http://www.gnu.org/software/bash/manual/html_node/Commands-For-Text.html
(cherry picked from commit 5be7f0228dea745aa8e1fe18036517675bbe8c1e)

Merge branch 'master' into for-erwan

Conflicts:

com32/hdt/Makefile

Merge branch 'for-erwan' of ssh://erwan@terminus.zytor.com/pub/git/users/pam/hdt-pierre into pam

Merge commit 'hdt-erwan/master' into for-erwan

Conflicts:

com32/hdt/hdt-ata.c
com32/hdt/hdt-cli-pxe.c
com32/hdt/hdt-cli-vesa.c
com32/hdt/hdt-cli.c
com32/hdt/hdt-common.c
com32/hdt/hdt-menu-pci.c

VPD: Adding the Virtual Product Data detection

Impact: It's now possible to reach VPD's info

Calling vpd_decode() populate a VPD structure to display :
- VPD's base address
- Bios Build ID
- Bios Release Date
- Bios Version
- Box Serial Number
- Motherboard Serial Number
- Machine Type/Model
- Default Flash Filename

hdt: Fixing typo. Thx to dag.

Impact: Cosmetic

Typo error

Merge commit 'syslinux-3.74'