diff options
| author | Yan, Zheng <zyan@redhat.com> | 2016-03-10 11:29:34 +0800 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2016-03-25 18:51:54 +0100 |
| commit | 4531126753aaf936e2674d28245400c6559ef0ee (patch) | |
| tree | 26e5a6a92e794dc7a6617ab7ddb56e2e52f3a7cc | |
| parent | a3d714c33632ef6bfdfaacc74ae6ba297b4c5820 (diff) | |
| download | op-kernel-dev-4531126753aaf936e2674d28245400c6559ef0ee.zip op-kernel-dev-4531126753aaf936e2674d28245400c6559ef0ee.tar.gz | |
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 888674c..fc5cae2 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, static int ceph_releasepage(struct page *page, gfp_t g) { - struct inode *inode = page->mapping ? page->mapping->host : NULL; - dout("%p releasepage %p idx %lu\n", inode, page, page->index); + dout("%p releasepage %p idx %lu\n", page->mapping->host, + page, page->index); WARN_ON(PageDirty(page)); /* Can we release the page from the cache? */ |
