diff options
author | Dan Carpenter <error27@gmail.com> | 2010-05-07 11:05:33 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-07-05 11:15:50 -0700 |
commit | e5c2181d15e358a0927edd545cb3f37b7ff8ca06 (patch) | |
tree | 79e18c4d5c292bf93c31ffdc5dcb4079a53e56bd /fs | |
parent | 0ac5ad9b26e8879728141ad76b56a8b9cc72cd98 (diff) | |
download | lwn-e5c2181d15e358a0927edd545cb3f37b7ff8ca06.tar.gz lwn-e5c2181d15e358a0927edd545cb3f37b7ff8ca06.zip |
exofs: confusion between kmap() and kmap_atomic() api
commit ddf08f4b90a413892bbb9bb2e8a57aed991cd47d upstream.
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.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
-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 4cfab1cc75c0..d91e9d829bc1 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); |