| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/3] xfs: consider freeze levels in xfs_fs_writable() |
| From: | Brian Foster <bfoster@xxxxxxxxxx> |
| Date: | Mon, 29 Sep 2014 08:18:21 -0400 |
| Cc: | Dave Chinner <david@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20140926094436.GA10692@xxxxxxxxxxxxx> |
| References: | <1411647632-28240-1-git-send-email-david@xxxxxxxxxxxxx> <1411647632-28240-2-git-send-email-david@xxxxxxxxxxxxx> <20140925161756.GA25798@xxxxxxxxxxxxx> <20140925210330.GG4945@dastard> <20140926094436.GA10692@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.23 (2014-03-12) |
On Fri, Sep 26, 2014 at 02:44:36AM -0700, Christoph Hellwig wrote: > On Fri, Sep 26, 2014 at 07:03:30AM +1000, Dave Chinner wrote: > > I can mention it, but it's so trivial I didn't think it was worth > > it. > > It does change behavior, so it should document why that was changed. > > > The changelog says "Hence allow the caller to pass in the freeze > > level it is allowed to write" which is exactly what this code is > > doing. > > The changelog says the you can pass the argument now. It doesn't > say that it did change a caller to pass a different argument, and more > importanly why. > Note that iirc the patch I posted had a bug so this one is preferred, but feel free to steal the commit log/problem description from the old one: http://oss.sgi.com/archives/xfs/2014-08/msg00073.html Brian > _______________________________________________ > xfs mailing list > xfs@xxxxxxxxxxx > http://oss.sgi.com/mailman/listinfo/xfs |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [RFC v2] Unicode/UTF-8 support for XFS, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 3/3] xfs: consolidate superblock logging functions, Brian Foster |
| Previous by Thread: | Re: [PATCH 1/3] xfs: consider freeze levels in xfs_fs_writable(), Christoph Hellwig |
| Next by Thread: | [PATCH 2/3] xfs: remove bitfield based superblock updates, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |