Re: [PATCH 1/5] drm/tests: shmem: Swap names of export tests

[Boris Brezillon <boris.brezillon@collabora.com>]

On Fri, 12 Dec 2025 17:00:32 +0100
Thomas Zimmermann <tzimmermann@suse.de> wrote:

> GEM SHMEM has 2 helpers for exporting S/G tables. Swap the names of
> the rsp. tests, so that each matches the helper it tests.
> 
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> Fixes: 93032ae634d4 ("drm/test: add a test suite for GEM objects backed by shmem")
> Cc: dri-devel@lists.freedesktop.org
> Cc: <stable@vger.kernel.org> # v6.8+

Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>

> ---
>  drivers/gpu/drm/tests/drm_gem_shmem_test.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/tests/drm_gem_shmem_test.c b/drivers/gpu/drm/tests/drm_gem_shmem_test.c
> index 68f2c3162354..872881ec9c30 100644
> --- a/drivers/gpu/drm/tests/drm_gem_shmem_test.c
> +++ b/drivers/gpu/drm/tests/drm_gem_shmem_test.c
> @@ -194,7 +194,7 @@ static void drm_gem_shmem_test_vmap(struct kunit *test)
>   * scatter/gather table large enough to accommodate the backing memory
>   * is successfully exported.
>   */
> -static void drm_gem_shmem_test_get_pages_sgt(struct kunit *test)
> +static void drm_gem_shmem_test_get_sg_table(struct kunit *test)
>  {
>  	struct drm_device *drm_dev = test->priv;
>  	struct drm_gem_shmem_object *shmem;
> @@ -236,7 +236,7 @@ static void drm_gem_shmem_test_get_pages_sgt(struct kunit *test)
>   * backing pages are pinned and a scatter/gather table large enough to
>   * accommodate the backing memory is successfully exported.
>   */
> -static void drm_gem_shmem_test_get_sg_table(struct kunit *test)
> +static void drm_gem_shmem_test_get_pages_sgt(struct kunit *test)
>  {
>  	struct drm_device *drm_dev = test->priv;
>  	struct drm_gem_shmem_object *shmem;
> @@ -366,8 +366,8 @@ static struct kunit_case drm_gem_shmem_test_cases[] = {
>  	KUNIT_CASE(drm_gem_shmem_test_obj_create_private),
>  	KUNIT_CASE(drm_gem_shmem_test_pin_pages),
>  	KUNIT_CASE(drm_gem_shmem_test_vmap),
> -	KUNIT_CASE(drm_gem_shmem_test_get_pages_sgt),
>  	KUNIT_CASE(drm_gem_shmem_test_get_sg_table),
> +	KUNIT_CASE(drm_gem_shmem_test_get_pages_sgt),
>  	KUNIT_CASE(drm_gem_shmem_test_madvise),
>  	KUNIT_CASE(drm_gem_shmem_test_purge),
>  	{}