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>

NEWS: document MEMDISK offset fix

memdisk: MBR not recognized correctly

This fixes a small inaccuracy in memdisk:

get_disk_image_geometry() does not use MBR data to detect disk
geometry, since it checks MBR signature at wrong offset.

This normally does not cause problems, because memdisk guesses a
reasonable value, if no MBR is found.

Signed-off-by: Bodo Stroesser <bstroesser at ts.fujitsu.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

NEWS: update

core: remove do_idle from pollchar

Remove do_idle from pollchar; it is highly inappropriate while, for
example, checking for a keypress during disk activity.  Furthermore,
make sure we reset the idle counter on events that indicate actual
disk activity.

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

cpuid.c32: very simple low-level CPUID info CLI routine

Add a very simple com32 program to dump a single CPUID leaf.

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

core: Workaround for Xen HVM HLT-in-real-mode bug

Xen HVM when used with the "vmxassist" program crashes if HLT is
executed in real mode.  Detect a version of Xen old enough to support
"vmxassist", and if it is found, then disable HLT.

See: https://bugzilla.redhat.com/show_bug.cgi?id=570496

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

Clean up the version/copyright strings reported via the comboot API

Skip leading garbage in the version and copyright strings reported via
the comboot API.  Clean up fixes for those, and document the situation.

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

Merge branch 'master' into pathbased

Resolved Conflicts:
MCONFIG.embedded
com32/MCONFIG
com32/include/com32.h
core/com32.inc
core/pxelinux.asm
doc/comboot.txt

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

com32: export the filename of a com32 module to the module itself

Export the filename of the com32 module to the module itself, setting
argv[0].

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

MCONFIG: update gcc options for embedded and com32

Update the gcc options used for embedded and for com32 and make sure
they match.

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

mboot: set boot loader name

Set the boot loader name based on syslinux_version()->version_string.
Based on a prior patch by Sebastian Herbszt <herbszt@gmx.de>.

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

gpllib: fix call to CPUID function 4

Only call CPUID function 4 if cpuid_level indicates its availability.

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

core: use MY_NAME for syslinux_banner

Use MY_NAME for syslinux_banner in isolinux and pxelinux.
This is already done in ldlinux and extlinux.

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

dir.c32: show long directories in pages

When columnizing large directories, columnize only inside of a page,
i.e. show:

a d g
b e h
c f i

j m p
k n q
l o r

This seems to have the best readability.

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

fat: fix the type of get_inode_mode()

get_inode_mode() returns enum dirent_type.

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

iso9660: set the type of get_inode_type()

get_inode_type() returns enum dirent_type.

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

fs: get rid of enum inode_mode

Replace enum inode_mode with the equivalent enum dirent_type.

btrfs: Implement readdir operation

Tested with dir.c32. Also use the new IFTODT macro to convert fs inode
mode to internal dirent mode.

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

sys/dirent.h: conversion macros between d_type and st_mode

Conversion macros to change between d_type and st_mode values.

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

loadconfig: replace printf() with dprintf()

It's a debugging statement, just dprintf() it.

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

Changes missing from previous checkin

Add files missing from previous checkin...

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

pxe: move port allocation to a separate file; use dynamic port for DNS

Move all local port number allocation to a separate file, and use that
API to allocate a dynamic port number for DNS queries.

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

pxe: make have_uuid a bool

Make have_uuid a bool, and clarify the logic for setting it.

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

disk.c32: add disk geometry check

Make sure the CHS geometry is valid before displaying disk
information to avoid showing garbage.

Debugging-info-by: Gert Hulselmans <gerth@zytor.com>
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

core: don't bother zeroing the disk cache and heap

No point in zeroing out the disk cache and the heap.  Put those in a
.hugebss section which we align, but don't zero.

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

core: page-align .earlybss

Leave the entire first page unmolested.

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

core: use assembly for the .stack section

Use assembly to define the .stack section, so that it get proper
attributes.

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

pxe: use an inode structure for pxe

Use an inode structure for PXE as well.  Even though this is really
rather superfluous in the case of PXE, it eliminates an ugly special
case in the generic code.

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

mem: increase the heap size to 128K

Use 128K for the heap, to allow TFTP buffers to be allocated there.

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

dir.c32: allow "dir" without an argument to mean "dir ."

People logically expect "dir" to display the current directory.

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

ext2: return the correct file types from readdir()

Return the global file types, not the ext2-specific ones.

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