[U-Boot] [PATCH v2 3/7] test: fs: run fsck after on fs image after the tests are run

[Tom Rini]

On Fri, Feb 01, 2019 at 03:33:36PM +0100, Jean-Jacques Hiblot wrote:

> This is to check the integrity of the FS after the test operations.
> This is useful to make sure that the operations are implemented properly,
> and are not going to create silent corruptions.
> Currently only the integrity of EXT4 filesystems is checked.
> 
> Signed-off-by: Jean-Jacques Hiblot <jjhiblot at ti.com>
> 
> ---
> 
> Changes in v2:
> - Add a FS integrity check at the end of the FS tests
> 
>  test/py/tests/test_fs/conftest.py | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/test/py/tests/test_fs/conftest.py b/test/py/tests/test_fs/conftest.py
> index 745ed0ed38..e742cda662 100644
> --- a/test/py/tests/test_fs/conftest.py
> +++ b/test/py/tests/test_fs/conftest.py
> @@ -216,6 +216,15 @@ def mount_fs(fs_type, device, mount_point):
>      except CalledProcessError:
>          raise
>  
> +
> +def fsck(img, fs_type):
> +    try:
> +        if fs_type == 'ext4':
> +            check_call('fsck.ext4 -n -f %s' % img, shell=True)
> +    except CalledProcessError:
> +        raise
> +
> +

Why don't we just call 'fsck -f -n ...' and check all filesystems?
Force and "make no changes" are both general fsck options and not
filesystem specific.  And if we're messing up FAT as part of the tests
we should know that now :)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190201/f02b6f9d/attachment.sig>