Re: [PATCH v6 07/11] drm/i915/selftests: Use to_gt() helper

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

 



On Thu, Dec 09, 2021 at 03:25:08PM +0200, Andi Shyti wrote:
> Use to_gt() helper consistently throughout the codebase.
> Pure mechanical s/i915->gt/to_gt(i915). No functional changes.
> 
> Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>
> Cc: Michał Winiarski <michal.winiarski@xxxxxxxxx>

Reviewed-by: Matt Roper <matthew.d.roper@xxxxxxxxx>

> ---
>  drivers/gpu/drm/i915/selftests/i915_active.c  |  2 +-
>  drivers/gpu/drm/i915/selftests/i915_gem.c     |  2 +-
>  .../gpu/drm/i915/selftests/i915_gem_evict.c   |  6 ++--
>  drivers/gpu/drm/i915/selftests/i915_gem_gtt.c |  4 +--
>  drivers/gpu/drm/i915/selftests/i915_perf.c    |  2 +-
>  drivers/gpu/drm/i915/selftests/i915_request.c | 10 +++----
>  .../gpu/drm/i915/selftests/i915_selftest.c    |  4 +--
>  .../gpu/drm/i915/selftests/igt_flush_test.c   |  2 +-
>  .../gpu/drm/i915/selftests/igt_live_test.c    |  4 +--
>  .../drm/i915/selftests/intel_memory_region.c  |  4 +--
>  drivers/gpu/drm/i915/selftests/intel_uncore.c |  2 +-
>  .../gpu/drm/i915/selftests/mock_gem_device.c  | 30 +++++++++----------
>  drivers/gpu/drm/i915/selftests/mock_gtt.c     |  6 ++--
>  drivers/gpu/drm/i915/selftests/mock_uncore.c  |  2 +-
>  14 files changed, 40 insertions(+), 40 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/selftests/i915_active.c b/drivers/gpu/drm/i915/selftests/i915_active.c
> index 61bf4560d8af..2dac9be1de58 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_active.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_active.c
> @@ -254,7 +254,7 @@ int i915_active_live_selftests(struct drm_i915_private *i915)
>  		SUBTEST(live_active_barrier),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_subtests(tests, i915);
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem.c b/drivers/gpu/drm/i915/selftests/i915_gem.c
> index 152d9ab135b1..b5576888cd78 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem.c
> @@ -248,7 +248,7 @@ int i915_gem_live_selftests(struct drm_i915_private *i915)
>  		SUBTEST(igt_gem_ww_ctx),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_live_subtests(tests, i915);
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> index 7e0658a77659..75b709c26dd3 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> @@ -545,7 +545,7 @@ int i915_gem_evict_mock_selftests(void)
>  		return -ENOMEM;
>  
>  	with_intel_runtime_pm(&i915->runtime_pm, wakeref)
> -		err = i915_subtests(tests, &i915->gt);
> +		err = i915_subtests(tests, to_gt(i915));
>  
>  	mock_destroy_device(i915);
>  	return err;
> @@ -557,8 +557,8 @@ int i915_gem_evict_live_selftests(struct drm_i915_private *i915)
>  		SUBTEST(igt_evict_contexts),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
> -	return intel_gt_live_subtests(tests, &i915->gt);
> +	return intel_gt_live_subtests(tests, to_gt(i915));
>  }
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c b/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
> index 46f4236039a9..48123c3e1ff0 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
> @@ -155,7 +155,7 @@ static int igt_ppgtt_alloc(void *arg)
>  	if (!HAS_PPGTT(dev_priv))
>  		return 0;
>  
> -	ppgtt = i915_ppgtt_create(&dev_priv->gt, 0);
> +	ppgtt = i915_ppgtt_create(to_gt(dev_priv), 0);
>  	if (IS_ERR(ppgtt))
>  		return PTR_ERR(ppgtt);
>  
> @@ -1053,7 +1053,7 @@ static int exercise_ppgtt(struct drm_i915_private *dev_priv,
>  	if (IS_ERR(file))
>  		return PTR_ERR(file);
>  
> -	ppgtt = i915_ppgtt_create(&dev_priv->gt, 0);
> +	ppgtt = i915_ppgtt_create(to_gt(dev_priv), 0);
>  	if (IS_ERR(ppgtt)) {
>  		err = PTR_ERR(ppgtt);
>  		goto out_free;
> diff --git a/drivers/gpu/drm/i915/selftests/i915_perf.c b/drivers/gpu/drm/i915/selftests/i915_perf.c
> index 9e9a6cb1d9e5..88db2e3d81d0 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_perf.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_perf.c
> @@ -424,7 +424,7 @@ int i915_perf_live_selftests(struct drm_i915_private *i915)
>  	if (!perf->metrics_kobj || !perf->ops.enable_metric_set)
>  		return 0;
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	err = alloc_empty_config(&i915->perf);
> diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
> index 9979ef9197cd..92a859b34190 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_request.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_request.c
> @@ -841,7 +841,7 @@ static struct i915_vma *empty_batch(struct drm_i915_private *i915)
>  	__i915_gem_object_flush_map(obj, 0, 64);
>  	i915_gem_object_unpin_map(obj);
>  
> -	intel_gt_chipset_flush(&i915->gt);
> +	intel_gt_chipset_flush(to_gt(i915));
>  
>  	vma = i915_vma_instance(obj, &i915->ggtt.vm, NULL);
>  	if (IS_ERR(vma)) {
> @@ -982,7 +982,7 @@ static struct i915_vma *recursive_batch(struct drm_i915_private *i915)
>  	if (IS_ERR(obj))
>  		return ERR_CAST(obj);
>  
> -	vma = i915_vma_instance(obj, i915->gt.vm, NULL);
> +	vma = i915_vma_instance(obj, to_gt(i915)->vm, NULL);
>  	if (IS_ERR(vma)) {
>  		err = PTR_ERR(vma);
>  		goto err;
> @@ -1014,7 +1014,7 @@ static struct i915_vma *recursive_batch(struct drm_i915_private *i915)
>  	__i915_gem_object_flush_map(obj, 0, 64);
>  	i915_gem_object_unpin_map(obj);
>  
> -	intel_gt_chipset_flush(&i915->gt);
> +	intel_gt_chipset_flush(to_gt(i915));
>  
>  	return vma;
>  
> @@ -1700,7 +1700,7 @@ int i915_request_live_selftests(struct drm_i915_private *i915)
>  		SUBTEST(live_breadcrumbs_smoketest),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_subtests(tests, i915);
> @@ -3091,7 +3091,7 @@ int i915_request_perf_selftests(struct drm_i915_private *i915)
>  		SUBTEST(perf_parallel_engines),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_subtests(tests, i915);
> diff --git a/drivers/gpu/drm/i915/selftests/i915_selftest.c b/drivers/gpu/drm/i915/selftests/i915_selftest.c
> index 484759c9409c..2d6d7bd13c3c 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_selftest.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_selftest.c
> @@ -298,10 +298,10 @@ int __i915_live_setup(void *data)
>  	struct drm_i915_private *i915 = data;
>  
>  	/* The selftests expect an idle system */
> -	if (intel_gt_pm_wait_for_idle(&i915->gt))
> +	if (intel_gt_pm_wait_for_idle(to_gt(i915)))
>  		return -EIO;
>  
> -	return intel_gt_terminally_wedged(&i915->gt);
> +	return intel_gt_terminally_wedged(to_gt(i915));
>  }
>  
>  int __i915_live_teardown(int err, void *data)
> diff --git a/drivers/gpu/drm/i915/selftests/igt_flush_test.c b/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> index a6c71fca61aa..b84594601d30 100644
> --- a/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> +++ b/drivers/gpu/drm/i915/selftests/igt_flush_test.c
> @@ -14,7 +14,7 @@
>  
>  int igt_flush_test(struct drm_i915_private *i915)
>  {
> -	struct intel_gt *gt = &i915->gt;
> +	struct intel_gt *gt = to_gt(i915);
>  	int ret = intel_gt_is_wedged(gt) ? -EIO : 0;
>  
>  	cond_resched();
> diff --git a/drivers/gpu/drm/i915/selftests/igt_live_test.c b/drivers/gpu/drm/i915/selftests/igt_live_test.c
> index 1c721542e277..72b58b66692a 100644
> --- a/drivers/gpu/drm/i915/selftests/igt_live_test.c
> +++ b/drivers/gpu/drm/i915/selftests/igt_live_test.c
> @@ -16,7 +16,7 @@ int igt_live_test_begin(struct igt_live_test *t,
>  			const char *func,
>  			const char *name)
>  {
> -	struct intel_gt *gt = &i915->gt;
> +	struct intel_gt *gt = to_gt(i915);
>  	struct intel_engine_cs *engine;
>  	enum intel_engine_id id;
>  	int err;
> @@ -57,7 +57,7 @@ int igt_live_test_end(struct igt_live_test *t)
>  		return -EIO;
>  	}
>  
> -	for_each_engine(engine, &i915->gt, id) {
> +	for_each_engine(engine, to_gt(i915), id) {
>  		if (t->reset_engine[id] ==
>  		    i915_reset_engine_count(&i915->gpu_error, engine))
>  			continue;
> diff --git a/drivers/gpu/drm/i915/selftests/intel_memory_region.c b/drivers/gpu/drm/i915/selftests/intel_memory_region.c
> index 0d5df0dc7212..8255561ff853 100644
> --- a/drivers/gpu/drm/i915/selftests/intel_memory_region.c
> +++ b/drivers/gpu/drm/i915/selftests/intel_memory_region.c
> @@ -1217,7 +1217,7 @@ int intel_memory_region_live_selftests(struct drm_i915_private *i915)
>  		return 0;
>  	}
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_live_subtests(tests, i915);
> @@ -1229,7 +1229,7 @@ int intel_memory_region_perf_selftests(struct drm_i915_private *i915)
>  		SUBTEST(perf_memcpy),
>  	};
>  
> -	if (intel_gt_is_wedged(&i915->gt))
> +	if (intel_gt_is_wedged(to_gt(i915)))
>  		return 0;
>  
>  	return i915_live_subtests(tests, i915);
> diff --git a/drivers/gpu/drm/i915/selftests/intel_uncore.c b/drivers/gpu/drm/i915/selftests/intel_uncore.c
> index bc8128170a99..cdd196783535 100644
> --- a/drivers/gpu/drm/i915/selftests/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/selftests/intel_uncore.c
> @@ -344,5 +344,5 @@ int intel_uncore_live_selftests(struct drm_i915_private *i915)
>  		SUBTEST(live_forcewake_domains),
>  	};
>  
> -	return intel_gt_live_subtests(tests, &i915->gt);
> +	return intel_gt_live_subtests(tests, to_gt(i915));
>  }
> diff --git a/drivers/gpu/drm/i915/selftests/mock_gem_device.c b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
> index eeb632aac4a7..8aa7b1d33865 100644
> --- a/drivers/gpu/drm/i915/selftests/mock_gem_device.c
> +++ b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
> @@ -45,7 +45,7 @@
>  
>  void mock_device_flush(struct drm_i915_private *i915)
>  {
> -	struct intel_gt *gt = &i915->gt;
> +	struct intel_gt *gt = to_gt(i915);
>  	struct intel_engine_cs *engine;
>  	enum intel_engine_id id;
>  
> @@ -64,7 +64,7 @@ static void mock_device_release(struct drm_device *dev)
>  		goto out;
>  
>  	mock_device_flush(i915);
> -	intel_gt_driver_remove(&i915->gt);
> +	intel_gt_driver_remove(to_gt(i915));
>  
>  	i915_gem_drain_workqueue(i915);
>  	i915_gem_drain_freed_objects(i915);
> @@ -73,7 +73,7 @@ static void mock_device_release(struct drm_device *dev)
>  	destroy_workqueue(i915->wq);
>  
>  	intel_region_ttm_device_fini(i915);
> -	intel_gt_driver_late_release(&i915->gt);
> +	intel_gt_driver_late_release(to_gt(i915));
>  	intel_memory_regions_driver_release(i915);
>  
>  	drm_mode_config_cleanup(&i915->drm);
> @@ -178,11 +178,11 @@ struct drm_i915_private *mock_gem_device(void)
>  	spin_lock_init(&i915->gpu_error.lock);
>  
>  	i915_gem_init__mm(i915);
> -	intel_gt_init_early(&i915->gt, i915);
> -	__intel_gt_init_early(&i915->gt, i915);
> +	intel_gt_init_early(to_gt(i915), i915);
> +	__intel_gt_init_early(to_gt(i915), i915);
>  	mock_uncore_init(&i915->uncore, i915);
> -	atomic_inc(&i915->gt.wakeref.count); /* disable; no hw support */
> -	i915->gt.awake = -ENODEV;
> +	atomic_inc(&to_gt(i915)->wakeref.count); /* disable; no hw support */
> +	to_gt(i915)->awake = -ENODEV;
>  
>  	ret = intel_region_ttm_device_init(i915);
>  	if (ret)
> @@ -195,19 +195,19 @@ struct drm_i915_private *mock_gem_device(void)
>  	mock_init_contexts(i915);
>  
>  	mock_init_ggtt(i915, &i915->ggtt);
> -	i915->gt.vm = i915_vm_get(&i915->ggtt.vm);
> +	to_gt(i915)->vm = i915_vm_get(&i915->ggtt.vm);
>  
>  	mkwrite_device_info(i915)->platform_engine_mask = BIT(0);
> -	i915->gt.info.engine_mask = BIT(0);
> +	to_gt(i915)->info.engine_mask = BIT(0);
>  
> -	i915->gt.engine[RCS0] = mock_engine(i915, "mock", RCS0);
> -	if (!i915->gt.engine[RCS0])
> +	to_gt(i915)->engine[RCS0] = mock_engine(i915, "mock", RCS0);
> +	if (!to_gt(i915)->engine[RCS0])
>  		goto err_unlock;
>  
> -	if (mock_engine_init(i915->gt.engine[RCS0]))
> +	if (mock_engine_init(to_gt(i915)->engine[RCS0]))
>  		goto err_context;
>  
> -	__clear_bit(I915_WEDGED, &i915->gt.reset.flags);
> +	__clear_bit(I915_WEDGED, &to_gt(i915)->reset.flags);
>  	intel_engines_driver_register(i915);
>  
>  	i915->do_release = true;
> @@ -216,13 +216,13 @@ struct drm_i915_private *mock_gem_device(void)
>  	return i915;
>  
>  err_context:
> -	intel_gt_driver_remove(&i915->gt);
> +	intel_gt_driver_remove(to_gt(i915));
>  err_unlock:
>  	destroy_workqueue(i915->wq);
>  err_drv:
>  	intel_region_ttm_device_fini(i915);
>  err_ttm:
> -	intel_gt_driver_late_release(&i915->gt);
> +	intel_gt_driver_late_release(to_gt(i915));
>  	intel_memory_regions_driver_release(i915);
>  	drm_mode_config_cleanup(&i915->drm);
>  	mock_destroy_device(i915);
> diff --git a/drivers/gpu/drm/i915/selftests/mock_gtt.c b/drivers/gpu/drm/i915/selftests/mock_gtt.c
> index cc047ec594f9..f0b87de0aca3 100644
> --- a/drivers/gpu/drm/i915/selftests/mock_gtt.c
> +++ b/drivers/gpu/drm/i915/selftests/mock_gtt.c
> @@ -70,7 +70,7 @@ struct i915_ppgtt *mock_ppgtt(struct drm_i915_private *i915, const char *name)
>  	if (!ppgtt)
>  		return NULL;
>  
> -	ppgtt->vm.gt = &i915->gt;
> +	ppgtt->vm.gt = to_gt(i915);
>  	ppgtt->vm.i915 = i915;
>  	ppgtt->vm.total = round_down(U64_MAX, PAGE_SIZE);
>  	ppgtt->vm.dma = i915->drm.dev;
> @@ -109,7 +109,7 @@ void mock_init_ggtt(struct drm_i915_private *i915, struct i915_ggtt *ggtt)
>  {
>  	memset(ggtt, 0, sizeof(*ggtt));
>  
> -	ggtt->vm.gt = &i915->gt;
> +	ggtt->vm.gt = to_gt(i915);
>  	ggtt->vm.i915 = i915;
>  	ggtt->vm.is_ggtt = true;
>  
> @@ -130,7 +130,7 @@ void mock_init_ggtt(struct drm_i915_private *i915, struct i915_ggtt *ggtt)
>  	ggtt->vm.vma_ops.clear_pages = clear_pages;
>  
>  	i915_address_space_init(&ggtt->vm, VM_CLASS_GGTT);
> -	i915->gt.ggtt = ggtt;
> +	to_gt(i915)->ggtt = ggtt;
>  }
>  
>  void mock_fini_ggtt(struct i915_ggtt *ggtt)
> diff --git a/drivers/gpu/drm/i915/selftests/mock_uncore.c b/drivers/gpu/drm/i915/selftests/mock_uncore.c
> index b3790ef137e4..f2d6be5e1230 100644
> --- a/drivers/gpu/drm/i915/selftests/mock_uncore.c
> +++ b/drivers/gpu/drm/i915/selftests/mock_uncore.c
> @@ -42,7 +42,7 @@ __nop_read(64)
>  void mock_uncore_init(struct intel_uncore *uncore,
>  		      struct drm_i915_private *i915)
>  {
> -	intel_uncore_init_early(uncore, &i915->gt);
> +	intel_uncore_init_early(uncore, to_gt(i915));
>  
>  	ASSIGN_RAW_WRITE_MMIO_VFUNCS(uncore, nop);
>  	ASSIGN_RAW_READ_MMIO_VFUNCS(uncore, nop);
> -- 
> 2.34.1
> 

-- 
Matt Roper
Graphics Software Engineer
VTT-OSGC Platform Enablement
Intel Corporation
(916) 356-2795



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux