memdisk: use char array for external symbols

Use "extern const char foo[];" for an external symbol rather than
"extern void" (gcc 4.5 fix.)

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

gfxboot: use a character array for an absolute symbol

The best way to access an absolute symbol is "extern const char
foo[];" -- older gccs let you get away with "extern void" but gcc 4.5
chokes on it.

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

disklib: remove improper "inline"

Remove improper inlines from static functions which cause gcc 4.5 to
abort due to inlining failure.

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

com32: add a centralized bitops header

Add a centralized bitops header <sys/bitops.h> which uses x86 bitops
instructions.  This is necessary to keep gcc 4.5 from aborting
compilation due to the inlined code being larger than the non-inlined
version, and well, we should really use the bitops.

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

Merge branch 'genec-rosh-for-hpa', remote branches 'genec/sample-make-for-hpa', 'genec/drmk-ver-for-hpa' and 'genec/pxechain-for-hpa'

PXELINUX: shorten code in pxenv call

Now that both scenarios are clear (do/do not call timer_cleanup/
timer_init), shorten it to eliminate a needless jmp.

PXELINUX: add gPXE PXENV_FILE_EXEC in addition to PXENV_RESTART_TFTP; comments

Both of these calls will not return if successful

ver.com: Update some of how DRMK is handled

PXELINUX: Fix timer bug

Certain calls to the PXE stack could potentially remove us from
memory.  Revert the INT 1Ch timer to its previous state.

ver.com: Fix month calculation for DRMK Build Date

sample/Makefile: re-add the include

Without the include, .ppm.gz to .lss messes up as $(GZIPPROG) is
not expanded properly

modules/Makefile: add ver.com

ver.com: Fixed width number output; Conditional skip DOS serial

Create two functions, writedecb[23], to output a byte AL as fixed with of
2 or 3 characters wide, 0 prepend

If all of the DOS Serialnumber is 0, don't bother with printing anything.

ver.com: Compact file by reuse of string

ver.com: Update to pull info from DRMK

core/writedec: Remove writechr; fix overflow in .cloop

writechr is found in other files and this prevents a compile error.

ver.com: Import from previously private branch as new file

gfxboot: fix buffer overrun when loading kernel/initramfs

If the file size wasn't a multiple of 64KB, we could overwrite the next
entry in the malloc arena so reading the initramfs would fail.

Signed-off-by: Colin Watson <cjwatson@ubuntu.com>
Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

rosh: rosh_argcat now has length.  if DO_DEBUG, show startup command

rosh: display command entered and a command unknown message when not known

Merge remote branch 'liu/master'

chain.c32: DRMK: Fix comment at top of source

chain.c32: DRMK: Add comment about only using a DWORD for partition offset

chain.c32: DRMK: clean up output; clean up and correct comments

chain.c: Progress on DRMK and notes that it's not perfect yet.

Merge branch 'master' into rosh_for_hpa

rosh: add another item in the ToDos comment

gfxboot: add include and menu include support

Ubuntu CDs use multiple configuration files and the include directive as
well as gfxboot, so it's useful for gfxboot to support these.

Signed-off-by: Colin Watson <cjwatson@ubuntu.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

doc: extlinux falls back to syslinux.cfg

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

doc: reorganize config file intro

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

doc: comments don't need a leading space

Signed-off-by: Ferenc Wagner <wferi@niif.hu>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

chain.c32: make CHS calculation match core/fs/diskio.c

Use the same format for the CHS calculation as in core/fs/diskio.c;
also, apply the correct limits, and propagate the fix to the write
routine.

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

strspn.c: drop "inline"

Drop "inline" from bit operations.  These should be recoded to use x86
bts/bt instructions, but for now just drop the "inline" which causes
failures on gcc 4.5 due to inlining failures.

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

NEWS: Update for 4.03

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

isohybrid: Use ftruncate instead of seek for final padding

Pad the image via ftruncate instead of seeking to the end (which was
done incorrectly).

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

chain.c32: fix conversion from LBA to CHS

Bits [9:8] of the cylinder goes into the top bits of the sector
register, not the bottom (and the math for that was wrong, anyway.)

Reported-by: Michal Soltys <soltys@ziu.info>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

rosh: Fix rosh_ls(); fix FTBFS on rosh_more(); add ROSH_DEBUG2_ARGV_V macro

lib: Update memmem function

The current memem implementation can not handle the case that the length
of the third parameter(needle) is equal *ONE*. So that the current
strstr implemention doesn't work when the lenght of substring is ONE.

For example, strstr("linux", "l") it will return NULL instead of
"linux", which is not right.

This code is stolen from klibc ;)

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

unistd.h: Add missing getopt declarations

Merge branch 'master' into rosh_for_hpa

rosh: Remove unneeded cases in rosh_command() switch statements

rosh: Clean up rosh_command to remove old stuff

rosh: code cleanup

rosh: Change rosh_ls()/rosh_dir() to use argc/argv; use getopt()

rosh: Change rosh_more()/rosh_less() to use argc/argv

rosh: remove cmdstr from rosh_pwd()

rosh: clear errno before possibly using it.  might be set by a previous call

rosh: Add argc/argv rosh_reboot(); Allow warm reboots

rosh: In main, use string macro directly

rosh: Change rosh_run() to use argc/argv

rosh: Change rosh_help() to use the string passed to it directly

rosh: ROSH_DEBUG_ARGV_V in header

rosh: convert rosh_cat() to argc/argv

rosh: Add function header comments that were missing

rosh: Fix warning on rosh_ls_arg(); remove rosh_echo() for rosh_pr_argv()

rosh: Rename function as it's verbose

rosh_pr_argv and ROSH_DEBUG_ARGV are verbose, printing the
argument number and single-quoted string out.

rosh: closedir() only if actually opened

rosh: compact previous commit

rosh: Fix rosh_ls_arg() to prevent segfault on access denied

rosh: main, prompt and command try to use argc/argv mostly

prompt still uses a string but parses it and passes the parsed
argc/argv pair to command.  main doesn't need a string anymore

rosh: rosh_cd fix DEBUG

rosh: fix errors in previous commit

rosh: change rosh_cd() to use argc/argv

rosh: Use internal parser for argc/argv; use argc/argv in rosh_command

rosh: start parse_args(); prep for use; make print_argv()

parse_args() for parsing a string into an argv/argc pair for easier
parsing and the potential use of getopt().

print_argv() for printing an argv/argc pair to show functionality

rosh: fix commented out WS usage in linux to grab screen size

rosh: add echo

Merge remote branch 'genec/rosh_for_hpa'

core: fix handling new directory "/" for CONFIG in ISOLINUX and EXTLINUX

Fix missing option -s in libinstaller

In 'libinstaller/syslxopt.c' on line 71 the "s" (undercast) is missing in:

   const char short_options[] = "t:fid:UuzS:H:rvho:OM:ma";

Adding the 's' there seems to solve our problem.

pxe: add a "pxeretry" option to deal with 404's in web apps

For web downloads, sometimes a mirror site will not be fully synced.
Add an option to retry the open a specific number of times before
giving up.

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

extlinux: try to get the disk offset from sysfs

It is possible(?) that HDIO_GETGEO can't return the full offset, and
in either case it is too small -- only 32 bits on 32-bit platforms.
Thus query sysfs for the real value, if available.

sysfs also contains information for slave devices -- this is going to
really matter for the md/lvm issues.

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

diskio: remove redundant register write

The disk number is set twice in diskio.c/edd_rdwr_sectors() this patch
removes the redundant setting.

version: next version will be 4.03

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

core: look for invalid MBR handover information address

The Win7 MBR doesn't provide proper handover information.  Instead, it
leaves DS:SI = 0:0x7e00, so reject that information as invalid on
grounds that it is pointed into the bootloader area.

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

chain: Extra comments; shrink the new space to minimum

chain: Just assign the value rather than using memcpy()

chain: add drmk= to usage

chain: Fix up and clean up DRMK load code; fix filename length;  Patch memory.

chain.c32: prepare for loading DRMK

syslxopt: remove lengthy warning message

Remove a lengthy warning message which cause the actual command
reference to overflow a 24-line screen.

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

syslxopt: fix fatal warnings

Fix fatal warnings in syslxopt.c partly due to legal but non-idiomatic
use of switch().  Since the only purpose was to conditionalize single
output statements, just do it explicitly.

In addition, since they are conditional messages, there is no need to
explain when they apply.

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

chain.c: fix public index value in mbr and gpt iterators

mbr and gpt iterators skip empty / non-data partitions properly, but don't
update publicly visible index. With this patch, such partitions are counted as
well.

Signed-off-by: Michal Soltys <soltys@ziu.info>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

libinstaller/syslxopt.c: remove unneeded usage text from certain variants.

Gert Hulselmans noticed this.

Make __free_tagged actually free memory

__free_tagged called by comboot_cleanup_lowmem (core/mem/free.c) doesn't
seem to work correctly. Memory allocated with lmalloc() isn't marked as free
after the allocating module exits:

boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000553f0
boot: mem
lowmem_buf: 0x00075400
boot: mem
lowmem_buf: 0x00000000
boot:

The expected behaviour

boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000353e0

can be achieved with the following patch.

Merge branch 'm' into rosh_for_hpa

core: define and use set_flags() helper

Rather than opencoding the arithmetic flag mask, define an inline to
make that happen properly.

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

sample: remove obsolete "raw" comboot/com32 samples

We really don't want people to write code without using the library,
so just remove these obsolete and no longer functional samples.

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

libmenu: Fix cursor on exit from showmenus()

This fixes the issue in hdt.c32 that Kent Robotti noticed where
exiting directly from Menu Mode caused the cursor to not be displayed.
This same issue also affects com32/cmenu/simple.c32 and probably
others using showmenus().

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core: don't hang if no config file is found

If we can't find a configuration file, we need to do something
slightly more friendly than hanging.

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

win64: experimental Win64 installer (syslinux64.exe)

Experimental Win64 installer.  This is necessary to be able to run on
WinPE64, since WinPE64 doesn't support running 32-bit binaries.

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

win32: fix README, adjust broken +x bits

Fix the win32/README file and adjust some stray +x bits which had
crept in.

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

lua: remove obsolete FIXME comment

Remove an obsolete FIXME comment; the problem was caused by a stale
library.

NEWS: document IP byte order fix.

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

pxe: fix byte order of output from parse_dotquad()

parse_dotquad() incorrectly returned output in host byte order.

Reported-by: Alexander Wuerstlein <arw@arw.name>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

extlinux, linux: Ignore --force in these installers

Previous commits described this behaviour, but did not
actually implement it. :)

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

dos: Use libinstaller option parser

We will produce an error message if a user attempts
to use an option we don't implement.

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

dos: add .size directives to int2526.S

Add .size directives to int2526.S.

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

dos: add .size attribute for syslinux_ldlinux_len

Just for the sake of good order, a .size attribute for
syslinux_ldlinux_len.

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

dos: add missing file ldlinux.S

Add missing assembly wrapper for ldlinux.sys.

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