linux-kselftest.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] selftests/arm64: Fix grammatical error in string literals
@ 2025-08-26 21:49 Nikola Z. Ivanov
  2025-08-26 23:27 ` Bagas Sanjaya
  2025-08-27 10:32 ` Mark Brown
  0 siblings, 2 replies; 3+ messages in thread
From: Nikola Z. Ivanov @ 2025-08-26 21:49 UTC (permalink / raw)
  To: bagasdotme, Markus.Elfring, broonie
  Cc: shuah, will, mark.rutland, ebiggers, catalin.marinas,
	martin.petersen, ardb, thiago.bauermann, linux-arm-kernel,
	linux-kselftest, linux-kernel, linux-kernel-mentees,
	Nikola Z. Ivanov

Fix grammatical error in <past tense verb> + <infinitive>
construct related to memory allocation checks.
In essence change "Failed to allocated" to "Failed to allocate".

Signed-off-by: Nikola Z. Ivanov <zlatistiv@gmail.com>
---
Changes in v2:
- More descriptive commit message

Original title is "Fix typos in malloc return value check"

 tools/testing/selftests/arm64/fp/fp-stress.c   | 2 +-
 tools/testing/selftests/arm64/fp/kernel-test.c | 4 ++--
 tools/testing/selftests/arm64/gcs/gcs-stress.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/testing/selftests/arm64/fp/fp-stress.c b/tools/testing/selftests/arm64/fp/fp-stress.c
index 74e23208b94c..ddc01efea3f9 100644
--- a/tools/testing/selftests/arm64/fp/fp-stress.c
+++ b/tools/testing/selftests/arm64/fp/fp-stress.c
@@ -549,7 +549,7 @@ int main(int argc, char **argv)
 
 	evs = calloc(tests, sizeof(*evs));
 	if (!evs)
-		ksft_exit_fail_msg("Failed to allocated %d epoll events\n",
+		ksft_exit_fail_msg("Failed to allocate %d epoll events\n",
 				   tests);
 
 	for (i = 0; i < cpus; i++) {
diff --git a/tools/testing/selftests/arm64/fp/kernel-test.c b/tools/testing/selftests/arm64/fp/kernel-test.c
index e3cec3723ffa..0c40007d1282 100644
--- a/tools/testing/selftests/arm64/fp/kernel-test.c
+++ b/tools/testing/selftests/arm64/fp/kernel-test.c
@@ -188,13 +188,13 @@ static bool create_socket(void)
 
 	ref = malloc(digest_len);
 	if (!ref) {
-		printf("Failed to allocated %d byte reference\n", digest_len);
+		printf("Failed to allocate %d byte reference\n", digest_len);
 		return false;
 	}
 
 	digest = malloc(digest_len);
 	if (!digest) {
-		printf("Failed to allocated %d byte digest\n", digest_len);
+		printf("Failed to allocate %d byte digest\n", digest_len);
 		return false;
 	}
 
diff --git a/tools/testing/selftests/arm64/gcs/gcs-stress.c b/tools/testing/selftests/arm64/gcs/gcs-stress.c
index bbc7f4950c13..cf316d78ea97 100644
--- a/tools/testing/selftests/arm64/gcs/gcs-stress.c
+++ b/tools/testing/selftests/arm64/gcs/gcs-stress.c
@@ -433,7 +433,7 @@ int main(int argc, char **argv)
 
 	evs = calloc(tests, sizeof(*evs));
 	if (!evs)
-		ksft_exit_fail_msg("Failed to allocated %d epoll events\n",
+		ksft_exit_fail_msg("Failed to allocate %d epoll events\n",
 				   tests);
 
 	for (i = 0; i < gcs_threads; i++)
-- 
2.50.1


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

* Re: [PATCH v2] selftests/arm64: Fix grammatical error in string literals
  2025-08-26 21:49 [PATCH v2] selftests/arm64: Fix grammatical error in string literals Nikola Z. Ivanov
@ 2025-08-26 23:27 ` Bagas Sanjaya
  2025-08-27 10:32 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Bagas Sanjaya @ 2025-08-26 23:27 UTC (permalink / raw)
  To: Nikola Z. Ivanov, Markus.Elfring, broonie
  Cc: shuah, will, mark.rutland, ebiggers, catalin.marinas,
	martin.petersen, ardb, thiago.bauermann, linux-arm-kernel,
	linux-kselftest, linux-kernel, linux-kernel-mentees

On 8/27/25 04:49, Nikola Z. Ivanov wrote:
> Fix grammatical error in <past tense verb> + <infinitive>
> construct related to memory allocation checks.
> In essence change "Failed to allocated" to "Failed to allocate".
> 

LGTM, thanks!

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

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

* Re: [PATCH v2] selftests/arm64: Fix grammatical error in string literals
  2025-08-26 21:49 [PATCH v2] selftests/arm64: Fix grammatical error in string literals Nikola Z. Ivanov
  2025-08-26 23:27 ` Bagas Sanjaya
@ 2025-08-27 10:32 ` Mark Brown
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Brown @ 2025-08-27 10:32 UTC (permalink / raw)
  To: Nikola Z. Ivanov
  Cc: bagasdotme, Markus.Elfring, shuah, will, mark.rutland, ebiggers,
	catalin.marinas, martin.petersen, ardb, thiago.bauermann,
	linux-arm-kernel, linux-kselftest, linux-kernel,
	linux-kernel-mentees

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

On Wed, Aug 27, 2025 at 12:49:13AM +0300, Nikola Z. Ivanov wrote:
> Fix grammatical error in <past tense verb> + <infinitive>
> construct related to memory allocation checks.
> In essence change "Failed to allocated" to "Failed to allocate".
> 
> Signed-off-by: Nikola Z. Ivanov <zlatistiv@gmail.com>
> ---
> Changes in v2:
> - More descriptive commit message

Repeating my previous tag:

Reviewed-by: Mark Brown <broonie@kernel.org>

Please don't drop tags, it causes people to have to spend time repeating
review.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

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

end of thread, other threads:[~2025-08-27 10:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-26 21:49 [PATCH v2] selftests/arm64: Fix grammatical error in string literals Nikola Z. Ivanov
2025-08-26 23:27 ` Bagas Sanjaya
2025-08-27 10:32 ` Mark Brown

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).