On Wed, Feb 28, 2018 at 11:30:22AM -0700, Jason Gunthorpe wrote:
> Have check-build run them through the c++ compiler to make sure they
> have no compilation issues, and very roughly check that headers have
> the required extern "C".
>
> Fix all public headers missing the extern "C"
>
> This fixes compilation of user applications in C++ mode that use these
> headers.
>
> Fixes: 5160e25267ee ("mlx4: Add mlx4 direct verbs")
> Fixes: a2ddaca1dd75 ("mlx5: Add mlx5 direct verbs")
> Cc: stable@xxxxxxxxxxxxxx
> Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx>
> ---
> buildlib/check-build | 28 ++++++++++++++++++++++++++--
> ibacm/include/infiniband/acm.h | 8 ++++++++
> ibacm/include/infiniband/acm_prov.h | 8 ++++++++
> providers/mlx4/mlx4dv.h | 9 +++++++++
> providers/mlx5/mlx5dv.h | 8 ++++++++
> 5 files changed, 59 insertions(+), 2 deletions(-)
>
> Nicholas, this should got to all the stables, but you can just drop the
> change to check-build when doing it.
>
Thanks, applied
https://github.com/linux-rdma/rdma-core/pull/299
Attachment:
signature.asc
Description: PGP signature
