[syslinux] [PATCH v2] extlinux: Make sure ldlinux.sys file is installed

Matt Fleming matt at console-pimps.org
Thu Jan 10 02:52:29 PST 2013


On Wed, 2013-01-09 at 21:02 -0300, Paulo Alcantara wrote:
> On Wed, Jan 9, 2013 at 6:36 PM, H. Peter Anvin <hpa at zytor.com> wrote:
> 
> > Actually it turns out that fsync() doesn't guarantee that the directory
> > entry has hit disk.  syncfs() is quite new (Linux 2.6.39) so it might be
> > better to just use sync() here.  Bootloader installation isn't too
> > horribly common anyway.
> 
> Yeah, you're right. In the first time I thought that making
> ldlinux.sys flushed out to disk would force the filesystem to create
> the directory entry and flush it out too - however it might get
> created on memory and kept there - whereas ldlinux.sys normally on
> disk.
> 
> Matt, wouldn't you mind to make a patch that uses sync() there ?

Sure, I've done that now.

-- 
Matt Fleming, Intel Open Source Technology Center



More information about the Syslinux mailing list