On Sat, Feb 18, 2017 at 9:16 AM, Andrei Borzenkov <arvidj...@gmail.com> 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 <ca...@endlessm.com>
>
> ---
>
> @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.

Thanks,


-- 
Carlo Caione  |  +39.340.80.30.096  |  Endless

_______________________________________________
Bug-grub mailing list
Bug-grub@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-grub

Reply via email to