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

H. Peter Anvin hpa at zytor.com
Sun Jan 6 20:36:56 PST 2013


On 01/06/2013 02:11 PM, Paulo Alcantara wrote:
> Due to delayed allocation feature on XFS filesystems, a write is not
> guaranteed to flushed out to the underlying file system after crash or
> something else - so we need to make sure that ldlinux.sys is installed
> correctly in the mounted filesystem.
>
> Signed-off-by: Paulo Alcantara <pcacjr at zytor.com>
> ---
>   extlinux/main.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/extlinux/main.c b/extlinux/main.c
> index 0204caa..f7eeeb7 100644
> --- a/extlinux/main.c
> +++ b/extlinux/main.c
> @@ -687,6 +687,8 @@ static int xfs_install_file(const char *path, int devfd, struct stat *rst)
>       }
>
>       close(dirfd);
> +
> +    syncfs(fd);
>       close(fd);
>
>       return 0;
>

Wouldn't fsync() make more sense?

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.



More information about the Syslinux mailing list