| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: xfs build warning |
| From: | Alex Elder <aelder@xxxxxxx> |
| Date: | Thu, 21 Apr 2011 15:08:10 -0500 |
| Cc: | Randy Dunlap <randy.dunlap@xxxxxxxxxx>, xfs@xxxxxxxxxxx |
| In-reply-to: | <20110415173902.GA27378@xxxxxxxxxxxxx> |
| References: | <4DA876C2.8050606@xxxxxxxxxx> <20110415173902.GA27378@xxxxxxxxxxxxx> |
| Reply-to: | aelder@xxxxxxx |
On Fri, 2011-04-15 at 13:39 -0400, Christoph Hellwig wrote: > On Fri, Apr 15, 2011 at 09:48:02AM -0700, Randy Dunlap wrote: > > on i386, when CONFIG_LBDAF is not enabled: > > > > fs/xfs/linux-2.6/./xfs_trace.h:1354: warning: format '%llx' expects type > > 'long long unsigned int', but argument 22 has type 'xfs_fsblock_t' > > This should fix it: Looks good. Christoph, can I assume your signoff? I wonder whether we'll ever need to cast the other fields that are based on abstractions from __uint32_t (like xfs_agblock_t, xfs_extlen_t, ...). Reviewed-by: Alex Elder <aelder@xxxxxxx> |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: Files full of zeros with coreutils-8.11 and xfs (FIEMAP related?), Jim Meyering |
|---|---|
| Next by Date: | Re: [PATCH 3/4] xfs: exact busy extent tracking, Alex Elder |
| Previous by Thread: | Re: xfs build warning, Randy Dunlap |
| Next by Thread: | Re: xfs build warning, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |