partiter.c: add stepall modifier; bugfixes + comment changes

In certain situations (new features to be added later), it's useful
to be able to step through all partitions in a partition list. This
patch adds necessary functionality.

Adjust chain.c to use new version of partiter.

Also small bugfixes and comment changes.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Split chain into smaller files

new file:   chain.h
new file:   common.h
new file:   mangle.c
new file:   mangle.h
new file:   options.c
new file:   options.h
new file:   utility.c
new file:   utility.h

Signed-off-by: Michal Soltys <soltys@ziu.info>

partiter.c: cosmetic changes

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Improve setdrv option, remove setbpb4/

Option 'setdrv' can autodetect offset now. setbpb4 is removed, as it's
no longer necessary.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: change file/sect/hand area handling logic, add 'hptr' option

The logic behind file/sect/hand area handling is more flexible
now. Furthermore, a user can explicitly control where ds:si/ds:bp
should point during chainloading.

Other misc. changes:

- registry setup is more conservative
- 'if' style

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: merge segs= and sect into single option, remove segf

There's no need to have separate 'sect' and 'segs='. 'segf' alias of
'seg' can also be removed now.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: option renames

Renamed some of the options to better fit the whole scheme.
Also adjusted usage().

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Always use console_ansi_raw(), not only with DEBUG

Signed-off-by: Michal Soltys <soltys@ziu.info>

Merge branch 'disklib' into chaindev

chain.c: adjust what ds:si and ds:bp point to after chainloading

Previously, chain module always set ds:si and ds:bp to handover
area (unless otherwise blocked). This is not necessarily valid in case
when both file and sector are being loaded - here file should receive
pointer to it's [relocated] sector, and it's unlikely interested in
mbr-like handover area at all.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: fixes, adjustments

- update usage()
- add chs2lba()
- update handover code to generate CHS values
- make sure 'write' is never default
- add wait_key()

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: add options, misc varaible renames

This patch adds following options:
- setgeo (bpb: set legacy geometry)
- setdrv (bpb: set physical drive number)
- read  (read mbr / bs - previously always attempted)
- write (write bs - for bpb mangling)
- filebpb (also mangle 'kernel' file in memory)
- msdos7 (dos7+ IP is set as required by io.sys)

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: add 'smap' and 'hand' options, update handover code

'[no]hand' allows explicit control whenever to create handover data

'[no]smap' allows loading bootsector without mapping it during
actual chainloading (meant for upcoming BPB mangling options)

Expanded handover code moved mostly to its own function

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain, partiter: bugfixes

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Don't bruteforce loops detecting fixed drives.

Previously chain module looped all possible fixed disks from 0x80 to
0xFF. It's prone to BIOS bugs, where BIOS can report nonexistent or
phantom drives at higher numbers. It's safer to use fixed disk count
value from 40:75 to limit such loops.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Introduce sseg and fseg options.

Add options for detailed specification of load and jump addresses.
Misc adjustments.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: fix trailing whitespace

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: move reg setup of dos drive to find_dp()

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: add nokeeppxe option and sethid shortcut of sethidden

Signed-off-by: Michal Soltys <soltys@ziu.info>

disklib: small fixes / addons

Functions disk_write_sector() and disk_write_verify_sector()
take 64bit lba as an argument now - similary to disk_read_sectors().

Structure disk_info additionally provides cylinders, besides head and sector.
Sanity checks during lba -> chs conversions have been adjusted to use
cylinders.

CX in cbios read/write code was not calculated properly.

Minor signed/unsigned changes.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Rename cur_part -> iter

Current iterators go over both disk and partitions, including raw disks.
More generic name fits better into the whole scheme.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c minor fix of ADDRMAX value

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: move regs to opt structure

regs, being a set of starting registers for chainloaded image, fits
logically into opt structure. Also simplifies function callings.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: fix all compiler warnings in very pedantic mode

Compilation of chain.c now passes: -Wextra -Wconversion -pedantic

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: move file mangling outside main()

All the file mangling code has been moved to
mangle_*() family of functions, currently:

mangle_isolinux()
mangle_grublegacy()
mangle_grldr()
mangle_drmk()

They will be further moved to separate file.

Also, file and sector loading routines are now near each other.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Adjust hide_unhide() and remove mbr_area from main()

hide_unhide() is adjusted to use new iterators. It's also fully
self sufficient.

mbr_area is no longer needed in main().

Check for proper magic when loading bs/mbr is removed. There can be valid bootable
code without 0xaa55 magic (e.g. for testing purposes or unusual tasks).

Signed-off-by: Michal Soltys <soltys@ziu.info>

partiter: Add raw iterators

As iterators cover the whole disk now, it's possible to be able to still
boot "something" from the 1st sector, even if it has no valid mbr or gpt
layout. This patch adds "raw" iterator, updating necessary code.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: Restructure finding of partition/drive to chainload.

Finding drive or partition to chainload has been moved outside
main() to find_dp(). Further adjustments and fixes made to accomodate
this change.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: move parsing outside main(), move some options to opt structure

Options' parsing can be easily moved outside main().
drivename and partition are part of opt structure now.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: adjust usage(), update placeholder doc/chain.txt

Make usage() info more precise, adjust indentation.
Update placeholder docs.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: add soi2sli() function

Add generic function to convert seg[:off:[ip]] into seg:lin:ip, mostly
in preparation for further updates.

Make opt.seg check use it.

Signed-off-by: Michal Soltys <soltys@ziu.info>

partiter.c: tone down non-data sanity checks, add comments in ebr helper

Sanity checks vs. programmer are compiled only with DEBUG.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: initial move of documentation to doc/chain.txt

This patch moves the commented documentation from chain.c to
placeholder doc/chain.txt.

Signed-off-by: Michal Soltys <soltys@ziu.info>

doc/gpt.txt: minor fix

Information about one field was incorrect.

Signed-off-by: Michal Soltys <soltys@ziu.info>

chain.c: minor fixes

This fixes segval comparison and two mistypes.

Signed-off-by: Michal Soltys <soltys@ziu.info>

split chain into chain + iterator, expand iterators' code

Patch:

- splits chain into chain and iterator parts and moves them into their
  own com32/chain directory
- extensively updates iterators' code
- adjusts chain.c to use new iterators
- fixes mbr/sect/hand/file allocation

Signed-off-by: Michal Soltys <soltys@ziu.info>

Merge branch 'master' into chaindev

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

Merge branch 'master' into disklib

Resolved Conflicts:
com32/modules/chain.c

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

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>

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.

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>

Merge remote branch 'sha0/com32lib_disk' into disklib

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>

chain.c32, libcom32: Move gpt_dump() as disk_gpt_header_dump()

Moving portions of chain.c32 into libcom32.

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

chain.c32, libcom32: Move struct gpt as disk_gpt_header

Also moved gpt_sig_magic as disk_gpt_sig_magic.
Moving portions of chain.c32 into libcom32.

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

chain.c32, libcom32: Move gpt_part_dump() as disk_gpt_part_dump()

Moving portions of chain.c32 into libcom32.

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

chain.c32, libcom32: Move guid_to_str() and str_to_guid()

Moving portions of chain.c32 into libcom32.
There might be a better header for these to belong to.

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