hdt: switch to libansi

Make hdt use libansi.

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

cmenu: switch to libansi

Make cmenu use libansi.

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

libansi: add missing prototype in the header

clearwindow() was not presented in libansi.h.

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

libansi: build libansi.o

Hook libansi.o into the lib/ build.

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

libansi: move to lib directory, as it is MIT, not GPL licensed

gplinclude/gpllib should be used for external, GPL, libraries
only. libansi has been developed for Syslinux and is MIT licensed.

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

gpllib: add more ANSI functions

Import the functions that I introduced in cmenu.

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

gpllib: add a new ansi library

HDT and cmenu are currently duplicating code: they both define their
own mini ansi library.

This patch extracts the HDT ansi lib into gpllib as a first step towards
unification.

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

cmenu: reset attributes instead of using ANSI sequence 22

hpa advised not to use ANSI sequence 22, as it may not work everywhere.
Instead, let's reset attributes and set last_attr to unknown. The next
background/foreground tests are then guaranteed to fire. We take a small
hit in terms of performance though.

Misc.: change initialization of last_attr to 0x300 (unknown) to trigger
properly the tests the first time.

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

cmenu: fix background color

The menu highlights some characters to mark them as key shortcuts.
To switch the bold attribute off, we were printing the reset code,
resetting the background color in the same time.

Use ANSI sequence 22 instead.

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

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

We shouldn't be using any pages, as there is no support on the
serial line.

This reverts commit e9b317fd0dc87dc93a49bc40520f4bdd0264d746.

cmenu: use 80x25 as terminal size by default

Let's not rely on the geometry returned by the BIOS for the
terminal size - we don't know anything about the one used via serial.

We now default to 80x25, regardless of what the BIOS returns.

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

cmenu: misc. cleanups

Fix a compiler warning and a header documentation.

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

cmenu: remove multi pages support

The cmenu used to display the help on a separate page. Since we
can't have multi pages on the serial line, get rid of them completely.

This has a fairly big impact as the external cmenu API has changed.
adv_menu.tpl and HDT have been patched accordingly.

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

cmenu: use < for menu exit character

Use plain '<' as the "back" symbol.

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

cmenu: fix menu separators

Use the alternate character set to draw horizontal lines (menu
separator). Refactor the c*print functions in the same time as they
were severely broken.

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

cmenu: use > instead of » for submenus

» doesn't exist in the alternate character set. Use plain
> instead.

Misc.: fix SO/SI invocation, putchar takes a char, fputs a string.

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

cmenu: remove extra getboxchars call

getboxchars is not used anymore in drawbox. Let's get rid of it.

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

cmenu: build cleanups

Fix build warnings and remove old cls declaration.

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

cmenu: refactor cswprint function

Now that cprint is smarter, cswprint can be simplified.

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

cmenu: refactor printmenuitem function

Simplify the printmenuitem function, now that putch is smarter.

cmenu: refactor beep function

Use \007 to make the system beep. Note "\007" works, but not '\007'.

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

cmenu: refactor cprint functions

Some refactoring:

* Make cprint_vga2ansi static, as it is an internal function.
* Simplify the logic in cprint().

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

cmenu: use VT-100 alternate character set for drawing boxes

Use the Special Characters and Line Drawing Character Set as
described in http://www.vt100.net/docs/vt102-ug/table5-13.html
to draw boxes around the menus.

Note that to work in xterm, G1 needs to be initialized (to point
to the alternate character set). This is done in the cls function
("\033)0").

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

cmenu: remove unused bit_reverse function

Not needed anymore: we use a lookup table (ansi_char)
in cprint_vga2ansi now to reverse Background and Foreground bits..

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

cmenu: better implementation of vga->ansi

hpa suggested a better implementation, that also fixes some color
issues and invalid ANSI codes. Thanks hpa!

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

cmenu: implement cprint using printf

First step towards a usable menu. Color is now sent over serial.

Testing Done: launched HDT over serial. The background got drawn blue,
as expected.

Note: vga attributes need to be converted to ansi codes.

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

cmenu: remove unused getchar function

char getchar(void) doesn't seem to be used.

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

cmenu: implement scrollup using SU escape sequence

Use the ANSI SU - Scroll Up escape sequence to scrollup the window.

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

cmenu: implement gotoxy using escape sequences

Use ansicon and the ANSI CUP - CUrsor Position escape sequence
to implement the gotoxy function.

Note: page switching is not supported (yet).

Testing Done: ran com32/cmenu/test.c32 in qemu.

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

ansicon: allow to pass page # to write functions

Some video modes support up to 8 different pages, the current one being
stored at 0x462.

The display page value is passed in BH when invoking the INT 10h Video
Service routines. This patch changes the interface of ansicon_write_char
and ansicon_set_cursor to allow the caller to specify the page.

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

NEWS: update for 3.83

hdt: enhance error reporting in the CLI (disk)

parse_partition_table can fail for two reasons: disk I/O
error or unrecognized partition layout (non msdos, corrupt, ...).

In the latter case, we were displaying

   Error parsing disk 0x80
   parse_partition_table: error 0

which is confusing. We now display

   Disk 0x80: unrecognized partition layout

or the errno_disk in case of I/O errors.

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

hdt: make get_error behaves like perror

This simplifies buffer handling.

Misc.: clean some old error code handling.

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

hdt: fix CLI crash when parsing unsupported geometries

Remove extra free() calls: these were needed before the introduction
of the disklib errno. We apparently forgot to remove them.

Reported-by: Gert Hulselmans <hulselmansgert@gmail.com>
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: 0.3.4

Impact: New release

0.3.4 is now out !

hdt: fix disklib display

We currently display the first sector after the end of an extended.
Fix it (-1).

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

disklib: fix extended partition code

gert1 reported some issues using disklib with HDT. As it turned out, the
code that iterated through ebrs in an extended partition was broken: ebr offsets
need to be relative to the start of that partition.

Misc.: fix memory leak.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
Signed-off-by: Erwan Velu <erwan.velu@free.fr>

hdt: Fixing memory corruption

Impact: Improve stability

In some case, sectors_to_size can return a 8 char long string like "1000
MiB". In such case, some buffer were corrupted.

This is now fixed by a size[9] instead of size[8].

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

Merge commit 'pam/disklib-fixes'

MEMDISK: generate map files

Generate map files to make debugging less painful.

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

memdisk: make the stack size configurable

Make it possible to configure the stack size on the command line,
so that we can rule that out in case of problems.

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

menu: initialize cm (current menu) before empty check

The empty check in the menu system used "cm", but before "cm" had
actually been initialized.  Move the "cm" initialization earlier so we
do the right thing here.

Reported-by: CKSoon <cksoon79@yahoo.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'origin/master'

menu: fix Ctrl-W (word erase) when editing a command line

The Ctrl-W (word erase) key was broken: it would think the command
line got longer, not shorter, resulting in havoc.

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

Revert "Run Nindent on com32/include/pngconf.h"

This reverts commit 6361b71f87466877f8fd1273240e8fa4460f8d05.

<pngconf.h> is part of libpng, it should never have been Nindented.

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

Revert "Run Nindent on com32/include/png.h"

This reverts commit ab7b6c3380cd9188edbc99693d5562dcca055317.

<png.h> is part of libpng and hence externally maintained; it should
not have been Nindented.

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

zlib: remove unused sample programs

Remove unused sample programs (example.c, minigzip.c); we don' tuse
them and they are just noise.

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

zlib: kill gzio.c

Remove gzio.c, which is a zlib-specific stdio-like interface.  We
implement this in the actual filesystem layer, which is much cleaner.

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

hdt: Remove extra more_printf calls

Let's rather use string concatenation.

Reported-by: Shao Miller <Shao.Miller@yrdsb.edu.on.ca>
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: Introduce typedefs for callbacks

typedefs are evil but useful for function pointers as it makes them more
readable and maintainable.

This fixes a bug by the way: we had

   void *callback(struct driveinfo *, struct part_entry *, int, int)

where we should have had

   void (*callback)(struct driveinfo *, struct part_entry *, int, int)

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

hdt: Don't show partition header if not needed

Don't blindly print the partition header: check if we are able to iterate
through them first.

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

hdt: Fix disk menu to use the new disklib interfaces

Adapt the disk menu to use the new disklib interfaces. This fixes bugs related
to the display of partition numbering and swsusp signature.

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

hdt: Make the menu and cli use the new disk interfaces

The disklib interfaces have changed (errno_disk introduced). Update
these changes in hdt (nobody else use them).

Misc: fix get_error declaration.

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

gpllib: More errno fixes

 * Fix header files due to previous commit
 * Prefix disk errnos with ED to avoid conflicts with classic errno
 * Fix swsusp code to use errno_disk

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

gpllib: Enhance Makefile

Leverage Make built-ins rather than listing manually all objects files to
compile.

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

gpllib: Fix compilation

ata.c was missing the inttypes.h header.

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

disklib: Change read/write interface

Get rid of the error pointer and use a errno-like error reporting mechanism.
Intent is to make these more like the standard read/write system calls.

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

disklib: Fix write_verify_sectors

Compare the entire chunk (all sectors), not only the first one.

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

Merge commit 'hpa/master' into for-erwan

Conflicts:
com32/cmenu/libmenu/com32io.h
com32/gplinclude/dmi/dmi.h
com32/gplinclude/dmi/dmi_base_board.h
com32/gplinclude/dmi/dmi_memory.h
com32/gplinclude/dmi/dmi_system.h
com32/gpllib/dmi/dmi.c
com32/gpllib/dmi/dmi_memory.c
com32/include/sys/pci.h
com32/lib/pci/scan.c
com32/modules/Makefile

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

gpllib: refactor partitions detection

Re-implement the logic to parse extended partitions to correctly go through the
linked list of partitions.

Updated the CLI in hdt and tested it both on Qemu (Debian Lenny installation)
and on DELL E6400 (6 partitions). The output seemed sane.

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

NEWS: document isohybrid -id fixed.

isohybrid: convert -id to a binary string

Shouldn't 'isohybrid -id ID' write ID as 32bit number? Currently it ends up
as string messing up the mbr.

memdisk: remove unnecessary %ifdef

Remove an %ifdef that the macros take care of anyway.

memdisk: don't set CX:DX for INT 13h AH=15h for floppies

Win98SE DOS apparently hangs if INT 13h, AH=15h returns the capacity
in CX:DX for floppies... *unless* EDD is enabled.  Confused yet?

This patch also adds some more tracing framework.

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

MEMDISK: additional EDD DPT Fields

We incorrectly used "equ" instead of "dd" (or "db") for strings in
memdisk.inc.  Oops!  My mistake in my 'freedos' branch.  Silly
mistake.  - Shao

memdisk: correct the types for the EDD DPT, and write a real one

Fix the sizes of some of the fields in the EDD DPT.  Write a real one
to the best of our ability.

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

memdisk: don't write the EDD DPT unless we are in EDD mode...

Don't write the EDD DPT unless we're actually in EDD mode!  Otherwise
we end up corrupting memory beyond the defined code.  This was the
reason 5ca2764460883f5e9a5bd83db00f242ea1272e33 worked, by moving
variables out of the corruption zone.

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

Merge commit 'andytim/freedos'

graphics: actually do 32-bit writes to VGA

Use "rep movsd" to copy data to VGA memory, not "rep movsb".

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

[memdisk] Additional EDD Device Parameter Table fields

Some additional fields from the EDD-4 spec. draft for the Device Parameter
Table have been added into the structure in setup.c and memdisk.inc.  These
were added in the hopes of resolving a FreeDOS MEMDISK bug on IBM ThinkPads.

graphics: trivial optimization of old-style graphic mode

Trivially optimize the old-style graphics by using another level of
temporary buffering in bitplane format, thereby enabling the use of
rep movsd to write the contents to the screen.  This can speed things
up substantially in virtualization environments.

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

NEWS: document Thinkpad fix

memdisk: move temporary variables away from the EDD data block

Move a couple of variables away from the EDD data block.  Without this
change, FreeDOS doesn't boot on Thinkpads.  Unfortunately, this should
have been a null change, so it is really not clear why this happens.
Stack overflows have been ruled out.  Probably need to examine the
FreeDOS kernel in some detail.

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

memdisk: handle recursive INT 13h invocation

Some systems, e.g. a lot of Thinkpads, invoke INT 13h recursively from
within the stack itself.  This is generally a bad idea as it re-runs
any INT 13h hooks (including the ones that DOS installs!), but it
should work.  Keep a counter of recursive invocations and simply
bypass the whole system including the DL shift if the invocation is
recursive.

With this, MS-DOS boots on a Thinkpad T61.  FreeDOS is still broken,
though, but apparently in a different manner...

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

PXELINUX: allow filenames up to 251 characters long

At least one users really need this, so be nice and provide it...

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

Merge commit 'syslinux-3.83-pre1'

Conflicts:
com32/modules/chain.c

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

NEWS: document chain.c32 ISOLINUX fix.

chain.c32: set DL properly when loaded from ISOLINUX

When loaded from ISOLINUX, we still have a valid boot drive.  Set it!
This permits the use of chainloaded boot loaders.

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

gfxboot: remove obsolete message regarding pxelinux support

PXELINUX is supported by the gfxboot module.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

gfxboot: parse TIMEOUT keyword

Parse the TIMEOUT keyword from the config file and pass it to gfxboot core.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

gfxboot: only call gfxboot core if labels are specified

Only call gfxboot core if labels are specified in the config file.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

gfxboot: display error message if bootlogo file not found

Display an error message if the user specified bootlogo file is not found.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

gfxboot: change handling of keywords

Makes handling of keywords more flexible.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

dosutil/Makefile: more complete structure

More complete and consistent structure for dosutil/Makefile.

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

dosutil: update mdiskchk, add Makefile

Update mdiskchk and add a Makefile.  Note that since this needs
OpenWatcom to compile, we still check in the binary file (which is
quite small, anyway.)

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

Create a dosutil subdirectory and move mdiskchk there

DOS utilities aren't really "samples".  Create a dedicated "dosutil"
directory, and move mdiskchk there for now.

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

extlinux: remove duplicate "const char *program;"

Remove redundant declaration.

Merge branch 'syslinux-3.8x'

NEWS: document chainloading fix

PXELINUX: when chaining an NBP, restore *all* registers

On at least ASUS A8N-E, unless *all* registers are restored to pre-PXE
status, localboot doesn't work.  We were clobbering DS, ESI and EDX
when chainloading an NBP, which meant that localboot wouldn't work in
the chainloaded NBP.  Fix this.

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

bootsect: zero memory above a chainloaded boot sector/NBP

Windows RIS has been reported to make inappropriate use of data found
in uninitialized memory.  To avoid that, clear memory between
a chainloaded boot sector or NBP and Free Base Memory.

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

com32.h: cleanups, make OFFS_VALID() take the pointer size into account

Clean up some of the constructs in com32.h, and make OFFS_VALID() take
the pointer type into account for the range check -- the whole object
needs to fit within the segment.

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

memdisk: don't use the stack before bootstrap

Under some certain circumstances, it might not be safe to use the
stack as it was set up, so use a jump instead of a call to invoke the
bootstrap.

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

Merge commit 'syslinux-3.82'

Conflicts:
com32/menu/menumain.c

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

pxelinux: actually stop parsing on a null option

Stop the OACK parsing on a null option, even if it isn't the very
first option.

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

pxelinux: revert to using TFTP error 8

The main reason for ERROR is OACK parsing failure, so revert to using
error code 8 (but a fixed text error.)

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

pxelinux: cleaner test for OACK trailing null bytes

Instead of looking for a string of null bytes at the end of the OACK
string, simply abort parsing if we run into a null byte where an
option is expected; either we are seeing junk at the end of the
packet, or we are hopelessly confused about how to make sense of the
rest of the packet -- in either case, ignoring is the "liberal"
option.

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

PXELINUX: handle OACK packets with extra NULs, cleaner TFTP error

There are apparently TFTP servers in the field which will send OACK
packets with extra NUL bytes appended at the end.  If we find an OACK
packet where the only thing left at some point during processing is
NULs, then just consider the packet processed.

We have reported all TFTP protocol errors as "tsize required", which
is definitely not true anymore.  Change error code to 0 (undefined)
and the error string to "TFTP error".  When this code gets converted
to C we'll do better.

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

memdisk: additional cleanups

Additional stylistic cleanups.  Rename "syscall" to "intcall" (we
can't call it __intcall without clashing with com32.h); use macros
instead of copying variables to different places with only the type
being different.

Also, only change the rm/pm jump instructions when actually relocating
the code.

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