[igt-dev] [PATCH i-g-t v9 10/16] Verify execbuf fails with stale PXP context after teardown
Rodrigo Vivi
rodrigo.vivi at intel.com
Fri Sep 10 19:12:07 UTC 2021
On Fri, Sep 10, 2021 at 07:04:13PM +0000, Teres Alexis, Alan Previn wrote:
>
> On Fri, 2021-09-10 at 14:53 -0400, Rodrigo Vivi wrote:
> > On Thu, Sep 02, 2021 at 10:30:37AM -0700, Alan Previn wrote:
> > > Add a subtest to verify that reusing a stale protected context
> > > in a gem_execbuff after a teardown fails with -EIO error because
> > > it's banned. Trigger the teardown via the pxp invalidation
> > > debugfs that simulates a HW teardown IRQ.
> > >
> > > NOTE: The end-to-end architecture requirement includes that
> > > any break in the links of the PXP sessions needs to trigger a
> > > full teardown and the application needs to be made aware of that
> > > allowing it to re-establish the end-to-end pipeline of buffers,
> > > contexts and renders again if it chooses to. This stricter
> > > behavior targets only contexts created with PXP enabled.
> > >
> > > Signed-off-by: Alan Previn <alan.previn.teres.alexis at intel.com>
> > > ---
> > > lib/intel_batchbuffer.c | 2 +-
> > > lib/intel_batchbuffer.h | 3 +
> > > tests/i915/gem_pxp.c | 151 ++++++++++++++++++++++++++++++++++++++++
> > > 3 files changed, 155 insertions(+), 1 deletion(-)
> > >
> > > +static void test_pxp_stale_ctx_execution(int i915)
> > > +{
> > > + int ret;
> > > + struct simple_exec_assets data = {0};
> > > +
> > > + /*
> > > + * Use normal buffers for testing for invalidation
> > > + * of protected contexts to ensure kernel is catching
> > > + * the invalidated context (not buffer)
> > > + */
> > > + prepare_exec_assets(i915, &data, true, false);
> > > + ret = gem_execbuf_flush_store_dw(i915, data.ibb, data.ctx, data.fencebuf);
> > > + igt_assert(ret == 0);
> > > +
> > > + trigger_pxp_debugfs_forced_teardown(i915);
> > > +
> > > + ret = gem_execbuf_flush_store_dw(i915, data.ibb, data.ctx, data.fencebuf);
> > > + igt_assert_f((ret == -EIO), "Executing stale pxp context didn't fail with -EIO\n");
> >
> > I believe you meant the opposite right?
> > other then that, the patch looks good to me...
>
> No, I actually DID want to ensure I get the -EIO failure when executing with stale PXP and so the error
> does read correctly. I can modify it to "Executing stale pxp was expected to fail with -EIO\n" if you
> like. Can i still get a conditional R-v-b at least? (i do have to double check that the error type hasnt
> changed - this had changed a few times on the kernel side and i need to check the latest rev today).
Sorry man...
I need coffee and a break... but before:
Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
>
> ...alan
More information about the igt-dev
mailing list