Add a simple test that checks whether the action is indeed called right
away and that it is not called on the final drm_dev_put().

Signed-off-by: Michał Winiarski <michal.winiar...@intel.com>
---
 drivers/gpu/drm/tests/drm_managed_test.c | 28 ++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/drivers/gpu/drm/tests/drm_managed_test.c 
b/drivers/gpu/drm/tests/drm_managed_test.c
index c1fc1f0aac9b2..91863642efc13 100644
--- a/drivers/gpu/drm/tests/drm_managed_test.c
+++ b/drivers/gpu/drm/tests/drm_managed_test.c
@@ -41,6 +41,33 @@ static void drm_test_managed_run_action(struct kunit *test)
        KUNIT_EXPECT_TRUE_MSG(test, priv->action_done, "Release action was not 
called");
 }
 
+/*
+ * The test verifies that the release action is called immediately when
+ * drmm_release_action is called and that it is not called for a second time
+ * when the device is released.
+ */
+static void drm_test_managed_release_action(struct kunit *test)
+{
+       struct managed_test_priv *priv = test->priv;
+       int ret;
+
+       ret = drmm_add_action_or_reset(priv->drm, drm_action, priv);
+       KUNIT_EXPECT_EQ(test, ret, 0);
+
+       ret = drm_dev_register(priv->drm, 0);
+       KUNIT_ASSERT_EQ(test, ret, 0);
+
+       drmm_release_action(priv->drm, drm_action, priv);
+       KUNIT_EXPECT_TRUE_MSG(test, priv->action_done, "Release action was not 
called");
+       priv->action_done = false;
+
+       drm_dev_unregister(priv->drm);
+       drm_kunit_helper_free_device(test, priv->drm->dev);
+
+       KUNIT_EXPECT_FALSE_MSG(test, priv->action_done,
+                              "Unexpected release action call during cleanup");
+}
+
 static int drm_managed_test_init(struct kunit *test)
 {
        struct managed_test_priv *priv;
@@ -67,6 +94,7 @@ static int drm_managed_test_init(struct kunit *test)
 }
 
 static struct kunit_case drm_managed_tests[] = {
+       KUNIT_CASE(drm_test_managed_release_action),
        KUNIT_CASE(drm_test_managed_run_action),
        {}
 };
-- 
2.43.0

Reply via email to