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>

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>

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

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>

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

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>

hdt: Fixing DMI version display

Impact: none

Version is unsigned, so using %u is prefered ;)

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

[chain] Run Nindent on chain.c

[chain] Add support for chain-loading another isolinux.bin

Add 'isolinux=' option to chain.c32 for chain-loading
another isolinux.bin.

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