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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <4802DA0D.4060502@rs.jp.nec.com>
Date:	Mon, 14 Apr 2008 13:14:05 +0900
From:	Akira Fujita <a-fujita@...jp.nec.com>
To:	cmm@...ibm.com
CC:	linux-ext4@...r.kernel.org
Subject: Re: [RFC][PATCH 0/8] ext4 online defrag (ver 0.8)

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html