Merge remote-tracking branch 'hpa/master'

extlinux: remove wrong use of strstr()

strstr() is only used for zero-terminated strings, so OEM Name is not
guaranteed to be zero-terminated. Instead, use fat_check_sb_fields() to
sanity check FAT superblock.

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

implement fat_check_sb_fields()

Sanity check for FAT volumes

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

isohybrid: unbreak hex/octal arguments in C version

Allow hex/octal arguments in the C version of isohybrid, since they
were accepted by the Perl version.

Merge remote-tracking branch 'genec/extlinux-geo-for-hpa'

com32: make syslinux_dump_*() pure debugging functions

Make the syslinux_dump_*() functions pure debugging functions; that is
the way that they are used anyway, and this way they log to the same
place as dprintf.

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

Use <dprintf.h> instead of an ad hoc definition

Instead of using ad hoc definitions of dprintf, dprintf2, vdprintf and
vdprintf2, use <dprintf.h> everywhere possible.

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

dprintf: always define dprintf2/vdprintf2

Always define dprintf2/vdprintf2 even with no debugging at all
enabled.

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

libupload: use %zx to print a size_t argument

To print an argument of type size_t we should use the %z size
modifier.

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

lib: sync <stdint.h> and <stddef.h> from klibc

The stdint.h definition appears to be a total mess.  Clean it up with
the current version from klibc, and revert change of size_t being long
rather than int (it is not on i386).

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

lib/shuffle: use <dprintf.h>

Another straggler not yet using <dprintf.h> for debugging.

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

dprintf: add the ability to log to stdio

Add the ability to redirect dprintf to stdio when there is no other
choice.

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

chain.c32: fix types in dprintf()

Correct invalid types in dprintf().

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

lib: Add missing bits to stddef.h, stdint.h; pointers are "long"

Add missing bits to stddef.h and stdint.h and correct the basic
pointer type to "long".  We really should use the gcc macros for
this...

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

hdt: New release 0.5.1

New release 0.5.1 to manage dynamic filenames based on "%" variables.

hdt: Ignoring checksums file

Checksums file are generated by release target. No need to track it.

strreplace: Fixing includes

Stdlib was missing for malloc definition

Merge remote-tracking branch 'hpa/master'

core: extend conventional memory warning with amount seen

Extend the conventional memory warning with the amount actually seen.

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

extlinux/main.c: Fix geometry handling

1) ioctl HDIO_GETGEO expects a pointer to a struct hd_geometry
2) struct stat's st_dev is the parent file used; st_rdev is what we want

Reported-by: Michael Tokarev <mjt@tls.msk.ru>
Tested-By: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Add "make strip" target

Add a "make strip" target, to make doing the official build easier.
We want the official build to have stripped binaries for size reasons,
so do it right and make it an actual build target.

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

Remove -s for host binaries

Remove -s for host binaries; current practice is to let the distro
packaging systems do that themselves.  For the official binaries, we
should probably strip them, but via an external tool.

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

linux: remove debugging OPTFLAGS

Remove debugging OPTFLAGS which really should have made it in.

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

syslinux.mk: $(MAKEDIR), not $(makefiledir)

Fix the path for the optional makefile segments.

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

chain.c32: allow specifying arbitrary seg/offs/entry

Augment the seg= option to also allow for the offset and entry point
to be specified.

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

chain: Support loading ReactOS' FreeLdr

Use something like:

  LABEL freeldr
    COM32 chain.c32
    APPEND freeldr=freeldr.sys

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

chain: Run Nindent on chain.c

Minor formatting fix-ups

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

diag/geodsp/Makefile: Use make variables

Use the appropriate variables but for now the flags/options are
overridden in this Makefile.  Also add XZ to mk/syslinux.mk

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

hdt: Adding dump_filename= option

This option allow the user to redefine the way filename is made-of.
By default that's "mac_address+dmi_vendor_name+dmi_product_name"

This can be overidded by the following syntax:

dump_filename=your_filename

Note that it's mandatory not to add "" or '' around your_filename.

your_filename could be a regular filename but can also includes %{}
directives as defined by :
%{m} = mac address
%{v} = vendor name of the machine
%{p} = product name of the machine
%{ba} = Asset tag of the base board
%{bs} = Serial number of the base board
%{ca} = Asset tag of the chassis
%{cs} = Serial number of the chassis
%{sk} = SKU number of the system
%{ss} = Serial number of the system

Here come a possible command line :
APPEND nomenu auto='dump;' dump_path=hdt2 tftp_ip=192.168.1.254 dump_filename=%{m}+%{bs}+%{ba}+%{cs}+%{ca}+%{sk}+%{ss}+%{p}+%{v}

string: Adding strreplace

diag/geodsp: Save the .bin files

They are a BTARGET file

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

diag/geodsp: Makefile cleanup

Unneeded statements; delete *.lst like other Makefiles

diag/geodsp: Fix .img.gz prerequisite

hdt: Adding hybrid iso image during release

When we release hdt, generating an usb hybrid image could be useful.

hdt: dumping boot flag status

If the boot flag is set on a partition, let's dump this information.

Revert "hdt: remove set-but-unused variable"

This reverts commit 842d7e35ccd8cd89105faf0266497e838d96c8b3.

In fact, you found a real bug. I missed at dumping some information.
So I revert this commit and then add the missing code.

hdt: add back -I$(com32)

hdt picks up include files from outside the include directory and so
needs an explicit -I$(com32).  This probably should be fixed.

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

sysdump: add $(LIBS) to the link rule

$(C_LIBS) doesn't include $(LIBS) so we need to add it explicitly.

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

Merge remote-tracking branch 'mfleming/for-hpa/makefile-cleanup'

Resolved Conflicts:
com32/hdt/Makefile
com32/sysdump/Makefile

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

isolinux: remove broken and obsolete open_file_t

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

isolinux: remove .img file support

Remove disk image support from ISOLINUX.  Very few BIOSes support this
call correctly, and it is likely to have been broken for a very long
time anyway.

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

com32: inet_ntoa() takes struct in_addr

The standard definition for inet_ntoa() is to take struct in_addr, and
not doing that causes a conflict on the lwip branch.

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

hdt: remove set-but-unused variable

Remove a set but unused variable in HDT, which prevents building with
gcc 4.6.  This is probably a bug; presumably the active flag should be
presented to the user, too.

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

Merge remote-tracking branch 'sherbszt/gfxboot32'

gfxboot: pass current working dir to gfxboot core

Pass current working dir to (and back from) gfxboot core via gfx_config
struct. This way I can implement a chdir() in the gfxboot core without
the need for additional 16-bit comboot calls. gfxboot may change the
directory, so you have to read and apply the current setting in boot() (to
read kernel etc. from right location).

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

Merge remote-tracking branch 'hdt/master'

diag/geodsp: zero-fill sectors in -1 mode

In -1 mode, zero-fill the sector instead of writing whatever garbage
was in the buffer already.

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

diag/geodsp: fix ordering of args to fread/fwrite in mk-lba-img

If we want the return value to be in bytes, make sure we pass the byte
count in the nmemb field instead of the size field.

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

diag/geodsp: Avoid large intermediate files

Allow the generation program mk-lba-img to merge the prefix file and
output to stdout, so it can be fed directly to the compressor; that
way we avoid generating huge intermediate files on disk.

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

ppmtolss16: remove obsolete Perl construct

defined() on an array is considered obsolete; instead check for a
nonzero number of elements with scalar().

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

gfxboot: handle INITRD config line

Handle case where there is no initrd= option but a separate initrd config line.

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

version: now working on version 4.05

4.04 is out, next version will be 4.05 (or later).

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

hdt: Final release 0.5.0

Thanks for all who participe in this release.

hdt: Using dashed instead of . into label name

When generating floppy images some version of mtools (< 4.0.14) are
buggy and generate incorrect labels. So using > 4.0.13 is recommended.

Anyway, let's use _ into the volume name, that cleaner :

We now have
  disk label="HDT_0_5_0_B"
instead of
  disk label="HDT_0~5~0-B"

hdt: Fixing ACPI display

Some screenshots shows that we have some troubles for aligning items.
This is now fixed.

hdt: New release beta5

libupload: Adding tftp.h missing file

Oups, this while wasn't commited :/

hdt: Showing specific message for auto mode

That's a nice info to know that some output were generated by the
automatic mode.

hdt: Removing useless printf in dump mode

This message was clearly boring to read ;o)

libupload: Fixing tftp return value

The default return value for the tftp mode.
This made the tftp to return error even if the upload was succesful.

hdt: New release Beta4

hdt: Adding dump mode on pre-built images

As we now have a dump mode, let's add an entry in our menu for that.

hdt: Preventing dump mode if no PXE config

Dump mode can only work if PXE is present.

Merge commit 'syslinux-4.04' into dynamic-sector

spec: Use BuildRequires, add diag/ subdirectory

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

diag: Don't remove BTARGET on make clean

"make clean" should not remove BTARGET

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

Merge branch 'master' of git://git./boot/syslinux/syslinux

Merge remote-tracking branch 'pcacjr/syslinux-update-for-hpa'

hdt: Reporting TFTP errors in dump mode

If we get some errors from the tftp, let's report it to the user.
This will make user's life easier to understand why the dumping mode
isn't working.

libupload: Implementing error code on tftp

When using the tftp backend, let's return the errors we got on the
server.

menu: allow "menu hiddenkey" to take multiple keys; run unlabel

- Allow "menu hiddenkey" to take a comma-separated list of keys.
- Run unlabel() on the commands passed to menu hiddenkey.
  XXX: Consider moving unlabelling to post-menu instead.

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

hdt: Fixing automatic mode parsing

Parsing of the auto='' mode was weak an non functionnal in many
configuration like

auto='dump'

This is now fixed.

NEWS: Document MENU HIDDENKEY

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

libupload: Fixing memset call

We have to use the size of the element, not the size of the pointer.

hdt: New release 0.5.0-beta3

cpuid: Using generic model if cyrix id is unknown

When using the specific Cyrix call to determine the model, if the model
is unknown, let's consider the generic call was right.

This insure a better display of the model id.

lua: Delete superfluous $(LIBS) prerequisite

com32.mk already includes the necessary library in $(C_LIBS), we don't
need to list them again in $(LIBS).

Signed-off-by: Matt Fleming <matt.fleming@linux.intel.com>

sysdump: Delete superfluous $(LIBS) prerequisite

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

Signed-off-by: Matt Fleming <matt.fleming@linux.intel.com>

hdt: Delete superfluous $(LIBS) prerequisites

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

Signed-off-by: Matt Fleming <matt.fleming@linux.intel.com>

menu.c32: Delete superfluous $(LIBS) prerequistite

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

Signed-off-by: Matt Fleming <matt.fleming@linux.intel.com>

cpuid: Improving Cyrix/NSC detection

This code add the specific detection code for Cyrix/NSC processor.
Code came from the Linux kernel.

hdt: Fixing buffer corruption in PXE menu

Jim Cromie reported that PXE menu behave like :

gPXE Detected: ]No

This commit avoid this buffer corruption.

syslinux: check --update option properly

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

extlinux: remove already_installed

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

extlinux: use syslinux_already_installed instead of already_installed

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

libinstaller: implement syslinux_already_installed

syslinux_already_installed function will be used in both extlinux and
syslinux for checking if the boot sector has either the string
"SYSLINUX" or "EXTLINUX" in the OEMID field.

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

cpuid: Managing UNKNOWN CPU better

It's safe to manage the unknown cpu as a regular one. This will avoid
weird behavior when CPU isn't well detected.

hdt: Changing boot flag semantic in menu mode

What we report is the presence of the boot flag and not the ability of a
partition to be bootable. A partition can have a valid bootloader with a
boot flag not set. So the 'Bootable' semantic was wrong.

cpuid: add cpuid info for "Geode by NSC"

Without this, cpu_detect(), and set_generic_info() would take 416
seconds on a soekris 4801 and hdt would report ?????????? as the
processor vendor.

hdt: fix precedence problem in double-space check

hdt: Display non-sequentials disks configurations

If the location of the disks wasn't sequential on the system, the menu
mode was missing some.

This commit fix ticket #46.

Thanks gerth for reporting.

hdt: Adding chain.c32 to pre-built images

This got a requirement from some users.

hdt: Fixing checksums deletion

We have to delete all the checksums files while cleaning.
Older release might exists.

hdt: New release 0.5.0-beta2

keytest: make somewhat more comprehensible

Someone may very well need to use keytest to figure out their
keymappings.  Make it a bit more user friendly.

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

keyname: better handling of named control characters

Named control character (Tab, Enter, Backspace etc.) should use those
names rather than caret sequences.

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

keytest: show the human name together with the keycode

Since we now have library support for showing the keycode name, make
keytest print that as well.

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

menu: clear the timeout line on MENU HIDDENKEY

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

menu hiddenkey: fix parsing the config file option

Fix parsing bugs in the config file.

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

menu: Add "menu hiddenkey" to make terminating menu hidden a command

Use "menu hiddenkey" together with "menu hidden" for a one-keystroke
action.

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