chain.c32: make CHS calculation match core/fs/diskio.c

Use the same format for the CHS calculation as in core/fs/diskio.c;
also, apply the correct limits, and propagate the fix to the write
routine.

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

strspn.c: drop "inline"

Drop "inline" from bit operations.  These should be recoded to use x86
bts/bt instructions, but for now just drop the "inline" which causes
failures on gcc 4.5 due to inlining failures.

Reported-by: Sebastian Herbszt <herbszt@gmx.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

NEWS: Update for 4.03

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

isohybrid: Use ftruncate instead of seek for final padding

Pad the image via ftruncate instead of seeking to the end (which was
done incorrectly).

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

chain.c32: fix conversion from LBA to CHS

Bits [9:8] of the cylinder goes into the top bits of the sector
register, not the bottom (and the math for that was wrong, anyway.)

Reported-by: Michal Soltys <soltys@ziu.info>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge remote branch 'genec/rosh_for_hpa'

core: fix handling new directory "/" for CONFIG in ISOLINUX and EXTLINUX

Fix missing option -s in libinstaller

In 'libinstaller/syslxopt.c' on line 71 the "s" (undercast) is missing in:

   const char short_options[] = "t:fid:UuzS:H:rvho:OM:ma";

Adding the 's' there seems to solve our problem.

pxe: add a "pxeretry" option to deal with 404's in web apps

For web downloads, sometimes a mirror site will not be fully synced.
Add an option to retry the open a specific number of times before
giving up.

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

extlinux: try to get the disk offset from sysfs

It is possible(?) that HDIO_GETGEO can't return the full offset, and
in either case it is too small -- only 32 bits on 32-bit platforms.
Thus query sysfs for the real value, if available.

sysfs also contains information for slave devices -- this is going to
really matter for the md/lvm issues.

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

diskio: remove redundant register write

The disk number is set twice in diskio.c/edd_rdwr_sectors() this patch
removes the redundant setting.

version: next version will be 4.03

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

core: look for invalid MBR handover information address

The Win7 MBR doesn't provide proper handover information.  Instead, it
leaves DS:SI = 0:0x7e00, so reject that information as invalid on
grounds that it is pointed into the bootloader area.

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

chain: Extra comments; shrink the new space to minimum

chain: Just assign the value rather than using memcpy()

chain: add drmk= to usage

chain: Fix up and clean up DRMK load code; fix filename length;  Patch memory.

chain.c32: prepare for loading DRMK

syslxopt: remove lengthy warning message

Remove a lengthy warning message which cause the actual command
reference to overflow a 24-line screen.

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

syslxopt: fix fatal warnings

Fix fatal warnings in syslxopt.c partly due to legal but non-idiomatic
use of switch().  Since the only purpose was to conditionalize single
output statements, just do it explicitly.

In addition, since they are conditional messages, there is no need to
explain when they apply.

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

chain.c: fix public index value in mbr and gpt iterators

mbr and gpt iterators skip empty / non-data partitions properly, but don't
update publicly visible index. With this patch, such partitions are counted as
well.

Signed-off-by: Michal Soltys <soltys@ziu.info>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

libinstaller/syslxopt.c: remove unneeded usage text from certain variants.

Gert Hulselmans noticed this.

Make __free_tagged actually free memory

__free_tagged called by comboot_cleanup_lowmem (core/mem/free.c) doesn't
seem to work correctly. Memory allocated with lmalloc() isn't marked as free
after the allocating module exits:

boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000553f0
boot: mem
lowmem_buf: 0x00075400
boot: mem
lowmem_buf: 0x00000000
boot:

The expected behaviour

boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000353e0
boot: mem
lowmem_buf: 0x000353e0

can be achieved with the following patch.

Merge branch 'm' into rosh_for_hpa

core: define and use set_flags() helper

Rather than opencoding the arithmetic flag mask, define an inline to
make that happen properly.

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

sample: remove obsolete "raw" comboot/com32 samples

We really don't want people to write code without using the library,
so just remove these obsolete and no longer functional samples.

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

libmenu: Fix cursor on exit from showmenus()

This fixes the issue in hdt.c32 that Kent Robotti noticed where
exiting directly from Menu Mode caused the cursor to not be displayed.
This same issue also affects com32/cmenu/simple.c32 and probably
others using showmenus().

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core: don't hang if no config file is found

If we can't find a configuration file, we need to do something
slightly more friendly than hanging.

Reported-by: Sebastian Herbszt <herbszt@gmx.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

win64: experimental Win64 installer (syslinux64.exe)

Experimental Win64 installer.  This is necessary to be able to run on
WinPE64, since WinPE64 doesn't support running 32-bit binaries.

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

win32: fix README, adjust broken +x bits

Fix the win32/README file and adjust some stray +x bits which had
crept in.

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

lua: remove obsolete FIXME comment

Remove an obsolete FIXME comment; the problem was caused by a stale
library.

NEWS: document IP byte order fix.

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

pxe: fix byte order of output from parse_dotquad()

parse_dotquad() incorrectly returned output in host byte order.

Reported-by: Alexander Wuerstlein <arw@arw.name>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

extlinux, linux: Ignore --force in these installers

Previous commits described this behaviour, but did not
actually implement it. :)

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

dos: Use libinstaller option parser

We will produce an error message if a user attempts
to use an option we don't implement.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

dos: add .size directives to int2526.S

Add .size directives to int2526.S.

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

dos: add .size attribute for syslinux_ldlinux_len

Just for the sake of good order, a .size attribute for
syslinux_ldlinux_len.

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

dos: add missing file ldlinux.S

Add missing assembly wrapper for ldlinux.sys.

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

win32: remove an archive before running ar on it

Always remove an archive before ar...

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

dos, win32: use our own getopt_long() for both

Use our own version of getopt_long() for both the DOS and Win32
installers.  Currently, on MinGW, getopt_long() is a static library,
but that could change in some installations.

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

dos: fix non-executable files which were +x

Don't make non-executable files +x...

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

Merge branch 'master' into libinstaller_dos_wip

Resolved Conflicts:
libinstaller/syslxopt.c
mtools/syslinux.c

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

dos: advance the pointer correctly

Advance the pointer into the payload segment correctly.

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

dos: add memcpy_from_sl() and use it where appropriate

Add memcpy_from_sl() and use it to handle pointers into the payload
section.

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

dos: change the link procedure to use negative pointers below DGROUP

Use negative pointer values below DGROUP, i.e. for the .payload
segment.  This makes it a lot easier to figure out if we are dealing
with pointers that shouldn't come from where they do, since the DGROUP
and .payload segments no longer overlap, and in fact are congruent.

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

NEWS: mtools, win32 uses new CLI

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

man: update syslinux.1.  It is still out of date, however.

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

Merge remote branch 'gerth/lua'

Merge remote branch 'gerth/ifplop_comments'

Merge remote branch 'sha0/for_hpa'

diskio: add a paranoia cast

Make sure we don't overflow the shift, even though that should never
be possible.

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

diskio: fix name for secpercyl field, remove unused type field

Fix the name for the incorrectly named "t" (track) field; it is in
fact the sectors/cylinder (secpercyl) field.  Furthermore, remove the
completely unused "type" field... right now the pointer to the
rdwr_sectors field acts as a proxy, and if we need more complex stuff
in the future it is probably better handled by pointing to an ops
structure.

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

NEWS: document EDD->CHS fallback.

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

diskio: remove locally coded ilog2()

Use the centrally coded ilog2() function.

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

diskio: expand most fields; cache the number of total tracks

Expand most fields in struct disk to 32 bits -- it makes the code
smaller in general, and struct disk is not a common data element.

Add a field for the number of tracks (number of heads*sectors).

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

diskio: move strict CHS start value check into chs_rdwr_sectors()

It really makes more sense in chs_rdwr_sectors()...

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

diskio: check early for an impossible LBA value for CHS requests

Make sure we don't end up trying to do a CHS request for a truly
absurd value, that could possibly be misconverted.

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

diskio: when falling back EDD->CHS make sure to get the offset right

When we decide to go from EDD to CHS, make sure we convert from
absolute back to a partition offset.  The cylinder check should still
be on the absolute value, however.

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

diskio: remove redundant sector count cap

The sector count cap on CHS is automatically given as a result of the
no-track-crossing rule.

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

diskio: allow fallback from EDD to CHS

At least one system has been found which require fallback from EDD to
CHS, so actually make it doable.

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

NEWS: Try again for proper lua.c32 credits

Trying again to get this right...

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

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

whichsys.c32: Fix example in the comments and Usage output

Fix example in the comments and Usage output.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

ifplop.c32: Fix spelling errors and other mistakes in comments

Fix various spelling errors and other mistakes in the comment section.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

doc: Recommend NASM 2.07+

Recommend NASM 2.07+ to avoid the overlong instructions problem around
NASM 2.05.

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

doc: NASM 2.03+ has been required since 4.00 at least

At least since the 4.00 code was merged, NASM 2.03 or higher has been
required.

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

dos: Work-in-progress 2

win32: Use libinstaller option parser

We will produce an error message if a user attempts
to use an option we don't implement.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

mtools: Use libinstaller option parser

We will produce an error message if a user attempts
to use an option we don't implement.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

libinstaller: Add --mbr, --active, and --force options

In order to consolidate the installer option-parsing system,
we add the --mbr (-m), --active (-a), and --force (-f)
options from the DOS, Win32, and mtools installers to
libinstaller.

Since the -m and -a options are not currently valid for the
Linux 'syslinux' and 'extlinux' installers, we add error
messages to both of these.  The -f option is allowed but
does nothing for them.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

NEWS: correct credit for new Lua work.

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

NEWS: update

Update in preparation of a 4.02 release cycle.

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

DMI: Fixing memory mgmnt in dmi_memory_module_types

In some case, dmi_memory_module_types could read the entire memory as a
result of a bogus snprintf usage.

rosh: show environment

rosh: more remove unneeded check

rosh: move ioctl.h include for Linux-only

rosh: bump build

rosh: more: overallocation, better feedback for debug

rosh: make getscreensize more effective and usable

rosh: move scrbuf to more's main function; malloc and free oversize buffer.

rosh: more: if it's the first line, dump it to screen for now even if it's too long

rosh: fix displaying too many lines because a line is too long.

rosh: fix the DATE macro stuff

rosh: begin to add Syslinux version string with DATE

Was from commit 226224fdd1dff5cfcac641c1405303932b59aa4f; This is incomplete but will be fixed in the next commit

lua: Add .syslinux.derivative() and .syslinux.version()

Add .syslinux.derivative() and .syslinux.version().
Add an example script which shows how you can use .syslinux.derivative().

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

zlib: rename DEBUG to DEBUG_ZLIB again

We don't want to use the global DEBUG for zlib...

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

lua: remove obsolete header used for old sleep function

Remove obsolete header used for old sleep function.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

mtools: slightly more obvious definition of tmpdir fallback

Make it a bit more obvious what actually happens for the tmpdir
fallback.

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

mtools: honor TMPDIR, check for errors writing mtools.conf

Honor TMPDIR when picking a place to write mtools.conf.

Check for errors while writing mtools.conf.

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

btrfs: print an error if finding compressed/encrypted data

If we find compressed or encrypted data, print an error message
instead of returning garbage.  This is suboptimal, but at least
handles the common subcase of an encrypted configuration file.

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

Merge remote branch 'gerth/ifcpu_remove_defines'

ifcpu.c32: Remove defines used in the old sleep function.

Remove useless defines which where used in the old sleep function.

Signed-off-by: Gert Hulselmans <gerth@zytor.com>

NEWS: add grubcfg= and whichsys.c32

Add grubcfg= and whichsys.c32 to the NEWS file.

NEWS: add grubcfg= and whichsys.c32

Add grubcfg= and whichsys.c32 to the NEWS file.

Merge remote branch 'gerth/whichsys'

Merge remote branch 'gerth/chain_grubcfg'

ifcpu.c32: need <unistd.h>

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

ifcpu.c32: remove open-coded sleep function

Remove open-coded sleep function, which uses an unreliable BIOS call.

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

lua: change syslinux.sleep() to seconds, syslinux.msleep() to ms

For consistency with just about everything else, make syslinux.sleep()
count seconds, and syslinux.msleep() count milliseconds.

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

com32: add proper sleep()/msleep() functions and make Lua use them

Add proper sleep() and msleep() functions, which among other things
call syslinux_idle(), and make Lua use them.

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