| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/7] XFS: Never call mark_inode_dirty_sync() directly |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Thu, 14 Aug 2008 15:47:02 -0400 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1218698083-11226-5-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1218698083-11226-1-git-send-email-david@xxxxxxxxxxxxx> <1218698083-11226-5-git-send-email-david@xxxxxxxxxxxxx> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.18 (2008-05-17) |
On Thu, Aug 14, 2008 at 05:14:40PM +1000, Dave Chinner wrote: > Once the Linux inode and the XFS inode are combined, we cannot rely > on just check if the linux inode exists as a method of determining > if it is valid or not. Hence we should always call > xfs_mark_inode_dirty_sync() instead as it does the correct checks to > determine if the liinux inode is in a valid state or not. Makes sense, and another candidate to put at the beginning of the series or rather just put in now. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 0/7] RFC: combine linux and XFS inodes, Christoph Hellwig |
|---|---|
| Next by Date: | Re: [PATCH 6/7] XFS: Combine the XFS and Linux inodes., Christoph Hellwig |
| Previous by Thread: | [PATCH 4/7] XFS: Never call mark_inode_dirty_sync() directly, Dave Chinner |
| Next by Thread: | Re: [PATCH 4/7] XFS: Never call mark_inode_dirty_sync() directly, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |