[PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Lijun Ou]

In order to keep the size and member order with the
hns_roce_ib_create_qp sturcture of the kernel driver,
it need to update hns_roce_create_qp's structure
members. Otherwise, it will run fail.

Signed-off-by: Lijun Ou <oulijun-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
Signed-off-by: Wei Hu (Xavier) <xavier.huwei-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
---
 providers/hns/hns_roce_u_abi.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/providers/hns/hns_roce_u_abi.h b/providers/hns/hns_roce_u_abi.h
index e78f967..276d29d 100644
--- a/providers/hns/hns_roce_u_abi.h
+++ b/providers/hns/hns_roce_u_abi.h
@@ -61,9 +61,10 @@ struct hns_roce_create_cq_resp {
 struct hns_roce_create_qp {
 	struct ibv_create_qp		ibv_cmd;
 	__u64				buf_addr;
+	__u64				rsv;
 	__u8				log_sq_bb_count;
 	__u8				log_sq_stride;
-	__u8				reserved[5];
+	__u8				reserved[6];
 };
 
 #endif /* _HNS_ROCE_U_ABI_H */
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Jason Gunthorpe]

On Sat, Sep 30, 2017 at 04:51:17PM +0800, Lijun Ou wrote:
> In order to keep the size and member order with the
> hns_roce_ib_create_qp sturcture of the kernel driver,
> it need to update hns_roce_create_qp's structure
> members. Otherwise, it will run fail.

What?

Did you break the uapi? What kernel patch does this match with?

Generally structs in the u_abi.h headers cannot be changed like this.

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Leon Romanovsky]

[-- Attachment #1: Type: text/plain, Size: 864 bytes --]

On Sat, Sep 30, 2017 at 09:45:18AM -0600, Jason Gunthorpe wrote:
> On Sat, Sep 30, 2017 at 04:51:17PM +0800, Lijun Ou wrote:
> > In order to keep the size and member order with the
> > hns_roce_ib_create_qp sturcture of the kernel driver,
> > it need to update hns_roce_create_qp's structure
> > members. Otherwise, it will run fail.
>
> What?
>
> Did you break the uapi? What kernel patch does this match with?
>
> Generally structs in the u_abi.h headers cannot be changed like this.

Jason,

It was my suggestion to change rdma-core, because before they tried to
change kernel.

https://patchwork.kernel.org/patch/9975329/


>
> Jason
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Leon Romanovsky]

[-- Attachment #1: Type: text/plain, Size: 1095 bytes --]

On Sat, Sep 30, 2017 at 07:05:15PM +0300, Leon Romanovsky wrote:
> On Sat, Sep 30, 2017 at 09:45:18AM -0600, Jason Gunthorpe wrote:
> > On Sat, Sep 30, 2017 at 04:51:17PM +0800, Lijun Ou wrote:
> > > In order to keep the size and member order with the
> > > hns_roce_ib_create_qp sturcture of the kernel driver,
> > > it need to update hns_roce_create_qp's structure
> > > members. Otherwise, it will run fail.
> >
> > What?
> >
> > Did you break the uapi? What kernel patch does this match with?
> >
> > Generally structs in the u_abi.h headers cannot be changed like this.
>
> Jason,
>
> It was my suggestion to change rdma-core, because before they tried to
> change kernel.
>
> https://patchwork.kernel.org/patch/9975329/
>

And according to the git log on providers/hns/hns_roce_u_abi.h and
include/uapi/rdma/hns-abi.h, it never worked.

>
> >
> > Jason
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html



[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Jason Gunthorpe]

On Sat, Sep 30, 2017 at 07:05:15PM +0300, Leon Romanovsky wrote:
> On Sat, Sep 30, 2017 at 09:45:18AM -0600, Jason Gunthorpe wrote:
> > On Sat, Sep 30, 2017 at 04:51:17PM +0800, Lijun Ou wrote:
> > > In order to keep the size and member order with the
> > > hns_roce_ib_create_qp sturcture of the kernel driver,
> > > it need to update hns_roce_create_qp's structure
> > > members. Otherwise, it will run fail.
> >
> > What?
> >
> > Did you break the uapi? What kernel patch does this match with?
> >
> > Generally structs in the u_abi.h headers cannot be changed like this.
> 
> Jason,
> 
> It was my suggestion to change rdma-core, because before they tried to
> change kernel.
> 
> https://patchwork.kernel.org/patch/9975329/

Ugh, better commit messages please.

Also, the patch is wrong, it needs to exactly duplicate the current
kernel struct word for word. You can send some thing else once the
kernel side is accepted.

How did we get to a point where hns in rdma-core doesn't work with the
kernel code? Is huawei not testing this stuff?

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Leon Romanovsky]

[-- Attachment #1: Type: text/plain, Size: 253 bytes --]

On Sat, Sep 30, 2017 at 10:29:53AM -0600, Jason Gunthorpe wrote:
> How did we get to a point where hns in rdma-core doesn't work with the
> kernel code? Is huawei not testing this stuff?

You probably know the answer by yourself now.

Thanks

>
> Jason

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH rdma-core] libhns: Update the size and member order of hns_roce_create_qp structure

[Wei Hu (Xavier)]

On 2017/10/1 0:29, Jason Gunthorpe wrote:
> On Sat, Sep 30, 2017 at 07:05:15PM +0300, Leon Romanovsky wrote:
>> On Sat, Sep 30, 2017 at 09:45:18AM -0600, Jason Gunthorpe wrote:
>>> On Sat, Sep 30, 2017 at 04:51:17PM +0800, Lijun Ou wrote:
>>>> In order to keep the size and member order with the
>>>> hns_roce_ib_create_qp sturcture of the kernel driver,
>>>> it need to update hns_roce_create_qp's structure
>>>> members. Otherwise, it will run fail.
>>> What?
>>>
>>> Did you break the uapi? What kernel patch does this match with?
>>>
>>> Generally structs in the u_abi.h headers cannot be changed like this.
>> Jason,
>>
>> It was my suggestion to change rdma-core, because before they tried to
>> change kernel.
>>
>> https://patchwork.kernel.org/patch/9975329/
> Ugh, better commit messages please.
>
> Also, the patch is wrong, it needs to exactly duplicate the current
> kernel struct word for word. You can send some thing else once the
> kernel side is accepted.
>
> How did we get to a point where hns in rdma-core doesn't work with the
> kernel codse? Is huawei not testing this stuff?
Hi, Jason
     In fact testing is mandatory in our development process. sorry for 
this fault.
     We will do our best to avoid the occurrence of this kind of problem.
     And we will send patch v2 at your suggestion. Thanks.

     Regards
Wei Hu
> Jason
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.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 majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html