linux-ext4 - Re: [RFC][PATCH 1/3] Add EXT4_IOC_MOVE

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20090613132148.GK24336@mit.edu>
Date:	Sat, 13 Jun 2009 09:21:48 -0400
From:	Theodore Tso <tytso@....edu>
To:	Akira Fujita <a-fujita@...jp.nec.com>
Cc:	linux-ext4@...r.kernel.org
Subject: Re: [RFC][PATCH 1/3] Add EXT4_IOC_MOVE_EXT ioctl and related
	functions

On Fri, May 22, 2009 at 04:06:16PM +0900, Akira Fujita wrote:
> ext4: online defrag -- Add EXT4_IOC_MOVE_EXT ioctl and related functions.
> 
> From: Akira Fujita <a-fujita@...jp.nec.com>
> 
> The EXT4_IOC_MOVE_EXT exchanges the blocks between orig_fd and donor_fd,
> and then write the file data of orig_fd to donor_fd.
> ext4_mext_move_extent() is the main fucntion of ext4 online defrag,
> and this patch includes all functions related to ext4 online defrag.

Akira-san,

Thank you for all of the hard work and preserverance with the online
defrag work!  This patch is much, *much* better; I've done a quick
review, and I've only noted two things, which I've updated in the
version I've now moved into the stable portion of the patch queue.
One is that nothing actually uses orig_fd in the move_extent
structure; so to avoid confusion, and I've renamed it to "reserved",
and used explicit __u32 fields for the reserved and donor_fd fields.
Also, I've renamed ext4_mext_move_extent() to ext4_move_extents();
since it is the one published interface, I wanted it to have an
easier-to-understand name.

As a side note, the static functions in fs/ext4/move_extent.c really
don't need the ext4_mext prefix, since static functions don't have
namespace issues that require a consistent naming scheme.  (Sometimes
a shorter name can also be useful since it avoids needing to line wrap
function calls with a long list of parameters.)

I haven't done extensive testing on the patch to make sure that
nothing bad happens if the file is actively been modified while the
defrag progam is running, but the interfaces look good, which is what
matters in terms of pushing it to the stable queue.

Again, many thanks,

Regards,

						- Ted
--
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