[Cluster-devel] [GFS2 Patch] GFS2: Eliminate gfs2_rg_lops

[Steven Whitehouse]

Hi,

Added to the -nmw tree. Thanks,

Steve.

On Tue, 2013-05-21 at 10:54 -0400, Bob Peterson wrote:
> Hi,
> 
> With recent changes to the transactions, it appears that we
> are no longer using the "log ops" for resource groups. Since the
> log commit code processes the array of log ops, eliminating this
> should be marginally better for performance. Therefore this patch
> eliminates it.
> 
> Regards,
> 
> Bob Peterson
> Red Hat File Systems
> 
> Signed-off-by: Bob Peterson <rpeterso at redhat.com> 
> ---
>  fs/gfs2/lops.c | 5 -----
>  fs/gfs2/lops.h | 1 -
>  2 files changed, 6 deletions(-)
> 
> diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c
> index e069041..0457768 100644
> --- a/fs/gfs2/lops.c
> +++ b/fs/gfs2/lops.c
> @@ -850,10 +850,6 @@ const struct gfs2_log_operations gfs2_revoke_lops = {
>  	.lo_name = "revoke",
>  };
>  
> -const struct gfs2_log_operations gfs2_rg_lops = {
> -	.lo_name = "rg",
> -};
> -
>  const struct gfs2_log_operations gfs2_databuf_lops = {
>  	.lo_before_commit = databuf_lo_before_commit,
>  	.lo_after_commit = databuf_lo_after_commit,
> @@ -865,7 +861,6 @@ const struct gfs2_log_operations gfs2_databuf_lops = {
>  const struct gfs2_log_operations *gfs2_log_ops[] = {
>  	&gfs2_databuf_lops,
>  	&gfs2_buf_lops,
> -	&gfs2_rg_lops,
>  	&gfs2_revoke_lops,
>  	NULL,
>  };
> diff --git a/fs/gfs2/lops.h b/fs/gfs2/lops.h
> index 87e062e..9ca2e64 100644
> --- a/fs/gfs2/lops.h
> +++ b/fs/gfs2/lops.h
> @@ -23,7 +23,6 @@
>  extern const struct gfs2_log_operations gfs2_glock_lops;
>  extern const struct gfs2_log_operations gfs2_buf_lops;
>  extern const struct gfs2_log_operations gfs2_revoke_lops;
> -extern const struct gfs2_log_operations gfs2_rg_lops;
>  extern const struct gfs2_log_operations gfs2_databuf_lops;
>  
>  extern const struct gfs2_log_operations *gfs2_log_ops[];
>