[PATCH i-g-t v4 1/2] lib/igt_kmemleak: library to interact with kmemleak

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Feb 12 13:41:37 UTC 2025


Hi Peter,
On 2025-02-12 at 14:12:44 +0100, Peter Senna Tschudin wrote:
> 
> 
> On 12.02.2025 11:01, Zbigniew Kempczyński wrote:
> > On Wed, Feb 12, 2025 at 10:18:17AM +0100, Peter Senna Tschudin wrote:

[...]

> >> Resume: I prefer to wait until the initial feedback from CI and developers.
> >> I will be happy to move this to runner later if we decide to not walk the
> >> unknown road of trying to catch as many transient leaks as we can.
> >>
> >> That being said, let me know how to move forward.
> > 
> > Question is to CI folks, for me this scan should be executed as
> > dedicated CI run with kmemleak on. And I repeat - I don't see reason
> > we should add this support to the tests, as few line script which
> > wraps around test by kmemleak clear/scan is enough. We already have
> > scripts directory so it is good place for such wrapper.
> 
> So I am moving the code to the runner directory, and using printf() instead
> of igt_warn(). Any other request for the series?
> 
> [...]
> 

or fprintf() or write() so you could write to a file instead
of stdout, as stdout isn't collected now by runner itself (it
goes to igt_runnerNN.txt but that is another story).

I also want to see ack from CI, on ML or in any other way.

Regards,
Kamil



More information about the igt-dev mailing list