1. Patchew
  2. linux
  3. View series

[PATCH v2] drm/tests: modeset: Add drm connector atomic_[duplicate/destroy]_state

Xiaolei Wang posted 1 patch 5 months, 2 weeks ago 
drivers/gpu/drm/tests/drm_client_modeset_test.c | 4 ++++
1 file changed, 4 insertions(+)
[PATCH v2] drm/tests: modeset: Add drm connector atomic_[duplicate/destroy]_state
Posted by Xiaolei Wang 5 months, 2 weeks ago 
Commit 66671944e176 ("drm/tests: helpers: Add atomic helpers")
added the atomic state path, so adding the drm connector
atomic_[duplicate/destroy]_state is also necessary.

WARNING: CPU: 0 PID: 96 at drivers/gpu/drm/drm_connector.c:234 drm_connector_init_only+0x934/0xee0
 Call trace:
  drm_connector_init_only+0x934/0xee0 (P)
  drmm_connector_init+0xe0/0x1b0
  drm_client_modeset_test_init+0x290/0x534
  kunit_try_run_case+0x110/0x3b4
  kunit_generic_run_threadfn_adapter+0x80/0xec
  kthread+0x3b8/0x6c0
  ret_from_fork+0x10/0x20

Fixes: 66671944e176 ("drm/tests: helpers: Add atomic helpers")
Signed-off-by: Xiaolei Wang <xiaolei.wang@windriver.com>
Reviewed-by: Maxime Ripard <mripard@kernel.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---

v1: https://patchew.org/linux/20250811063403.98739-1-xiaolei.wang@windriver.com/

v2: compared with v1, add Fixes tag

 drivers/gpu/drm/tests/drm_client_modeset_test.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/tests/drm_client_modeset_test.c b/drivers/gpu/drm/tests/drm_client_modeset_test.c
index 3f44fe5e92e4..e702a27c2368 100644
--- a/drivers/gpu/drm/tests/drm_client_modeset_test.c
+++ b/drivers/gpu/drm/tests/drm_client_modeset_test.c
@@ -12,6 +12,7 @@
 #include <drm/drm_modes.h>
 #include <drm/drm_modeset_helper_vtables.h>
 #include <drm/drm_probe_helper.h>
+#include <drm/drm_atomic_state_helper.h>
 
 struct drm_client_modeset_test_priv {
 	struct drm_device *drm;
@@ -48,6 +49,9 @@ static const struct drm_connector_helper_funcs drm_client_modeset_connector_help
 };
 
 static const struct drm_connector_funcs drm_client_modeset_connector_funcs = {
+	.reset                  = drm_atomic_helper_connector_reset,
+	.atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
+	.atomic_destroy_state   = drm_atomic_helper_connector_destroy_state,
 };
 
 static int drm_client_modeset_test_init(struct kunit *test)
-- 
2.43.0
Re: [PATCH v2] drm/tests: modeset: Add drm connector atomic_[duplicate/destroy]_state
Posted by Alexandre Ghiti 5 months, 1 week ago 
Hi Xiaolei,


On 8/27/25 10:45, Xiaolei Wang wrote:
> Commit 66671944e176 ("drm/tests: helpers: Add atomic helpers")
> added the atomic state path, so adding the drm connector
> atomic_[duplicate/destroy]_state is also necessary.
>
> WARNING: CPU: 0 PID: 96 at drivers/gpu/drm/drm_connector.c:234 drm_connector_init_only+0x934/0xee0
>   Call trace:
>    drm_connector_init_only+0x934/0xee0 (P)
>    drmm_connector_init+0xe0/0x1b0
>    drm_client_modeset_test_init+0x290/0x534
>    kunit_try_run_case+0x110/0x3b4
>    kunit_generic_run_threadfn_adapter+0x80/0xec
>    kthread+0x3b8/0x6c0
>    ret_from_fork+0x10/0x20
>
> Fixes: 66671944e176 ("drm/tests: helpers: Add atomic helpers")
> Signed-off-by: Xiaolei Wang <xiaolei.wang@windriver.com>
> Reviewed-by: Maxime Ripard <mripard@kernel.org>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
>
> v1: https://patchew.org/linux/20250811063403.98739-1-xiaolei.wang@windriver.com/
>
> v2: compared with v1, add Fixes tag
>
>   drivers/gpu/drm/tests/drm_client_modeset_test.c | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/drivers/gpu/drm/tests/drm_client_modeset_test.c b/drivers/gpu/drm/tests/drm_client_modeset_test.c
> index 3f44fe5e92e4..e702a27c2368 100644
> --- a/drivers/gpu/drm/tests/drm_client_modeset_test.c
> +++ b/drivers/gpu/drm/tests/drm_client_modeset_test.c
> @@ -12,6 +12,7 @@
>   #include <drm/drm_modes.h>
>   #include <drm/drm_modeset_helper_vtables.h>
>   #include <drm/drm_probe_helper.h>
> +#include <drm/drm_atomic_state_helper.h>
>   
>   struct drm_client_modeset_test_priv {
>   	struct drm_device *drm;
> @@ -48,6 +49,9 @@ static const struct drm_connector_helper_funcs drm_client_modeset_connector_help
>   };
>   
>   static const struct drm_connector_funcs drm_client_modeset_connector_funcs = {
> +	.reset                  = drm_atomic_helper_connector_reset,
> +	.atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
> +	.atomic_destroy_state   = drm_atomic_helper_connector_destroy_state,
>   };
>   
>   static int drm_client_modeset_test_init(struct kunit *test)


Tested-by: Alexandre Ghiti <alexghiti@rivosinc.com>

Thanks,

Alex
Re: [PATCH v2] drm/tests: modeset: Add drm connector atomic_[duplicate/destroy]_state
Posted by Xiaolei Wang 1 month, 2 weeks ago 
On 9/1/25 18:41, Alexandre Ghiti wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender 
> and know the content is safe.
>
> Hi Xiaolei,
>
>
> On 8/27/25 10:45, Xiaolei Wang wrote:
>> Commit 66671944e176 ("drm/tests: helpers: Add atomic helpers")
>> added the atomic state path, so adding the drm connector
>> atomic_[duplicate/destroy]_state is also necessary.
>>
>> WARNING: CPU: 0 PID: 96 at drivers/gpu/drm/drm_connector.c:234 
>> drm_connector_init_only+0x934/0xee0
>>   Call trace:
>>    drm_connector_init_only+0x934/0xee0 (P)
>>    drmm_connector_init+0xe0/0x1b0
>>    drm_client_modeset_test_init+0x290/0x534
>>    kunit_try_run_case+0x110/0x3b4
>>    kunit_generic_run_threadfn_adapter+0x80/0xec
>>    kthread+0x3b8/0x6c0
>>    ret_from_fork+0x10/0x20
>>
>> Fixes: 66671944e176 ("drm/tests: helpers: Add atomic helpers")
>> Signed-off-by: Xiaolei Wang <xiaolei.wang@windriver.com>
>> Reviewed-by: Maxime Ripard <mripard@kernel.org>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
>> ---
>>
>> v1: 
>> https://patchew.org/linux/20250811063403.98739-1-xiaolei.wang@windriver.com/
>>
>> v2: compared with v1, add Fixes tag
>>
>>   drivers/gpu/drm/tests/drm_client_modeset_test.c | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/tests/drm_client_modeset_test.c 
>> b/drivers/gpu/drm/tests/drm_client_modeset_test.c
>> index 3f44fe5e92e4..e702a27c2368 100644
>> --- a/drivers/gpu/drm/tests/drm_client_modeset_test.c
>> +++ b/drivers/gpu/drm/tests/drm_client_modeset_test.c
>> @@ -12,6 +12,7 @@
>>   #include <drm/drm_modes.h>
>>   #include <drm/drm_modeset_helper_vtables.h>
>>   #include <drm/drm_probe_helper.h>
>> +#include <drm/drm_atomic_state_helper.h>
>>
>>   struct drm_client_modeset_test_priv {
>>       struct drm_device *drm;
>> @@ -48,6 +49,9 @@ static const struct drm_connector_helper_funcs 
>> drm_client_modeset_connector_help
>>   };
>>
>>   static const struct drm_connector_funcs 
>> drm_client_modeset_connector_funcs = {
>> +     .reset                  = drm_atomic_helper_connector_reset,
>> +     .atomic_duplicate_state = 
>> drm_atomic_helper_connector_duplicate_state,
>> +     .atomic_destroy_state   = 
>> drm_atomic_helper_connector_destroy_state,
>>   };
>>
>>   static int drm_client_modeset_test_init(struct kunit *test)
>
Humble ping...

thanks

xiaolei

>
> Tested-by: Alexandre Ghiti <alexghiti@rivosinc.com>
>
> Thanks,
>
> Alex
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.