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

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>

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

hdt: Implement exit with the new cli framework; Cleanups

Impact: Cleanups, add the concept of 1 word commands, make hdt_cli global

Add support for one token commands. Each mode can now have "default
commands", which are commands made of only one word.
Some of these are generic (exit, help, ...). They are stored in the
hdt mode structures. Only 'exit' is implemented so far.

Note: this is different from default callbacks for modules:

dmi> exit # default command (shared, stored into
# hdt structures)
dmi> show chassis # will execute the chassis module
# callback stored in the show modules
# list of the dmi mode
dmi> show # will execute the default callback for the
# show modules

The structure is the following:

|
----> mode A
| |
| ------> default_modules
| | |
| | ----> module default
| | |
| | ----> module a
| | |
| | ----> module b
| |
| ------> show_modules
| |
| ----> module default
| |
| ----> module c
| |
| ----> module d
| |
| ----> module e

If A is dmi, c is cpu, d memory, e chassis, etc and default (of
show_modules) displays a summary (command 'show'). a and b are single
commands ('foo', 'bar'). 'exit', 'help', ... fall into that category but are
defined in the hdt mode.

Note2: none of the modules list is mandatory. Most of the time actually,
the default_modules list will be empty.

Misc: Refactor the new framework by extracting logic into helpers.

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

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

Conflicts:

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

hdt: Fixing indentation

Impact: None.

spaces -> tabs so that I can merge easily.

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

hdt: Fixing backspace

Impact: Backspace now have the proper behaviour

Since we can move in the string, backspace have to behave differently.
Fixing style.

hdt: Adding append mode to cli

Impact: Users can now edit a line

Append mode is now the default mode
This features moving using LEFT,RIGHT,HOME & END keys.

hdt: ctrl+a or HOME key return to the beginning of line

Impact: Users can now uses ctrl+a or HOME to return at the beginning of
a line

Ctrl+a or home key is now implemented

hdt: Skip heading spaces

Impact: Allow user to put some space char before the real command

Before that patch, it was required not having any space before the
command. That's not so user friendly ;o)

hdt: Using a single cli structure to manage the input line

Impact: Makes the code easier to write

Using a single s_cli structure for managing the input line, the cursor
position and the prompt

hdt: Missing include for getkey

Impact: remove a warning

more_printf now uses the proper includes

hdt: more_printf must use getkey

Impact: Users can now press a key to continue

Since we switch to an ansi raw console, the fgets() used
to catch the return when we display to much lines didn't worked.
We now use the getkey function to grab __any__ key, which is better
then.

hdt: Rewrite DMI module using a new syntax for the CLI

Impact: Start refactoring the CLI.

In order to have an extensible CLI, the commands handling needs to be
refactored. This patch starts the migration by refactoring the DMI
module.

Note that the core of the new commands parser has been implemented.
Adapting other modules to use it simply requires to declare
new structures that maps the command string to the callback.

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

hdt: fixing one remaining display_prompt

Impact: make is now successfull ;o)

A display_prompt didn't get updated

hdt: Renaming display_prompt to reset_prompt

Impact: Fix buffer corruption in cli.

The rename occurs to match the reality.
We actually do more than displaying the prompt as we reset some vars.
One of this parameters where wrong too and could occurs corruption.

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

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

runkernel: further simplify the command-line parser

Further simplifications to the command-line parser, both from a code
size and a usability perspective.

Refactor command line parsing; support "quiet" option

Refactor the command line parser to be more of a general parser
instead of treating each option as an ad hoc feature.

Suppress the Loading... prompt if "quiet" is specified on the command
line.  Some messed-up people want it this way.

hdt: Rewrite DMI module using a new syntax for the CLI

Impact: Start refactoring the CLI.

In order to have an extensible CLI, the commands handling needs to be
refactored. This patch starts the migration by refactoring the DMI
module.

Note that the core of the new commands parser has been implemented.
Adapting other modules to use it simply requires to declare
new structures that maps the command string to the callback.

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

hdt: Reset buffer in cli

Impact: Fix buffer corruption in cli.

After displaying the line, cur_pos and cli_line need to be reset.

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

hdt: Handling CTRL+C

Impact: User can use the CTRL+C

Basic Ctrl+C implementation

hdt: cli now uses the ansi raw interface

Impact: cli is now managed char by char instead of line by line

This patch is a first step toward a new version of the cli.
Current code isn't scalable to manage a complex cli code.
First step is to manage input char by char.
It will now possible to implement :
- completion using tab,
- using special keys (F1-F12)
This patch implements F1 as "show help"

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

hdt: Adding vesa in available modes & help message

Impact: Fixes CLI behavior. show vesa is a valid command

"help" command didn't reported vesa as possible mode
"show vesa" command was also missing even if already implemented

hdt: putting some vesa function static

Impact: none

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

hdt: Making the build silent

Impact: Silence the build.

It is easier to detect warnings and errors if make doesn't print out the
commands before executing them.

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

*/Makefile: fixes for older versions of NASM

Deal with issues that made older versions of nasm:

a) prevent "make spotless" from running
b) generate bad dependencies (no final newline)

*/Makefile: use -include when including dependencies

This seems to have better behavior for "make spotless".

Revert "memdisk: memdisk16.asm needs %ifndef DEPEND around %include"

This reverts commit 8f3919c8f5dd37ccd3b7565d32ef1dac8b2522d5.
Wrong fix.

memdisk: memdisk16.asm needs %ifndef DEPEND around %include

Older versions of NASM puke if there is an %include of a non-existent
file during dependency generation.  Make it more forgiving.

memdisk: align .bss to a 16-byte boundary for better zeroing perf

Some CPUs really want a reasonable alignment for a rep;stosl, so give
it to them...

memdisk: zero the .bss section

It is just too confusing to have a .bss section that isn't zeroed, to
explicitly zero it.

memdisk: move the floppy disk geometry code in parallel with the MBR code

Move the floppy disk geometry generation code into parallel with the
MBR code, which pretty much mimics the previous behavior.

memdisk: don't key FAT vs MBR on the type of disk

We can't rely on the disk type to say if this is FAT vs MBR, since we
haven't determined that yet.  Thus, move the FAT test up before the
MBR test; both are heuristic, but the FAT test has much less
probability of false positives, and can be further tightened if
needed.

SYSLINUX: fix handling of .bss files for FAT12/FAT16

The size of the superblock is different for FAT12/FAT16 and FAT32.  We
should only copy the relevant portions of the superblock over.
Technically speaking, it is possible for FAT12/FAT16 to have an older,
smaller superblock as well (without the 0x29 boot signature); we don't
handle that at the moment.

Merge commit 'hdt/master'

memdisk: auto-detect large floppy geometry if it is FAT

If a "large floppy" image is formatted with a FAT filesystem, we can
use the headers in the FAT image to derive the geometry.  This is nice
and user-friendly, so do it that way.

isolinux: split MaxTransfer for HD and CD modes

The MaxTransfer for CD modes has to be smaller than for HD modes to
avoid 64K overrun (even though in theory it's permitted, we don't
trust it.)  Thus have separate variables for HD and CD modes.

hdt: Improving vesa output

Impact: Use proper words to be better understood by users

Cosmetic fix.

hdt: Adding missing install target

Impact: Allow the build process to complete properly

The install: target was missing and prevented the global make install to
end properly

hdt: Uploading missing vesa files

The vesa files were missing

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

hdt: Command-line arguments should be const

Impact: Fix compilation warnings, signatures changes.

argc and argv should be const.

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

hdt: Force cast when dereferencing menu->disk_sub_menu

Impact: Fix compilation warning.

A cast is necessary in compute_disks.

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

hdt: Fix irq references

Impact: Fix build compilation

d213082d76a24d3ba2e033416891a170b39c1ded moved irq to dev_info struct.
Make the changes accordingly.

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

cmd.c32: simplify by passing the unparsed command line

Instead of parsing and unparsing the command line, just pass the raw
command line the way we already have it.

hdt: Remove unused parameter in handle_vesa_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter.

Note: The file hdt-cli-vesa is missing for the tree. This change will need to
be propagated once the file is integrated.

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

hdt: find_argument should return a const char*

Impact: Fix compilation warning, signature change.

Handle command line arguments as const. This fixes a compilation
warning.

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

hdt: Honor syslinux_version const qualifier

Impact: Fix compilation warning

syslinux_version returns a const pointer to a syslinux_version struct.
This fixes a compilation warning.

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

hdt: Fix duplicate struct entry

Impact: fix compilation error

An entry was accidentally added twice in s_pxe causing the build to
fail.

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

hdt: Make compute_memory_module static

Impact: None.

compute_memory_module is only used in hdt-menu-dmi.c. Let's make it
static.

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

hdt: Make compute_pci_device static

Impact: None.

compute_pci_device is only used in hdt-menu-pci.c. Let's make it static.

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

hdt: Remove unused cli_mode parameter

Impact: Signatures changes.

Uniform the main_show_* signatures by removing the parameter cli_mode from
main_show_dmi. The call to do_exit has been removed (it was useless anyway).

This allowed us to remove the cli_mode parameter from handle_hdt_commands
as well.

This change fixes all compilation warnings related to the cli.

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

hdt: Make main_show_summary static

Impact: None.

main_show_summary is only used in hdt-cli.c. Let's make it static.

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

hdt: Remove unused parameters in start_cli_mode

Impact: Signature change.

argc and argv aren't used in start_cli_mode. This fixes 2 compilation
warnings.

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

hdt: Make show_cli_help static

Impact: None.

show_cli_help is only used in hdt-cli.c. Let's make it static.

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

hdt: Make handle_hdt_commands static

Impact: None.

handle_hdt_commands is only used in hdt-cli.c. Let's make it static.

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

hdt: Make set_mode static

Impact: None.

set_mode is only used in hdt-cli.c. Let's make it static.

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

hdt: Make pci_show static

Impact: None.

pci_show is only used in hdt-cli-pci.c. Let's make it static.

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

hdt: Remove unused parameter in handle_pci_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make show_pci_devices static

Impact: None.

show_pci_devices is only used in hdt-cli-pci.c. Let's make it static.

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

hdt: Make show_pci_help static

Impact: None.

show_pci_help is only used in hdt-cli-pci.c. Let's make it static.

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

hdt: Remove unused variables in main_show_pci

Impact: Cleanups.

This fixes 8 compilation warnings.

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

hdt: Make most dmi helpers static

Impact: None.

Most of the dmi helpers are only used in hdt-cli-dmi.c. Let's make them
static.

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

hdt: Remove unused parameter in handle_dmi_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make show_dmi_help static

Impact: None.

show_dmi_help is only used in hdt-cli-dmi.c. Let's make it static.

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

hdt: Remove unused parameter in main_show_cpu

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make show_cpu_help static

Impact: None.

show_cpu_help is only used in hdt-cli-cpu.c. Let's make it static.

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

hdt: Remove unused parameter in handle_cpu_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make cpu_show static

Impact: None.

cpu_show is only used in hdt-cli-cpu.c. Let's make it static.

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

hdt: Make pxe_show static

Impact: None.

pxe_show is only used in hdt-cli-pxe.c. Let's make it static.

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

hdt: Remove unused parameter in main_show_pxe

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Remove unused parameter in handle_pxe_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make show_pxe_help static

Impact: None.

show_pxe_help is only used in hdt-cli-pxe.c. Let's make it static.

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

hdt: Remove unused variables

Impact: Cleanups.

first_line and second_line are not used. This fixes compilation
warnings.

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

hdt: Remove unused variable

Impact: Cleanups.

i is not used.

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

hdt: Remove unused parameter in handle_kernel_commands

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Remove unused parameter in main_show_kernel

Impact: Signature change.

The cli mode information is already in the function name. No need to pass it
as a parameter. This fixes a compilation warning.

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

hdt: Make show_kernel_help static

Impact: None.

show_kernel_help is only used in hdt-cli-kernel.c. Let's make it static.

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