* next: ext4 build failure
@ 2012-12-04 9:29 Jiri Slaby
2012-12-04 13:12 ` Tao Ma
2012-12-04 13:27 ` Theodore Ts'o
0 siblings, 2 replies; 6+ messages in thread
From: Jiri Slaby @ 2012-12-04 9:29 UTC (permalink / raw)
To: boyu.mt; +Cc: Theodore Ts'o, linux-ext4, LKML
Hi,
commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
Author: Tao Ma <boyu.mt@taobao.com>
Date: Sun Dec 2 20:30:52 2012 -0500
ext4: let fallocate handle inline data correctly
added
static int int ext4_convert_inline_data(struct inode *inode)
^^^^^^^
when EXT4_FS_XATTR is unset.
Care to fix that?
thanks,
--
js
suse labs
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: next: ext4 build failure
2012-12-04 9:29 next: ext4 build failure Jiri Slaby
@ 2012-12-04 13:12 ` Tao Ma
2012-12-04 13:27 ` Theodore Ts'o
1 sibling, 0 replies; 6+ messages in thread
From: Tao Ma @ 2012-12-04 13:12 UTC (permalink / raw)
To: Jiri Slaby; +Cc: boyu.mt, Theodore Ts'o, linux-ext4, LKML
Hi Jiri,
Thanks for the report.
The corresponding fix is already sent to Ted in the patch named "ext4:
Fix inline data build warning found by kernel build testing.", but it
seems that Ted hasn't pushed it into ext4dev yet.
Thanks
Tao
On 12/04/2012 05:29 PM, Jiri Slaby wrote:
> Hi,
>
> commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
> Author: Tao Ma <boyu.mt@taobao.com>
> Date: Sun Dec 2 20:30:52 2012 -0500
>
> ext4: let fallocate handle inline data correctly
>
> added
>
> static int int ext4_convert_inline_data(struct inode *inode)
>
> ^^^^^^^
>
> when EXT4_FS_XATTR is unset.
>
> Care to fix that?
>
> thanks,
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: next: ext4 build failure
2012-12-04 9:29 next: ext4 build failure Jiri Slaby
2012-12-04 13:12 ` Tao Ma
@ 2012-12-04 13:27 ` Theodore Ts'o
2012-12-04 13:29 ` Jiri Slaby
2012-12-04 13:40 ` Tao Ma
1 sibling, 2 replies; 6+ messages in thread
From: Theodore Ts'o @ 2012-12-04 13:27 UTC (permalink / raw)
To: Jiri Slaby; +Cc: boyu.mt, linux-ext4, LKML
On Tue, Dec 04, 2012 at 10:29:00AM +0100, Jiri Slaby wrote:
> Hi,
>
> commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
> Author: Tao Ma <boyu.mt@taobao.com>
> Date: Sun Dec 2 20:30:52 2012 -0500
>
> ext4: let fallocate handle inline data correctly
>
> added
>
> static int int ext4_convert_inline_data(struct inode *inode)
>
> ^^^^^^^
>
> when EXT4_FS_XATTR is unset.
>
> Care to fix that?
Oops. Thanks for catching this!!!
- Ted
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: next: ext4 build failure
2012-12-04 13:27 ` Theodore Ts'o
@ 2012-12-04 13:29 ` Jiri Slaby
2012-12-04 13:40 ` Tao Ma
1 sibling, 0 replies; 6+ messages in thread
From: Jiri Slaby @ 2012-12-04 13:29 UTC (permalink / raw)
To: Theodore Ts'o, boyu.mt, linux-ext4, LKML
On 12/04/2012 02:27 PM, Theodore Ts'o wrote:
> On Tue, Dec 04, 2012 at 10:29:00AM +0100, Jiri Slaby wrote:
>> Hi,
>>
>> commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
>> Author: Tao Ma <boyu.mt@taobao.com>
>> Date: Sun Dec 2 20:30:52 2012 -0500
>>
>> ext4: let fallocate handle inline data correctly
>>
>> added
>>
>> static int int ext4_convert_inline_data(struct inode *inode)
>>
>> ^^^^^^^
>>
>> when EXT4_FS_XATTR is unset.
>>
>> Care to fix that?
>
> Oops. Thanks for catching this!!!
This is not the only one BTW, see:
http://decibel.fi.muni.cz/gitweb/?p=linux.git;a=commitdiff;h=1a166497fe3d76e5af8af02e2ee5a967b2829b93
thanks,
--
js
suse labs
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: next: ext4 build failure
2012-12-04 13:27 ` Theodore Ts'o
2012-12-04 13:29 ` Jiri Slaby
@ 2012-12-04 13:40 ` Tao Ma
2012-12-04 13:42 ` Tao Ma
1 sibling, 1 reply; 6+ messages in thread
From: Tao Ma @ 2012-12-04 13:40 UTC (permalink / raw)
To: Theodore Ts'o, Jiri Slaby, boyu.mt, linux-ext4, LKML
Hi Ted,
On 12/04/2012 09:27 PM, Theodore Ts'o wrote:
> On Tue, Dec 04, 2012 at 10:29:00AM +0100, Jiri Slaby wrote:
>> Hi,
>>
>> commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
>> Author: Tao Ma <boyu.mt@taobao.com>
>> Date: Sun Dec 2 20:30:52 2012 -0500
>>
>> ext4: let fallocate handle inline data correctly
>>
>> added
>>
>> static int int ext4_convert_inline_data(struct inode *inode)
>>
>> ^^^^^^^
>>
>> when EXT4_FS_XATTR is unset.
>>
>> Care to fix that?
>
> Oops. Thanks for catching this!!!
Please push the patch with the subject "[PATCH] ext4: Fix inline data
build warning found by kernel build testing." and these error should go
away. And also Dan's smatch has found another issue and I have sent out
the patch with subject "[PATCH] ext4: Fix inline data build warning
found by kernel build testing.".
Thanks
Tao
>
> - Ted
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: next: ext4 build failure
2012-12-04 13:40 ` Tao Ma
@ 2012-12-04 13:42 ` Tao Ma
0 siblings, 0 replies; 6+ messages in thread
From: Tao Ma @ 2012-12-04 13:42 UTC (permalink / raw)
To: Theodore Ts'o, Jiri Slaby, boyu.mt, linux-ext4, LKML
On 12/04/2012 09:40 PM, Tao Ma wrote:
> Hi Ted,
> On 12/04/2012 09:27 PM, Theodore Ts'o wrote:
>> On Tue, Dec 04, 2012 at 10:29:00AM +0100, Jiri Slaby wrote:
>>> Hi,
>>>
>>> commit a7b0168d4a9bb78535827ddaf9c055963f5bd7aa
>>> Author: Tao Ma <boyu.mt@taobao.com>
>>> Date: Sun Dec 2 20:30:52 2012 -0500
>>>
>>> ext4: let fallocate handle inline data correctly
>>>
>>> added
>>>
>>> static int int ext4_convert_inline_data(struct inode *inode)
>>>
>>> ^^^^^^^
>>>
>>> when EXT4_FS_XATTR is unset.
>>>
>>> Care to fix that?
>>
>> Oops. Thanks for catching this!!!
> Please push the patch with the subject "[PATCH] ext4: Fix inline data
> build warning found by kernel build testing." and these error should go
> away. And also Dan's smatch has found another issue and I have sent out
> the patch with subject "[PATCH] ext4: Fix inline data build warning
> found by kernel build testing.".
sorry, I hit the send button too fast. The subject for the 2nd patch is
"[PATCH] ext4: Fix a build warning in __ext4_journalled_writepage."
Sorry for all the trouble.
Thanks
Tao
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-12-04 16:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-04 9:29 next: ext4 build failure Jiri Slaby
2012-12-04 13:12 ` Tao Ma
2012-12-04 13:27 ` Theodore Ts'o
2012-12-04 13:29 ` Jiri Slaby
2012-12-04 13:40 ` Tao Ma
2012-12-04 13:42 ` Tao Ma
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).