ldlinux: Pass entire cmdline to execute()

We need to pass the entire cmdline to execute(), not just the kernel
name. execute() does all the required processing of cmdline arguments
and passes them to the loader in the correct format, e.g. for COM32
modules the cmdline arguments need to be passed in the traditional
char *argv[] way.

So stop using strtok() which modifies its first argument and means we
therefore lose the cmdline after the kernel name.

Previously, the "argc" and "argv" arguments that are passed to a
module's main() function contained bogus values.

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

ldlinux: Move cmdline processing out of cli.c

While it makes perfect sense for cli.c to read input from the user and
parse certain characters (Ctrl-R, TABs, etc), once that processing is
done it should be up to the caller to decide what to do with the
expanded cmdline string. For example, the logic to figure out what
type of kernel we're loading (calculated from a kernel's filename
extension as typed on the command-line) is currently in cli.c, and
that is just wrong. That kind of logic belongs in ldlinux.c

So move process_command() from cli.c to ldlinux.c and extend it so
that if the user types a label name at the prompt, we load the kernel
for that label as described in the config file.

Now that we've moved all cli history code into edit_cmdline() it seems
like a good time to turn it on permanently.

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

ldlinux, cli: Do not assign 'comm_counter' unnecessarily

There's no need to assign a value to 'comm_counter' at the start of
the edit_cmdline() because it is always assigned before being used.

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

Merge commit 'syslinux-4.04-pre18' into elflink

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

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

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

NEWS: update the NEWS file

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

pxe: disable the "force INT 18h" hack

It seems to break as many machines as it fixes, so disable it for now.

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

com32, bitops: an "m" constraint takes an object, not a pointer to one

The argument to an "m" constraint is an object, not a pointer to an
object.  For a void pointer it needs to be cast to an indirectable
type (like char) and then indirected.

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

com32: fix range handling in vsscanf

Fix multiple errors in building the bitmap for ranges in %[.

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

bitops: the bt instructions returns result in CF, not ZF

The bit test instructions returns the tested bit in CF, not the
inverse of the tested bit in ZF.  I don't know how I got that idea...

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

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: Perform auto-boot if NOESCAPE set in config file

The previous logic for detecting when to perform an auto-boot was
incorrect, we should try to auto-boot if NOESCAPE is set in the config
file.

Signed-off-by: Matt Fleming <matt.fleming@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