Re: [PATCH 04/18] RMDA/core: Introduce ib_scatterlist structure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 07, 2019 at 08:02:30PM +0200, Max Gurtovoy wrote:
> From: Israel Rukshin <israelr@xxxxxxxxxxxx>
> 
> Instead of adding many arguments to RDMA mapping functions with PI,
> create a new data structure 'ib_scatterlist' which contains the needed
> information on a mapped scatterlist. In the future, this will also be
> used by ULPs that define these attributes explicitly.

I still don't like the idea of another magic structure at all.  Even
if we absolutely need something like this for the signature MR case
(which I'm not sure about) we should not require any of that for the
normal case.

We basically add 4 lines of boilerplate code to each ib_map_mr_sg
caller, and probably have worse code generation as well.

Also the name for the structure is extremely misleading - it is
not a scatterlist structure, but really the arguments structure
for ib_map_mr_sg.



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux