| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/6] fs: add iomap_file_dirty |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Mon, 19 Sep 2016 10:11:44 +1000 |
| Cc: | Christoph Hellwig <hch@xxxxxx>, viro@xxxxxxxxxxxxxxxxxx, darrick.wong@xxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, linux-api@xxxxxxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20160905145751.GC7662@xxxxxxxxxxxxx> |
| References: | <147216784041.525.7722906502172299465.stgit@xxxxxxxxxxxxxxxx> <147216786712.525.5901639662288553230.stgit@xxxxxxxxxxxxxxxx> <20160905145751.GC7662@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Mon, Sep 05, 2016 at 07:57:51AM -0700, Christoph Hellwig wrote: > On Thu, Aug 25, 2016 at 04:31:07PM -0700, Christoph Hellwig wrote: > > Originally-from: Christoph Hellwig <hch@xxxxxx> > > This should be a > > From: Christoph Hellwig <hch@xxxxxx> > > so that git picks up authorship information correctly. > > > XXX: might want a bigger name, and possible a better implementation > > that doesn't require two lookups in the radix tree. > > And these need to be looked into. I can take a stab at it, but I need > to get a few other things off my plate first. Seeing as it works and isn't too ugly to live, I think I'm going to merge this as is. When a better implementation comes along we can replace it with that... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: xfs_file_splice_read: possible circular locking dependency detected, Al Viro |
|---|---|
| Next by Date: | Re: [PATCH] mkfs.xfs: clarify ftype defaults in manpage, Dave Chinner |
| Previous by Thread: | Re: [PATCH 4/6] fs: add iomap_file_dirty, Christoph Hellwig |
| Next by Thread: | Re: [PATCH 6/6] iomap: add a flag to report shared extents, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |