[ANNOUNCE]: SCST 3.2 released

2016-12-15 Thread Vladislav Bolkhovitin
Hi All, I'm glad to announce SCST 3.2 has just been released You can download it from http://scst.sourceforge.net/downloads.html SCST is alternative SCSI target stack for Linux. SCST allows creation of sophisticated storage devices, which can provide advanced functionality, like replication, t

Re: [PATCH 08/22] qla2xxx: Refactore target code to remove symbol dependency

2016-12-15 Thread Tran, Quinn
Christoph, Qlogic was asked to support other Target Stack (SC ST) using the same qla2xxx.ko upstream driver. I had mentioned this during the last 2015 LSF get together. Hopefully, that gives a better context. The understanding that came away was “Qlogic do what is needed to provide this supp

Re: iscsi_trx going into D state

2016-12-15 Thread Robert LeBlanc
Nicholas, I've found that the kernels I used were not able to be inspected using crash and I could not build the debug info for them. So I built a new 4.9 kernel and verified that I could inspect the crash. It is located at [1]. [1] http://mirrors.betterservers.com/trace/crash2.tar.xz ---

Re: [PATCH 5/8] linux: drop __bitwise__ everywhere

2016-12-15 Thread Lee Duncan
On 12/14/2016 09:15 PM, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S.

Re: [PATCH 05/22] qla2xxx: Add DebugFS node of irq vector cpuid

2016-12-15 Thread Madhani, Himanshu
Hi Christoph, On 12/14/16, 1:13 PM, "Christoph Hellwig" wrote: >Doesn't make sense - an interrupt can be assigned to multiple >CPUIDs, and you can see the assignment based on the irq count >in /proc/interrupts > We Added this for our debug purpose. We’ll drop this patch in next series. Tha

Re: [PATCH 08/22] qla2xxx: Refactore target code to remove symbol dependency

2016-12-15 Thread Madhani, Himanshu
Hi Christoph, On 12/14/16, 1:18 PM, "Christoph Hellwig" wrote: > - the new qla2x00_free_fcport is entirely pointless, please drop > that part of the patch (and even if it wasn't pointless it should > have been a patch on it's own) > - please use struct names and not typedefs for your new

[PATCH RESEND] block: allow WRITE_SAME commands with the SG_IO ioctl

2016-12-15 Thread Mauricio Faria de Oliveira
The WRITE_SAME commands are not present in the blk_default_cmd_filter write_ok list, and thus are failed with -EPERM when the SG_IO ioctl() is executed without CAP_SYS_RAWIO capability (e.g., unprivileged users). [ sg_io() -> blk_fill_sghdr_rq() > blk_verify_command() -> -EPERM ] The problem can b

Re: [PATCH 5/8] linux: drop __bitwise__ everywhere

2016-12-15 Thread Krzysztof Kozlowski
On Thu, Dec 15, 2016 at 07:15:20AM +0200, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-o

Re: [PATCH 15/22] qla2xxx: Rename qlini_mode parameter

2016-12-15 Thread Madhani, Himanshu
On 12/14/16, 1:07 PM, "Christoph Hellwig" wrote: >On Tue, Dec 06, 2016 at 12:30:44PM -0800, Himanshu Madhani wrote: >> From: Quinn Tran >> >> Qlogic's adapter is able to behave in multiple modes: >> initiator, target, exclusive/either, and dual/both. >> >> This patch renames the qlini_mode ->

Re: [PATCH 01/22] qla2xxx: Fix wrong IOCB type assumption.

2016-12-15 Thread Madhani, Himanshu
On 12/14/16, 1:09 PM, "Christoph Hellwig" wrote: >On Tue, Dec 06, 2016 at 12:30:30PM -0800, Himanshu Madhani wrote: >> +uint32_t unpacked_lun = 0; > >Please remove the unpacked_lun variable as well, and just directly >pass 0 to qlt_issue_task_mgmt. Ack, will update and resend this patch. >

Re: [PATCH 03/22] qla2xxx: Set tcm_qla2xxx version to automatically track qla2xxx version.

2016-12-15 Thread Madhani, Himanshu
On 12/14/16, 1:12 PM, "Christoph Hellwig" wrote: >Please just remove TCM_QLA2XXX_VERSION entirely and use QLA2XXX_VERSION >directly instead. Ack. Will update patch and resubmit. >

Re: [PATCH 14/22] qla2xxx: Add interrupt polling mechanism

2016-12-15 Thread Madhani, Himanshu
Hi Bart, Christoph, On 12/15/16, 1:27 AM, "Bart Van Assche" wrote: >On 12/14/2016 10:06 PM, Christoph Hellwig wrote: >> On Tue, Dec 06, 2016 at 12:30:43PM -0800, Himanshu Madhani wrote: >>> From: Quinn Tran >>> >>> This patch adds capability to poll for an interrupt, If hardware >>> does not

Re: [PATCH] target: fix spelling mistake and unwrap multi-line text

2016-12-15 Thread Bart Van Assche
On 11/07/2016 05:31 PM, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake "Obselete" to "Obsolete" > in error message and also turn multi-line text into one > line. Thanks for the patch. It has been applied on my for-v4.10 branch. Bart. -- To unsubscribe from this li

Re: [PATCH V5 00/11] megaraid_sas: Updates for scsi-next

2016-12-15 Thread Tomas Henzl
On 14.12.2016 23:12, Sasikumar Chandrasekaran wrote: > Sasikumar Chandrasekaran (11): > megaraid_sas: Add new pci device Ids for SAS3.5 Generic Megaraid > Controllers > megaraid_sas: 128 MSIX Support > megaraid_sas: EEDP Escape Mode Support for SAS3.5 Generic Megaraid > Controllers >

Re: [PATCH V5 08/11] megaraid_sas: Enable or Disable Fast path based on the PCI Threshold Bandwidth

2016-12-15 Thread Tomas Henzl
On 14.12.2016 23:13, Sasikumar Chandrasekaran wrote: > Large SEQ IO workload should sent as non fast path commands > > This patch is depending on patch 7 > This patch is same as V4 and there is no specific update for V5 That means, that you have ignored my question regarding 'bytes_wrote' I asked

Re: [PATCH V4 08/11] megaraid_sas: Enable or Disable Fast path based on the PCI Threshold Bandwidth

2016-12-15 Thread Tomas Henzl
On 14.12.2016 22:54, Sasikumar PC wrote: > Hi Tomas, > > Please see my response inline > > Thanks > sasi > > -Original Message- > From: Tomas Henzl [mailto:the...@redhat.com] > Sent: Friday, December 09, 2016 8:59 AM > To: Sasikumar Chandrasekaran; j...@kernel.org; h...@infradead.org > Cc:

Re: [PATCH V5 06/11] megaraid_sas: Dynamic Raid Map Changes for SAS3.5 Generic Megaraid Controllers

2016-12-15 Thread Tomas Henzl
On 14.12.2016 23:13, Sasikumar Chandrasekaran wrote: > SAS3.5 Generic Megaraid Controllers FW will support new dynamic RaidMap to > have different > sizes for different number of supported VDs. > > This patch is depending on patch 5 > Code indentation is fixed for VD_EXT_DEBUG macro > > Signed-off

Re: [PATCH V5 04/11] megaraid_sas: SAS3.5 Generic Megaraid Controllers Stream Detection and IO Coalescing

2016-12-15 Thread Tomas Henzl
On 14.12.2016 23:13, Sasikumar Chandrasekaran wrote: > Detect sequential IO streams and pass those IOs directly to FW. > > This patch is depending on patch 3 > This patch is same as V4 and there is no specific update for V5 That is not correct, I'm glad to see several whitespace changes (for examp

[PATCH] mptlan: Remove linux/miscdevice.h from mptlan.h

2016-12-15 Thread Corentin Labbe
This patch remove linux/miscdevice.h from mptlan.h since mptlan.h does not contain any miscdevice. The only user of it is mptctl.c which already include linux/miscdevice.h So no need to include it twice. Signed-off-by: Corentin Labbe --- drivers/message/fusion/mptlan.h | 1 - 1 file changed, 1 d

Re: [PATCH 5/8] linux: drop __bitwise__ everywhere

2016-12-15 Thread Greg Kroah-Hartman
On Thu, Dec 15, 2016 at 07:15:20AM +0200, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-o

Re: [PATCH 14/22] qla2xxx: Add interrupt polling mechanism

2016-12-15 Thread Bart Van Assche
On 12/14/2016 10:06 PM, Christoph Hellwig wrote: > On Tue, Dec 06, 2016 at 12:30:43PM -0800, Himanshu Madhani wrote: >> From: Quinn Tran >> >> This patch adds capability to poll for an interrupt, If hardware >> does not generate any interrupt for 2 seconds. > > This description sounds like the ha

Re: [PATCH 5/8] linux: drop __bitwise__ everywhere

2016-12-15 Thread Stefan Schmidt
Hello. On 15/12/16 06:15, Michael S. Tsirkin wrote: __bitwise__ used to mean "yes, please enable sparse checks unconditionally", but now that we dropped __CHECK_ENDIAN__ __bitwise is exactly the same. There aren't many users, replace it by __bitwise everywhere. Signed-off-by: Michael S. Tsirkin