Hi Aneesh,
> On Mon, Mar 24, 2008 at 08:00:54PM +0900, Akira Fujita wrote:
>>>
>> Is EXT4_IOC_FIEMAP going to be added to the ext4 patch queue?
>> I will try to use EXT4_IOC_FIEMAP instead of EXT4_IOC_EXTENTS_INFO
>> in ext4 online defrag when kernel supports it.
>
> Can you also look at the sparse warnings with the patches ?
>
I have already released the fixed patch before, but it is not in the ext4 patch queue yet.
Mingming, could you replace new ext4-online-defrag-free-space-fragmentation.patch
with old one in the ext4 patch queue?
Regards, Akira
Signed-off-by: Akira Fujita <[email protected]>
---
defrag.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -Nurp linux-2.6.25-rc6-full/fs/ext4/defrag.c linux-2.6.25-rc6-fix/fs/ext4/defrag.c
--- linux-2.6.25-rc6-full/fs/ext4/defrag.c 2008-03-24 20:37:10.000000000 +0900
+++ linux-2.6.25-rc6-fix/fs/ext4/defrag.c 2008-03-24 20:40:22.000000000 +0900
@@ -454,7 +454,7 @@ static int ext4_defrag_move_victim(struc
ext_info->ext[i].len, goal, DEFRAG_FORCE_VICTIM, &ext);
if (ret < 0) {
printk(KERN_ERR "ext4 defrag: "
- "Moving victim file failed. ino [%lu]\n",
+ "Moving victim file failed. ino [%llu]\n",
ext_info->ino);
goto err;
}
On Mon, Mar 24, 2008 at 09:31:03PM +0900, Akira Fujita wrote:
> Hi Aneesh,
> > On Mon, Mar 24, 2008 at 08:00:54PM +0900, Akira Fujita wrote:
> >>>
> >> Is EXT4_IOC_FIEMAP going to be added to the ext4 patch queue?
> >> I will try to use EXT4_IOC_FIEMAP instead of EXT4_IOC_EXTENTS_INFO
> >> in ext4 online defrag when kernel supports it.
> >
> > Can you also look at the sparse warnings with the patches ?
> >
> I have already released the fixed patch before, but it is not in the ext4 patch queue yet.
> Mingming, could you replace new ext4-online-defrag-free-space-fragmentation.patch
> with old one in the ext4 patch queue?
>
CHECK fs/ext4/defrag.c
fs/ext4/defrag.c:181:7: warning: cast to restricted type
fs/ext4/defrag.c:181:7: warning: cast from restricted type
fs/ext4/defrag.c:613:21: warning: incorrect type in argument 1
(different address spaces)
fs/ext4/defrag.c:613:21: expected void [noderef] <asn:1>*to
fs/ext4/defrag.c:613:21: got struct ext4_group_data_info *<noident>
fs/ext4/defrag.c:629:24: warning: incorrect type in argument 1
(different address spaces)
fs/ext4/defrag.c:629:24: expected void [noderef] <asn:1>*to
fs/ext4/defrag.c:629:24: got struct ext4_extents_info *<noident>
fs/ext4/defrag.c:849:29: warning: potentially expensive pointer
subtraction
fs/ext4/defrag.c:912:22: warning: potentially expensive pointer
subtraction
you would need http://www.kernel.org/pub/software/devel/sparse/
to find and fix these warnings.
-aneesh
Aneesh Kumar K.V wrote:
> On Mon, Mar 24, 2008 at 09:31:03PM +0900, Akira Fujita wrote:
>
>> Hi Aneesh,
>>
>>> On Mon, Mar 24, 2008 at 08:00:54PM +0900, Akira Fujita wrote:
>>>
>>>>>
>>>>>
>>>> Is EXT4_IOC_FIEMAP going to be added to the ext4 patch queue?
>>>> I will try to use EXT4_IOC_FIEMAP instead of EXT4_IOC_EXTENTS_INFO
>>>> in ext4 online defrag when kernel supports it.
>>>>
>>> Can you also look at the sparse warnings with the patches ?
>>>
>>>
>> I have already released the fixed patch before, but it is not in the ext4 patch queue yet.
>> Mingming, could you replace new ext4-online-defrag-free-space-fragmentation.patch
>> with old one in the ext4 patch queue?
>>
>>
>
> CHECK fs/ext4/defrag.c
> fs/ext4/defrag.c:181:7: warning: cast to restricted type
> fs/ext4/defrag.c:181:7: warning: cast from restricted type
> fs/ext4/defrag.c:613:21: warning: incorrect type in argument 1
> (different address spaces)
> fs/ext4/defrag.c:613:21: expected void [noderef] <asn:1>*to
> fs/ext4/defrag.c:613:21: got struct ext4_group_data_info *<noident>
> fs/ext4/defrag.c:629:24: warning: incorrect type in argument 1
> (different address spaces)
> fs/ext4/defrag.c:629:24: expected void [noderef] <asn:1>*to
> fs/ext4/defrag.c:629:24: got struct ext4_extents_info *<noident>
> fs/ext4/defrag.c:849:29: warning: potentially expensive pointer
> subtraction
> fs/ext4/defrag.c:912:22: warning: potentially expensive pointer
> subtraction
>
> you would need http://www.kernel.org/pub/software/devel/sparse/
> to find and fix these warnings.
>
>
Oops, there are a lot of warnings. =-O
Thank you for letting me know.
I will release fixed defrag patches in a few days.
Regards, Akira
On Thu, 2008-03-27 at 19:15 +0900, Akira Fujita wrote:
> Aneesh Kumar K.V wrote:
> > On Mon, Mar 24, 2008 at 09:31:03PM +0900, Akira Fujita wrote:
> >
> >> Hi Aneesh,
> >>
> >>> On Mon, Mar 24, 2008 at 08:00:54PM +0900, Akira Fujita wrote:
> >>>
> >>>>>
> >>>>>
> >>>> Is EXT4_IOC_FIEMAP going to be added to the ext4 patch queue?
> >>>> I will try to use EXT4_IOC_FIEMAP instead of EXT4_IOC_EXTENTS_INFO
> >>>> in ext4 online defrag when kernel supports it.
> >>>>
> >>> Can you also look at the sparse warnings with the patches ?
> >>>
> >>>
> >> I have already released the fixed patch before, but it is not in the ext4 patch queue yet.
> >> Mingming, could you replace new ext4-online-defrag-free-space-fragmentation.patch
> >> with old one in the ext4 patch queue?
> >>
> >>
> >
> > CHECK fs/ext4/defrag.c
> > fs/ext4/defrag.c:181:7: warning: cast to restricted type
> > fs/ext4/defrag.c:181:7: warning: cast from restricted type
> > fs/ext4/defrag.c:613:21: warning: incorrect type in argument 1
> > (different address spaces)
> > fs/ext4/defrag.c:613:21: expected void [noderef] <asn:1>*to
> > fs/ext4/defrag.c:613:21: got struct ext4_group_data_info *<noident>
> > fs/ext4/defrag.c:629:24: warning: incorrect type in argument 1
> > (different address spaces)
> > fs/ext4/defrag.c:629:24: expected void [noderef] <asn:1>*to
> > fs/ext4/defrag.c:629:24: got struct ext4_extents_info *<noident>
> > fs/ext4/defrag.c:849:29: warning: potentially expensive pointer
> > subtraction
> > fs/ext4/defrag.c:912:22: warning: potentially expensive pointer
> > subtraction
> >
> > you would need http://www.kernel.org/pub/software/devel/sparse/
> > to find and fix these warnings.
> >
> >
> Oops, there are a lot of warnings. =-O
> Thank you for letting me know.
> I will release fixed defrag patches in a few days.
As part of transition to move ext4 related header files out of
include/linux, and move to fs/ext4, I also updated the defrag patches to
adapt this change. You might want to checkout the defrag patches in
patch queue and merge the updates before release the new version.
Cheers,
Mingming