diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-05-07 11:05:33 +0200 |
|---|---|---|
| committer | Boaz Harrosh <bharrosh@panasas.com> | 2010-05-17 13:50:58 +0300 |
| commit | ddf08f4b90a413892bbb9bb2e8a57aed991cd47d (patch) | |
| tree | aa17308fb0ad06898111a8356ebe702522e5a6d8 /fs/exofs | |
| parent | 200b0700420a78c3ade543761f0901985f41f96b (diff) | |
| download | op-kernel-dev-ddf08f4b90a413892bbb9bb2e8a57aed991cd47d.zip op-kernel-dev-ddf08f4b90a413892bbb9bb2e8a57aed991cd47d.tar.gz | |
exofs: confusion between kmap() and kmap_atomic() api
For kmap_atomic() we call kunmap_atomic() on the returned pointer.
That's different from kmap() and kunmap() and so it's easy to get them
backwards.
Cc: Stable <stable@kernel.org>
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Diffstat (limited to 'fs/exofs')
| -rw-r--r-- | fs/exofs/dir.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/exofs/dir.c b/fs/exofs/dir.c index 4cfab1c..d91e9d8 100644 --- a/fs/exofs/dir.c +++ b/fs/exofs/dir.c @@ -608,7 +608,7 @@ int exofs_make_empty(struct inode *inode, struct inode *parent) de->inode_no = cpu_to_le64(parent->i_ino); memcpy(de->name, PARENT_DIR, sizeof(PARENT_DIR)); exofs_set_de_type(de, inode); - kunmap_atomic(page, KM_USER0); + kunmap_atomic(kaddr, KM_USER0); err = exofs_commit_chunk(page, 0, chunk_size); fail: page_cache_release(page); |
