core: fix mismerges with the master branch

Fix a couple of minor issues caused by mismerge with the master
branch.

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

Merge branch 'master' into core32

Conflicts:
core/bcopy32.inc
core/cleanup.inc
core/conio.inc

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

Don't set the autocr flag on the serial console; clean up crap

libutil would set the autocr flag on the serial console, which really
never was any point -- we already do \n -> \r\n conversion explicitly
in the serial code.  This was always very annoying to deal with if the
menu was interrupted.

Furthermore, drop completely unnecessary
initialization/deinitialization routines that completely duplicated
other code.

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

core: don't flush the serial port queue for a serial command

Don't flush the serial port queue when encountering a "serial"
command.  We don't actually want to lose data due to a repeated
"serial", since it's quite likely it's exactly the same as before.

Do flush the queue on hardware cleanup, however.

Also fix some minor bugs, including a bunch of code in the .data
segment.

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

core: add file missing from previous checkin (serirq.inc)

Add the file serirq.inc missing from previous checkin.

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

core: remove obsolete comment

Remove comment about flipping A20 for each 64K block copied.

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

core: add a proper interrupt handler for the serial console

If we enable interrupts for the serial console, add a proper interrupt
handler.  Since we don't know what vector we'll end up using, or if we
are shared with other devices, simply hook *all* the interrupts and
poll the serial port then.

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

Merge branch 'master' into core32

Conflicts:
core/idle.inc

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

idle: set NoHalt back to 0 as it should be

NoHalt was set to 1 temporarily for debugging; return it to 0.

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

isohybrid: add missing isohdppx.S file

Add a file missing from previous checkin
e462c28ffaca0132c1761736bc93cb06a41dc7a6.

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

Drop support for ACPI 3 E820 extended memory attributes

Drop all support for ACPI 3 E820 extended memory attributes.  There
are BIOSes in the field that report completely bogus information here,
resulting in no memory at all being detected (we then fall back to
E801 detection, but that is problematic in its own ways.)

There is strong reasons to believe at this point that the extended
memory attributes are not usable in their current form, so drop them
and revert back to simple 20-byte support, including for MEMDISK
spoofing.

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

NEWS: update isohybrid changes

isohybrid: support booting from partition; fix CBIOS booting

Fix CBIOS in isohybrid mode.  Also allow an isohybrid image to be
booted from a partition.  Unfortunately this breaks compatibility
between differing versions of isohybrid and isolinux.bin.

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

isohybrid: allow selecting the _c and _f versions of the prefix

Allow selecting the _f or _c versions of the prefix in addition to the
default one.  This is specified with the -forcehd0 or -ctrlhd0
options.

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

idle: handle PXE stacks which improperly disable interrupts

At least Etherboot (and all-but-super-recent versions of gPXE) PXE
ROMs improperly disable interrupts when calling an intercepted version
of INT 15h and 1Ah; this is due to the old trick of using "ret 2" to
return... this avoids resetting the flags for status, but it also
doesn't restore the value of the interrupt flag.  Needless to say,
this causes serious issues.

Work around it by adding explicit pushf/popf or STI in places known to
have issues, but also add an STI in reset_idle, and add an error alert
in do_idle if we ever get called with interrupts disabled.

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

chain.c32: remove clobber of drivename/partition

Remove a clobber of drivename and partition, after we have spent time
computing what the should have been...

Reported-by: Luciano Miguel Ferreira Rocha <strange@nsk.no-ip.org>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

isohybrid: add options, change default ptype to 0x17

Add support for setting a variety of options in isohybrid.  Also
change the default partition type to 0x17, "Windows hidden IFS", as
that seems to make Windows less unhappy.

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

core: drop double clearing of .uibss

The .uibss and .auxseg are cleared together now, but .uibss was also
cleared separately.

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

Merge branch 'master' into core32

NEWS: add note about linux.c32 having a quiet command

meminfo: distinguish between no flags and flags=1

For debugging reasons it's important to know if we have no flags at
all or if we get flags = 1.  Print [-] in the case of no flags at all.

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

core: LZO compress the PM part of the core

Use LZO to compress the PM part of the core.  LZO is not the best
compression algorithm, but it is very fast, and the decompressor is
only 447 bytes long.  The LZO code is part of the LZO 2.03 library.

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

core: fill unused space in .text with NOP not zero

Fill unused space in the .text section with NOP, not with zero.

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

upx: try --lzma first, since --ultra-brute doesn't... sigh

You'd think upx --ultra-brute would test everything that is possible,
but not so.  --lzma needs to be specified separately.

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

dos: run UPX on the DOS installer binary if we have it

If UPX is installed on the host system, we might as well use it to
compress the DOS installer binary.

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

dos: add a prototype for calloc()

Add a prototype for calloc() to the DOS installer library.

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

Standardize the names SECTOR_SIZE and SECTOR_SHIFT

Use the names SECTOR_SIZE and SECTOR_SHIFT everywhere, instead of an
odd mix of symbols and hard-coded constants.

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

FAT: change DOS installer to EXE; additional 32K limit fixes

Additional fixes for the 32K limits in the installers.  In the case
of the DOS installer, that means changing it from COM format to EXE
format (since COM format has a 63K hard limit); retain the name
syslinux.com for user compatibility, though (DOS doesn't care what the
extension except for pathname search; if it finds an MZ EXE header it
will use it.)

With the change to EXE means having to handle more than one segment.
Since we don't have a real DOS compiler we have to wing it a bit.

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

libinstaller: "nsect" is the size of ldlinux.sys, not the total

"nsect" here is the size of ldlinux.sys, not the total structure;
therefore it should not be decremented.  Again, all this should really
be unified.

With this patch the FAT derivatives boot with the current code.

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

libinstaller: fix definition of bsHeadLen

Correct the definition of bsHeadLen, the size of the region at the
start of the boot sector that should always be copied.

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

FAT: update the FAT installers to match the new diskstart.inc

Update the FAT installers to match the new patch area format in
diskstart.inc.  Also, update the coding style to match extlinux; this
code should eventually get unified.

This does not resolve the current issues with the DOS installer when
ldlinux.sys exceeds about 48K, at which point the DOS installer needs
to become a segment-aware EXE program.

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

pxelinux: revert to a separate stack implementation

Return to using a private stack, except immediately around a pxenv
call.  The systems on which this broke has been lost in time, and with
more code in 32-bit space there is a risk of deeper stacks (due to
more mode-switch calls.)

It is still a single configuration variable to flip back to the
unified 16-bit stack, however.  If most of the code is in 32-bit mode
and there is small likelihood of several transitions, it might still
be the best option in the end.

Note: the PXE stack is 1.5K in size (and we must account for the
possibility of receiving an interrupt.)

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

pxelinux: set StackTop == StackBuf

We need StackTop to be <= StackBuf since we use memory immediately
above StackBuf during replace_bootstrap.

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

core: use reserved stack space only for replace_bootstrap

When running replace_bootstrap, our regular stack is still live.
Therefore, (a) make sure we have 44 bytes reserved above the normal
stack, and (b) use those, i.e. StackBuf, not StackTop.

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

Merge branch 'master' into core32

Don't run to comboot API with interrupts off

We have historically run the comboot API with interrupts off due to
concerns about stack overflow.  However, this is a really bad idea: we
can easily spend a fair bit of time servicing one of these routines,
especially when doing things like waiting for I/O.

In particular, do_idle should *never* be run with interrupts disabled.

Switch to running with interrupts enabled everywhere unless we have
specific reasons not to do so.

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

Merge branch 'master' into core32

core/conio.inc: when reading the serial port, drop read of IIR

Drop the read of the IIR when reading the serial port.  First of all,
we weren't protecting AL so we clobbered the actual data; second of
all, this isn't actually necessary as reading RDR will clear the
interrupt condition per the UART spec.

It's worth noting that enabling interrupts here will do bad things if
the interrupts aren't edge-triggered (since we don't actually have an
interrupt routine to clear the interrupt condition); this also means
bad things will happen if the interrupt line is shared.

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

Merge branch 'master' into core32

Merge branch 'master' into core32

Conflicts:
core/extlinux.asm
core/isolinux.asm
core/ldlinux.asm
core/pxeidle.inc
core/pxelinux.asm

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

core/pxeidle.inc: fix pops with no matching push

In check_for_arp, we dropped the segment register pushes, but we need
to drop the pops, too.

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

Try to HLT the processor during idle

Try to HLT the processor during idle.  All the events we care about
should have interrupts associated with them, except possibly the
serial console.  Try to deal with the serial console by waiting some
time before going into HLT, and giving the user the option of enabling
the serial console interrupt, on the assumption that the BIOS will
simply IRET.

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

core: change "Linux needs..." to "Syslinux needs..."

Linux really only needs about 64K or so of low memory these days, but
Syslinux needs more.

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

version: set this version to 4.00

Assume that whatever version ends up using the core32 stuff it will be
called 4.00.

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

Move the NASM debugging options to a separate variable

Move the NASM debugging options to a separate Makefile variable,
NASMDEBUG, so they can be overridden if necessary.

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

core: move common module includes to common.inc

A lot of modules are common to *all* the derivatives (as opposed to
just "most"); move those to common.inc.

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

core: move real-mode callback code to callback.inc

The real-mode callbacks are not specific to the comboot API, nor are
they com32-specific anymore.  Move them to a separate file, and let
com32.inc be only the items related to comboot files proper.

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

core: add .textnr to layout.inc

Add .textnr to layout.inc, so it gets the proper flags.

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

core: move 32-bit code out of the .text16 segment

Move all 32-bit code out of the .text16 segment.  The code that is
used during the relocation of the .text segment (i.e. anywhere on the
bcopy path) is moved to .textnr (other than the code already in
.bcopyxx) for "no relocate", the rest of the code to .text.

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

core: move "-g -F stabs" to NASMOPT

Move -g -F stabs to NASMOPT.  Eventually we might need to disable them
for older versions of NASM, since the linker seems to get unhappy
about mixing older NASM stabs with gcc stabs.

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

Merge branch 'master' into core32

Conflicts:
core/graphics.inc

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

bcopyxx: remove 16-bitisms

Remove a couple of 16-bitisms in the bcopy code, since it is now
running in perfectly ordinary 32-bit mode.  In particular,
prefer 32-bit registers to 16-bit registers, and drop "a32" prefixes
(which do nothing.)

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

bcopyxx/memmove: fix alignment logic for reverse moves

The alignment logic for reverse moves is reversed, because the initial
edi, and therefore edx, points to the last byte, not to one byte
beyond the end.  Therefore, in the fully aligned case it will end in
11 binary, not in 00 binary as for the forward case.

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

Fix clobber of the command line when using F-keys+LSS graphics

Fix a half-entered command line when getting clobbered when pressing
an F key that in turn invokes an LSS graphic.  This was used by
duplexing the command line buffer as LSS decompression space.

Allocate a separate buffer, but move the legacy graphic variables to
.bss2 to avoid overflow.

Reported-by: Михаил <from.miha@gmail.com>
Debugged-by: Sebastian Herbszt <herbszt@gmx.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: make auxseg a piece of address space assigned by the linker

Let the linker assign a place for the auxseg using normal linking
rules.  Place it between the main 16-bit and 32-bit code.  Eventually
the fontbuf will probably move into the 32-bit bss.

This also avoids the weirdly aligned real_mode_seg.

In the future, expect xfer_buf_seg and real_mode_seg to merge into a
single 16-bit bounce buffer and cache_seg to go away.

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

core: add pm_call convenience macro

Add a pm_call convenience macro, instead of using stub routines
everywhere.  Stubs would still make sense if we have a routine which
gets invoked from a lot of places, though, since the pm_call expands
to 9 bytes as opposed to 3 bytes per call site plus a 10-byte stub.

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

core: make the intcall/farcall/cfarcall routines reentrant

Make the intcall/farcall/cfarcall routines reentrant, by dropping the
use of RealModeEAX and by saving/restoring Com32SysSP on the PM stack.

Furthermore, drop the saving and restoring of EAX, EDX, ECX; trying to
save all registers complicates the code, and those registers will be
assumed clobbered by the caller anyway.  Might as well save a little
bit of stack space, too.

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

core: make the COMBOOT API available to in-kernel PM code

Make it possible to call the COMBOOT API from in-kernel PM code,
simply by setting up the COMBOOT API earlier and only tearing it down
during final shutdown.

WARNING: the COMBOOT API is quite possibly probably not reentrant; I
haven't checked it...

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

core: move core definitions to include/core.h

Create a new include file for core-specific definitions, and put it in
the include directory.

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

core: rewrite rllpack in C

Rewrite the rllpack implementation in C, as a first step and test
case.

In particular, this illustrates how to access real-mode registers from
protected-mode C.

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

core: link with libcom32 and libgcc

Link libcom32 and libgcc into the core.  That doesn't mean all the
functionality of libcom32 is usable in the core!!!

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

core/diskstart.inc: break transactions on 64K boundaries

getlinsec doesn't watch for 64K boundaries, so we need to do it
ourselves.  Break a loading run if we reach a 64K boundary.

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

core/diskstart: handle more than 2^16 dwords

Handle more than 2^16 dwords in the checksumming loop.

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

BUGS, TODO: deleted outdated, no longer maintained files

These files are outdated and haven't been updated for years.  This
kind of stuff probably would work better on the wiki anyway.

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

core: handle more than 32K of code for disk-based derivatives

Handle more than 32K worth of code for disk-based derivatives.  We do
this by allowing the sector pointers to overflow past sector 1; this
is OK because we limit a run to be based on only the pointers that we
have read so far.

XXX: This is implemented for EXTLINUX, but breaks SYSLINUX.  Need to
update (and unify!) the SYSLINUX installers to cope.

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

core: adopt the naming convention core_* for exported symbols

Adopt the naming convention core_* for symbols exported from the core
code.

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

core: syslinux.ld: explicitly mark .adv and .uibss as NOLOAD

Explicitly mark the .adv and .uibss sections as NOLOAD; this seems to
avoid a linker error for reasons (VMA of these sections colliding with
the LMA of the 32-bit code.)

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

First cut of doing 32-bit C code in the core

A very preliminary implementation of 32-bit C code in the core.  This
currently breaks ADV, because the installer assumes the ADV is the
last two sectors in extlinux.sys -- need to figure out a way to deal
with this hopefully without breaking the migration protocol.

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

core: rename .text, .data and .bss to .text16, .data16, .bss16

Rename the .text, .data and .bss sections to .text16, .data16 and
.bss16, in anticipation of being linked with compiler-generated 32-bit
code, which presumably would like to use the standard section names.

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

core: zero bss and uibss; fix some section confusions

Set .bss and .uibss as soon as we are fully loaded.  This gives us the
more familiar behavior of most normal execution environments.  The
.earlybss section is not zeroed; therefore, all variables that are set
before we have the opportunity to zero need to go in this section.

This checkin also fixes some incorrect section directives.

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

core: obsolete .bss1 and .bss2

Remove the obsolete .bss1 and .bss2 segments; they represent segment
balancing which is no longer useful.  The current balance between
.earlybss, .bss and .uibss seems to be sufficient; if we need more
balancing in the future then we need to start over anyway.

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

core: move initial code to a new .init segment; unify .bss+.bss1

Move code used before loading is complete to a new .init segment;
.text is now only the stuff that is used at any time.  Move the .bss1
segment down to where .bss and .bss2 already are; it seems to fit
better there now.

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

core: simplify pm_call usage

Simplify the pm_call usage:

- pm_call now saves and restores all registers
  callees that want to return values in registers can mess with
  the frame on the stack
- the PM function now enters with EAX = EBP = RM stack
  (EAX for the use of C code, EBP for assembly)
- the RM stack frame matches com32sys_t
- pmcall.inc defined macros for the stack frame based on EBP

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

core/pm.inc: shorten the time until we have a valid stack

Cut the time until we have a valid RM stack as much as possible.

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

core/bcopy32.inc: provide correct output esi/edi from bcopy

Make sure the outputs from the bcopy routine are set correctly.
pm_bcopy doesn't do this for us, so we have to do that ourselves.

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

core/com32.inc: fix bits 16/bits 32 confusion

The unification dropped a bunch of bits 16/bits 32 that are needed to
make things work.  Add those back where needed.

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

pm_call: only enable interrupts in PM if interrupts on in RM

When executing pm_call, only enable interrupts in PM if we were called
with interrupts enabled in RM.

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

New attempt at unify protected mode entry/exit

Another attempt at unify protected mode entry/exit, based on the
previous bcopyint branch.  This should, among other things, give a
"full service" PM environment including BIOS upcalls and interrupt
service to the core-internal code.

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

linux.c32: replace the kernel name with BOOT_IMAGE=

The BOOT_IMAGE= argument is generated by replacing the kernel name in
the argument array.  As a result, we shouldn't advance argp.  Move the
code around slightly, to make it more obvious that that is what is
happening.

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

linux.c32: honor the "quiet" flag

Honor the "quiet" flag for linux.c32, since some distros have odd
notions about these things.

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

bcopyxx: EDX already points to the GDT; no reason to recompute

We already have the EDX register pointing to the GDT, so recomputing
the address of the GDT is completely pointless.  This shaves an
additional few bytes off the bcopyxx safe area (now down to 543 bytes
including the 128-byte stack and the 15-byte alignment slop.)

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

load_linux.c: quiet warning

Quiet a (harmless) compiler warning.

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

load_linux.c: consider a zImage kernel to be non-relocatable

It's pretty safe to say there has never been a working relocatable
Linux zImage kernel, and the semantics for them are iffy at best.
Consider zImage kernels to be non-relocatable.

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

load_linux.c: when relocating, need to update code32_start

The normal value for code32_start is based on the kernel load address
(in fact, it usually *is* the kernel load address); accordingly, it
needs to be updated if the kernel is relocated.

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

load_linux.c: add missing header file

We need <minmax.h> for min().

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

linux.c32: cap the stack pointer to 0xfff0

Linux itself is fine with sp = 0 for a 64K segment, but perhaps other
things aren't.  Just to be safe, set the stack pointer to 64K-16
bytes.

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

NEWS: document linux.c32 fix.

shuffler: fix setting up esp in real-mode shuffles

The value of esp set in real-mode shuffles was wrong.  This
unfortunately broke linux.c32 :(

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

poweroff: add copyright notice

Add a copyright notice.

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

Next version will probably be 3.81

core: remove vestiges of using nasm -f bin

Remove a few things that are specific to NASM's "bin" backend, which
we no longer use.  Newer versions of NASM print warning messages which
are unnecessary.

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

NEWS: drop plans for a 3.76 release

The aim now is for a 3.80 release instead...

Document poweroff module.

poweroff COMBOOT module

This module is able to power off a system via APM.
It was tested on QEMU, Bochs and VMware.

Possible usage:

TIMEOUT 3000
TOTALTIMEOUT 9000
ONTIMEOUT poweroff.com

- Sebastian

mboot: handle ELF Multiboot kernel where paddr != vaddr

The way Grub handles ELF Multiboot kernels where paddr != vaddr is to
load at the paddr, but to also adjust the entry point (e_entry) so
that the previous vaddr becomes a paddr.  Since the Multiboot spec is
pretty much "Grub wins", follow this behavior.

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

shuffler: when looking for temporary memory, it has to be *free!*

free_area() ended up looking for any area of memory, not necessarily a
*free* area of memory.  This had predictably disastrous consequences.

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

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>