gpllibs: Running Nindent

Impact: visual

Running Nindent on gpllib / gplinclude

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

hdt: Running Nindent

Impact: Visual cleanup

HDT now uses the same indent as syslinux

Merge remote branch 'pam/hdt-0.3.6'

hdt: fix memory corruption in the CLI

More memory corruption with overlooked. parse_command_line and
expand_aliases both allocate memory (for command, module, argc and argv).

We did fix the memory corruption for the autocompletion feature but
but didn't do it for the actual execution of the commands.

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

memory: Commenting workaround

Impact: none

HDT is using a memory detection code which duplicates some syslinux
code. Once addr_t will be moved into 64bit, this can be removed by using
syslinux_scan_memory().

hdt: replacing e88 by 88

Impact: e88 doesn't have any meaning

Fixing e88 by 88

hdt: Rounding detected memory size

Impact: Visual

Rounding values to have more realistic values

hdt: Improving memory detection

Impact: Detect memory size even if dmi is broken

This commit uses e820/e801/e88 memory detection to find the memory size.
That allow to display the memory size even if the dmi table is broken

memory: Adding memory size detection

Impact: allow com32 module to detect the amount of memory installed

This commit adds several function:

sanitize_e820_map() to generate a new e820 map that doesn't have
overlaping

memsize_e820() to detect the available memory (in KB)

detect_memsize() to detect the amount of memory by using e820/e801/e88.
It first try to detect the memory via e820. If it fails, it uses e801 as
fallback. If this e801 fails, e88 is used.

hdt: Cleaning mbr_ids

Impact: none

Let's just clean mbr_ids as we do for the orther structures

hdt: More coherency in messages

Impact: more coherency

More coherency in messages

hdt: Adding more coherency in memory messages

Impact: more coherency

More coherency in memory messages

hdt: memory menu have to be shown in every case

Impact: memory menu have to be always visible

In virtualbox, dmi table isn't correctly feed. In that situation, the
memory menu isn't displayed whereas e88, e820 & e801 are present.
So let's remove the condition where the memory menu have to be shown
only when memory modules are detected.

Tested by Gert.

hdt: Renaming "show 88" to "show e88"

Impact: increasing coherency

Let's rename that function to keep more coherency with e820 & e801

Merge remote branch 'pam/hdt-0.3.6'

gpllib: mbr: fix grub2 detection

gerth reports that hdt-0.3.6 doesn't detect grub2 anymore.
It looks like some break were missing in the case statement.

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

hdt: Only copy needed art files

Impact: Reduces binary image size

This commit only copy the needed art files to put in the floppy/iso
images.

hdt: Allow user to override the background image

Impact: none

This commit allow user using vesa=<filename> to override the background
image in vesa mode

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