[PATCH] tests/intel/perf_pmu: Add a test to repro NPD during FLR

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jan 12 10:35:12 UTC 2024


Hi Umesh,
On 2024-01-12 at 02:20:38 +0000, Umesh Nerlige Ramappa wrote:
> Open a PMU event (actual-frequency) and then do an unbind, followed by a bind.
> This is causing NPD during bind.
------------------^^^
Please describe what NPD means here.

> 
> Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
> ---
>  tests/intel/perf_pmu.c | 60 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 60 insertions(+)
> 
> diff --git a/tests/intel/perf_pmu.c b/tests/intel/perf_pmu.c
> index c6e6a8b77..06b02e18a 100644
> --- a/tests/intel/perf_pmu.c
> +++ b/tests/intel/perf_pmu.c
> @@ -249,6 +249,11 @@
>   * Description: Test the i915 pmu perf interface
>   * Feature: i915 pmu perf interface, pmu
>   * Test category: Perf
> + *
> + * SUBTEST: npd
---------------^^^
Please use some other name for subtest, NPD may happen in other
scenarios, for example:

* SUBTEST: bind-openevent-unbind-bind

Regards,
Kamil

> + * Description: Test null pointer dereference case
> + * Feature: i915 pmu perf interface, pmu
> + * Test category: Perf
>   */
>  
>  IGT_TEST_DESCRIPTION("Test the i915 pmu perf interface");
> @@ -2417,6 +2422,58 @@ static void test_unload(unsigned int num_engines)
>  	igt_assert_eq(__igt_i915_driver_unload(NULL), 0);
>  }
>  
> +static int sysfs_driver(int i915)
> +{
> +	int dir, drv;
> +
> +	dir = igt_sysfs_open(i915);
> +	igt_assert(dir != -1);
> +
> +	drv = openat(dir, "device/driver", O_DIRECTORY);
> +	close(dir);
> +
> +	igt_assert(drv != -1);
> +	return drv;
> +}
> +
> +static void test_npd(void)
> +{
> +	struct pci_device *pdev;
> +	int fd, i915, drv;
> +	char *snd = NULL;
> +	char addr[80];
> +	uint64_t val;
> +
> +	i915 = __drm_open_driver(DRIVER_INTEL);
> +	pdev = igt_device_get_pci_device(i915);
> +
> +	snprintf(addr, sizeof(addr), "%04x:%02x:%02x.%d",
> +		 pdev->domain, pdev->bus, pdev->dev, pdev->func);
> +	igt_debug("Opened %s\n", addr);
> +
> +	fd = open_pmu(i915, I915_PMU_ACTUAL_FREQUENCY);
> +
> +	drv = sysfs_driver(i915);
> +	close(i915);
> +
> +	igt_audio_driver_unload(&snd);
> +
> +	val = pmu_read_single(fd);
> +
> +	igt_set_timeout(30, "Driver unbind timeout!");
> +	igt_assert_f(igt_sysfs_set(drv, "unbind", addr),
> +		     "Driver unbind failure (%s)!\n", addr);
> +	igt_reset_timeout();
> +
> +	igt_set_timeout(30, "Driver bind timeout!");
> +	igt_assert_f(igt_sysfs_set(drv, "bind", addr),
> +		     "Driver bind failure (%s)!\n", addr);
> +	igt_reset_timeout();
> +
> +	close(drv);
> +	close(fd);
> +}
> +
>  static void pmu_read(int i915)
>  {
>  	char val[128];
> @@ -2745,6 +2802,9 @@ igt_main
>  		free(drpc);
>  	}
>  
> +	igt_subtest("npd")
> +		test_npd();
> +
>  	igt_subtest("module-unload") {
>  		igt_require(igt_i915_driver_unload() == 0);
>  		for (int pass = 0; pass < 3; pass++)
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list