diff options
author | Kautuk Consul <consul.kautuk@gmail.com> | 2012-02-20 03:46:12 -0500 |
---|---|---|
committer | Boaz Harrosh <bharrosh@panasas.com> | 2012-08-02 07:52:12 -0400 |
commit | 0e8d96dd2c99405c707f540b5922ec869b848979 (patch) | |
tree | 4279e079a86060037f6f99c4a0fc1e20825269f9 /fs/exofs | |
parent | c999ff68029ebd0f56ccae75444f640f6d5a27d2 (diff) |
exofs: readpage_strip: Add a BUG_ON to check for PageLocked(page)
readpage_strip can be called from several code paths all of which
require that the page be locked before any operations are carried
out.
Since we export the exofs_readpage callback to the VFS, add a
BUG_ON to check for PageLocked(page) to make sure that this
understanding is never compromised.
Signed-off-by: Kautuk Consul <consul.kautuk@gmail.com>
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Diffstat (limited to 'fs/exofs')
-rw-r--r-- | fs/exofs/inode.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/exofs/inode.c b/fs/exofs/inode.c index 5badb0c039de..9a5ed30b8a35 100644 --- a/fs/exofs/inode.c +++ b/fs/exofs/inode.c | |||
@@ -389,6 +389,8 @@ static int readpage_strip(void *data, struct page *page) | |||
389 | size_t len; | 389 | size_t len; |
390 | int ret; | 390 | int ret; |
391 | 391 | ||
392 | BUG_ON(!PageLocked(page)); | ||
393 | |||
392 | /* FIXME: Just for debugging, will be removed */ | 394 | /* FIXME: Just for debugging, will be removed */ |
393 | if (PageUptodate(page)) | 395 | if (PageUptodate(page)) |
394 | EXOFS_ERR("PageUptodate(0x%lx, 0x%lx)\n", pcol->inode->i_ino, | 396 | EXOFS_ERR("PageUptodate(0x%lx, 0x%lx)\n", pcol->inode->i_ino, |