memdisk/dskprobe: Be consistent in debug output

Only probe_int13h_15h() showed the AH01h output; show it in the others also

memdisk/dskprobe: Reorder operations to compact the debug output

By listing current regs, evaluate present, then display present last, it
allows it to conveniently use only 1 output line.

memdisk/dskprobe: Code cleanup

memdisk/dskprobe: Use status of last command to affect present

memdisk/dskprobe: Show the status of the last command when debugging

INT13h AH01h is supposed to show the status of the last command.  Show
its status and determine if it's a known failure.

This does not affect the "present" variable at this time.

memdisk/dskprobe: Increase the debug output

Include all registers that should include anything of value and display
the value of the "present" variable.

memdisk/dskprobe.c: Additional checks in INT13h AH08h and AH41h

Some BIOSs lie and don't set CF when they should.

Additional comments from Shao:

Fix INT 0x13, AH==0x08 disk drive probe.

This function might return CF==0 ("success"), but could return a
standard response no matter which drive number is passed in DL, but
based only on DL's bit 7.  Fortunately, AH should be 0 for an existent
drive and 01h for a non-existent drive.  Now we check AH.

Fix INT 0x13, AH==0x41 disk drive probe

This function might return CF==0 ("success"), but could also include BX
left unchanged.  If extensions are present, BX should be returned with
0xAA55.  So we now check for this special value in our determination of
a drive's presence.

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

Merge remote branch 'hdt/master'

Merge remote branch 'sha0/memdskprobe'

disk: put a magic at the end of the boot sector

Put a magic signature at the end of the boot sector, and a backpointer
to the code that contains the pointer to the main code extent.  This
is useful for integrity-checking tools, and could help the installer
in the future.

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

memdisk: Fix INT 0x13, AH==0x15 disk drive probe

The function might return CF==0 ("success"), but could
include AH==0 ("no such drive").  This is the case on
at least a number of Dell models.  Now we check AH, too.

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

diskstart: move writestr_early out of boot sector

We can save 8 bytes in the boot sector by moving writestr_early out
and putting the real routine in sector 1.

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

memdisk: Enhance disk-probe debugging output

In an effort to trouble-shoot a problem report on some Dell
models (including an Optiplex GX260), we add further debugging
output to try to find out at which point things go wrong.  The
problem units are apparently probing as "drive present" for all
drives, which obviously isn't right.

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

disklib: Detect DiskCryptor MBR

Add detection code for DiskCryptor MBR to disklib.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

disklib: Detect "FreeDOS (eXtended FDisk)" MBR

Add detection code for "FreeDOS (eXtended FDisk)" MBR to disklib.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

disklib: Detect "MS-DOS 3.30 through Windows 95 (A)" MBR

Add detection code for "MS-DOS 3.30 through Windows 95 (A)" MBR to disklib.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

disklib: Detect new Syslinux (normal + isohybrid) MBR

Add detection code for new Syslinux (normal + isohybrid) MBR to disklib.
Switch old Syslinux MBR and Master Boot LoaDeR detection code.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

disklib: Detect Paragon MBR

Add detection code for Paragon MBR to disklib.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

memdisk: Make debug-mode a tad prettier

By removing the use of #ifdef-#endif blocks.

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

diskio: Support EDD 4 16-byte device paths

EDD 4 has 16-byte device path information.  Make the EDD buffer big
enough to capture that.  Note that the location of the device path
checksum actually depends on the device path length information field!

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

diskio: add reminder that we need clear in the future

Currently, disk_init() can only be called once, and edd_params get
statically initialized to zero.  This will change once this function
can be called more than once, and we depend on the pre-initialization
to zero to avoid known BIOS bugs.

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

NEWS: Document MBR fix.

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

mbr: Make sure the MBR code starts with the byte 0x33

Apparently some BIOSes (including some Acer Travelmate machines)
require an MBR to start with 0x33; apparently Micro$oft MBRs start
with 33 C0, an alternate coding of the "xorw %ax,%ax" instruction.  As
such, follow suit to work on these braindead BIOSes.

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

core/diskboot.inc: fix bug; Sect1Ptr1_VAL

core/diskstart.inc: Remove code copied to diskboot.inc

core/diskboot.inc: The boot sector code from diskstart.inc

Split to allow it to be used by debugging/diagnostic images

core/diskstart.inc: Sect1Ptr values moved to constants

This allows another program to re-use the boot sector and define
different values, once split.

core/diskstart.inc: Use xint13 when it saves space or protects

Additional note on where it will cost to possibly not protect anything

core/diskstart.inc: Wrap INT13h in xint13 to preserve ES

Sometimes INT13h AH08h kills ES to return info about a "floppy" (since
an unpartitioned HDD might look like a floppy).  Also, change to tabs
on previous line.

core: Add workaround in the Makefile for GNU Make 3.82 bug

Apparently GNU Make 3.82 mishandles pattern rules with fixed
dependencies, sigh.

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

core: remove HAS_LOCALBOOT

HAS_LOCALBOOT is set unconditionally in config.inc.

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

NEWS: document fix for non-partitioned devices.

core, diskstart: add more sanity checks for handover info

In particular, somehow we failed to actually test for DS:SI = 0:0!
This tests both for SI < 16 and DS:SI < 1024.  The former of those
tests is a bit iffy and probably should be removed.

This should also be promoted into isohdpfx.

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

syslinux.ld: ld 2.20.51 seems to want .bss16 explicitly (NOLOAD)

ld 2.20.51 seems to want the .bss16 explicitly marked (NOLOAD).  We do
that for most of the other bss sections, so we might as well.

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

memdiskfind: abstract get page size so we can compile against klibc

klibc doesn't have sysconf(), because it involves a hideously
inefficient multiplex.  Thus, if _SC_PAGESIZE is not defined, invoke
getpagesize().  Call the routine get_page_size() so it doesn't
conflict on a platform which has both sysconf(_SC_PAGESIZE) and
getpagesize().

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

NEWS: further updates

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

com32: fix a ffile descriptor leak on open() of a nonexistent file

If we try to open a nonexistent file, free the resulting file
descriptor.

Reported-by: Antonio Carlini <arcarlini@iee.org>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

gfxboot: require LABEL keywords

Require LABEL keywords in config file.

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

gfxboot: support TEXT and ENDTEXT keywords

Support TEXT and ENDTEXT keywords; ignore the help text.

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

NEWS: Update for 4.04 as of 4.04-pre1

Bump version to 4.04

man: remove long-since-obsolete note about name mangling

We haven't mangled label names for a very long time...

core, pxe: Fix handling of unqualified DNS names

Actually append the domain name to the end of an unqualified DNS name
like we should have done all along.

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

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.