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

Leon Romanovsky
2016-05-26T05:08:26Z

[-- 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 --]