com32: Put deleted object files back into the Makefile

For some reason that I can't understand, some object files were
removed from this Makefile in the following commit,

    |commit 0cb6490fa96f752c01bbb9add2c9ca6dbf99ece2
    |Author: Feng Tang <feng.tang@intel.com>
    |Date:   Thu Jun 3 10:48:22 2010 +0800
    |
    |   elflink: start merge with pathbased branch
    |
    |   modify these files to make compile pass
    |
    |       modified:   com32/MCONFIG
    |       modified:   com32/Makefile
    |       modified:   com32/include/klibc/compiler.h
    |       modified:   com32/include/sys/elfcommon.h
    |       modified:   com32/lib/Makefile
    |       modified:   com32/lib/free.c
    |       modified:   com32/lib/malloc.c

However, with this patch things continue to compile correctly so I'm
unsure why they were ever removed. Furthermore, pieces of code in
com32/ refer to symbols defined in these these object files and so
they are required in order to load modules.

These missing object files were discovered when loading libcom32.c32,
which wouldn't load because it was complaining about unresolved
symbols.

The removal of core/strncasecmp.c needs some explanation. There are
currently two identical copies of strncasecmp.c, one in com32/lib and
one in core/, and while the core copy is part of core/libcom32.a, the
copy in com32/lib isn't compiled. Now, because there are no references
to strncasecmp within core/ the symbol isn't pulled in from libcom32.a
and exported as a global symbol by the core. So, if ELF modules have
references to strncasecmp they cannot be resolved at runtime.

To fix this I've included strncasecmp.o in libcom32min.a which
isolinux.elf, ldlinux.elf and pxelinux.elf link against with
--whole-archive, which means that even if there are no references to
strncasecmp within the core, that symbol is still exported.

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

elflink: Move more code from core/ into ldlinux

ldlinux now contains all the code necessary to load and execute
modules, none is contained in the core.

This change also allows us to change the spawn_load() prototype and to
push the job of processing arguments to executable functions (e.g. the
contents of char *argv[] as passed to a module's main function) into
ldlinux/execute.c instead of doing it in spawn_load(). Moving it into
ldlinux/ makes sense because the only core user of spawn_load() is
load_env32() and we don't require any sort of argument processing in
that path.

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

elflink: Remove MODULE_MAIN() from executable ELF modules

To make it easier to move existing COM32 modules over to ELF modules
without having to modify them we need to search for and run their
main() functions when they're loaded instead of requiring their
initial function to be labeled with MODULE_MAIN().

Currently, we require all executable ELF modules to specify their
initial function with the use of the MODULE_MAIN() macro and not via
the traditional method of naming it main(). However, there are weird
restrictions on what functions can be passed to MODULE_MAIN(), for
instance, they must be declared static.

This patch makes life much simpler and allows executable ELF modules
to be loaded without the MODULE_MAIN() wrapper, but rather by naming
their initial function main(). All the modules in com32/modules can
now be run as ELF modules without any modifications.

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

com32: console is already initialised when we enter main()

There's no need to initialise the console when we enter main() as
ldlinux.c32 will already have initialised it for us. Leaving the calls
to openconsole() causes the console settings to be reset whenever we
run a module.

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

exit: Delete exit.S and references to __exit_handler

We stopped using exit.S (and thus __exit_handler) in 71d5d1ba646e
("elflink: change the exit.c, remove the exit.S to make process exit
working") when we moved to using the C version of _exit(). If we don't
remove the symbol __exit_handler we'll encounter undefined symbol
errors when moving libcom32.a over to an ELF shared library.

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

mk: LDFLAGS use elf_i386 emulation for ld

elf32_i386 isn't a valid ld emulation mode. Presumably no one is using
$(LDFLAGS) set in this file.

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

mk: Add rule for emitting .c32 ELF files

We currently open code the Makefile rules for outputting .c32 files,
so provide a pattern rule to make things easier.

There are no users of this rule yet, but we'll make use of it when we
switch all modules over to ELF.

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

elflink: Move elf MCONFIG to mk/elf.mk

Catch up with recent changes to the Makefile fragments in the MCONFIG
files.

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

Merge branch 'for-hpa/elflink/core' into elflink

Merge branch 'for-hpa/elflink/ldlinux' into elflink

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

We need the recent Makefile filename changes to be merged into the
elflink branch because it will make things simpler when converting all
modules to ELF format.

Conflicts:
com32/Makefile
com32/modules/Makefile
version

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.

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>

elflink: Jump to kaboom if we fail to load ldlinux.c32

Previously if we failed to load ldlinux.c32, we would drop out of
load_env32() and try to execute the assembly kernel loader.

With this patch, if we drop out of load_env32() we jump to kaboom
which drops the user at a command-line, at least that way they have a
chance of fixing the reason ldlinux.c32 failed to load.

Also, delete the comment that talks about loading and parsing the
configuration file - that is now done by ldlinux.c32.

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

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>

hdt: Adding dump entry in the menu

If the PXE is enabled, let's show a dump menu at the main menu.

hdt: Adding checksums file

When generating a release, let's create a list of the checksums

hdt: Adding dump_mode & tftp_ip boot option

dump_mode=<dir> give the user a chance to select a directory on the tftp
server.

tftp_ip=<w.x.y.z> give the user a chance to select another tftp server
for dumping data. By default, we use the tftp that serves the pxe
booting.

hdt: new beta 0.5.0-beta1

This first beta is featuring the dumping code.

hdt: Dumping HDT

HDT information is now dumped in the 'hdt' file

hdt: Adding Kernel dumping

Kernel is now dumped into th 'kernel' file

Merge remote branch 'hpa/master' into libupload

mboot.c32: Delete superfluous $(LIBS) prerequisite

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

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

Makefile: Move Makefile fragments into mk/

Move the MCONFIG files into a mk/ directory and give them more
descriptive names.

This is purely a cosmetic change to make the 'include' directives a
bit more coherent by making it obvious exactly which MCONFIG file
we're including. For example, in com32/lua/src/Makefile we exchange
the line,

include ../../MCONFIG

for the much more comprehensible,

include $(MAKEDIR)/com32.mk

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

diskstart: use the correct register for the starting address

The starting address is obtained in si, not edi.

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

installer: Update and correct generate_extents()

1. Make sure extents are strictly less than 64K in size
2. Make sure extents do not cross 64K boundaries

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

core: Move sector 1 to address 0x8000 so it is aligned

Move sector 1 to 0x8000, so that it will be sector-aligned no matter
what the sector size is (well, as long as it is <= 32K).  This avoids
potential conflicts between the 64K "seams" and the sector size.

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

core/head.inc: require NASM 2.03+

We have needed NASM 2.03+ for a long time.

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

Merge commit 'syslinux-4.04-pre22' into dynamic-sector

diskstart: Add a pointer to the syslinux_banner to the EPA

For the benefit of third-party tools, add a pointer to the
syslinux_banner in the extended patch area.

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

disk: Make the sector size dynamic

Make the sector size dynamic in anticipation of disks with
non-512-byte logical sectors.

With this change, bsBytesPerSec *must* be correct; this still needs to
be added to the installers to enforce.  Furthermore, the number of
sectors in one extent can still only be < 64K.

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

com32, bitops: proper constraint for byte-sized output

The proper output for the byte-sized output of "setc" is "=qm".

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

NEWS: iso9660 also had issues with ../ paths.

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

core: Adjust formatting around the date string

Adjust the formatting around the date string to avoid missing and
duplicated spaces.

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

tinyjpeg: change DEBUG to JPEG_DEBUG

Rename the debugging option for tinyjpeg to JPEG_DEBUG to avoid
conflict with the global dprintf option.

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

iso9660: use generic_mangle_name

There is no longer any need to do special name-mangling for iso9660,
and doing so, in fact, breaks the handling of paths starting with ../.

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