Re: [PATCH rdma-next] RDMA/rdmavt: Decouple QP and SGE lists allocations

From: Dennis Dalessandro
Date: Wed May 19 2021 - 15:49:41 EST


On 5/19/21 2:29 PM, Jason Gunthorpe wrote:
On Wed, May 19, 2021 at 07:56:32AM -0400, Dennis Dalessandro wrote:

Perhaps the code can be enhanced to move more stuff into the driver's own
structs as Jason points out, but that should happen first. For now I still
don't understand why the core can't optionally make the allocation per node.

Because I think it is wrong in the general case to assign all
allocations to a single node?

If by general case you mean for all drivers, sure, totally agree. We aren't talking about all drivers though, just the particular case of rdmavt.

-Denny