linux-kselftest.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] kselftest: mm: fix typos in test_vmalloc.sh
@ 2025-08-23 17:02 ally heev
  2025-08-24  6:53 ` Uladzislau Rezki
  0 siblings, 1 reply; 4+ messages in thread
From: ally heev @ 2025-08-23 17:02 UTC (permalink / raw)
  To: skhan, akpm, david, shuah
  Cc: ally heev, linux-mm, linux-kselftest, linux-kernel

Signed-off-by: ally heev <allyheev@gmail.com>
---
 tools/testing/selftests/mm/test_vmalloc.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/testing/selftests/mm/test_vmalloc.sh b/tools/testing/selftests/mm/test_vmalloc.sh
index d73b846736f1..d39096723fca 100755
--- a/tools/testing/selftests/mm/test_vmalloc.sh
+++ b/tools/testing/selftests/mm/test_vmalloc.sh
@@ -47,14 +47,14 @@ check_test_requirements()
 	fi
 }
 
-run_perfformance_check()
+run_performance_check()
 {
 	echo "Run performance tests to evaluate how fast vmalloc allocation is."
 	echo "It runs all test cases on one single CPU with sequential order."
 
 	modprobe $DRIVER $PERF_PARAM > /dev/null 2>&1
 	echo "Done."
-	echo "Ccheck the kernel message buffer to see the summary."
+	echo "Check the kernel message buffer to see the summary."
 }
 
 run_stability_check()
@@ -160,7 +160,7 @@ function run_test()
 		usage
 	else
 		if [[ "$1" = "performance" ]]; then
-			run_perfformance_check
+			run_performance_check
 		elif [[ "$1" = "stress" ]]; then
 			run_stability_check
 		elif [[ "$1" = "smoke" ]]; then
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] kselftest: mm: fix typos in test_vmalloc.sh
  2025-08-23 17:02 [PATCH] kselftest: mm: fix typos in test_vmalloc.sh ally heev
@ 2025-08-24  6:53 ` Uladzislau Rezki
  2025-08-25  0:17   ` Andrew Morton
  0 siblings, 1 reply; 4+ messages in thread
From: Uladzislau Rezki @ 2025-08-24  6:53 UTC (permalink / raw)
  To: ally heev
  Cc: skhan, akpm, david, shuah, linux-mm, linux-kselftest,
	linux-kernel

On Sat, Aug 23, 2025 at 10:32:08PM +0530, ally heev wrote:
> Signed-off-by: ally heev <allyheev@gmail.com>
> ---
>  tools/testing/selftests/mm/test_vmalloc.sh | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/testing/selftests/mm/test_vmalloc.sh b/tools/testing/selftests/mm/test_vmalloc.sh
> index d73b846736f1..d39096723fca 100755
> --- a/tools/testing/selftests/mm/test_vmalloc.sh
> +++ b/tools/testing/selftests/mm/test_vmalloc.sh
> @@ -47,14 +47,14 @@ check_test_requirements()
>  	fi
>  }
>  
> -run_perfformance_check()
> +run_performance_check()
>  {
>  	echo "Run performance tests to evaluate how fast vmalloc allocation is."
>  	echo "It runs all test cases on one single CPU with sequential order."
>  
>  	modprobe $DRIVER $PERF_PARAM > /dev/null 2>&1
>  	echo "Done."
> -	echo "Ccheck the kernel message buffer to see the summary."
> +	echo "Check the kernel message buffer to see the summary."
>  }
>  
>  run_stability_check()
> @@ -160,7 +160,7 @@ function run_test()
>  		usage
>  	else
>  		if [[ "$1" = "performance" ]]; then
> -			run_perfformance_check
> +			run_performance_check
>  		elif [[ "$1" = "stress" ]]; then
>  			run_stability_check
>  		elif [[ "$1" = "smoke" ]]; then
> -- 
> 2.34.1
> 
> 
./scripts/checkpatch.pl ./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch
WARNING: Missing commit description - Add an appropriate one

total: 0 errors, 1 warnings, 24 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Can you add a commit message?

Thank you!

--
Uladzislau Rezki

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] kselftest: mm: fix typos in test_vmalloc.sh
  2025-08-24  6:53 ` Uladzislau Rezki
@ 2025-08-25  0:17   ` Andrew Morton
  2025-08-25  2:53     ` ally heev
  0 siblings, 1 reply; 4+ messages in thread
From: Andrew Morton @ 2025-08-25  0:17 UTC (permalink / raw)
  To: Uladzislau Rezki
  Cc: ally heev, skhan, david, shuah, linux-mm, linux-kselftest,
	linux-kernel

On Sun, 24 Aug 2025 08:53:26 +0200 Uladzislau Rezki <urezki@gmail.com> wrote:

> >  run_stability_check()
> > @@ -160,7 +160,7 @@ function run_test()
> >  		usage
> >  	else
> >  		if [[ "$1" = "performance" ]]; then
> > -			run_perfformance_check
> > +			run_performance_check
> >  		elif [[ "$1" = "stress" ]]; then
> >  			run_stability_check
> >  		elif [[ "$1" = "smoke" ]]; then
> > -- 
> > 2.34.1
> > 
> > 
> ./scripts/checkpatch.pl ./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch
> WARNING: Missing commit description - Add an appropriate one
> 
> total: 0 errors, 1 warnings, 24 lines checked
> 
> NOTE: For some of the reported defects, checkpatch may be able to
>       mechanically convert to the typical style using --fix or --fix-inplace.
> 
> ./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch has style problems, please review.
> 
> NOTE: If any of the errors are false positives, please report
>       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> 
> Can you add a commit message?

A changelog would be identical to the subject.  checkpatch is being a
bit excessive here.

Oh well, I'll add the obvious text.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] kselftest: mm: fix typos in test_vmalloc.sh
  2025-08-25  0:17   ` Andrew Morton
@ 2025-08-25  2:53     ` ally heev
  0 siblings, 0 replies; 4+ messages in thread
From: ally heev @ 2025-08-25  2:53 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Uladzislau Rezki, skhan, david, shuah, linux-mm, linux-kselftest,
	linux-kernel

> A changelog would be identical to the subject.  checkpatch is being a
> bit excessive here.

Yes. Hence omitted it


On Mon, Aug 25, 2025 at 5:47 AM Andrew Morton <akpm@linux-foundation.org> wrote:
>
> On Sun, 24 Aug 2025 08:53:26 +0200 Uladzislau Rezki <urezki@gmail.com> wrote:
>
> > >  run_stability_check()
> > > @@ -160,7 +160,7 @@ function run_test()
> > >             usage
> > >     else
> > >             if [[ "$1" = "performance" ]]; then
> > > -                   run_perfformance_check
> > > +                   run_performance_check
> > >             elif [[ "$1" = "stress" ]]; then
> > >                     run_stability_check
> > >             elif [[ "$1" = "smoke" ]]; then
> > > --
> > > 2.34.1
> > >
> > >
> > ./scripts/checkpatch.pl ./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch
> > WARNING: Missing commit description - Add an appropriate one
> >
> > total: 0 errors, 1 warnings, 24 lines checked
> >
> > NOTE: For some of the reported defects, checkpatch may be able to
> >       mechanically convert to the typical style using --fix or --fix-inplace.
> >
> > ./0001-kselftest-mm-fix-typos-in-test_vmalloc.sh.patch has style problems, please review.
> >
> > NOTE: If any of the errors are false positives, please report
> >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> >
> > Can you add a commit message?
>
> A changelog would be identical to the subject.  checkpatch is being a
> bit excessive here.
>
> Oh well, I'll add the obvious text.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-08-25  2:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-23 17:02 [PATCH] kselftest: mm: fix typos in test_vmalloc.sh ally heev
2025-08-24  6:53 ` Uladzislau Rezki
2025-08-25  0:17   ` Andrew Morton
2025-08-25  2:53     ` ally heev

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).