Re: [PATCH] lpfc: Fix possible NULL pointer dereference

2016-07-29 Thread Johannes Thumshirn
On Thu, Jul 28, 2016 at 09:21:47PM -0400, Martin K . Petersen wrote: > > "James" == James Smart writes: > > James> This patch is good. > > Johannes: You were going to tweak a few things and resubmit. Please do. Oh yes, sorry totally forgot about this one. My bad. -- Johannes Thumshirn

Re: [PATCH 1/3] nvme-fabrics: rework nvmf_get_address() for variable options

2016-07-29 Thread Johannes Thumshirn
On Thu, Jul 28, 2016 at 05:00:25PM -0700, James Smart wrote: > > Revise nvmf_get_address() string to account for not all options being > present. > > Signed-off-by: James Smart Acked-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse

Re: [PATCH 2/3] nvme-fabrics: revise host transport option descriptions

2016-07-29 Thread Johannes Thumshirn
On Thu, Jul 28, 2016 at 05:00:26PM -0700, James Smart wrote: > > Revise some of the comments so not so ethernet-network centric > > Signed-off-by: James Smart Acked-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de

Re: [PATCH 3/3] nvme-fabrics: Add host_traddr options field to host infrastructure

2016-07-29 Thread Johannes Thumshirn
On Thu, Jul 28, 2016 at 05:00:27PM -0700, James Smart wrote: > > Add the host_traddr field to allow specification of the host-port > connection info for the transport. Will be used by FC transport. > > Signed-off-by: James Smart Acked-by: Johannes Thumshirn -- Johannes Thumshirn

[PATCH] ipr: Add asynchronous error notification

2016-07-29 Thread Heitor Ricardo Alves de Siqueira
This patch implements functions for pushing HCAM (host controlled asynchronous messages) error buffers to userspace through sysfs attributes. Reads to the "async_err_log" attribute will result in a single HCAM buffer being copied to userspace; one can process the next HCAM buffer by writing any str

[PATCH v2] lpfc: Fix possible NULL pointer dereference

2016-07-29 Thread Johannes Thumshirn
Check for the existence of piocb->vport before accessing it. Signed-off-by: Johannes Thumshirn --- Changes to v1: * Fix spelling of piocb (Tyrel) * re-introduce the BU() assertion if piocb->vport is NULL (Tyrel) drivers/scsi/lpfc/lpfc_sli.c | 15 ++- 1 file changed, 6 insertions(+)

[PATCH -next] fcoe: add missing destroy_workqueue() on error in fcoe_init()

2016-07-29 Thread Wei Yongjun
Add the missing destroy_workqueue() before return from fcoe_init() in the fcoe transport register failed error handling case. Signed-off-by: Wei Yongjun --- drivers/scsi/fcoe/fcoe.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe

[PATCH -next] ipr: Fix error return code in ipr_probe_ioa()

2016-07-29 Thread Wei Yongjun
Fix to return error code -ENOMEM from the workqueue alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/scsi/ipr.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index 95e4834..bf85974 100

[PATCH v6 2/2] Add ioctl to issue ZBC/ZAC commands via block layer

2016-07-29 Thread Shaun Tancheff
Add support for ZBC ioctl's BLKREPORT- Issue Report Zones to device. BLKOPENZONE - Issue Zone Action: Open Zone command. BLKCLOSEZONE - Issue Zone Action: Close Zone command. BLKRESETZONE - Issue Zone Action: Reset Zone command. Signed-off-by: Shaun Tancheff --- v6: - Added

[PATCH v6 1/2] Add bio/request flags to issue ZBC/ZAC commands

2016-07-29 Thread Shaun Tancheff
Add op flags to access to zone information as well as open, close and reset zones: - REQ_OP_ZONE_REPORT - Query zone information (Report zones) - REQ_OP_ZONE_OPEN - Explictly open a zone for writing - REQ_OP_ZONE_CLOSE - Explictly close a zone - REQ_OP_ZONE_RESET - Reset Write Pointer to st

[PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-07-29 Thread Shaun Tancheff
Hi Jens, This series is based on linus' current tip after the merge of 'for-4.8/core' As Host Aware drives are becoming available we would like to be able to make use of such drives. This series is also intended to be suitable for use by Host Managed drives. ZAC/ZBC drives add new commands for d

Re: [PATCH -next] ipr: Fix error return code in ipr_probe_ioa()

2016-07-29 Thread Brian King
Acked-by: Brian King -- Brian King Power Linux I/O IBM Linux Technology Center -- 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 3/5] nvme-fabrics: Add host FC transport support

2016-07-29 Thread J Freyensee
On Fri, 2016-07-22 at 17:23 -0700, James Smart wrote: A couple of minor comments: > Add nvme-fabrics host FC transport support > > Implements the FC-NVME T11 definition of how nvme fabric capsules are > performed on an FC fabric. Utilizes a lower-layer API to FC host > adapters > to send/receiv

Re: [PATCH 2/5] nvme-fabrics: Add FC transport LLDD api definitions

2016-07-29 Thread J Freyensee
On Mon, 2016-07-25 at 11:12 +0200, Johannes Thumshirn wrote: > On Fri, Jul 22, 2016 at 05:23:56PM -0700, James Smart wrote: > > > > nvme-fabrics: Add FC transport LLDD api definitions: > > > > Host: > > -LLDD registration with the host transport > > -registering host ports (local ports) and targe

Re: [PATCH 1/5] nvme-fabrics: Add FC transport FC-NVME definitions

2016-07-29 Thread J Freyensee
On Mon, 2016-07-25 at 10:56 +0200, Johannes Thumshirn wrote: > On Fri, Jul 22, 2016 at 05:23:55PM -0700, James Smart wrote: > > > > nvme-fabrics: Add FC transport FC-NVME definitions: > > - Formats for Cmd, Data, Rsp IUs > > - Formats FC-4 LS definitions > > > > > > Signed-off-by: James Smart >