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>

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>

gfxboot: better error handling

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

gfxboot: fix localboot

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

gfxboot: turn off graphics before starting kernel

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

gfxboot: handle IPAPPEND

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

gfxboot: make config statements case-insensitive

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

gfxboot: support MENU LABEL statement

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

chain.c32 sethidden option fix

Hidden sectors value was applied at the wrong offset.

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

com32/tools: nothing to install here

This directory contains builttime tools, so they don't need to be
installed anywhere.

This fixes "make install-all".

Reported-by: Dag Wieers <dag@wieers.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core: fs: verify call to fs_ops->readdir

Check if fs_ops->readdir is available before calling it.
At least PXELINUX doesn't implement it.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>
LKML-Reference: <1267974170$3058@local>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge commit 'syslinux-3.86' into pathbased

Resolved Conflicts:
core/cpuinit.inc

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

core: make sure we observe EFLAGS_ID only

When looking for CPUID, make sure we only look at EFLAGS_ID and no
other flags that could possibly have changed for extrinsic reasons.

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

cpuid.c32: output the correct register names

Output the correct register names in cpuid.c32; fix copy-and-paste
error.

Reported-by: Andrew Jones <drjones@redhat.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: document mBFT fix

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

memdisk: Calculate mBFT checksum later

We were calculating the mBFT checksum too early; dpt_ptr could
change after checksumming, so we now do it after that possibility.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

core: remove stray debugging statement

Remove a stray debugging printout when running on Xen.

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

core: move __jiffies and _IdleTimer to .data16

Move the __jiffies and _IdleTimer variables to the .data16 segment.
This ensures that if do_idle is invoked very early that it will not
jump off into a non-existent idle routine.

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

Merge branch 'master' into pathbased

Resolved Conflicts:
com32/modules/Makefile
core/comboot.inc
core/cpuinit.inc
core/idle.inc

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

core: move idle handling into protected mode

Do the actual idling in protected mode.  This both allows PM code a
more efficient interface, but also handles bugs in HVM implementations
which don't handle HLT in real mode.

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