Re: [PATCH 16/25] vfs: make remapping to source file eof more explicit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: "Darrick J. Wong" <darrick.wong@xxxxxxxxxx>
Subject: Re: [PATCH 16/25] vfs: make remapping to source file eof more explicit
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Mon, 15 Oct 2018 11:28:58 -0700
Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx>, david@xxxxxxxxxxxxx, sandeen@xxxxxxxxxx, linux-nfs@xxxxxxxxxxxxxxx, linux-cifs@xxxxxxxxxxxxxxx, Amir Goldstein <amir73il@xxxxxxxxx>, linux-unionfs@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-btrfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, ocfs2-devel@xxxxxxxxxxxxxx
In-reply-to: <20181015153219.GG28243@magnolia>
References: <153938912912.8361.13446310416406388958.stgit@magnolia> <153938925737.8361.3995899966552253527.stgit@magnolia> <20181014172433.GG30673@infradead.org> <20181015153219.GG28243@magnolia>
User-agent: Mutt/1.9.2 (2017-12-15)

On Mon, Oct 15, 2018 at 08:32:19AM -0700, Darrick J. Wong wrote:
> > Why not put that in the name to make it more descriptive?
> 
> I'm all ears for better suggestions. :)

I think the best idea is no flag - just carry through the special
zero..

References:
- [PATCH v4 00/25] fs: fixes for serious clone/dedupe problems
  - From: Darrick J. Wong
- [PATCH 16/25] vfs: make remapping to source file eof more explicit
  - From: Darrick J. Wong
- Re: [PATCH 16/25] vfs: make remapping to source file eof more explicit
  - From: Christoph Hellwig
- Re: [PATCH 16/25] vfs: make remapping to source file eof more explicit
  - From: Darrick J. Wong

Prev by Date: Re: [PATCH 10/25] vfs: create generic_remap_file_range_touch to update inode metadata
Next by Date: Re: [PATCH 07/25] vfs: combine the clone and dedupe into a single remap_file_range
Previous by thread: Re: [PATCH 16/25] vfs: make remapping to source file eof more explicit
Next by thread: [PATCH 17/25] vfs: enable remap callers that can handle short operations
Index(es):
- Date
- Thread

[Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]