GFS2: Clean up releasepage
authorSteven Whitehouse <swhiteho@redhat.com>
Tue, 26 Nov 2013 13:21:08 +0000 (13:21 +0000)
committerSteven Whitehouse <swhiteho@redhat.com>
Fri, 3 Jan 2014 09:58:41 +0000 (09:58 +0000)
commite4f2920625cc2c562941b280914823a553f6973d
tree154c0fc597abecba1dcd1c72be4c16f428d49832
parent5ea5050cec9c02e86ceb5e707a889003f895a690
GFS2: Clean up releasepage

For historical reasons, we drop and retake the log lock in ->releasepage()
however, since there is no reason why we cannot hold the log lock over
the whole function, this allows some simplification. In particular,
pinning a buffer is only ever done under the log lock, so it is possible
here to remove the test for pinned buffers in the second loop, since it
is impossible for that to happen (it is also tested in the first loop).

As a result, two tests made later in the second loop become constants
and can also be reduced to the only possible branch. So the net result
is to remove various bits of unreachable code and make this more
readable.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
fs/gfs2/aops.c