Re: [PATCH v2 1/8] scsi: Add ufs transport class

2018-08-09 Thread Douglas Gilbert
On 2018-08-09 07:51 PM, Bart Van Assche wrote: On Thu, 2018-08-09 at 23:32 +, Avri Altman wrote: And as I said before, we think that maintaining the flexibility to have more-than-one bsg device nodes, will be useful serving as a testing and validation environment. That is a very vague

Re: [PATCH v2 1/8] scsi: Add ufs transport class

2018-08-09 Thread Bart Van Assche
On Thu, 2018-08-09 at 23:32 +, Avri Altman wrote: > And as I said before, we think that maintaining the flexibility to have > more-than-one > bsg device nodes, will be useful serving as a testing and validation > environment. That is a very vague statement. Please clarify. > >> "... > >>

Re: [PATCH v2 1/8] scsi: Add ufs transport class

2018-08-09 Thread Bart Van Assche
On Thu, 2018-08-09 at 16:22 +, Avri Altman wrote: > > -Original Message- > > From: Bart Van Assche > > Sent: Wednesday, August 08, 2018 7:58 PM > > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > >

RE: [PATCH v2 1/8] scsi: Add ufs transport class

2018-08-09 Thread Avri Altman
> -Original Message- > From: Bart Van Assche > Sent: Wednesday, August 08, 2018 7:58 PM > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > j...@linux.vnet.ibm.com > Cc: Vinayak Holikatti ; Avi Shchislowski ;

Re: [PATCH] aha1542: convert to DMA mapping API

2018-08-09 Thread Ondrej Zary
On Thursday 09 August 2018 07:35:33 Christoph Hellwig wrote: > On Wed, Aug 08, 2018 at 10:30:19PM +0200, Ondrej Zary wrote: > > Then it crashes with null-pointer dereference in aha1542_reset. > > Must be the dma_unmap. Updated patch below: No change in behavior, unfortunately. The crash is

Re: [PATCH 0/3] scsi: fcoe: memleak fixes

2018-08-09 Thread ard
Hi, On Thu, Aug 09, 2018 at 12:01:30PM +0200, ard wrote: > This to determine if we have a single regression in just the > login handling or both. As a matter of fact, I think this will not work on vn2vn:

Re: [PATCH] scsi: csiostor: remove automatic irq affinity assignment

2018-08-09 Thread Varun Prakash
On Wed, Aug 01, 2018 at 05:18:25PM +0530, Varun Prakash wrote: > On Wed, Aug 01, 2018 at 08:33:23AM +0200, Hannes Reinecke wrote: > > On 07/31/2018 05:07 PM, Varun Prakash wrote: > > > If number of interrupt vectors are more than num_online_cpus() > > > then pci_alloc_irq_vectors_affinity()

Re: [PATCH] Simplify nvmet_init to remove extra out label

2018-08-09 Thread Milan P. Gandhi
Thank you Johannes, I will send it on linux-n...@lists.infradead.org On Thu, Aug 09, 2018 at 01:51:18PM +0200, Johannes Thumshirn wrote: > please send this patch to the correct mailing list > (linux-n...@lists.infradead.org) > > -- > Johannes Thumshirn

Re: [PATCH] Simplify nvmet_init to remove extra out label

2018-08-09 Thread Johannes Thumshirn
please send this patch to the correct mailing list (linux-n...@lists.infradead.org) -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer,

Re: [PATCH] Remove extra goto label from nvmet_ns_enable

2018-08-09 Thread Johannes Thumshirn
please send this patch to the correct mailing list (linux-n...@lists.infradead.org) -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer,

[PATCH] Simplify nvmet_init to remove extra out label

2018-08-09 Thread Milan P. Gandhi
The out label at end could easily be removed by returning an error upon receiving non-zero error return code from nvmet_init_discovery. This would avoid need for 2nd out label. Signed-off-by: Milan P. Gandhi --- drivers/nvme/target/core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH] Remove extra goto label from nvmet_ns_enable

2018-08-09 Thread Milan P. Gandhi
Currently nvmet_ns_enable has two labels viz. out_unlock and out_dev_put, and the reverse call from out_dev_put to out_unlock is little bit confusing. We could simplify it by calling nvmet_ns_dev_disable before. This would eliminate need for 2nd out_dev_put label. Also, this function already

[PATCH V8 0/2] Add UFS provisioning support in driver

2018-08-09 Thread Sayali Lokhande
This patch adds Configfs support to provision UFS device at runtime. This feature can be primarily useful in factory or assembly line as some devices may be required to be configured multiple times during initial system development phase. Configuration Descriptors can be written multiple times

RE: [PATCH v2 7/8] scsi: ufs-bsg: Add support for raw upiu in ufs_bsg_request()

2018-08-09 Thread Avri Altman
Thanks, Avri > -Original Message- > From: Bart Van Assche > Sent: Wednesday, August 08, 2018 7:40 PM > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > j...@linux.vnet.ibm.com > Cc: Vinayak Holikatti ; Avi

RE: [PATCH v2 6/8] scsi: ufs: Add API to execute raw upiu commands

2018-08-09 Thread Avri Altman
Thanks, Avri > -Original Message- > From: Bart Van Assche > Sent: Wednesday, August 08, 2018 7:39 PM > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > j...@linux.vnet.ibm.com > Cc: Vinayak Holikatti ; Avi

RE: [PATCH v2 2/8] scsi: ufs: Add ufs-bsg module

2018-08-09 Thread Avri Altman
Thanks, Avri > -Original Message- > From: Bart Van Assche > Sent: Wednesday, August 08, 2018 7:27 PM > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > j...@linux.vnet.ibm.com > Cc: Vinayak Holikatti ; Avi

RE: [PATCH v2 1/8] scsi: Add ufs transport class

2018-08-09 Thread Avri Altman
> -Original Message- > From: Bart Van Assche > Sent: Wednesday, August 08, 2018 7:21 PM > To: h...@lst.de; Avri Altman ; linux-scsi@vger.kernel.org; > jthumsh...@suse.de; h...@suse.com; martin.peter...@oracle.com; > j...@linux.vnet.ibm.com > Cc: Vinayak Holikatti ; Avi Shchislowski ;