movebits: make the user-space test cases slightly easier

Make it a bit easier to write user-space test cases; use fgets+sscanf
so comment lines end up getting ignored.

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

shuffler: correctly handle one-to-many relationships

One-to-many relationships, in which one chunk of a file is used in
more than one place, tends to naturally show up in decoding certain
fileformats, including (but not limited to) Microsoft SDI.  Make the
shuffler library handle those cases correctly, and remove a
special-purpose hack in sdi.c.

This is based on the observation that all one-to-many relationships
can be treated as a one-to-one shuffle followed by
destination-to-destination copies; i.e. one copy is (arbitrarily)
assigned the "master copy" status, and all aliases are then copied
from the master copy when the master copy is already in its final
place.  All other copies can then be simply ignored for the duration
of the shuffle, just as zero-memory is.

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

shuffle: align the shuffle safe area

For performance, align the shuffle safe area to a dword boundary.

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

syslinux_dump_memmap(): make it easier to spot errors

Instead of stopping the memmap dump on SMT_END, stop it only on a null
pointer.  That way we can see if we have any bogus entries with
SMT_END.

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

syslinux_add_memmap(): fix failures at address zero, more?

syslinux_add_memmap() would fail miserably and corrupt the list if an
entry was added at address zero.  Quite possibly other addresses would
have similar problems.  Furthermore, we did an extra "optimization
pass" which should never have been necessary if the algorithm had been
correct in the first place.

This should hopefully fix ALL those bugs.

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

mboot: align the stack to a 16-byte boundary

Align the stack to a 16-byte boundary, just in case...

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

mboot: move setting regs.eax to mboot_run()

The setting of regs.eax in map_image() doesn't really make any sense;
move it to mboot_run() instead.

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

mboot: move map initialization out of map_image()

The amap/mmap initialization were buried randomly inside map_image(),
which at the very least makes the code needlessly hard to read.
Furthermore, it is at least possible that we may want to be able to
map multiple images in the future.

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

ui.inc: when looking for whitespace in kernel names look for <= ' '

Our somewhat generous definition of whitespace is less than or equal
to 32 (' '); we do not include 127 and 255 which are printable
characters in the PC codepages (in the latter case, NBSP.)

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

pxechain: remove unused ipaddrbuf

Remove unused ipaddrbuf.

core: make vk_rname default to mangled vk_vname

Change pc_label in parseconfig.inc to default vk_rname to mangled
vk_vname.  Also change spaces to tabs.  This gives a usable error
message in the case where the user selects a label without any kernel
information.

gfxboot: increase bss alignment to 4K

Increase the bss alignment (and therefore the alignment of buffers) to
4K rather than 512 bytes -- this in anticipation of future hard disks
with 4K sectors.  This only changes the actual bss location by 1.5K.

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

gfxboot: align buffers and move them into bss

Align I/O buffers to a multiple of 512 bytes, as required by the
Syslinux ABI.  Furthermore, move them from data to bss, so that they
don't occupy space in the image file.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
Acked-by: Sebastian Herbszt <herbszt@gmx.de>

NEWS: document isohybrid fix

isohybrid: preserve MBR id in isohybrid

When running isohybrid again it's nice to preserve the MBR id

There was some partial code to support setting the id using non
existing to_int, but no way to use it.

Signed-off-by: Pascal Terjan <pterjan@mandriva.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

isohybrid: support iso images over 2GB

"use integer" limits integers to 2^31-1 on 32 bits, which makes
imgsize to be -1 on DVD images

Signed-off-by: Pascal Terjan <pterjan@mandriva.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Document boot-once fix.

ADV: EDD "write with verify" is AL=01h or AL=02h, not AL=80h

The "write with verify" functionality in EDD is AL=01h or AL=02h
depending on EDD version, not AL=80h.  Since it is apparently
version-dependent, don't even try to do it; furthermore, some BIOSes
are known to reject a write with this feature requested no matter
what.  At some point we may want to explicitly read back to verify.

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

cmenu: add .gitignore file

Add .gitignore file for .c files created from .menu files.

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

mboot: disable DEBUG

Disable DEBUG in mboot.c32, which was accidentally enabled in a
previous commit.

Reported-by: Sebastian Herbszt <herbszt@gmx.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: update

linux.c32: move handling of mem= and vga= into the library function

Move the handling of mem= and vga= into syslinux_boot_linux(), so that
the user of that function doesn't need to worry about it.

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

core: make localboot another vkernel type

Change "localboot" to be another vkernel type.  It still needs some
special treatment (because it doesn't take a filename), but overall
this makes it a lot cleaner than what it was before.  It should also
avoid the problem of empty labels (e.g. menu quit) doing weird things.

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

shuffle_pm: remove unused variables and includes

Remove unused variables and #include statements
from syslinux_shuffle_boot_pm().

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

memscan: use the contents of the ebda_seg, not the pointer itself

Use the contents of the ebda_seg variable, not the address...

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

mboot: silence warning

Silence warning of unused argc.

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

Makefile: drop references to "make depend"

We no longer need "make depend" for anything.

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

mboot: include module filenames in the command line

Grub includes the kernel and module filenames in the command lines it
passes, so match that behavior and don't strip them off.

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

Fix NASM dependency generation

Fix missing -M from NASM dependency generation calls; adopt the
uniform stanza "-M -DDEPEND" even for sites that were already correct.

Also, use ">" not ">>" for obvious reasons...

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

Unify dependency generation: MCONFIG.embedded

Unify dependency generation and move common rules into
MCONFIG.embedded.

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

Unify dependency generation: com32/lib, com32/cmenu

Add the unified dependency generation to com32/lib and com32/cmenu.

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

Unify dependency generation

Make the dependency generation more common; have a general pattern in
MCONFIG, and use it in rules (not in CFLAGS).

For NASM source, in order to stay compatible with old versions of
NASM, run NASM twice; newer versions of NASM is capable of generating
dependencies simultaneously like gcc can, but that would break
compatibility with older distros.

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

rllpack: make all pointers 32 bits wide

The messing around with partial pointers for rllpack/rllunpack turned
out to be a source of bugs.  Instead, have all the values be 32 bits
wide, and require the callers to pass them accordingly.

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

memscan: correctly handle the DOS memory fallback case

Actually (try to) handle the case of finding the DOS memory amount.

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

bcopyxx: clear all flags before jumping to the successor code

Clear the flags register while we still have a stack to do so.
This makes the state in the target OS more predictable.  It's worth
noting that none of the instructions we use in either the PM or RM
trampolines change the flags, either (except the IF setting in the RM
trampoline, but that is intentional.)

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

bcopyxx: align the relocated code to a 16-byte boundary

Align the relocated bcopyxx code to a 16-byte boundary.  There are
CPUs which have errata relating to GDTs which are not 16-byte aligned.

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

mboot: add header guards; use <inttypes.h>; formatting changes

Reformat the mboot header files slightly, use <inttypes.h> types, and
add header inclusion guards.

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

mboot/solaris.c: fix failure case; correct copyright notice

Better handling of failed mapping; correct copyright notice.

mboot: reimplement the Solaris DHCP hack, add compliant a.out mode

Reimplement the Solaris DHCP information passing hack.

Add a spec-compliant mode for the "a.out kludge".  The spec is pretty
clear that the bit should override the ELF header (after all,
otherwise there wouldn't be any need for the bit), but Grub
disagrees.  We default to Grub-compliant mode, as Solaris seems to set
the bit even though it's an ELF kernel, but add the option to enable
spec-compliant mode, as apparently some versions of FreeBSD need it.

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

NEWS: add updates for 3.80

Merge branch 'syslinux-3.7x'

Conflicts:
version

Remove the old mboot module

mboot: fix cmdline; a few more layout tweaks

Fix module command lines (it was overwriting the main kernel command
line); a few minor layout tweaks.  In particular, we require the
section header to be page-aligned, but not the subsequent sections.

With this, I can get Xen to boot.

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

mboot: skip --- marker; decompress all files

When walking the argument list we need to skip --- markers.

Grub, and the old mboot.c32, seem to transparently decompress all
files, not just the main one, so do the same.

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

mboot: make sure we actually succeed when we finished

When we actually finished mapping the image, return 0 and don't bail.
Add error messages to most failure cases.

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

com32/MCONFIG: generate dependency files by default

Generate the dependency files by default.  They should perhaps even
move into the actual rule lines.

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

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>

NEWS, version: prepare for a 3.76 release

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>