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

Conflicts:

com32/hdt/hdt-ata.c
com32/hdt/hdt-cli-pxe.c
com32/hdt/hdt-cli-vesa.c
com32/hdt/hdt-cli.c
com32/hdt/hdt-common.c
com32/hdt/hdt-menu-pci.c

VPD: Adding the Virtual Product Data detection

Impact: It's now possible to reach VPD's info

Calling vpd_decode() populate a VPD structure to display :
- VPD's base address
- Bios Build ID
- Bios Release Date
- Bios Version
- Box Serial Number
- Motherboard Serial Number
- Machine Type/Model
- Default Flash Filename

hdt: Fixing typo. Thx to dag.

Impact: Cosmetic

Typo error

hdt: Don't display unsupported modes

Impact: Prevent from displaying 0x0 modes

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

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

hdt: cursor pos must be uint

Impact: Prevent warning while comparing with strlen() & co.

Cursor pos is usually compared with string pos, so it must be an uint

hdt: Implement DEL behavior in cli mode

Impact: Users can now use DEL

DEL now deletes the next char

Merge branch 'for-erwan' of ssh://erwan@terminus.zytor.com/pub/git/users/pam/hdt-pierre into pam

hdt: Trim spaces in the parser

Impact: Fix potential bugs when extra spaces are present

It is now possible to enter:

'  set      fooo      bar          baz      '

which will be understood as

'set fooo bar baz'

The real benefit is for the autocompletion:

'set   <TAB>'

does work.

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

Merge branch 'for-erwan' of ssh://erwan@terminus.zytor.com/pub/git/users/pam/hdt-pierre into pam

hdt: Autocomplete modes

Impact: Add feature

Add modes in the list of potential matches when autocompleting.
For instance, 'px'<TAB> will autocomplete in 'pxe', which is an
alias for 'set mode pxe'.

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

hdt: Fix !!! BUG when exiting

Impact: Fix error messages

There is no cli_mode_descr for the exit mode. Add a condition not to
print a !!! BUG message if cli_mode_descr is NULL when exiting.

Misc.: fix messages. hdt_cli.mode is an integer, not a string.

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

hdt: Adding lib-ansi to manage ansi escape codes

Impact: code is easier to read

Basic implementation of ansi escape codes. This make code easier to
read.

hdt: skip_spaces is requires for completion

Impact: Completion works again

The previous commit broke the completion

hdt: renaming skipspace to remove_spaces

Impact: this function now remove trailing spaces too

We need to remove both begining & trailing spaces to insure a good match
while reading cli's input.

hdt: Don't print "Flags :" when there is no cpu flags

Impact: Nicer output

The cpu module used to print a "Flags :" even if there was nothing to
print (using Qemu, I had a third Flags line empty). Fixing it.

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

hdt: Fix cpu summary alignment

Impact: Nicer output

The fields in the cpu summary weren't aligned. Fixing it.

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

hdt: NULLify autocomplete_tail on destroy

Impact: Fix memory corruption

autocomplete_tail needs to be NULLed when freeing the list of
autocomplete tokens or autocomplete_add_token_to_list can get really
confused when starting a new list.

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

hdt: Add "menu" command

Impact: Switching back from CLI to menu is possible

The "menu" command allows the user to display the menu from the CLI.

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

hdt: Execute default callback only if no module is specified

Impact: 'show' in hdt shows the main summary

Previously, 'show bidule' would execute the default callback for the
list of show modules. This is confusing. Let's execute it only if no
module is specified.

Note: same behaviour for 'set'.

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

hdt: Add "?" alias for help

Impact: New alias

"?" is now mapped to help.

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

hdt: Add show_modes

Impact: help more verbose w.r.t. modes

show modes (in hdt mode) will list the availables modes.
help (in any mode) will also list them.

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

hdt: Add autocompletion support

Impact: current_mode is global; cleanups

Autocompletion is now supported.

Without anything on the line, hitting TAB at the prompt once will show the list
of available commands. Hitting TAB a second time will start cycling
through these commands, replacing one by one on the prompt.

If the start of a command is entered, e.g. 'cl', the list of possible
matches is narrowed ('cl' will only yield 'clear').

The same behaviour exists with modules. Having entered 'show' on the
command line, the list of possible show modules will be presented.
Same for narrow searches: 'show s' in the hdt mode would only yield
'show summary' and 'show syslinux'.

Misc.: cleanups, enhanced some messages.

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

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: Missing include for getkey

Impact: Fix the build

more_printf now uses the proper includes

hdt: prevent unless disk's menu

Impact: none

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

hdt: Converting printf to more_printf

Impact: Improving display management

more_printf is better to manage output as it ask users to press return
when the content is too long

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

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: Fixing modules count for hdt mode

Impact: "show hdt" command now works

We have 9 modules

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: 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: Execute hdt default callbacks in each mode if needed

Impact: One can switch modes without going back through hdt

Given a command `dmi> show foo', the ordering of callbacks is now
the following:
1/ execute module 'foo' of commands set 'show' of mode dmi
2/ execute default callback of commands set 'show' of mode dmi
3/ [NEW] execute module 'foo' of commands set 'show' of mode hdt

(2/ is executed if 1/ doesn't exist, 3/ is executed if 2/ doesn't exist
either, if 3/ doesn't exist an error is displayed).

This allows for instance to switch between modes without exiting first
to hdt.

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

hdt: Implement aliases

Impact: New alias feature; CLi backward compatible

`dmi' is now a shorthand for `set mode dmi' (idem for all modes). This
is performed via a new routine: expand_aliases.

It is now also possible to match aliases (e.g. `q', `quit') to a single
valid command (e.g. `exit'). The list needs to be populated manually and
is intented to be kept short. Eventually, the CLI will support
autocompletion that will help user experience.

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

hdt: Fix NULL dereference in core

Impact: Bug fixing; add debug info

There was a NULL dereference in the CLI core for modes that doesn't
implement a set of modules (show, set, ...).

Also, check the command name on the `command' variable, not the raw line.

Misc.: add debug info.

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

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: Fixing merge

Impact: Fix the build.

The previous merge seems to have gone wrong. Fixing it.

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

Merge branch 'master' into pierre

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

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.

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

Conflicts:

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

hdt: Final cleanups to switch to the new cli

Impact: Old cli code has been removed.

This patch ends the transition to the new framework, the entire old code
has been removed.

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

hdt: Make VESA module use the new framework

Impact: Cleanups in the VESA module.

The VESA module now uses the new framework.

Misc.: fix indentation in the VESA module.

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

hdt: Make PCI module use the new framework

Impact: Cleanups in the PCI module.

The PCI module now uses the new framework.

Misc.: fix indentation in the PCI module.

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

hdt: Make CPU module use the new framework

Impact: Cleanups in the CPU module.

The CPU module now uses the new framework.

Misc.: fix indentation in the CPU module.

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

hdt: Make KERNEL module use the new framework

Impact: Cleanups in the KERNEL module.

The KERNEL module now uses the new framework.

Misc.: fix indentation in the KERNEL module.

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

hdt: Make PXE module use the new framework

Impact: Cleanups in the PXE module.

The PXE module now uses the new framework.

Misc.: fix indentation in the PXE module.

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

hdt: Add default callback for the DMI show module

Impact: Extend the cli.

`show' in the dmi mode now lists the available modules.

Misc.: remove noisy printf.

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

hdt: Convert syslinux module to use the new framework

Impact: Add the concept of default_callback per set of modules

Syslinux only implements a default show command. Previously, this was
triggered by the `show list' command line. It is not really intuitive
and may be confusing (it has another meaning in the DMI module).

There is now the concept of default callback, one that would be
triggered when only `show', `set', ... is entered in the command line.
A common usecase may be that `show' triggers a summary display.

Misc.: small fix in the DMI module.

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

hdt: Hide show/set help when no module is defined

Impact: Help output improved.

Small improvement in the module help output: the module_descr structure can be
defined but without any module. Add this sanity check.

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

hdt: Implement show commands (hdt mode) with the new framework

Impact: Cleanups.

The summary of each mode (show <module> in hdt mode) is now implemented
using the new framework, as show modules of the hdt mode.

Misc.: cleanups.

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

hdt: Implement set_mode with the new framework; Cleanups

Impact: Cleanups, rename cli module structures.

set_mode now uses the new framework.

Add a new file: hdt-cli-hdt.c. hdt-cli.c will contain the core functions
for the cli, whereas hdt-cli-hdt.c will contain callbacks for the hdt
(default) mode.

Misc.: rename core structures to be more meaningful.

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

hdt: Cleanups in the DMI module

Impact: Remove compilation warnings, Cleanups, DMI CLI output changed.

Add __unused macro to remove compilation warnings. argc and argv are
needed for consistency sake in the cli, but are not always used.

Slightly change the output of `list' to be more precise. It was
confusing for the first time user to list the modules with 'help' but
then have an error message when a module wasn't available.

Remove unused help function (there is a dynamic global one).

Misc.: move dprintf to the header file.

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

hdt: Refactor help option; Cleanups

Impact: Help is usable in all modes, cleanups.

Help is now a generique helper that is shared by all modes.
It prints:
* default commands for the current mode
* the list of modules associated to show for the current mode
* default commands for hdt

For the last item, it doesn't print overwritten commands, e.g. if the
dmi module implements a generic 'list' command and there is a generic,
shared, 'list' command in the hdt mode, the former overwrites the
latter.

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

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

hdt: Adding an history

Impact: UP & Down keys can be used to reach the command history

UP & Down keys can be used to parse the history

vesacon: migrate disable fixes from ansicon

Copy the disabled fixes already done in ansicon from checkin
22cd4e378e3aafd402f43030941d6f6b300d0cea.

Document menu.c32 + console 0 fix.

ansicon: handle the disabled case properly

Even when disabled, we need to set the screen size, because the
downstream code depends on it.  Set it to 25x80 (maybe we should
swtich this is 24x80) in this case.

Furthermore, when disabled, we should return success == bytes
"written", not zero.

getc: move parseint_esdi out of the getint control flow

parseint_esdi was located in the middle of the getint -> parseint fall
through, with expectedly disastrous results.