[PATCH] Fix unused variable warnings for pl_index, sg_l

[PATCH] Fix unused variable warnings for pl_index, sg_l

[Jarod Wilson]

These variables will be unused if certain #define's aren't set, so wrap
them with the same #ifdef's as the places they're used.

Signed-off-by: Jarod Wilson <jarod-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
 src/perftest_resources.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/perftest_resources.c b/src/perftest_resources.c
index 71a1541..4071989 100755
--- a/src/perftest_resources.c
+++ b/src/perftest_resources.c
@@ -2778,8 +2778,12 @@ int run_iter_bw(struct pingpong_context *ctx,struct perftest_parameters *user_pa
 	int 			cpu_mhz = 0;
 	int 			return_value = 0;
 	int			wc_id;
+	#ifdef HAVE_VERBS_EXP
+	#ifdef HAVE_ACCL_VERBS
 	int pl_index;
 	struct ibv_sge		*sg_l;
+	#endif
+	#endif
 
 	ALLOCATE(wc ,struct ibv_wc ,CTX_POLL_BATCH);
 
-- 
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Fix unused variable warnings for pl_index, sg_l

[Leon Romanovsky]

[-- Attachment #1: Type: text/plain, Size: 1265 bytes --]

On Wed, May 18, 2016 at 01:08:18PM -0400, Jarod Wilson wrote:
> These variables will be unused if certain #define's aren't set, so wrap
> them with the same #ifdef's as the places they're used.
> 
> Signed-off-by: Jarod Wilson <jarod-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> ---
>  src/perftest_resources.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/perftest_resources.c b/src/perftest_resources.c
> index 71a1541..4071989 100755
> --- a/src/perftest_resources.c
> +++ b/src/perftest_resources.c
> @@ -2778,8 +2778,12 @@ int run_iter_bw(struct pingpong_context *ctx,struct perftest_parameters *user_pa
>  	int 			cpu_mhz = 0;
>  	int 			return_value = 0;
>  	int			wc_id;
> +	#ifdef HAVE_VERBS_EXP
> +	#ifdef HAVE_ACCL_VERBS

Maybe "#if defined(HAVE_VERBS_EXP) && defined(HAVE_ACCL_VERBS)" will be
better?

Anyway, it is the same.

>  	int pl_index;
>  	struct ibv_sge		*sg_l;
> +	#endif
> +	#endif
>  
>  	ALLOCATE(wc ,struct ibv_wc ,CTX_POLL_BATCH);
>  
> -- 
> 1.8.3.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]