| To: | xfs@xxxxxxxxxxx |
|---|---|
| Subject: | [PATCH 1/3] xfs: inode allocation should use unmapped buffers. |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Fri, 23 Nov 2012 14:24:23 +1100 |
| In-reply-to: | <1353641065-14983-1-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1353641065-14983-1-git-send-email-david@xxxxxxxxxxxxx> |
From: Dave Chinner <dchinner@xxxxxxxxxx>
Inode buffers do not need to be mapped as inodes are read or written
directly from/to the pages underlying the buffer. This fixes a
regression introduced by commit 611c994 ("xfs: make XBF_MAPPED the
default behaviour").
Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>
---
fs/xfs/xfs_ialloc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_ialloc.c b/fs/xfs/xfs_ialloc.c
index 2d6495e..a815412 100644
--- a/fs/xfs/xfs_ialloc.c
+++ b/fs/xfs/xfs_ialloc.c
@@ -200,7 +200,8 @@ xfs_ialloc_inode_init(
*/
d = XFS_AGB_TO_DADDR(mp, agno, agbno + (j * blks_per_cluster));
fbuf = xfs_trans_get_buf(tp, mp->m_ddev_targp, d,
- mp->m_bsize * blks_per_cluster, 0);
+ mp->m_bsize * blks_per_cluster,
+ XBF_UNMAPPED);
if (!fbuf)
return ENOMEM;
/*
--
1.7.10
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: high load and xfsaild in d, Dave Chinner |
|---|---|
| Next by Date: | [PATCH 2/3] xfs: fix direct IO nested transaction deadlock., Dave Chinner |
| Previous by Thread: | [PATCH 0/3, V2] xfs: regression fixes for the 3.8 cycle, Dave Chinner |
| Next by Thread: | Re: [PATCH 1/3] xfs: inode allocation should use unmapped buffers., Mark Tinguely |
| Indexes: | [Date] [Thread] [Top] [All Lists] |