diff mbox

Fix unused variable warnings for pl_index, sg_l

Message ID 1463591298-30102-1-git-send-email-jarod@redhat.com (mailing list archive)
State Superseded
Headers show

Commit Message

Jarod Wilson May 18, 2016, 5:08 p.m. UTC 
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@redhat.com>
---
 src/perftest_resources.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Leon Romanovsky May 26, 2016, 5:08 a.m. UTC | #1 
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@redhat.com>
> ---
>  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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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);