Re: [PATCH v2 07/21] crypto: shash, caam: Make use of the new sg_map helper function

2017-04-27 Thread Herbert Xu
On Thu, Apr 27, 2017 at 09:45:57AM -0600, Logan Gunthorpe wrote: > > > On 26/04/17 09:56 PM, Herbert Xu wrote: > > On Tue, Apr 25, 2017 at 12:20:54PM -0600, Logan Gunthorpe wrote: > >> Very straightforward conversion to the new function in the caam driver > >> and shash library. > >> > >> Signed-

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Logan Gunthorpe
On 27/04/17 05:20 PM, Jason Gunthorpe wrote: > It seems the most robust: test for iomem, and jump to a slow path > copy, otherwise inline the kmap and memcpy > > Every place doing memcpy from sgl will need that pattern to be > correct. Ok, sounds like a good place to start to me. I'll see what

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Jason Gunthorpe
On Thu, Apr 27, 2017 at 05:03:45PM -0600, Logan Gunthorpe wrote: > > > On 27/04/17 04:11 PM, Jason Gunthorpe wrote: > > On Thu, Apr 27, 2017 at 03:53:37PM -0600, Logan Gunthorpe wrote: > > Well, that is in the current form, with more users it would make sense > > to optimize for the single page c

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Logan Gunthorpe
On 27/04/17 04:11 PM, Jason Gunthorpe wrote: > On Thu, Apr 27, 2017 at 03:53:37PM -0600, Logan Gunthorpe wrote: > Well, that is in the current form, with more users it would make sense > to optimize for the single page case, eg by providing the existing > call, providing a faster single-page-only

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Jason Gunthorpe
On Thu, Apr 27, 2017 at 03:53:37PM -0600, Logan Gunthorpe wrote: > On 27/04/17 02:53 PM, Jason Gunthorpe wrote: > > blkfront is one of the drivers I looked at, and it appears to only be > > memcpying with the bvec_data pointer, so I wonder why it does not use > > sg_copy_X_buffer instead.. > > But

[PATCH v3] lpfc: Fix panic on BFS configuration.

2017-04-27 Thread jsmart2021
From: James Smart To select the appropriate shost template, the driver is issuing a mailbox command to retrieve the wwn. Turns out the sending of the command precedes the reset of the function. On SLI-4 adapters, this is inconsequential as the mailbox command location is specified by dma via the

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Logan Gunthorpe
On 27/04/17 02:53 PM, Jason Gunthorpe wrote: > blkfront is one of the drivers I looked at, and it appears to only be > memcpying with the bvec_data pointer, so I wonder why it does not use > sg_copy_X_buffer instead.. Yes, sort of... But you'd potentially end up calling sg_copy_to_buffer multip

RE: [PATCH 09/12] hpsa: separate monitor events from heartbeat worker

2017-04-27 Thread Don Brace
> -Original Message- > From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi- > ow...@vger.kernel.org] On Behalf Of Martin Wilck > Sent: Tuesday, April 11, 2017 7:19 AM > To: Don Brace ; joseph.szczy...@hpe.com; > Gerry Morong ; John Hall > ; j...@linux.vnet.ibm.com; Kevin Barnett > ; M

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Jason Gunthorpe
On Thu, Apr 27, 2017 at 02:19:24PM -0600, Logan Gunthorpe wrote: > > > On 26/04/17 01:37 AM, Roger Pau Monné wrote: > > On Tue, Apr 25, 2017 at 12:21:02PM -0600, Logan Gunthorpe wrote: > >> Straightforward conversion to the new helper, except due to the lack > >> of error path, we have to use SG_

Re: [PATCH v2 15/21] xen-blkfront: Make use of the new sg_map helper function

2017-04-27 Thread Logan Gunthorpe
On 26/04/17 01:37 AM, Roger Pau Monné wrote: > On Tue, Apr 25, 2017 at 12:21:02PM -0600, Logan Gunthorpe wrote: >> Straightforward conversion to the new helper, except due to the lack >> of error path, we have to use SG_MAP_MUST_NOT_FAIL which may BUG_ON in >> certain cases in the future. >> >> S

Re: [PATCH v2 01/21] scatterlist: Introduce sg_map helper functions

2017-04-27 Thread Logan Gunthorpe
On 26/04/17 01:44 AM, Christoph Hellwig wrote: > I think we'll at least need a draft of those to make sense of these > patches. Otherwise they just look very clumsy. Ok, what follows is a draft patch attempting to show where I'm thinking of going with this. Obviously it will not compile because

Re: [PATCH] lpfc: fix compile error when CONFIG_NVME_TARGET_FC not enabled

2017-04-27 Thread Jens Axboe
On 04/27/2017 01:35 PM, jsmart2...@gmail.com wrote: > From: James Smart > > fix compile error when CONFIG_NVME_TARGET_FC not enabled > > return statement needs to be outside of ifdef > > Signed-off-by: Dick Kennedy > Signed-off-by: James Smart > --- > Patch to be pulled in via block trees >

[PATCH] lpfc: fix compile error when CONFIG_NVME_TARGET_FC not enabled

2017-04-27 Thread jsmart2021
From: James Smart fix compile error when CONFIG_NVME_TARGET_FC not enabled return statement needs to be outside of ifdef Signed-off-by: Dick Kennedy Signed-off-by: James Smart --- Patch to be pulled in via block trees Patch cut against nvme-4.12 tree drivers/scsi/lpfc/lpfc_nvmet.c | 2 +-

Re: [PATCH v2] lpfc: Fix panic on BFS configuration.

2017-04-27 Thread James Smart
On 4/27/2017 9:14 AM, Ewan D. Milne wrote: Actually, it seems like there is a problem with this patch, see below: It did not compile for me. Argh. Yes. Please ignore the patch. I'll repost. -- james

Re: [PATCH v2] lpfc: Fix panic on BFS configuration.

2017-04-27 Thread Martin K. Petersen
James, Please run checkpatch on your submissions. > This issue was introduced by this patch: > http://www.spinics.net/lists/linux-scsi/msg105908.html > which is in the stable pools with commit id: > 96418b5e2c8867da3279d877f5d1ffabfe460c3d This information needs to go in the tag section: Fixes

Re: target: fixup error message in target_tg_pt_gp_tg_pt_gp_id_store()

2017-04-27 Thread Bart Van Assche
On Thu, 2017-04-27 at 16:17 +0200, Hannes Reinecke wrote: > When setting up an ALUA target port group with an invalid ID the > error message > > kstrtoul() returned -22 for tg_pt_gp_id > > is displayed, which is not really helpful. > Convert it to something sane. > > Reviewed-by: Hannes Reinecke

Re: target: fixup error message in target_tg_pt_gp_alua_access_type_store()

2017-04-27 Thread Bart Van Assche
On Thu, 2017-04-27 at 16:17 +0200, Hannes Reinecke wrote: > When setting up a target the error message: > > Unable to do set ##_name ALUA state on non valid tg_pt_gp ID: 0 > > is displayed. > Apparently concatenation doesn't work in a string; one should be using > implicit string concatenation he

Re: [PATCH v2] lpfc: Fix panic on BFS configuration.

2017-04-27 Thread Ewan D. Milne
Actually, it seems like there is a problem with this patch, see below: It did not compile for me. On Thu, 2017-04-27 at 09:18 -0400, Ewan D. Milne wrote: > On Wed, 2017-04-26 at 12:19 -0700, jsmart2...@gmail.com wrote: > > From: James Smart > > > > To select the appropriate shost template, the d

Re: [PATCH] libfc: do not flood console with messages 'libfc: queue full ...'

2017-04-27 Thread Bart Van Assche
On Thu, 2017-04-27 at 16:25 +0200, Hannes Reinecke wrote: > @@ -1896,11 +1899,11 @@ int fc_queuecommand(struct Scsi_Host *shost, struct > scsi_cmnd *sc_cmd) > > if (!fc_fcp_lport_queue_ready(lport)) { > if (lport->qfull) { > - fc_fcp_can_queue_ramp_down(lp

Re: [PATCH v2 01/21] scatterlist: Introduce sg_map helper functions

2017-04-27 Thread Logan Gunthorpe
On 27/04/17 09:27 AM, Jason Gunthorpe wrote: > On Thu, Apr 27, 2017 at 08:53:38AM +0200, Christoph Hellwig wrote: > How about first switching as many call sites as possible to use > sg_copy_X_buffer instead of kmap? Yeah, I could look at doing that first. One problem is we might get more Naks o

Re: [PATCH v2 07/21] crypto: shash, caam: Make use of the new sg_map helper function

2017-04-27 Thread Logan Gunthorpe
On 26/04/17 09:56 PM, Herbert Xu wrote: > On Tue, Apr 25, 2017 at 12:20:54PM -0600, Logan Gunthorpe wrote: >> Very straightforward conversion to the new function in the caam driver >> and shash library. >> >> Signed-off-by: Logan Gunthorpe >> Cc: Herbert Xu >> Cc: "David S. Miller" >> --- >>

Re: [PATCH v2 01/21] scatterlist: Introduce sg_map helper functions

2017-04-27 Thread Logan Gunthorpe
On 27/04/17 12:53 AM, Christoph Hellwig wrote: > I think you'll need to follow the existing kmap semantics and never > fail the iomem version either. Otherwise you'll have a special case > that's almost never used that has a different error path. > > Again, wrong way. Suddenly making things fai

sym53c8xx and Large disk (DCB-16)

2017-04-27 Thread Alexander Voropay
Hi! I have a nasty bug with ‘sym53c8xx_2’ SCSI driver. I’m trying to attach a large (>2Tb) storage to this controller but it does not work. Seems nobody tested such combination before... My SCSI controller is noname OEM for Intel, PCI-X card with SYN53C1010 chip. Storage: Infortrend A24U-G2421-1

Re: [PATCH v2 01/21] scatterlist: Introduce sg_map helper functions

2017-04-27 Thread Jason Gunthorpe
On Thu, Apr 27, 2017 at 08:53:38AM +0200, Christoph Hellwig wrote: > > The main difficulty we > > have now is that neither of those functions are expected to fail and we > > need them to be able to in cases where the page doesn't map to system > > RAM. This patch series is trying to address it for

[PATCH] libfc: do not flood console with messages 'libfc: queue full ...'

2017-04-27 Thread Hannes Reinecke
When the FCoE sending side becomes congested libfc tries to reduce the queue depth on the host; however due to the built-in lag before attempting to ramp down the queue depth _again_ the message log is flooded with messages libfc: queue full, reducing can_queue to 512 With this patch the message

target: fixup error message in target_tg_pt_gp_tg_pt_gp_id_store()

2017-04-27 Thread Hannes Reinecke
When setting up an ALUA target port group with an invalid ID the error message kstrtoul() returned -22 for tg_pt_gp_id is displayed, which is not really helpful. Convert it to something sane. Reviewed-by: Hannes Reinecke --- drivers/target/target_core_configfs.c | 4 ++-- 1 file changed, 2 ins

target: fixup error message in target_tg_pt_gp_alua_access_type_store()

2017-04-27 Thread Hannes Reinecke
When setting up a target the error message: Unable to do set ##_name ALUA state on non valid tg_pt_gp ID: 0 is displayed. Apparently concatenation doesn't work in a string; one should be using implicit string concatenation here. Signed-off-by: Hannes Reinecke --- drivers/target/target_core_con

Re: [PATCH v2] lpfc: Fix panic on BFS configuration.

2017-04-27 Thread Ewan D. Milne
On Wed, 2017-04-26 at 12:19 -0700, jsmart2...@gmail.com wrote: > From: James Smart > > To select the appropriate shost template, the driver is issuing > a mailbox command to retrieve the wwn. Turns out the sending of > the command precedes the reset of the function. On SLI-4 adapters, > this is

RE: [RESEND][PATCH 07/10][SCSI]mpt2sas: Added Reply Descriptor Post Queue (RDPQ) Array support

2017-04-27 Thread Kashyap Desai
> -Original Message- > From: linux-scsi-ow...@vger.kernel.org [mailto:linux-scsi- > ow...@vger.kernel.org] On Behalf Of Martin K. Petersen > Sent: Thursday, April 27, 2017 3:55 AM > To: Sreekanth Reddy > Cc: Martin K. Petersen; j...@kernel.org; linux-scsi@vger.kernel.org; linux- > ker...@vg