From: "Liam R. Howlett" <Liam.Howlett@oracle.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: maple-tree@lists.infradead.org, linux-mm@kvack.org,
linux-kernel@vger.kernel.org,
"Matthew Wilcox (Oracle)" <willy@infradead.org>
Subject: Re: [PATCH] test suite: Use %zu to print size_t
Date: Wed, 2 Apr 2025 22:53:52 -0400 [thread overview]
Message-ID: <ubvc2fxn6zl6g6kb5ndzazyjj6ds54onawxvar5j5vfu6gz2dy@k2mfcvhatoza> (raw)
In-Reply-To: <20250403003311.359917-1-Liam.Howlett@oracle.com>
* Liam R. Howlett <Liam.Howlett@oracle.com> [250402 20:33]:
> From: "Matthew Wilcox (Oracle)" <willy@infradead.org>
>
> On 32-bit, we can't use %lu to print a size_t variable and gcc warns us
> about it. Shame it doesn't warn about it on 64-bit.
>
> Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
> Fixes: cc86e0c2f306 (radix tree test suite: add support for slab bulk APIs)
> Signed-off-by: Liam R. Howlett <Liam.Howlett@oracle.com>
> ---
>
> Note that this requires the patch to fix the headers [1] for the testing
> to build. Unfortunately, that has not landed, but the patches do not
> conflict so we should push this upstream anyways.
>
> [1]. https://lore.kernel.org/all/20250321-fix-radix-tree-build-v1-1-838a1e6540e2@samsung.com/
Sorry, This should not have been 'in-reply-to' the previous older patch
set. Let me know if you want me to resend.
Thanks,
Liam
>
> tools/testing/shared/linux.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/testing/shared/linux.c b/tools/testing/shared/linux.c
> index 66dbb362385f3..0f97fb0d19e19 100644
> --- a/tools/testing/shared/linux.c
> +++ b/tools/testing/shared/linux.c
> @@ -150,7 +150,7 @@ void kmem_cache_free(struct kmem_cache *cachep, void *objp)
> void kmem_cache_free_bulk(struct kmem_cache *cachep, size_t size, void **list)
> {
> if (kmalloc_verbose)
> - pr_debug("Bulk free %p[0-%lu]\n", list, size - 1);
> + pr_debug("Bulk free %p[0-%zu]\n", list, size - 1);
>
> pthread_mutex_lock(&cachep->lock);
> for (int i = 0; i < size; i++)
> @@ -168,7 +168,7 @@ int kmem_cache_alloc_bulk(struct kmem_cache *cachep, gfp_t gfp, size_t size,
> size_t i;
>
> if (kmalloc_verbose)
> - pr_debug("Bulk alloc %lu\n", size);
> + pr_debug("Bulk alloc %zu\n", size);
>
> pthread_mutex_lock(&cachep->lock);
> if (cachep->nr_objs >= size) {
> --
> 2.43.0
>
next prev parent reply other threads:[~2025-04-03 2:54 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-01 17:16 [PATCH 00/12] maple_tree: iterator state changes Liam R. Howlett
2023-11-01 17:16 ` [PATCH 01/12] maple_tree: Remove unnecessary default labels from switch statements Liam R. Howlett
2023-11-01 17:16 ` [PATCH 02/12] maple_tree: Make mas_erase() more robust Liam R. Howlett
2023-11-01 17:16 ` [PATCH 03/12] maple_tree: Move debug check to __mas_set_range() Liam R. Howlett
2023-11-01 17:16 ` [PATCH 04/12] maple_tree: Add end of node tracking to the maple state Liam R. Howlett
2023-11-01 17:16 ` [PATCH 05/12] maple_tree: Use cached node end in mas_next() Liam R. Howlett
2023-11-01 17:16 ` [PATCH 06/12] maple_tree: Use cached node end in mas_destroy() Liam R. Howlett
2023-11-01 17:16 ` [PATCH 07/12] maple_tree: Clean up inlines for some functions Liam R. Howlett
2023-11-01 17:16 ` [PATCH 08/12] maple_tree: Separate ma_state node from status Liam R. Howlett
2023-11-02 8:42 ` kernel test robot
2023-11-02 17:39 ` Liam R. Howlett
2023-11-06 15:41 ` [PATCH] maple_tree: Fix comments about MAS_* Liam R. Howlett
2023-11-06 15:45 ` [PATCH] maple_tree: Update forking to separate maple state and node Liam R. Howlett
2025-04-03 0:33 ` [PATCH] test suite: Use %zu to print size_t Liam R. Howlett
2025-04-03 2:53 ` Liam R. Howlett [this message]
2023-11-01 17:16 ` [PATCH 09/12] maple_tree: Remove mas_searchable() Liam R. Howlett
2023-11-01 17:16 ` [PATCH 10/12] maple_tree: Use maple state end for write operations Liam R. Howlett
2023-11-01 17:16 ` [PATCH 11/12] maple_tree: Don't find node end in mtree_lookup_walk() Liam R. Howlett
2023-11-01 17:16 ` [PATCH 12/12] maple_tree: mtree_range_walk() clean up Liam R. Howlett
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ubvc2fxn6zl6g6kb5ndzazyjj6ds54onawxvar5j5vfu6gz2dy@k2mfcvhatoza \
--to=liam.howlett@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=maple-tree@lists.infradead.org \
--cc=willy@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox