Re: [PATCH 10/12] tests: Better handling of valgrind errors saving blobs

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



On 26 March 2018 at 07:25, David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> wrote:
> Currently we have 3 valgrind suppression files in the tests, all of which
> are to handle memcheck errors that originate from saving entire buffers
> containing blobs where the gaps between sub-blocks might not be
> initialized.
>
> We can more simply suppress those errors by having the save_blob() helper
> use valgrind's client interface to mark the data as initialized before we
> write it out.
>
> Signed-off-by: David Gibson <david@xxxxxxxxxxxxxxxxxxxxx>
> ---
>  tests/mangle-layout.supp |  7 -------
>  tests/open_pack.supp     |  7 -------
>  tests/sw_tree1.supp      | 18 ------------------
>  tests/testutils.c        | 15 +++++++++++++--
>  4 files changed, 13 insertions(+), 34 deletions(-)
>  delete mode 100644 tests/mangle-layout.supp
>  delete mode 100644 tests/open_pack.supp
>  delete mode 100644 tests/sw_tree1.supp
>

Reviewed-by: Simon Glass <sjg@xxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe devicetree-compiler" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux