linux-ext4 - Re: Review of ext4-online-defrag-for-relevant-files.patch

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: <20080914005024.GH26128@mit.edu>
Date:	Sat, 13 Sep 2008 20:50:24 -0400
From:	Theodore Tso <tytso@....EDU>
To:	a-fujita@...jp.nec.com
Cc:	Takashi Sato <t-sato@...jp.nec.com>, linux-ext4@...r.kernel.org
Subject: Re: Review of ext4-online-defrag-for-relevant-files.patch

On Sat, Sep 13, 2008 at 03:22:14PM -0400, Theodore Ts'o wrote:
> There is also no mention if this new ioctl anywhere in the patch
> commit.  Was this intentional?  If so, what is the justification for the
> new ioctl, and is it safe to simply remove the ioctl and change the
> userspace program to use the FIBMAP ioctl?

Ah, I see one potential reason.  FIBMAP only allows 32-bit block
numbers, and EXT4_IOC_FIBMAP allows 64-bit block numbers.  Still, it's
only used in one place, to get the physical block number of the first
block of the file to use as the "goal" block.  This could be retrieved
using either FIEMAP or the EXT4_IOC_EXTENTS_INFO ioctl (not
surprising, since the two are equivalent.  :-)

Of course, the fact that the defrag code is using the first block of
the inode is the goal block, while other places get_free_extents()
assumes that the only free extents which are interesting are the ones
in the block group of the inode seems funny....  but I haven't had the
time to thoroughly understand the algorithms used by the defrag
program.

Still, it seems clear to me that step one is getting the helper ioctls
designed correctly and merged into ext4, and then we can gradually
work to get the rest of the defrag patches merged.

Best 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