Merge remote branch 'hpa/master'

hdt: Adding another logo

Impact: none

Adding another logo (black)

hdt: Adding logo

Impact: none

Adding transparent logo

hdt: Updating contributors

Impact: Visual

Adding Alexander Andino for design. Adding other contributors's role

hdt: fixing wrong type

Impact: Avoid compilation error

Fixing return type

hdt: 0.3.6-pre1

Impact: Pre-release

Releasing 0.3.6-pre1

hdt: Adding background image for the CLI

Impact: Visual

The cli now features a background image when started in vesa mode

hdt: Adding vesa mode support

Impact: cli can now be turned into vesa

Using the "vesa" keyword at boot time, you can enable the vesa mode for
the cli.

hdt: hdt.img must use 8.3 naming

Impact: Loading files is now working again

Prior to that commit, the hdt.img was failing loading files

memdisk: pptr->cd_pkt.sect_count is in virtual sectors

Per the El Torito spec, pptr->cd_pkt.sect_count are in virtual
sectors, i.e. 512-byte sectors.

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

memdisk: Use boot_lba logic for booting an offset within the disk image

We have two fundamental quantities: the offset of the disk image
itself and the LBA from which we boot.  Keep them straight.  This
should make at least some no-emulation isos work correctly.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memdisk: Restore DL for INT 13h, AH=15h, DL=floppy

At least PC-DOS requires this to avoid an infinite loop; the DL we
restore is not the same DL that we call the underlying BIOS with, but
rather the original one (before the decrement.)  PC-DOS loops over all
the floppies and relies on the call preserving DL in order to advance
the loop.

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

open: close the file descriptor if the file fails to open

Close the file descriptor (freeing the file structure) when opening a
file fails.

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

dprintf: fix uninitialized pointer; return void

Fix an uninitialized pointer bug; return void rather than returning
int like normal printfs... if we're depending on the return value of a
debugging function we're screwed when debugging is disabled.

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

dprintf: fix dprintf() and vdprintf()

Fix incorrect prototype for vdprintf() used in dprintf(), masked by
the non-use of the header file.

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

menu: debugging hooks

Add (disabled) debugging hooks to the menu system.

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

dprintf: a generic debug-to-serial infrastructure

Actually provide real infrastructure for debug-to-serial.  Very
useful when debugging fullscreen applications, especially under an
emulator like Qemu.

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

hdt: fix double lines in the menu

Ralf Carl reports some UI glitches in the summary menu
(unnecessary line separator at the bottom).
This patch rewrites the separator logic.

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

hdt: trivial printf fix (CLI)

Remove a typo. This fixes a compilation warning.

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

hdt: more memory corruption fixes

A cli_module_descr can be defined with no modules (only a default
callback). Cf. the syslinux module for instance.
This patch adds a check for NULL structures before trying to dereference them.

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

hdt: fix memory corruption in CLI

Ralf Carl has reported some corruption in the autocompletion feature
of the CLI. This patch fixes it.

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

hdt: Using dmi when cpu doesn't support identify

Impact: Avoid free field

On old processors, identify isn't working. In that situation, let's use
the dmi to find the processor model

hdt: Summary mode use type6 as workaround

Impact: Avoid empty field

The summary mode should also use type6 as fallback of type 17

hdt: Adding includes to avoid gcc warning

Impact: Avoid warnings at compile time

hdt.c was calling start_cli without including the according .h file

hdt: Fixing type 6 detection in cli mode

Impact: Type 6 only systems would have been unable to show modules in
cli

This commit is completing bc6105b409291f0f15bbe99d348e5641a0f7c729

hdt: Removing dead code, adding comments

Impact: Removing dead code

This code was used in the ancient cli model. That's just dead code.
Let's add a few comments too

hdt: Summary in cli mode can use type 6

Impact: Show summary displays modules when type 17 isn't available

When dmi type 17 isn't available, the "show summary" command must use
type 6 to show the current memory configuration

Merge branch 'master' into memdisk-iso

isohybrid: make the -offset option actually work

Actually implement the -offset option.  Credit for the patch goes to
"cmdr from the gparted-forum", reported by
Sven-Hendrik Haase <sh@lutzhaase.com>.

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

hdt: Disabling SERIAL 0 support

Impact: Improves speed

As reported in ticket #24, removing SERIAL output increase performances

hdt: DMI Type 6 used as a workaround for Type 17

Impact: Allow to have a memory entry when only type 6 is available

Prior to that commit, systems that only featured type 6 weren't able to
display the memory information.

Merge branch 'memdisk-iso-SAFE' into memdisk-iso

Conflicts:

memdisk/memdisk_chs.asm
memdisk/memdisk_edd.asm
memdisk/setup.c

Resolved:

Deleted memdisk/memdisk_chs.asm
Deleted memdisk/memdisk_edd.asm
Modified memdisk/setup.c

gpllib: mbr: add detection for Grub2 minor versions

Grub2 v1.96 and v1.97 have different opcodes in their MBR.

Reported-by: gert
This fixes #26.

Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>

gpllib: mbr: change "Gag" to "GAG"

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

gpllib: mbr: change "Plop" to "PloP"

Reported-by: gert
Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>

gpllib: add XOSL mbr detection

Feature request originally reported at

http://sourceforge.net/projects/bootinfoscript/forums/forum/905692/topic/3459317

and forwarded to HDT by gert.

This fixes #25.

Signed-off-by: Pierre-Alexandre Meyer <pierre@ning.com>

mboot.c32: fix the video mode selection algorithm

Make the video mode selection algorithm do what was intended...

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

mboot.c32: add VESA mode setting support

Add VESA mode setting support per the Multiboot specification.

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

Merge branch 'master' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux

dos: int 25/26 may be register-dirty; wrap them in assembly

int 25h and int 26h are known to be register-dirty for some versions
of DOS -- unlike int 21h, which is usually clean.  As such, wrap those
in assembly functions.

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

dmi: fix bank connections information

Ralf Carl reports that bank connections are incomplete (DMI type 6).
Some information was left out during output formatting.

This should fix #23.

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

dmi: fix DMI type 6 output

Ralf Carl reports some garbage in the DMI type 6 output. It appears
that some output formatting wasn't quite right.

This should close #22.

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

dos: don't add the linker script twice on the ld command line

My version of the linker can deal with it, but others might not be so
smart.

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

dos: drop unneeded .eh_frame section

gcc will produce an .eh_frame section, which we have no use for.

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

dos: always try the FAT32-aware direct I/O calls first

Always try the FAT32-aware direct I/O calls before trying the legacy
raw I/O calls.  The reason for doing this is that the FAT32 stuff may
be implemented as an add-on.

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

dos: DOS version 7.10 is 0x070a, not 0x0710...

Fix DOS version 7.10 (first FAT32-capable version) check.

Perhaps we should just call the FAT32 calls blindly and fall back to
the old calls if they fail?

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

NEWS: merged hdt 0.3.5

Merge commit 'hdt-0.3.5'

version: next version on this branch will be 3.84

NEWS: document fix for DOS installer

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

dos: fix the soft-locking loop

Code the soft-locking loop correctly.

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

dos: try to handle both raw DOS mode and Windows mode

The locking API works very different in raw DOS mode and in Windows
mode.  The hierarchial locking is only available in the latter mode;
in the former mode we can only use levels 0 and 4.

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

dos: make DOS installer work under WinME

At least under WinME, the DOS installer did not work as advertised.
With these modifications, it seems to work okay.

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

hdt: Fixing subrelease

Impact: none

That's an alpha not a pre

hdt: Generating gzipped content in binary images

Impact: Reduces image size

Floppies & ISO are now using gzipped content to reduce the size

hdt: Switching to 0.3.6-pre1

Impact: none

Let's work on the future product

pci: using zfopen instead of fopen

Impact: allow to open gzipped files

This commit allow people using gzipped files
(pci.ids/modules.pcimap/modules.alias)

hdt: iso/floppy have to be quiet

Impact: Quieten output of hdt

By default, the provided iso/floppies are now quieter.
This will increase the readibility of HDT's output

pci: Removing useless initialisation

Impact: None

The kernel module count is set to zero when the dev_info is allocated.

pci: Prevent kernel modules to be listed twice

Impact: Prevent duplicated modules

If both get_module_name_from_pcimap() &  get_module_name_from_alias()
are called, we didn't checked if the module we are detecting already got
detected. This leads to a situation where modules got listed twice.

This patch add a test to insure that we aren't adding an already
detected module.

hdt: 0.3.5

Impact: new release

New release : 0.3.5

cmenu: Increasing limits

Impact: avoid memory corruption with hdt

Some users reported systems with 89 pci devices. This case requires to
increase theses values

hdt: Fixing building warning

Impact: Avoid warning during compilation

Avoid warning during gcc compilation

Revert "ansicon: allow to pass page # to write functions"

This reverts commit e9b317fd0dc87dc93a49bc40520f4bdd0264d746.

(This commit breaks vesamenu, and $DEITY knows what else.)

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

Merge commit 'sha0/chain_isolinux'

hdt: Putting auto mode in coherence with cli

Impact: visual

The output generated by the auto mode should stay in coherence with the
default cli mode.

hdt: Switching debug mode in quiet mode

Impact: "quiet" keyword also a quiet display

quiet is a new keyword to prevent HDT beeing too much verbose

Revert "Revert "gpllib: Reident fbinst code""

This reverts commit 28d7917dd52b34e2669fa9ad44445124b1d10584.

Merge remote branch 'pam/hdt-0.3.5'

Revert "gpllib: Reident fbinst code"

This reverts commit e4e28b8fd43a971466ff38b243784d20437d6252.

hdt: Prevent auto mode from using empty commands

Impact: prevent running stupid commands

The code didn't prevents from executing stupid "'" commands

gpllib: Reident fbinst code

Impact: none

Reindent the fbinst patch

hdt: fix arguments parsing

Do not increment the argument counter in the loop after having found
the last auto= argument. It is taken care of by the for.
With it, an extra argument after the auto= string will be ignored.

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

hdt: optimize more_printf macro

disable_more_printf should be set only in auto mode (for now).
Add __likely macro to optimize the code.

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

disklib: add support for detection of the fbinst mbr

Note that fbinst and grub4dos mbrs share the first two bytes.

Patch originally submitted by gerth.

This fixes #20.

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

hdt: add debug option

Pass 'debug' on the kernel command line for the output to be more verbose.
Making hdt quiet is really helpful in auto mode, when a lot of information
is flowing.

This fixes #21.

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

hdt: fix arguments parsing

Do not increment the argument counter in the loop after having found
the last auto= argument. It is taken care of by the for.
With it, an extra argument after the auto= string will be ignored.

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

hdt: optimize more_printf macro

disable_more_printf should be set only in auto mode (for now).
Add __likely macro to optimize the code.

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

disklib: add support for detection of the fbinst mbr

Note that fbinst and grub4dos mbrs share the first two bytes.

Patch originally submitted by gerth.

This fixes #20.

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

Merge commit 'hdt-0.3.5-pre3' into hdt-0.3.5

hdt: Release 0.3.5-pre3

Impact: new pre-version

Releasing pre3 with reboot & auto= features

hdt: Setting up a flag to disable more_printf

Impact: None.

Some modes could requires disabling more_printf

hdt: Removing debugging message in auto mode

Impact: Visual

This debugging info isn't interesting

hdt: Improving readbility of auto mode

Impact: Visual

This patch enhance the readibility of the auto mode

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

Merge branch 'master' into auto

hdt: Improving automatic code (memory mgmt)

Impact: Freeing memory

We have to be more careful about the malloc & free. We also have to
remove spaces around commands

hdt: Adding automatic mode

Impact: HDT can be controlled via the cmdline

When the auto= parameter is set, HDT execute a set of commands and
return to the cli after.

i.e auto='show summary; show pci' will show the summary then the pci and
return to the cli.

Note that the auto= feature, requires a the "'" char to define the
complete set of commands and the ";" char to separate commands.

Some more examples:
auto='show syslinux; exit' -> this will show the syslinux release and
then exit to return to the bootloader

auto='show summary; reboot' -> this will show the summary and
then reboot the computer

Merge remote branch 'pam/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: Adding serial output when building HDT images

Impact: Serial output have to be activated

That's almost useful having the serial output when running HDT

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

hdt: Adding reboot.c32 in iso & floppy images

Impact: reboot is now working in floppy & iso images

The reboot features requires the reboot.c32 file to be copied in our
floppy & iso images

Merge remote branch 'pam/hdt-0.3.5'

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>