NEWS: Update for 6.03 release

Thanks to Ady <ady-sf@hotmail.com> for most of this writeup.

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

define _DEFAULT_SOURCE for glibc-2.20

_BSD_SOURCE was deprecated in favour of _DEFAULT_SOURCE since glibc
2.20[1]. To avoid build warning on glibc2.20, _DEFAULT_SOURCE should
also be defined.

[1]: https://sourceware.org/glibc/wiki/Release/2.20

Signed-off-by: Chanho Park <chanho61.park@samsung.com>
_BSD_SOURCE was fist used at 1769d57c94d7a965168b72b6fd8d48251710b452
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

chain/partiter: correct gpt header checks

In gpt header, lba_cur and lba_alt alternate depending on whether we
read primary or backup copy.

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

chain: implement strict=<0|1|2>

This provides more fine grained control than single relax flag. to cover
case with wrong disk sizes.

relax and nostrict are equivalent to strict=0
norelax and strict are equivalent to strict=2

strict=1 does the same as strict=2, but ignores checks against disk size

The current default is strict=1. Options: 'fixchs', '[un]hide[all]' and
'save' will forcibly enable strict=2 (can be overridden by the user).

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

chain: add missing pi_del() in find*() functions

As partiter doesn't deallocate itself after finish (anymore), it should
be deleted after each loop iteration.

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

chain/partiter: adjust error reporting

Use <0 for errors, 0 for normal state, and >0 for clean completion.

In future this would be necessary if it's decided to make partiter
a generic lib (similar to disklib) - though it has to be quieted first
and provide strerr()-like functionality in place of its verbosity.

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

chain/partiter: fix possible non-NULL value returned by pi_begin() on error

This patch fixes non-NULL value returned from pi_begin() in case of an
error (which resulted in further hang instead of proper exit). Probably
a leftover from pi_dealloc() times.

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

chain/partiter: fix and improve gpt handling in buggy cases

Previous version had some issues in case of error situations, among
those:

- backup gpt header was not read if reading of primary failed
- alternating nature of lba_cur and lba_alt was ignored

This patch fixes this and extends the gpt handling.
The current behavior is:

- try primary header; if unreadable or checksum fails (or sanity checks
  fail unless relax flag is set) - try secondary header

- try main partition table referenced in the header that was read; if
  it's unreadable or if its crc check fails - try alternative (so if we
  managed to read primary header, table at the end would be considered
  as alternative - if we managed to read secondary header, table at the
  beginning would be treated as such)

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

PXE ISR: Force polling on select hardware #3 WORKAROUND

Like 2fe3a7bd, certain Dell machines (Latitude E6510) state interrupts
should work but effectively don't.  Force polling.

Reported-by:
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core, pxe: Don't push on one stack and pop from the other in pxenv

When saving/restoring the flags around the stack switch test, we
cannot leave live data on the stack *across* the stack switch that we
are going to use on the other side (unlike the "big" stack frame which
we only use once we are back on the original stack.)

Use register BP, which is not live at either point, as a temporary
holding place for the flags from the stack.

Reported-by: Frank Mehnert <frank.mehnert@googlemail.com>
Link: http://bugzilla.syslinux.org/show_bug.cgi?id=54
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

PXE ISR: Force polling on select hardware WORKAROUND

Like 2fe3a7bd, certain Dell machines (OptiPlex 990) state interrupts
should work but effectively don't.  Force polling.

Reported-by: Alexander Perlis <aperlis@math.lsu.edu>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

lzo: Upgrade to LZO 2.07

Use version 2.07 of the LZO compression library.

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

Test for Watcom C presence

This adds checks for Watcom C DOS cross-compiler in a manner analogous to mingw
compiler for Windows. The compiler is not commonly present in Linux distributions
due to a non-free license.

Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

utils/isohybrid.c: Enable promised options -u, -m, -b

This change enables the single-letter options -u, -m, -b as promised
by the help text.

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

efi: Tag __syslinux_adv* with __export

commit 415d571 ("adv: Remove double defintion") introduced a regression
for the EFI boot loader. The commit is correct, but should have tagged
the __syslinux_adv_ptr and __syslinux_adv_size data objects in the EFI
core as __export.

This change is required because symbols in the EFI code are not global
by default, and so, when ldlinux.{e32,e64} tries to bind to the adv
symbols it fails.

The following demonstrates the ELF symbol changes,

Before:
  659: 0000000000034f40     8 OBJECT  LOCAL  DEFAULT   16 __syslinux_adv_ptr

After:
  957: 0000000000036fe0     8 OBJECT  GLOBAL DEFAULT   16 __syslinux_adv_ptr

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

isohybrid: Function to write UTF-16LE strings

Rather than open-coding the content of UTF-16LE strings, just add a
simple function to convert ASCII (only) strings to UTF-16LE.

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

utils/isohybrid.c: Introduce option --mbr and make isohybrid.c compilable standalone

Although isohybrid.c is supposed to be a companion of the local SYSLINUX
installation, there may be situations where the file isolinux.bin and the
matching MBR template do not stem directly from such an installation.

This change adds an option --mbr, which allows to load an MBR template
file. This may be an isohdp[fp]x*.bin MBR template from the local SYSLINUX
installation, or the first 512 bytes of the isohybrid-capable ISO image
from which isolinux.bin and the other ISOLINUX files are taken.

If macro ISOHYBRID_C_STANDALONE is defined, then the hardcoded MBR templates
are not accessible and isohdpfx.o is not needed at compile time. In this
case, option --mbr becomes mandatory.
I used this for testing my changes with Fedora-Live-Desktop-x86_64-20-1.iso.

isohybrid.c is then compilable without further components of ISOLINUX by:

  cc -DISOHYBRID_C_STANDALONE -Wall -o isohybrid isohybrid.c -luuid

Test run:

  cp Fedora-Live-Desktop-x86_64-20-1.iso \
     Fedora-Live-Desktop-x86_64-20-1-rehybrid.iso

  valgrind ./isohybrid --uefi --mac \
           --mbr Fedora-Live-Desktop-x86_64-20-1.mbr \
           Fedora-Live-Desktop-x86_64-20-1-rehybrid.iso

yields:

  ==13828== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 1)
  ...
  ==13828== LEAK SUMMARY:
  ==13828==    definitely lost: 2,048 bytes in 1 blocks.

(Not that valgrind would have detected the memcpy() abuse of patch 001.
 But at least i seem to have not introduced more obvious sins.)

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

utils/isohybrid.c: Change all fseek(3) to fseeko(3)

It seems unwise to offer future programmers fseek(3) calls for copy+paste.
They are simply insufficient for large image files.

This change switches all calls of fseek(3) to fseeko(3) and takes care
that the offset value if of type off_t.

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

utils/isohybrid.c: Write GPT backup to the very end of the image

The GPT backup header block should start 512 bytes before the end of the
image file (resp. end of the disk device). This block and the backup GPT
array were wrongly written 512 bytes too early.

This change brings the backup GPT at its correct position.

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

utils/isohybrid.c: Correct end block address of first GPT partition

The GPT partition 1 covers the whole ISO filesystem size. GPT specs
demand that the partition end block number shall be the last valid
block in the partition. isohybrid.c rather wrote the number of the
first block after the partition end.

This change reduces the number by 1.

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

utils/isohybrid.c: Correct blocking factor in APM partition block counts

The block counts in the APM partitions assumed 512 bytes per block,
whereas the start block numbers assume 2048 as announced in the APM header.

This change divides the affected block counts by 4 to correct the assumption.

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

utils/isohybrid.c: Encode GPT partition names as UTF-16LE

The worst sin of isohybrid.c was to compose GPT partition names by
8-bit characters and to memcpy() them as if they were 16 bit wide.
GPT names are encoded as UTF-16LE. It is trivial to create this encoding
from 7-bit ASCII.

This change introduces two byte arrays with the desired UTF-16LE names
which replace the string constants "ISOHybrid ISO" and "ISOHybrid".

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

isohybrid: define _FILE_OFFSET_BITS in the file itself

On general principles, define _FILE_OFFSET_BITS in the file itself
(even though it is also in the Makefile.)

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

isohybrid: fix overflow on 32 bit system

When call isohybrid with option '-u', it overflows on a 32 bits host. It
seeks to 512 bytes before the end of the image to install gpt header. If
the size of image is larger than LONG_MAX, it overflows fseek() and
cause error:

isohybrid: image-x86-64-20140505110100.iso: seek error - 8: Invalid argument

Replace fseek with fseeko to fix this issue.

Signed-off-by: Kai Kang <kai.kang@windriver.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxelinux-options: Fix data corruption bug

Fix bug where the data would invariably be incorrectly written.

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

pxe: Add support for embedded options in EFI

For EFI, rather than mucking with the PECOFF or ELF headers (we have
both!) just use a fixed-size buffer embedded in the image with a large
magic number that can be scanned for safely.

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

pxe: Re-add support for embedded DHCP options

Fix the support for embedded DHCP options.  Although we were
diligently saving them away, we never actually parsed them.

This fixes embedded options for BIOS only -- for EFI we need to modify
the encoding scheme so that it can fit inside an EFI PECOFF image.

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

Revert "ldlinux: Allow a held shift key to override UI as well as DEFAULT"

This reverts commit 7e7139788c8ac6ffbf474976ad4d1727e0677b9f.

Requested by Ady <ady-sf@hotmail.com> who also requested the original
patch:

After some tests with MENU SHIFTKEY with different versions of
Syslinux, I came to the conclusion that, at least for now, it would
be better to undo commit 7e7139788c8ac6ffbf474976ad4d1727e0677b9f:

 "ldlinux: Allow a held shift key to override UI as well as DEFAULT"
committed on Thu, 22 May 2014.

Although the aforementioned commit makes the current 6.03 branch
behave as version 4.xx, my tests show that the behavior is
effectively inadequate and inconsistent.

Although I had been previously nagging you in irc about maintaining
backward compatibility of the "force-prompt" keys, I have to ask you
to undo the aforementioned commit before the final release of 6.03
stable. The behavior of Syslinux 6.03 without this particular commit
will slightly "break" with the prior "force-prompt" behavior as seen
in 4.xx when using [vesa]menu.c32, but it will be more consistent and
clean.

Without this commit, the user can still avoid the automatic launch of
the default label by the same means (the "escape" "shift" keys),
which is the most important characteristic of MENU SHIFTKEY.

Please consider this request before the release of 6.03 stable.

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

com32: Fixing lua's dependencies

Lua shall be one of the latest module to build as it requires some other
components to be built before.

Prior this patch, it was impossible to make a parallel build when "j"
was greater than 26.

This patch fix the ordering and allow a "make -j"  of com32 directory.

memdump: Remove old obsolete COM16 binary

memdump was a com16 binary.  If still useful, it should be replaced
with a com32 library using libupload, but it has largely been replaced
by sysdump anyway.

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

sysdump: Remove README that was for memdump, not sysdump

Sysdump was forked from memdump, but the README was never fixed.  It
should be updated, but until then, delete it.

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

Make symbols defined in linker script HIDDEN

This makes symbols defined in com32.ld and syslinux.ld HIDDEN, to
avoid collisions.  Additional linker scripts should have this done, too.

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

adv: Remove double defintion

__syslinux_adv_* symbols are now part of the core, so don't export
them again elsewhere.  This broke all uses of the ADV, including
--once.

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

core/fs: Add support to Unix File system 1/2.

It's already loading modules successfully, booting Linux, and both
UFS version 1 and 2 seem to be working correctly.

Signed-off-by: Raphael S. Carvalho <raphael.scarv@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

installer: Add UFS1/2 support for Extlinux installer

It's needed to enumerate both UFS1 and UFS2 as they have different magic numbers
and super block offsets.

Neither UFS1 nor UFS2 move files around, therefore, ldlinux.sys can be
installed as a regular file.

Signed-off-by: Raphael S. Carvalho <raphael.scarv@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

ntfs: fix incorrect file->offset usage in ntfs_readdir

file->offset is used to store position in index root between
ntfs_readdir calls.  Previously, pointer to buffer was stored in this
field. However this buffer is reallocated and read each ntfs_readdir
call so the pointer may become incorrect.  Now offset in index root
rather than pointer is stored in this field.

[ hpa: applied patch manually as it arrived whitespace-corrupted ]

Signed-off-by: Andy Alex <andy at r-tt.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

cache, btrfs: Provide a general cached read routine

btrfs_read() was really just a function to read data linearly while
using the metadata cache.  Move it to cache.c and rename it
cache_read() so other filesystems can make use of it as well.

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

btrfs: Remove static variables

Remove the use of static variables and replace them with per-instance
allocations that can be found from the fs_info structure.

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

btrfs: Fix stack smash with node size > 4K

Newer btrfs has a node size of more than 4K.  Make sure we have a
buffer big enough to hold a node -- instead of allocating it on the
stack, allocate it at startup time.

While changing this code, remove a completely unnecessary arbitrary
64-bit divide.

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

ldlinux: Allow a held shift key to override UI as well as DEFAULT

For compatibility with 4.xx, make a held shift escape to override UI
as well as DEFAULT.

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

btrfs: fix typo in definition of BTRFS_ADV_OFFSET

Correct the macros used in BTRFS_ADV_OFFSET.

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

btrfs: Move ldlinux.sys to Boot Area B

ldlinux.sys is frequently too big to fit into Boot Area A (64K).  Per
a recent ruling by Chris Mason, we have a 768K Boot Area B at offset
256K, so use that to store ldlinux.sys.  In theory we could split it
up and make it slightly larger, however, ldlinux.sys is loaded below
the 640K mark so it has to be smaller than that size anyway.

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

Centralize shift_is_held(), make it work to force the command line

Holding down shift is supposed to force a command line unless noescape
is provided; make it behave that way.

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

kbd: Use the extended keyboard state query for bios_shiftflags()

Mixing extended and non-extended keyboard functions can be a bad idea.

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

kbd: Make getting the keyboard shift state a firmware method

Getting the keyboard shift state is a firmware method.  It is
unfortunately unclear if it is even possible on EFI.

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

bios: Use int 0x16, ah=0x02 to probe for shift flags

At least one USB keyboard BIOS has been reported to not properly
handle the keyboard flags in memory.  See if actually using INT 16h
works better.

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

NTFS: Fragmented $MFT file was not handled

NTFS $MFT file may be fragmented by itself (and actually is in most
cases).  However, such a situation was not handled.  This patch adds
support for a fragmented $MFT file.

Signed-off-by: Andy Alex <andy at r-tt.com>
Fixed-by: Ady <ady-sf@hotmail.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: Avoid initializing the cache more than once

Most of file system drivers initialize the cache themselves.
The problem is that the same cache could be again initialized later,
then invalidating the previous one. This patch fixes this.

Problem found while auditing the code.

Signed-off-by: Raphael S. Carvalho <raphael.scarv@gmail.com>

lua: Remove additional old cruft from sl_boot_linux()

More obsolete cruft...

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

ntfs: Make byte_shift a const

byte_shift is a constant, label it as such.

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

NTFS: Incorrect parsing of file runs

Length of some extents of NTFS files is parsed in wrong way.

Signed-off-by: Andy Alex <andy at r-tt.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

FSUUID for ext2 filesystem

The ext2 filesystem supports volume UUID now.
The FSUUID variable can be set to kernel command line.
Patch is based on FSUUID for FAT patch.

Signed-off-by: Serj Kalichev <serj.kalichev@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Add filesystem UUID to SYSAPPEND for FAT

Filesystem UUID shows the partition we boot kernel from.
The kernel parameter has format like FSUUID=DA1A-0B2E.
The SYSAPPEND bit is 0x40000. Now the FAT only supports FSUUID.
The patch is based on 67aaaeeb228.

Signed-off-by: Serj Kalichev <serj.kalichev@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge branch 'nocomapi'

Trying again.

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

biosboot: This file is not for EFI

This file causes dynamic link failures on EFI, and of course has
no use in the EFI environment.

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

lua: Remove obsolete crap from sl_boot_linux()

sl_boot_linux() contained a bunch of random obsolete garbage.

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

lua: In sl_initramfs_add_file(), actually use the real file name

Instead of "/testfile1", actually use the real file name...

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

Merge branch 'nocomapi'

lua: replace the syslinux module test with an automatic boot menu generator

lua: base the package load paths on the Syslinux path

lua: upgrade to 5.2.3

lua: add demo of the cmenu binding (the simple example)

lua: start of a binding module for libmenu.c32

lua: make the dmi module standalone

By copying two short functions from the cpu module.
They do not warrant a separate module.
Also make those functions static in both modules.

lua: also reactivate the cpu, dhcp, dmi, pci and vesa extension modules

And document the change in usage.

lua: bind get_key() in the syslinux module

lua: reactivate the syslinux extension module

Under Lua 5.2 modules are not expected to set global variables
to reduce namespace pollution. Explicit require() is preferred.

lua: add the LuaFileSystem library

lua: enable dynamic module loading

sys/module.h: fix some typos in function documentations

sys/module.h: remove the #ifdef DYNAMIC_MODULE condition

This makes the module_* functions accessible to COM32 modules.
MODULE_INIT and MODULE_EXIT are not used anymore.

lua: the COM32 API does not support freopen()

Nor the concept of text vs. binary open mode, so it does not matter.

lua: the COM32 API does not support time()

lua: the COM32 API actually supports exit() of oslib

stdlib.h: provide a stub definition for getenv()

Which always returns NULL, representing an empty environment.

lua: the COM32 API supports only part of iolib

lua: the COM32 API does not provide strcoll()

lua: use integer arithmetic

lua: the COM32 API does not provide locale.h

lua: explicitly put the console in standard mode

lua: the COM32 API does not provide signal.h

dev.h: get size_t definition

lua: replace src/Makefile to build Lua as a Syslinux COM32 module

Start with the src/Makefile in Syslinux before importing Lua 5.2.2.
Refresh the core Lua object list based on the original Makefile.
Add the new builtin library objects.
Temporarily remove the Syslinux extension library objects, as they
need adjustments for the new API.

lua: import Lua 5.2.2

Source downloaded from http://www.lua.org/ftp/lua-5.2.2.tar.gz.
The com32/lua directory now matches exactly the source distribution,
plus the various Syslinux extension modules and their documentation.

core, bios: Move __syslinux_shuffler_size to assembly

Putting the __syslinux_shuffler_size in a C file by itself caused it
to not get included in the core, causing failures.  Put it in assembly
to make sure it really gets included there.

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

shuffler: Actually get the shuffler size sanely

Actually get the shuffler size sanely.  At least for now, we are BIOS
only, so stub out the functions on EFI (they shouldn't even be
compiled for EFI; this stuff should be moved into some kind of
BIOS-only directory.)

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

Makefiles: export the firmware type as a cpp variable

Export the firmware type as a cpp variable so we can #ifdef.

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

core, bios: Incorrect detection of EDD in /core/fs/diskio_bios.c

DL register is not set to drive number when detecting EDD for drive,
so detection may fail.

Signed-off-by: Andy Alex <andy at r-tt.com>

Merge branch 'syslinux-5.xx'

Resolved Conflicts:
core/init.c

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

bios: Don't truncate memory size needed to 16 bits

We can't truncate the memory size needed to 16 bits *before* we
convert it to kilobytes...

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

bios: Fix lowmem check

The lowmem check was backwards and would report error when there
wasn't any.  This was missed because the Ctrl key check was *also*
backwards and so the check would only be done when Ctrl was held down.

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

Makefile.private: archive the gnu-efi submodule, too.

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

mk/efi.mk: Add libefi.a dependency

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

efi: fix up gnu-efi build

This eliminates the forking and pushes the build under the efi32/ and
 efi64/ object directories eliminating a build race and preventing the
 build from breaking as gnu-efi takes a few seconds to complete.

Depends on gnu-efi commit 52d88dd

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

gnu-efi: update to ab54e2b4

This allows gnu-efi to be built outside the source tree properly.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Makefile: make "make spotless" actually work

The other cleaning targets might still have issues...

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

efi: We need -m32/-m64 even when processing assembly

We need to specify -m32/-m64 even when we are processing assembly.

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

mk/efi.mk: Disable stack protector

Disable the stack protector if on by default.

Copied from https://bugs.gentoo.org/show_bug.cgi?id=495146.

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

efi: Useless relocation section in PE file

There is no need to have a relocation section that nothing points at.
The image is still seen as relocatable as long as the Characteristics of
the FileHeader do not say otherwise.

Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

efi: PE file size differ from in-memory size

PE headers code_sz and image_sz indicate more or less, the size of the
file and the size of the in-memory image. They are now given the right
value.

In the ELF format, only the program headers are reliable to determine
the actually needed part of the file and the in-memory size.

The .bss section should always be marked as NOLOAD for ld since its
content shouldn't be included into the binary file.

Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>