diff options
| author | Christoph Hellwig <hch@infradead.org> | 2007-12-07 14:07:08 +1100 |
|---|---|---|
| committer | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-02-07 18:18:50 +1100 |
| commit | a9759f2de38a3443d5107bddde03b4f3f550060e (patch) | |
| tree | 3a61d9818f5e457bc073958363a3136ac229f532 /fs/xfs/linux-2.6/xfs_buf.h | |
| parent | 40ebd81d1a7635cf92a59c387a599fce4863206b (diff) | |
| download | kernel_samsung_smdk4412-a9759f2de38a3443d5107bddde03b4f3f550060e.zip kernel_samsung_smdk4412-a9759f2de38a3443d5107bddde03b4f3f550060e.tar.gz kernel_samsung_smdk4412-a9759f2de38a3443d5107bddde03b4f3f550060e.tar.bz2 | |
[XFS] kill superflous buffer locking (2nd attempt)
There is no need to lock any page in xfs_buf.c because we operate on our
own address_space and all locking is covered by the buffer semaphore. If
we ever switch back to main blockdeive address_space as suggested e.g. for
fsblock with a similar scheme the locking will have to be totally revised
anyway because the current scheme is neither correct nor coherent with
itself.
SGI-PV: 971186
SGI-Modid: xfs-linux-melb:xfs-kern:30156a
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Diffstat (limited to 'fs/xfs/linux-2.6/xfs_buf.h')
| -rw-r--r-- | fs/xfs/linux-2.6/xfs_buf.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/xfs/linux-2.6/xfs_buf.h b/fs/xfs/linux-2.6/xfs_buf.h index b5908a3..a3d207d 100644 --- a/fs/xfs/linux-2.6/xfs_buf.h +++ b/fs/xfs/linux-2.6/xfs_buf.h @@ -143,7 +143,6 @@ typedef struct xfs_buf { void *b_fspriv2; void *b_fspriv3; unsigned short b_error; /* error code on I/O */ - unsigned short b_locked; /* page array is locked */ unsigned int b_page_count; /* size of page array */ unsigned int b_offset; /* page offset in first page */ struct page **b_pages; /* array of page pointers */ |