On Wed, Aug 16, 2023 at 01:33:29PM -0700, Cavitt, Jonathan wrote: > -----Original Message----- > From: Roper, Matthew D <[email protected]> > Sent: Tuesday, August 15, 2023 12:29 PM > To: Cavitt, Jonathan <[email protected]> > Cc: [email protected]; Dutt, Sudeep <[email protected]>; > [email protected]; Gupta, saurabhg <[email protected]>; > Shyti, Andi <[email protected]>; Summers, Stuart > <[email protected]>; Das, Nirmoy <[email protected]>; Belgaumkar, > Vinay <[email protected]>; Winiarski, Michal > <[email protected]> > Subject: Re: [PATCH 1/2] drm/i915/selftests: Align igt_spinner_create_request > with hangcheck > > > > On Tue, Aug 15, 2023 at 09:53:44AM -0700, Jonathan Cavitt wrote: > > > Align igt_spinner_create_request with the hang_create_request > > > implementation in selftest_hangcheck.c. > > > > > > Signed-off-by: Jonathan Cavitt <[email protected]> > > > > Reviewed-by: Matt Roper <[email protected]> > > > > > > For the second patch in the series, the general direction looks good to > > me, but I'm not familiar enough with the spinner implementation and > > context handling to do a detailed review there. Hopefully someone more > > familiar with that code can take a look. > > > > > Thank you for the RB! If it's not too much trouble, could you please push > this patch (independent of the rest of the series) upstream? Sudeep would > like to see this patch upstreamed as soon as possible to get MTL prepared > for PV. I figured I'd ask you first since you provided the RB. > > Of course, if you're busy, that's understandable as well.
Since you'd previously sent this patch by itself, I used the CI results from that submission, which were clean aside from false positive hits of fdo#9121. Applied to drm-intel-gt-next. Thanks for the patch. Matt > -Jonathan Cavitt > > > > > > Matt > > > > > --- > > > drivers/gpu/drm/i915/selftests/igt_spinner.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/i915/selftests/igt_spinner.c > > > b/drivers/gpu/drm/i915/selftests/igt_spinner.c > > > index 0f064930ef11..8c3e1f20e5a1 100644 > > > --- a/drivers/gpu/drm/i915/selftests/igt_spinner.c > > > +++ b/drivers/gpu/drm/i915/selftests/igt_spinner.c > > > @@ -179,6 +179,9 @@ igt_spinner_create_request(struct igt_spinner *spin, > > > > > > *batch++ = arbitration_command; > > > > > > + memset32(batch, MI_NOOP, 128); > > > + batch += 128; > > > + > > > if (GRAPHICS_VER(rq->i915) >= 8) > > > *batch++ = MI_BATCH_BUFFER_START | BIT(8) | 1; > > > else if (IS_HASWELL(rq->i915)) > > > -- > > > 2.25.1 > > > > > > > -- > > Matt Roper > > Graphics Software Engineer > > Linux GPU Platform Enablement > > Intel Corporation > > -- Matt Roper Graphics Software Engineer Linux GPU Platform Enablement Intel Corporation
