| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 1/5] xfs: take the ilock around xfs_bmapi_read in xfs_zero_remaining_bytes |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 5 Dec 2013 12:40:08 -0800 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20131205203115.GA29897@dastard> |
| References: | <20131205155830.620826868@xxxxxxxxxxxxxxxxxxxxxx> <20131205155951.199565525@xxxxxxxxxxxxxxxxxxxxxx> <20131205203115.GA29897@dastard> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Fri, Dec 06, 2013 at 07:31:15AM +1100, Dave Chinner wrote: > This now holds the ilock over data IO, which is not allowed to be > done as data IO completion can require the ilock to be taken. Yes, > the code specifically avoids all these problems, but the general > rule is that ilock is only held over metadata IO operations, not > data IO. If we need data IO serialisation, then we use the iolock. And we already hold the iolock here. So yeah, we probably should just move it to protect the xfs_bmapi_read call only. |
| Previous by Date: | Re: [PATCH 1/5] xfs: take the ilock around xfs_bmapi_read in xfs_zero_remaining_bytes, Ben Myers |
|---|---|
| Next by Date: | Re: [PATCH 2/5] xfs: use xfs_ilock_map_shared in xfs_qm_dqtobp, Dave Chinner |
| Previous by Thread: | Re: [PATCH 1/5] xfs: take the ilock around xfs_bmapi_read in xfs_zero_remaining_bytes, Ben Myers |
| Next by Thread: | [PATCH] xfs: free the efi AIL entry on log recovery failure, Mark Tinguely |
| Indexes: | [Date] [Thread] [Top] [All Lists] |