[PATCH v2] kselftests:grammer correction

Pavan Bobba posted 1 patch 1 month, 2 weeks ago
tools/testing/selftests/acct/acct_syscall.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v2] kselftests:grammer correction
Posted by Pavan Bobba 1 month, 2 weeks ago
correct a minor grammer mistake

Signed-off-by: Pavan Bobba <opensource206@gmail.com>
---
v1 -> v2 : changed the "corrected" word to "correct" word in message
           description to follow the convention
 tools/testing/selftests/acct/acct_syscall.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/acct/acct_syscall.c b/tools/testing/selftests/acct/acct_syscall.c
index 87c044fb9293..ee2894e4f7bc 100644
--- a/tools/testing/selftests/acct/acct_syscall.c
+++ b/tools/testing/selftests/acct/acct_syscall.c
@@ -22,7 +22,7 @@ int main(void)
 	ksft_print_header();
 	ksft_set_plan(1);
 
-	// Check if test is run a root
+	// Check if test is run as root
 	if (geteuid()) {
 		ksft_exit_skip("This test needs root to run!\n");
 		return 1;
-- 
2.43.0
Re: [PATCH v2] kselftests:grammer correction
Posted by Al Viro 1 month, 1 week ago
On Wed, Aug 20, 2025 at 06:28:36PM +0530, Pavan Bobba wrote:
> correct a minor grammer mistake

Demonstrations of the Muphry's law are amusing, but this is the the
6th copy (all with unique Message-ID, at that) all with the same patch.
You might at least vary the misspelling...
Re: [PATCH v2] kselftests:grammer correction
Posted by Shivam Chaudhary 1 month, 1 week ago
On 20/08/25 6:28 PM, Pavan Bobba wrote:
> correct a minor grammer mistake

grammer should be grammar, i think not present in spelling.txt and 
missed by checkpatch.

Also fix your subsystem prefix it should be selftests: acct:

send v3

-cvam

>
> Signed-off-by: Pavan Bobba <opensource206@gmail.com>
> ---
> v1 -> v2 : changed the "corrected" word to "correct" word in message
>             description to follow the convention
>   tools/testing/selftests/acct/acct_syscall.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/acct/acct_syscall.c b/tools/testing/selftests/acct/acct_syscall.c
> index 87c044fb9293..ee2894e4f7bc 100644
> --- a/tools/testing/selftests/acct/acct_syscall.c
> +++ b/tools/testing/selftests/acct/acct_syscall.c
> @@ -22,7 +22,7 @@ int main(void)
>   	ksft_print_header();
>   	ksft_set_plan(1);
>   
> -	// Check if test is run a root
> +	// Check if test is run as root
>   	if (geteuid()) {
>   		ksft_exit_skip("This test needs root to run!\n");
>   		return 1;
Re: [PATCH v2] kselftests:grammer correction
Posted by Shivam Chaudhary 1 month, 2 weeks ago
On 20/08/25 6:28 PM, Pavan Bobba wrote:
> correct a minor grammer mistake

grammer should be grammar , spelling.txt does not have this, that is why 
missed by checkpatch.

Also use proper subject prefix kselftests: should be selftests: acct:

please Send v3

> Signed-off-by: Pavan Bobba <opensource206@gmail.com>
> ---
> v1 -> v2 : changed the "corrected" word to "correct" word in message
>             description to follow the convention
>   tools/testing/selftests/acct/acct_syscall.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/acct/acct_syscall.c b/tools/testing/selftests/acct/acct_syscall.c
> index 87c044fb9293..ee2894e4f7bc 100644
> --- a/tools/testing/selftests/acct/acct_syscall.c
> +++ b/tools/testing/selftests/acct/acct_syscall.c
> @@ -22,7 +22,7 @@ int main(void)
>   	ksft_print_header();
>   	ksft_set_plan(1);
>   
> -	// Check if test is run a root
> +	// Check if test is run as root
>   	if (geteuid()) {
>   		ksft_exit_skip("This test needs root to run!\n");
>   		return 1;