Merge remote-tracking branch 'mfleming/for-hpa/elflink/cmdline' into elflink

Merge remote-tracking branch 'mfleming/for-hpa/elflink/core' into elflink

Merge remote-tracking branch 'zytor/master' into elflink-merge

Conflicts:
com32/lib/Makefile
com32/lib/sys/open.c
version

diskstart: Move DATE_STR out of sector 1

Save a few bytes in sector 1 by moving DATE_STR out.  This string can
be fairly long especially when using a git tree.

Reported-by: Matt Fleming <matt.fleming@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge remote-tracking branch 'genec/memdiskdbg-for-sha0'

memdisk/dskprobe.c: Add status check to probes 08h and 41h

If this fails, the machine should never be working.

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

Merge remote-tracking branch 'genec/diag-mbr-handoff-for-hpa'

Merge remote-tracking branch 'genec/diag-geodsp-for-hpa'

gpxe: Inhibit parallel make

Don't parallel make the gpxe directory.

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

Update .gitignore

Add *.img, *.img.xz and mk-lba-img to .gitignore.

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

Merge remote-tracking branch 'sherbszt/gfxboot32'

ld: Use ABSOLUTE() when taking differences between relative symbols

binutils 2.21.51.0.6, and possibly other versions, generate the wrong
result when subtracting two section-relative symbols (the final result
ends up having the base of the section incorrectly added to it.)  Work
around this by using ABSOLUTE() on the symbols to force the result to
become absolute.

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

ldlinux, cli: Add support for F-keys

The old asm command-line interface had support for printing files
containing help messages when the F-keys (F1-F10) were pressed. Add
this support to ldlinux.

The config parser already had support for parsing config files with
F-key directives and the cli code already had support for executing
callbacks when one of the F-keys was hit, so this patch simply glues
all the pieces together and provides a function to cat the help file.

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

elflink: Detect circular dependencies during module load

It's possible for the 'modules.dep' file to contain circular
dependencies where module A has a dependency on module B, and module B
has a depedency on module A.

Currently what happens in the face of circular dependencies when
trying to parse and load the modules in 'modules.dep' is that we get
stuck in a loop in module_load_dependencies(), presumably eventually
blowing the stack.

To fix this we maintain a singly-linked list of module names that are
in the process of having their dependencies loaded, but that have not
completed the loading. Everytime we begin loading the dependencies for
a new module we check to see if that module name is already on the
linked list. If it is, we've discovered a circular reference so we
refuse to load the dependencies and print a helpful message to the
user which includes the circular dependency chain.

It would be possible to cope with these circular references if we were
to skip loading the dependencies for any modules that are on the
linked list and defer resolving any module relocs until after all
module images have been loaded into memory. However, it's more than
likely that a circular reference is indicative of a bug and that the
correct fix would be to extract the symbols causing the circular
dependency into their own module. So we treat circular dependencies as
an error case for now.

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

gfxboot: remove save_read() function

Remove the no longer needed save_read() function.

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

gfxboot: remove lowmem_buf_size variable

Remove the no longer needed lowmem_buf_size variable.

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

gfxboot: don't use the bounce buffer

Allocate lowmem buffer instead of using the bounce buffer.

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

gfxboot: use skipspace()

Use skipspace() instead of local skip_spaces().

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

diag/geodsp/README: Additional use notes

Merge remote-tracking branch 'mcb30/empty-tftp-ip'

elflink, cli: TAB key now displays labels

Now, if the user hits the TAB key at a prompt a list of all labels to
complete the current label typed on the command-line is printed. If
nothing is typed at the command-line all labels are printed.

This feature is available in the asm command-line interface so lets
make it available in the C version.

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

pxe: allow TFTP server IP to be empty when not using TFTP

Reported-by: Marcus Grando <Marcus.Grando@corp.terra.com.br>
Signed-off-by: Michael Brown <mcb30@ipxe.org>

diag/mbr/handoff: Add optional printing of floppy descriptor

diag/mbr/handoff: Code comments; reflow output

The printing of CR/LF was misplaced such that it'd print a blank line if
no PnP data was found.

linux.c32: For protocol 2.10+, use init_size when relocating

If we have to relocate the kernel, use the init_size parameter if it
is available, rather than an arbitrary fudge factor.

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

gpxe: Make the build parallel-safe

Make sure we don't have multiple build processes wrecking havoc in the
gpxe directory.

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

Merge remote-tracking branch 'genec/gpxelinux-kpxe-for-hpa'

Makefile: Add gpxe/gpxelinuxk.0 to INSTALL_AUX

core/Makefile: another attempt at dealing with GNU make 3.82

Trying another tack...

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

Makefile: Add diag/geodsp/ .xz images to install target

Merge branch 'master' into diag-mbr-handoff-for-hpa

Conflicts:
diag/mbr/handoff.S

diag/mbr/handoff.S: Remove the attempt to zero pad/prevent overflow

Add diag/ targets into top Makefile for install target

diag/mbr/: use checksize.pl to pad/prevent overflow

HPA noticed that using Fedora 15 Alpha and gcc-4.6 broke on this code

diag/geodsp/Makefile: use ./ to run a file in the current dir

When running a file in the current directory, use ./ as a prefix.

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

Makefile: build the diag directory

Build the diag directory (diagnostic aids).

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

diag/mbr/handoff.S: Fix compilation problems with gas 2.21.51

The construct used for zero-padding in handoff.S doesn't work with gas
2.21.51.

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

Merge remote-tracking branch 'genec/diag-geodsp-for-hpa'

NEWS: Document isohdpfx*.bin available.

mbr: Install isohdpfx*.bin

GNU xorriso can build isohybrid images in one step without the need
for the isohybrid program, but for that to work the isohdpfx*.bin
files need to be available.

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

README: update copyright to 2011

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

doc: update copyright to 2011

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

Merge remote-tracking branch 'genec/news-for-hpa'

Merge remote-tracking branch 'genec/doc-syslinux-for-hpa'

com32/cmenu/adv_menu.tpl: Use the proper value for ipappend

If x is NULL, we're supposed to pass 0 for ipappend; this was
incorrect in the template and the gcc 4.6 unused variable warning
caught it.

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

libinstaller/fat.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

extlinux/main.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

core/fs/pxe/pxe.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

core/fs/iso9660/iso9660.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

core/fs/fat/fat.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/tools/relocs.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/rosh/rosh.h: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/rosh/rosh.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lua/src/vesa.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lua/src/syslinux.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lib/vsscanf.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lib/syslinux/shuffle.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lib/sys/rawcon_read.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/lib/jpeg/tinyjpeg.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/hdt/hdt-menu-summary.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/hdt/hdt-cli-kernel.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

com32/cmenu/libmenu/help.c: remove variables set but not used

gcc 4.6 warns on variables set but not used, so remove them.

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

dos/dosexe.ld: Work around linker issue in binutils 2.21.51

binutils 2.21.51 seems to have problem subtracting two
section-relative symbols, so make them global.

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

memdisk: make sure the disk type is actually set

Make sure we actually set the disk CMOS type.

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

Merge remote-tracking branch 'mfleming/for-hpa/elflink/fix-compiler-warnings' into elflink

elflink: Return zero from draw_background()

Fix the following comiler warning,

background.c:37:1: warning: control reaches end of non-void function

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

elflink: Add draw_message_file() prototype

To fix the following warning,

menumain.c: In function ‘show_message_file’:
menumain.c:196:5: warning: implicit declaration of function ‘draw_message_file’

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

elflink: Fix compiler warnings by including core-elf.h

Numerous files need to include core-elf.h to get the definition of
'enum kernel_type'.

In file included from mytest.c:19:0:
menu.h: At top level:
menu.h:206:40: warning: ‘enum kernel_type’ declared inside parameter list
menu.h:206:40: warning: its scope is only this definition or declaration, which is probably not what you want

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

elflink: Fix compiler warnings in hello.c

Fix the following warnings,

hello.c: In function ‘hello_main’:
hello.c:23:5: warning: too few arguments for format
hello.c:20:9: warning: unused variable ‘i’

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

ldlinux: Fix compiler warning in colors.c

Include core-elf.h to get the definition of enum kernel_type to fix
the following warning,

In file included from colors.c:17:0:
../modules/menu.h:206:40: warning: ‘enum kernel_type’ declared inside parameter list
../modules/menu.h:206:40: warning: its scope is only this definition or declaration, which is probably not what you want

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

ldlinux: Delete unused variables and functions

Fix the following compiler warnings,

readconfig.c: In function ‘parse_one_config’:
readconfig.c:1059:6: warning: unused variable ‘i’
readconfig.c: At top level:
readconfig.c:1110:13: warning: ‘dump_menu’ defined but not used

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

ldlinux: Sprinkle const type qualifers

The following warnings are all caused because we're assigning from
const char * to char *. As the strings are never modified, it makes
sense to add the const type qualifer instead of casting away const.

readconfig.c: In function ‘parse_config_file’:
readconfig.c:708:11: warning: assignment discards qualifiers from pointer target type
readconfig.c:937:10: warning: assignment discards qualifiers from pointer target type
readconfig.c:987:20: warning: assignment discards qualifiers from pointer target type
readconfig.c:998:15: warning: assignment discards qualifiers from pointer target type
readconfig.c:1004:18: warning: assignment discards qualifiers from pointer target type
readconfig.c:1041:11: warning: assignment discards qualifiers from pointer target type

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

ldlinux: Fix compiler warnings in cli.c

Make the compiler happy by casting away the following warnings,

cli.c: In function ‘process_command’:
cli.c:381:2: warning: passing argument 1 of ‘strtok’ discards qualifiers from pointer target type
../../../com32/include/string.h:44:16: note: expected ‘char *’ but argument is of type ‘const char *’
cli.c:400:3: warning: passing argument 2 of ‘spawn_load’ from incompatible pointer type
../../../com32/include/sys/exec.h:43:12: note: expected ‘const char **’ but argument is of type ‘char **’

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

ldlinux: Fix the following warnings

ldlinux.c: In function ‘enter_cmdline’:
ldlinux.c:15:23: warning: unused variable ‘aux’
ldlinux.c: In function ‘ldlinux_main’:
ldlinux.c:75:5: warning: implicit declaration of function ‘kaboom’

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

elflink: handle the NULL return of edit_cmdline

When use ctrl-c to cancle the current input, the edit_cmdline
function will return a NULL string. While, the called didn't
handle that case, and an error would be happened.

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

elflink: Add ctrl-R key bind support

This patch would add the CTRL-R key bind support. It would have
the basic feature of the CTRL-R key bind in bash. While, don't
expect it will do exactly like what bash will do.

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

elflink: use 'input' as the prompt of the CLI

Use the paramter 'input' of fucntion edit_cmdline as the prompt of the
CLI. I guess this is what the 'input' parameter used for. It's is more
extendable than using a specify prompt.

And, for now, let's use the 'syslinux' as the prompt.:)

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

elflink: Add Ctrl-p + Ctrl-n key binds

Add ctrl-p and ctrl-n key binds to get the prev and next
command. They are much easier to reach than KEY_UP and
KEY_DOWN.

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

elflink: Do clear screen even if we have no pDraw_Menu method

Do clear screen even if we have no pDraw_Menu method, since user
may work in CLI mode and want to clear screen by pressing 'Ctrl-L'.

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

elflink: Fix the wrong malloc size in enter_cmdline

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

elflink: Cleanup some warnings

Cleanup some unused variables, goto lables, and add the missing
header files.

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

chain.c32: Allow both "guid" and "uuid"

The GPT specification calls it "guid", but the Unix world generally
calls it "uuid".

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

chain.c32: fix specification of disk by GUID (UUID)

Fix specification of disk by GUID, broken due to an incorrect pointer.

Reported-by: Szymon H <bodwick@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

diag: Add geodsp to Makefile

diag/geodsp: Display Geometry based on read

This produces two images that are 8MiB+ in size (in order to exceed the
size of the largest CHS cylinder) containing 1 or multiple sectors of
code/data and the remaining sectors filled with their LBA.

diag/mbr: Remove unneeded dependency from Makefile

diag/mbr/handoff: Make filename in comment consistent

diag/mbr/README: consistent docs

diag/: Add Makefile,README here and mbr/

Handoff *BR: Move to diag

mbr_ho.bin: Fix stack bug; also save/restore cx in another location

mbr_ho.bin: Fix several non-8086 issues

mbr_ho.bin: Update install instructions

mbr_ho.bin: install instructions

mbr_ho.bin: Fix bug in $PnP Scan; PressAnyKey; compare [ES:DI]

Also numerous uses of *w opcodes and spacing fixes

mbr_ho.bin: Handoff MBR

This mbr code entity shows DL, DS, SI, 16 bytes of [DS:SI], ES, DI,
4 bytes of [ES:DI], and scans F0000h-FFFFFh on 16 byte alignment for $PnP.

This is useful for testing when MBR or VBR/PBR handoff goes awry.

NEWS: wrapping

doc/pxelinux.txt: Info on gPXE-enhanced binaries

doc/memdisk.txt: Add mem= parameter

NEWS: update

ldlinux: Parse and store the "onerror" command line

If we fail to load a kernel and the parser found an "onerror"
directive, execute the onerror command line.

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