[Cluster-devel] [GFS2 PATCH] Only wake the reclaim daemon if we need to

Steven Whitehouse <[email protected]>
Newsgroups com.redhat.cluster-devel
Message-ID <1201595758.22038.322.camel@quoit>
Hi,

Now in the -nmw git tree. Thanks,

Steve.

On Mon, 2008-01-28 at 14:54 -0600, Bob Peterson wrote:
> Hi,
> 
> This patch only wakes up the glock reclaim daemon if there is
> actually something to be reclaimed.
> 
> Regards,
> 
> Bob Peterson
> Red Hat GFS
> 
> Signed-off-by: Bob Peterson <[email protected]> 
> --
>  fs/gfs2/glock.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c
> index 80e09c5..894c70e 100644
> --- a/fs/gfs2/glock.c
> +++ b/fs/gfs2/glock.c
> @@ -1586,10 +1586,10 @@ void gfs2_glock_schedule_for_reclaim(struct gfs2_glock *gl)
>  		gfs2_glock_hold(gl);
>  		list_add(&gl->gl_reclaim, &sdp->sd_reclaim_list);
>  		atomic_inc(&sdp->sd_reclaim_count);
> -	}
> -	spin_unlock(&sdp->sd_reclaim_lock);
> -
> -	wake_up(&sdp->sd_reclaim_wq);
> +		spin_unlock(&sdp->sd_reclaim_lock);
> +		wake_up(&sdp->sd_reclaim_wq);
> +	} else
> +		spin_unlock(&sdp->sd_reclaim_lock);
>  }
>  
>  /**
> 
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.