Re: [PATCH rdma-core] mlx5: Return pointer to CQ doorbell

Yishai Hadas
2017-08-23T12:13:38Z

On 8/22/2017 7:30 PM, Jason Gunthorpe wrote:
> On Tue, Aug 22, 2017 at 11:36:42AM +0300, Yishai Hadas wrote:
>> On 8/21/2017 6:22 PM, Jason Gunthorpe wrote:
>>> Is there existing code out there that uses cq_out->uar and works
>>> properly today? Yes or No?
>>
>> No, only this fix enables that to work properly.

> Particularly since a patch to do this with proper compatibility
> exists, and there is no reason at all to take a shortcut.
>

We are not looking for a shortcut but for a solution that will match
majority users if not all.

Changing the symbol version for this API will cause that applications
that will be linked against this version won't be able to run at all on
previous RH packages (e.g RH 7.3/7.4) even if this flow wasn't used at
all as we expect. As pointed, from our knowledge there are no customers
for that specific *rare* flow in mlx5 DV and we prefer to stay with
current bug fix and same API version.