All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime@cerno.tech>
To: Maxime Ripard <mripard@kernel.org>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	 Daniel Vetter <daniel@ffwll.ch>,
	David Airlie <airlied@gmail.com>,
	Thomas Zimmermann <tzimmermann@suse.de>
Cc: David Gow <davidgow@google.com>,
	Brendan Higgins <brendan.higgins@linux.dev>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Dave Stevenson <dave.stevenson@raspberrypi.com>,
	Javier Martinez Canillas <javierm@redhat.com>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	linaro-mm-sig@lists.linaro.org,
	Maíra Canal <mairacanal@riseup.net>,
	Maxime Ripard <maxime@cerno.tech>,
	linux-kselftest@vger.kernel.org, kunit-dev@googlegroups.com,
	linux-media@vger.kernel.org
Subject: [PATCH 06/24] drm/tests: kunit: Allow for a custom device struct to be allocated
Date: Wed, 23 Nov 2022 16:25:48 +0100	[thread overview]
Message-ID: <20221123-rpi-kunit-tests-v1-6-051a0bb60a16@cerno.tech> (raw)
In-Reply-To: <20221123-rpi-kunit-tests-v1-0-051a0bb60a16@cerno.tech>

The current helper to allocate a DRM device doesn't allow for any
subclassing by drivers, which is going to be troublesome as we work on
getting some kunit testing on atomic modesetting code.

Let's use a similar pattern to the other allocation helpers by providing
the structure size and offset as arguments.

Signed-off-by: Maxime Ripard <maxime@cerno.tech>
---
 drivers/gpu/drm/tests/drm_client_modeset_test.c |  4 +++-
 drivers/gpu/drm/tests/drm_kunit_helpers.c       | 19 ++++++++-----------
 drivers/gpu/drm/tests/drm_kunit_helpers.h       | 11 +++++++++--
 3 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/tests/drm_client_modeset_test.c b/drivers/gpu/drm/tests/drm_client_modeset_test.c
index fe95fda350ae..086469273580 100644
--- a/drivers/gpu/drm/tests/drm_client_modeset_test.c
+++ b/drivers/gpu/drm/tests/drm_client_modeset_test.c
@@ -45,7 +45,9 @@ static int drm_client_modeset_test_init(struct kunit *test)
 	priv->dev = drm_kunit_helper_alloc_device(test);
 	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv->dev);
 
-	priv->drm = drm_kunit_helper_alloc_drm_device(test, priv->dev, DRIVER_MODESET);
+	priv->drm = __drm_kunit_helper_alloc_drm_device(test, priv->dev,
+							sizeof(*priv->drm), 0,
+							DRIVER_MODESET);
 	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, priv->drm);
 
 	ret = drmm_connector_init(priv->drm, &priv->connector,
diff --git a/drivers/gpu/drm/tests/drm_kunit_helpers.c b/drivers/gpu/drm/tests/drm_kunit_helpers.c
index 8bd7c40d0e1a..6d614f9e62d8 100644
--- a/drivers/gpu/drm/tests/drm_kunit_helpers.c
+++ b/drivers/gpu/drm/tests/drm_kunit_helpers.c
@@ -13,10 +13,6 @@
 #define WQ_TIMEOUT_MS		100
 #define FAKE_DEVICE_NAME	"drm-kunit-fake-device"
 
-struct kunit_dev {
-	struct drm_device base;
-};
-
 static const struct drm_mode_config_funcs drm_mode_config_funcs = {
 };
 
@@ -104,12 +100,13 @@ void drm_kunit_helper_free_device(struct kunit *test, struct device *dev)
 }
 
 struct drm_device *
-drm_kunit_helper_alloc_drm_device(struct kunit *test, struct device *dev,
-				  u32 features)
+__drm_kunit_helper_alloc_drm_device(struct kunit *test, struct device *dev,
+				    size_t size, size_t offset,
+				    u32 features)
 {
-	struct kunit_dev *kdev;
 	struct drm_device *drm;
 	struct drm_driver *driver;
+	void *container;
 	int ret;
 
 	driver = kunit_kzalloc(test, sizeof(*driver), GFP_KERNEL);
@@ -117,11 +114,11 @@ drm_kunit_helper_alloc_drm_device(struct kunit *test, struct device *dev,
 		return ERR_PTR(-ENOMEM);
 
 	driver->driver_features = features;
-	kdev = devm_drm_dev_alloc(dev, driver, struct kunit_dev, base);
-	if (IS_ERR(kdev))
-		return ERR_CAST(kdev);
+	container = __devm_drm_dev_alloc(dev, driver, size, offset);
+	if (IS_ERR(container))
+		return ERR_CAST(container);
 
-	drm = &kdev->base;
+	drm = container + offset;
 	drm->mode_config.funcs = &drm_mode_config_funcs;
 
 	ret = drmm_mode_config_init(drm);
diff --git a/drivers/gpu/drm/tests/drm_kunit_helpers.h b/drivers/gpu/drm/tests/drm_kunit_helpers.h
index b4277fe92c38..70abc8c806c9 100644
--- a/drivers/gpu/drm/tests/drm_kunit_helpers.h
+++ b/drivers/gpu/drm/tests/drm_kunit_helpers.h
@@ -10,7 +10,14 @@ struct device *drm_kunit_helper_alloc_device(struct kunit *test);
 void drm_kunit_helper_free_device(struct kunit *test, struct device *dev);
 
 struct drm_device *
-drm_kunit_helper_alloc_drm_device(struct kunit *test, struct device *dev,
-				  u32 features);
+__drm_kunit_helper_alloc_drm_device(struct kunit *test, struct device *dev,
+				    size_t size, size_t offset,
+				    u32 features);
+
+#define drm_kunit_helper_alloc_drm_device(_test, _dev, _type, _member, _feat)	\
+	((_type *) __drm_kunit_helper_alloc_drm_device(_test, _dev,		\
+						       sizeof(type),		\
+						       offsetof(type, member),	\
+						       _feat))
 
 #endif // DRM_KUNIT_HELPERS_H_

-- 
2.38.1-b4-0.11.0-dev-d416f

  parent reply	other threads:[~2022-11-23 15:29 UTC|newest]

Thread overview: 95+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-23 15:25 [PATCH 00/24] drm: Introduce Kunit Tests to VC4 Maxime Ripard
2022-11-23 15:25 ` [PATCH 01/24] drm/tests: helpers: Rename the device init helper Maxime Ripard
2022-11-25  8:50   ` Javier Martinez Canillas
2022-11-25  8:50     ` Javier Martinez Canillas
2022-11-25 14:10   ` Maíra Canal
2022-11-25 14:10     ` Maíra Canal
2022-11-25 14:33     ` Maxime Ripard
2022-11-25 14:33       ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 02/24] drm/tests: helpers: Remove the name parameter Maxime Ripard
2022-11-25  8:57   ` Javier Martinez Canillas
2022-11-25  8:57     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 03/24] drm/tests: helpers: Create the device in another function Maxime Ripard
2022-11-25  9:06   ` Javier Martinez Canillas
2022-11-25  9:06     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 04/24] drm/tests: helpers: Switch to a platform_device Maxime Ripard
2022-11-25  9:25   ` Javier Martinez Canillas
2022-11-25  9:25     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 05/24] drm/tests: helpers: Make sure the device is bound Maxime Ripard
2022-11-25 10:19   ` Javier Martinez Canillas
2022-11-25 10:19     ` Javier Martinez Canillas
2022-11-23 15:25 ` Maxime Ripard [this message]
2022-11-25 10:46   ` [PATCH 06/24] drm/tests: kunit: Allow for a custom device struct to be allocated Javier Martinez Canillas
2022-11-25 10:46     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 07/24] drm/tests: helpers: Allow to pass a custom drm_driver Maxime Ripard
2022-11-25 10:48   ` Javier Martinez Canillas
2022-11-25 10:48     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 08/24] drm/tests: Add a test for DRM managed actions Maxime Ripard
2022-11-25 10:52   ` Javier Martinez Canillas
2022-11-25 10:52     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 09/24] drm/atomic: Constify the old/new state accessors Maxime Ripard
2022-11-25 10:54   ` Javier Martinez Canillas
2022-11-25 10:54     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 10/24] drm/vc4: kms: Sort the CRTCs by output before assigning them Maxime Ripard
2022-11-25 11:00   ` Javier Martinez Canillas
2022-11-25 11:00     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 11/24] drm/vc4: Constify container_of wrappers Maxime Ripard
2022-11-25 11:01   ` Javier Martinez Canillas
2022-11-25 11:01     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 12/24] drm/vc4: Move HVS state to main header Maxime Ripard
2022-11-25 11:04   ` Javier Martinez Canillas
2022-11-25 11:04     ` Javier Martinez Canillas
2022-11-23 15:25 ` [PATCH 13/24] drm/vc4: kms: Constify the HVS old/new state helpers Maxime Ripard
2022-11-25 11:05   ` Javier Martinez Canillas
2022-11-25 11:05     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 14/24] drm/vc4: txp: Reorder the variable assignments Maxime Ripard
2022-11-25 11:07   ` Javier Martinez Canillas
2022-11-25 11:07     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 15/24] drm/vc4: Add TXP encoder type Maxime Ripard
2022-11-25 11:11   ` Javier Martinez Canillas
2022-11-25 11:11     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 16/24] drm/vc4: txp: Initialise the CRTC before the encoder and connector Maxime Ripard
2022-11-25 11:12   ` Javier Martinez Canillas
2022-11-25 11:12     ` Javier Martinez Canillas
2022-11-28 11:04   ` (subset) " Maxime Ripard
2022-11-28 11:04     ` Maxime Ripard
2022-11-23 15:25 ` [PATCH 17/24] drm/vc4: crtc: Pass the device and data in vc4_crtc_init Maxime Ripard
2022-11-25 11:13   ` Javier Martinez Canillas
2022-11-25 11:13     ` Javier Martinez Canillas
2022-11-28 11:05   ` (subset) " Maxime Ripard
2022-11-28 11:05     ` Maxime Ripard
2022-11-23 15:26 ` [PATCH 18/24] drm/vc4: crtc: Introduce a lower-level crtc init helper Maxime Ripard
2022-11-25 11:29   ` Javier Martinez Canillas
2022-11-25 11:29     ` Javier Martinez Canillas
2022-11-23 15:26 ` [PATCH 19/24] drm/vc4: crtc: Make encoder lookup helper public Maxime Ripard
2022-11-25 11:30   ` Javier Martinez Canillas
2022-11-25 11:30     ` Javier Martinez Canillas
2022-11-23 15:26 ` [PATCH 20/24] drm/vc4: crtc: Provide a CRTC name Maxime Ripard
2022-11-25 11:32   ` Javier Martinez Canillas
2022-11-25 11:32     ` Javier Martinez Canillas
2022-11-28 11:05   ` (subset) " Maxime Ripard
2022-11-28 11:05     ` Maxime Ripard
2022-11-23 15:26 ` [PATCH 21/24] drm/vc4: hvs: Provide a function to initialize the HVS structure Maxime Ripard
2022-11-25 11:41   ` Javier Martinez Canillas
2022-11-25 11:41     ` Javier Martinez Canillas
2022-11-23 15:26 ` [PATCH 22/24] drm/vc4: tests: Introduce a mocking infrastructure Maxime Ripard
2022-11-25 12:53   ` kernel test robot
2022-11-25 12:53     ` kernel test robot
2022-11-23 15:26 ` [PATCH 23/24] drm/vc4: tests: Fail the current test if we access a register Maxime Ripard
2022-11-23 15:26 ` [PATCH 24/24] drm/vc4: tests: Add unit test suite for the PV muxing Maxime Ripard
2022-11-24  8:31 ` [PATCH 00/24] drm: Introduce Kunit Tests to VC4 David Gow
2022-11-24  8:31   ` David Gow
2022-11-24 14:01   ` Maxime Ripard
2022-11-24 14:01     ` Maxime Ripard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221123-rpi-kunit-tests-v1-6-051a0bb60a16@cerno.tech \
    --to=maxime@cerno.tech \
    --cc=airlied@gmail.com \
    --cc=brendan.higgins@linux.dev \
    --cc=daniel@ffwll.ch \
    --cc=dave.stevenson@raspberrypi.com \
    --cc=davidgow@google.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=javierm@redhat.com \
    --cc=kunit-dev@googlegroups.com \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mairacanal@riseup.net \
    --cc=mripard@kernel.org \
    --cc=tzimmermann@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.