diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-05-07 05:05:33 -0400 | 
|---|---|---|
| committer | Boaz Harrosh <bharrosh@panasas.com> | 2010-05-17 06:50:58 -0400 | 
| commit | ddf08f4b90a413892bbb9bb2e8a57aed991cd47d (patch) | |
| tree | aa17308fb0ad06898111a8356ebe702522e5a6d8 | |
| parent | 200b0700420a78c3ade543761f0901985f41f96b (diff) | |
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>
| -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) | |||
| 608 | de->inode_no = cpu_to_le64(parent->i_ino); | 608 | de->inode_no = cpu_to_le64(parent->i_ino); | 
| 609 | memcpy(de->name, PARENT_DIR, sizeof(PARENT_DIR)); | 609 | memcpy(de->name, PARENT_DIR, sizeof(PARENT_DIR)); | 
| 610 | exofs_set_de_type(de, inode); | 610 | exofs_set_de_type(de, inode); | 
| 611 | kunmap_atomic(page, KM_USER0); | 611 | kunmap_atomic(kaddr, KM_USER0); | 
| 612 | err = exofs_commit_chunk(page, 0, chunk_size); | 612 | err = exofs_commit_chunk(page, 0, chunk_size); | 
| 613 | fail: | 613 | fail: | 
| 614 | page_cache_release(page); | 614 | page_cache_release(page); | 
