| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH for-2.6.33] xfs: flush all log buffers in xlog_dealloc_log |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 4 Feb 2010 11:10:50 -0500 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| In-reply-to: | <20100204081422.GF5332@xxxxxxxxxxxxxxxx> |
| References: | <20100201220813.GA3519@xxxxxxxxxxxxx> <20100203105545.GA1047@xxxxxxxxxxxxx> <20100204081422.GF5332@xxxxxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.19 (2009-01-05) |
On Thu, Feb 04, 2010 at 07:14:22PM +1100, Dave Chinner wrote: > On Wed, Feb 03, 2010 at 05:55:45AM -0500, Christoph Hellwig wrote: > > Actually I'll take this one back - the log buffers aren't delwri > > so xfs_flush_buftarg probably only helped by timing or the > > xfs_buf_runall_queues on xfslogd_workqueue. Need to think about > > this a bit more. > > Can you point me at the bug report? IIRC I've seen this in the > past where we freed the log before we've done all the correct > shutdown processing and they got fixed by correcting the > order of shutdown to ensure the log is idle before freeing > it... Yes, that I was going to look into next. The recent report is http://bugzilla.kernel.org/show_bug.cgi?id=15150 and there's also a very similar one from Ed Cashin <ecashin@xxxxxxxxxx> in mail archives from November. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 08/10] xfs: move the inode locking outside xfs_fsync(), Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH, RFC] more reserved blocks fixups, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH for-2.6.33] xfs: flush all log buffers in xlog_dealloc_log, Dave Chinner |
| Next by Thread: | Re: [PATCH for-2.6.33] xfs: flush all log buffers in xlog_dealloc_log, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |