linux-kernel - Re: [PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAKYAXd_xpWg-=T_1PghL_u9wDKTX_Ey5mJxbgVVKth8NoexQQg@mail.gmail.com>
Date:	Tue, 18 Feb 2014 18:50:54 +0900
From:	Namjae Jeon <linkinjeon@...il.com>
To:	Lukáš Czerner <lczerner@...hat.com>
Cc:	viro@...iv.linux.org.uk, david@...morbit.com, bpm@....com,
	tytso@....edu, adilger.kernel@...ger.ca, jack@...e.cz,
	mtk.manpages@...il.com, linux-fsdevel@...r.kernel.org,
	xfs@....sgi.com, linux-ext4@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Namjae Jeon <namjae.jeon@...sung.com>,
	Ashish Sangwan <a.sangwan@...sung.com>
Subject: Re: [PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE_RANGE
 for fallocate

2014-02-18 18:05 GMT+09:00, Lukáš Czerner <lczerner@...hat.com>:
> On Wed, 12 Feb 2014, Namjae Jeon wrote:
>
>> Date: Wed, 12 Feb 2014 11:28:35 +0900
>> From: Namjae Jeon <linkinjeon@...il.com>
>> To: Lukáš Czerner <lczerner@...hat.com>
>> Cc: viro@...iv.linux.org.uk, david@...morbit.com, bpm@....com,
>> tytso@....edu,
>>     adilger.kernel@...ger.ca, jack@...e.cz, mtk.manpages@...il.com,
>>     linux-fsdevel@...r.kernel.org, xfs@....sgi.com,
>>     linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org,
>>     Namjae Jeon <namjae.jeon@...sung.com>,
>>     Ashish Sangwan <a.sangwan@...sung.com>
>> Subject: Re: [PATCH RESEND 3/10] ext4: Add support
>> FALLOC_FL_COLLAPSE_RANGE
>>     for fallocate
>>
>> 2014-02-11 20:59 GMT+09:00, Lukáš Czerner <lczerner@...hat.com>:
>> > On Sun, 2 Feb 2014, Namjae Jeon wrote:
>> >
>> >> Date: Sun,  2 Feb 2014 14:44:34 +0900
>> >> From: Namjae Jeon <linkinjeon@...il.com>
>> >> To: viro@...iv.linux.org.uk, david@...morbit.com, bpm@....com,
>> >> tytso@....edu,
>> >>     adilger.kernel@...ger.ca, jack@...e.cz, mtk.manpages@...il.com
>> >> Cc: linux-fsdevel@...r.kernel.org, xfs@....sgi.com,
>> >>     linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org,
>> >>     Namjae Jeon <linkinjeon@...il.com>, Namjae Jeon
>> >> <namjae.jeon@...sung.com>,
>> >>     Ashish Sangwan <a.sangwan@...sung.com>
>> >> Subject: [PATCH RESEND 3/10] ext4: Add support
>> >> FALLOC_FL_COLLAPSE_RANGE
>> >> for
>> >>     fallocate
>> >>
>> >> From: Namjae Jeon <namjae.jeon@...sung.com>
>> >>
>> >> Add support FALLOC_FL_COLLAPSE_RANGE for fallocate
>> Hi Lukas.
>> >
>> > Description is missing here, please provide FALLOC_FL_COLLAPSE_RANGE
>> > description so people know what it's supposed to be doing.
>> >
>> > more comments bellow.
>> Okay, I will udpate.
>> >
>> > Thanks!
>> > -Lukas
>
> Hi,
Hi Lukas.
>
> you may have noticed my patches for new FALLOC_FL_ZERO_RANGE
> fallocate flag. This changes things around the same area as your
> patches does so we should probably figure out which are going to
> land in first and then rebase the other on top of that.
Yes, I have seen your patches. I think that VFS + XFS patches for
FALLOC_FL_COLLAPSE_RANGE have got a fair amount of review from Dave.
So they are almost ready. I will post collapse range v5 patches after
fixing some minor comments tonight or tommorow.
If there will be no more comments for XFS patch, VFS + XFS patches can
go through to xfs tree by Dave.
I think that Ext4 collapse range could go at a later time.(Need your
reviewed-by tag)
If still there are comments for xfs patch, I will rebase on top of your patch.
>
> One concern I have is that I have not seen any tests provided to
> verify the feature. But I just may have missed it. Do you have any
> xfstests test or at least fsx and fsstress patches to provide
> support for testing FALLOC_FL_COLLAPSE_RANGE ? Patches for
> util_linux might also be handy.
Yes, There are 5 new xfstests cases in this patch-set. these will also
be posted with collapse range patch-set soon.

Thanks :)
>
> Thanks!
> -Lukas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/