hdt: Fix partition display (menu)

Pass the correct disk structure when extracting partitioning
information.

Misc.: remove extra paren in output.

(Impact: fixes #2)

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

gpllib: Specify size of the edd structure in the buffer (disklib)

The first word of the result buffer should contain the size of the expected
result, as specified in the ATA/ATAPI Command set:

   The caller shall set this value to the maximum Result Buffer length, in bytes.

(Impact: fixes #1)

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

Merge commit 'hdt-erwan/master' into for-erwan

Conflicts:
com32/hdt/hdt-cli-dmi.c

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

hdt: Add e801 and 88 calls in the memory mode (CLI)

Enhance the new memory mode:

 * show e801: to show the memory layout reported by the
              INT 15, AX = 0xE801 call
 * show 88: to show the memory size (contiguous Kb above 1M)
            reported by the INT 0x15, AH = 0x88 call

Note: `show' now defaults to the DMI banks listing.

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

gllib: Add e801 and 88 memory detection

Add Bios calls INT 0x15, AX = 0xE801 and INT 0x15, AH = 0x88
to detect the memory. These can be used as a fallback when e820 is not
available.

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

gpllib: Fix ACPI 3.0 quirk (memory)

If 24 bytes are returned in the e820 map, the entry should be ignored
if bit 0 of the Extended Attributes is clear.

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

hdt: 0.3.2

Impact: new release

New 0.3.2 release

gpllib: Add copyright/license information (disklib)

The disklib is distributed under GPLv2.

Add copyright holders for borrowed code:

 * H. Peter Anvin/Intel Corporation (read/write BIOS calls)
 * Matt Domsch <Matt_Domsch@dell.com> (edd structures)

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

hdt: Add memory mode (CLI)

The memory mode will be used to dump the memory map
(via e820, e802, e801 and 88). Only `show e820' is implemented.

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

gpllib: Add detect_memory_e820 call

detect_memory_e820 populates an array of struct e820entry using
INT 15, AX = E820h.

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

hdt: removing useless clear screen on dmi> show memory

Impact: Improving display usage

No need to clearscreen for such display

gpllib: Add copyright/license information (disklib)

The disklib is distributed under GPLv2.

Add copyright holders for borrowed code:

 * H. Peter Anvin/Intel Corporation (read/write BIOS calls)
 * Matt Domsch <Matt_Domsch@dell.com> (edd structures)

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

cmenu: fixing getnumrows values

Impact: Allow users to use 25 lines instead of 24

Prior to that commit it was impossible to setup a 25 lines menu

hdt: Remove useless comments

Impact: code cleaning

Removing useless comments

hdt: 0.3.2-pre1

Impact: pre version

pre1

hdt: optmizing disk's menu

Impact: Improving visuals

Optimizing disk'screen to have an efficient display

hdt: updating disk's menu to match latest disk api

Impact: Menu mode now display every partition

Updating the menu to use the latest API.
The menu now display all detected partitions

hdt: detect_disks requires detect_vesa (FIXME)

Impact: Prevent empty strings while detecting disks

Calling detect_vesa before doing int13 make it works....

hdt: missing \n on disk detection

Impact: visual improvements

Fixing missing return while printing "DISKS: Detecting"

pci: prevent modules.alias to destroy modules.pcimap content

Impact: multiple get_module_name_from_* corrupt result

Prior to that commit, calling both get_module_name_from_pcimap and
get_module_name_from_alias were reporting unknown modules.

hdt: preventing mac address to be shown of all pci devices

Impact: Fixing wrong display of MAC Address

This commit fix a wrong behavior where the MAC address was displaying
for every pci device instead of only for the related pci device.

hdt: fix wrong kernel module display in menu mode

Impact: prevent wrong module display for some pci devices

The buffer wasn't clean and could contain some previous kernel modules.
This bug leads to display too many kernel modules for a single pci device

hdt: Adding modules.alias support

Impact: It is now possible to use modules.alias to search kernel
modules.

This commit rename modules= command line parameter to modules_pcimap=
It introduce modules_alias= parameter to override the default
"modules.alias" filename.
Both methods (pcimap & alias) are tried when detecting hardware.
A failure message will occurs only if both methods fails.

pci: Adding get_module_name_from_alias()

Impact: Users can now use modules.alias to find kernel modules

get_module_name_from_alias() parses modules.alias to find the
appropriate kernel modules required by the local pci devices.

hdt: Fix bug in module lookup (CLI)

Compare the exact length. The previous version allowed
`show diskkkkkk' to match `show disk'.

Using strcmp is safe here, both strings are null-terminated by
construction.

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

hdt: Add disk callbacks (CLI)

In the hdt mode, `show disk' outputs a summary for all disks
(C/H/S, EDD (version, size) and host/interface).

In the disk mode, `show' outputs the same as above, `show disks'
a full description for all disks (with partitions) and `show disk 0x80'
detailed information for one disk only (0x80 in that example).

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

hdt: Fix modules lookup to compare against full callback name (CLI)

Impact: remove auto-aliases feature but fix potential bug

Comparing the first letters of the supplied command to look for
the callback was interesting, since it added aliases automatically
(`show bi' matched `show bios'). Unfortunately, that caused some issues
in certain situation (`show disk' matching `show disks').

Removing the feature.

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

Merge commit 'hdt-erwan/master' into for-erwan

Merge branch 'disklib' into for-erwan

Conflicts:

com32/hdt/hdt-common.c

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

hdt: Make the disk mode use parse_partition_table() (CLI)

Impact: code refactoring

Leverage the new call parse_partition_table() in the disk CLI.

Misc.: enhance UI output, add space in the columns for up to 46 TB disks.

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

gpllib: Add parse_partition_table() to iterate through all partitions on a disk

Impact: expand gpllib disk API

parse_partition_table() can be used to execute a callback on each partition.

The signature of the callback is:

   void callback(struct driveinfo *drive_info,
                struct part_entry *ptab,
                struct part_entry *ptab_root,
                int offset_root,
                int local_partition_number,
                int ebr_seen)

 * drive_info represents the disk.
 * ptab is the current partition entry processed.
 * ptab_root is the partition entry linking to the current one processed (extended only).
 * offset_root is the offset of the ebr when iterating through extended partitions
   (0 otherwise).
 * local_partition_number is the number of the partition processed (0->3).
 * ebr_seen counts the total number of ebr processed.

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

hdt: Improve disk output (CLI)

Do not print EDD C/H/S info, since it is not really useful.
Add disk size to the summary.

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

hdt: Adding hdt> show vpd

Impact: Adding a missing CLI command

Prior to that commit, "hdt> show vpd" didn't worked.

hdt: Adding hotkey for VPD

Impact: VPD menu is now reachable via its "V" hotkey

Prior to that commit, VPD menu wasn't selectable using a direct keyboard key.

hdt: displaying only populated VPD' items

Impact: No need to display empty items

This commit improve VPD menu by not displaying empty items

hdt: 0.3.1

Impact: new release

New release

Merge commit 'mouraf/for-erwan' into for-erwan

Conflicts:

com32/hdt/hdt-cli-hdt.c

hdt: Do not print information about buggy extended partitions

Impact: bug fix

Add a sanity check before printing extended partitions.
As hpa notes in chain.c:

   Sanity check entry: must not extend outside the extended partition.
   This is necessary since some OSes put crap in some entries.

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

hdt: Do not modify offsets in the part_entry itself

Impact: fix bug

When printing extended partitions information, we need to adjust
the start lba. Do not modify it in the part_entry struct but just
print the adjusted one.

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

gpllib: Handle unknown labels (disk)

Return "Unknown" for unknown labels.

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

gpllib: Add default error handling (disk)

In case of an unknown error, display a default message.

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

hdt: Add missing header in hdt-util.c

Impact: fix build

Add missing #include <string.h> in hdt-util.c.

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

hdt: Check if a swsusp image exists on swap partitions

Impact: new feature, add error debug information

When printing partitions information, add a message in the CLI if a swap device
contains a valid swsusp (hibernate) image.

Misc.: check and print, if needed, errors after each I/O operation.

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

gpllib: Build new objects (disk)

Add error.o and swsusp.o as part of the gpllib build.

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

gpllib: Add swsusp_check to check for swsusp signature

swsusp is a Linux mechanism to put a computer to sleep. In case of hibernation,
an image is written to the disk, on a swap partition.

swsusp_check can be used to check if a device contains the magic swsusp signature,
meaning that an image is present.

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

gpllib: Propagate error codes in disk library

It is worth propagating read/write error codes back to the caller. He can
pass NULL to ignore them, or an int* that will be populated.

To decode it, one can use the get_error function (see disk/error.c).

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

gpllib: Move partition declarations to disk/partition.h

Add a new file for partition related structures.

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

gpllib: Move get_error from disk/util.c to disk/error.c

The get_error function is fairly big. It is worth creating
a new file for it.

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

gpllib: Add disk/common.h for shared definitions

A couple of structures/definitions are shared between several files
in the disk library. Add them in disk/common.h

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

hdt: Add IEC 60027 format for disk size (menu)

Impact: move the decimal format code to hdt-util.c

Display size in MiB/GiB/TiB as well in the disk menu.

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

hdt: Use IEC 60027 standard for capacity (CLI)

Impact: add hdt-util.c for generic helpers

Display size of partitions in kibi, mebi, gibi ot tebibytes.
There is no point in displaying the number of sectors in the CLI, a
human readable size is more useful.

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

hdt: Process extended partitions (CLI)

Print information about extended partitions in the CLI.

Note: the offsets reported are the offsets found in the EBR. We may want to
adjust it with the offset in the MBR.

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

hdt: Add partitions information in disk menu

Show information about valid partitions found (start, end, length, type, ...).

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

hdt: Dump partition tables (CLI)

Use the new disk API to read mbrs and dump information about detected
primary partitions.

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

gpllib: Add partition table structure (disk)

Impact: disk API extension

Add part_entry structure to describe DOS partition tables.

The new get_label call translates a label id to a meaningful string
describing the partition.

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

gpllib: Add read_mbr helper (disk)

Impact: disk API extension

read_mbr takes a drive number and returns a pointer to a malloced buffer
containing its mbr.

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

disk.c32: Fix indentation

Impact: none

Fix indentation for the loop ending curly brace.

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

hdt: Adapt disk CLI/menu to new struct devinfo

The struct devinfo has slightly changed. Reflect the changes in hdt.

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

gpllib: Don't clobber legacy C/H/S parameters with EDD ones

Impact: driveinfo structure change, new disk.c32 module

Adapt driveinfo structure to store both legacy and EDD parameters. Change utility
functions to use EDD when available (C/H/S being a fallback).

Add a new disk.c32 module to test the disk library. It will print information
(geometry) about every detected drive.

Misc.: refactoring, set ES:DI to 0:0 when querying legacy C/H/S

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

hdt: Fix typo in device_parameter struct

cheksum should read checksum.

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

hdt: Remove misguiding printf() (disks)

When detecting disks, we iterate through all ports. The number of disks
scanned will always be 127, no matter how many are actually plugged in.
Remove extra printf() when scanning.

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

hdt: Fix EDD format (menu)

The EDD version is more meaningful if displayed using an hexadecimal
format:

01h = 1.x
20h = 2.0 / EDD-1.0
21h = 2.1 / EDD-1.1
30h = EDD-3.0

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

hdt: Remove inappropriate free()

Impact: fix crash

This extra free() is not needed, as the error buffer is already freed in the
loop.

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

hdt: removing useless more_printf

Impact: improving user experience

hdt>show modes was using more_printf. This is pretty useless and
annoying for users.

hdt: Add `show disk' in hdt mode (CLI)

It is now possible to see disks from hdt mode, via the `show disk'
command.

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

Merge commit 'mouraf/for-erwan' into disklib

Conflicts:
com32/hdt/hdt-cli.c
com32/hdt/hdt-cli.h

hdt: Refactor disk menu

Make the disk menu use the new disk library.

Misc.: remove firmware, serial number and model information as they aren't
detected yet.

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

hdt: Add disk mode (CLI)

The disk mode features drives autodetection (geometry). Generic geometry
code from hdt-ata.c has been replaced by the new disk gpllib.

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

gpllib: Add disk library

This library features read and write operations, as well as geometry detection.
It will prevent duplicating code between several modules (e.g. hdt and chain.c32).

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

hdt: removing useless more_printf

Impact: none

more_printf isn't needed for such display

hdt: more_printf was eating a line

Impact: Visual. Some lines were missing at display

The more_printf call was missing some lines to display

hdt: MAC Address should only be displayed on the PXE Device

Impact: MAC address were shown on invalid devices

Prior to that commit, the mac address of the PXE booted network card was
displayed on every pci devices.... weird.

hdt: fixing typo

Impact: fixing compilation errors

Wrong call :(

hdt: Cleaning dmi chassis asset tag output

Impact: Visual

Some users reported this dmi/chassis/asset tag could reports many
spaces.

hdt: removing multiple spaces

Impact: Improving visual experience

Some hardware reported strings contains multiple spaces.
This isn't very nice for managing display. The del_multiple_spaces()
function is removing multiple spaces.

Merge commit 'origin/for-erwan' into disklib

hdt: moving contact adress to the mailing list

Impact: none

Mailing list is the best place to have user inputs

hdt: Add VPD menu item

Add an item to display VPD information, if detected.

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

hdt: Add vpd mode (CLI)

Add the vpd mode to dump the vpd structure, if found.

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

Merge commit 'erwan/master' into for-erwan

Conflicts:
com32/hdt/hdt-cli-hdt.c
com32/hdt/hdt-cli.c

hdt: Get rid of nb_modules in struct cli_module_descr

End list of modules commands by NULL instead of managing manually
the number in nb_modules. This should limit the number of bugs when adding new
features.

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

hdt: Get rid of MAX_MODES

End the list of modes by NULL instead of managing manually the length.

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

hdt: removing useless clear_screen

Impact: Improve user experience

Prevent clear screen when not necessary

hdt: Improving more_printf & clear_screen management

Impact: greatly improve user experience

This commit adds more coherency in more_printf & clear_screen usage

hdt: Fixing wrong printf calls

Impact: Removing memory corruption

Some printf called were badly setup

hdt: Adding reset_more_printf()

Impact: first step toward more_printf() rationalization

reset_more_printf() reset the counter used by more_printf to count
lines.

hdt: menu should be 80x25

Impact: increasing menu size to improve display

80x24 was a mistake, it have to be 80x25

Merge branch 'master' of git://git./boot/syslinux/syslinux into erwan

Merge branch 'syslinux-3.7x'

rosh: don't compile rosh.lnx by default

Don't compile rosh.lnx by default.  It causes unnecessary dependencies
on the host system.

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

NEWS: document HDT PCI fix

hdt: Fixing latency indentation

Impact: Improving display

Removing useless space

pci: Using for_each_pci_func3

Impact: cleaning code

Using for_each_pci_func3 instead of for_each_pci_func + pci_mkaddr

pci: add pci_for_each_func3() which produces an address

Add pci_for_each_func3() iterator, which produces an address in
addition to the device pointer.

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

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.

pci: Using for_each_pci_func3

Impact: cleaning code

Using for_each_pci_func3 instead of for_each_pci_func + pci_mkaddr

Merge branch 'master' of git://git./boot/syslinux/syslinux

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>