diff options
author | Yan, Zheng <zyan@redhat.com> | 2016-03-09 22:29:34 -0500 |
---|---|---|
committer | Ilya Dryomov <idryomov@gmail.com> | 2016-03-25 13:51:54 -0400 |
commit | 4531126753aaf936e2674d28245400c6559ef0ee (patch) | |
tree | 26e5a6a92e794dc7a6617ab7ddb56e2e52f3a7cc | |
parent | a3d714c33632ef6bfdfaacc74ae6ba297b4c5820 (diff) |
ceph: remove unnecessary NULL check
If page->mapping is NULL, releasepage() callback does not get called.
Remove the unnecessary NULL check to make static code analysis tool
happy
Signed-off-by: Yan, Zheng <zyan@redhat.com>
-rw-r--r-- | fs/ceph/addr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c index 888674c311c5..fc5cae2a0db2 100644 --- a/fs/ceph/addr.c +++ b/fs/ceph/addr.c | |||
@@ -175,8 +175,8 @@ static void ceph_invalidatepage(struct page *page, unsigned int offset, | |||
175 | 175 | ||
176 | static int ceph_releasepage(struct page *page, gfp_t g) | 176 | static int ceph_releasepage(struct page *page, gfp_t g) |
177 | { | 177 | { |
178 | struct inode *inode = page->mapping ? page->mapping->host : NULL; | 178 | dout("%p releasepage %p idx %lu\n", page->mapping->host, |
179 | dout("%p releasepage %p idx %lu\n", inode, page, page->index); | 179 | page, page->index); |
180 | WARN_ON(PageDirty(page)); | 180 | WARN_ON(PageDirty(page)); |
181 | 181 | ||
182 | /* Can we release the page from the cache? */ | 182 | /* Can we release the page from the cache? */ |