Re: [RFC][PATCH 0/8] ext4 online defrag (ver 0.8)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

Mingming Cao wrote:
>> Summary of patches:
>> * These patches are replaced with ext4 online defrag patches
>>   which are already in the ext4 git tree (linux-2.6.26-rc8).
>>
>> [PATCH 1/8]  Exchange the blocks between two inodes
>> - Exchange the data blocks between the temporary inode and
>>   the original inode.
>>
>> [PATCH 2/8] Read and write file data with memory page
>> - Read the file data from the old blocks to the page and
>>   write the file data on the page into the new blocks.
>>
>> [PATCH 3/8] Allocate new contiguous blocks with mballoc
>> - Search contiguous free blocks with mutil-block allocation
>>   and allocate them for the temporary inode.
>>
>> [PATCH 4/8] Main function of ext4 online defrag and ioctl implementation
>> - Create the temporary inode and do defrag per
>>   defrag_size (defalut 64MB).
>>
> 
> I am curious about the ordering here. It seems reverse the order of the
> first 4 patches make the logic more clear to me, start from patch 4,
> then 3, then 2 and last 1?  At lease when I read the describtion of the
> first patch I was puzzled by a minute what is this coming from
> 
> Just share my thoughts.
> 

Ah, as you said, the ordering the patches is confusing.
I'll reconstruct my patches order(4-3-2-1-5-6-7-8) to be clear
in the future release.

Thanks,
Akira
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux