binder: mark binder_alloc_exhaustive_test as slow

[PATCH] binder: mark binder_alloc_exhaustive_test as slow

Posted by Carlos Llamas 3 months, 2 weeks ago

The binder_alloc_exhaustive_test kunit test takes over 30s to complete
and the kunit framework reports:

  # binder_alloc_exhaustive_test: Test should be marked slow (runtime: 33.842881934s)

Mark the test as suggested to silence the warning.

Cc: Tiffany Yang <ynaffit@google.com>
Signed-off-by: Carlos Llamas <cmllamas@google.com>
---
 drivers/android/tests/binder_alloc_kunit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/android/tests/binder_alloc_kunit.c b/drivers/android/tests/binder_alloc_kunit.c
index 9b884d977f76..7f9cc003bbe3 100644
--- a/drivers/android/tests/binder_alloc_kunit.c
+++ b/drivers/android/tests/binder_alloc_kunit.c
@@ -554,7 +554,7 @@ static void binder_alloc_test_exit(struct kunit *test)
 static struct kunit_case binder_alloc_test_cases[] = {
 	KUNIT_CASE(binder_alloc_test_init_freelist),
 	KUNIT_CASE(binder_alloc_test_mmap),
-	KUNIT_CASE(binder_alloc_exhaustive_test),
+	KUNIT_CASE_SLOW(binder_alloc_exhaustive_test),
 	{}
 };
 
-- 
2.51.1.821.gb6fe4d2222-goog

Re: [PATCH] binder: mark binder_alloc_exhaustive_test as slow

Posted by Alice Ryhl 3 months, 1 week ago

On Fri, Oct 24, 2025 at 04:15:02PM +0000, Carlos Llamas wrote:
> The binder_alloc_exhaustive_test kunit test takes over 30s to complete
> and the kunit framework reports:
> 
>   # binder_alloc_exhaustive_test: Test should be marked slow (runtime: 33.842881934s)
> 
> Mark the test as suggested to silence the warning.
> 
> Cc: Tiffany Yang <ynaffit@google.com>
> Signed-off-by: Carlos Llamas <cmllamas@google.com>

Reviewed-by: Alice Ryhl <aliceryhl@google.com>

Re: [PATCH] binder: mark binder_alloc_exhaustive_test as slow

Posted by Tiffany Yang 3 months, 2 weeks ago

Carlos Llamas <cmllamas@google.com> writes:

> The binder_alloc_exhaustive_test kunit test takes over 30s to complete
> and the kunit framework reports:

>    # binder_alloc_exhaustive_test: Test should be marked slow (runtime:  
> 33.842881934s)

> Mark the test as suggested to silence the warning.

> Cc: Tiffany Yang <ynaffit@google.com>
> Signed-off-by: Carlos Llamas <cmllamas@google.com>
> ---
>   drivers/android/tests/binder_alloc_kunit.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

> diff --git a/drivers/android/tests/binder_alloc_kunit.c  
> b/drivers/android/tests/binder_alloc_kunit.c
> index 9b884d977f76..7f9cc003bbe3 100644
> --- a/drivers/android/tests/binder_alloc_kunit.c
> +++ b/drivers/android/tests/binder_alloc_kunit.c
> @@ -554,7 +554,7 @@ static void binder_alloc_test_exit(struct kunit *test)
>   static struct kunit_case binder_alloc_test_cases[] = {
>   	KUNIT_CASE(binder_alloc_test_init_freelist),
>   	KUNIT_CASE(binder_alloc_test_mmap),
> -	KUNIT_CASE(binder_alloc_exhaustive_test),
> +	KUNIT_CASE_SLOW(binder_alloc_exhaustive_test),
>   	{}
>   };

Reviewed-by: Tiffany Yang <ynaffit@google.com>

-- 
Tiffany Y. Yang