Re: [PATCH] aic7xxx: fix firmware build with O=path

2017-07-03 Thread Jakub Kicinski
On Wed, 28 Jun 2017 21:17:16 -0700, Jakub Kicinski wrote: > Building firmware with O=path was apparently broken in aic7 for ever. > Message of the previous commit to the Makefile (from 2008) mentions > this unfortunate state of affairs already. Fix this, mostly to make > randconfig builds more

Re: [PATCH] scsi: ufs: set host_byte to DID_REQUEUE when ocs = OCS_ABORTED

2017-07-03 Thread Subhash Jadavani
On 2017-06-28 19:25, Zang Leigang wrote: On Wed, Jun 28, 2017 at 04:42:36PM -0700, Subhash Jadavani wrote: On 2017-06-24 03:27, Zang Leigang wrote: >Host set ocs to OCS_ABORTED when clear a doorbell in err handler. OCS field is valid after host controller has cleared the corresponding doorbell

[PATCH][V2] scsi: qedf: fix spelling mistake: "offlading" -> "offloading"

2017-07-03 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in QEDF_INFO message and remove duplicated "since" (thanks to Tyrel Datwyler for spotting the latter issue). Signed-off-by: Colin Ian King --- drivers/scsi/qedf/qedf_main.c | 2 +- 1 file

Re: [PATCH] scsi: qedf: fix spelling mistake: "offlading" -> "offloading"

2017-07-03 Thread Tyrel Datwyler
On 07/03/2017 02:44 AM, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake in QEDF_INFO message > > Signed-off-by: Colin Ian King > --- > drivers/scsi/qedf/qedf_main.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH] Make "SCSI Transports" a menuconfig to ease disabling it all

2017-07-03 Thread kbuild test robot
-to-ease-disabling-it-all/20170703-222401 base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next config: tile-tilegx_defconfig (attached as .config) compiler: tilegx-linux-gcc (GCC) 4.6.2 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin

Re: [PATCH] Make "SCSI Transports" a menuconfig to ease disabling it all

2017-07-03 Thread kbuild test robot
-to-ease-disabling-it-all/20170703-222401 base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next config: m68k-sun3_defconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 4.9.0 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross

Re: [PATCH] scsi: qedi: fix another spelling mistake: "alloction" -> "allocation"

2017-07-03 Thread Rangankar, Manish
On 03/07/17 3:54 PM, "Colin King" wrote: >From: Colin Ian King > >Trivial fix to spelling mistake in QEDF_ERR message. I should have >also included this in a previous fix, but I only just spotted this one. > >Signed-off-by: Colin Ian King

[PATCH] scsi: qedi: fix another spelling mistake: "alloction" -> "allocation"

2017-07-03 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in QEDF_ERR message. I should have also included this in a previous fix, but I only just spotted this one. Signed-off-by: Colin Ian King --- drivers/scsi/qedi/qedi_fw.c | 2 +- 1 file

[PATCH] scsi: isci: fix typo in function names

2017-07-03 Thread Colin King
From: Colin Ian King There are a couple of typos in function names and spelling of request where the letters u and e are swapped: scu_ssp_reqeust_construct_task_context scu_sata_reqeust_construct_task_context Fix the spelling of request. Signed-off-by: Colin Ian King

[PATCH] scsi: qedf: fix spelling mistake: "offlading" -> "offloading"

2017-07-03 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in QEDF_INFO message Signed-off-by: Colin Ian King --- drivers/scsi/qedf/qedf_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/qedf/qedf_main.c

[PATCH v7 3/6] g_NCR5380: Cleanup comments and whitespace

2017-07-03 Thread Finn Thain
Signed-off-by: Finn Thain --- drivers/scsi/g_NCR5380.c | 61 ++-- 1 file changed, 28 insertions(+), 33 deletions(-) diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c index 911a4300ea51..dedaed2d16e4 100644

[PATCH v7 5/6] g_NCR5380: Re-work PDMA loops

2017-07-03 Thread Finn Thain
From: Ondrej Zary The polling loops in pread() and pwrite() can easily become infinite loops and hang the machine. Merge the IRQ check into host buffer wait loop and add polling limit. Also place a limit on polling for 53C80 registers accessibility. [Use

Re: [PATCH v6 0/6] g_NCR5380: PDMA fixes and cleanup

2017-07-03 Thread Finn Thain
On Sun, 2 Jul 2017, Ondrej Zary wrote: > On Sunday 02 July 2017 05:11:27 Finn Thain wrote: > > On Sat, 1 Jul 2017, Ondrej Zary wrote: > > > The write corruption is still present - "start" must be rolled back > > > in both IRQ and timeout cases. > > > > Your original algorithm aborts the transfer

[PATCH v7 6/6] g_NCR5380: Two DTC436 PDMA workarounds

2017-07-03 Thread Finn Thain
From: Ondrej Zary Limit PDMA send to 512 B to avoid data corruption on DTC3181E. The corruption is always the same: one byte missing at the beginning of a 128 B block. It happens only with slow Quantum LPS 240 drive, not with faster IBM DORS-32160. It's not clear what

[PATCH v7 1/6] g_NCR5380: Fix PDMA transfer size

2017-07-03 Thread Finn Thain
From: Ondrej Zary generic_NCR5380_dma_xfer_len() incorrectly uses cmd->transfersize which causes rescan-scsi-bus and CD-ROM access to hang the system. Use cmd->SCp.this_residual instead, like other NCR5380 drivers. Signed-off-by: Ondrej Zary

[PATCH v7 0/6] g_NCR5380: PDMA fixes and cleanup

2017-07-03 Thread Finn Thain
Ondrej, would you please test this new series? Changed since v1: - PDMA transfer residual is calculated earlier. - End of DMA flag check is now polled (if there is any residual). Changed since v2: - Bail out of transfer loops when Gated IRQ gets asserted. - Make udelay conditional on board type.

[PATCH v7 2/6] g_NCR5380: End PDMA transfer correctly on target disconnection

2017-07-03 Thread Finn Thain
From: Ondrej Zary When an IRQ arrives during PDMA transfer, pread() and pwrite() return without waiting for the 53C80 registers to be ready and this ends up messing up the chip state. This was observed with SONY CDU-55S which is slow enough to disconnect during

[PATCH v7 4/6] g_NCR5380: Use unambiguous terminology for PDMA send and receive

2017-07-03 Thread Finn Thain
The word "read" may be used to mean "DMA read operation" or "SCSI READ command", though a READ command implies writing to memory. Signed-off-by: Finn Thain --- drivers/scsi/g_NCR5380.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

RE: [PATCH 1/2] Drop legacy megaraid controller

2017-07-03 Thread Sumit Saxena
>-Original Message- >From: Hannes Reinecke [mailto:h...@suse.de] >Sent: Friday, June 30, 2017 11:46 PM >To: Christoph Hellwig >Cc: Martin K. Petersen; James Bottomley; Kashyap Desai; Sumit Saxena; >linux- >s...@vger.kernel.org; Hannes Reinecke >Subject: Re: [PATCH 1/2] Drop legacy megaraid

RE: [PATCH 09/15] megaraid_sas: use vmalloc for crash dump buffers and driver's local RAID map

2017-07-03 Thread Shivasharan Srikanteshwara
> -Original Message- > From: Tomas Henzl [mailto:the...@redhat.com] > Sent: Friday, June 30, 2017 8:01 PM > To: Shivasharan S; linux-scsi@vger.kernel.org > Cc: martin.peter...@oracle.com; j...@linux.vnet.ibm.com; > kashyap.de...@broadcom.com; sumit.sax...@broadcom.com; > h...@suse.com;