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>

dir.c32: sort, columize and page the output

Make the output of dir.c32 actually usable for an average user.
Present the directory contents sorted in columns, and pause for every
screenful.

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

core/fs: clean up, fix and improve the internal readdir() interfaces

- Avoid a completely unnecessary malloc/free pair by passing a pointer
  to the filesystem driver;
- Make sure d_reclen is always set correctly;
- Make sure the d_type field is set correctly.

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

dirent.h: types for the d_type field

Add an enum for the DT_* constants for the d_type field.

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

stdcon: make the screen size available

Make the screen size available when using stdcon, not just ansicon.

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

qsort: handle the case of zero entries

Avoid bugs when the qsorted array is empty.

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

Add 32-bit versions of open file/close file

Add 32-bit API calls for open file and close file.

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

fat: fixing filenames with multiple-of-13 names broke all others

The fix for filenames with names that were multiples of 13 broke all
*other* filenames.  Fix both, this time...

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

Merge remote branch 'origin/master' into pathbased

Resolved Conflicts:
version

Merge remote branch 'origin/master' into pathbased

NEWS, version: update for 3.86

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

chain.c32: Fix chaining to MBRs

The change to load the bootsector even when loading a bootstrap from a
file broke the handing of chainloading the MBR of a disk.  Fix this.

Debugging-info-by: Thomas Mieslinger <thomas.mieslinger@lundl.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

VFAT: handle filenames with are exact multiples of 13

Filenames in VFAT that are exact multiples of 13 are not
null-terminated; handle that particular subcase.

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

lmalloc: add <string.h> for lzalloc

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

readdir: replace opendir/readdir/closedir API with a 32-bit API

The 16-bit API to opendir/readdir/closedir was confused, had a memory
leak, and was incompatible with Syslinux 3.x anyway.  Replace it with
a pure 32-bit API.

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

extlinux: centralize file flags setting; add FAT support

Centralize the poking at file flags.
Add FAT support to file flags setting (S bit).

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

extlinux: don't report failure after writing a FAT superblock

Missing parens made us return failure after successfully writing a FAT
superblock.

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

fat: fix longname matching

Fix longname matching, which was broken due to missing counter
decrement.

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

btrfs: remove global fs ptr and clean up code

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

btrfs: get rid of unnecessary _fs variable

Get rid of the unnecessary _fs variable in btrfs_fs_init().  The
compiler would have optimized it out anyway, but it's cleaner if it's
in the source.

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

btrfs: Implement next_extent and call generic_getfssec

Now multiple extent allocation is supported.
Also fixed the symlink file handling and fs_init typo.

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

pxe: don't allow URL parsing to produce zero bytes; retune

Don't allow URL escapes to produce zero bytes (%00).  Also, assume
TFTP URLs are going to be less common than "plain" filenames, so defer
the type testing until we know it's a URL.

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

Add simplified toupper, tolower, str[n]casecmp to core

(Missing from previous checkin)

Add simplified toupper(), tolower() and str[n]casecmp()
implementations to the core, good enough for ASCII e.g. for use in URL
parsing, or in config file decoding.

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

pxe: handle tftp:// URLs natively

Handle tftp:// URLs natively, instead of relying on gPXE.  This also
allows for specifying a nonstandard TFTP port.

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

pxe: add a single-tick timeout for PXE stacks which eat transmits

Add a single-tick first-round timeout for PXE stacks which eat the
first transmit under certain circumstances, i.e. gPXE if the ARP table
isn't populated yet.

TODO: encapsulate all send packet/wait for reply code into a common
function.

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

gfxboot: remove 16-bit gfxboot module

Get rid of the 16-bit gfxboot module; use gfxboot.c32 instead.

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

fs: add no_next_extent() dummy function

Add a no_next_extent() dummy function for filesystems (like iso9660)
which should never end up calling next_extent (because there is only
one extent...)

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

core: fs: get rid of the prev_extent member

The prev_extent member was only ever used transiently; instead pass
the new lstart in as an argument and leave the previous value of
next_extent unmolested and thus can be used in-place.

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

core: get rid of unmangle_name completely

The notion of "mangle name" is now restricted to producing a canonical
representation, e.g. changing \ -> / for FAT filesystems.  However,
the resulting mangled name is now always human-readable, so there
never is any reason to unmangle a name.

Get rid of not just the mangling, but of the resulting unmangled-name
buffers and so on.

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

core: remove unmangle_name

Remove the now-obsolete unmangle_name method.

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

core: add missing dependencies

We need to account for dependences as far down as we care about.

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

pxe: change debugging message to dprintf()

Marking debug messages is what dprintf() is for...

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

memdisk: drop DEBUG from unzip.c

DEBUG doesn't work in unzip.c/inflate.c, so always #undef it.

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

pxe: drop UDP close from unload sequence

UDP close is part of the reset sequence, not the unload sequence.  Now
we always execute the reset sequence, so drop it from the unload
sequence.

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