Re: [PATCH 17/41] scsi: isci: phy: Mark expected switch fall-through

2018-12-20 Thread Artur Paszkiewicz
On 11/28/18 5:29 AM, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Addresses-Coverity-ID: 703127 ("Missing break in switch") > Signed-off-by: Gustavo A. R. Silva Acked-by: Artur Paszkiewicz

Re: [PATCH 20/41] scsi: isci: request: mark expected switch fall-through

2018-12-20 Thread Artur Paszkiewicz
On 11/28/18 5:30 AM, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Notice that, in this particular case, a dash is added as a token in > order to separate the "Fall through" annotation from the rest o

Re: [PATCH 19/41] scsi: isci: remote_node_context: mark expected switch fall-throughs

2018-12-20 Thread Artur Paszkiewicz
On 11/28/18 5:30 AM, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Notice that, in this particular case, a dash is added as a token in > order to separate the "Fall through" annotations from the rest

[PATCH] scsi: smartpqi: call pqi_free_interrupts() in pqi_shutdown()

2018-12-20 Thread Yanjiang Jin
We must free all irqs during shutdown, else kexec's 2nd kernel would hang in pqi_wait_for_completion_io() as below: Call trace: pqi_wait_for_completion_io pqi_submit_raid_request_synchronous.constprop.78+0x23c/0x310 [smartpqi] pqi_configure_events+0xec/0x1f8 [smartpqi] pqi_ctrl_init+0x814/0xc

Re: [PATCH 18/41] scsi: isci: remote_device: Mark expected switch fall-throughs

2018-12-20 Thread Artur Paszkiewicz
On 11/28/18 5:30 AM, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Notice that, in this particular case, a dash is added as a token in > order to separate the "fall through" annotations from the rest

[PATCH] How to reproduce this issue

2018-12-20 Thread Yanjiang Jin
This happens only in the following combinations: 1. smartpqi is built as module, not built-in; 2. We have a disk connected to smartpqi card; 3. Both kexec's 1st and 2nd kernels use this disk as Rootfs' mount point. This issue can be found on my ARM64 board: QDF2400: kexec -l /boot/vmlinuz-4.20.0

Re: [PATCH -next] scsi: fcoe: remove set but not used variable 'port'

2018-12-20 Thread Johannes Thumshirn
I somehow have a deja vu feeling I have already acked it, but anyways: Acked-by: Johannes Thumshirn

Aw: [ANNOUNCE v2]: Broadcom (Emulex) FC Target driver - efct

2018-12-20 Thread Sebastian Herbszt
Hello James, > I noticed this version does only build here on kernel 4.18. I pushed the > required changes for kernel 4.19 to my repository [1]. this patch [1] is required on 32 bit to fix WARNING: "__udivdi3" [efct.ko] undefined! It seems v2 contains some regressions: My simple setup triggere

Re: scsi: ufs: Problem at init on msm8998

2018-12-20 Thread Marc Gonzalez
On 19/12/2018 18:27, Marc Gonzalez wrote: > On 18/12/2018 17:54, Marc Gonzalez wrote: > >> Now the fun begins, to find the tiny difference that makes/breaks everything >> :-) > I thought I had found the problem: > "clk_set_rate: ufs_unipro_core_clk_src" was not occurring. > But, in fact, I think

Re: scsi: ufs: Problem at init on msm8998

2018-12-20 Thread Marc Gonzalez
On 20/12/2018 13:38, Marc Gonzalez wrote: > Found something fishy... [ufs_phy_aux_clk rate returned 0] Even with that "fixed" [or rather hacked around, since the syntax of drivers/clk/qcom/gcc-msm8998.c remains nebulous, even after staring at it, literally, for hours] the UFSHC refuses to functio

Re: [PATCH 31/41] scsi: mpt3sas: mpt3sas_scsih: Mark expected switch fall-through

2018-12-20 Thread Gustavo A. R. Silva
Hi Suganath, On 12/19/18 10:19 PM, Suganath Prabu Subramani wrote: Hi Gustavo, This patch may not apply smoothly over 4.21/scsi-queue. Our previous patch for Aero had some changes in this switch case. Can you resend this patch with latest code base ? Okay. I'll send a separate patch. Thanks

[PATCH] scsi: mpt3sas: mpt3sas_scsih: Mark expected switch fall-through

2018-12-20 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 1475400 ("Missing break in switch") Signed-off-by: Gustavo A. R. Silva --- drivers/scsi/mpt3sas/mpt3sas_scsih.c | 1 + 1 file changed, 1 insertion(+) diff --git a/

[Bug 201935] Writing data to tape broken

2018-12-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201935 --- Comment #4 from Todd Aiken (tai...@mvtech.ca) --- Just installed 4.19.11, and the fix works. Many thanks to everyone who helped squash this bug. :-) -- You are receiving this mail because: You are the assignee for the bug.

[PATCH][next] scsi: lpfc: replace snprintf with strscpy

2018-12-20 Thread Colin King
From: Colin Ian King Using snprintf without a format specifier is potentially risky if the string phba->ModelDesc contains format specifiers. Replace this with the safer and more efficient strscpy. Cleans up clang warning: drivers/scsi/lpfc/lpfc_scsi.c:4543:31: warning: format string is not a st

Re: [PATCH] bsg: deprecate BIDI support in bsg

2018-12-20 Thread Jens Axboe
On 12/20/18 12:12 AM, Christoph Hellwig wrote: > On Wed, Dec 19, 2018 at 04:23:14PM -0700, Jens Axboe wrote: >> On 12/19/18 7:39 AM, Christoph Hellwig wrote: >>> Besides the OSD command set that never got traction, the only SCSI >>> command using bidirectional buffers is XDWRITEREAD in the 10 and 3

Re: [PATCH 1/3] dt-bindings: ufs: Add HI3670 UFS controller binding

2018-12-20 Thread Rob Herring
On Fri, Dec 07, 2018 at 11:21:49PM +0530, Manivannan Sadhasivam wrote: > Add devicetree binding for HI3670 UFS controller. HI3760 SoC is very > similar to HI3660 SoC with almost same IPs. Only major difference interms s/interms/in terms/ > of UFS is the PHY. HI3670 has 10nm PHY. > > Signed-off-b

[PATCH v2] bsg: deprecate BIDI support in bsg

2018-12-20 Thread Christoph Hellwig
Besides the OSD command set that never got traction, the only SCSI command using bidirectional buffers is XDWRITEREAD in the 10 and 32 byte variants, which is extremely esoteric and has been removed from the spec again as of SBC4r15. It probably doesn't make sense to keep the support code around j

Re: [PATCH v2] bsg: deprecate BIDI support in bsg

2018-12-20 Thread Jens Axboe
On 12/20/18 10:36 AM, Christoph Hellwig wrote: > Besides the OSD command set that never got traction, the only SCSI > command using bidirectional buffers is XDWRITEREAD in the 10 and 32 byte > variants, which is extremely esoteric and has been removed from the spec > again as of SBC4r15. It probab

Re: [PATCH] bsg: deprecate BIDI support in bsg

2018-12-20 Thread Douglas Gilbert
On 2018-12-20 12:09 p.m., Jens Axboe wrote: On 12/20/18 12:12 AM, Christoph Hellwig wrote: On Wed, Dec 19, 2018 at 04:23:14PM -0700, Jens Axboe wrote: On 12/19/18 7:39 AM, Christoph Hellwig wrote: Besides the OSD command set that never got traction, the only SCSI command using bidirectional bu

[PATCH V2 1/3] smartpqi: increase fw status register read timeout

2018-12-20 Thread Don Brace
From: Mahesh Rajashekhara Problem: - during the driver initialization, driver will poll fw for KERNEL_UP in a 30 seconds timeout. - if the firmware is not ready after 30 seconds, driver will not be loaded. Fix: - change timeout from 30 seconds to 3 minutes. Reported-by: Feng Li Review

[PATCH V2 2/3] smartpqi: add ofa support

2018-12-20 Thread Don Brace
From: Mahesh Rajashekhara - when OFA event occurs, driver will stop traffic to RAID/HBA path. Driver waits for all the outstanding requests to complete. - Driver sends OFA event acknowledgment to firmware. - Driver will wait until the new firmware is up and running. - Driver will free up the

[PATCH V2 0/3] smartpqi additional patches

2018-12-20 Thread Don Brace
These patches are based on Linus's tree There are two more patches that need to be added that were not fully tested until now. The changes are: - smartpqi-increase-fw-status-register-read-timeout . wait longer for fw to fully initialize - smartpqi-add-ofa-support . allow on-line firmware upd

[PATCH V2 3/3] smartpqi: update driver version

2018-12-20 Thread Don Brace
- need to bump up the driver version because of the OFA patch and the fw status register read timeout patch. Reviewed-by: Gerry Morong Signed-off-by: Don Brace --- drivers/scsi/smartpqi/smartpqi_init.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/s

RE: [PATCH] scsi: smartpqi: call pqi_free_interrupts() in pqi_shutdown()

2018-12-20 Thread Don.Brace
-Original Message- From: Yanjiang Jin [mailto:yanjiang@hxt-semitech.com] Sent: Thursday, December 20, 2018 2:33 AM To: don.br...@microsemi.com; j...@linux.vnet.ibm.com; martin.peter...@oracle.com; yanjiang@hxt-semitech.com Cc: yu.zh...@hxt-semitech.com; jinyanji...@gmail.com; esc

RE: remove exofs, the T10 OSD code and block/scsi bidi support V3

2018-12-20 Thread Elliott, Robert (Persistent Memory)
> -Original Message- > From: linux-kernel-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Douglas Gilbert > Sent: Wednesday, December 19, 2018 9:02 PM > Subject: Re: remove exofs, the T10 OSD code and block/scsi bidi support V3 > ... > While on the subject of bidi, the order

Re: [PATCH] iscsi: Capture iscsi debug messages using tracepoints

2018-12-20 Thread Fred Herard
ping(Doug); Thanks, Fred On 12/9/2018 5:01 PM, Fred Herard wrote: > Gentle reminder... > > Reviewed-by: Rajan Shanmugavelu > Reviewed-by: Lee Duncan > > Thanks, > Fred > > On 12/2/2018 1:47 AM, Fred Herard wrote: >> Hi Doug, >> >> Here's an updated patch with removal of the misplaced semicol

Re: [PATCH] iscsi: Capture iscsi debug messages using tracepoints

2018-12-20 Thread Douglas Gilbert
On 2018-12-20 4:45 p.m., Fred Herard wrote: ping(Doug); Thanks, Fred On 12/9/2018 5:01 PM, Fred Herard wrote: Gentle reminder... Reviewed-by: Rajan Shanmugavelu Reviewed-by: Lee Duncan Reviewed-by: Douglas Gilbert Thanks. Doug Gilbert On 12/2/2018 1:47 AM, Fred Herard wrote: Hi Dou

Re: Aw: [ANNOUNCE v2]: Broadcom (Emulex) FC Target driver - efct

2018-12-20 Thread James Smart
On 12/19/2018 10:04 AM, Sebastian Herbszt wrote: Hello James, I'd like to announce the availability of the Broadcom (Emulex) FC target driver - efct. This is the 2nd round of announcement.   In the first round, after discussion with community members, it was decided that the driver would consi

Re: scsi: ufs: Problem at init on msm8998

2018-12-20 Thread Jeffrey Hugo
On 12/20/2018 5:38 AM, Marc Gonzalez wrote: On 19/12/2018 18:27, Marc Gonzalez wrote: On 18/12/2018 17:54, Marc Gonzalez wrote: Now the fun begins, to find the tiny difference that makes/breaks everything :-) I thought I had found the problem: "clk_set_rate: ufs_unipro_core_clk_src" was not

Re: [PATCH] scsi: smartpqi: call pqi_free_interrupts() in pqi_shutdown()

2018-12-20 Thread Martin K. Petersen
Yanjiang, > We must free all irqs during shutdown, else kexec's 2nd kernel would > hang in pqi_wait_for_completion_io() as below: Applied to 4.21/scsi-queue, thanks! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH V2 0/3] smartpqi additional patches

2018-12-20 Thread Martin K. Petersen
Don, > Changes since V1: > . fix kbuild test robot and compile warnings (MIPS) > . patch smartpqi-add-ofa-support > . replace do_div MACRO with straight division Since I had already pushed the series, this had to be an incremental patch. I did it by hand and also addressed the fall t

Re: [PATCH] scsi: mpt3sas: fix memory ordering on 64bit writes

2018-12-20 Thread Martin K. Petersen
Stephan, > Thanks. Is there any way to get that backported to older kernels? > Without that little patch, HBAs relying on the mpt3sas are quite > useless on the affected powerpc64 systems. Once the commit hits Linus' tree you can send an inclusion request to linux-stable. -- Martin K. Peterse

Re: [PATCH -next] scsi: fcoe: remove set but not used variable 'port'

2018-12-20 Thread Martin K. Petersen
YueHaibing, > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/scsi/fcoe/fcoe.c: In function 'fcoe_recv_frame': > drivers/scsi/fcoe/fcoe.c:1672:20: warning: > variable 'port' set but not used [-Wunused-but-set-variable] > > drivers/scsi/fcoe/fcoe.c: In function 'fcoe_device_notificati

Re: [PATCH] scsi: mpt3sas: mpt3sas_scsih: Mark expected switch fall-through

2018-12-20 Thread Martin K. Petersen
Gustavo A., > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. Applied to 4.21/scsi-queue. Thanks! -- Martin K. Petersen Oracle Linux Engineering

Re: [RESEND PATCH v2] megaraid: fix out-of-bound array accesses

2018-12-20 Thread Martin K. Petersen
Qian, > UBSAN reported those with MegaRAID SAS-3 3108, Applied to 4.21/scsi-queue, thanks! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH 32/41] scsi: myrb: Mark expected switch fall-throughs

2018-12-20 Thread Martin K. Petersen
Gustavo A., > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. Applied to 4.21/scsi-queue. Thanks. -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH 17/41] scsi: isci: phy: Mark expected switch fall-through

2018-12-20 Thread Martin K. Petersen
Artur, > On 11/28/18 5:29 AM, Gustavo A. R. Silva wrote: >> In preparation to enabling -Wimplicit-fallthrough, mark switch cases >> where we are expecting to fall through. >> >> Addresses-Coverity-ID: 703127 ("Missing break in switch") >> Signed-off-by: Gustavo A. R. Silva > > Acked-by: Artur

Re: [PATCH] iscsi: Capture iscsi debug messages using tracepoints

2018-12-20 Thread Martin K. Petersen
Fred, > This commit enhances iscsi initiator modules to capture iscsi debug > messages using linux kernel tracepoint facility: Applied to 4.21/scsi-queue, thanks! -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH 1/3] dt-bindings: ufs: Add HI3670 UFS controller binding

2018-12-20 Thread Manivannan Sadhasivam
Hi Rob, On Thu, Dec 20, 2018 at 11:12:44AM -0600, Rob Herring wrote: > On Fri, Dec 07, 2018 at 11:21:49PM +0530, Manivannan Sadhasivam wrote: > > Add devicetree binding for HI3670 UFS controller. HI3760 SoC is very > > similar to HI3660 SoC with almost same IPs. Only major difference interms > >

Clang build error in drivers/scsi/smartpqi/smartpqi_init.c

2018-12-20 Thread Nathan Chancellor
Hi all, After commit 4fd22c13ad44 ("scsi: smartpqi: add ofa support"), an arm allyesconfig build with Clang fails with the following warning/error: drivers/scsi/smartpqi/smartpqi_init.c:7473:2: warning: comparison of distinct pointer types ('typeof ((sg_count)) *' (aka 'unsigned int *') and 'uin

Re: Clang build error in drivers/scsi/smartpqi/smartpqi_init.c

2018-12-20 Thread Nathan Chancellor
On Fri, Dec 21, 2018 at 12:11:00AM -0700, Nathan Chancellor wrote: > Hi all, > > After commit 4fd22c13ad44 ("scsi: smartpqi: add ofa support"), an arm > allyesconfig build with Clang fails with the following warning/error: > > drivers/scsi/smartpqi/smartpqi_init.c:7473:2: warning: comparison of d