Re: [PATCH igt] igt/kms_force_connector_basic: Clear any previous connector override

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewed-by: Marta Lofstedt <marta.lofstedt@xxxxxxxxx>

> -----Original Message-----
> From: Chris Wilson [mailto:chris@xxxxxxxxxxxxxxxxxx]
> Sent: Monday, February 26, 2018 12:44 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: igt-dev@xxxxxxxxxxxxxxxxxxxxx; Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>;
> Lofstedt, Marta <marta.lofstedt@xxxxxxxxx>
> Subject: [PATCH igt] igt/kms_force_connector_basic: Clear any previous
> connector override
> 
> When searching for a VGA connector to use to test overriding the connector
> status, we require the system to have a disconnected VGA connector, but if
> a previous test left an override inplace, that may not exist. Before we check
> whether the connector is attached to real HW, first reset the connector
> status override so that we always get the actual HW result.
> 
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: Marta Lofstedt <marta.lofstedt@xxxxxxxxx>
> ---
>  tests/kms_force_connector_basic.c | 24 +++++++++++++++++-------
>  1 file changed, 17 insertions(+), 7 deletions(-)
> 
> diff --git a/tests/kms_force_connector_basic.c
> b/tests/kms_force_connector_basic.c
> index d2c804c8..c51b5cdf 100644
> --- a/tests/kms_force_connector_basic.c
> +++ b/tests/kms_force_connector_basic.c
> @@ -86,29 +86,39 @@ int main(int argc, char **argv)
>  				    opt_handler,
> NULL);
> 
>  	igt_fixture {
> +		unsigned vga_connector_id = 0;
> +
>  		drm_fd =
> drm_open_driver_master(DRIVER_INTEL);
>  		res = drmModeGetResources(drm_fd);
>  		igt_assert(res);
> 
>  		/* find the vga connector */
>  		for (int i = 0; i < res->count_connectors; i++) {
> -
>  			vga_connector =
> drmModeGetConnectorCurrent(drm_fd,
> 
> 			   res->connectors[i]);
> 
>  			if (vga_connector-
> >connector_type == DRM_MODE_CONNECTOR_VGA) {
> -				start_n_modes =
> vga_connector->count_modes;
> -				start_connection =
> vga_connector->connection;
> -				break;
> +				/* Ensure that no
> override was left in place */
> +
> 	kmstest_force_connector(drm_fd,
> +
> 		vga_connector,
> +
> 		FORCE_CONNECTOR_UNSPECIFIED);
> +
> +				/* only use the first
> connector */
> +				if
> (!vga_connector_id)
> +
> 	vga_connector_id = res->connectors[i];
>  			}
> 
> 
> 	drmModeFreeConnector(vga_connector);
> -
> -			vga_connector = NULL;
>  		}
> 
> -		igt_require(vga_connector);
> +		igt_require(vga_connector_id);
> +
> +		/* Reacquire status after clearing any previous
> overrides */
> +		vga_connector =
> drmModeGetConnector(drm_fd, vga_connector_id);
>  		igt_skip_on(vga_connector->connection ==
> DRM_MODE_CONNECTED);
> +
> +		start_n_modes = vga_connector-
> >count_modes;
> +		start_connection = vga_connector->connection;
>  	}
> 
>  	igt_subtest("force-load-detect") {
> --
> 2.16.2

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux