Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-13 Thread Janusz Krzysztofik
On Monday, 13 February 2023 10:51:39 CET Zbigniew Kempczyński wrote: > On Fri, Feb 10, 2023 at 10:33:21PM +0100, Janusz Krzysztofik wrote: > > On Thursday, 9 February 2023 20:32:31 CET Janusz Krzysztofik wrote: > > > If any of *-without-i915 subtests fails or skips for any reason, it may > > >

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-13 Thread Zbigniew Kempczyński
On Fri, Feb 10, 2023 at 10:33:21PM +0100, Janusz Krzysztofik wrote: > On Thursday, 9 February 2023 20:32:31 CET Janusz Krzysztofik wrote: > > If any of *-without-i915 subtests fails or skips for any reason, it may > > leave the i915 module unloaded while keeping our device list populated > > with

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-10 Thread Janusz Krzysztofik
On Thursday, 9 February 2023 20:32:31 CET Janusz Krzysztofik wrote: > If any of *-without-i915 subtests fails or skips for any reason, it may > leave the i915 module unloaded while keeping our device list populated > with initially collected data. In a follow up igt_fixture section we then > try

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-10 Thread Kamil Konieczny
Hi Janusz, On 2023-02-10 at 15:21:20 +0100, Janusz Krzysztofik wrote: > On Friday, 10 February 2023 15:02:59 CET Kamil Konieczny wrote: > > Hi Janusz, > > > > I have one nit, see below. > > > > On 2023-02-09 at 20:32:31 +0100, Janusz Krzysztofik wrote: > > > If any of *-without-i915 subtests

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-10 Thread Janusz Krzysztofik
On Friday, 10 February 2023 15:02:59 CET Kamil Konieczny wrote: > Hi Janusz, > > I have one nit, see below. > > On 2023-02-09 at 20:32:31 +0100, Janusz Krzysztofik wrote: > > If any of *-without-i915 subtests fails or skips for any reason, it may > > leave the i915 module unloaded while keeping

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-10 Thread Kamil Konieczny
Hi Janusz, I have one nit, see below. On 2023-02-09 at 20:32:31 +0100, Janusz Krzysztofik wrote: > If any of *-without-i915 subtests fails or skips for any reason, it may > leave the i915 module unloaded while keeping our device list populated > with initially collected data. In a follow up

[Intel-gfx] [PATCH i-g-t] tests/i915_suspend: Free device list after *-without-i915 subtests

2023-02-09 Thread Janusz Krzysztofik
If any of *-without-i915 subtests fails or skips for any reason, it may leave the i915 module unloaded while keeping our device list populated with initially collected data. In a follow up igt_fixture section we then try to reopen the device. If the test has been executed with a device filter