Re: [MAINLINE 0/2] Enable DIM for legacy ULPs and use it in RDS

From: Haakon Bugge
Date: Tue Sep 24 2024 - 11:17:48 EST




> On 24 Sep 2024, at 15:59, Zhu Yanjun <yanjun.zhu@xxxxxxxxx> wrote:
>
> 在 2024/9/24 14:54, Christoph Hellwig 写道:
>> On Tue, Sep 24, 2024 at 09:58:24AM +0800, Zhu Yanjun wrote:
>>> The users that I mentioned is not in the kernel tree.
>> And why do you think that would matter the slightest?
>
> I noticed that the same cq functions are used. And I also made tests with this patch series. Without this patch series, dim mechanism will not be invoked.

Christoph alluded to say: Do not modify the old cq_create_cq() code in order to support DIM, it is better to change the ULP to use ib_alloc_cq(), and get DIM enabled that way.


Thxs, Håkon