[PATCH v2 27/28] fs/squashfs: sqfs_read: fragmented files are not supported

[Simon Glass]

Hi Joao,

On Wed, 25 Nov 2020 at 01:58, Richard Genoud <richard.genoud at posteo.net> wrote:
>
> Hi,
>
> Le 20/11/2020 à 02:35, Tom Rini a écrit :
> > On Tue, Nov 03, 2020 at 12:11:25PM +0100, Richard Genoud wrote:
> >
> >> The code for reading a fragmented file is not functionnal.
> >> It's better to signal this to the user.
> >>
> >> Signed-off-by: Richard Genoud <richard.genoud at posteo.net>
> >
> > This change causes the test.py squashfs tests to fail.  I am unsure if
> > the problem is with the tests or this exposing further problems in the
> > code.
> Actually, reading a fragmented file doesn't work.
> The test only check if the file is read, but not it's content.
>
> With this following patch, we'll see that the file content is not the same :
>
>
>  From 68f87301c059aaae8e90e42fbec9b560aee0c6eb Mon Sep 17 00:00:00 2001
> From: Richard Genoud <richard.genoud at posteo.net>
> Date: Tue, 24 Nov 2020 17:45:07 +0100
> Subject: [PATCH] test/py: SquashFS: Check if loaded file is corrupted
>
> After loading the file in memory, its content should be checked for
> errors.
>
> Signed-off-by: Richard Genoud <richard.genoud at posteo.net>
> ---
>   test/py/tests/test_fs/test_squashfs/sqfs_common.py    | 5 ++++-
>   test/py/tests/test_fs/test_squashfs/test_sqfs_load.py | 6 +++++-
>   2 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/test/py/tests/test_fs/test_squashfs/sqfs_common.py b/test/py/tests/test_fs/test_squashfs/sqfs_common.py
> index c96f92c1d8f..a7673c73762 100644
> --- a/test/py/tests/test_fs/test_squashfs/sqfs_common.py
> +++ b/test/py/tests/test_fs/test_squashfs/sqfs_common.py

This test works the first time I run it but fails the second time,
since the directory already exists. This makes it necessary to disable
the test for development.

It also uses the wrong quoting style - we have settled on a single
quote by default in U-Boot.

Finally, the tests and some functions need comments about what they do
and what the arguments are.

Please can you take a look?

Thanks,
Simon