FAT: For FAT32, the root directory is a cluster chain

The old code would set the FAT32 root directory right after the FAT
like everything else... which would just happen to work if the root
directory was in cluster 2.  This is very common for a newly formatted
filesystem, but isn't guaranteed!

Reported-by: Gert Huselmans <gerth@zytor.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'origin/master' into fsc

mkdiskimage: add option -s to make a sparse image

It is often fine to have a sparse file for the output image, so add an
-s option to generate a sparse image.

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

core: set QUIET_FLAG if we load Linux "quiet"

If we see the "quiet" flag for the Linux kernel, also suppress the
early Linux kernel messages; for people who want a completely clean
boot.

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

FAT: remove debugging printf

Remove debugging printf which snuck in...

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

FAT: fix FAT16/32 next cluster calculation, make more similar to FAT12

The FAT16/32 next cluster calculations were changed to be somewhat
more like the FAT12 one, but that was done incorrectly: we would end
up reading off the end of the FAT sector we were looking at.  Make it
structurally more similar to the FAT12 calculation, and actually make
it compute correctly.

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

FAT: fix next_sector implementation

"return ++sector;" is a bit subtle and I brainfarted and changed to
"return sector++;" -- change it to "return sector+1;" to make it
obvious we're not looking for a side effect at all here...

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

Merge branch 'master' into fsc

Resolved Conflicts:
core/ldlinux.asm

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

chain.c32: Handle nonlinear logical partitions

Some partitioning program, including at least parted, apparently
create the list of logical partition headers nonlinearly with the
partition themselves.  Thus, make it a requirement that then fit
inside the overall extended partition, but not inside the
corresponding sublogical partition.

Reported-by: Gert Huselmans <gerth@zytor.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

openmem: fix openmem()

Correct the implementation of openmem().

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

core/fs: abstract filesystem, use cache for ext2 group descriptors

Abstract out the filesystem, remove all references to "this_fs" in the
main filesystem drivers. There is still some in the core (dir.c, fs.c)
which eventually need to be replaced by a properly defined device
marker and root.

The inode structure now contains a reference to its parent filesystem.

The inode structure can now contain additional data at the end of the
structure, this is used for filesystem private data.

TODO: move the filesystem private data into proper structures.

Finally, use the block cache for ext2 block group descriptors.  Trying
to allocate them all at filesystem mount time doesn't work for large
filesystems.

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

FAT: use an anonymous union

We can use an anonymous union here... we're more or less gcc-specific
anyway, and it cleans up the namespace a lot.

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

FAT: deal with non-mirrored FATs for FAT32

FAT32 has the option of non-mirrored FATs, used to improve atomicity
of some operations.  If we have non-mirrored FATs, we need to make
sure to read the one FAT which is currently marked active.

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

core/fs: make it more explicit sector size is per filesystem

Sector size is per device, and can vary between filesystems.  In
particular, it is time to be getting rid of assumptions of 512-byte
sectors whereever possible.

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

fs.h: fix prototype

Another () -> (void)

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

FAT: Compare cluster count to total number of clusters

We have to know the total number of clusters, so we might as well make
use of that knowledge.  Treat any impossible cluster number as EOF.

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

FAT: generate EOF depending on exceeding the number of clusters

Generate EOF by comparing to the actual number of clusters.  We have
to compute the number of clusters in the filesystem anyway during
setup, so save the number and use it for the EOF computation.

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

ldlinux: fix FAT EOF detection

We had the wrong highest possible cluster for the FAT; adjust it so
that it matches the definition.

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

runkernel: we only look at the first letter in vga=...

We only look at the first letter in the handling of the special vga=
strings, make that manifest in the assembly code so we don't end up
with unnecessary warnings.

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

FAT: handle the cluster number being 1

Any reference to cluster 1 is an error, so treat it the same as
cluster 0 or one of the EOF cluster numbers.

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

FAT: clean up cluster arithmetic

a) clean up the computation of the FAT12 FAT entry offset.
b) the highest allocatable cluster is ...ff6, not ...ff0.

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

FAT: be more explicit why we think a filesystem is bad

Give readable error messages as to why we think a filesystem is bad.
Given the current sizes of the installer, these extra strings are
worth it.

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

FAT: fix installer on FAT32 filesystems

The FAT32 superblock has a 12-byte reserved field which wasn't
accounted for in the structure.  As a result, the installer refused to
run on a FAT32 filesystem.  Fix the structure.

Reported-by: Kim Mik <kimmik999999@yahoo.co.uk>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'origin/master' into fsc

Merge branch 'master' into fsc

Conflicts:
com32/lib/MCONFIG
com32/lib/readdir.c

Add support for "vga=current"

"vga=current" is a user-friendly synonym for "vga=0x0f04".

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

core: don't reset the video mode when invoking the kernel

Don't reset the video mode when invoking the kernel if we're booting
quiet -- it breaks "vga=current" which some distros use for a smooth
graphical transition all the way into the kernel.

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

core/diskio: correct the EDD parameter query

We need to tell the BIOS how big of a buffer we have for the EDD
parameter query.  This caused us to *never* use EDD, which meant that
large filesystems never worked.

Reported-by: Gert Hulselmans <kimmik999999@yahoo.co.uk>
Isolated-by: Alek Du <alek.du@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core/diskstart: we don't have 64-bit partition offset just yet...

We don't actually have a way to receive a 64-bit partition offset yet,
so don't pretend to (on FAT, this field contains other information.)

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

MCONFIG: add debugging version of GCCWARN

Add the options that help smoke out bugs.

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

dmi_processor.c: remove unreachable table values

A uint8_t can't have values all the way up to 0x12e...

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

disk/bootloaders.c: fix bogus const in get_bootloader_string()

get_bootloader_string() takes a mutable first argument; the call chain
does have call paths on which this argument will be modified.

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

disk/ata.c: fix signedness weirdness in ata_id_c_string()

Fix signedness weirdness in ata_id_c_string(), thus killing a warning.

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

com32/lib: remove unused variable in syslinux_getadv()

Remove unused variable, thus removing a warning.

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

zlib: fix warning in crc32.c

Fix a signed/unsigned warning in crc32.c.

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

com32/lib: fix external prototype for __vesacon_open()

The external prototype for __vesacon_open() was incorrect; this is a
good example of why external prototypes are bad, incidentally.

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

com32/lib: fix inappropriate & in sys/colortable.c

The & is incorrect... not actively harmful, but generates a warning.

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

com32/lib: add cast in __parse_argv()

ALIGN_UP_FOR() returns the type of the original pointer, but we want
it to be the type that we're actually going to use.  Perhaps
ALIGN_UP_FOR() should be changed, but for now, just add the proper
cast.

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

libgcc: fix signedness warning in __moddi3()

Intentional cross-signedness pointer passing, add a cast.

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

com32/lib: fix unused argument warning in fdopendir()

fdopendir() is a stub, and probably will remain such.

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

com32/lib: fix unused parameter warning in chdir()

chdir() is a stub at the moment... to be fixed in Syslinux 4.

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

memdisk: fix non-prototype function declaration

() is not a prototype, use (void)

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

com32/lib: fix warning in str*cmp()

Shut up a warning about initializing pointer across signedness.

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

com32/lib: fix warnings in opendir/readdir/closedir

Missing #include <stdlib.h>, nested comment in readdir.

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

Makefile: replace -W -Wall with centralized $(GCCWARN)

Replace -W -Wall hardcoded into a bunch of Makefiles with $(GCCWARN),
a centralized variable defined in the root MCONFIG.  Add
-Wstrict-prototypes to the list of global warnings: we should never
have non-prototyped declarations.

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

cmenu: clean up some () that should be (void)

() is not a prototype, and means (...) not (void) in C.
Replace with (void).

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

NEWS: document Grub4DOS chainloading

chain.c32: simplify the grub4dos support

Grub4dos (like Grub in general) uses the same partition numbers as
Linux (and chain.c32), minus one.

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

chain.c32: add grldr= command for Grub4dos

grldr of Grub4dos wants the partition number in DH:
0xff: whole drive
0-3:  primary partitions
4-*:  logical partitions

Hmmm... there really isn't a huge reason not to do this
unconditionally, at least unless it's known to cause problems.  It
would be better, of course, if grldr used the standard DS:SI, but it
doesn't, so oh well.

Some info of a Grub4dos developer (Tinybit):
  GRLDR can be loaded at any address with alignment 16(i.e., a possible
  segment base address). Generally you want to load it at 0000:7C00, or at
  2000:0000. Of course you never load it at 0000:0000 or similar.

  Before jumping to the entry point at the very beginning of GRLDR, you
  should setup DL=(BIOS drive) and DH=(partition number). For partition
  numbers, 0 - 3 are primary, 4 - 0xFE are logical. (DH=0xFF) stands for
  whole drive(unpartitioned). DH will later be passed to
  install_partition(the third byte, from bit 16 to bit 23).

http://www.boot-land.net/forums/index.php?showtopic=8457&st=20&start=20 post #22

Update README for 2010

Change () prototypes to (void)

() means the same as (...) in C, not the same as (void) as it does in
C++.  It is generally misused to mean (void), though.  Actually write
what we mean... this is C, after all.

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

Merge branch 'master' into fsc

Resolved Conflicts:
core/configinit.inc
version

Resolved Undetected Conflicts:
 core/ui.inc

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

Update date to 2010

core: Remove the built-in default

The built-in default is rarely if ever used anymore and highly
confusing to users.  Remove it entirely.

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

Merge branch 'master' into fsc

dosutil: don't remove targets which need Watcom even for "make spotless"

Most systems won't have Watcom installed, so don't remove
Watcom-generated binaries even with "make spotless".

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

btrfs: use malloc instead of static allocation for chunk map

Now when we have malloc() available, use it instead of static allocation.

Signed-off-by: Alek Du <alek.du@intel.com>
LKML-Reference: <1262073486-21321-1-git-send-email-alek.du@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

btrfs: change func() to func(void)

"void func()" means "void func(...)" in C, but "void func(void)" in
C++.  However, we want the "void func(void)" meaning, but this is C.

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

chain.c32: stylistic comments/usage output changes

Comments fix and usage output corrected for chain.c32

- Gert Hulselmans

Signed-off-by: Gert Hulselmans <kimmik999999@yahoo.co.uk>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'origin/master' into fsc

Resolved Conflicts:
version

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

Merge commit 'sherbszt/gfxboot32'

gfxboot: add .gitignore file

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

isolinux: export kaboom

We need to export kaboom in all cases...

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

Merge commit 'liu/master' into fsc

Resolved Conflicts:
core/fs.c
core/fs/ext2/ext2.c

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

eltorito: align some pointlessly misaligned data items

Align some data items (including the jump table) which were misaligned
for no clear reason.

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

chain.c: do a proper copy operation for the cmldr signature

Instead of writing into memory that we might not own, create a proper
copy operation using syslinux_add_movelist().

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

chain.c32: add support for booting the Recovery Console of Windows NT/2K/XP

The following patch for chain.c32 adds support to boot the Recovery Console of Windows NT/2K/XP.
Normally when you want to boot the recovery console, you need to patch "cmdcons\0" into the
bootsectorof the partition (8 bytes starting at 0x3 (4th byte)).

We can do this patching in memory:
  strcpy((char *) 0x7c03, "cmdcons");

To boot the recovery console with chain.c32:
  chain.c32 cmldr=/cmldr

- Gert Hulselmans

Merge commit 'contrib/master'

chain.c32: fixing implicit declaration of function ‘syslinux_force_text_mode’

When building chain.c32:

$ make chain.c32
gcc -Wp,-MT,chain.o,-MD,./..chain.o.d -std=gnu99 -m32 -fno-stack-protector -mregparm=3 -DREGPARM=3 -march=i386 -Os -W -Wall -march=i386 -fomit-frame-pointer -D__COM32__ -nostdinc -iwithprefix include -I../../com32/libutil/include -I../../com32/include -I../../com32/gplinclude -c -o chain.o chain.c
chain.c: In function ‘do_boot’:
chain.c:552: warning: implicit declaration of function ‘syslinux_force_text_mode’
ld -m elf_i386 -T ../../com32/lib/com32.ld -o chain.elf chain.o ../../com32/libutil/libutil_com.a ../../com32/gpllib/libcom32gpl.a ../../com32/lib/libcom32.a /usr/lib/gcc/i486-linux-gnu/4.4.1/libgcc.a
objcopy -O binary chain.elf chain.c32

The following was missing:
  #include <syslinux/video.h>

- Gert Hulselmans

NEWS, version: prepare for 3.85

Merge commit 'origin/master' into fsc

com32: syslinux_report_video_mode returns void

syslinux_report_video_mode() never gave a return value, to have it
return void unless we need something else.

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

vesa: formatting fix

Code formatting fix in screencpy.c

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

vesamenu: unbreak the default background

Unbreak the default background image; it would clobber all of memory.

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

core: clean up the call to disk_init()

Move direct references to registers up as far as possible to make the
code easier to read.

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

core: canonicalize ROOT_FS_OPS

ROOT_FS_OPS belongs in .rodata, and make it the same format.

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

isolinux: fs_init now takes a pointer to an array of filesystems

fs_init now takes a pointer to an array of filesystems, not a pointer
to a single filesystem type.

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

pxe: update to match the new fs_init interface

fs_init now takes a pointer to an array of filesystems, so provide it
as such.  Furthermore, fs_init needs to return a nonnegative value to
be considered successful.

Finally, handle the case of 64-bit partition offsets being passed in
to fs_init (in ebx:ecx in the disk-based derivatives.)

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

btrfs: reduce size by computing the crc32c table on the fly

The crc32c algorithm requires a largish table (1K), however, it can be
computed at runtime using a fairly trivial piece of code.  Even with
compression, this is substantially smaller.

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

btrfs: add btrfs file system support to extlinux

the extlinux.sys will be installed in btrfs first 64K blank area, and the
extlinux.conf must be in root dir...

Signed-off-by: Alek Du <alek.du@intel.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'syslinux-3.84'

Resolved Conflicts:
Makefile

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

Merge branch 'master' into fsc

Resolved Conflicts:
com32/Makefile
com32/lib/sys/open.c
com32/modules/Makefile
dos/Makefile
dos/com16.ld
dos/syslinux.c
version

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

spec: don't actually package the dosutil directory proper

No need to actually package the dosutil directory per se.

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

spec: NASM 2.03, new dosutil directory

Require NASM 2.03, and add new dosutil directory.

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

NEWS: document eltorito.sys

NEWS: chain.c32 now always returns to text mode.

chain.c32: force text mode

When loading an unknown system, revert to text mode first.

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

video: implement long-promised video functions; use them

Implement long-since-promised video functions defined in
<syslinux/video.h>.  Use these functions in initvesa.c instead of
open-coding the same functionality.

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

NEWS: document chain.c32 -> isolinux.bin addition

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

vfs: add the readdir stuff support

For now, the EXT, VFAT and ISO fs support readdir lib. So, the dir.c32
module can do the right work.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

EXTLINUX: fix the complie error

I fogot to  remove the removed filed of inode in bmap.c file.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

isofs: fix some coding style problems in iso9660.c

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

core: some cleanups

The blksize filed in inode structure removed, since we can simply get it
by BLOCK_SIZE(fs) macro.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

ISOLINUX: applying the generic path-lookup to iso9660 fs

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

memdisk: Fix "might be used uninitialized" warning

Impact: Initializing variables

The checksum_buf function was providing an incorrect checksum
on at least one build.

memdisk: Fix mBFT alignment

Impact: Fixing mBFT alignment

It was previously possible that the mBFT might not be 16-byte
aligned because the .data section was not.

vfs: turn off the cache debug information

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

LDLINUX:vfat: Oh, NO, I did fix the above bug in a wrong way!

I shouldn't count on the 'dname' parameter, we should ZERO the
'long_name' buffer at every time we met a new long name entry,
and that's the right way to fix the bug.

And we shouldn't count the 'id == 0' to check if we have a long
name matched first or not. Say we have one entry of a long name
entry, it will always be 'id == 0' even it doesn't matches. So,
add a new flag variable to do this.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

LDLINUX:vfat: fix the bug that can't find the program even it exists

Here comes a TINY bug but hard to find! After we run hdt.c32(it can run it
well), then we will get failed if we run cpuidtest.c32, and it does exist.

Well, it's weird since it doesn't happened in the fstk implementation.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>

LDLINUX:vfat: Improve the fat fs relative path searching

Added a new flag, FS_THISIND, used in the fat and iso fs relative path
searching. It means tries to get the last directory of the config file
path in the 'this_inode' variable to do a relative searching while loading
the kernel images in after.

And also fixed a bug in fs.c which wouldn't close the file structure when
doing path searching failed! And as a result, we could just run a few com32
programs.

Signed-off-by: Liu Aleaxander <Aleaxander@gmail.com>