Re: [PATCH rdma-next v2 1/2] lib/scatterlist: Add support in dynamic allocation of SG table from pages

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

 



On Mon, Sep 21, 2020 at 09:57:25AM +0200, Christoph Hellwig wrote:
> I'm still not really sold on the explosion of specific sgl APIs, so
> I ended up implementing my original suggestion to reuse
> __sg_alloc_table_from_pages and just pass two additional parameters.
> I also ended up moving the memset out of __sg_alloc_table into its
> two callers, and I think the result looks much better, what do you
> think?

I think that the API call is really hard to grasp now with too many
arguments. Fun part will start when someone will decide to use this API
without some (expected for now) parameters.

I'm fond of more explicit interfaces.

Anyway, it is your area, we will retest it and resend.

Thanks



[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