> -----Original Message-----
> From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-
> ow...@vger.kernel.org] On Behalf Of Hal Rosenstock
> Sent: Wednesday, January 14, 2015 6:07 AM
> To: Weiny, Ira
> Cc: linux-rdma (linux-rdma@vger.kernel.org); Or Gerlitz
> Subject: [PATCH v3 infiniband-diags] ibdiag_common.c: Add more supported
> device IDs in is_mlnx_ext_port_info_supported
>
> Add support for some ConnectX-IB, ConnectX-4, and Switch-IB in
> is_mlnx_ext_port_info_supported.
>
> This only affects tool(s) which invoke this routine
> (is_mlnx_ext_port_info_supported) which is currently just ibportstate.
>
> is_mlnx_ext_port_info_supported determines whether or not the MLNX vendor
> extended port info SMP is sent which is used to determine FDR10 speed.
>
> Not having the device in the list would only result in FDR10 being
> misinterpreted as QDR.
>
> Signed-off-by: Hal Rosenstock <h...@mellanox.com>
Thanks, applied,
Ira
> ---
> Change since v2:
> Changed ConnectX-3 to ConnectX-IB in patch description
>
> Change since v1:
> Improved patch description
>
> diff --git a/src/ibdiag_common.c b/src/ibdiag_common.c index
> 8c749c7..384d342 100644
> --- a/src/ibdiag_common.c
> +++ b/src/ibdiag_common.c
> @@ -499,9 +499,9 @@ conv_cnt_human_readable(uint64_t val64, float *val,
> int data) int is_mlnx_ext_port_info_supported(uint32_t devid) {
> if (ibd_ibnetdisc_flags & IBND_CONFIG_MLX_EPI) {
> - if (devid == 0xc738)
> + if (devid == 0xc738 || devid == 0xcb84)
> return 1;
> - if (devid >= 0x1003 && devid <= 0x1011)
> + if (devid >= 0x1003 && devid <= 0x1016)
> return 1;
> }
> return 0;
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the
> body
> of a message to majord...@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
