| Message ID | 1473438884-674-2-git-send-email-hch@lst.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Sep 09, 2016 at 06:34:35PM +0200, Christoph Hellwig wrote: > Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Ross Zwisler <ross.zwisler@linux.intel.com> Though I guess I've never seen a patch with a completely empty changelog before? :) Maybe it's okay for really easy patches? I just thought I'd get shouted down for doing it. > --- > fs/xfs/xfs_iomap.c | 2 ++ > include/linux/iomap.h | 1 + > 2 files changed, 3 insertions(+) > > diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c > index f96c8ff..5d06a2d 100644 > --- a/fs/xfs/xfs_iomap.c > +++ b/fs/xfs/xfs_iomap.c > @@ -635,6 +635,7 @@ retry: > if (end_fsb != orig_end_fsb) > xfs_inode_set_eofblocks_tag(ip); > > + iomap->flags = IOMAP_F_NEW; > trace_xfs_iomap_alloc(ip, offset, count, 0, &got); > done: > if (isnullstartblock(got.br_startblock)) > @@ -1000,6 +1001,7 @@ xfs_file_iomap_begin( > if (error) > return error; > > + iomap->flags = IOMAP_F_NEW; > trace_xfs_iomap_alloc(ip, offset, length, 0, &imap); > } else { > ASSERT(nimaps); > diff --git a/include/linux/iomap.h b/include/linux/iomap.h > index 3d70ece..14d7067 100644 > --- a/include/linux/iomap.h > +++ b/include/linux/iomap.h > @@ -22,6 +22,7 @@ struct vm_fault; > * Flags for iomap mappings: > */ > #define IOMAP_F_MERGED 0x01 /* contains multiple blocks/extents */ > +#define IOMAP_F_NEW 0x02 /* blocks have been newly allocated */ > > /* > * Magic value for blkno: > -- > 2.1.4 > > _______________________________________________ > Linux-nvdimm mailing list > Linux-nvdimm@lists.01.org > https://lists.01.org/mailman/listinfo/linux-nvdimm -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Sep 13, 2016 at 04:43:57PM -0600, Ross Zwisler wrote: > On Fri, Sep 09, 2016 at 06:34:35PM +0200, Christoph Hellwig wrote: > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > Reviewed-by: Ross Zwisler <ross.zwisler@linux.intel.com> > > Though I guess I've never seen a patch with a completely empty changelog > before? :) Maybe it's okay for really easy patches? I just thought I'd get > shouted down for doing it. I think it's ok and others do it a lot too (e.g. Al Viro). But I've been shouted at for it from others nevertheless.. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/xfs/xfs_iomap.c b/fs/xfs/xfs_iomap.c index f96c8ff..5d06a2d 100644 --- a/fs/xfs/xfs_iomap.c +++ b/fs/xfs/xfs_iomap.c @@ -635,6 +635,7 @@ retry: if (end_fsb != orig_end_fsb) xfs_inode_set_eofblocks_tag(ip); + iomap->flags = IOMAP_F_NEW; trace_xfs_iomap_alloc(ip, offset, count, 0, &got); done: if (isnullstartblock(got.br_startblock)) @@ -1000,6 +1001,7 @@ xfs_file_iomap_begin( if (error) return error; + iomap->flags = IOMAP_F_NEW; trace_xfs_iomap_alloc(ip, offset, length, 0, &imap); } else { ASSERT(nimaps); diff --git a/include/linux/iomap.h b/include/linux/iomap.h index 3d70ece..14d7067 100644 --- a/include/linux/iomap.h +++ b/include/linux/iomap.h @@ -22,6 +22,7 @@ struct vm_fault; * Flags for iomap mappings: */ #define IOMAP_F_MERGED 0x01 /* contains multiple blocks/extents */ +#define IOMAP_F_NEW 0x02 /* blocks have been newly allocated */ /* * Magic value for blkno:
Signed-off-by: Christoph Hellwig <hch@lst.de> --- fs/xfs/xfs_iomap.c | 2 ++ include/linux/iomap.h | 1 + 2 files changed, 3 insertions(+)