PCI: Fixing freeze when detecting many PCI devices

Impact: Prevent hard freeze when having many PCI devices

A user reported that calling gather_additional_pci_config() when many
PCI devices exists on a host could lead to a complete freeze.
This patch remove some useless code and use for_each_pci_func() instead.
This prevent useless memory allocation.

NEWS: add the latest stuff...

Document additional changes since 3.74.

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

gpxe: Don't use "lret $2" to return from an interrupt

Using "lret $2" to return from an interrupt causes interrupts to be
disabled in the calling program, since the INT instruction will have
disabled interrupts.  Instead, patch CF on the stack and use iret to
return.

Interestingly, the original PC BIOS had this bug in at least one
place.

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

menu: modify MENU SAVE to be a cascading default

Change MENU SAVE to be a cascading default that can be reverted with
MENU NOSAVE.

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

doc: fix stray parenthesis in syslinux.txt

Remove stay parenthesis in syslinux.txt

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

doc: update copyright in syslinux.txt

Update and clarify copyright in syslinux.txt.

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

NEWS: the extlinux(1) is from Brian Pellin, not Daniel Baumann.

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

NEWS: document MEMDISK fixes, updates.

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

Adding manpage from Brian Pellin <bpellin@gmail.com> for extlinux.

Correcting typo in extlinux documentation.

MEMDISK: traceability for the geometry

Print the source of the geometry determination.

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

MEMDISK: fix MBR geometry extraction

Fix the MBR geometry extraction; there was an off-by-2 error in the
MBR signature offset.

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

NEWS, version: bump version to 3.75, document keeppxe fix

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

pxelinux: fix the handling of KeepPXE

KeepPXE was broken in 3.74, because we tried to use both bits 0 and 1,
but failed to actually test both bits.  Instead, just use bit 0, but
move the place where we clear the bits.

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

gfxboot: parse DEFAULT keywork in syslinux config file

The gfxboot COM module currently ignores the DEFAULT entry set in
syslinux config file and always default to 0 instead. This patch parses
DEFAULT entries in the config file and set the default entry
accordingly.

Signed-off-by: Christophe Fergeau <cfergeau@mandriva.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

hdt: clear memtest_label

Clear memtest_label in init_hardware().

com32: the keyboard map size is in CX not in DX

Impact: fixes kbdmap.c32

The keyboard map size is passed in CX, not in DX.  Fix typo in
libcom32.

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

gfxboot module: fix copyright notice

Include copyright notices of reused code.

NEWS: document gfxboot.com

Gfxboot COMBOOT module

A COMBOOT module to interface to Steffen Winterfeldt's Gfxboot
graphical front end (http://gfxboot.sourceforge.net/).

Merge commit 'hdt/master'

Add back "install: all" to the com32 devel directories

"make install" in the com32 directory currently installs the devel
environment, which should be build using the host toolchain... so make
it so.

Makefiles: try to make "make install" do the right thing

One more attempt at getting semi-sanity to "make install".
Realistically, the install/netinstall/install-all/... needs to be
pushed into the com32 Makefiles, but this is at least a quick hack.

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

Implement MENU SAVE; fix COM32 setadv function

Implement MENU SAVE, which allows the menu system to retain the
previous selection from one boot to another.  In the process, fix the
syslinux_setadv() function.

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

core: unbreak "make depend", fix make spotless

cmdline.inc: Unbreak "make depend"
Makefile: fix "make spotless"

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

NEWS: document cmdline fix

ipappend: clean up ipappend code; add ipappend to plain cmdline

ipappend information is needed for the plain cmdline too, since it may
be used by linux.c32.  While we're at it, clean up the handling of
ipappend lines -- we can rather obviously use a loop for them.

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

hdt: Don't display unsupported modes

Impact: Prevent from displaying 0x0 modes

Some vesa bioses reports 0x0 modes, no need to display them

Provide a shared MIME file for the LSS16 image format

Add a MIME file for the LSS16 image format.  Currently only included
in source, as it is somewhat unclear what the proper way to package
this thing is.

memscan: protect 0-0x7c00 until the new shuffler is merged

Mark the memory range 0-0x7c00 unavailable until the new shuffler code
is ready, at which point we can go for a tighter bound.

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

Makefile: install new MBR variants; NEWS: document

Make the new MBR variants installables, and add to the NEWS file.

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

memdisk: fix comment syntax in memdisk.ld

Fix build failure due to incorrect comment syntax in memdisk.ld

Merge branch 'ctrl-mbr'

mbr: generate normal versions, force-80 versions, and ctrl-80 versions

Produce MBR variants with normal (DL) behavior, force to drive 80
behavior, and force to 80 on Ctrl pressed behavior.

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

Update copyright notices

Update copyright notices; add Intel copyright notices where
appropriate.

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

Update copyright notices; add Intel notices where appropriate

Update copyright notices.  Per agreement with my new employer, Intel
Corporation, add Intel copyright notices where appropriate.

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

Merge commit 'hdt/master'

linux.c32: don't relocate the real-mode code *above* 0x90000

Do not relocate the real-mode code above address 0x90000... that would
not be prudent.

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

linux.c32: make load_linux() support relocation

In the case where the preferred kernel locations are not available,
make load_linux() smart enough to be able to relocate the kernel if at
all possible.

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

com32: add _t to scan_memory_callback

Change scan_memory_callback to scan_memory_callback_t

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

com32: merge all memory map discovery to one file

Merge the memory map discovery for malloc and the memory map discovery
for memmap into one file that scans memory and invokes a callback.

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

hdt: Adding exit menu in menu mode

Impact: Users can now exit hdt :)

It's now possible to exit HDT properly. Thx to fcrozat report.

hdt: Fixing aligment in about menu

Impact: cosmetic

Fixing contact in about mode

hdt: Fixing author name

Impact: cosmetic

Fixing author name

NEWS: document new com32 memory management

com32: fix the argument order

Impact: bug fix

Fix the order of arguments to a com32 program.

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

com32: make memory beyond the core HighMem available to malloc

Impact: should deal with memory holes more gracefully

Right now, if we find a memory hole, we simply don't use the memory
beyond that point.  This makes it possible for com32 modules to
use that memory.  After this, we should be able to add relocation
support to allow loading when the memory at 1 MB is unavailable.

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

NEWS: fix typo

Banner: "and contributors" is just too long; try "et al"

Impact: aestetics

Change "and contributors" to "et al" so that the message actually fits
on one line.  This is unfortunate, but what can you do...

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

altmbr: an alternative MBR which ignores the active flag

Impact: new feature

Create an alternative MBR which takes a partition number in byte 439
instead of looking at the active flag.  This is useful when
dual-booting legacy operating systems which assume that their
particular partition must be the active partition.

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

mbr/*.S: use a symbolic constant for BIOS_kbdflags

Impact: cleanup

Use a symbolic constant for BIOS_kbdflags instead of open-coding the
address.

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

mbr/*.S: if Ctrl is pressed, force the drive number to 0x80

Impact: BIOS bug workaround, possible unexpected consequences?

Escape hatch for BIOSes which pass in garbage in DL: if the Ctrl key
is pressed during boot, force the drive number to 0x80.

Based on a patch by "TJ <ubuntu@tjworld.net>".

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

Fix the new kernel command-line parser

Impact: bug fix

We were incorrectly looking at [ds:di] instead of [es:di] for
verifying that a command was properly terminated.  Fix.

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

memdisk: Int13FuncsCnt is a constant, not a memory location

Impact: bug fix

Int13FuncsCnt is a compile-time constant, not a memory location.  This
is a rather important distinction.

pci: Rename get_module_name_from_pci_ids

Impact: none

Rename get_module_name_from_pci_ids to get_module_name_from_pcimap since
the module names are read from the modules.pcimap and not pci.ids file.

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

hdt: prevent unless disk's menu

Impact: none

No need to compute disk's menu if no disk are detected

hdt: Adding memtest= boot parameter

Impact: memtest=<label> allow users to give another label name for the
memtest call

Prior to that version, memtest label was hardcoded to "memtest".
Now, "memtest" is the default label but could be overrided by users.

hdt: Adding PXE information on the booted pci device

Impact: Display which pci device is the one used for PXE

Sync with cli's behavior

doc: Adding more initrd details

Signed-off-by: Gert Hulselmans (hulselmansgert@gmail.com)

doc: Fixing typo

Signed-off-by: Gert Hulselmans (hulselmansgert@gmail.com)

hdt: Updating author name

Impact: none

Made for testing purpose

hdt: bump version to 0.2.7

Impact: None

Bump to 0.2.7

Merge commit 'origin/repair'

hdt: in the cli, we have to Skip empty lines

Impact: empty lines were confusing history

History works better when we ignore empty lines

hdt: Workaround for PXE detection

Impact: The PCI device matching could fails in some cases

In some cases, trying to find the exact bus/dev/fun/vendor/product
fails reported by the pxe rom fails. In such case, that workaround will
only look at the vendor/product id

hdt: Fixing output of vesa software revision

Impact: Vesa's software revision was misdisplayed and could lead to
corruption

Displaying an int with %s isn't a good idea ;o)

hdt: Fix typo in vesa initialization

Impact: Bug fixing.

This caused some garbage to be displayed when detecting vesa.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: Pxe mode might generate corruption

Impact: Prevent some hard lock due to pxe detection

In some cases, we can't find the associated PCI device.
In such situation, we shouldn't try to display its properties as
pci_device is set to NULL.

hdt: Fixing crash on some optical drives

Impact: Prevent some hard crashes

On some optical drives, requesting int 13, func 0x48 make the system
crash. In fact, if the geometry isn't valid (optical drives), no need to
check for extended attributes

menu: Increasing the maximum number of menus

Impact: some hosts can generate many menus when used with hdt

In some cases, we can run out of menus

pci: Check if configuration type is PCI_CFG_NONE

Impact: prevent from grabbing configuration when PCI is PCI_CFG_NONE

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

pci: Remove "(void)cfgtype"

Impact: none

Remove useless "(void)cfgtype"
Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

pci modules: Check for pci_scan() == NULL.

Impact: we don't have to grab more details if pci_scan returns NULL

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

hdt: Check for pci_scan() == NULL.

Impact: we don't have to grab more details if pci_scan returns NULL

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

hdt: Adding IPMI detection

Impact: HDT can now reports if an IPMI base board exists

By using DMI, it's now possible to detect IPMI base boards
Bump to 0.2.6

dmi: Adding IPMI detection

Impact: IPMI base board can now be detected

Detecting IPMI via DMI structures

Merge commit 'origin/erwan'

hdt: in the cli, we have to Skip empty lines

Impact: empty lines were confusing history

History works better when we ignore empty lines

Merge branch 'erwan'

hdt: Workaround for PXE detection

Impact: The PCI device matching could fails in some cases

In some cases, trying to find the exact bus/dev/fun/vendor/product
fails reported by the pxe rom fails. In such case, that workaround will
only look at the vendor/product id

hdt: Fixing output of vesa software revision

Impact: Vesa's software revision was misdisplayed and could lead to
corruption

Displaying an int with %s isn't a good idea ;o)

hdt: Fix typo in vesa initialization

Impact: Bug fixing.

This caused some garbage to be displayed when detecting vesa.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: Fix typo in vesa initialization

Impact: Bug fixing.

This caused some garbage to be displayed when detecting vesa.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: Pxe mode might generate corruption

Impact: Prevent some hard lock due to pxe detection

In some cases, we can't find the associated PCI device.
In such situation, we shouldn't try to display its properties as
pci_device is set to NULL.

hdt: Fixing crash on some optical drives

Impact: Prevent some hard crashes

On some optical drives, requesting int 13, func 0x48 make the system
crash. In fact, if the geometry isn't valid (optical drives), no need to
check for extended attributes

menu: Increasing the maximum number of menus

Impact: some hosts can generate many menus when used with hdt

In some cases, we can run out of menus

pci: Check if configuration type is PCI_CFG_NONE

Impact: prevent from grabbing configuration when PCI is PCI_CFG_NONE

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

pci: Remove "(void)cfgtype"

Impact: none

Remove useless "(void)cfgtype"
Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

pci modules: Check for pci_scan() == NULL.

Impact: we don't have to grab more details if pci_scan returns NULL

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

hdt: Check for pci_scan() == NULL.

Impact: we don't have to grab more details if pci_scan returns NULL

Signed-off-by: Sebastian Herbszt (herbszt@gmx.de)

hdt: Adding IPMI detection

Impact: HDT can now reports if an IPMI base board exists

By using DMI, it's now possible to detect IPMI base boards
Bump to 0.2.6

dmi: Adding IPMI detection

Impact: IPMI base board can now be detected

Detecting IPMI via DMI structures

Add "and contributors" to the copyright banner

Clarify that there are nowadays more copyright holders than just
myself.

memdisk: additional comments in memdisk.inc and postprocess.pl

Just a sprinkling of comments adding further description to the code.

vesacon: remove unused com32sys_t variables

No more direct __intcall()'s in this code, so remove unused register
set variables.

Merge branch 'memdisk-cd'

Merge commit 'hdt/master'

hdt: Adding contributors

Impact: A new cli command : hdt> show hdt

Bump to 0.2.5
Adding contributor's list

hdt: commenting some devel messages

Impact: remove useless error messages

Some error messages were displayed even if commands were valid

hdt: Fixing more_printf to display a proper \n

Impact: Improve the visibility of the message

more_printf could have printed that message without any \n