Re: [PATCH i-g-t 3/9] tests/perf: Add testcase to verify ctx id

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

 



On Wed, Sep 13, 2017 at 04:22:02PM +0530, Sagar Arun Kamble wrote:
> This subtest verifies that the CS perf samples contains
> proper HW context ID as captured through CONTEXT_PARAM_HW_ID.
> 
> v2: Updated property enum names.
> 
> Signed-off-by: Sagar Arun Kamble <sagar.a.kamble@xxxxxxxxx>
Reviewed-by: Ewelina Musial <ewelina.musial@xxxxxxxxx>
> ---
>  lib/ioctl_wrappers.h    |   1 +
>  tests/intel_perf_dapc.c | 102 +++++++++++++++++++++++++++++++++++++++++++++++-
>  2 files changed, 101 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
> index 090c125..44f15a0 100644
> --- a/lib/ioctl_wrappers.h
> +++ b/lib/ioctl_wrappers.h
> @@ -131,6 +131,7 @@ struct local_i915_gem_context_param {
>  #define LOCAL_CONTEXT_PARAM_GTT_SIZE	0x3
>  #define LOCAL_CONTEXT_PARAM_NO_ERROR_CAPTURE	0x4
>  #define LOCAL_CONTEXT_PARAM_BANNABLE	0x5
> +#define LOCAL_CONTEXT_PARAM_HW_ID	0x6
>  	uint64_t value;
>  };
>  void gem_context_require_bannable(int fd);
> diff --git a/tests/intel_perf_dapc.c b/tests/intel_perf_dapc.c
> index 92b4dee..23d6ffe 100644
> --- a/tests/intel_perf_dapc.c
> +++ b/tests/intel_perf_dapc.c
> @@ -644,10 +644,28 @@ read_perf_reports(int stream_fd,
>  	return true;
>  }
>  
> +static unsigned int
> +context_get_hw_ctx_id(int fd, unsigned int ctx)
> +{
> +	struct local_i915_gem_context_param param;
> +
> +	memset(&param, 0, sizeof(param));
> +	param.context = ctx;
> +	param.param = LOCAL_CONTEXT_PARAM_HW_ID;
> +	param.value = 0;
> +	param.size = 0;
> +
> +	if (__gem_context_get_param(fd, &param) == -EINVAL)
> +		igt_assert(param.value == 0);
> +
> +	return param.value;
> +}
> +
>  static void
>  perf_stream_capture_workload_samples(struct drm_i915_perf_open_param *param,
>  				     uint8_t *perf_reports,
> -				     int num_reports, int report_size)
> +				     int num_reports, int report_size,
> +				     uint64_t *hw_ctx_id)
>  {
>  	drm_intel_bufmgr *bufmgr;
>  	drm_intel_context *context0;
> @@ -676,6 +694,9 @@ retry:
>  	igt_assert_eq(ret, 0);
>  	igt_assert_neq(ctx_id, 0xffffffff);
>  
> +	if (hw_ctx_id)
> +		*hw_ctx_id = context_get_hw_ctx_id(drm_fd, ctx_id);
> +
>  	igt_debug("opening i915-perf stream\n");
>  	stream_fd = __perf_open(drm_fd, param);
>  
> @@ -776,7 +797,8 @@ test_oa_source(void)
>  		igt_assert(perf_reports);
>  
>  		perf_stream_capture_workload_samples(&param, perf_reports,
> -						     num_reports, report_size);
> +						     num_reports, report_size,
> +						     NULL);
>  		verify_source(perf_reports, num_reports, report_size);
>  		free(perf_reports);
>  	}
> @@ -784,6 +806,79 @@ test_oa_source(void)
>  	igt_waitchildren();
>  }
>  
> +struct oa_ctxid_sample {
> +	uint64_t ctx_id;
> +	uint8_t oa_report[];
> +};
> +
> +static void
> +verify_ctxid(uint8_t *perf_reports, int num_reports, size_t report_size,
> +	     uint64_t hw_ctx_id)
> +{
> +	struct oa_ctxid_sample *sample;
> +	uint32_t *oa_report;
> +
> +	for (int i = 0; i < num_reports; i++) {
> +		size_t offset = i * report_size;
> +
> +		sample = (struct oa_ctxid_sample *) (perf_reports + offset);
> +		oa_report = (uint32_t *) sample->oa_report;
> +
> +		igt_debug("read report: ctx_id= %lu, reason = %x, "
> +			  "timestamp = %x\n",
> +			  sample->ctx_id, oa_report[0], oa_report[1]);
> +
> +		if (!oa_report[0])
> +			igt_assert(sample->ctx_id == hw_ctx_id);
> +	}
> +}
> +
> +static void
> +test_perf_ctxid(void)
> +{
> +	uint64_t properties[] = {
> +		/* Include OA reports in samples */
> +		DRM_I915_PERF_PROP_SAMPLE_OA, true,
> +
> +		/* OA unit configuration */
> +		DRM_I915_PERF_PROP_OA_METRICS_SET, test_metric_set_id,
> +		DRM_I915_PERF_PROP_OA_FORMAT, test_oa_format,
> +		DRM_I915_PERF_PROP_OA_EXPONENT, oa_exp_1_millisec,
> +
> +		/* CS parameters */
> +		local_DRM_I915_PERF_PROP_ENGINE, I915_EXEC_RENDER,
> +		local_DRM_I915_PERF_PROP_SAMPLE_CTX_ID, true,
> +	};
> +	struct drm_i915_perf_open_param param = {
> +		.flags = I915_PERF_FLAG_FD_CLOEXEC,
> +		.num_properties = sizeof(properties) / 16,
> +		.properties_ptr = to_user_pointer(properties),
> +	};
> +
> +	/* should be default, but just to be sure... */
> +	write_u64_file("/proc/sys/dev/i915/perf_stream_paranoid", 1);

This function is used to write parameter to only one file so path could be
defined inside function

> +
> +	igt_fork(child, 1) {
> +		int prop_size = ARRAY_SIZE(properties);
> +		int num_reports = 10;
> +		int report_size = get_perf_report_size(properties, prop_size,
> +						       test_oa_format);
> +		int total_size = num_reports * report_size;
> +		uint8_t *perf_reports = malloc(total_size);
> +		uint64_t hw_ctx_id;
> +
> +		igt_assert(perf_reports);
> +
> +		perf_stream_capture_workload_samples(&param, perf_reports,
> +						     num_reports, report_size,
> +						     &hw_ctx_id);
> +		verify_ctxid(perf_reports, num_reports, report_size, hw_ctx_id);
> +		free(perf_reports);
> +	}
> +
> +	igt_waitchildren();
> +}
> +
>  igt_main
>  {
>  	igt_skip_on_simulation();
> @@ -805,6 +900,9 @@ igt_main
>  	igt_subtest("oa-source")
>  		test_oa_source();
>  
> +	igt_subtest("perf-ctxid")
> +		test_perf_ctxid();
> +
>  	igt_fixture {
>  		close(drm_fd);
>  	}
> -- 
> 1.9.1

--
Cheers,
Ewelina

> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
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