All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] managed drm_device, absolute final leftover bits
@ 2020-09-18 13:25 ` Daniel Vetter
  0 siblings, 0 replies; 56+ messages in thread
From: Daniel Vetter @ 2020-09-18 13:25 UTC (permalink / raw)
  To: DRI Development; +Cc: Daniel Vetter, Intel Graphics Development, amd-gfx

Hi all,

These are the leftovers of the leftovers of my initial drmm series to
manage drm_device.

Changes:
- bugfixed i915 selftests
- patch from Luben to finalize the admgpu conversion

Alex & i915 maintainers, pls ack for merging this all through
drm-misc-next since otherwise the final patch (and the resulting confusion
with outdated docs) is held up another round.

Cheers, Daniel

Daniel Vetter (3):
  drm/i915/selftest: Create mock_destroy_device
  drm/i915/selftests: align more to real device lifetimes
  drm/dev: Remove drm_dev_init

Luben Tuikov (1):
  drm/amdgpu: Convert to using devm_drm_dev_alloc() (v2)

 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c       | 16 ++----
 drivers/gpu/drm/drm_drv.c                     | 41 ++--------------
 drivers/gpu/drm/drm_internal.h                |  1 +
 drivers/gpu/drm/drm_managed.c                 | 13 -----
 .../gpu/drm/i915/gem/selftests/huge_pages.c   |  2 +-
 .../drm/i915/gem/selftests/i915_gem_context.c |  2 +-
 .../drm/i915/gem/selftests/i915_gem_dmabuf.c  |  2 +-
 .../drm/i915/gem/selftests/i915_gem_object.c  |  2 +-
 .../drm/i915/gem/selftests/i915_gem_phys.c    |  2 +-
 drivers/gpu/drm/i915/gt/selftest_timeline.c   |  2 +-
 .../gpu/drm/i915/selftests/i915_gem_evict.c   |  2 +-
 drivers/gpu/drm/i915/selftests/i915_gem_gtt.c |  2 +-
 drivers/gpu/drm/i915/selftests/i915_request.c |  2 +-
 drivers/gpu/drm/i915/selftests/i915_vma.c     |  2 +-
 .../drm/i915/selftests/intel_memory_region.c  |  2 +-
 .../gpu/drm/i915/selftests/mock_gem_device.c  | 49 ++++++++++++-------
 .../gpu/drm/i915/selftests/mock_gem_device.h  |  2 +
 include/drm/drm_drv.h                         |  4 --
 18 files changed, 51 insertions(+), 97 deletions(-)

-- 
2.28.0

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 56+ messages in thread
* [PATCH 43/44] drm/i915/selftests: align more to real device lifetimes
@ 2020-04-03 13:58 Daniel Vetter
  2020-04-03 22:31 ` [PATCH] " Daniel Vetter
  0 siblings, 1 reply; 56+ messages in thread
From: Daniel Vetter @ 2020-04-03 13:58 UTC (permalink / raw)
  To: DRI Development; +Cc: Daniel Vetter, Intel Graphics Development, Daniel Vetter

The big change is device_add so that device_del can auto-cleanup
devres resources. This allows us to use devm_drm_dev_alloc, which
removes the last user of drm_dev_init.

Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
 .../gpu/drm/i915/selftests/mock_gem_device.c  | 31 +++++++++----------
 .../gpu/drm/i915/selftests/mock_gem_device.h  |  2 +-
 2 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/i915/selftests/mock_gem_device.c b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
index 03607647cdeb..ea73d1f7cf12 100644
--- a/drivers/gpu/drm/i915/selftests/mock_gem_device.c
+++ b/drivers/gpu/drm/i915/selftests/mock_gem_device.c
@@ -123,12 +123,6 @@ struct drm_i915_private *mock_gem_device(void)
 	pdev = kzalloc(sizeof(*pdev), GFP_KERNEL);
 	if (!pdev)
 		return NULL;
-	i915 = kzalloc(sizeof(*i915), GFP_KERNEL);
-	if (!i915) {
-		kfree(pdev);
-		return NULL;
-	}
-
 	device_initialize(&pdev->dev);
 	pdev->class = PCI_BASE_CLASS_DISPLAY << 16;
 	pdev->dev.release = release_dev;
@@ -139,8 +133,23 @@ struct drm_i915_private *mock_gem_device(void)
 	/* hack to disable iommu for the fake device; force identity mapping */
 	pdev->dev.archdata.iommu = (void *)-1;
 #endif
+	err = device_add(&pdev->dev);
+	if (err) {
+		kfree(pdev);
+		return NULL;
+	}
+
+	i915 = devm_drm_dev_alloc(&pdev->dev, &mock_driver,
+				  struct drm_i915_private, drm);
+	if (err) {
+		pr_err("Failed to allocate mock GEM device: err=%d\n", err);
+		put_device(&pdev->dev);
+
+		return NULL;
+	}
 
 	pci_set_drvdata(pdev, i915);
+	i915->drm.pdev = pdev;
 
 	dev_pm_domain_set(&pdev->dev, &pm_domain);
 	pm_runtime_enable(&pdev->dev);
@@ -148,16 +157,6 @@ struct drm_i915_private *mock_gem_device(void)
 	if (pm_runtime_enabled(&pdev->dev))
 		WARN_ON(pm_runtime_get_sync(&pdev->dev));
 
-	err = drm_dev_init(&i915->drm, &mock_driver, &pdev->dev);
-	if (err) {
-		pr_err("Failed to initialise mock GEM device: err=%d\n", err);
-		put_device(&pdev->dev);
-		kfree(i915);
-
-		return NULL;
-	}
-	i915->drm.pdev = pdev;
-	drmm_add_final_kfree(&i915->drm, i915);
 
 	intel_runtime_pm_init_early(&i915->runtime_pm);
 
diff --git a/drivers/gpu/drm/i915/selftests/mock_gem_device.h b/drivers/gpu/drm/i915/selftests/mock_gem_device.h
index 2e3c7585a7bb..4f309a05c85a 100644
--- a/drivers/gpu/drm/i915/selftests/mock_gem_device.h
+++ b/drivers/gpu/drm/i915/selftests/mock_gem_device.h
@@ -9,7 +9,7 @@ void mock_device_flush(struct drm_i915_private *i915);
 
 static inline void mock_destroy_device(struct drm_i915_private *i915)
 {
-	drm_dev_put(&i915->drm);
+	device_del(i915->drm.dev);
 }
 
 #endif /* !__MOCK_GEM_DEVICE_H__ */
-- 
2.25.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 56+ messages in thread

end of thread, other threads:[~2020-09-21  9:10 UTC | newest]

Thread overview: 56+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-18 13:25 [PATCH 0/4] managed drm_device, absolute final leftover bits Daniel Vetter
2020-09-18 13:25 ` Daniel Vetter
2020-09-18 13:25 ` [Intel-gfx] " Daniel Vetter
2020-09-18 13:25 ` [PATCH 1/4] drm/i915/selftest: Create mock_destroy_device Daniel Vetter
2020-09-18 13:25   ` Daniel Vetter
2020-09-18 13:25   ` [Intel-gfx] " Daniel Vetter
2020-09-18 13:25 ` [PATCH 2/4] drm/i915/selftests: align more to real device lifetimes Daniel Vetter
2020-09-18 13:25   ` Daniel Vetter
2020-09-18 13:25   ` [Intel-gfx] " Daniel Vetter
2020-09-18 17:50   ` Matthew Auld
2020-09-18 17:50     ` Matthew Auld
2020-09-18 17:50     ` [Intel-gfx] " Matthew Auld
2020-09-18 18:22     ` Daniel Vetter
2020-09-18 18:22       ` Daniel Vetter
2020-09-18 18:22       ` [Intel-gfx] " Daniel Vetter
2020-09-18 18:31       ` Matthew Auld
2020-09-18 18:31         ` Matthew Auld
2020-09-18 18:31         ` [Intel-gfx] " Matthew Auld
2020-09-18 18:46         ` Daniel Vetter
2020-09-18 18:46           ` Daniel Vetter
2020-09-18 18:46           ` [Intel-gfx] " Daniel Vetter
2020-09-18 20:00   ` [PATCH] " Daniel Vetter
2020-09-18 20:00     ` Daniel Vetter
2020-09-18 20:00     ` [Intel-gfx] " Daniel Vetter
2020-09-19 13:38   ` Daniel Vetter
2020-09-19 13:38     ` Daniel Vetter
2020-09-19 13:38     ` [Intel-gfx] " Daniel Vetter
2020-09-19 13:40   ` Daniel Vetter
2020-09-19 13:40     ` Daniel Vetter
2020-09-19 13:40     ` [Intel-gfx] " Daniel Vetter
2020-09-18 13:25 ` [PATCH 3/4] drm/amdgpu: Convert to using devm_drm_dev_alloc() (v2) Daniel Vetter
2020-09-18 13:25   ` Daniel Vetter
2020-09-18 13:25   ` [Intel-gfx] " Daniel Vetter
2020-09-18 13:25 ` [PATCH 4/4] drm/dev: Remove drm_dev_init Daniel Vetter
2020-09-18 13:25   ` Daniel Vetter
2020-09-18 13:25   ` [Intel-gfx] " Daniel Vetter
2020-09-18 14:26   ` Thomas Zimmermann
2020-09-18 14:26     ` Thomas Zimmermann
2020-09-18 14:26     ` [Intel-gfx] " Thomas Zimmermann
2020-09-18 14:31 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for managed drm_device, absolute final leftover bits Patchwork
2020-09-18 14:56 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2020-09-18 15:03 ` [PATCH 0/4] " Alex Deucher
2020-09-18 15:03   ` Alex Deucher
2020-09-18 15:03   ` [Intel-gfx] " Alex Deucher
2020-09-18 17:12   ` Rodrigo Vivi
2020-09-18 17:12     ` Rodrigo Vivi
2020-09-18 17:12     ` Rodrigo Vivi
2020-09-21  9:09     ` Daniel Vetter
2020-09-21  9:09       ` Daniel Vetter
2020-09-21  9:09       ` Daniel Vetter
2020-09-18 18:01 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for " Patchwork
2020-09-18 20:15 ` [Intel-gfx] ✗ Fi.CI.BUILD: failure for managed drm_device, absolute final leftover bits (rev2) Patchwork
2020-09-19 13:57 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for managed drm_device, absolute final leftover bits (rev4) Patchwork
2020-09-19 14:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2020-09-19 15:29 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2020-04-03 13:58 [PATCH 43/44] drm/i915/selftests: align more to real device lifetimes Daniel Vetter
2020-04-03 22:31 ` [PATCH] " Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.