12 years ago[settings] Ensure fetch_string_setting() returns a NUL-terminated string
Michael Brown [Fri, 7 Nov 2008 03:46:21 +0000 (03:46 +0000)]
[settings] Ensure fetch_string_setting() returns a NUL-terminated string

This fixes a regression introduced in commit 612f4e7:

  [settings] Avoid returning uninitialised data on error in fetch_xxx_setting()

in which the memset() was moved from fetch_string_setting() to
fetch_setting(), in order that it would be useful for non-string
setting types.  However, this neglects to take into account the fact
that fetch_string_setting() shrinks its buffer by one byte (to allow
for the NUL) before calling fetch_setting().

Restore the memset() in fetch_string_setting(), so that the
terminating NUL is guaranteed to actually be a NUL.

12 years ago[i386] Add data32 prefixes to all lgdt/lidt instructions
Michael Brown [Thu, 6 Nov 2008 23:08:10 +0000 (23:08 +0000)]
[i386] Add data32 prefixes to all lgdt/lidt instructions

With a 16-bit operand, lgdt/lidt will load only a 24-bit base address,
ignoring the high-order bits.  This meant that we could fail to fully
restore the GDT across a call into gPXE, if the GDT happened to be
located above the 16MB mark.

Not all of our lgdt/lidt instructions require a data32 prefix (for
example, reloading the real-mode IDT can never require a 32-bit base
address), but by adding them everywhere we will hopefully not forget
the necessary ones in future.

12 years ago[phantom] Allow for PXE boot to be enabled/disabled on a per-port basis
Michael Brown [Sat, 1 Nov 2008 01:55:13 +0000 (01:55 +0000)]
[phantom] Allow for PXE boot to be enabled/disabled on a per-port basis

This is something of an ugly hack to accommodate an OEM requirement.
The NIC has only one expansion ROM BAR, rather than one per port.  To
allow individual ports to be selectively enabled/disabled for PXE boot
(as required), we must therefore leave the expansion ROM always
enabled, and place the per-port enable/disable logic within the gPXE

12 years ago[romprefix] Add vendor branding facilities and guidelines
Michael Brown [Fri, 31 Oct 2008 19:10:28 +0000 (19:10 +0000)]
[romprefix] Add vendor branding facilities and guidelines

Some hardware vendors have been known to remove all gPXE-related
branding from ROMs that they build.  While this is not prohibited by
the GPL, it is a little impolite.

Add a facility for adding branding messages via two #defines
(PRODUCT_NAME and PRODUCT_SHORT_NAME) in config/general.h.  This
should accommodate all known OEM-mandated branding requirements.
Vendors with branding requirements that cannot be satisfied by using
PRODUCT_NAME and/or PRODUCT_SHORT_NAME should contact us so that we
can extended this facility as necessary.

12 years ago[phantom] Assume one net device per PCI function
Michael Brown [Fri, 31 Oct 2008 01:52:55 +0000 (01:52 +0000)]
[phantom] Assume one net device per PCI function

The Phantom firmware selectively disables PCI functions based on the
board type, with the end result that we see one PCI function for each
network port.  This allows us to eliminate the code for reading from
flash and, more importantly, removes knowledge of the board type magic
number from the gPXE driver.

12 years ago[hacks] Improve the guess_boot_netdev() logic
Michael Brown [Fri, 31 Oct 2008 01:51:26 +0000 (01:51 +0000)]
[hacks] Improve the guess_boot_netdev() logic

This function is a major kludge, but can be made slightly more
accurate by ignoring net devices that aren't open.  Eventually it
needs to be removed entirely.

12 years ago[settings] Add the notion of a "tag magic" to numbered settings
Michael Brown [Wed, 29 Oct 2008 18:17:02 +0000 (18:17 +0000)]
[settings] Add the notion of a "tag magic" to numbered settings

Settings can be constructed using a dotted-decimal notation, to allow
for access to unnamed settings.  The default interpretation is as a
DHCP option number (with encapsulated options represented as
"<encapsulating option>.<encapsulated option>".

In several contexts (e.g. SMBIOS, Phantom CLP), it is useful to
interpret the dotted-decimal notation as referring to non-DHCP
options.  In this case, it becomes necessary for these contexts to
ignore standard DHCP options, otherwise we end up trying to, for
example, retrieve the boot filename from SMBIOS.

Allow settings blocks to specify a "tag magic".  When dotted-decimal
notation is used to construct a setting, the tag magic value of the
originating settings block will be ORed in to the tag number.
Store/fetch methods can then check for the magic number before
interpreting arbitrarily-numbered settings.

12 years ago[romprefix] Further sanity checks for the PCI 3 runtime segment address
Michael Brown [Wed, 29 Oct 2008 01:10:33 +0000 (01:10 +0000)]
[romprefix] Further sanity checks for the PCI 3 runtime segment address

This extends the sanity checks on the runtime segment address provided
in %bx, first implemented in commit 5600955.

We now allow the ROM to be placed anywhere above a000:0000 (rather
than c000:0000, as before), since this is the region allowed by the
PCI 3 spec.  If the BIOS asks us to place the runtime image such that
it would overlap with the init-time image (which is explicitly
prohibited by the PCI 3 spec), then we assume that the BIOS is faulty
and ignore the provided runtime segment address.

Testing on a SuperMicro BIOS providing overlapping segment addresses
shows that ignoring the provided runtime segment address is safe to do
in these circumstances.

12 years ago[phantom] Add CLP settings interface
Michael Brown [Tue, 28 Oct 2008 18:49:58 +0000 (18:49 +0000)]
[phantom] Add CLP settings interface

This interface provides access to firmware settings (e.g. MAC address)
that will apply to all drivers loaded for the duration of the current
system boot.

12 years ago[phantom] Unhalt/halt all PEGs during driver startup/shutdown
Michael Brown [Fri, 24 Oct 2008 02:49:11 +0000 (03:49 +0100)]
[phantom] Unhalt/halt all PEGs during driver startup/shutdown

A hardware bug means that reads through the expansion ROM BAR can
return corrupted data if the PEGs are running.  This breaks platforms
that re-read the expansion ROM after invoking gPXE code, such as IBM
blade servers.

Halt PEGs during driver shutdown, and unhalt PEGs during driver
startup if we detect that this is not the first startup since

12 years ago[phantom] Add line count limit to phantom_dmesg()
Michael Brown [Fri, 24 Oct 2008 20:41:58 +0000 (21:41 +0100)]
[phantom] Add line count limit to phantom_dmesg()

The Phantom firmware debug logs can be extremely long, so add the
ability to restrict the output to the last N lines.

12 years ago[ioapi] Fix broken implementation of insX() in the x86 I/O API
Michael Brown [Mon, 27 Oct 2008 00:29:16 +0000 (00:29 +0000)]
[ioapi] Fix broken implementation of insX() in the x86 I/O API

12 years ago[uri] Avoid interpreting DOS-style path names as opaque URIs
Michael Brown [Fri, 24 Oct 2008 03:08:43 +0000 (04:08 +0100)]
[uri] Avoid interpreting DOS-style path names as opaque URIs

A DOS-style full path name such as "C:\Program Files\tftpboot\nbp.0"
satisfies the syntax requirements for a URI with a scheme of "C" and
an opaque portion of "\Program Files\tftpboot\nbp.0".

Add a check in parse_uri() to ignore schemes that are apparently only
a single character long; this avoids interpreting DOS-style paths in
this way, and shouldn't affect any practical URI scheme.

12 years ago[phantom] Change register space abstraction to match other drivers
Michael Brown [Thu, 23 Oct 2008 22:35:01 +0000 (23:35 +0100)]
[phantom] Change register space abstraction to match other drivers

Most other Phantom drivers define a register space in terms of a 64M
virtual address space.  While this doesn't map in any meaningful way
to the actual addresses used on the latest cards, it makes maintenance
easier if we do the same.

12 years ago[pcbios] Guard against register corruption in INT 15,e820 implementations
Michael Brown [Thu, 23 Oct 2008 03:16:31 +0000 (04:16 +0100)]
[pcbios] Guard against register corruption in INT 15,e820 implementations

Someone at Dell must have a full-time job designing ways to screw up
implementations of INT 15,e820.  This latest gem is courtesy of a Dell
Xanadu system, which arbitrarily decides to obliterate the contents of

Preserve %esi, %edi and %ebp across calls to INT 15,e820, in case
someone tries a variation on this trick in future.

12 years ago[settings] Avoid returning uninitialised data on error in fetch_xxx_setting()
Michael Brown [Wed, 22 Oct 2008 20:18:15 +0000 (21:18 +0100)]
[settings] Avoid returning uninitialised data on error in fetch_xxx_setting()

Callers (e.g. usr/autoboot.c) may not check the return values from
fetch_xxx_setting(), assuming that in error cases the returned setting
value will be "empty" (for some sensible value of "empty").

In particular, if the DHCP server did not specify a next-server
address, this would result in gPXE using uninitialised data for the
TFTP server IP address.

12 years ago[tftp] Add EUNIQ_xx values to differentiate the many EINVAL errors
Michael Brown [Tue, 21 Oct 2008 22:00:13 +0000 (23:00 +0100)]
[tftp] Add EUNIQ_xx values to differentiate the many EINVAL errors

12 years ago[tg3] Add PCI ID for BCM5786
Michael Brown [Tue, 21 Oct 2008 21:31:10 +0000 (22:31 +0100)]
[tg3] Add PCI ID for BCM5786

Contributed by Steve Cayford <>

12 years ago[contrib] Add patch to enable writable ROM images in bochs
Michael Brown [Tue, 21 Oct 2008 19:49:10 +0000 (20:49 +0100)]
[contrib] Add patch to enable writable ROM images in bochs

Some of the gPXE romprefix.S code attempts to write to the ROM image,
as allowed by the PCI spec.  We do gracefully handle the case of
read-only ROM images (for ISA ROMs and for normal bochs/qemu
operation), but it can be handy to be able to use bochs to debug the
code paths that depend on writable ROM images.

12 years ago[build] Fix building on FreeBSD
Michael Brown [Fri, 17 Oct 2008 17:24:04 +0000 (18:24 +0100)]
[build] Fix building on FreeBSD

FreeBSD requires the object format to be specified as elf_i386_fbsd,
rather than elf_i386.

Based on a patch from Eygene Ryabinkin <>

12 years ago[romprefix] Sanity-check the runtime segment address for PCI 3
Michael Brown [Fri, 17 Oct 2008 17:12:24 +0000 (18:12 +0100)]
[romprefix] Sanity-check the runtime segment address for PCI 3

Some PCI 3 BIOSes seem to provide a garbage value in %bx, which should
contain the runtime segment address.  Perform a basic sanity check: we
reject the segment if it is below the start of option ROM space.  If
the sanity check fails, we assume that the BIOS was not expecting us
to be a PCI 3 ROM, and we just leave our image in situ.

12 years ago[build] Use ".bss.*" names for uninitialised-data sections
Michael Brown [Fri, 17 Oct 2008 02:12:11 +0000 (03:12 +0100)]
[build] Use ".bss.*" names for uninitialised-data sections

The section name seems to have significance for some versions of

There is no way to instruct gcc that sections such as .bss16 contain
uninitialised data; it will emit them with contents explicitly set to
zero.  We therefore have to rely on the linker script to force these
sections to become uninitialised-data sections.  We do this by marking
them as NOLOAD; this seems to be the closest semantic equivalent in the
linker script language.

However, this gets ignored by some versions of ld (including 2.17 as
shipped with Debian Etch), which mark the resulting sections with
(CONTENTS,ALLOC,LOAD,DATA).  Combined with the fact that this version of
ld seems to ignore the specified LMA for these sections, this means that
they end up overlapping other sections, and so parts of .prefix (for
example) get obliterated by .data16's bss section.

Rename the .bss sections from .section_bss to .bss.section; this seems to
cause these versions of ld to treat them as uninitialised data.

12 years ago[build] Fix building on Ubuntu 8.04
Michael Brown [Fri, 17 Oct 2008 00:39:48 +0000 (20:39 -0400)]
[build] Fix building on Ubuntu 8.04

Not fully understood, but it seems that the LMA of bss sections matters
for some newer binutils builds.  Force all bss sections to have an LMA
at the end of the file, so that they don't interfere with other

The symptom was that objcopy -O binary -j .zinfo would extract the
.zinfo section from bin/xxx.tmp as a blob of the correct length, but
with zero contents.  This would then cause the [ZBIN] stage of the
build to fail.

Also explicitly state that .zinfo(.*) sections have @progbits, in case
some future assembler or linker variant decides to omit them.

12 years ago[etherfabric] Merge changes from vendor tree
Michael Brown [Thu, 16 Oct 2008 16:21:48 +0000 (17:21 +0100)]
[etherfabric] Merge changes from vendor tree

Also clean up minor errors in the use of debug macros.

12 years agovirtio-net: Fix kick/wait logic
Eduardo Habkost [Wed, 8 Oct 2008 20:02:33 +0000 (17:02 -0300)]
virtio-net: Fix kick/wait logic

The virtnet_transmit() logic for waiting the packet to be transmitted is
reversed: we can't wait the packet to be transmitted if we didn't kick()
the ring yet. The vring_more_used() while loop logic is reversed also,
that explains why the code works today.

The current code risks trying to free a buffer from the used ring
when none was available, that will happen most times because KVM
doesn't handle the packet immediately on kick(). Luckily it was working
because it was unlikely to have a buffer still queued for transmit when
virtnet_transmit() was called.

Also, adds a BUG_ON() to vring_get_buf(), to catch cases where we try
to free a buffer from the used ring when there was none available.

Patch for Etherboot. gPXE has the same problem on the code, but I hadn't
a chance to test gPXE using virtio-net yet.

Signed-off-by: Eduardo Habkost <>
Signed-off-by: Stefan Hajnoczi <>
12 years ago[efi] Add basic implementation of EFI SIMPLE_NETWORK_PROTOCOL
Michael Brown [Thu, 16 Oct 2008 04:38:23 +0000 (05:38 +0100)]
[efi] Add basic implementation of EFI SIMPLE_NETWORK_PROTOCOL

This implementation is tested and working, but not currently tied in
to the EFI build.

12 years ago[netdevice] Change link-layer push() and pull() methods to take raw types
Michael Brown [Wed, 15 Oct 2008 03:17:48 +0000 (04:17 +0100)]
[netdevice] Change link-layer push() and pull() methods to take raw types

EFI requires us to be able to specify the source address for
individual transmitted packets, and to be able to extract the
destination address on received packets.

Take advantage of this to rationalise the push() and pull() methods so
that push() takes a (dest,source,proto) tuple and pull() returns a
(dest,source,proto) tuple.

12 years ago[netdevice] Split multicast hashing out into an mc_hash method
Michael Brown [Wed, 15 Oct 2008 01:00:44 +0000 (02:00 +0100)]
[netdevice] Split multicast hashing out into an mc_hash method

Multicast hashing is an ugly overlap between network and link layers.
EFI requires us to provide access to this functionality, so move it
out of ipv4.c and expose it as a method of the link layer.

12 years ago[netdevice] Add maximum packet length as a net device property
Michael Brown [Tue, 14 Oct 2008 23:49:59 +0000 (00:49 +0100)]
[netdevice] Add maximum packet length as a net device property

Currently this length is set at device allocation time, and is never

12 years ago[efi] Fix linker script for ld
Michael Brown [Mon, 13 Oct 2008 09:41:38 +0000 (10:41 +0100)]
[efi] Fix linker script for ld

Some versions of ld choke on the "AT ( _xxx_lma )" in with an
error saying "nonconstant expression for load base".  Since these were
only explicitly setting the LMA to the address that it would have had
anyway, they can be safely omitted.

12 years ago[efi] Add EFI image format and basic runtime environment
Michael Brown [Sun, 12 Oct 2008 00:55:55 +0000 (01:55 +0100)]
[efi] Add EFI image format and basic runtime environment

We have EFI APIs for CPU I/O, PCI I/O, timers, console I/O, user
access and user memory allocation.

EFI executables are created using the vanilla GNU toolchain, with the
EXE header handcrafted in assembly and relocations generated by a
custom efilink utility.

12 years ago[sanboot] Quick and dirty hack to make SAN boot protocols selectable
Michael Brown [Mon, 13 Oct 2008 09:05:23 +0000 (10:05 +0100)]
[sanboot] Quick and dirty hack to make SAN boot protocols selectable

12 years ago[config] Make the default image type selection platform-dependent
Michael Brown [Mon, 13 Oct 2008 08:37:51 +0000 (09:37 +0100)]
[config] Make the default image type selection platform-dependent

12 years ago[monojob] Release reference on completed job
Michael Brown [Mon, 13 Oct 2008 04:30:04 +0000 (05:30 +0100)]
[monojob] Release reference on completed job

monojob_wait() was holding a reference to the completed job, meaning that
various objects would not be freed until the next job was plugged in to
the monojob interface.

12 years ago[umalloc] Formalise the user memory allocation API
Michael Brown [Mon, 13 Oct 2008 03:32:11 +0000 (04:32 +0100)]
[umalloc] Formalise the user memory allocation API

12 years ago[uaccess] Formalise the uaccess API
Michael Brown [Mon, 13 Oct 2008 02:47:33 +0000 (03:47 +0100)]
[uaccess] Formalise the uaccess API

The userptr_t is now the fundamental type that gets used for conversions.
For example, virt_to_phys() is implemented in terms of virt_to_user() and

12 years ago[timer] Remove now-obsolete references to TIMER_BIOS and TIMER_RDTSC
Michael Brown [Sun, 12 Oct 2008 23:23:18 +0000 (00:23 +0100)]
[timer] Remove now-obsolete references to TIMER_BIOS and TIMER_RDTSC

12 years ago[nap] Formalise the CPU sleeping API
Michael Brown [Sun, 12 Oct 2008 22:34:45 +0000 (23:34 +0100)]
[nap] Formalise the CPU sleeping API

12 years ago[monojob] Allow for extremely slow system timers
Michael Brown [Sun, 12 Oct 2008 20:27:57 +0000 (21:27 +0100)]
[monojob] Allow for extremely slow system timers

The EFI timer runs at one tick per second, so using ">" rather than ">="
results in a two-second gap between dots.

12 years ago[timer] Formalise the timer API
Michael Brown [Sun, 12 Oct 2008 18:56:52 +0000 (19:56 +0100)]
[timer] Formalise the timer API

We now have two implementations for the timer API: one using the
time-of-day counter at 40:70 and one using RDTSC.  Both make use of

12 years ago[process] Add DBG2() messages to help track down frozen processes
Michael Brown [Sun, 12 Oct 2008 14:06:07 +0000 (15:06 +0100)]
[process] Add DBG2() messages to help track down frozen processes

12 years ago[retry] Use a separate flag to indicate that a retry timer is running
Michael Brown [Sun, 12 Oct 2008 14:04:37 +0000 (15:04 +0100)]
[retry] Use a separate flag to indicate that a retry timer is running

Using start==0 to indicate a stopped timer is dangerous, because 0 is a
valid value for the current tick counter.

12 years ago[commands] Fix config command to accept zero arguments
Michael Brown [Sun, 12 Oct 2008 13:37:51 +0000 (14:37 +0100)]
[commands] Fix config command to accept zero arguments

12 years ago[pci] Formalise the PCI I/O API
Michael Brown [Sun, 12 Oct 2008 11:50:44 +0000 (12:50 +0100)]
[pci] Formalise the PCI I/O API

12 years ago[legacy] Add missing #include <gpxe/io.h>
Michael Brown [Sun, 12 Oct 2008 11:37:25 +0000 (12:37 +0100)]
[legacy] Add missing #include <gpxe/io.h>

12 years ago[phantom] Add missing #include <gpxe/io.h>
Michael Brown [Sun, 12 Oct 2008 11:37:08 +0000 (12:37 +0100)]
[phantom] Add missing #include <gpxe/io.h>

12 years ago[config] Split console configuration out to config/console.h
Michael Brown [Sun, 12 Oct 2008 01:30:31 +0000 (02:30 +0100)]
[config] Split console configuration out to config/console.h

12 years ago[ioapi] Absorb virt_to_phys() and phys_to_virt() into the I/O API
Michael Brown [Sun, 12 Oct 2008 01:11:50 +0000 (02:11 +0100)]
[ioapi] Absorb virt_to_phys() and phys_to_virt() into the I/O API

12 years ago[i386] Remove obsolete functions from virtaddr.h
Michael Brown [Sun, 12 Oct 2008 01:01:18 +0000 (02:01 +0100)]
[i386] Remove obsolete functions from virtaddr.h

The copy_{to,from}_phys() functions were obsoleted long ago by

relocate_to() also disappeared some time ago.

12 years ago[rtl8139] Add missing #include <string.h>
Michael Brown [Sun, 12 Oct 2008 00:59:57 +0000 (01:59 +0100)]
[rtl8139] Add missing #include <string.h>

12 years ago[natsemi] Add missing #include <string.h>
Michael Brown [Sun, 12 Oct 2008 00:59:49 +0000 (01:59 +0100)]
[natsemi] Add missing #include <string.h>

12 years ago[e1000] Add missing #include <string.h>
Michael Brown [Sun, 12 Oct 2008 00:59:34 +0000 (01:59 +0100)]
[e1000] Add missing #include <string.h>

12 years ago[ioapi] Remove old io.h file and switch all users over to <gpxe/io.h>
Michael Brown [Sat, 11 Oct 2008 23:52:30 +0000 (00:52 +0100)]
[ioapi] Remove old io.h file and switch all users over to <gpxe/io.h>

12 years ago[ioapi] Formalise the I/O API as used in i386-pcbios
Michael Brown [Sun, 12 Oct 2008 00:03:17 +0000 (01:03 +0100)]
[ioapi] Formalise the I/O API as used in i386-pcbios

12 years ago[cs89x0] Simplify obscure loop syntax
Michael Brown [Sat, 11 Oct 2008 23:39:57 +0000 (00:39 +0100)]
[cs89x0] Simplify obscure loop syntax

12 years ago[e1000] Fix dubious syntax in e1000 I/O wrapper macros
Michael Brown [Sat, 11 Oct 2008 23:38:56 +0000 (00:38 +0100)]
[e1000] Fix dubious syntax in e1000 I/O wrapper macros

12 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

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

Inspired by a patch from Carl Karsten <> and an
identical patch from Rorschach <>.

12 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 script in

Note that 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

12 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.

12 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

12 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

12 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

12 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

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

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

12 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

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

12 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.

12 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

12 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.

12 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

12 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>

12 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.

12 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.

12 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.

12 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

12 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

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

12 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()

12 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()

12 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 <>
12 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 <>
12 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 <>

12 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.

12 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.

12 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.

12 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.

12 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.

12 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.

12 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

12 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+

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

12 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()

12 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

12 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

12 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).

12 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.

12 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.

12 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.

12 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"

12 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

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