| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/3] xfs: consider freeze levels in xfs_fs_writable() |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Fri, 26 Sep 2014 02:44:36 -0700 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20140925210330.GG4945@dastard> |
| References: | <1411647632-28240-1-git-send-email-david@xxxxxxxxxxxxx> <1411647632-28240-2-git-send-email-david@xxxxxxxxxxxxx> <20140925161756.GA25798@xxxxxxxxxxxxx> <20140925210330.GG4945@dastard> |
| User-agent: | Mutt/1.5.23 (2014-03-12) |
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. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Paramount Group Of Companies Cordially Invites You To Golfforeste Carnival Of Colors, Smriti Ahuja |
|---|---|
| Next by Date: | Re: [PATCH 2/3] xfs: remove bitfield based superblock updates, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH 1/3] xfs: consider freeze levels in xfs_fs_writable(), Dave Chinner |
| Next by Thread: | Re: [PATCH 1/3] xfs: consider freeze levels in xfs_fs_writable(), Brian Foster |
| Indexes: | [Date] [Thread] [Top] [All Lists] |