Re: [PATCH v2 0/5] mempool based chained scatterlist alloc/free api
On Sun, 2016-04-03 at 23:07 -0700, Ming Lin wrote: > On Mon, Mar 28, 2016 at 7:48 AM, Ming Linwrote: > > On Thu, Mar 24, 2016 at 8:46 AM, James Bottomley > > wrote: > > > On Thu, 2016-03-24 at 08:09 -0700, Ming Lin wrote: > > > > On Wed, Mar 23, 2016 at 12:40 AM, Christoph Hellwig > > > > > > > > wrote: > > > > > On Tue, Mar 22, 2016 at 03:03:11PM -0700, Ming Lin wrote: > > > > > > From: Ming Lin > > > > > > > > > > > > The fist 4 patches make the SG related > > > > > > definitions/structs/functions > > > > > > in SCSI code generic and the last patch move it to > > > > > > lib/sg_pool.c. > > > > > > > > > > > > I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 > > > > > > places. > > > > > > > > > > I don't ѕee the point, but I'm not going to block the series > > > > > over > > > > > it either. > > > > > > > > > > The new series looks really nice to me! > > > > > > > > > > Reviewed-by: Christoph Hellwig > > > > > > > > Hi James, > > > > > > > > This series touches several sub-systems. > > > > What's the best way to merge it? > > > > > > It has a minor intrusion into > > > > > > drivers/ata/pata_icside.c | 2 +- > > > drivers/infiniband/ulp/srp/ib_srp.c | 4 +- > > > drivers/usb/storage/scsiglue.c | 2 +- > > > > > > Apart from that, it's all SCSI, so the SCSI tree would seem to be > > > the > > > best one. > > > > Are you OK to merge it? > > Hi James, > > Ping ... > > Are you OK to apply this series? > Or any changes needed? It's got the needed reviews, but we need acks from tejun (ATA) and Sagi (IB) and a repost fixing the test robot objection to patch 5. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/5] mempool based chained scatterlist alloc/free api
On Thu, Mar 24, 2016 at 8:46 AM, James Bottomleywrote: > On Thu, 2016-03-24 at 08:09 -0700, Ming Lin wrote: >> On Wed, Mar 23, 2016 at 12:40 AM, Christoph Hellwig >> wrote: >> > On Tue, Mar 22, 2016 at 03:03:11PM -0700, Ming Lin wrote: >> > > From: Ming Lin >> > > >> > > The fist 4 patches make the SG related >> > > definitions/structs/functions >> > > in SCSI code generic and the last patch move it to lib/sg_pool.c. >> > > >> > > I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 >> > > places. >> > >> > I don't ѕee the point, but I'm not going to block the series over >> > it either. >> > >> > The new series looks really nice to me! >> > >> > Reviewed-by: Christoph Hellwig >> >> Hi James, >> >> This series touches several sub-systems. >> What's the best way to merge it? > > It has a minor intrusion into > > drivers/ata/pata_icside.c | 2 +- > drivers/infiniband/ulp/srp/ib_srp.c | 4 +- > drivers/usb/storage/scsiglue.c | 2 +- > > Apart from that, it's all SCSI, so the SCSI tree would seem to be the > best one. Are you OK to merge it? Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/5] mempool based chained scatterlist alloc/free api
On Thu, 2016-03-24 at 08:09 -0700, Ming Lin wrote: > On Wed, Mar 23, 2016 at 12:40 AM, Christoph Hellwig> wrote: > > On Tue, Mar 22, 2016 at 03:03:11PM -0700, Ming Lin wrote: > > > From: Ming Lin > > > > > > The fist 4 patches make the SG related > > > definitions/structs/functions > > > in SCSI code generic and the last patch move it to lib/sg_pool.c. > > > > > > I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 > > > places. > > > > I don't ѕee the point, but I'm not going to block the series over > > it either. > > > > The new series looks really nice to me! > > > > Reviewed-by: Christoph Hellwig > > Hi James, > > This series touches several sub-systems. > What's the best way to merge it? It has a minor intrusion into drivers/ata/pata_icside.c | 2 +- drivers/infiniband/ulp/srp/ib_srp.c | 4 +- drivers/usb/storage/scsiglue.c | 2 +- Apart from that, it's all SCSI, so the SCSI tree would seem to be the best one. James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/5] mempool based chained scatterlist alloc/free api
On Wed, Mar 23, 2016 at 12:40 AM, Christoph Hellwigwrote: > On Tue, Mar 22, 2016 at 03:03:11PM -0700, Ming Lin wrote: >> From: Ming Lin >> >> The fist 4 patches make the SG related definitions/structs/functions >> in SCSI code generic and the last patch move it to lib/sg_pool.c. >> >> I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 places. > > I don't ѕee the point, but I'm not going to block the series over > it either. > > The new series looks really nice to me! > > Reviewed-by: Christoph Hellwig Hi James, This series touches several sub-systems. What's the best way to merge it? Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 0/5] mempool based chained scatterlist alloc/free api
On Tue, Mar 22, 2016 at 03:03:11PM -0700, Ming Lin wrote: > From: Ming Lin> > The fist 4 patches make the SG related definitions/structs/functions > in SCSI code generic and the last patch move it to lib/sg_pool.c. > > I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 places. I don't ѕee the point, but I'm not going to block the series over it either. The new series looks really nice to me! Reviewed-by: Christoph Hellwig -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH v2 0/5] mempool based chained scatterlist alloc/free api
From: Ming LinThe fist 4 patches make the SG related definitions/structs/functions in SCSI code generic and the last patch move it to lib/sg_pool.c. I still keep the macro "SG_MEMPOOL_NR" since it's used in 3 places. v2: - do modification in scsi code first then move to lib/sg_pool.c - address Christoph's comments Ming Lin (5): scsi: replace "scsi_data_buffer" with "sg_table" in SG functions scsi: replace "mq" with "first_chunk" in SG functions scsi: rename SG related struct and functions scsi: rename SCSI_MAX_{SG, SG_CHAIN}_SEGMENTS lib: scatterlist: move SG pool code from SCSI driver to lib/sg_pool.c drivers/ata/pata_icside.c | 2 +- drivers/infiniband/ulp/srp/ib_srp.c | 4 +- drivers/scsi/Kconfig| 1 + drivers/scsi/arm/cumana_2.c | 2 +- drivers/scsi/arm/eesox.c| 2 +- drivers/scsi/arm/powertec.c | 2 +- drivers/scsi/esas2r/esas2r_main.c | 4 +- drivers/scsi/hisi_sas/hisi_sas.h| 2 +- drivers/scsi/mpt3sas/mpt3sas_base.c | 4 +- drivers/scsi/mpt3sas/mpt3sas_base.h | 2 +- drivers/scsi/scsi_debug.c | 2 +- drivers/scsi/scsi_lib.c | 172 +--- drivers/usb/storage/scsiglue.c | 2 +- include/linux/scatterlist.h | 25 ++ include/scsi/scsi.h | 19 include/scsi/scsi_host.h| 2 +- lib/Kconfig | 7 ++ lib/Makefile| 1 + lib/sg_pool.c | 172 19 files changed, 241 insertions(+), 186 deletions(-) create mode 100644 lib/sg_pool.c -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html