[igt-dev] [PATCH i-g-t] tests/kms_hdr: Fix failure to read output_bpc on AMD hardware

[Petri Latvala]

On Sat, Oct 02, 2021 at 11:15:11AM -0400, Rodrigo Siqueira wrote:
> Hi Lakshminarayana and Petri,
> 
> I noticed that the CI was not triggered for this patch [1]. Do you know
> why? Is it possible to trigger it?
> 
> 1. https://patchwork.freedesktop.org/series/95313/

CI was down for a bit due to server cert issues, testing that and the
other affected series happening in a bit.


-- 
Petri Latvala


> 
> Thanks
> Siqueira
> 
> On 10/01, Stylon Wang wrote:
> > The first commit with TEST_ONLY flag in bpc-switch subtests
> > could in some cases cause reading of output_bpc on connectors
> > to fail on AMD hardware.
> > 
> > Use ALLOW_MODESET to prevent this from happening.
> > 
> > Signed-off-by: Stylon Wang <stylon.wang at amd.com>
> > ---
> >  tests/kms_hdr.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c
> > index 5b8529c8..a4983b9a 100644
> > --- a/tests/kms_hdr.c
> > +++ b/tests/kms_hdr.c
> > @@ -217,7 +217,7 @@ static void test_bpc_switch_on_output(data_t *data, igt_output_t *output,
> >  		igt_plane_set_fb(data->primary, &afb);
> >  		igt_plane_set_size(data->primary, data->w, data->h);
> >  		igt_output_set_prop_value(data->output, IGT_CONNECTOR_MAX_BPC, 8);
> > -		ret = igt_display_try_commit_atomic(display, DRM_MODE_ATOMIC_TEST_ONLY, NULL);
> > +		ret = igt_display_try_commit_atomic(display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
> >  		if (!ret) {
> >  			data->w = afb.width;
> >  			data->h = afb.height;
> > -- 
> > 2.32.0
> > 
> 
> -- 
> Rodrigo Siqueira
> https://siqueira.tech