[GFS2] Move pin/unpin into lops.c, clean up locking
authorSteven Whitehouse <swhiteho@redhat.com>
Mon, 27 Aug 2007 12:54:05 +0000 (13:54 +0100)
committerSteven Whitehouse <swhiteho@redhat.com>
Wed, 10 Oct 2007 07:56:00 +0000 (08:56 +0100)
commit9b9107a5a8b190e6cf09bbdf893869c6a9c482cc
tree77016dd5476f6ba72605ac0cdcbccec3ce794057
parenteaf965270ffff3086ef929e660ace45e862cfd2d
[GFS2] Move pin/unpin into lops.c, clean up locking

gfs2_pin and gfs2_unpin are only used in lops.c, despite being
defined in meta_io.c, so this patch moves them into lops.c and
makes them static. At the same time, its possible to clean up
the locking in the buf and databuf _lo_add() functions so that
we only need to grab the spinlock once. Also we have to move
lock_buffer() around the _lo_add() functions since we can't
do that in gfs2_pin() any more since we hold the spinlock
for the duration of that function.

As a result, the code shrinks by 12 lines and we do far fewer
operations when adding buffers to the log. It also makes the
code somewhat easier to read & understand.

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