Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1358325

Re: [PATCH RFC 0/2] mempool based chained scatterlist alloc/free api api

Show all headers | View raw

On Tue, 2016-03-15 at 15:39 -0700, Ming Lin wrote:
> From: Ming Lin <ming.l@ssi.samsung.com>
> 
> Hi list,
> 
> This moves the mempool based chained scatterlist alloc/free code from
> scsi_lib.c to lib/scatterlist.c.
> 
> So other drivers(for example, the under development NVMe over fabric 
> drivers) can also use it.
> 
> Ming Lin (2):
>   scatterlist: add mempool based chained SG alloc/free api
>   scsi: use the new chained SG api
> 
>  drivers/scsi/scsi_lib.c     | 129 ++----------------------------------
>  include/linux/scatterlist.h |  12 ++++
>  lib/scatterlist.c           | 156 ++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 175 insertions(+), 122 deletions(-)

I'd really rather this were a single patch so git can tell us the code
motion.  If you add in one patch and remove in another the code motion
trackers don't see it.

Secondly, you said "This copied code from scsi_lib.c to scatterlist.c
and modified it a bit" could you move in one patch and modify in
another, so we can see exactly what you're changing.

Thirdly, are you sure the pool structure for NVMe should be the same as
for SCSI?  We don't do buddy pools for 1,2 or 4 entry transactions in
SCSI just basically because of heuristics, but the packetised io
characteristics of NVMe make single entry lists more likely for it,
don't they?

James

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH RFC 0/2] mempool based chained scatterlist alloc/free api api Ming Lin <mlin@kernel.org> - 2016-03-15 23:50 +0100
  [PATCH RFC 1/2] scatterlist: add mempool based chained SG alloc/free  api Ming Lin <mlin@kernel.org> - 2016-03-15 23:50 +0100
    Re: [PATCH RFC 1/2] scatterlist: add mempool based chained SG  alloc/free api Christoph Hellwig <hch@lst.de> - 2016-03-16 09:30 +0100
  [PATCH RFC 2/2] scsi: use the new chained SG api Ming Lin <mlin@kernel.org> - 2016-03-15 23:50 +0100
  Re: [PATCH RFC 0/2] mempool based chained scatterlist alloc/free  api api James Bottomley <James.Bottomley@HansenPartnership.com> - 2016-03-16 00:20 +0100
    Re: [PATCH RFC 0/2] mempool based chained scatterlist alloc/free  api api Ming Lin <mlin@kernel.org> - 2016-03-16 06:20 +0100
    Re: [PATCH RFC 0/2] mempool based chained scatterlist alloc/free  api api Christoph Hellwig <hch@lst.de> - 2016-03-16 09:30 +0100

csiph-web