From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ot1-x342.google.com (mail-ot1-x342.google.com [IPv6:2607:f8b0:4864:20::342]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 4EDBD21A02937 for ; Tue, 27 Nov 2018 14:34:58 -0800 (PST) Received: by mail-ot1-x342.google.com with SMTP id w25so21695288otm.13 for ; Tue, 27 Nov 2018 14:34:58 -0800 (PST) MIME-Version: 1.0 References: <154170028986.12967.2108024712555179678.stgit@ahduyck-desk1.jf.intel.com> <154170044652.12967.17419321472770956712.stgit@ahduyck-desk1.jf.intel.com> <2031cf4705d76dd4d0f722a600a6a106cce2ba41.camel@linux.intel.com> <1543350780.185366.81.camel@acm.org> <1543353746.185366.90.camel@acm.org> In-Reply-To: <1543353746.185366.90.camel@acm.org> From: Dan Williams Date: Tue, 27 Nov 2018 14:34:46 -0800 Message-ID: Subject: Re: [driver-core PATCH v6 9/9] libnvdimm: Schedule device registration on node local to the device List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: bvanassche@acm.org Cc: "Brown, Len" , linux-nvdimm , Greg KH , Linux-pm mailing list , jiangshanlai@gmail.com, Linux Kernel Mailing List , Pavel Machek , zwisler@kernel.org, Tejun Heo , alexander.h.duyck@linux.intel.com, Andrew Morton , "Rafael J. Wysocki" List-ID: On Tue, Nov 27, 2018 at 1:22 PM Bart Van Assche wrote: > > On Tue, 2018-11-27 at 12:50 -0800, Dan Williams wrote: > > Thanks Bart, so tying this back to Alex's patches, does the ordering > > problem that Alex's patches solve impact the SCSI case? I'm looking > > for something like "SCSI depends on asynchronous probing and without > > 'driver core: Establish clear order of operations for deferred probe > > and remove' probing is often needlessly serialized". I.e. does it > > suffer from the same platform problem that libnvdimm ran into where > > it's local async probing implementation was hindered by the driver > > core? > > (+Martin) > > Hi Dan, > > Patch 6/9 reduces the time needed to scan SCSI LUNs significantly. The only > way to realize that speedup is by enabling more concurrency. That's why I > think that patch 6/9 is a significant driver core improvement. Perfect, Alex, with that added to the 6/9 changelog and moving to device_private for the async state tracking you can add my Reviewed-by. _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm