Re: [RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

2014-10-15 Thread Zach Brown
On Tue, Oct 14, 2014 at 10:18:09PM +0200, Fabian Frederick wrote: > > > > On 14 October 2014 at 21:15 Zach Brown wrote: > > > > > > On Tue, Oct 14, 2014 at 07:46:14PM +0200, Fabian Frederick wrote: > > > use function defined in include/linux/highmem.h > > > Note that this reverts 2 last function

Re: [RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

2014-10-14 Thread Fabian Frederick
> On 14 October 2014 at 21:15 Zach Brown wrote: > > > On Tue, Oct 14, 2014 at 07:46:14PM +0200, Fabian Frederick wrote: > > use function defined in include/linux/highmem.h > > Note that this reverts 2 last function call order > > And adds a BUG_ON(PAGE_CACHE_SIZE > PAGE_SIZE).  We can take bets

Re: [RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

2014-10-14 Thread Zach Brown
On Tue, Oct 14, 2014 at 07:46:14PM +0200, Fabian Frederick wrote: > use function defined in include/linux/highmem.h > Note that this reverts 2 last function call order And adds a BUG_ON(PAGE_CACHE_SIZE > PAGE_SIZE). We can take bets on whether that will ever trigger. - z -- To unsubscribe from t

[RFC 1/1 linux-next] btrfs: don't opencode zero_user_segment

2014-10-14 Thread Fabian Frederick
use function defined in include/linux/highmem.h Note that this reverts 2 last function call order Signed-off-by: Fabian Frederick --- fs/btrfs/scrub.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/fs/btrfs/scrub.c b/fs/btrfs/scrub.c index efa0831..fa063cd 100644 ---