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@xxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
