Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Fix prototype for generate_extents

Add missing const

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

syslinux: synchronize generate_extents() with extlinux

Synchronize the code for generate_extents() from extlinux to syslinux.

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

extlinux: remove debugging printf's

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

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Switch to 64-bit sector pointers everywhere

Switch to consistent use of 64-bit sector pointers; this should enable
booting even for individual *partitions* larger than 2 TB.  In order
to not slow down the boot too much, switch the initial load from an
enumeration to an extent map.  This means the table gets larger (since
we have to assume the worst case), but it simplifies the Sector 1 code
(since we can push all the hard stuff into the installer), and will
speed up booting in the general case.

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

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

core: add hlt to infinite loop

We should never hit it, but if we do, don't burn the CPU to a crisp.

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

Implement !GPT protocol in Syslinux core; handle offset > 2 TB

Implement the !GPT handover protocol in the Syslinux core, and handle
partition offsets above 2 TB.  We do not yet handle filesystem sizes
above 2 TB, but that should be a reasonably straightforward extension
at this time (need to switch to 8-byte block pointers).

This finally meant moving getlinsec out of the boot sector, since it
no longer fits.  Instead have a very simple getonesec implementation
in the boot sector.  getlinsec still fits in Sector 1, although it
doesn't leave space for very many block pointers.  That's still better
than adding yet another boot loader stage, however.

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

gptmbr: change synthetic partition type to 0xED

The committee didn't like re-using 0xEE for this purpose and wants
0xED instead.

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

Merge remote branch 'sherbszt/gfxboot32' into pathbased

fat: handle .. pointing back to the root directory

.. pointing back to the root directory will have a cluster number of
0, even for FAT32 where there is an actual cluster number for the root
directory.  Handle this as a special case.

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

gfxboot: set media type

Set media type according to the drive number and cd mode.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

gptmbr: save four bytes

Save four bytes by observing that none of our code relies on
saturate_stosl not actually corrupting %eax.

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

gptmbr: fix offset for MSW of partition end

Fix the offset for the MSW of the partition endpoint.

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

gptmbr: implement the new T13-approved GPT protocol

My GPT-based protocol was modified by the UEFI and T13 committees (the
former responsible for the GPT format, the latter for EDD and
therefore for the disk-related part of the BIOS specification.)  This
is thus now on its way to become an official protocol, so change the
implementation to match.

This still needs testing, and the Syslinux core needs to be adjusted
to leverage the extended information.

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

syslinux: Correctly handle the case of installing into the root

When installing into the root directory, we should logically search
the root directory first.  Thus, we need to make sure the current path
is "/" not "".

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

setadv: asprintf() doesn't return an error code

asprintf() doesn't return an error number, but a count of characters
(which we don't need.)

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

syslxopt: add missing break;

Add missing break; in the command-line parser.

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

loadconfig: fold FAT pathname search into generic_load_config

Fold the FAT pathname searching into generic_load_config; make it a
simple set of loops.

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

syslinux: provide for backwards compatibility

Provide for backwards command-line compatibility, together with
warning messages where appropriate.

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

syslinux: don't break -o just yet; print warning and resume

-o has been supported for a very long time; recommend using -t
instead, but for now proceed with an error message.

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

Unify instances of integer log2

Unify multiple open-coded instances of integer binary logarithm.

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

Clean up warnings in previous checkin

Clean up warnings in checkin
146c34a2 Make syslinux installer real "pathbased"

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

Make syslinux installer real "pathbased"

The previous merging extlinux/syslinux patch is not sufficient, the
syslinux is not real "patchbased", hence:

* fill currentdir in patcharea
* search "extlinux.conf" and "syslinux.cfg" for generic_load_config
* define boot_image and boot_sector macro for ldsyslinux
  this could make further merging extlinux and syslinux possible

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

mboot.c32: don't rely on possibly incomplete fields

Since load_end_addr and bss_end_addr may be incomplete, we can't
actually use them for information.  An alternative would be to
actually modify these fields in-memory, but it is unclear if that is
permitted by the spec.

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

mboot.c32: handle non-self-terminating images

Handle non-ELF Multiboot images with 0 entries for data and/or bss
end, which means "until end of the file" and "nothing" for the two fields,
respectively.

Reported-and-tested-by: Thomas Bächler <thomas@archlinux.org>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

menu docs: document MENU SAVE caveat with RAID-1

The usual RAID-1 ADV constraint...

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

extlinux: add a --menu-save option

Add the --menu-save option to set the MENU SAVE value from the running
system using extlinux.

From: Thomas Bächler <thomas@archlinux.org>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

extlinux docs: recommend the --raid option

When booting from RAID, recommend --raid.

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

docs: explain the danger of IPAPPEND 1

Explain what the actual issue with IPAPPEND 1 is, and what the
constraints on using it really are.

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

chain.c32: add support for loading GRUB stage2

This adds (basic) support for GRUB stage2 image files.

Loading a stage2 image will probably give you a GRUB prompt, with GRUB's
"root" set at "(hd0)" (ie, entire first disk). Maybe the "root" will
differ in less common setups. (One can of course select another disk
and/or partition with GRUB's "root" command.)

This has only been tested with version 3.2 stage2 images (as used by
GRUB 0.97). I'm not familiair with differences with other versions.

GRUB's loading mechanism allows to somehow provide stage2 with the
selected disk and partition, BSD slice, etc. (ie, to tell stage2 what
it's "root" is). I don't yet understand the notation used in that
mechanism. Besides, since stage2 images will not necessarily be loaded
from the disk (and partition, etc.) they were installed to, it seems
best to just use the first disk.

GRUB stage1_5 image files load quite similarly. However, for some
reason, a short test only got those images to print an error ("Error
17"). This could be related to the partition info these images are
provided with when they're loaded. I have never used stage1_5 images,
and do not know how to properly use and configure those, so my test
stopped there, and stage1_5 images are not supported.

Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

gfxboot: use syslinux_derivative_info()

Use syslinux_derivative_info() instead of open coding it.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

MENU IMMEDIATE: hotkeys which do not require Enter

The normal behavior for a hotkey is to jump to a specific menu entry.
With MENU IMMEDIATE, it activates the menu entry as well.

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

Merge remote branch 'origin/master' into pathbased

isohybrid: use getopt_long_only()

For compatibility with isohybrid.pl, use getopt_long_only() so long
options are accepted with a single dash.

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

Merge remote branch 'origin/master' into pathbased

Resolved Conflicts:
com32/modules/cat.c

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

isohybrid: create a C version

Some Linux distributions have complained about Perl as a prerequisite
for isohybrid, so create a C version.

[ hpa: modified the array generator to put it all in one script ]

cat.c32: handle multiple files, use argv[0], copy 4K at a time

Loop over multiple files, use argv[0] for the program name, and copy
4K at a time.

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

unify common parts of extlinux and syslinux installer

Thus we can share same command line options and reduce a lot of dup
code...

Seems like a big patch, but the changes are quite safe, no much logical
change.

Signed-off-by: Alek Du <alek.du@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

syslinux: fix sector arrays

The first sector ptr is in bs->NextSector not in the array in
the patch_area. And actually the ADV sectors counting is wrong for a
while ...

Signed-off-by: Alek Du <alek.du@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Restore skip CR and LF in the comboot API copies of the banner strings

The comboot API should not report leading CR and LF in the banner
strings.  This was done in 3.86, but apparently was lost due to
mismerge (52c9d5ba2af4e1f77aa2486a2af2bcab0b0af52c) in the 4.00 branch.

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

core: fix "sector size" confusion

Fix the case where the "sector size" used by the pm filesystem driver
isn't the same thing as the SECTOR_SIZE/SECTOR_SHIFT macros used in
the assembly code.  This is a per-device property, and in the
particular case of isolinux hybrid, they are not even currently the
same (for all others, they are the same for now, but not necessarily
in the future.)

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

diskstart: print CHS/EDD instead of CBIOS/EBIOS

The terms CHS or EDD are more well-known, so use them.

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

comboot: fix comapi_open

Fix breakage introduced by commit e375515ddc712f1f69ee21337db2a3267caa5d49
(Add 32-bit versions of open file/close file).

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

FAT: need to track the current position past discontinuities

When we have a discontiguous file (broken extent), then we need to
keep track of where the next extent starts, not where the current
extent ends.

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

diskio: compare to the proper pointer

We need to compare against the advanced pointer, not the original one.

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

fat: fix confusion between byte and sector counts

clust_shift is in units of sectors, not in bytes; this was mixed up in
the extent finder.

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

diskio: make maxtransfer per-device, cap to 127, imported from head

Make the maxtransfer per device, as it should be; properly imported
from the head loader (in case it is patched with -s).  Also enforce
capping to 127 for EBIOS and 63 for CBIOS.  This is structured so that
once EDD4 is approved we can remove the capping for that particular
subcase.

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

syslinux.ld: don't pad between .got and .data

Adding padding between .got and .data is unnecessary, and can
cause spurious decompression failures when .data is empty (which it
almost is already...)

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

diskio: sanitize the reduced transfer sizes

Simply shift the size left by 1 when computing the transfer sizes.
This will always end with the values ..., 1, 0 as it should.

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

cmenu: remove obsolete Makefile rules

Remove Makefile rules which were both redundant (../MCONFIG contains
the rules in a central manner) and just plain wrong.

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

MCONFIG: add MCONFIG.devel for use from git, removed in tarballs

Add an MCONFIG.devel file which is present in git, but removed in
tarballs.  This lets us set -Werror there and still not break users
who are using a different compiler.

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

installers: fix warnings

CLean up warnings in the installers.

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

hdt-cli.c: clean up unused warnings

-Werror cleanup.

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

adv_menu.tpl: fix type of login_handler()

Fix the type of the login_handler() function.

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

complex.c: fix warnings

Clean up warnings, even though this is sample code.

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

libmenu/des.c: fix signedness error

Fix a signedness error, which in turn was caused by needlessly
complicated code.

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

libmenu/passwords.h: function declaration isn't a prototype

Make it a prototype

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

libmenu/syslnx.c: mark ipappend used

Is this really correct?!

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

rosh.c: mark argument used

-Werror cleanup.

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

readconfig.c: avoid uninitialized variable warnings

-Werror cleanup.

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

cat.c32: remove unused variable

Remove unused variable as part of -Werror cleanup.

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

vpdtest.c32: remove unused variable

Remove unused variable as part of -Werror cleanup.

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

sdi.c32: remove unused variable

Remove unused variable as part of -Werror cleanup.

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

pcitest.c32: mark argc/argv used

Part of -Werror cleanup.

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

disk.c32: remove unused variable

Remove unused variable as part of -Werror cleanup.

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

sha256/512: fix signedness errors

Fix signedness errors as part of -Werror cleanup.

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

get_key.c: fix signedness error

Fix signedness error, part of -Werror cleanup.

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

runimage.c: remove unused variable

-Werror cleanup: remove unused variable.

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

Merge branch 'master' into pathbased

CMENU: Fixing login_handler prototype

login_handler have to follow the (t_menusystem * ms, t_menuitem * mi)
prototype defined by t_handler_return

com32: Fixing cast in argv to avoid warning

const char * confuse the *argv++ assignation
Casting to char * removes this warning

hdt: casting to avoid useless warning

hdt: Fixing print_history() prototype

print_history have to follow the exec prototype

cpuid: Typo between argv & argc

argv isn't a number ;)

cmd: Adding proper include to syslinux_run_command

disk: Fixing get_error() call

get_error prototype changed since this module was written

Merge branch 'master' into pathbased

Resolved Conflicts:
com32/modules/Makefile

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

cat.c32: move from samples to modules; remove debug code

Move cat.c32 from samples to modules, and remove debugging messages.
Real users may want to use this.

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

Merge branch 'master' into pathbased

cat.c32: check if filename exists

Don't crash if the file doesn't exist.

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

Merge branch 'master' into pathbased

Merge remote branch 'hdt-pierre/fixes-for-3.86'

Merge branch 'master' into pathbased

Merge remote branch 'erwan/master'

VPD: Fixing base_address

This commit now save properly VPD's address.

cpuid.c32: print unsigned values

Print unsigned values; formatting was wrong for unsigned values
anyway.

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

sysdump: use <sys/cpu.h>

Use <sys/cpu.h> where it makes sense.

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

com32: Unbreak <sys/cpu.h>

CPUID functions were apparently broken when made PIC-safe; clean up
and fix.

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

Merge branch 'master' into pathbased

Merge remote branch 'erwan/master'

core: remove "Hello, World!" debugging message

Remove "Hello, World!" debugging message -- no longer relevant.

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

DMI: Increasing processor buffer size

Impact: avoid string truncation

The version field could be bigger than 32 char.

DMI: Fixing memory corruption

Impact: avoid memory corruption

Too much usage of strcpy & sprintf was leading to serious memory
corruption. Using snprintf & strncpy is prefered.

DMI: removing useless variable

Impact: none

Removing useless variable

mboot.c32: autodetect Solaris

Autodetect Solaris kernels (based on the ELF header OSABI field) and
use the Solaris workarounds in that case.

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

Merge remote branch 'origin/master' into pathbased

extlinux: handle cases of a single level of directories

We had a boundary condition error where a single-level directory from
the global root (e.g. /boot) would be incorrectly truncated; fix that.

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

gfxboot: fix menu label handling

Signed-off-by: Steffen Winterfeldt <wfeldt@opensuse.org>
Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>