[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