[PATCH] bnx2fc: fix incorrect DMA memory mapping in bnx2fc_map_sg()

2014-08-04 Thread Maurizio Lombardi
In the bnx2fc_map_sg() function, the original behaviour is to allocate the DMA memory by directly calling dma_map_sg() instead of using scsi_dma_map(). In contrast, bnx2fc_unmap_sg_list() calls scsi_dma_unmap(). The problem is that bnx2fc_map_sg() passes to the dma_map_sg() function the pointer

[PATCH V3 0/4] Add XEN pvSCSI support

2014-08-04 Thread jgross
This series adds XEN pvSCSI support. With pvSCSI it is possible to use physical SCSI devices from a XEN domain. The support consists of a backend in the privileged Domain-0 doing the real I/O and a frontend in the unprivileged domU passing I/O-requests to the backend. The code is taken (and

[PATCH V3 1/4] Add XEN pvSCSI protocol description

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com Add the definition of pvSCSI protocol used between the pvSCSI frontend in a XEN domU and the pvSCSI backend in a XEN driver domain (usually Dom0). This header was originally provided by Fujitsu for XEN based on Linux 2.6.18. Changes are: - added comment -

[PATCH V3 4/4] add xen pvscsi maintainer

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com Add myself as maintainer for the Xen pvSCSI stuff. Signed-off-by: Juergen Gross jgr...@suse.com --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index c2066f4..d020bfd 100644 --- a/MAINTAINERS +++

[PATCH V3 2/4] Introduce xen-scsifront module

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com Introduces the XEN pvSCSI frontend. With pvSCSI it is possible for a XEN domU to issue SCSI commands to a SCSI LUN assigned to that domU. The SCSI commands are passed to the pvSCSI backend in a driver domain (usually Dom0) which is owner of the physical device.

[PATCH V3 3/4] Introduce XEN scsiback module

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com Introduces the XEN pvSCSI backend. With pvSCSI it is possible for a XEN domU to issue SCSI commands to a SCSI LUN assigned to that domU. The SCSI commands are passed to the pvSCSI backend in a driver domain (usually Dom0) which is owner of the physical device.

[PATCH] scsi: use correct formats in printk()

2014-08-04 Thread Martin Kepplinger
Use %llu for u64 and %u for int. Not the other way round. Signed-off-by: Martin Kepplinger mart...@posteo.de --- applies to -next20140801 drivers/scsi/u14-34f.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/u14-34f.c b/drivers/scsi/u14-34f.c index

[PATCH v2 6/18] arcmsr: precise checking adapter ID

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw This patch rewrites the arcmsr_define_adapter_type function to precisely check Areca adapter's ID. This can prevent an unknown adapter being used as a default adapter type by driver. Signed-off-by: Ching Huang ching2...@areca.com.tw --- Thanks for

[PATCH v1.3 7/18] arcmsr: revise messages_isr_bh_fn to remove duplicate code

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Revise message_isr_bh_fn to remove the duplicate code for each adapter type. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c --- a/drivers/scsi/arcmsr/arcmsr_hba.c

[PATCH v1.3 8/18] arcmsr: remove calling arcmsr_hbb_enable_driver_mode

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Remove calling arcmsr_hbb_enable_driver_mode by in-line code. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c --- a/drivers/scsi/arcmsr/arcmsr_hba.c 2014-08-04

[PATCH v1.3 9/18] arcmsr: modify printing adapter model number and F/W messages

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Adjust printing order of adapter model name and firmware version. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c --- a/drivers/scsi/arcmsr/arcmsr_hba.c 2014-08-04

[PATCH v1.3 10/18] arcmsr: clear outbound doorbell buffer completely

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Clear outbound doorbell buffer completely for adapter type C. This is to prevent getting bad data input from IOP before ioctl command beginning. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c

[PATCH v1.3 11/18] arcmsr: rename function and variable

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Rename some variable and function name for readability and consistency. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr.h b/drivers/scsi/arcmsr/arcmsr.h --- a/drivers/scsi/arcmsr/arcmsr.h 2014-05-06

[PATCH v1.3 12/18] arcmsr: revise allocation of second dma_coherent_handle for type B adapter

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw This modification is for consistency with up-coming adapter type D. They have similar H/W and S/W structure. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr.h b/drivers/scsi/arcmsr/arcmsr.h ---

[PATCH v1.3 13/18] arcmsr: fix ioctl data read/write error for adapter type C

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Rewrite ioctl entry and its relate functions. This patch fix ioctl data read/write error and change data I/O access from byte to Dword. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_attr.c

Re: [Xen-devel] [PATCH V3 1/4] Add XEN pvSCSI protocol description

2014-08-04 Thread Jan Beulich
On 04.08.14 at 10:27, jgr...@suse.com wrote: +/* Requests from the frontend to the backend */ + +/* + * Request a SCSI operation specified via a CDB in vscsiif_request.cmnd. + * The target is specified via channel, id and lun. + */ +#define VSCSIIF_ACT_SCSI_CDB 1 + +/* + *

[PATCH v1.3 14/18] arcmsr: fix sparse waring and error

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Fix sparse utility checking error and warning. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c --- a/drivers/scsi/arcmsr/arcmsr_hba.c 2014-08-04 16:46:30.0

[PATCH v1.3 15/18] arcmsr: modify some character string

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Revise comment and some character strings. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c --- a/drivers/scsi/arcmsr/arcmsr_hba.c 2014-08-04 16:45:46.0 +0800

[PATCH v1.3 17/18] arcmsr: modify calling scsi_scan_host after all initialization done

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw Modify calling scsi_scan_host until all initialization done. And fix error path of free allocated resource. Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c ---

[PATCH v1.3 18/18] arcmsr: replace flush_scheduled_work() by flush_work()

2014-08-04 Thread Ching Huang
From: Ching Huang ching2...@areca.com.tw To prevent flushing entire workqueue, replace flush_scheduled_work() by flush_work(). Signed-off-by: Ching Huang ching2...@areca.com.tw --- diff -uprN a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c ---

Re: [Xen-devel] [PATCH V3 1/4] Add XEN pvSCSI protocol description

2014-08-04 Thread Juergen Gross
On 08/04/2014 12:10 PM, Jan Beulich wrote: On 04.08.14 at 10:27, jgr...@suse.com wrote: +/* Requests from the frontend to the backend */ + +/* + * Request a SCSI operation specified via a CDB in vscsiif_request.cmnd. + * The target is specified via channel, id and lun. + */ +#define

Re: [PATCH] scsi: use correct formats in printk()

2014-08-04 Thread Christoph Hellwig
On Mon, Aug 04, 2014 at 10:48:37AM +0200, Martin Kepplinger wrote: Use %llu for u64 and %u for int. Not the other way round. I've already queued up an equivalent fix in my core-for-3.17 branch, I just need James to pull it into the scsi for-next tree. -- To unsubscribe from this list: send the

Re: [PATCH] Save command pool address of Scsi_Host

2014-08-04 Thread Christoph Hellwig
On Mon, Aug 04, 2014 at 06:22:59AM +0200, Juergen Gross wrote: OTOH it would be possible to just delete .cmd_pool in the template when deleting the pool. I'll send a patch doing this and you can decide whether to take it or to use the other solution. I'm not sure which to prefer: the

Re: [PATCH] Save command pool address of Scsi_Host

2014-08-04 Thread Christoph Hellwig
On Sat, Aug 02, 2014 at 12:24:34AM +0400, James Bottomley wrote: Wait, that's not right at all. There looks to be a thinko in the command pool handling code. We have both a cmd_pool in the host structure and in the host template structure, but there's confusion about which one we're supposed

Re: [PATCH V3] Save command pool address of Scsi_Host

2014-08-04 Thread Christoph Hellwig
On Mon, Aug 04, 2014 at 06:26:09AM +0200, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but scsi_alloc_host_cmd_pool()

Re: [ANNOUNCE] scsi patch queue tree updated

2014-08-04 Thread Christoph Hellwig
On Fri, Aug 01, 2014 at 04:32:01PM -0400, James Bottomley wrote: On Fri, 2014-08-01 at 05:20 -0700, Christoph Hellwig wrote: I've pushed out updates to both the core-for-3.17 and drivers-for-3.17 branches. So I'm afraid we missed the last -next build on these, so they can't go in with the

Re: [PATCH] scsi/arcmsr: Add timeout module parameter

2014-08-04 Thread Christoph Hellwig
To modify the timeout on a queue please use blk_queue_rq_timeout in the slave_configure method instead of poking directly into the block timer, which won't work e.g. for the blk-mq path. But this really needs an explanation on why you'd need a configurable timeout to start with. -- To

Re: [PATCH V3] Save command pool address of Scsi_Host

2014-08-04 Thread Juergen Gross
On 08/04/2014 01:08 PM, Christoph Hellwig wrote: On Mon, Aug 04, 2014 at 06:26:09AM +0200, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will

[PATCH V4] Save command pool address of Scsi_Host

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but scsi_alloc_host_cmd_pool() isn't saving the pool address in the host template. This will result in an

Re: [PATCH V4] Save command pool address of Scsi_Host

2014-08-04 Thread Juergen Gross
On 08/04/2014 01:24 PM, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but scsi_alloc_host_cmd_pool() isn't saving the pool

[PATCH V5] Save command pool address of Scsi_Host

2014-08-04 Thread jgross
From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but scsi_alloc_host_cmd_pool() isn't saving the pool address in the host template. This will result in an

Re: [ANNOUNCE] scsi patch queue tree updated

2014-08-04 Thread Stephen Rothwell
Hi Christoph, On Mon, 4 Aug 2014 04:11:47 -0700 Christoph Hellwig h...@infradead.org wrote: On Fri, Aug 01, 2014 at 04:32:01PM -0400, James Bottomley wrote: On Fri, 2014-08-01 at 05:20 -0700, Christoph Hellwig wrote: I've pushed out updates to both the core-for-3.17 and drivers-for-3.17

Re: [ANNOUNCE] scsi patch queue tree updated

2014-08-04 Thread Christoph Hellwig
On Mon, Aug 04, 2014 at 09:30:59PM +1000, Stephen Rothwell wrote: You realise that neither of these in linux-next as the scsi-core and scsi-drivers trees had for-3.16 branches in the last round. I assume I should just drop these trees completely from linux-next? Well, they get pulled in

Re: [ANNOUNCE] scsi patch queue tree updated

2014-08-04 Thread Stephen Rothwell
Hi Christoph, On Mon, 4 Aug 2014 04:34:04 -0700 Christoph Hellwig h...@infradead.org wrote: On Mon, Aug 04, 2014 at 09:30:59PM +1000, Stephen Rothwell wrote: You realise that neither of these in linux-next as the scsi-core and scsi-drivers trees had for-3.16 branches in the last round. I

Re: [PATCH V5] Save command pool address of Scsi_Host

2014-08-04 Thread Boaz Harrosh
On 08/04/2014 02:30 PM, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but scsi_alloc_host_cmd_pool() isn't saving the pool

Re: [PATCH V5] Save command pool address of Scsi_Host

2014-08-04 Thread Juergen Gross
On 08/04/2014 01:49 PM, Boaz Harrosh wrote: On 08/04/2014 02:30 PM, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com If a scsi host driver specifies .cmd_len in it's scsi_host_template, a driver's private command pool is needed. scsi_find_host_cmd_pool() will locate it, but

Re: [PATCH V3 2/4] Introduce xen-scsifront module

2014-08-04 Thread David Vrabel
On 04/08/14 09:27, jgr...@suse.com wrote: + +static void scsifront_backend_changed(struct xenbus_device *dev, + enum xenbus_state backend_state) +{ + struct vscsifrnt_info *info = dev_get_drvdata(dev-dev); + + DPRINTK(%p %u %u\n, dev, dev-state,

Re: [PATCH V3 3/4] Introduce XEN scsiback module

2014-08-04 Thread David Vrabel
On 04/08/14 09:27, jgr...@suse.com wrote: From: Juergen Gross jgr...@suse.com Introduces the XEN pvSCSI backend. With pvSCSI it is possible for a XEN domU to issue SCSI commands to a SCSI LUN assigned to that domU. The SCSI commands are passed to the pvSCSI backend in a driver domain

Re: [RFC] hpsa: work in progress lockless monster patches

2014-08-04 Thread Tomas Henzl
On 07/31/2014 05:16 PM, scame...@beardog.cce.hp.com wrote: On Thu, Jul 31, 2014 at 03:56:13PM +0200, Tomas Henzl wrote: Hi Steve, Webb, let me start with the part most important for me - from my previous mail And please have a look at [PATCH] hpsa: refine the pci enble/disable handling I

Re: [RFC] hpsa: work in progress lockless monster patches

2014-08-04 Thread Tomas Henzl
On 08/01/2014 06:40 PM, Webb Scales wrote: On 7/31/14 9:56 AM, Tomas Henzl wrote: In cmd_tagged_alloc Thus, there should never be a collision here between two requests if this is true you don't need the refcount and just a simple flag were sufficient for your other needs. (And maybe you get to

Re: [PATCH 1/15] block copy: initial XCOPY offload support

2014-08-04 Thread Pavel Machek
On Tue 2014-07-15 15:34:47, Mikulas Patocka wrote: This is Martin Petersen's xcopy patch (https://git.kernel.org/cgit/linux/kernel/git/mkp/linux.git/commit/?h=xcopyid=0bdeed274e16b3038a851552188512071974eea8) with some bug fixes, ported to the current kernel. This patch makes it possible to

Re: [ANNOUNCE] scsi patch queue tree updated

2014-08-04 Thread James Bottomley
On Mon, 2014-08-04 at 04:11 -0700, Christoph Hellwig wrote: On Fri, Aug 01, 2014 at 04:32:01PM -0400, James Bottomley wrote: On Fri, 2014-08-01 at 05:20 -0700, Christoph Hellwig wrote: I've pushed out updates to both the core-for-3.17 and drivers-for-3.17 branches. So I'm afraid we

Re: [PATCH 2/5] kexec: Export kexec_in_progress

2014-08-04 Thread Brian King
On 07/28/2014 03:28 PM, Brian King wrote: Export kexec_in_progress for use by device drivers and other modules to optimize kexec boot. Signed-off-by: Brian King brk...@linux.vnet.ibm.com --- kernel/kexec.c |2 ++ 1 file changed, 2 insertions(+) diff -puN

Re: [PATCH] Save command pool address of Scsi_Host

2014-08-04 Thread James Bottomley
On Mon, 2014-08-04 at 04:03 -0700, Christoph Hellwig wrote: On Mon, Aug 04, 2014 at 06:22:59AM +0200, Juergen Gross wrote: OTOH it would be possible to just delete .cmd_pool in the template when deleting the pool. I'll send a patch doing this and you can decide whether to take it or to use