Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fa1bbdea authored by Bob Peterson's avatar Bob Peterson Committed by Steven Whitehouse
Browse files

GFS2: Optimize glock multiple-dequeue code 



This is a small patch that optimizes multiple glock dequeue
operations.  It changes the unlock order to be more efficient
and makes it easier for lock debugging tools to unravel.  It
also eliminates the need for the temp variable x, although
that would likely be optimized out.

Signed-off-by: default avatarBob Peterson <rpeterso@redhat.com>
Signed-off-by: default avatarSteven Whitehouse <swhiteho@redhat.com>
parent 0a33443b

fs/gfs2/glock.c

+4 −8
Original line number Diff line number Diff line
@@ -1248,10 +1248,8 @@ int gfs2_glock_nq_m(unsigned int num_gh, struct gfs2_holder *ghs) 


void gfs2_glock_dq_m(unsigned int num_gh, struct gfs2_holder *ghs)

{

	unsigned int x;



	for (x = 0; x < num_gh; x++)

		gfs2_glock_dq(&ghs[x]);

	while (num_gh--)

		gfs2_glock_dq(&ghs[num_gh]);

}



/**
@@ -1263,10 +1261,8 @@ void gfs2_glock_dq_m(unsigned int num_gh, struct gfs2_holder *ghs) 


void gfs2_glock_dq_uninit_m(unsigned int num_gh, struct gfs2_holder *ghs)

{

	unsigned int x;



	for (x = 0; x < num_gh; x++)

		gfs2_glock_dq_uninit(&ghs[x]);

	while (num_gh--)

		gfs2_glock_dq_uninit(&ghs[num_gh]);

}



void gfs2_glock_cb(struct gfs2_glock *gl, unsigned int state)