Re: [PATCH v3 01/10] iov_iter: Fix some checkpatch complaints in kunit tests
From: Joe Perches <joe@perches.com>
Date: 2023-11-16 04:08:23
Also in:
linux-block, linux-fsdevel, linux-kselftest, linux-mm, lkml
On Wed, 2023-11-15 at 15:49 +0000, David Howells wrote:
Fix some checkpatch complaints in the new iov_iter kunit tests:
(1) Some lines had eight spaces instead of a tab at the start.
(2) Checkpatch doesn't like (void*)(unsigned long)0xnnnnnULL, so switch to
using POISON_POINTER_DELTA plus an offset instead.That's because checkpatch is fundamentally stupid and that's a false positive.
quoted hunk ↗ jump to hunk
diff --git a/lib/kunit_iov_iter.c b/lib/kunit_iov_iter.c
[]
quoted hunk ↗ jump to hunk
@@ -548,7 +548,7 @@ static void __init iov_kunit_extract_pages_kvec(struct kunit *test) size_t offset0 = LONG_MAX; for (i = 0; i < ARRAY_SIZE(pagelist); i++) - pagelist[i] = (void *)(unsigned long)0xaa55aa55aa55aa55ULL; + pagelist[i] = (void *)POISON_POINTER_DELTA + 0x5a;
I think the original is easier to understand or would best be replaced by a single #define without the addition.
quoted hunk ↗ jump to hunk
@@ -626,7 +626,7 @@ static void __init iov_kunit_extract_pages_bvec(struct kunit *test) size_t offset0 = LONG_MAX; for (i = 0; i < ARRAY_SIZE(pagelist); i++) - pagelist[i] = (void *)(unsigned long)0xaa55aa55aa55aa55ULL; + pagelist[i] = (void *)POISON_POINTER_DELTA + 0x5a;
etc...