On Fri, Apr 26, 2019 at 04:49:28PM +0200, Christoph Hellwig wrote: > On Fri, Apr 26, 2019 at 08:53:44AM +0800, Ming Lei wrote: > > Introduces __sg_alloc_table_chained() and __sg_free_table_chained() with > > one extra parameter to specify size of the pre-allocated SGL, then the > > 'first_chunk' SGL can include any number of entries. > > Can you just update the existing sg_alloc_table_chained and > sg_free_table_chained chained instead? We only have a handful users > of them in total, and most of them should eventually use your new > semantics anyway. Yeah, I'd like to do that too. Not do that in V3 just because both nvme and network change are involved. Will do this way if our maintainers don't object. Thanks, Ming