xfs
[Top] [All Lists]

Re: [GIT PULL] XFS update for 2.6.33-rc1

To: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
Subject: Re: [GIT PULL] XFS update for 2.6.33-rc1
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 17 Dec 2009 11:20:07 +1100
Cc: Alex Elder <aelder@xxxxxxx>, torvalds@xxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx
In-reply-to: <20091216135512.2675a0e3.randy.dunlap@xxxxxxxxxx>
References: <200912162126.nBGLQAbZ012632@xxxxxxxxxxxxxxxxxxxxxx> <20091216135512.2675a0e3.randy.dunlap@xxxxxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17)
On Wed, Dec 16, 2009 at 01:55:12PM -0800, Randy Dunlap wrote:
> On Wed, 16 Dec 2009 15:26:10 -0600 Alex Elder wrote:
> 
> > One more XFS update for -rc1, now that the new XFS tracing code has
> > been committed.
> 
> Sorry if I have missed it, but have the printk format warnings
> on i386 builds been fixed?

I don't see any here w/ gcc 4.3.4 x86-64 compiling i386 kernels....

/me hates gcc's warning inconsistencies with a passion.

> e.g.:
> 
> In file included from linux-next-20091216/include/trace/ftrace.h:398,
>                  from linux-next-20091216/include/trace/define_trace.h:69,
>                  from linux-next-20091216/fs/xfs/linux-2.6/xfs_trace.h:1369,
>                  from linux-next-20091216/fs/xfs/linux-2.6/xfs_trace.c:75:
> linux-next-20091216/fs/xfs/linux-2.6/./xfs_trace.h: In function 
> 'ftrace_raw_output_id_xfs_iomap_enter':
> linux-next-20091216/fs/xfs/linux-2.6/./xfs_trace.h:830: warning: format 
> '%llx' expects type 'long long unsigned int', but argument 12 has type 
> 'xfs_fsblock_t'

Patch that should fix this below.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx


XFS: Format xfs_fsblock_t correctly in tracing code.

Fixes printk build warnings on i386.

Reported-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>
---
 fs/xfs/linux-2.6/xfs_trace.h |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/fs/xfs/linux-2.6/xfs_trace.h b/fs/xfs/linux-2.6/xfs_trace.h
index c40834b..d4ded59 100644
--- a/fs/xfs/linux-2.6/xfs_trace.h
+++ b/fs/xfs/linux-2.6/xfs_trace.h
@@ -815,7 +815,7 @@ TRACE_EVENT(name, \
        ), \
        TP_printk("dev %d:%d ino 0x%llx size 0x%llx new_size 0x%llx " \
                  "offset 0x%llx count %zd flags %s " \
-                 "startoff 0x%llx startblock 0x%llx blockcount 0x%llx", \
+                 "startoff 0x%llx startblock %s blockcount 0x%llx", \
                  MAJOR(__entry->dev), MINOR(__entry->dev), \
                  __entry->ino, \
                  __entry->size, \
@@ -824,7 +824,7 @@ TRACE_EVENT(name, \
                  __entry->count, \
                  __print_flags(__entry->flags, "|", BMAPI_FLAGS), \
                  __entry->startoff, \
-                 __entry->startblock, \
+                 xfs_fmtfsblock(__entry->startblock), \
                  __entry->blockcount) \
 )
 DEFINE_IOMAP_EVENT(xfs_iomap_enter);
@@ -1201,7 +1201,7 @@ TRACE_EVENT(name, \
        TP_printk("dev %d:%d agno %u agbno %u minlen %u maxlen %u mod %u " \
                  "prod %u minleft %u total %u alignment %u minalignslop %u " \
                  "len %u type %s otype %s wasdel %d wasfromfl %d isfl %d " \
-                 "userdata %d firstblock 0x%llx", \
+                 "userdata %d firstblock %s", \
                  MAJOR(__entry->dev), MINOR(__entry->dev), \
                  __entry->agno, \
                  __entry->agbno, \
@@ -1220,7 +1220,7 @@ TRACE_EVENT(name, \
                  __entry->wasfromfl, \
                  __entry->isfl, \
                  __entry->userdata, \
-                 __entry->firstblock) \
+                 xfs_fmtfsblock(__entry->firstblock)) \
 )
 
 DEFINE_ALLOC_EVENT(xfs_alloc_exact_done);

<Prev in Thread] Current Thread [Next in Thread>