[PATCH i-g-t] lib|tests: Don't dereference engine outside for_each_ctx_engine macro

Andi Shyti andi.shyti at linux.intel.com
Wed Aug 14 09:34:09 UTC 2024


Hi Peter,

On Wed, Aug 14, 2024 at 09:11:22AM +0200, Peter Senna Tschudin wrote:
> On 13.08.2024 21:51, Andi Shyti wrote:
> > On Tue, Aug 13, 2024 at 04:12:30PM +0200, Peter Senna Tschudin wrote:
> >> On 13.08.2024 15:56, Zbigniew Kempczyński wrote:
> >>> On Tue, Aug 13, 2024 at 03:36:50PM +0200, Peter Senna Tschudin wrote:
> >>>> Please do not apply this patch. I am working on a proper fix.
> >>>
> >>> Applying fix stops you from working on a your fix?
> >>
> >> Please do not fix a pothole just before I replace the road.
> > 
> > The patch is correct and Zbigniew's fix looks straight forward.
> > 
> > What is your fix aiming at? Can it go on top of this one?
> 
> Please ignore this. I have reviewed and compile tested the patch. Sorry for the noise.

if you have more fixes, please send them.

I was the originator of the for_each_ctx_engine() and I knew that
it was a big monster that needed some cleanups.

Thanks,
Andi


More information about the igt-dev mailing list