Re: [PATCH 05/28] IB/srp: remove old request_fn_active check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2018-10-26 at 08:32 -0600, Jens Axboe wrote:
+AD4 On 10/26/18 1:08 AM, Hannes Reinecke wrote:
+AD4 +AD4 On 10/25/18 11:10 PM, Jens Axboe wrote:
+AD4 +AD4 +AD4 This check is only viable for non scsi-mq. Since that is going away,
+AD4 +AD4 +AD4 kill this legacy check.
+AD4 +AD4 +AD4 
+AD4 +AD4 +AD4 Cc: Bart Van Assche +ADw-bvanassche+AEA-acm.org+AD4
+AD4 +AD4 +AD4 Cc: Parav Pandit +ADw-parav+AEA-mellanox.com+AD4
+AD4 +AD4 +AD4 Cc: linux-scsi+AEA-vger.kernel.org
+AD4 +AD4 +AD4 Signed-off-by: Jens Axboe +ADw-axboe+AEA-kernel.dk+AD4
+AD4 +AD4 +AD4 ---
+AD4 +AD4 +AD4   drivers/infiniband/ulp/srp/ib+AF8-srp.c +AHw 7 -------
+AD4 +AD4 +AD4   1 file changed, 7 deletions(-)
+AD4 +AD4 +AD4 
+AD4 +AD4 +AD4 diff --git a/drivers/infiniband/ulp/srp/ib+AF8-srp.c b/drivers/infiniband/ulp/srp/ib+AF8-srp.c
+AD4 +AD4 +AD4 index 0b34e909505f..5a79444c2f3c 100644
+AD4 +AD4 +AD4 --- a/drivers/infiniband/ulp/srp/ib+AF8-srp.c
+AD4 +AD4 +AD4 +-+-+- b/drivers/infiniband/ulp/srp/ib+AF8-srp.c
+AD4 +AD4 +AD4 +AEAAQA -1334,13 +-1334,6 +AEAAQA static void srp+AF8-terminate+AF8-io(struct srp+AF8-rport +ACo-rport)
+AD4 +AD4 +AD4   	struct scsi+AF8-device +ACo-sdev+ADs
+AD4 +AD4 +AD4   	int i, j+ADs
+AD4 +AD4 +AD4   
+AD4 +AD4 +AD4 -	/+ACo
+AD4 +AD4 +AD4 -	 +ACo Invoking srp+AF8-terminate+AF8-io() while srp+AF8-queuecommand() is running
+AD4 +AD4 +AD4 -	 +ACo is not safe. Hence the warning statement below.
+AD4 +AD4 +AD4 -	 +ACo-/
+AD4 +AD4 +AD4 -	shost+AF8-for+AF8-each+AF8-device(sdev, shost)
+AD4 +AD4 +AD4 -		WARN+AF8-ON+AF8-ONCE(sdev-+AD4-request+AF8-queue-+AD4-request+AF8-fn+AF8-active)+ADs
+AD4 +AD4 +AD4 -
+AD4 +AD4 +AD4   	for (i +AD0 0+ADs i +ADw target-+AD4-ch+AF8-count+ADs i+-+-) +AHs
+AD4 +AD4 +AD4   		ch +AD0 +ACY-target-+AD4-ch+AFs-i+AF0AOw
+AD4 +AD4 +AD4   
+AD4 +AD4 +AD4 
+AD4 +AD4 
+AD4 +AD4 I +AF8-think+AF8 I've already killed that one+ADs please do check the latest tree 
+AD4 +AD4 from Doug Ledford.
+AD4 
+AD4 I don't know which tree is Doug's, but as long as it's queued up
+AD4 for mainline, that's all I care about. I'm still rebasing this
+AD4 branch, so it'll just get dropped when it shows up in mainline.

Doug and Jason share a git repository. I think this is the git repository they
ask Linus to pull from: git://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git.
Although Doug had reported to have queued the above patch I haven't found that patch
in the RDMA git repository.

Bart.



[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux