| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 2/5] xfs: use per-filesystem I/O completion workqueues |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Wed, 9 Nov 2011 10:11:18 +1100 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20111108085846.291838677@xxxxxxxxxxxxxxxxxxxxxx> |
| References: | <20111108085614.478431403@xxxxxxxxxxxxxxxxxxxxxx> <20111108085846.291838677@xxxxxxxxxxxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Tue, Nov 08, 2011 at 03:56:16AM -0500, Christoph Hellwig wrote: > The new concurrency managed workqueues are cheap enough that we can create > per-filesystem instead of global workqueues. This allows us to remove the > trylock or defer scheme on the ilock, which has the potential of delaying > size updates, and is not helpful once we start to log the inode size. So why does the per-mount workqueues allow removal of the trylock? IOWs, it might be worthwhile pointing to the commit (77d7a0c "xfs: Non-blocking inode locking in IO completion") to indicate that the functionality being removed was introduced to avoid IO completion deadlocks between dependent filesystems (e.g. XFS on loop device on XFS) and that moving to per-mount completion queues removes that dependency.... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [XFS updates] XFS development tree branch, master, updated. xfs_for_3.2-rc1_v1-10720-g1ea6b8f, xfs |
|---|---|
| Next by Date: | [XFS updates] XFS development tree annotated tag, v3.2-rc1, created. v3.2-rc1, xfs |
| Previous by Thread: | [PATCH 2/5] xfs: use per-filesystem I/O completion workqueues, Christoph Hellwig |
| Next by Thread: | Re: [PATCH 2/5] xfs: use per-filesystem I/O completion workqueues, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |