From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DD33BC5519F for ; Wed, 18 Nov 2020 06:21:00 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DBA132468F for ; Wed, 18 Nov 2020 06:20:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="amsGAsmX"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="d7XNZzIy" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DBA132468F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 0EA8F1728; Wed, 18 Nov 2020 07:20:06 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 0EA8F1728 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1605680456; bh=JyKrBTD9KmRY4xEmDo2NZTmBTy+QJ2mMIRc2QKOzzDc=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=amsGAsmXoAQNneQSg4YXHVw3wcwfCSH/NqHq2X0rP3l05Ed/4XPq/YbEddxsKNsys VaGGYTJ9BMeVqvCEvAkOB9U5E3xRxR0/B4bs6wI7G1z6bP2ZMRhnWH/G7sMD/KpI5h MNCvBkez56wvV4fYv0SsRrYn8pvTVMyoc2T34eZk= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 7E4D1F80168; Wed, 18 Nov 2020 07:20:05 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 57A6BF8016C; Wed, 18 Nov 2020 07:20:03 +0100 (CET) Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id B515CF800EE for ; Wed, 18 Nov 2020 07:19:49 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz B515CF800EE Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="d7XNZzIy" Received: from localhost (83-86-74-64.cable.dynamic.v4.ziggo.nl [83.86.74.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id BA81224655; Wed, 18 Nov 2020 06:19:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1605680386; bh=JyKrBTD9KmRY4xEmDo2NZTmBTy+QJ2mMIRc2QKOzzDc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=d7XNZzIyCJHcdRwphjUy/Ndt5XCEjL6KToKLBaElJF3RoB6avy6xGyclvEDNnyyxo 10NQwAmikNMCQx6p+YuEMhQvZnWjHkeXTSXeXsB+uDhiDvDp2y9RT4nbVk0gTbTLZa Px+oY3SifEt+uys+r2S6/xakQCY6rFsc+dzOMLyM= Date: Wed, 18 Nov 2020 07:19:43 +0100 From: Greg KH To: Dan Williams Subject: Re: [PATCH v4 01/10] Add auxiliary bus support Message-ID: References: <20201113161859.1775473-1-david.m.ertman@intel.com> <20201113161859.1775473-2-david.m.ertman@intel.com> <20201117053000.GM47002@unreal> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Cc: alsa-devel@alsa-project.org, Parav Pandit , Takashi Iwai , Netdev , Ranjani Sridharan , Pierre-Louis Bossart , Fred Oh , linux-rdma , Doug Ledford , Mark Brown , "Saleem, Shiraz" , Jason Gunthorpe , Jakub Kicinski , Dave Ertman , Leon Romanovsky , David Miller , Linux Kernel Mailing List , "Patil, Kiran" X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On Tue, Nov 17, 2020 at 01:04:56PM -0800, Dan Williams wrote: > On Mon, Nov 16, 2020 at 11:02 PM Greg KH wrote: > > > > On Tue, Nov 17, 2020 at 07:30:00AM +0200, Leon Romanovsky wrote: > > > On Fri, Nov 13, 2020 at 08:18:50AM -0800, Dave Ertman wrote: > > > > Add support for the Auxiliary Bus, auxiliary_device and auxiliary_driver. > > > > It enables drivers to create an auxiliary_device and bind an > > > > auxiliary_driver to it. > > > > > > > > The bus supports probe/remove shutdown and suspend/resume callbacks. > > > > Each auxiliary_device has a unique string based id; driver binds to > > > > an auxiliary_device based on this id through the bus. > > > > > > > > Co-developed-by: Kiran Patil > > > > Signed-off-by: Kiran Patil > > > > Co-developed-by: Ranjani Sridharan > > > > Signed-off-by: Ranjani Sridharan > > > > Co-developed-by: Fred Oh > > > > Signed-off-by: Fred Oh > > > > Co-developed-by: Leon Romanovsky > > > > Signed-off-by: Leon Romanovsky > > > > Reviewed-by: Pierre-Louis Bossart > > > > Reviewed-by: Shiraz Saleem > > > > Reviewed-by: Parav Pandit > > > > Reviewed-by: Dan Williams > > > > Signed-off-by: Dave Ertman > > > > --- > > > > > > Greg, > > > > > > This horse was beaten to death, can we please progress with this patch? > > > Create special topic branch or ack so I'll prepare this branch. > > > > > > We are in -rc4 now and we (Mellanox) can't hold our submissions anymore. > > > My mlx5_core probe patches [1] were too intrusive and they are ready to > > > be merged, Parav's patches got positive review as well [2] and will be > > > taken next. > > > > > > We delayed and have in our internal queues the patches for VDPA, eswitch > > > and followup for mlx5_core probe rework, but trapped due to this AUX bus > > > patch. > > > > There are no deadlines for kernel patches here, sorry. Give me some > > time to properly review this, core kernel changes should not be rushed. > > > > Also, if you really want to blame someone for the delay, look at the > > patch submitters not the reviewers, as they are the ones that took a > > very long time with this over the lifecycle of this patchset, not me. I > > have provided many "instant" reviews of this patchset, and then months > > went by between updates from them. > > Please stop this finger pointing. It was already noted that the team, > out of abundance of caution / deference to the process, decided not to > push the patches while I was out on family leave. It's cruel to hold > that against them, and if anyone is to blame it's me for not > clarifying it was ok to proceed while I was out. I'm not blaming anyone, I'm just getting pissed when people are insisting that I do "quick reviews" for this patchset, which has been happening by different people since the very beginning of this whole feature, so I am trying to explain where others should be pointing their frustration at instead of me if they really want to do such a thing (hint, they shouldn't, but I wasn't explicit about that, sorry). Combine this with the long delays between my reviews and a new patchset submission, and on my end it's an extremely frustrating situation, which frankly, makes me want to review this thing even less and less as I know it's not going to be a fun or easy time when I do so. Everyone needs to remember that there are no deadlines here, and the people involved all have other things to work on at the same time, and that there are a lot of different subsystems and moving parts all involved. So someone is going to get grumpy about it, and right now, it seems to be me. I know I need to review this, and complaining that I haven't done so within 3 days of sending an updated patch set is not helping anyone. I'm going to try to carve out some time this week to review this properly. Hopefully there's no other major security "scares" popping up like there was the past few weeks to divert me from this... thanks, greg k-h