Merge commit 'hdt-0.3.5-pre2' into hdt-0.3.5

libansi: reset SGR parameters cache in cls()

There was a bug in HDT when switching from the menu to the cli and
back to the menu: the background wasn't redrawn.

This was due to the fact that the cli bypasses the libansi library
and calls printf() directly. Unfortunately, the SGR attributes were
cached between menu initializations and the background wasn't properly
redrawn.

To fix it, invalidate the cache in the cls() routine. cls() does much
more than just erasing the screen already (e.g. initialize G1) - we
may want to rename the function at some point.

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

hdt: add reboot command in HDT mode (CLI)

Reboot the system via the 'reboot' command in the CLI.

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

hdt: add reboot entry (MENU)

Add a reboot entry to reboot the system. Uses reboot.c32 behind the scenes.

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

hdt: 0.3.5-pre2

Impact: new pre-release

Releasing 0.3.5-pre2. Adding codename for 0.3.5

hdt: Adding core developper role

Impact: Visual

Pierre is more than a contributor ;)

Merge remote branch 'pam/hdt-0.3.5'

hdt: use plain printf for help, not more_printf (CLI)

The help prints several elements on the same line. This can't
work with more_printf() as the macro assumes each call contains
one and only one \n.

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

hdt: change remaining printf to more_printf (CLI)

A lot of places still had printf() instead of more_printf().
This patch does a bulk update and add missing reset_more_printf(),
used to reset the pager counter between each command call.

This also changes the more_printf() routine to delete the "--More--"
line while scrolling.

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

Revert "hdt: Adding core developper role"

This reverts commit 57689f219c5155e186ced1c9f5f972d5d0b340ed.

hdt: Adding core developper role

Impact: Visual

Pierre is more than a contributor ;)

dmi: enhance SMBIOS/DMI detection

Add missing sanity checks when parsing the SMBIOS table entry point.

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

hdt: fix printf typo (CLI)

Correction of an English grammar mistake:

  doesn't exists -> doesn't exist

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

hdt: reset position of the cursor at startup (CLI)

When starting HDT in cli mode (nomenu), reset the cursor to be at the
top left corner.

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

dmi: fix a strcpy (typo)

Not sure why sprintf is being called here. It's probably strcpy that was meant.

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

hdt: change inactive attributes (MENU)

The default inactive attributes in cmenu make little sense in HDT.
They actually make the menus hard to read.

Instead, use the same attributes as the active entries for the inactive
ones.

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

cmenu: reset the cursor position on shutdown

Reset the cursor at the top left corner of the screen when exiting the
menu. This fixes odd behaviors when switching back and forth between CLI
and menu in HDT.

Closes: #14

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

hdt: reset position of the cursor at startup (CLI)

When starting HDT in cli mode (nomenu), reset the cursor to be at the
top left corner.

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

hdt: use plain printf for help, not more_printf (CLI)

The help prints several elements on the same line. This can't
work with more_printf() as the macro assumes each call contains
one and only one \n.

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

hdt: change remaining printf to more_printf (CLI)

A lot of places still had printf() instead of more_printf().
This patch does a bulk update and add missing reset_more_printf(),
used to reset the pager counter between each command call.

This also changes the more_printf() routine to delete the "--More--"
line while scrolling.

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

hdt: change inactive attributes (MENU)

The default inactive attributes in cmenu make little sense in HDT.
They actually make the menus hard to read.

Instead, use the same attributes as the active entries for the inactive
ones.

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

cmenu: reset the cursor position on shutdown

Reset the cursor at the top left corner of the screen when exiting the
menu. This fixes odd behaviors when switching back and forth between CLI
and menu in HDT.

Closes: #14

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

hdt: fix printf typo (CLI)

Correction of an English grammar mistake:

  doesn't exists -> doesn't exist

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

dmi: fix a strcpy (typo)

Not sure why sprintf is being called here. It's probably strcpy that was meant.

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

dmi: enhance SMBIOS/DMI detection

Add missing sanity checks when parsing the SMBIOS table entry point.

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

dmi: Adding smbios detection

Impact: Avoid misdetection of dmi version

On some hosts, the legacy DMI version is reported as 0.0.
We can use SMBIOS to get the real version.
Solves HDT's ticket #8

hdt: Fixing {s}printf typo

Impact: Avoid some unwanted printf

Some dmi calls were using printf instead of sprintf.
This solves ticket #11

hdt: Adding hdt.img.gz target

Impact: It's now possible to have a gzipped floppy image

'make hdt.img.gz' now generates a gzipped floppy image

disklib: Adding new grub2 mbr magic

Impact: Detecting newer release of Grub2

Newer releases of grub2 have a eb63 signature

disklib: Fixing Syslinux Detection

Impact: Syslinux is now detected as bootloader

A test wasn't correct and prevented the Syslinux mbr detection

hdt: Fixing hdt.iso target, pci.ids was missing

Impact: Iso file is now having pci.ids

The makefile did forget to copy the pci.ids into the iso.
This stupid test is fixed.
Thx gert from reporting it.

hdt: Adding Gert as contributor

Impact: none

Gert is contributing in hdt, he have to be in the contributor list

Merge commit 'erwan/master' into hdt-0.3.5

Improving chain.c32 help message

Impact: Error message is more detailed

In case of syntax error, the message is far more detailed and provides
the same content as the documentation integrated in the source code.

hdt: Removing useless Author Name

Impact: visual

Pierre do much work on it, no need to keep my name everywhere ;)

Merge branch 'master' of ssh://terminus.zytor.com//pub/git/users/erwan/hdt

hdt: Cleaning Makefile

Impact: none

We use more variables to have a clean makefile

Merge commit 'hpa/master'

hdt: Adding hdt.iso target

Impact: It's now possible to generate a bootable iso

make hdt.iso now generates a bootable iso

mboot: correct the detection of the multiboot header

Correct the detection of the multiboot header.  The logic for
when the header extends past the file was backwards, which broke
non-ELF multiboot kernels completely.  Apparently they are few and far
between.

Reported-by: Klaus T. <nightos@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'hdt/master'

hdt: Adding missing hdt.c32 in 'make hdt.img'

Impact: hdt.img is now complete and running

After several modification of the makefile, the copy of hdt.c32 had
disapeared from the makefile :(
This commit reintroduce this missing.
'make hdt.img' is now working again

pxelinux: handle hardcoded options both "before" and "after" DHCP

Make it possible to specify hardcoded options both "before" and
"after" the processing of DHCP/PXE options.

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

hdt: Removing pci.ids

Impact: Removing optional file

The pci.ids have to be taken on the system or downloaded from the pciids
website. Syslinux doesn't have to provide it.

hdt: Adding hdt.img target

Impact: add a bootable floppy image

The 'make hdt.img' command generates a hdt.img file.
hdt.img is a 2.88MB floppy bootable image.

Merge commit 'hpa/master'

utils/Makefile: install pxelinux-options

Make sure the pxelinux-options script gets installed.

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

NEWS: document pxelinux-options

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

Merge branch 'hdt'

pxelinux-options: add help text

Add help text to pxelinux-options.

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

chain: missing some \n in error messages

Impact: visual

Some \n where missing in some error() calls.

Signed-off-by: Gert Hulselmans <kimmik999999@yahoo.co.uk>

pxelinux: support hardcoded options "before" or "after"; tool

Support hardcoded DHCP options both "before" or "after" the
PXE-provided options.

Add a tool to manipulate these hardcoded options.

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

chain: missing some \n one error messages

Impact: visual

Some \n where missing in some error() calls

Signed-off-by: Gert Hulselmans <kimmik999999@yahoo.co.uk>

Adding VMX & SVM cpu flags

Impact: Hardware virtualization is now detected

Minor update to detect vmx & svm cpu flags

pxelinux: support "hard-coding" DHCP options into the binary

Make it possible to patch pxelinux.0 so that it includes specific
options.  This isn't possible for gpxelinux.0, but there it is
possible to instead modify the embedded script.

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

int18: module for local booting using INT 18h

At least until we can get a better handle on return vs INT 18h,
provide a module so we can at least tell people how to test.

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

gpxecmd: module to send an arbitrary command to gPXE

Simple module to send an arbitrary command to gPXE, assuming the gPXE
CLI API is available.

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

Merge branch 'cmenu-video' into hdt-next

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

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

Merge branch 'bootloader-detection' into hdt-next

Conflicts:
com32/hdt/hdt-cli-disk.c
com32/hdt/hdt-common.h
com32/hdt/hdt-menu-disk.c

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

cmenu: use ASCII characters for scrollbars

Use ^ and v as hints for scrolling. Use \141 from the Special Characters and
Line Drawing Character Set as the scrollbox.

Note: printf "\016\141\017" doesn't seem to work well on xterm, although
it does work on the vga console.

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

cmenu: do not invoke help if helpid is 0xFFFF (complex.c)

0xFFFF is invalid (no help)

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

hdt: do not invoke help if helpid is 0xFFFF

0xFFFF is invalid (no help).

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

cmenu: redraw the current menu after exiting the keys_handler

After exiting the keys_handler, reset_ui() is not enough since it
only resets the status line and title. The current menu needs to be redrawn.

Note: only a submenu will be redrawn, not its parents. This means that not
all the UI will be displayed after exiting the handler if invoked from whithin a
submenu. We have already this problem nowadays when exiting a submenu
of a submenu. The root parent is not redrawn, leading to some ugly states
(the background color overrides part of the root menu).
This will be fixed when we implement a real hierarchy (each submenu having
access to its parents).

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

hdt: remove extra LF

In a few places, extra LFs cause the menu to scroll ('\n' in the status bar).
Remove them (they are not needed anyway).

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

cmenu: optimize redraw to be faster

cmenu currently redraws everything after each key press. This doesn't scale
well, especially over serial. Instead, redraw only the previous/current selected
entries. A full redraw is still needed when scrolling though.

Misc.: add CTRL-L combo to force a redraw of the current menu.

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

cmenu: force refresh after checkbox_handler in complex.c

complex.c has a special checkbox_handler that disables mutual exclusive
entries. This works today because each key press triggers a redraw of the whole
menu. We can't rely on this if we want to optimize the redraw.

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

cmenu: redraw menu when exiting keys_handler

The keys_handler can change the UI (e.g. the help system draws a black
background by default). We need to redraw the background and title on exit.

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

cmenu: pad help filename with 0 instead of spaces

The manual states that the help filename format is hlp<NNNNN>.txt where
<NNNNN> is the helpid.

If the helpid contains less than 5 digits, pad with zero, not spaces,
i.e. look for hlp00023.txt, not hlp   23.txt.

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

cmenu: remove obsolete com32io code

Remove legacy scrollupwindow definition and checkkbdbuf declaration.

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

cmenu: make the help menu actually work

With the recent changes in cmenu, the help feature was completely broken.
Fix it.

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

hdt: fix build due to cmenu keyboard changes

Sames changes as the ones needed on the cmenu examples.

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

libansi: add function to scroll up multiple lines

Impact: API change. scrollup() has been renamed scrollup_once().

Use scrollup(nb_of_lines) to specify the number of lines the window needs
to scroll.

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

cmenu: fix handlers signature in the examples

The keys handler accepts three arguments: the current menu system, the current
menu entry and the key pressed.

Misc: fix timeout in complex.c to actually be 10 seconds.

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

cmenu: use get_key library for keyboard functions

Syslinux already uses a get_key library to handle stdin. Let's leverage it.
cmenu now works over serial completely.

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

libansi: correctly reset attributes

When resetting the attributes, we were updating last_attr to the unknown value
0x300, which doesn't always work.

For instance, when requesting reset and normal attributes, i.e.
"\e[0;30;47m" with the previous attribute being "\e[1;30;47m", we where printing
"\e[0;47m", which behaves like "\e[0;39;47m". This is incorrect.

This patch adds a flag to explicitly set background and foreground colors after
a reset.

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

libansi: reset_colors should use csprint

cprint_vga2ansi takes only a char, not a string.

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

cmenu: fill backgorund with printable character

To have a blue background on the console (serial), we need to fill
the screen with a printable character (e.g. space).

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

cmenu: remove legacy CHABSATTR/CHRELATTR code

CHABSATTR and CHRELATTR have been replaced by SO/SI.

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

cmenu: remove legacy box drawing code

Lines are now printed via the Line Drawing Character Set.

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

cmenu: unify normal menu/radio menu handling

A lot of code was duplicated between regular and normal menus
(menu printing and selection handling). This patch unifies this logic
by adding an extra bool in printmenu() and getmenuoption() to distinguish
between the two types of menus.

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

cmenu: fix Login/Password and Kernel Arguments lines (complex.c)

Update the cmenu example complex.c to use clear_line(): cprint behavior has
changed w.r.t. cursor positioning (it is automatically advanced now).

Note: need to reset the colors before clearing the line since the status bar is
printed just before the Kernel Arguments line and it changes the background
color.

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

libansi: add reset_colors function

Purpose of reset_colors is to reset the attribute to VGA 0x07.

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

cmenu: turn off auto wrap

Auto wrap made the menu scroll because of the status line (when printing
the last character of the bottom right of the screen, the ansicon scrolled
the whole menu).

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

ansi: add support for no-wrap mode, cleanups

Add support for nowrap mode CSI ? 7 l (unfortunately, on ANSI.SYS it
is CSI = 7 l; which means that the only way to support both is to
print both sequences and then erase the resulting garbage.)

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

core: clean up SerialIRQPort when uninstalling the interrupt system

Make sure SerialIRQPort is zero when there is no active interrupt
system installed.

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

core: when enabling serial IRQs, unmask interrupts at the PIC

Some BIOSes only enable interrupts in active use by the BIOS at the
PIC, which presumably don't include the serial ports.  As such, enable
the interrupt lines at the PIC and hope to $DEITY we don't hang due to
a stuck interrupt line.

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

Merge commit 'origin/libansi' into cmenu-video

Merge branch 'cmenu-video' of git://git.zytor.com/users/pam/hdt-pierre into cmenu-video

core: remove overzealous calls to cleanup_hardware

cleanup_hardware is not needed in core/runkernel.inc: cleanup_hardware
is already invoked from replace_bootstrap.  Furthermore,
cleanup_hardware is defined to preserve all registers, so there is no
need to push/pop ax around it.

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

core: add missing calls to cleanup_hardware

As we're starting to rely more and more on a modified hardware state,
it is really important to clean up on all exit paths.

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

pxelinux: workaround for EFI CSM bug - BEV stack overwrite on localboot

On at least some machines with an EFI-based BIOS, the Compatibility
Service Module (CSM) puts the BEV stack at an address where the boot
loader is likely to overwrite it.  Implement a workaround which forces
INT 18h instead if an EFI signature is detected in memory.

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

cmenu: use printf instead of csprint with default attribute

We don't change the attribute in the help screen. Refactor the code
to simply use printf.

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

cmenu: use putchar instead of putch when using default attribute

There is no real need to do putch('x', 0x07), 0x07 being the default
VGA attribute. Use putchar instead.

Note that showhelp will trigger a drawbox first with 0x07 as attribute.
Since we don't update the attribute, it is guaranteed to still be 0x07
when invoking putch.

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

hdt: remove call to legacy getnumcols (menu)

Use getscreensize instead.

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

cmenu: use getscreensize to access number of rows/columns

The simple menu system already uses getscreensize. Another step towards
unification.

Impact: exported getnumcols and getnumrows are now deprecated.

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

gpllib: add const qualifier in cpu_dev (cpuid)

The Syslinux build turns on -Wwrite-strings, which gives string constants
the type const char[length].

Note: -Wall doesn't trigger this warning.

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

gpllib: add missing header in ata.c

This fixes an implicit definition warning.

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

hdt: fix signedness warnings

host_bus_type and interface_type are uint8_t. Force a cast when calling remove_spaces
on these fields.

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

hdt: unify disk headers in hdt-common.h

Both cli and menu need to access disk functions. Include disk headers
in hdt-common.h, instead of duplicating code.

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

gpllib: remove get_error declaration from disk/util.h

In d1890409251949b80c2026557b9b12ca00d55144, it seems we
have forgotten to update util.h.

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