ipxe.git
11 years ago[makefile] Add -Wformat-nonliteral as an extra warning category
Michael Brown [Fri, 10 Oct 2008 17:41:24 +0000 (18:41 +0100)] 
[makefile] Add -Wformat-nonliteral as an extra warning category

-Wformat-nonliteral is not enabled by -Wall and needs to be explicitly
 specified.

Modified the few files that use nonliteral format strings to work with
this new setting in place.

Inspired by a patch from Carl Karsten <carl@personnelware.com> and an
identical patch from Rorschach <r0rschach@lavabit.com>.

11 years ago[efi] Add EFI headers from the EFI Development Kit (edk2)
Michael Brown [Wed, 8 Oct 2008 01:30:48 +0000 (02:30 +0100)] 
[efi] Add EFI headers from the EFI Development Kit (edk2)

The intention is to include near-verbatim copies of the EFI headers
required by gPXE.  This is achieved using the import.pl script in
src/include/gpxe/efi.

Note that import.pl will modify any #include lines in each imported
header to reflect its new location within the gPXE tree.  It will also
tidy up the file by removing carriage return characters and trailing
whitespace.

11 years ago[i386] Simplify linker script and standardise linker-defined symbol names
Michael Brown [Thu, 9 Oct 2008 21:22:01 +0000 (22:22 +0100)] 
[i386] Simplify linker script and standardise linker-defined symbol names

Reduce the number of sections within the linker script to match the
number of practical sections within the output file.

Define _section, _msection, _esection, _section_filesz, _section_memsz,
and _section_lma for each section, replacing the mixture of symbols that
previously existed.

In particular, replace _text and _end with _textdata and _etextdata, to
make it explicit within code that uses these symbols that the .text and
.data sections are always treated as a single contiguous block.

11 years ago[makefile] Reduce verbosity of various NON_AUTO_MEDIA rules
Michael Brown [Fri, 10 Oct 2008 02:57:31 +0000 (03:57 +0100)] 
[makefile] Reduce verbosity of various NON_AUTO_MEDIA rules

11 years ago[makefile] Split platform-dependent portions out of arch/i386/Makefile
Michael Brown [Fri, 10 Oct 2008 01:14:22 +0000 (02:14 +0100)] 
[makefile] Split platform-dependent portions out of arch/i386/Makefile

11 years ago[makefile] Inhibit warnings if AUTO_DEPS or MEDIA_DEPS are empty lists
Michael Brown [Fri, 10 Oct 2008 01:09:55 +0000 (02:09 +0100)] 
[makefile] Inhibit warnings if AUTO_DEPS or MEDIA_DEPS are empty lists

11 years ago[makefile] Add missing dependency on arch/$(ARCH)/Makefile to MAKEDEPS
Michael Brown [Fri, 10 Oct 2008 01:08:31 +0000 (02:08 +0100)] 
[makefile] Add missing dependency on arch/$(ARCH)/Makefile to MAKEDEPS

11 years ago[util] Don't die on undefined symbols in sortobjdump.pl
Michael Brown [Fri, 10 Oct 2008 01:07:04 +0000 (02:07 +0100)] 
[util] Don't die on undefined symbols in sortobjdump.pl

Undefined symbols shouldn't reach sortobjdump.pl, but if they do then it
is not a helpful place to report the error.

11 years ago[util] Add optional debug messages to zbin.c
Michael Brown [Thu, 9 Oct 2008 17:02:19 +0000 (18:02 +0100)] 
[util] Add optional debug messages to zbin.c

11 years ago[makefile] Split config.h out into config/*.h and kill off mkconfig.pl
Michael Brown [Tue, 7 Oct 2008 23:37:40 +0000 (00:37 +0100)] 
[makefile] Split config.h out into config/*.h and kill off mkconfig.pl

11 years ago[makefile] Add support for multiple build platforms
Michael Brown [Tue, 7 Oct 2008 22:18:15 +0000 (23:18 +0100)] 
[makefile] Add support for multiple build platforms

Allow for the build CPU architecture and platform to be specified as part
of the make command goals.  For example:

  make bin/rtl8139.rom      # Standard i386 PC-BIOS build

  make bin-efi/rtl8139.efi  # i386 EFI build

The generic syntax is "bin[-[arch-]platform]", with the default
architecture being "i386" (regardless of the host architecture) and the
default platform being "pcbios".

Non-path targets such as "srcs" can be specified using e.g.

  make bin-efi srcs

Note that this changeset is merely Makefile restructuring to allow the
build architecture and platform to be determined by the make command
goals, and to export these to compiled code via the ARCH and PLATFORM
defines.  It doesn't actually introduce any new build platforms.

11 years ago[makefile] Kill off arch/$(ARCH)/Config
Michael Brown [Tue, 7 Oct 2008 21:46:43 +0000 (22:46 +0100)] 
[makefile] Kill off arch/$(ARCH)/Config

The arch/i386/Config file has long been marked as deprecated.  Move all
the pertinent bits to arch/i386/Makefile instead and remove
arch/i386/Config.

11 years ago[libc] Define wchar_t in a gcc-compatible way
Michael Brown [Tue, 7 Oct 2008 21:27:55 +0000 (22:27 +0100)] 
[libc] Define wchar_t in a gcc-compatible way

gcc defines the magic __WCHAR_TYPE__ macro, in order to convey
information about whether or not the user selected -fshort-wchar.

11 years ago[libc] Add function declaration for main() in stdlib.h
Michael Brown [Fri, 12 Sep 2008 01:06:49 +0000 (02:06 +0100)] 
[libc] Add function declaration for main() in stdlib.h

11 years ago[crypto] Rename <gpxe/bitops.h> to <gpxe/rotate.h>
Michael Brown [Tue, 30 Sep 2008 00:20:04 +0000 (01:20 +0100)] 
[crypto] Rename <gpxe/bitops.h> to <gpxe/rotate.h>

11 years ago[i2c] Generalise i2c bit-bashing support to addressless devices
Michael Brown [Wed, 1 Oct 2008 18:15:07 +0000 (19:15 +0100)] 
[i2c] Generalise i2c bit-bashing support to addressless devices

Some devices (e.g. the Atmel AT24C11) have no concept of a device
address; they respond to every device address and use this value as
the word address.  Some other devices use part of the device address
field to extend the word address field.

Generalise the i2c bit-bashing support to handle this by defining the
device address length and word address length as properties of an i2c
device.  The word address is assumed to overflow into the device
address field if the address used exceeds the width of the word
address field.

Also add a bus reset mechanism.  i2c chips don't usually have a reset
line, so rebooting the host will not clear any bizarre state that the
chip may be in.  We reset the bus by clocking SCL until we see SDA
high, at which point we know we can generate a start condition and
have it seen by all devices.  We then generate a stop condition to
leave the bus in a known state prior to use.

Finally, add some extra debugging messages to i2c_bit.c.

11 years ago[makefile] Fix -fno-stack-protector test on older versions of gcc
Michael Brown [Wed, 1 Oct 2008 18:05:39 +0000 (19:05 +0100)] 
[makefile] Fix -fno-stack-protector test on older versions of gcc

Some older versions of gcc don't complain about unknown compiler flags
unless you ask them to actually compile; asking them to merely
preprocess won't trigger the error.

Fix the -fno-stack-protector test by making it attempt to compile an
empty file, rather than preprocess an empty file.

11 years ago[compiler] Allow for selective disabling of debug levels at runtime
Michael Brown [Wed, 1 Oct 2008 17:58:21 +0000 (18:58 +0100)] 
[compiler] Allow for selective disabling of debug levels at runtime

The usefulness of DBGLVL_IO is limited by the fact that many cards
require large numbers of uninteresting I/O reads/writes at device
probe time, typically when driving a bit-bashing I2C/SPI bus to read
the MAC address.

This patch adds the DBG_DISABLE() and DBG_ENABLE() macros, which can
be used to temporarily disable and re-enable selected debug levels.
Note that debug levels must still be enabled in the build in order to
function at all: you can't use DBG_ENABLE(DBGLVL_IO) in an object
built with DEBUG=object:1 and expect it to do anything.

11 years ago[compiler] Add __always_inline macro
Michael Brown [Sat, 13 Sep 2008 15:21:47 +0000 (16:21 +0100)] 
[compiler] Add __always_inline macro

11 years ago[release] Update version to 0.9.5+ post release
Marty Connor [Wed, 1 Oct 2008 17:46:16 +0000 (13:46 -0400)] 
[release] Update version to 0.9.5+ post release

11 years ago[release] Update version to 0.9.5 for release v0.9.5
Marty Connor [Wed, 1 Oct 2008 17:38:45 +0000 (13:38 -0400)] 
[release] Update version to 0.9.5 for release

11 years ago[arbel] Fix off-by-one error in the failure path in arbel_probe()
Michael Brown [Mon, 29 Sep 2008 23:38:10 +0000 (00:38 +0100)] 
[arbel] Fix off-by-one error in the failure path in arbel_probe()

11 years ago[hermon] Fix off-by-one error in the failure path in hermon_probe()
Michael Brown [Mon, 29 Sep 2008 23:31:02 +0000 (00:31 +0100)] 
[hermon] Fix off-by-one error in the failure path in hermon_probe()

11 years ago[makefile] Quote argument to sed containing []
H. Peter Anvin [Mon, 29 Sep 2008 20:50:21 +0000 (13:50 -0700)] 
[makefile] Quote argument to sed containing []

[] are shell metacharacters, and they can get confused by files in the
local directory.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
11 years ago[makefile] Avoid using ?=
H. Peter Anvin [Mon, 29 Sep 2008 20:48:51 +0000 (13:48 -0700)] 
[makefile] Avoid using ?=

?= in a Makefile means that that variable can be overridden by the
environment.  This is confusing to users, especially with a generic
name like "ARCH".

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
11 years ago[tg3] Add PCI ID
Michael Brown [Mon, 29 Sep 2008 15:00:09 +0000 (16:00 +0100)] 
[tg3] Add PCI ID

Reported working by Martin Kreiner <m.kreiner@levigo.de>

11 years ago[pcbios] Allow for larger-than-20-byte buffers in e820mangler.S
Michael Brown [Mon, 29 Sep 2008 04:11:51 +0000 (05:11 +0100)] 
[pcbios] Allow for larger-than-20-byte buffers in e820mangler.S

Although the E820 API allows for a caller to provide only a 20-byte
buffer, there exists at least one combination (HP BIOS, 32-bit WinPE)
that relies on information found only in the "extended attributes"
field, which requires a 24-byte buffer.

Allow for up to a 64-byte E820 buffer, in the hope of coping with
future idiocies like this one.

11 years ago[pcbios] Print INT 15,E820 extended attributes, if present
Michael Brown [Mon, 29 Sep 2008 02:55:13 +0000 (03:55 +0100)] 
[pcbios] Print INT 15,E820 extended attributes, if present

The ACPI specification defines an additional 4-byte field at offset 20
for an E820 memory map entry.  This field is presumably optional,
since generally E820 gets given only a 20-byte buffer to fill.
However, the bits of this optional field are defined as:

  bit 0 : region is enabled
  bit 1 : region is non-volatile memory rather than RAM

so it seems as though callers that pass in only a 20-byte buffer may
be missing out on some rather important information.

11 years ago[gdb] Fix a compiler warning that shows up only when assertions are enabled
Michael Brown [Mon, 29 Sep 2008 00:00:14 +0000 (01:00 +0100)] 
[gdb] Fix a compiler warning that shows up only when assertions are enabled

gcc should (I think) be warning about this anyway, but seems to do so
only when assertions are enabled for this object.

11 years ago[pcbios] Save/restore %es in INT 15,e820
Michael Brown [Sun, 28 Sep 2008 23:36:11 +0000 (00:36 +0100)] 
[pcbios] Save/restore %es in INT 15,e820

Our INT 15,e820 code was setting %es=%ss (as part of the "look ahead
in the memory map" logic), but failing to restore %es afterwards.
This is a serious bug, but wasn't affecting many platforms because
almost all callers seem to set %es=%ss anyway.

11 years ago[i386] Add dump_regs() debug call
Michael Brown [Sun, 28 Sep 2008 22:06:53 +0000 (23:06 +0100)] 
[i386] Add dump_regs() debug call

Use as "call dump_regs" from any real-mode code within .text16.
Should preserve all registers and flags.

11 years ago[hermon] Allocate only as much memory as is needed for ICM and ICM AUX
Michael Brown [Sat, 27 Sep 2008 22:50:14 +0000 (23:50 +0100)] 
[hermon] Allocate only as much memory as is needed for ICM and ICM AUX

Use individual page mappings rather than a single whole-region
mapping, to avoid the waste of memory that occurs due to the
constraint that each mapped block must be aligned on its own size.

11 years ago[iscsi] Fix LUN parsing in the iSCSI root-path
Michael Brown [Fri, 26 Sep 2008 20:30:53 +0000 (21:30 +0100)] 
[iscsi] Fix LUN parsing in the iSCSI root-path

11 years ago[release] Update version post release to 0.9.4+
Marty Connor [Fri, 26 Sep 2008 20:05:55 +0000 (16:05 -0400)] 
[release] Update version post release to 0.9.4+

11 years ago[release] Update version information for 0.9.4 release v0.9.4
Marty Connor [Fri, 26 Sep 2008 19:14:34 +0000 (15:14 -0400)] 
[release] Update version information for 0.9.4 release

11 years ago[settings] Allow for setting fetchers to fail in fetchf_hex()
Michael Brown [Fri, 26 Sep 2008 02:39:35 +0000 (03:39 +0100)] 
[settings] Allow for setting fetchers to fail in fetchf_hex()

11 years ago[settings] Expose MAC address setting for general use
Michael Brown [Fri, 26 Sep 2008 02:39:17 +0000 (03:39 +0100)] 
[settings] Expose MAC address setting for general use

11 years ago[romprefix] Fully clear the "Press B to boot..." message when INT19 is used
Michael Brown [Fri, 26 Sep 2008 00:36:22 +0000 (01:36 +0100)] 
[romprefix] Fully clear the "Press B to boot..." message when INT19 is used

11 years ago[pcbios] Fetch INT 15,e820 entry directly into our e820 cache
Michael Brown [Thu, 25 Sep 2008 17:45:30 +0000 (18:45 +0100)] 
[pcbios] Fetch INT 15,e820 entry directly into our e820 cache

Some BIOSes require us to pass in not only the continuation value (in
%ebx) as returned by the previous call to INT 15,e820 but also the
unmodified buffer (at %es:%di) as returned by the previous call to INT
15,e820.  Apparently, someone thought it would be a worthwhile
optimisation to fill in only the low dword of the "length" field and
the low byte of the "type field", assuming that the buffer would
remain unaltered from the previous call.

This problem was being triggered by the "peek ahead" logic in
get_mangled_e820(), which would read the next entry into a temporary
buffer in order to be able to guarantee terminating the map with
%ebx=0 rather than CF=1.  (Terminating with CF=1 upsets some Windows
flavours, despite being documented legal behaviour.)

Work around this problem by always fetching directly into our e820
cache; that way we can guarantee that the underlying call always sees
the previous buffer contents (and the same buffer address).

11 years ago[arbel] Allocate sufficient space for firmware buffer
Michael Brown [Thu, 25 Sep 2008 06:23:34 +0000 (07:23 +0100)] 
[arbel] Allocate sufficient space for firmware buffer

We were accidentally allocating only half the required amount of
memory (given the alignment method) for the firmware buffer, leading
to conflicts between the firmware buffer and gPXE code/data segments.

11 years ago[hermon] Allocate sufficient space for firmware buffer
Michael Brown [Thu, 25 Sep 2008 06:22:06 +0000 (07:22 +0100)] 
[hermon] Allocate sufficient space for firmware buffer

We were accidentally allocating only half the required amount of
memory (given the alignment method) for the firmware buffer, leading
to conflicts between the firmware buffer and gPXE code/data segments.

11 years ago[pcbios] Add facility for testing arbitrary E820 memory maps
Michael Brown [Thu, 25 Sep 2008 02:34:26 +0000 (03:34 +0100)] 
[pcbios] Add facility for testing arbitrary E820 memory maps

We seem to be having issues with various E820 memory maps.  These
problems are often difficult to reproduce, requiring access to the
specific system exhibiting the problem.

Add a facility for hooking in a fake E820 map generator, using an
arbitrary map defined in a C array, solely in order to be able to test
the map-mangling code against arbitrary E820 maps.

11 years ago[crypto] Remove spurious #include "config.h"
Michael Brown [Tue, 16 Sep 2008 01:38:29 +0000 (02:38 +0100)] 
[crypto] Remove spurious #include "config.h"

11 years ago[romprefix] Allow BANNER_TIMEOUT to control banners in romprefix.S
Michael Brown [Thu, 25 Sep 2008 00:53:42 +0000 (01:53 +0100)] 
[romprefix] Allow BANNER_TIMEOUT to control banners in romprefix.S

In particular, allow BANNER_TIMEOUT=0 to inhibit the prompt banners
altogether.

Ironically, this request comes from the same OEM that originally
required the prompts to be present during POST.

11 years ago[dhcp] Accept BOOTP as well as DHCP
Michael Brown [Thu, 25 Sep 2008 00:35:15 +0000 (01:35 +0100)] 
[dhcp] Accept BOOTP as well as DHCP

11 years ago[pxe] Enable interrupts before starting PXE NBP execution
Michael Brown [Wed, 24 Sep 2008 20:23:50 +0000 (21:23 +0100)] 
[pxe] Enable interrupts before starting PXE NBP execution

Based on a patch provided by XenSource for Etherboot 5.4.

11 years ago[pcbios] Inhibit INT 15 memory map hiding on brain-dead BIOSes
Michael Brown [Wed, 24 Sep 2008 17:33:31 +0000 (18:33 +0100)] 
[pcbios] Inhibit INT 15 memory map hiding on brain-dead BIOSes

Some really moronic BIOSes bring up the PXE stack via the UNDI loader
entry point during POST, and then don't bother to unload it before
overwriting the code and data segments.  If this happens, we really
don't want to leave INT 15 hooked, because that will cause any loaded
OS to die horribly as soon as it attempts to fetch the system memory
map.

We use a heuristic to detect whether or not we are being loaded at the
top of free base memory.  If we determine that we are being loaded at
some other arbitrary location in base memory, then we assume that it's
not safe to hook INT 15.

11 years ago[settings] Add the uristring setting type
Michael Brown [Wed, 24 Sep 2008 06:22:42 +0000 (07:22 +0100)] 
[settings] Add the uristring setting type

This allows settings to be expanded in a way that is safe to include
within a URI string, such as

  kernel http://10.0.0.1/boot.php?mf=${manufacturer:uristring}

where the ${manufacturer} setting may contain characters that are not
permitted (or have reserved purposes) within a URI.

Since whitespace characters will be URI-encoded (e.g. "%20" for a
space character), this also works around the problem that spaces
within an expanded setting would cause the shell to split command-line
arguments incorrectly.

11 years ago[uri] Add uri_encode() and uri_decode() functions for URI character encoding
Michael Brown [Wed, 24 Sep 2008 06:21:47 +0000 (07:21 +0100)] 
[uri] Add uri_encode() and uri_decode() functions for URI character encoding

11 years ago[romprefix] If we hook INT 19, prompt before attempting boot
Michael Brown [Tue, 23 Sep 2008 23:53:40 +0000 (00:53 +0100)] 
[romprefix] If we hook INT 19, prompt before attempting boot

On non-BBS systems we hook INT 19, since there is no other way we can
guarantee gaining control of the flow of execution.  If we end up
doing this, prompt the user before attempting boot, since forcibly
capturing INT 19 is rather antisocial.

11 years ago[phantom] Skip command PEG initialisation if PEG is already running
Michael Brown [Tue, 23 Sep 2008 22:36:55 +0000 (23:36 +0100)] 
[phantom] Skip command PEG initialisation if PEG is already running

It is possible for the BIOS to use the UNDI API to bring up the NIC
prior to system boot.  If this happens, UNM_NIC_REG_CMDPEG_STATE will
contain the value 0xf00f (UNM_NIC_REG_CMDPEG_STATE_INITIALIZE_ACK),
and we should skip initialising the command PEG.

11 years ago[pcbios] Sanity-check the INT15,e820 and INT15,e801 memory maps
Michael Brown [Tue, 23 Sep 2008 22:35:19 +0000 (23:35 +0100)] 
[pcbios] Sanity-check the INT15,e820 and INT15,e801 memory maps

Some systems seem to report insane memory maps (particularly at POST
time).  Detect and work around some of the common failure cases.

11 years ago[phantom] Omit an initialisation step now performed by the firmware
Michael Brown [Tue, 23 Sep 2008 22:11:05 +0000 (23:11 +0100)] 
[phantom] Omit an initialisation step now performed by the firmware

The firmware will now determine the right port mode on all cards, so
the PXE driver doesn't have to set it.  (Setting the port mode
apparently breaks some newer cards.)

11 years ago[pxe] Display PXE_LOADER debug message after call to initialise()
Michael Brown [Tue, 23 Sep 2008 03:15:56 +0000 (04:15 +0100)] 
[pxe] Display PXE_LOADER debug message after call to initialise()

At least one Dell system calls the UNDI loader entry point with the
BIOS console disabled.  The serial console is active only after a call
to initialise(), so move the debug message in undi_loader() so that it
can be displayed via the serial console.

11 years ago[relocate] Guard against systems that report empty memory regions
Michael Brown [Tue, 23 Sep 2008 00:13:28 +0000 (01:13 +0100)] 
[relocate] Guard against systems that report empty memory regions

If the INT 15,e820 memory map reports a region [0,0), this confuses
the "truncate to even megabytes" logic, which ends up rounding the
region 'down' to [0,fff00000).

Fix by ensuring that the region's end address is at least 1, before we
subtract 1 to obtain the "last byte in region" address.

11 years ago[memmap] Allow for 4GB+ systems in debug message
Michael Brown [Mon, 22 Sep 2008 23:58:12 +0000 (00:58 +0100)] 
[memmap] Allow for 4GB+ systems in debug message

INT 15,e801 is capable of returning a memory range that extends to
4GB, so allow for this in the debug message that shows the data
returned by INT 15,e801.

11 years ago[iscsi] Change default initiator name prefix to "iqn.2000-01.org.etherboot:"
Michael Brown [Fri, 19 Sep 2008 16:46:07 +0000 (17:46 +0100)] 
[iscsi] Change default initiator name prefix to "iqn.2000-01.org.etherboot:"

The domain etherboot.org was actually registered on 2000-01-09, not
2000-09-01.  (To put it another way, it was registered on 1/9/2000 (US
date format) rather than 1/9/2000 (sensible date format); this may
illuminate the cause of the error.)

"iqn.2000-09.org.etherboot:" is still valid as per RFC3720, but may be
surprising to users, so change it to something less unexpected.

Thanks to the anonymous contributor for pointing this one out.

11 years ago[undi] Scan for UNDI ROMs on 512-byte boundaries rather than 2kB boundaries
Michael Brown [Fri, 12 Sep 2008 02:10:27 +0000 (03:10 +0100)] 
[undi] Scan for UNDI ROMs on 512-byte boundaries rather than 2kB boundaries

Apparently some BIOSes will place option ROMs on 512-byte boundaries.
While this is against specification, it doesn't actually hurt
anything, so we may as well increase our scan granularity to 512
bytes.

Contributed by Luca <lucarx76@gmail.com>

11 years ago[util] Disable automatic CR->LF conversion in serial-console utility
Michael Brown [Sun, 7 Sep 2008 21:08:40 +0000 (22:08 +0100)] 
[util] Disable automatic CR->LF conversion in serial-console utility

DUET (the EFI test environment) seems not to handle LF, so inhibit the
CR->LF conversion that the pty does for us by default.  This doesn't
affect operation of gPXE, which will happily accept either CR or LF.

11 years ago[pcbios] Add extra debugging messages relating to the system memory map
Michael Brown [Sun, 7 Sep 2008 20:38:19 +0000 (21:38 +0100)] 
[pcbios] Add extra debugging messages relating to the system memory map

11 years ago[contrib] Bring bochs/qemu READMEs up to date with latest upstream code
Michael Brown [Sun, 7 Sep 2008 10:31:42 +0000 (11:31 +0100)] 
[contrib] Bring bochs/qemu READMEs up to date with latest upstream code

11 years ago[pxe] Add a dummy PXENV+ entry point at UNDI_CS:0000
Michael Brown [Sat, 6 Sep 2008 02:19:03 +0000 (03:19 +0100)] 
[pxe] Add a dummy PXENV+ entry point at UNDI_CS:0000

Wyse Streaming Manager server (WLDRM13.BIN) assumes that the PXENV+
entry point is at UNDI_CS:0000; apparently, somebody at Wyse has
difficulty distinguishing between the words "may" and "must"...

Add a dummy entry point at UNDI_CS:0000, which just jumps to the
correct entry point.

11 years ago[multiboot] Allow for unspecified {load,bss}_end_addr for raw images
Michael Brown [Sat, 6 Sep 2008 00:57:25 +0000 (01:57 +0100)] 
[multiboot] Allow for unspecified {load,bss}_end_addr for raw images

The multiboot specification states that, for raw images, if
load_end_addr is zero then it should be interpreted as meaning "use
the entire file", and if bss_end_addr is zero it should be interpreted
as meaning "no bss".

11 years ago[util] Avoid calling fclose(NULL) in zbin.c
Eygene Ryabinkin [Sun, 31 Aug 2008 18:17:30 +0000 (22:17 +0400)] 
[util] Avoid calling fclose(NULL) in zbin.c

Must check that argument to a fclose() is not NULL -- we can get to the
'err' label when file was not opened.  fclose(NULL) is known to produce
core dump on some platforms and we don't want zbin to fail so loudly.

Signed-off-by: Eygene Ryabinkin <rea-fbsd@codelabs.ru>
11 years ago[libprefix] Add addr32 prefix required by older assemblers
Eygene Ryabinkin [Sun, 31 Aug 2008 18:05:50 +0000 (22:05 +0400)] 
[libprefix] Add addr32 prefix required by older assemblers

Explicitly state that we are using 32-bit addressing in 16-bit code.

GNU as 2.15 (FreeBSD/amd64 7-STABLE) got confused that 32-bit registers
are used in the code that was declared as 16-bit.  Add explicit modifier
'addr32' to make assembler happy.

Signed-off-by: Eygene Ryabinkin <rea-fbsd@codelabs.ru>
11 years ago[comboot] Add COMBOOT and COM32 support
Daniel Verkamp [Thu, 28 Aug 2008 22:52:19 +0000 (23:52 +0100)] 
[comboot] Add COMBOOT and COM32 support

11 years ago[librm] Add rm stack copying functions
Daniel Verkamp [Fri, 15 Aug 2008 18:16:16 +0000 (14:16 -0400)] 
[librm] Add rm stack copying functions

11 years ago[librm] Make rm_sp and rm_ss globals again
Daniel Verkamp [Fri, 15 Aug 2008 17:56:11 +0000 (13:56 -0400)] 
[librm] Make rm_sp and rm_ss globals again

11 years ago[libc] Mark setjmp and longjmp as __cdecl
Daniel Verkamp [Tue, 15 Jul 2008 16:15:51 +0000 (12:15 -0400)] 
[libc] Mark setjmp and longjmp as __cdecl

11 years ago[pxe] Fill in UNDIROMID pointer in !PXE structure
Michael Brown [Wed, 27 Aug 2008 22:45:59 +0000 (23:45 +0100)] 
[pxe] Fill in UNDIROMID pointer in !PXE structure

IBM's iSCSI Firmware Initiator checks the UNDIROMID pointer in the
!PXE structure that gets created by the UNDI loader.  We didn't
previously fill this value in.

11 years ago[romprefix] Add UNDI ROMID structure bus type field
Michael Brown [Wed, 27 Aug 2008 21:08:57 +0000 (22:08 +0100)] 
[romprefix] Add UNDI ROMID structure bus type field

The IBM iSCSI Firmware Initiator complains that we don't have a bus
type field in the UNDI ROMID structure, so include one.

11 years ago[phantom] Fix P3 B1 silicon bug workaround
Michael Brown [Wed, 27 Aug 2008 20:25:06 +0000 (21:25 +0100)] 
[phantom] Fix P3 B1 silicon bug workaround

Commit f58cc3f introduced a temporary workaround for a bug in current
prototype silicon, but failed to apply it to all eight PCI functions
within the device.

11 years ago[util] Fix interpretation of short jumps in Option::ROM
Michael Brown [Wed, 27 Aug 2008 19:36:30 +0000 (20:36 +0100)] 
[util] Fix interpretation of short jumps in Option::ROM

Option::ROM was assuming that ROM images using a short jump
instruction for the init entry point would have a zero byte at offset
5; this is not necessarily true.

11 years ago[romprefix] Preserve %edi when issuing INT 1A,B101
Michael Brown [Tue, 26 Aug 2008 04:03:19 +0000 (05:03 +0100)] 
[romprefix] Preserve %edi when issuing INT 1A,B101

INT 1A,B101 (get PCI BIOS version) will overwrite %edi.

11 years ago[phantom] Remove comment that no longer matches reality
Michael Brown [Tue, 26 Aug 2008 03:45:31 +0000 (04:45 +0100)] 
[phantom] Remove comment that no longer matches reality

11 years ago[romprefix] Add more diagnostic messages to ROM prefix
Michael Brown [Mon, 25 Aug 2008 22:36:24 +0000 (23:36 +0100)] 
[romprefix] Add more diagnostic messages to ROM prefix

Include PMM allocation result in POST banner.

Include full product string in "starting execution" message.

Also mark ourselves as supporting DDIM in PnP header, for
completeness.

11 years ago[phantom] Temporary workaround for bug in prototype (P3 B1) silicon
Michael Brown [Mon, 25 Aug 2008 22:25:33 +0000 (23:25 +0100)] 
[phantom] Temporary workaround for bug in prototype (P3 B1) silicon

11 years ago[romprefix] Fix regression in UNDI loader
Michael Brown [Fri, 22 Aug 2008 22:58:35 +0000 (23:58 +0100)] 
[romprefix] Fix regression in UNDI loader

Commit 12f203c introduced a bug that caused the UNDI loader to attempt
to load the UNDI code and data segments to incorrect addresses.

11 years ago[romprefix] On a PCI3.0, non-BBS system, use the correct %cs for INT19
Michael Brown [Fri, 22 Aug 2008 02:01:46 +0000 (03:01 +0100)] 
[romprefix] On a PCI3.0, non-BBS system, use the correct %cs for INT19

On a system that doesn't support BBS, we end up hooking INT19 to gain
control of the boot process.  If the system is PCI3.0, we must take
care to use the runtime value for %cs, rather than the POST-time
value, otherwise we end up pointing INT19 to the temporary option ROM
POST scratch area.

11 years ago[elf] Add execution entry point debug messages for elf/multiboot images
Michael Brown [Fri, 22 Aug 2008 01:12:44 +0000 (02:12 +0100)] 
[elf] Add execution entry point debug messages for elf/multiboot images

11 years ago[pxe] Add extra debug messages to the PXE UDP API calls
Michael Brown [Thu, 21 Aug 2008 15:52:08 +0000 (16:52 +0100)] 
[pxe] Add extra debug messages to the PXE UDP API calls

11 years ago[pxe] Fix broadcast transmissions via PXENV_UDP_WRITE
Michael Brown [Thu, 21 Aug 2008 15:50:51 +0000 (16:50 +0100)] 
[pxe] Fix broadcast transmissions via PXENV_UDP_WRITE

pxenv_udp_write() needs to specify a net device, otherwise ipv4 is
unable to route broadcast transmissions.

11 years ago[undi] Fill in ProtType correctly in PXENV_UNDI_ISR
Michael Brown [Wed, 20 Aug 2008 02:21:37 +0000 (03:21 +0100)] 
[undi] Fill in ProtType correctly in PXENV_UNDI_ISR

Determine the network-layer packet type and fill it in for UNDI
clients.  This is required by some NBPs such as emBoot's winBoot/i.

This change requires refactoring the link-layer portions of the
gPXE netdevice API, so that it becomes possible to strip the
link-layer header without passing the packet up the network stack.

11 years ago[undi] Work around broken UNDI polling behaviour in winBoot/i
Michael Brown [Wed, 20 Aug 2008 01:02:51 +0000 (02:02 +0100)] 
[undi] Work around broken UNDI polling behaviour in winBoot/i

Some dumb NBPs (e.g. emBoot's winBoot/i) never call PXENV_UNDI_ISR
with FuncFlag=PXENV_UNDI_ISR_START; they just sit in a tight polling
loop merrily violating the PXE spec with repeated calls to
PXENV_UNDI_ISR_IN_PROCESS.  Force a extra calls to netdev_poll() to
cope with these out-of-spec clients.

11 years ago[smbios] Print SMBIOS version number in debugging messages
Michael Brown [Tue, 19 Aug 2008 21:01:38 +0000 (22:01 +0100)] 
[smbios] Print SMBIOS version number in debugging messages

11 years ago[pcbios] Support arbitrary splits of the e820 memory map
Michael Brown [Mon, 18 Aug 2008 06:14:27 +0000 (07:14 +0100)] 
[pcbios] Support arbitrary splits of the e820 memory map

Allow for an arbitrary number of splits of the system memory map via
INT 15,e820.

Features of the new map-mangling algorithm include:

  Supports random access to e820 map entries.

  Requires only sequential access support from the underlying e820
  map, even if our caller uses random access.

  Empty regions will always be stripped.

  Always terminates with %ebx=0, even if the underlying map terminates
  with CF=1.

  Allows for an arbitrary number of hidden regions, with underlying
  regions split into as many subregions as necessary.

Total size increase to achieve this is 193 bytes.

11 years ago[pcbios] Prepare for multiple splits of hidden e820 memory regions
Michael Brown [Mon, 18 Aug 2008 00:01:45 +0000 (01:01 +0100)] 
[pcbios] Prepare for multiple splits of hidden e820 memory regions

Define a list of N allowed memory regions, and split each underlying
e820 region into up to N subregions.  Strip resulting empty regions
out of the map, avoiding using the "return with CF set to strip last
empty region" trick, because it seems that bootmgr.exe in Win2k8 gets
upset if the memory map is terminated with CF set.

This is an intermediate checkin that defines a single allowed memory
region covering the entire 64-bit address space, and uses the existing
map-mangling code on top of the new region-splitting code.  This
sanitises the memory map to the point that Win2k8 is able to boot even
on a system that defines a final zero-length region at the 4GB mark.

I'm checking this in because it may be useful for future debugging
efforts to be able to run with the existing and known-working map
mangling code together with the map sanitisation capabilities of the
new map mangling code.

11 years ago[util] Allow Option::ROM to understand and modify initialisation entry point
Michael Brown [Fri, 15 Aug 2008 03:10:35 +0000 (04:10 +0100)] 
[util] Allow Option::ROM to understand and modify initialisation entry point

Add support for manipulating the jump instruction that forms the
option ROM initialisation entry point, so that mergerom.pl can treat
it just like other entry points.

Add support for merging the initialisation entry point (and IBM BOFM
table) to mergerom.pl; this is another slightly icky but unfortunately
necessary GPL vs. NDA workaround.  When mergerom.pl replaces an entry
point in the original ROM, it now fills in the corresponding entry
point in the merged ROM with the original value; this allows (for
example) a merged initialisation entry point to do some processing and
then jump back to the original entry point.

11 years ago[settings] Avoid overwriting the start of .text in fetch_string_setting()
Michael Brown [Thu, 14 Aug 2008 02:03:53 +0000 (03:03 +0100)] 
[settings] Avoid overwriting the start of .text in fetch_string_setting()

fetch_string_setting() was subtracting one from the length of the
to-be-NUL-terminated buffer in order to obtain the length of the
unterminated buffer to be passed to fetch_setting().  This works
extremely well unless the length of the to-be-NUL-terminated buffer is
zero, at which point we end up giving fetch_setting() a buffer of
length -1UL, thereby inviting it to overwrite as much memory as it
wants...

11 years ago[dhcp] Do not restrict minimum retry time for ProxyDHCPREQUEST
Michael Brown [Wed, 13 Aug 2008 20:51:53 +0000 (21:51 +0100)] 
[dhcp] Do not restrict minimum retry time for ProxyDHCPREQUEST

The ProxyDHCPREQUEST is a unicast packet, so the first request will
almost always be lost due to not having the IP address in the ARP
cache.  If the minimum retry time is set to one second (as per commit
ff2b6a5), then ProxyDHCP will time out and give up before managing to
successfully transmit a request.

The DHCP timers need to be reworked anyway, so this mild hack is
acceptable for now.

11 years ago[retry] Added configurable timeouts to retry timer
Andrew Schran [Tue, 12 Aug 2008 00:05:26 +0000 (01:05 +0100)] 
[retry] Added configurable timeouts to retry timer

New min_timeout and max_timeout fields in struct retry_timer allow
users of this timer to set their own desired minimum and maximum
timeouts, without being constrained to a single global minimum and
maximum. Users of the timer can still elect to use the default global
values by leaving the min_timeout and max_timeout fields as 0.

11 years ago[iSCSI] Add support for mutual CHAP
Michael Brown [Mon, 11 Aug 2008 02:12:38 +0000 (03:12 +0100)] 
[iSCSI] Add support for mutual CHAP

Allow initiator to verify target authentication using CHAP.

11 years ago[libc] Add missing __attribute__ (( format ( printf ) )) to ssnprintf()
Michael Brown [Mon, 11 Aug 2008 01:30:46 +0000 (02:30 +0100)] 
[libc] Add missing __attribute__ (( format ( printf ) )) to ssnprintf()

11 years ago[romprefix] Fix ROM image copy on PCI 3.0 BIOSes
Michael Brown [Tue, 5 Aug 2008 22:18:07 +0000 (23:18 +0100)] 
[romprefix] Fix ROM image copy on PCI 3.0 BIOSes

Add the missing instructions to the code path that I never tested due
to not having a PCI 3.0 system available.

(D'oh)

11 years ago[pxe] If no ProxyDHCPACK exists, use DHCPACK for the fake ProxyDHCPACK packet
Michael Brown [Sat, 2 Aug 2008 02:03:42 +0000 (03:03 +0100)] 
[pxe] If no ProxyDHCPACK exists, use DHCPACK for the fake ProxyDHCPACK packet

WinPE seems to have a bug that causes it to always use the TFTP server
IP address and filename from the ProxyDHCPACK packet, even if the
ProxyDHCPACK packet doesn't exist.  This causes it to end up
attempting to fetch a file such as

  tftp://0.0.0.0/bootmgr.exe

If we don't have a ProxyDHCPACK to use, we pretend that it was a copy
of the DHCPACK packet.  This works around the problem, and hopefully
won't surprise any NBPs.

11 years ago[dhcp] Work around a bug in Altiris RDP
Michael Brown [Fri, 1 Aug 2008 20:49:11 +0000 (21:49 +0100)] 
[dhcp] Work around a bug in Altiris RDP

Altiris erroneously cares about the ordering of DHCP options, and will
get confused if we don't construct them in the order it expects.

This is observed (so far) only when attempting to deploy 64-bit Win2k3.

11 years ago[virtio] Add legacy driver for KVM virtio-net
Stefan Hajnoczi [Fri, 1 Aug 2008 15:55:49 +0000 (16:55 +0100)] 
[virtio] Add legacy driver for KVM virtio-net

This patch adds support for the virtio-net adapter provided by KVM.

Written by Laurent Vivier <Laurent.Vivier@bull.net> for Etherboot.
Wrapped as legacy driver for gPXE by Stefan Hajnoczi
<stefanha@gmail.com>.

11 years ago[autoboot] Retain initial-slash (if present) when constructing TFTP URIs
Michael Brown [Thu, 31 Jul 2008 15:55:46 +0000 (16:55 +0100)] 
[autoboot] Retain initial-slash (if present) when constructing TFTP URIs

When we boot from a DHCP-supplied filename, we previously relied on
the fact that the current working URI is set to tftp://[next-server]/
in order to resolve the filename into a full tftp:// URI.  However,
this process will eliminate the distinction between filenames with and
without initial slashes:

 cwuri="tftp://10.0.0.1/" filename="vmlinuz"  => URI="tftp://10.0.0.1/vmlinuz"
 cwuri="tftp://10.0.0.1/" filename="/vmlinuz" => URI="tftp://10.0.0.1/vmlinuz"

This distinction is important for some TFTP servers.  We now
explicitly construct a string of the form

 "tftp://[next-server]/filename"

so that a filename with an initial slash will result in a URI
containing a double-slash, e.g.

 "tftp://10.0.0.1//vmlinuz"

The TFTP code always strips a single initial slash, and so ends up
presenting the correct path to the server.

URIs entered explicitly by users at the command line must include a
double slash if they want an initial slash presented to the TFTP
server:

  "kernel tftp://10.0.0.1/vmlinuz"  => filename="vmlinuz"
  "kernel tftp://10.0.0.1//vmlinuz" => filename="/vmlinuz"

11 years ago[cleanup] Remove obsolete README.cvs file
Michael Brown [Thu, 31 Jul 2008 15:24:16 +0000 (16:24 +0100)] 
[cleanup] Remove obsolete README.cvs file

11 years ago[util] Add mergerom.pl
Michael Brown [Thu, 31 Jul 2008 05:28:46 +0000 (06:28 +0100)] 
[util] Add mergerom.pl

This utility is required as a workaround for legal restrictions on
including GPL and non-GPL code within the same expansion ROM image.

While this is not encouraged, we are prepared to accept that
concatenation of ROM images and updating of the ROM header data
structures can be classed as "mere aggregation" within the terms of
the GPL.

If in any doubt, assume that you cannot include GPL and non-GPL code
within the same expansion ROM image.  Contact the Etherboot team for
clarification on your specific circumstances.