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.
