[syslinux] [PATCH 1/1] core: Add a check at ldlinux.sys build time.
H. Peter Anvin
hpa at zytor.com
Fri Jul 26 12:58:28 PDT 2013
On 07/26/2013 08:10 AM, Matt Fleming wrote:
> On Wed, 24 Jul, at 08:05:16AM, Raphael S.Carvalho wrote:
>> Check if ldlinux.sys is larger than 64k at build time.
>>
>> Signed-off-by: Raphael S.Carvalho <raphael.scarv at gmail.com>
>> ---
>> core/Makefile | 5 +++--
>> core/ldlinux_limit.pl | 31 +++++++++++++++++++++++++++++++
>> 2 files changed, 34 insertions(+), 2 deletions(-)
>> create mode 100644 core/ldlinux_limit.pl
>
> Thanks, this looks good, though the patch was whitespace damaged. I also
> expanded on the commit message to explain *why* 64K is the limit we use.
>
> I'll push this out once I've shrunk ldlinux.sys down to size.
>
The test is wrong: we need it to fit into 64K *including* the boot
sector and two copies of the ADV. Currently the ADV is 512 bytes, but
we probably need to make it 4K.
-hpa
More information about the Syslinux
mailing list