[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] squash4: fix handling of fragments and sparse files

From: Carlo Caione
Subject: Re: [PATCH] squash4: fix handling of fragments and sparse files
Date: Sun, 19 Feb 2017 10:46:29 +0100

On Sat, Feb 18, 2017 at 9:16 AM, Andrei Borzenkov <address@hidden> wrote:
> 1. Do not assume block list and fragment are mutually exclusive. Squash
> can pack file tail as fragment (unless -no-fragments is specified); so
> check read offset and read either from block list or from fragments as
> appropriate.
> 2. Support sparse files with zero blocks.
> 3. Fix fragment read - frag.offset is absolute fragment position,
> not offset relative to ino.chunk.
> Reported and tested by Carlo Caione <address@hidden>
> ---
> @Vladimir: we need regression tests for both of these cases. We could real 
> small
> files only by accident - block list was zero, so it appeared to work.
> @Carlo, please test. I'm surprised it worked for you even without fragments as
> your file is sparse and grub immediately choked on the first zero block.

Just tested. And it works fine for me.


Carlo Caione  |  +39.340.80.30.096  |  Endless

reply via email to

[Prev in Thread] Current Thread [Next in Thread]