On Mon, Apr 06, 2020 at 08:35:01PM +0300, Leon Romanovsky wrote:
> From: Zhu Yanjun <yanjunz@xxxxxxxxxxxx>
>
> The RXE driver doesn't set vendor_id and user space applications
> see zeros. This causes to pyverbs tests to fail with the following
> traceback, because the expectation is to have valid vendor_id.
>
> Traceback (most recent call last):
> File "tests/test_device.py", line 51, in test_query_device
> self.verify_device_attr(attr)
> File "tests/test_device.py", line 77, in verify_device_attr
> assert attr.vendor_id != 0
>
> In order to fix it, we will set vendor_id 0XFFFFFF, according
> to the IBTA v1.4 A3.3.1 VENDOR INFORMATION section.
>
> """
> A vendor that produces a generic controller (i.e., one that supports a
> standard I/O protocol such as SRP), which does not have vendor specific
> device drivers, may use the value of 0xFFFFFF in the VendorID field.
> """
>
> Before:
> "
> hca_id: rxe0
> transport: InfiniBand (0)
> fw_ver: 0.0.0
> node_guid: 5054:00ff:feaa:5363
> sys_image_guid: 5054:00ff:feaa:5363
> vendor_id: 0x0000
> "
>
> After:
> "
> hca_id: rxe0
> transport: InfiniBand (0)
> fw_ver: 0.0.0
> node_guid: 5054:00ff:feaa:5363
> sys_image_guid: 5054:00ff:feaa:5363
> vendor_id: 0xffffff
> "
>
> Fixes: 8700e3e7c485 ("Soft RoCE driver")
> Signed-off-by: Zhu Yanjun <yanjunz@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> ---
> drivers/infiniband/sw/rxe/rxe.c | 1 +
> drivers/infiniband/sw/rxe/rxe_param.h | 3 +++
> 2 files changed, 4 insertions(+)
Applied to for-next, thanks
Jason
