| To: | Ben Myers <bpm@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/4] xfs: fix the logspace waiting algorithm |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 1 Dec 2011 02:21:49 -0500 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| In-reply-to: | <20111130235641.GX29840@xxxxxxx> |
| References: | <20111128081732.350228200@xxxxxxxxxxxxxxxxxxxxxx> <20111128081925.981681380@xxxxxxxxxxxxxxxxxxxxxx> <20111130235641.GX29840@xxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Wed, Nov 30, 2011 at 05:56:41PM -0600, Ben Myers wrote: > Christoph, > > I feel that the ordering of the accesses to l_grant_head and the writeq > list may be important in the lockless path, and notice that they used to > be in opposite order. It could use a comment explaining why (if) it is > ok. Do you mean moving the xlog_space_left after the first list_empty_careful check in xlog_grant_log_space? It doesn't matter given that we re-check the available space after each wakeup. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Next by Date: | Re: [PATCH 5/5] xfs: log file size updates at I/O completion time, Christoph Hellwig |
|---|---|
| Next by Thread: | Re: [PATCH 4/4] xfs: fix the logspace waiting algorithm, Ben Myers |
| Indexes: | [Date] [Thread] [Top] [All Lists] |