* [PATCH 1/1 linux-next] ceph: use %zu for size_t display
@ 2015-01-03 17:02 Fabian Frederick
2015-01-05 9:31 ` Ilya Dryomov
0 siblings, 1 reply; 2+ messages in thread
From: Fabian Frederick @ 2015-01-03 17:02 UTC (permalink / raw)
To: linux-kernel; +Cc: Fabian Frederick, Sage Weil, ceph-devel
Fixes the following 32-bit compilation warning:
fs/ceph/addr.c:1419:2: warning: format '%lu' expects argument of type
'long unsigned int', but argument 5 has type 'size_t' [-Wformat=]
Signed-off-by: Fabian Frederick <fabf@skynet.be>
---
fs/ceph/addr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c
index f5013d9..c81c0e0 100644
--- a/fs/ceph/addr.c
+++ b/fs/ceph/addr.c
@@ -1416,7 +1416,7 @@ void ceph_fill_inline_data(struct inode *inode, struct page *locked_page,
}
}
- dout("fill_inline_data %p %llx.%llx len %lu locked_page %p\n",
+ dout("fill_inline_data %p %llx.%llx len %zu locked_page %p\n",
inode, ceph_vinop(inode), len, locked_page);
if (len > 0) {
--
2.1.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/1 linux-next] ceph: use %zu for size_t display
2015-01-03 17:02 [PATCH 1/1 linux-next] ceph: use %zu for size_t display Fabian Frederick
@ 2015-01-05 9:31 ` Ilya Dryomov
0 siblings, 0 replies; 2+ messages in thread
From: Ilya Dryomov @ 2015-01-05 9:31 UTC (permalink / raw)
To: Fabian Frederick; +Cc: Linux Kernel Mailing List, Sage Weil, Ceph Development
On Sat, Jan 3, 2015 at 8:02 PM, Fabian Frederick <fabf@skynet.be> wrote:
> Fixes the following 32-bit compilation warning:
> fs/ceph/addr.c:1419:2: warning: format '%lu' expects argument of type
> 'long unsigned int', but argument 5 has type 'size_t' [-Wformat=]
>
> Signed-off-by: Fabian Frederick <fabf@skynet.be>
> ---
> fs/ceph/addr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c
> index f5013d9..c81c0e0 100644
> --- a/fs/ceph/addr.c
> +++ b/fs/ceph/addr.c
> @@ -1416,7 +1416,7 @@ void ceph_fill_inline_data(struct inode *inode, struct page *locked_page,
> }
> }
>
> - dout("fill_inline_data %p %llx.%llx len %lu locked_page %p\n",
> + dout("fill_inline_data %p %llx.%llx len %zu locked_page %p\n",
> inode, ceph_vinop(inode), len, locked_page);
>
> if (len > 0) {
This was fixed a while ago but hasn't been pushed to linux-next. I'll
try to make a point of pushing such fixes immediately in the future.
Thanks,
Ilya
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-01-05 9:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-03 17:02 [PATCH 1/1 linux-next] ceph: use %zu for size_t display Fabian Frederick
2015-01-05 9:31 ` Ilya Dryomov
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).