On Mon, Feb 26, 2018 at 03:02:21PM +0200, Leon Romanovsky wrote:
> From: Yishai Hadas <yishaih@xxxxxxxxxxxx>
>
> This patch explicitly aligns 'mlx4_uverbs_ex_query_device_resp' layout
> to u64 and use the correct response's length when copying the data to
> the user area.
>
> The uAPI should be u64 aligned, currently on 64 bit systems this is done
> implicitly as of the next u64 field in the rss_caps structure.
>
> Legacy user space 32 bit system application in any case is broken with
> 64 bit system kernel before this change.
>
> A matching change will be done in the user area for future use also on
> 32 bit systems.
>
> In addition, we move the structure to the uAPI file where it's expected
> to be from day one.
>
> Fixes: 09d208b258a2 ("IB/mlx4: Add report for RSS capabilities by vendor channel")
> Reviewed-by: Mark Bloch <markb@xxxxxxxxxxxx>
> Signed-off-by: Yishai Hadas <yishaih@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxx>
> drivers/infiniband/hw/mlx4/main.c | 5 +++--
> drivers/infiniband/hw/mlx4/mlx4_ib.h | 14 --------------
> include/uapi/rdma/mlx4-abi.h | 16 ++++++++++++++++
> 3 files changed, 19 insertions(+), 16 deletions(-)
Okay, so I've audited the subsystem for 32/64 problems and there is
only this and the rxe issue I previously sent.
What I'd like to do is solve all the 32/64 problems in one patch with
a really long commit description.
So I plan to take this patch, but remove the alignment fix to the
patch I'm working on above.
OK?
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
