lkml.org 
[lkml]   [2010]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRE: [GIT PULL] XFS update for 2.6.33-rc1
Date
From
Randy Dunlap wrote:
> On Wed, 16 Dec 2009 21:41:12 -0600 Alex Elder wrote:
>
>> Dave Chinner wrote:
>>> 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....
>>
>> I didn't get them either when I built in my environment.
>>
>> Dave, your patch looks good to me, and Randy has acked it
>> so I'm going to pull it into our XFS top-of-tree. I'll
>> hold off sending yet another pull request to Linus for
>> now. Thanks for addressing it quickly.
>>
>> -Alex
>
> Hi Alex,
> Dave's patch is now needed in Linus's mainline kernel tree...

Yes, I noticed that. I forgot I hadn't requested it get pulled.
Sorry. Working on that today.

-Alex


>>> /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.
>
>
> ---
> ~Randy



\
 
 \ /
  Last update: 2010-01-08 19:13    [W:0.065 / U:0.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site