[PATCH i-g-t v2 resend 2/2] runner/executor: Integrate igt_kmemleak scans

[Cavitt, Jonathan]

-----Original Message-----
From: Peter Senna Tschudin <peter.senna at linux.intel.com> 
Sent: Monday, January 27, 2025 7:28 AM
To: igt-dev at lists.freedesktop.org
Cc: Peter Senna Tschudin <peter.senna at linux.intel.com>; Rodrigo.Siqueira at amd.com; Gandi, Ramadevi <ramadevi.gandi at intel.com>; Knop, Ryszard <ryszard.knop at intel.com>; Lattannavar, Sameer <sameer.lattannavar at intel.com>; De Marchi, Lucas <lucas.demarchi at intel.com>; Saarinen, Jani <jani.saarinen at intel.com>; Piecielska, Katarzyna <katarzyna.piecielska at intel.com>; Roper, Matthew D <matthew.d.roper at intel.com>; Taylor, Clinton A <clinton.a.taylor at intel.com>; Yu, Jianshui <jianshui.yu at intel.com>; Vivekanandan, Balasubramani <balasubramani.vivekanandan at intel.com>; Cavitt, Jonathan <jonathan.cavitt at intel.com>
Subject: [PATCH i-g-t v2 resend 2/2] runner/executor: Integrate igt_kmemleak scans
> 
> Modifies igt_runner to include calls to igt_kmemleak(). Kmemleak
> scanning is disabled by default, so add command line options to
> igt_runner to enable kmemleak scanning:  -k, -k<option>, --kmemleak,
> --kmemleak=<option>. The options are:
>  - once: run one kmemleak scan after all tests from the test-list
>          complete. This is the default when not using option.
>  - each: run one kmemleak scan after each test completes.
> 
> If kmemleaks are found they will be saved to the igt_runner results
> directory in a file named kmemleak.txt.
> 
> Updates serialize_settings() and read_settings_from_file() to save
> and restore igt_runner settings to and from disk. This is used when
> calling igt_runner with '--dry-run' and then by calling igt_resume
> instead of igt_runner.
> 
> Updates unit testing for igt_runner to test that:
>  - Kmemleak scans are disabled by default
>  - Kmemleak scans  be enabled by command line arguments
>  - The choice about kmemleaks being enabled or not is saved to disk
>    and restored from disk
> 
> CC: Rodrigo.Siqueira at amd.com
> CC: ramadevi.gandi at intel.com
> CC: ryszard.knop at intel.com
> CC: sameer.lattannavar at intel.com
> CC: lucas.demarchi at intel.com
> CC: jani.saarinen at intel.com
> CC: katarzyna.piecielska at intel.com
> CC: matthew.d.roper at intel.com
> CC: clinton.a.taylor at intel.com
> CC: jianshui.yu at intel.com
> CC: balasubramani.vivekanandan at intel.com
> 
> Reviewed-by: jonathan.cavitt at intel.com

By the way, it's:
"Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>"
You need more than just the email address here.
-Jonathan Cavitt

> Signed-off-by: Peter Senna Tschudin <peter.senna at linux.intel.com>

[...]

> ---
> -- 
> 2.34.1
> 
>