Re: [PATCH igt] tests: add pc8

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

 



On Mon, Aug 19, 2013 at 01:20:02PM -0300, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> 
> This test chekcs our code that enables Package C8+. The environment
> requirements for this test are quite complicated:
>   - The machine needs to be properly configured to reach PC8+ when
>     possible, which means all the power management policies and device
>     drivers must be properly configured.
>   - You need at least one output connected.
>   - You need the /dev/cpu/0/msr file available.
>   - You need the /dev/i2c-* files available.
> 
> v2: - Many many changes due to the latest review comments and rebase.
> 
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>

Applied and pushed with a small fixup patch on top to convert your new
testcase to the latest igt infrastructure changes.

Thanks, Daniel

> ---
>  tests/.gitignore  |   1 +
>  tests/Makefile.am |   1 +
>  tests/pc8.c       | 804 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 806 insertions(+)
>  create mode 100644 tests/pc8.c
> 
> diff --git a/tests/.gitignore b/tests/.gitignore
> index b5eb7a5..19f827a 100644
> --- a/tests/.gitignore
> +++ b/tests/.gitignore
> @@ -93,6 +93,7 @@ getversion
>  kms_flip
>  kms_render
>  kms_setmode
> +pc8
>  prime_nv_api
>  prime_nv_pcopy
>  prime_nv_test
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index c724829..370a700 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -45,6 +45,7 @@ TESTS_progs_M = \
>  	kms_render \
>  	kms_setmode \
>  	$(NOUVEAU_TESTS_M) \
> +	pc8 \
>  	prime_self_import \
>  	$(NULL)
>  
> diff --git a/tests/pc8.c b/tests/pc8.c
> new file mode 100644
> index 0000000..06c54b7
> --- /dev/null
> +++ b/tests/pc8.c
> @@ -0,0 +1,804 @@
> +/*
> + * Copyright © 2013 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + * Authors:
> + *    Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
> + *
> + */
> +
> +#include <stdio.h>
> +#include <stdint.h>
> +#include <stdbool.h>
> +#include <string.h>
> +
> +#include <unistd.h>
> +#include <fcntl.h>
> +#include <dirent.h>
> +#include <sys/ioctl.h>
> +#include <sys/types.h>
> +#include <sys/stat.h>
> +#include <linux/i2c.h>
> +#include <linux/i2c-dev.h>
> +
> +#include "drm.h"
> +#include "drmtest.h"
> +#include "intel_batchbuffer.h"
> +#include "intel_gpu_tools.h"
> +#include "i915_drm.h"
> +
> +#define MSR_PC8_RES	0x630
> +#define MSR_PC9_RES	0x631
> +#define MSR_PC10_RES	0x632
> +
> +#define MAX_CONNECTORS	32
> +#define MAX_ENCODERS	32
> +#define MAX_CRTCS	16
> +
> +int drm_fd, msr_fd;
> +struct mode_set_data ms_data;
> +
> +/* Stuff used when creating FBs and mode setting. */
> +struct mode_set_data {
> +	drmModeResPtr res;
> +	drmModeConnectorPtr connectors[MAX_CONNECTORS];
> +	drmModePropertyBlobPtr edids[MAX_CONNECTORS];
> +
> +	drm_intel_bufmgr *bufmgr;
> +	uint32_t devid;
> +};
> +
> +/* Stuff we query at different times so we can compare. */
> +struct compare_data {
> +	drmModeResPtr res;
> +	drmModeEncoderPtr encoders[MAX_ENCODERS];
> +	drmModeConnectorPtr connectors[MAX_CONNECTORS];
> +	drmModeCrtcPtr crtcs[MAX_CRTCS];
> +	drmModePropertyBlobPtr edids[MAX_CONNECTORS];
> +};
> +
> +struct compare_registers {
> +	/* We know these are lost */
> +	uint32_t arb_mode;
> +	uint32_t tilectl;
> +
> +	/* Stuff touched at init_clock_gating, so we can make sure we
> +	 * don't need to call it when reiniting. */
> +	uint32_t gen6_ucgctl2;
> +	uint32_t gen7_l3cntlreg1;
> +	uint32_t transa_chicken1;
> +
> +	uint32_t deier;
> +	uint32_t gtier;
> +
> +	uint32_t ddi_buf_trans_a_1;
> +	uint32_t ddi_buf_trans_b_5;
> +	uint32_t ddi_buf_trans_c_10;
> +	uint32_t ddi_buf_trans_d_15;
> +	uint32_t ddi_buf_trans_e_20;
> +};
> +
> +/* If the read fails, then the machine doesn't support PC8+ residencies. */
> +static bool supports_pc8_plus_residencies(void)
> +{
> +	int rc;
> +	uint64_t val;
> +
> +	rc = pread(msr_fd, &val, sizeof(uint64_t), MSR_PC8_RES);
> +	if (!rc == sizeof(val))
> +		return false;
> +	rc = pread(msr_fd, &val, sizeof(uint64_t), MSR_PC9_RES);
> +	if (!rc == sizeof(val))
> +		return false;
> +	rc = pread(msr_fd, &val, sizeof(uint64_t), MSR_PC10_RES);
> +	if (!rc == sizeof(val))
> +		return false;
> +
> +	return true;
> +}
> +
> +static uint64_t get_residency(uint32_t type)
> +{
> +	int rc;
> +	uint64_t ret;
> +
> +	rc = pread(msr_fd, &ret, sizeof(uint64_t), type);
> +	igt_assert(rc == sizeof(ret));
> +
> +	return ret;
> +}
> +
> +static bool pc8_plus_residency_changed(unsigned int timeout_sec)
> +{
> +	unsigned int i;
> +	uint64_t res_pc8, res_pc9, res_pc10;
> +	int to_sleep = 100 * 1000;
> +
> +	res_pc8 = get_residency(MSR_PC8_RES);
> +	res_pc9 = get_residency(MSR_PC9_RES);
> +	res_pc10 = get_residency(MSR_PC10_RES);
> +
> +	for (i = 0; i < timeout_sec * 1000 * 1000; i += to_sleep) {
> +		if (res_pc8 != get_residency(MSR_PC8_RES) ||
> +		    res_pc9 != get_residency(MSR_PC9_RES) ||
> +		    res_pc10 != get_residency(MSR_PC10_RES)) {
> +			return true;
> +		}
> +		usleep(to_sleep);
> +	}
> +
> +	return false;
> +}
> +
> +/* Checks not only if PC8+ is allowed, but also if we're reaching it.
> + * We call this when we expect this function to return quickly since PC8 is
> + * actually enabled, so the 30s timeout we use shouldn't matter. */
> +static bool pc8_plus_enabled(void)
> +{
> +	return pc8_plus_residency_changed(30);
> +}
> +
> +/* We call this when we expect PC8+ to be actually disabled, so we should not
> + * return until the 5s timeout expires. In other words: in the "happy case",
> + * every time we call this function the program will take 5s more to finish. */
> +static bool pc8_plus_disabled(void)
> +{
> +	return !pc8_plus_residency_changed(5);
> +}
> +
> +static void disable_all_screens(struct mode_set_data *data)
> +{
> +	int i, rc;
> +
> +	for (i = 0; i < data->res->count_crtcs; i++) {
> +		rc = drmModeSetCrtc(drm_fd, data->res->crtcs[i], -1, 0, 0,
> +				    NULL, 0, NULL);
> +		igt_assert(rc == 0);
> +	}
> +}
> +
> +static uint32_t create_fb(struct mode_set_data *data, int width, int height)
> +{
> +	struct kmstest_fb fb;
> +	cairo_t *cr;
> +	uint32_t buffer_id;
> +
> +	buffer_id = kmstest_create_fb(drm_fd, width, height, 32, 24, false,
> +				      &fb);
> +	cr = kmstest_get_cairo_ctx(drm_fd, &fb);
> +	kmstest_paint_test_pattern(cr, width, height);
> +	return buffer_id;
> +}
> +
> +static void enable_one_screen(struct mode_set_data *data)
> +{
> +	uint32_t crtc_id = 0, buffer_id = 0, connector_id = 0;
> +	drmModeModeInfoPtr mode = NULL;
> +	int i, rc;
> +
> +	for (i = 0; i < data->res->count_connectors; i++) {
> +		drmModeConnectorPtr c = data->connectors[i];
> +
> +		if (c->connection == DRM_MODE_CONNECTED && c->count_modes) {
> +			connector_id = c->connector_id;
> +			mode = &c->modes[0];
> +			break;
> +		}
> +	}
> +
> +	crtc_id = data->res->crtcs[0];
> +	buffer_id = create_fb(data, mode->hdisplay, mode->vdisplay);
> +
> +	igt_assert(crtc_id);
> +	igt_assert(buffer_id);
> +	igt_assert(connector_id);
> +	igt_assert(mode);
> +
> +	rc = drmModeSetCrtc(drm_fd, crtc_id, buffer_id, 0, 0, &connector_id,
> +			    1, mode);
> +	igt_assert(rc == 0);
> +}
> +
> +static drmModePropertyBlobPtr get_connector_edid(drmModeConnectorPtr connector,
> +						 int index)
> +{
> +	unsigned int i;
> +	drmModeObjectPropertiesPtr props;
> +	drmModePropertyBlobPtr ret = NULL;
> +
> +	props = drmModeObjectGetProperties(drm_fd, connector->connector_id,
> +					   DRM_MODE_OBJECT_CONNECTOR);
> +
> +	for (i = 0; i < props->count_props; i++) {
> +		drmModePropertyPtr prop = drmModeGetProperty(drm_fd,
> +							     props->props[i]);
> +
> +		if (strcmp(prop->name, "EDID") == 0) {
> +			igt_assert(prop->flags & DRM_MODE_PROP_BLOB);
> +			igt_assert(prop->count_blobs == 0);
> +			ret = drmModeGetPropertyBlob(drm_fd,
> +						     props->prop_values[i]);
> +		}
> +
> +		drmModeFreeProperty(prop);
> +	}
> +
> +	drmModeFreeObjectProperties(props);
> +	return ret;
> +}
> +
> +static void init_mode_set_data(struct mode_set_data *data)
> +{
> +	int i;
> +
> +	data->res = drmModeGetResources(drm_fd);
> +	igt_assert(data->res);
> +	igt_assert(data->res->count_connectors <= MAX_CONNECTORS);
> +
> +	for (i = 0; i < data->res->count_connectors; i++) {
> +		data->connectors[i] = drmModeGetConnector(drm_fd,
> +						data->res->connectors[i]);
> +		data->edids[i] = get_connector_edid(data->connectors[i], i);
> +	}
> +
> +	data->bufmgr = drm_intel_bufmgr_gem_init(drm_fd, 4096);
> +	data->devid = intel_get_drm_devid(drm_fd);
> +
> +	do_or_die(igt_set_vt_graphics_mode());
> +	drm_intel_bufmgr_gem_enable_reuse(data->bufmgr);
> +}
> +
> +static void fini_mode_set_data(struct mode_set_data *data)
> +{
> +	int i;
> +
> +	drm_intel_bufmgr_destroy(data->bufmgr);
> +
> +	for (i = 0; i < data->res->count_connectors; i++) {
> +		drmModeFreeConnector(data->connectors[i]);
> +		drmModeFreePropertyBlob(data->edids[i]);
> +	}
> +	drmModeFreeResources(data->res);
> +}
> +
> +static void get_drm_info(struct compare_data *data)
> +{
> +	int i;
> +
> +	data->res = drmModeGetResources(drm_fd);
> +	igt_assert(data->res);
> +
> +	igt_assert(data->res->count_connectors <= MAX_CONNECTORS);
> +	igt_assert(data->res->count_encoders <= MAX_ENCODERS);
> +	igt_assert(data->res->count_crtcs <= MAX_CRTCS);
> +
> +	for (i = 0; i < data->res->count_connectors; i++) {
> +		data->connectors[i] = drmModeGetConnector(drm_fd,
> +						data->res->connectors[i]);
> +		data->edids[i] = get_connector_edid(data->connectors[i], i);
> +	}
> +	for (i = 0; i < data->res->count_encoders; i++)
> +		data->encoders[i] = drmModeGetEncoder(drm_fd,
> +						data->res->encoders[i]);
> +	for (i = 0; i < data->res->count_crtcs; i++)
> +		data->crtcs[i] = drmModeGetCrtc(drm_fd, data->res->crtcs[i]);
> +}
> +
> +static void get_registers(struct compare_registers *data)
> +{
> +	intel_register_access_init(intel_get_pci_device(), 0);
> +	data->arb_mode = INREG(0x4030);
> +	data->tilectl = INREG(0x101000);
> +	data->gen6_ucgctl2 = INREG(0x9404);
> +	data->gen7_l3cntlreg1 = INREG(0xB0C1);
> +	data->transa_chicken1 = INREG(0xF0060);
> +	data->deier = INREG(0x4400C);
> +	data->gtier = INREG(0x4401C);
> +	data->ddi_buf_trans_a_1 = INREG(0x64E00);
> +	data->ddi_buf_trans_b_5 = INREG(0x64E70);
> +	data->ddi_buf_trans_c_10 = INREG(0x64EE0);
> +	data->ddi_buf_trans_d_15 = INREG(0x64F58);
> +	data->ddi_buf_trans_e_20 = INREG(0x64FCC);
> +	intel_register_access_fini();
> +}
> +
> +static void free_drm_info(struct compare_data *data)
> +{
> +	int i;
> +
> +	for (i = 0; i < data->res->count_connectors; i++) {
> +		drmModeFreeConnector(data->connectors[i]);
> +		drmModeFreePropertyBlob(data->edids[i]);
> +	}
> +	for (i = 0; i < data->res->count_encoders; i++)
> +		drmModeFreeEncoder(data->encoders[i]);
> +	for (i = 0; i < data->res->count_crtcs; i++)
> +		drmModeFreeCrtc(data->crtcs[i]);
> +
> +	drmModeFreeResources(data->res);
> +}
> +
> +#define COMPARE(d1, d2, data) igt_assert(d1->data == d2->data)
> +#define COMPARE_ARRAY(d1, d2, size, data) do { \
> +	for (i = 0; i < size; i++) \
> +		igt_assert(d1->data[i] == d2->data[i]); \
> +} while (0)
> +
> +static void assert_drm_resources_equal(struct compare_data *d1,
> +				       struct compare_data *d2)
> +{
> +	COMPARE(d1, d2, res->count_connectors);
> +	COMPARE(d1, d2, res->count_encoders);
> +	COMPARE(d1, d2, res->count_crtcs);
> +	COMPARE(d1, d2, res->min_width);
> +	COMPARE(d1, d2, res->max_width);
> +	COMPARE(d1, d2, res->min_height);
> +	COMPARE(d1, d2, res->max_height);
> +}
> +
> +static void assert_modes_equal(drmModeModeInfoPtr m1, drmModeModeInfoPtr m2)
> +{
> +	COMPARE(m1, m2, clock);
> +	COMPARE(m1, m2, hdisplay);
> +	COMPARE(m1, m2, hsync_start);
> +	COMPARE(m1, m2, hsync_end);
> +	COMPARE(m1, m2, htotal);
> +	COMPARE(m1, m2, hskew);
> +	COMPARE(m1, m2, vdisplay);
> +	COMPARE(m1, m2, vsync_start);
> +	COMPARE(m1, m2, vsync_end);
> +	COMPARE(m1, m2, vtotal);
> +	COMPARE(m1, m2, vscan);
> +	COMPARE(m1, m2, vrefresh);
> +	COMPARE(m1, m2, flags);
> +	COMPARE(m1, m2, type);
> +	igt_assert(strcmp(m1->name, m2->name) == 0);
> +}
> +
> +static void assert_drm_connectors_equal(drmModeConnectorPtr c1,
> +					drmModeConnectorPtr c2)
> +{
> +	int i;
> +
> +	COMPARE(c1, c2, connector_id);
> +	COMPARE(c1, c2, connector_type);
> +	COMPARE(c1, c2, connector_type_id);
> +	COMPARE(c1, c2, mmWidth);
> +	COMPARE(c1, c2, mmHeight);
> +	COMPARE(c1, c2, count_modes);
> +	COMPARE(c1, c2, count_props);
> +	COMPARE(c1, c2, count_encoders);
> +	COMPARE_ARRAY(c1, c2, c1->count_props, props);
> +	COMPARE_ARRAY(c1, c2, c1->count_encoders, encoders);
> +
> +	for (i = 0; i < c1->count_modes; i++)
> +		assert_modes_equal(&c1->modes[0], &c2->modes[0]);
> +}
> +
> +static void assert_drm_encoders_equal(drmModeEncoderPtr e1,
> +				      drmModeEncoderPtr e2)
> +{
> +	COMPARE(e1, e2, encoder_id);
> +	COMPARE(e1, e2, encoder_type);
> +	COMPARE(e1, e2, possible_crtcs);
> +	COMPARE(e1, e2, possible_clones);
> +}
> +
> +static void assert_drm_crtcs_equal(drmModeCrtcPtr c1, drmModeCrtcPtr c2)
> +{
> +	COMPARE(c1, c2, crtc_id);
> +}
> +
> +static void assert_drm_edids_equal(drmModePropertyBlobPtr e1,
> +				   drmModePropertyBlobPtr e2)
> +{
> +	if (!e1 && !e2)
> +		return;
> +	igt_assert(e1 && e2);
> +
> +	COMPARE(e1, e2, id);
> +	COMPARE(e1, e2, length);
> +
> +	igt_assert(memcmp(e1->data, e2->data, e1->length) == 0);
> +}
> +
> +static void compare_registers(struct compare_registers *d1,
> +			      struct compare_registers *d2)
> +{
> +	COMPARE(d1, d2, gen6_ucgctl2);
> +	COMPARE(d1, d2, gen7_l3cntlreg1);
> +	COMPARE(d1, d2, transa_chicken1);
> +	COMPARE(d1, d2, arb_mode);
> +	COMPARE(d1, d2, tilectl);
> +	COMPARE(d1, d2, arb_mode);
> +	COMPARE(d1, d2, tilectl);
> +	COMPARE(d1, d2, gtier);
> +	COMPARE(d1, d2, ddi_buf_trans_a_1);
> +	COMPARE(d1, d2, ddi_buf_trans_b_5);
> +	COMPARE(d1, d2, ddi_buf_trans_c_10);
> +	COMPARE(d1, d2, ddi_buf_trans_d_15);
> +	COMPARE(d1, d2, ddi_buf_trans_e_20);
> +}
> +
> +static void assert_drm_infos_equal(struct compare_data *d1,
> +				   struct compare_data *d2)
> +{
> +	int i;
> +
> +	assert_drm_resources_equal(d1, d2);
> +
> +	for (i = 0; i < d1->res->count_connectors; i++) {
> +		assert_drm_connectors_equal(d1->connectors[i],
> +					    d2->connectors[i]);
> +		assert_drm_edids_equal(d1->edids[i], d2->edids[i]);
> +	}
> +
> +	for (i = 0; i < d1->res->count_encoders; i++)
> +		assert_drm_encoders_equal(d1->encoders[i], d2->encoders[i]);
> +
> +	for (i = 0; i < d1->res->count_crtcs; i++)
> +		assert_drm_crtcs_equal(d1->crtcs[i], d2->crtcs[i]);
> +}
> +
> +static void blt_color_fill(struct intel_batchbuffer *batch,
> +			   drm_intel_bo *buf,
> +			   const unsigned int pages)
> +{
> +	const unsigned short height = pages/4;
> +	const unsigned short width = 4096;
> +
> +	BEGIN_BATCH(5);
> +	OUT_BATCH(COLOR_BLT_CMD | COLOR_BLT_WRITE_ALPHA | COLOR_BLT_WRITE_RGB);
> +	OUT_BATCH((3 << 24) | /* 32 Bit Color */
> +		  0xF0 | /* Raster OP copy background register */
> +		  0); /* Dest pitch is 0 */
> +	OUT_BATCH(width << 16 | height);
> +	OUT_RELOC(buf, I915_GEM_DOMAIN_RENDER, I915_GEM_DOMAIN_RENDER, 0);
> +	OUT_BATCH(rand()); /* random pattern */
> +	ADVANCE_BATCH();
> +}
> +
> +static void test_batch(struct mode_set_data *data)
> +{
> +	struct intel_batchbuffer *batch;
> +	int64_t timeout_ns = 1000 * 1000 * 1000 * 2;
> +	drm_intel_bo *dst;
> +	int i, rc;
> +
> +	dst = drm_intel_bo_alloc(data->bufmgr, "dst", (8 << 20), 4096);
> +
> +	batch = intel_batchbuffer_alloc(data->bufmgr,
> +					intel_get_drm_devid(drm_fd));
> +
> +	for (i = 0; i < 1000; i++)
> +		blt_color_fill(batch, dst, ((8 << 20) >> 12));
> +
> +	rc = drm_intel_gem_bo_wait(dst, timeout_ns);
> +	igt_assert(!rc);
> +
> +	intel_batchbuffer_free(batch);
> +}
> +
> +/* We could check the checksum too, but just the header is probably enough. */
> +static bool edid_is_valid(const unsigned char *edid)
> +{
> +	char edid_header[] = {
> +		0x0, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x0,
> +	};
> +
> +	return (memcmp(edid, edid_header, sizeof(edid_header)) == 0);
> +}
> +
> +static int count_drm_valid_edids(struct mode_set_data *data)
> +{
> +	int i, ret = 0;
> +
> +	for (i = 0; i < data->res->count_connectors; i++)
> +		if (data->edids[i] && edid_is_valid(data->edids[i]->data))
> +			ret++;
> +	return ret;
> +}
> +
> +static int count_drm_edp_outputs(struct mode_set_data *data)
> +{
> +	int i, ret = 0;
> +
> +	for (i = 0; i < data->res->count_connectors; i++)
> +		if (data->connectors[i]->connector_type ==
> +		    DRM_MODE_CONNECTOR_eDP)
> +			ret++;
> +	return ret;
> +}
> +
> +static bool i2c_edid_is_valid(int fd)
> +{
> +	int rc;
> +	unsigned char edid[128] = {};
> +	struct i2c_msg msgs[] = {
> +		{ /* Start at 0. */
> +			.addr = 0x50,
> +			.flags = 0,
> +			.len = 1,
> +			.buf = edid,
> +		}, { /* Now read the EDID. */
> +			.addr = 0x50,
> +			.flags = I2C_M_RD,
> +			.len = 128,
> +			.buf = edid,
> +		}
> +	};
> +	struct i2c_rdwr_ioctl_data msgset = {
> +		.msgs = msgs,
> +		.nmsgs = 2,
> +	};
> +
> +	rc = ioctl(fd, I2C_RDWR, &msgset);
> +	return (rc >= 0) ? edid_is_valid(edid) : false;
> +}
> +
> +static int count_i2c_valid_edids(void)
> +{
> +	int fd, ret = 0;
> +	DIR *dir;
> +
> +	struct dirent *dirent;
> +	char full_name[32];
> +
> +	dir = opendir("/dev/");
> +	igt_assert(dir);
> +
> +	while ((dirent = readdir(dir))) {
> +		if (strncmp(dirent->d_name, "i2c-", 4) == 0) {
> +			snprintf(full_name, 32, "/dev/%s", dirent->d_name);
> +			fd = open(full_name, O_RDWR);
> +			igt_assert(fd != -1);
> +			if (i2c_edid_is_valid(fd))
> +				ret++;
> +			close(fd);
> +		}
> +	}
> +
> +	closedir(dir);
> +
> +	return 0;
> +}
> +
> +/* We run this test when all outputs are turned off. When eDP panels are turned
> + * off, I2C doesn't work on them and we get a nice dmesg backtrace. So count how
> + * many eDP panels we have and subtract them. */
> +static bool test_i2c(struct mode_set_data *data)
> +{
> +	int i2c_edids = count_i2c_valid_edids();
> +	int drm_edids = count_drm_valid_edids(data);
> +	int edps = count_drm_edp_outputs(data);
> +
> +	return i2c_edids + edps == drm_edids;
> +}
> +
> +static bool setup_environment(void)
> +{
> +	int i2c_dev_files;
> +	DIR *dev_dir;
> +	struct dirent *dirent;
> +
> +	drm_fd = drm_open_any();
> +	if (drm_fd <= 0)
> +		return false;
> +
> +	init_mode_set_data(&ms_data);
> +
> +	/* Only Haswell supports the PC8 feature. */
> +	if (!IS_HASWELL(ms_data.devid)) {
> +		printf("PC8+ feature only supported on Haswell.\n");
> +		return false;
> +	}
> +
> +	/* Make sure our Kernel supports MSR and the module is loaded. */
> +	msr_fd = open("/dev/cpu/0/msr", O_RDONLY);
> +	if (msr_fd == -1) {
> +		printf("Can't open /dev/cpu/0/msr.\n");
> +		return false;
> +	}
> +
> +	/* Make sure the /dev/i2c-* files exist. */
> +	dev_dir = opendir("/dev");
> +	if (!dev_dir) {
> +		printf("Can't open /dev.\n");
> +		return false;
> +	}
> +	i2c_dev_files = 0;
> +	while ((dirent = readdir(dev_dir))) {
> +		if (strncmp(dirent->d_name, "i2c-", 4) == 0)
> +			i2c_dev_files++;
> +	}
> +	closedir(dev_dir);
> +	if (!i2c_dev_files) {
> +		printf("No /dev/i2c-* files.\n");
> +		return false;
> +	}
> +
> +	/* Non-ULT machines don't support PC8+. */
> +	if (!supports_pc8_plus_residencies()) {
> +		printf("Machine doesn't support PC8+ residencies.\n");
> +		return false;
> +	}
> +
> +	/* Make sure PC8+ residencies move! */
> +	disable_all_screens(&ms_data);
> +	if (!pc8_plus_enabled()) {
> +		printf("Machine is not reaching PC8+ states, please check its "
> +		       "configuration.\n");
> +		return false;
> +	}
> +
> +	/* Make sure PC8+ residencies stop! */
> +	enable_one_screen(&ms_data);
> +	if (!pc8_plus_disabled()) {
> +		printf("PC8+ residency didn't stop with screen enabled.\n");
> +		return false;
> +	}
> +
> +	return true;
> +}
> +
> +static void teardown_environment(void)
> +{
> +	fini_mode_set_data(&ms_data);
> +	drmClose(drm_fd);
> +	close(msr_fd);
> +}
> +
> +/* Test of the DRM resources reported by the IOCTLs are still the same. This
> + * ensures we still see the monitors with the same eyes. We get the EDIDs and
> + * compare them, which ensures we use DP AUX or GMBUS depending on what's
> + * connected. */
> +static void drm_resources_equal_subtest(void)
> +{
> +	struct compare_data pre_pc8, during_pc8, post_pc8;
> +
> +	printf("Checking the if the DRM resources match.\n");
> +
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +	get_drm_info(&pre_pc8);
> +	igt_assert(pc8_plus_disabled());
> +
> +	disable_all_screens(&ms_data);
> +	igt_assert(pc8_plus_enabled());
> +	get_drm_info(&during_pc8);
> +	igt_assert(pc8_plus_enabled());
> +
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +	get_drm_info(&post_pc8);
> +	igt_assert(pc8_plus_disabled());
> +
> +	assert_drm_infos_equal(&pre_pc8, &during_pc8);
> +	assert_drm_infos_equal(&pre_pc8, &post_pc8);
> +
> +	free_drm_info(&pre_pc8);
> +	free_drm_info(&during_pc8);
> +	free_drm_info(&post_pc8);
> +}
> +
> +/* Make sure interrupts are working. */
> +static void batch_subtest(void)
> +{
> +	printf("Testing batchbuffers.\n");
> +
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +
> +	disable_all_screens(&ms_data);
> +	igt_assert(pc8_plus_enabled());
> +	test_batch(&ms_data);
> +	igt_assert(pc8_plus_enabled());
> +}
> +
> +/* Try to use raw I2C, which also needs interrupts. */
> +static void i2c_subtest(void)
> +{
> +	printf("Testing raw i2c protocol calls.\n");
> +
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +
> +	disable_all_screens(&ms_data);
> +	igt_assert(pc8_plus_enabled());
> +	igt_assert(test_i2c(&ms_data));
> +	igt_assert(pc8_plus_enabled());
> +
> +	enable_one_screen(&ms_data);
> +}
> +
> +/* Make us enter/leave PC8+ many times. */
> +static void stress_test(void)
> +{
> +	int i;
> +
> +	printf("Stress testing.\n");
> +
> +	for (i = 0; i < 100; i++) {
> +		disable_all_screens(&ms_data);
> +		igt_assert(pc8_plus_enabled());
> +		test_batch(&ms_data);
> +		igt_assert(pc8_plus_enabled());
> +	}
> +}
> +
> +/* Just reading/writing registers from outside the Kernel is not really a safe
> + * thing to do on Haswell, so don't do this test on the default case. */
> +static void register_compare_subtest(void)
> +{
> +	struct compare_registers pre_pc8, post_pc8;
> +
> +	printf("Testing register compare.\n");
> +
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +	get_registers(&pre_pc8);
> +	igt_assert(pc8_plus_disabled());
> +
> +	disable_all_screens(&ms_data);
> +	igt_assert(pc8_plus_enabled());
> +	enable_one_screen(&ms_data);
> +	igt_assert(pc8_plus_disabled());
> +	/* Wait for the registers to be restored. */
> +	sleep(1);
> +	get_registers(&post_pc8);
> +	igt_assert(pc8_plus_disabled());
> +
> +	compare_registers(&pre_pc8, &post_pc8);
> +}
> +
> +int main(int argc, char *argv[])
> +{
> +	bool do_register_compare = false;
> +
> +	if (argc > 1 && strcmp(argv[1], "--do-register-compare") == 0)
> +		do_register_compare = true;
> +
> +	igt_subtest_init(argc, argv);
> +
> +	/* Skip instead of failing in case the machine is not prepared to reach
> +	 * PC8+. We don't want bug reports from cases where the machine is just
> +	 * not properly configured. */
> +	printf("Checking if the environment is properly configured.\n");
> +	if (!setup_environment())
> +		return 77;
> +
> +	igt_subtest("drm-resources-equal")
> +		drm_resources_equal_subtest();
> +	igt_subtest("batch")
> +		batch_subtest();
> +	igt_subtest("i2c")
> +		i2c_subtest();
> +	igt_subtest("stress-test")
> +		stress_test();
> +	if (do_register_compare)
> +		igt_subtest("register-compare")
> +			register_compare_subtest();
> +
> +	teardown_environment();
> +
> +	printf("Done!\n");
> +	return 0;
> +}
> -- 
> 1.8.1.2
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://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