Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1654114 > unrolled thread

[PATCH] selftests: sync: Skip the test if kernel support is not found

Started byMichael Ellerman <mpe@ellerman.id.au>
First post2017-05-31 12:50 +0200
Last post2017-06-02 23:50 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] selftests: sync: Skip the test if kernel support is not found Michael Ellerman <mpe@ellerman.id.au> - 2017-05-31 12:50 +0200
    Re: [PATCH] selftests: sync: Skip the test if kernel support is not  found Gustavo Padovan <gustavo.padovan@collabora.com> - 2017-05-31 16:50 +0200
      Re: [PATCH] selftests: sync: Skip the test if kernel support is not  found Shuah Khan <shuahkh@osg.samsung.com> - 2017-05-31 17:00 +0200
    Re: [PATCH] selftests: sync: Skip the test if kernel support is not  found Shuah Khan <shuahkh@osg.samsung.com> - 2017-06-02 23:50 +0200

#1654114 — [PATCH] selftests: sync: Skip the test if kernel support is not found

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-05-31 12:50 +0200
Subject[PATCH] selftests: sync: Skip the test if kernel support is not found
Message-ID<tN9mq-3zx-9@gated-at.bofh.it>
The "Sync framework" test doesn't work if the kernel has no support,
obviously. Rather than reporting a failure, check for the kernel support
by looking for /sys/kernel/debug/sync/sw_sync, and if not found skip the
test.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
---
 tools/testing/selftests/sync/sync_test.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/tools/testing/selftests/sync/sync_test.c b/tools/testing/selftests/sync/sync_test.c
index 9ea08d9f0b13..62fa666e501a 100644
--- a/tools/testing/selftests/sync/sync_test.c
+++ b/tools/testing/selftests/sync/sync_test.c
@@ -29,6 +29,7 @@
 #include <unistd.h>
 #include <stdlib.h>
 #include <sys/types.h>
+#include <sys/stat.h>
 #include <sys/wait.h>
 
 #include "synctest.h"
@@ -52,10 +53,22 @@ static int run_test(int (*test)(void), char *name)
 	exit(test());
 }
 
+static int sync_api_supported(void)
+{
+	struct stat sbuf;
+
+	return 0 == stat("/sys/kernel/debug/sync/sw_sync", &sbuf);
+}
+
 int main(void)
 {
 	int err = 0;
 
+	if (!sync_api_supported()) {
+		printf("SKIP: Sync framework not supported by kernel\n");
+		return 0;
+	}
+
 	printf("[RUN]\tTesting sync framework\n");
 
 	err += RUN_TEST(test_alloc_timeline);
-- 
2.7.4

[toc] | [next] | [standalone]

#1654322 — Re: [PATCH] selftests: sync: Skip the test if kernel support is not found

FromGustavo Padovan <gustavo.padovan@collabora.com>
Date2017-05-31 16:50 +0200
SubjectRe: [PATCH] selftests: sync: Skip the test if kernel support is not found
Message-ID<tNd6G-67G-15@gated-at.bofh.it>
In reply to#1654114
Hi Michael,

On 05/31/2017 07:40 PM, Michael Ellerman wrote:
> The "Sync framework" test doesn't work if the kernel has no support,
> obviously. Rather than reporting a failure, check for the kernel support
> by looking for /sys/kernel/debug/sync/sw_sync, and if not found skip the
> test.
> 
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
> ---
>  tools/testing/selftests/sync/sync_test.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/tools/testing/selftests/sync/sync_test.c b/tools/testing/selftests/sync/sync_test.c
> index 9ea08d9f0b13..62fa666e501a 100644
> --- a/tools/testing/selftests/sync/sync_test.c
> +++ b/tools/testing/selftests/sync/sync_test.c
> @@ -29,6 +29,7 @@
>  #include <unistd.h>
>  #include <stdlib.h>
>  #include <sys/types.h>
> +#include <sys/stat.h>
>  #include <sys/wait.h>
>  
>  #include "synctest.h"
> @@ -52,10 +53,22 @@ static int run_test(int (*test)(void), char *name)
>  	exit(test());
>  }
>  
> +static int sync_api_supported(void)
> +{
> +	struct stat sbuf;
> +
> +	return 0 == stat("/sys/kernel/debug/sync/sw_sync", &sbuf);
> +}
> +
>  int main(void)
>  {
>  	int err = 0;
>  
> +	if (!sync_api_supported()) {
> +		printf("SKIP: Sync framework not supported by kernel\n");
> +		return 0;
> +	}
> +
>  	printf("[RUN]\tTesting sync framework\n");
>  
>  	err += RUN_TEST(test_alloc_timeline);
> 

Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>

-- 
Gustavo Padovan
Collabora Ltd.

[toc] | [prev] | [next] | [standalone]

#1654331 — Re: [PATCH] selftests: sync: Skip the test if kernel support is not found

FromShuah Khan <shuahkh@osg.samsung.com>
Date2017-05-31 17:00 +0200
SubjectRe: [PATCH] selftests: sync: Skip the test if kernel support is not found
Message-ID<tNdgm-6b3-27@gated-at.bofh.it>
In reply to#1654322
On 05/31/2017 08:44 AM, Gustavo Padovan wrote:
> Hi Michael,
> 
> On 05/31/2017 07:40 PM, Michael Ellerman wrote:
>> The "Sync framework" test doesn't work if the kernel has no support,
>> obviously. Rather than reporting a failure, check for the kernel support
>> by looking for /sys/kernel/debug/sync/sw_sync, and if not found skip the
>> test.
>>
>> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
>> ---
>>  tools/testing/selftests/sync/sync_test.c | 13 +++++++++++++
>>  1 file changed, 13 insertions(+)
>>
>> diff --git a/tools/testing/selftests/sync/sync_test.c b/tools/testing/selftests/sync/sync_test.c
>> index 9ea08d9f0b13..62fa666e501a 100644
>> --- a/tools/testing/selftests/sync/sync_test.c
>> +++ b/tools/testing/selftests/sync/sync_test.c
>> @@ -29,6 +29,7 @@
>>  #include <unistd.h>
>>  #include <stdlib.h>
>>  #include <sys/types.h>
>> +#include <sys/stat.h>
>>  #include <sys/wait.h>
>>  
>>  #include "synctest.h"
>> @@ -52,10 +53,22 @@ static int run_test(int (*test)(void), char *name)
>>  	exit(test());
>>  }
>>  
>> +static int sync_api_supported(void)
>> +{
>> +	struct stat sbuf;
>> +
>> +	return 0 == stat("/sys/kernel/debug/sync/sw_sync", &sbuf);
>> +}
>> +
>>  int main(void)
>>  {
>>  	int err = 0;
>>  
>> +	if (!sync_api_supported()) {
>> +		printf("SKIP: Sync framework not supported by kernel\n");
>> +		return 0;
>> +	}
>> +
>>  	printf("[RUN]\tTesting sync framework\n");
>>  
>>  	err += RUN_TEST(test_alloc_timeline);
>>
> 
> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com>
> 

Thanks both. I will pull this into kselftest for 4.13-rc1

-- Shuah

[toc] | [prev] | [next] | [standalone]

#1656583 — Re: [PATCH] selftests: sync: Skip the test if kernel support is not found

FromShuah Khan <shuahkh@osg.samsung.com>
Date2017-06-02 23:50 +0200
SubjectRe: [PATCH] selftests: sync: Skip the test if kernel support is not found
Message-ID<tO2Cd-6B6-7@gated-at.bofh.it>
In reply to#1654114
On 05/31/2017 04:40 AM, Michael Ellerman wrote:
> The "Sync framework" test doesn't work if the kernel has no support,
> obviously. Rather than reporting a failure, check for the kernel support
> by looking for /sys/kernel/debug/sync/sw_sync, and if not found skip the
> test.
> 
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>

Thanks. Applied to linux-kselftest next for 4.13-rc1

-- Shuah

> ---
>  tools/testing/selftests/sync/sync_test.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/tools/testing/selftests/sync/sync_test.c b/tools/testing/selftests/sync/sync_test.c
> index 9ea08d9f0b13..62fa666e501a 100644
> --- a/tools/testing/selftests/sync/sync_test.c
> +++ b/tools/testing/selftests/sync/sync_test.c
> @@ -29,6 +29,7 @@
>  #include <unistd.h>
>  #include <stdlib.h>
>  #include <sys/types.h>
> +#include <sys/stat.h>
>  #include <sys/wait.h>
>  
>  #include "synctest.h"
> @@ -52,10 +53,22 @@ static int run_test(int (*test)(void), char *name)
>  	exit(test());
>  }
>  
> +static int sync_api_supported(void)
> +{
> +	struct stat sbuf;
> +
> +	return 0 == stat("/sys/kernel/debug/sync/sw_sync", &sbuf);
> +}
> +
>  int main(void)
>  {
>  	int err = 0;
>  
> +	if (!sync_api_supported()) {
> +		printf("SKIP: Sync framework not supported by kernel\n");
> +		return 0;
> +	}
> +
>  	printf("[RUN]\tTesting sync framework\n");
>  
>  	err += RUN_TEST(test_alloc_timeline);
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web