Re: [igt-dev] [PATCH i-g-t] kms_content_protection: Disable authentication when failed

[Ramalingam C <ramalingam.c@intel.com>]

On 2019-04-16 at 10:54:43 +0200, Daniel Vetter wrote:
> On Tue, Apr 16, 2019 at 11:12:30AM +0300, Martin Peres wrote:
> > On 11/04/2019 14:23, Ramalingam C wrote:
> > > When content protection authentication is failed in kernel after all
> > > requried retries, before declaring the test failure, set the content
> > > protection to UNDESIRED state.
> > > 
> > > This will avoid the HDCP authentication attempts in subsequent modesets
> > > from other tests.
> > > 
> > > Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110376
> 
> exit handler would be another option, but they're also not without issues.
> 
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Martin and Daniel, 

Thanks for the review. Change is merged in upstream.

-Ram
> 
> > > ---
> > >  tests/kms_content_protection.c | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > > 
> > > diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c
> > > index 7fc8542e769a..ae6ab497ea21 100644
> > > --- a/tests/kms_content_protection.c
> > > +++ b/tests/kms_content_protection.c
> > > @@ -212,6 +212,10 @@ static void test_cp_enable_with_retry(igt_output_t *output,
> > >  		if (!ret && --retry)
> > >  			igt_debug("Retry (%d/2) ...\n", 3 - retry);
> > >  	} while (retry && !ret);
> > > +
> > > +	if (!ret)
> > > +		test_cp_disable(output, s);
> > > +
> > >  	igt_assert_f(ret, "Content Protection not enabled\n");
> > >  }
> > >  
> > > 
> > _______________________________________________
> > igt-dev mailing list
> > igt-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> 
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev