re: [SCSI] fnic: FIP VLAN Discovery Feature Support

2014-01-08 Thread Dan Carpenter
Hello Hiral Patel, The patch d3c995f1dcf9: [SCSI] fnic: FIP VLAN Discovery Feature Support from Feb 25, 2013, leads to the following static checker warning: drivers/scsi/fnic/fnic_fcs.c:278 is_fnic_fip_flogi_reject() warn: is it ok to set 'els_len' to negative? This is some

RE: [patch] [SCSI] megaraid: missing bounds check in mimd_to_kioc()

2014-01-08 Thread Saxena, Sumit
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Wednesday, October 30, 2013 10:44 PM To: DL-MegaRAID Linux Cc: James E.J. Bottomley; linux-scsi@vger.kernel.org; secur...@kernel.org; Nico Golde; Fabian Yamaguchi Subject: [patch] [SCSI] megaraid: missing

RE: [PATCH] megaraid_sas: make the log message about rescanning more informative.

2014-01-08 Thread Saxena, Sumit
-Original Message- From: Jasper Spaans [mailto:spa...@fox-it.com] Sent: Friday, October 25, 2013 5:53 PM To: DL-MegaRAID Linux Cc: linux-scsi@vger.kernel.org Subject: [PATCH] megaraid_sas: make the log message about rescanning more informative. Hi, I was working on one of our servers,

RE: [PATCH] [SCSI] megaraid: use GFP_ATOMIC under spin lock

2014-01-08 Thread Saxena, Sumit
-Original Message- From: Wei Yongjun [mailto:weiyj...@gmail.com] Sent: Friday, December 20, 2013 8:38 AM To: DL-MegaRAID Linux; jbottom...@parallels.com; simon.pu...@gmail.com; jkos...@suse.cz Cc: yongjun_...@trendmicro.com.cn; linux-scsi@vger.kernel.org Subject: [PATCH] [SCSI] megaraid:

Re: [SCSI] fnic: FIP VLAN Discovery Feature Support

2014-01-08 Thread Dan Carpenter
Hiral isn't at Cisco any more. regards, dan carpenter On Wed, Jan 08, 2014 at 12:57:41PM +0300, Dan Carpenter wrote: Hello Hiral Patel, The patch d3c995f1dcf9: [SCSI] fnic: FIP VLAN Discovery Feature Support from Feb 25, 2013, leads to the following static checker warning:

Re: Terrible performance of sequential O_DIRECT 4k writes in SAN environment. ~3 times slower then Solars 10 with the same HBA/Storage.

2014-01-08 Thread Christoph Hellwig
On Tue, Jan 07, 2014 at 08:37:23PM +0200, Sergey Meirovich wrote: Actually my initial report (14.67Mb/sec 3755.41 Requests/sec) was about ext4 However I have tried XFS as well. It was a bit slower than ext4 on all occasions. I wasn't trying to say XFS fixes your problem, but that we could

Re: Terrible performance of sequential O_DIRECT 4k writes in SAN environment. ~3 times slower then Solars 10 with the same HBA/Storage.

2014-01-08 Thread Sergey Meirovich
Hi James, On 7 January 2014 22:57, James Smart james.sm...@emulex.com wrote: Sergey, The Thor chipset is a bit old - a 4Gig adapter. Most of our performance improvements, including parallelization, have gone into the 8G and 16G adapters. But you still should have seen significantly beyond

Re: [PATCH] drivers: target: target_core_mod: use div64_u64_rem() instead of operator '%' for u64

2014-01-08 Thread Chen Gang
On 01/08/2014 03:32 PM, Hannes Reinecke wrote: On 12/24/2013 04:35 AM, Chen Gang wrote: On 12/23/2013 02:51 PM, Nicholas A. Bellinger wrote: On Sun, 2013-12-22 at 17:17 +0800, Chen Gang wrote: On 12/22/2013 10:56 AM, Nicholas A. Bellinger wrote: Hi Chen, On Sat, 2013-12-21 at 10:08 +0800,

Re: Terrible performance of sequential O_DIRECT 4k writes in SAN environment. ~3 times slower then Solars 10 with the same HBA/Storage.

2014-01-08 Thread Sergey Meirovich
Hi Christoph, On 8 January 2014 16:03, Christoph Hellwig h...@infradead.org wrote: On Tue, Jan 07, 2014 at 08:37:23PM +0200, Sergey Meirovich wrote: Actually my initial report (14.67Mb/sec 3755.41 Requests/sec) was about ext4 However I have tried XFS as well. It was a bit slower than ext4 on

Re: status of block-integrity

2014-01-08 Thread Martin K. Petersen
James == James Bottomley james.bottom...@hansenpartnership.com writes: James No, I think you're confusing algorithms with protocols. DIF and James DIX are two names for protection envelopes. DIF verifies James integrity from the HBA to the device surface. DIX verifies James integrity from an

Re: [RFC 2/2] dual scan thread bug fix

2014-01-08 Thread Alan Stern
On Wed, 8 Jan 2014, James Bottomley wrote: OK, Agreed, but that means modifying the 1/2 patch with the below. This should make the proposed diff to 2/2 correct. James --- diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index ef3f958..5fad646 100644 ---

RE: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread David Laight
From: Sarah Sharp On Tue, Jan 07, 2014 at 03:57:00PM -0800, walt wrote: On 01/07/2014 01:21 PM, Sarah Sharp wrote: Can you please try the attached patch, on top of the previous three patches, and send me dmesg? Hi Sarah, I just now finished running 0001-More-debugging.patch for the

Re: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread Alan Stern
On Tue, 7 Jan 2014, Sarah Sharp wrote: On Tue, Jan 07, 2014 at 03:57:00PM -0800, walt wrote: On 01/07/2014 01:21 PM, Sarah Sharp wrote: Can you please try the attached patch, on top of the previous three patches, and send me dmesg? Hi Sarah, I just now finished running

RE: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread David Laight
From: Alan Stern This may be a foolish question, but why is xhci-hcd using no-op TRBs in the first place? Because it can't write in a link TRB because other parts of the code use link TRBs to detect the end of the ring. The problem is that it can't put a link TRB in the middle of a chain of

RE: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread Alan Stern
On Wed, 8 Jan 2014, David Laight wrote: From: Alan Stern This may be a foolish question, but why is xhci-hcd using no-op TRBs in the first place? Because it can't write in a link TRB because other parts of the code use link TRBs to detect the end of the ring. The problem is that

Re: Terrible performance of sequential O_DIRECT 4k writes in SAN environment. ~3 times slower then Solars 10 with the same HBA/Storage.

2014-01-08 Thread Sergey Meirovich
On 8 January 2014 17:26, Christoph Hellwig h...@infradead.org wrote: On my laptop SSD I get the following results (sometimes up to 200MB/s, sometimes down to 100MB/s, always in the 40k to 50k IOps range): time elapsed (sec.):5 bandwidth (MiB/s): 160.00 IOps:

RE: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread David Laight
From: Alan Stern On Wed, 8 Jan 2014, David Laight wrote: From: Alan Stern This may be a foolish question, but why is xhci-hcd using no-op TRBs in the first place? Because it can't write in a link TRB because other parts of the code use link TRBs to detect the end of the ring.

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Alan Stern
On Tue, 7 Jan 2014, Phillip Susi wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 01/07/2014 02:18 PM, Alan Stern wrote: I don't know how you manually spin down your disk, but one reasonable way to do it is to set the autosuspend timeout to 0. If you use this approach and

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/8/2014 2:00 AM, Aaron Lu wrote: Then I would say we just keep the pm_request_resume call in their system resume callback. For drives that have that cool feature turned on, it will be in runtime active state while it's actually in standby

Re: [PATCH] [SCSI] advansys: change buildtime warning into runtime error

2014-01-08 Thread Paul Bolle
On Tue, 2013-01-29 at 11:20 +0100, Paul Bolle wrote: On Mon, 2012-11-05 at 11:58 +0100, Paul Bolle wrote: Building advansys.o triggers this warning: drivers/scsi/advansys.c:71:2: warning: #warning this driver is still not properly converted to the DMA API [-Wcpp] This warning can

Re: [PATCH] [SCSI] advansys: change buildtime warning into runtime error

2014-01-08 Thread Ondrej Zary
On Wednesday 08 January 2014 21:05:14 Paul Bolle wrote: On Tue, 2013-01-29 at 11:20 +0100, Paul Bolle wrote: On Mon, 2012-11-05 at 11:58 +0100, Paul Bolle wrote: Building advansys.o triggers this warning: drivers/scsi/advansys.c:71:2: warning: #warning this driver is still not

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/8/2014 12:46 PM, Alan Stern wrote: I/O _was_ done. To spin up the disk requires sending it a START command. That's I/O. Yes, currently, but that's exactly what I'm trying to get rid of. In essence, you want your disk's state to move

Re: [PATCH] [SCSI] advansys: change buildtime warning into runtime error

2014-01-08 Thread Paul Bolle
On Wed, 2014-01-08 at 21:15 +0100, Ondrej Zary wrote: The driver works fine with narrow boards. Please don't break working drivers because of some stupid warning. My patch changed a buildtime warning for both narrow and wide boards in a runtime error on wide boards only. That wouldn't break

[PATCH 00/14] target: Initial support for DIF Type1+Type3 emulation

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org Hi MKP SCSI folks, This series contains initial support for target mode DIF Type1+Type3 emulation within target core, RAMDISK_MCP device backend, and tcm_loop fabric driver. DIF emulation is enabled via a new 'pi_prot_type' device attribute within

[PATCH 06/14] target/spc: Add protection related bits to INQUIRY EVPD=0x86

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates spc_emulate_evpd_86() (extended INQUIRY) to report GRD_CHK (Guard Check) and REF_CHK (Reference Check) bits when DIF emulation is enabled by the backend device. Cc: Martin K. Petersen martin.peter...@oracle.com Cc: Christoph

[PATCH 10/14] target: Add protection SGLs to target_submit_cmd_map_sgls

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds support to target_submit_cmd_map_sgls() for accepting 'sgl_prot' + 'sgl_prot_count' parameters for DIF protection information. Note the passed parameters are stored at se_cmd-t_prot_sg and se_cmd-t_prot_nents respectively. Also,

[PATCH 12/14] target/rd: Add support for protection SGL setup + release

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds rd_build_prot_space() + rd_release_prot_space() logic to setup + release protection information scatterlists. It also adds rd_init_prot() + rd_free_prot() se_subsystem_api callbacks used by target core code for setup + release of

[PATCH 14/14] tcm_loop: Enable DIF/DIX modes in SCSI host LLD

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates tcm_loop_driver_probe() to set protection information using scsi_host_set_prot() and scsi_host_set_guard(), which currently enabled all modes of DIF/DIX protection, minus DIF TYPE0. Also, update tcm_loop_submission_work() to pass

[PATCH 11/14] target/rd: Refactor rd_build_device_space + rd_release_device_space

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch refactors rd_build_device_space() + rd_release_device_space() into rd_allocate_sgl_table() + rd_release_device_space() so that they may be used seperatly for setup + release of protection information scatterlists. Also add explicit memset

[PATCH 09/14] target/configfs: Expose protection device attributes

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds support for exposing DIF protection device attributes via configfs. This includes: pi_prot_type: Protection Type (0, 1, 3 currently support) pi_prot_version: Protection Version (DIF v1 currently supported) pi_guard_type:

[PATCH 13/14] target/rd: Add DIF protection into rd_execute_rw

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds support for DIF protection into rd_execute_rw() code for WRITE/READ I/O using sbc_dif_verify_[write,read]() logic. It also adds rd_get_prot_table() for locating protection SGLs assoicated with the ramdisk backend device. Cc: Martin

[PATCH 07/14] target/sbc: Add P_TYPE + PROT_EN bits to READ_CAPACITY_16

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates sbc_emulate_readcapacity_16() to set P_TYPE and PROT_EN bits when DIF emulation is enabled by the backend device. Cc: Martin K. Petersen martin.peter...@oracle.com Cc: Christoph Hellwig h...@lst.de Cc: Hannes Reinecke h...@suse.de

[PATCH 08/14] target/spc: Expose ATO bit in control mode page

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates spc_modesense_control() to set the Application Tag Owner (ATO) bit when when DIF emulation is enabled by the backend device. Cc: Martin K. Petersen martin.peter...@oracle.com Cc: Christoph Hellwig h...@lst.de Cc: Hannes Reinecke

[PATCH 05/14] target/spc: Add protection bit to standard INQUIRY output

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch updates spc_emulate_inquiry_std() to set the PROTECT bit when DIF emulation is enabled by the backend device. Cc: Martin K. Petersen martin.peter...@oracle.com Cc: Christoph Hellwig h...@lst.de Cc: Hannes Reinecke h...@suse.de Cc: Sagi

[PATCH 02/14] target: Add DIF CHECK_CONDITION ASC/ASCQ exception cases

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds support for DIF related CHECK_CONDITION ASC/ASCQ exception cases into transport_send_check_condition_and_sense(). This includes: LOGICAL BLOCK GUARD CHECK FAILED LOGICAL BLOCK APPLICATION TAG CHECK FAILED LOGICAL BLOCK

[PATCH 04/14] target/sbc: Add DIF TYPE1+TYPE3 read/write verify emulation

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds support for DIF read/write verify emulation for TARGET_DIF_TYPE1_PROT + TARGET_DIF_TYPE3_PROT operation. This includes sbc_dif_verify_write() + sbc_dif_verify_read() calls accessable by backend drivers to perform DIF verify for SGL

[PATCH 01/14] target: Add DIF related base definitions

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds DIF related definitions to target_core_base.h that includes enums for target_prot_op + target_prot_type + target_prot_version + target_guard_type + target_pi_error. Also included is struct se_dif_v1_tuple, along with changes to struct

[PATCH 03/14] target/sbc: Add sbc_check_prot + update sbc_parse_cdb for DIF

2014-01-08 Thread Nicholas A. Bellinger
From: Nicholas Bellinger n...@linux-iscsi.org This patch adds sbc_check_prot() for performing various DIF related CDB sanity checks, along with setting SCF_PROT once sanity checks have passed. Also, add calls in sbc_parse_cdb() for READ_[10,12,16] + WRITE_[10,12,16] to perform DIF sanity

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Alan Stern
On Wed, 8 Jan 2014, Phillip Susi wrote: In essence, you want your disk's state to move directly from unpowered (system asleep) to runtime-suspended with no intermediate active -- but only if the disk doesn't spin up all by itself. One problem: How can the kernel tell whether the disk

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 1/8/2014 4:21 PM, Alan Stern wrote: I never saw your patches. Where were they posted? Higher up in this thread on linux-ide and linux-scsi. Subject was Let sleeping disks lie. If you issue the REQUEST SENSE command in the usual way (going

Re: [PATCH] drivers: target: target_core_mod: use div64_u64_rem() instead of operator '%' for u64

2014-01-08 Thread Nicholas A. Bellinger
On Wed, 2014-01-08 at 08:32 +0100, Hannes Reinecke wrote: On 12/24/2013 04:35 AM, Chen Gang wrote: On 12/23/2013 02:51 PM, Nicholas A. Bellinger wrote: On Sun, 2013-12-22 at 17:17 +0800, Chen Gang wrote: SNIP The related fix patch changed start_lba = lba % ... to start_lba = lba / ...,

Re: [PATCH 3.12 033/118] usb: xhci: Link TRB must not occur within a USB payload burst

2014-01-08 Thread walt
On 01/03/2014 03:29 PM, Sarah Sharp wrote: I'll let you know when I have some diagnostic patches ready. Hi Sarah. I see today gregkh committed the patches you've already sent me, so I assume someone (other than me) has tested those patches and discovered some benefit from them? I'm still

Re: REQ_PM vs REQ_TYPE_PM_RESUME

2014-01-08 Thread Aaron Lu
On 01/09/2014 05:21 AM, Alan Stern wrote: On Wed, 8 Jan 2014, Phillip Susi wrote: You issue a REQUEST SENSE command and that returns status indicating whether the drive is stopped, or in standby. See my patches. One of I never saw your patches. Where were they posted? If you issue the

[PATCH] iscsi: fix wrong order of opcode and itt in iscsi_handle_reject prompt

2014-01-08 Thread Vaughan Cao
This patch makes reject messages show right value for opcode and itt, which is converse previously. Signed-off-by: Vaughan Cao vaughan@oracle.com --- drivers/scsi/libiscsi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/libiscsi.c

Re: [PATCH] iscsi: fix wrong order of opcode and itt in iscsi_handle_reject prompt

2014-01-08 Thread Mike Christie
On 01/08/2014 08:21 PM, Vaughan Cao wrote: This patch makes reject messages show right value for opcode and itt, which is converse previously. Signed-off-by: Vaughan Cao vaughan@oracle.com --- drivers/scsi/libiscsi.c | 8 1 file changed, 4 insertions(+), 4 deletions(-)