Re: [PATCH] IB: make INFINIBAND_ADDR_TRANS configurable

From: Bart Van Assche
Date: Fri Apr 13 2018 - 08:48:10 EST


On Fri, 2018-04-13 at 00:06 -0700, Greg Thelen wrote:
> Allow INFINIBAND without INFINIBAND_ADDR_TRANS.
>
> Signed-off-by: Greg Thelen <gthelen@xxxxxxxxxx>
> Cc: Tarick Bedeir <tarick@xxxxxxxxxx>
> ---
> drivers/infiniband/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/infiniband/Kconfig b/drivers/infiniband/Kconfig
> index ee270e065ba9..f20a3977087c 100644
> --- a/drivers/infiniband/Kconfig
> +++ b/drivers/infiniband/Kconfig
> @@ -61,7 +61,7 @@ config INFINIBAND_ON_DEMAND_PAGING
> pages on demand instead.
>
> config INFINIBAND_ADDR_TRANS
> - bool
> + bool "InfiniBand address translation"
> depends on INFINIBAND
> default y

Hello Greg,

Please change the short description from "InfiniBand address translation" (which
is a misleading description) into "RDMA/CM". Please also add a help text that
explains to people who are not RDMA experts what the RDMA/CM is.

Thanks,

Bart.