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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,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 C68F8C433FE for ; Fri, 4 Dec 2020 17:12:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8659F20769 for ; Fri, 4 Dec 2020 17:12:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726173AbgLDRMS (ORCPT ); Fri, 4 Dec 2020 12:12:18 -0500 Received: from mga05.intel.com ([192.55.52.43]:16320 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726152AbgLDRMS (ORCPT ); Fri, 4 Dec 2020 12:12:18 -0500 IronPort-SDR: lYxAXEnAOKK+z93SOsH9dCYfPxiaZNEqNl2OAQjrQ5C4vWo4vuARSGg49/Wo8MzbfsT+s7jRjN Eci49+sifX2g== X-IronPort-AV: E=McAfee;i="6000,8403,9825"; a="258121283" X-IronPort-AV: E=Sophos;i="5.78,393,1599548400"; d="scan'208";a="258121283" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Dec 2020 09:10:35 -0800 IronPort-SDR: Q+kE0BgnPU7r7p9k4wspq1KsSVeTDsFrzsFNtygkwpTILxTTJpT1uFQm7NdtGN+n3JtkEsQNsh z4MleMyQy3BQ== X-IronPort-AV: E=Sophos;i="5.78,393,1599548400"; d="scan'208";a="551015315" Received: from mwalsh7-mobl1.amr.corp.intel.com ([10.212.248.252]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Dec 2020 09:10:34 -0800 Message-ID: Subject: Re: [resend/standalone PATCH v4] Add auxiliary bus support From: Ranjani Sridharan To: Greg KH , Leon Romanovsky Cc: Pierre-Louis Bossart , alsa-devel@alsa-project.org, Kiran Patil , linux-rdma@vger.kernel.org, netdev@vger.kernel.org, Martin Habets , lgirdwood@gmail.com, Fred Oh , broonie@kernel.org, jgg@nvidia.com, Dave Ertman , kuba@kernel.org, Dan Williams , Shiraz Saleem , davem@davemloft.net, linux-kernel@vger.kernel.org, Parav Pandit Date: Fri, 04 Dec 2020 09:10:34 -0800 In-Reply-To: References: <160695681289.505290.8978295443574440604.stgit@dwillia2-desk3.amr.corp.intel.com> <20201204123207.GH16543@unreal> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.36.3-0ubuntu1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org On Fri, 2020-12-04 at 13:59 +0100, Greg KH wrote: > On Fri, Dec 04, 2020 at 02:32:07PM +0200, Leon Romanovsky wrote: > > On Fri, Dec 04, 2020 at 12:42:46PM +0100, Greg KH wrote: > > > On Wed, Dec 02, 2020 at 04:54:24PM -0800, Dan Williams wrote: > > > > From: Dave Ertman > > > > > > > > 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 > > > > Co-developed-by: Ranjani Sridharan < > > > > ranjani.sridharan@linux.intel.com> > > > > Co-developed-by: Fred Oh > > > > Co-developed-by: Leon Romanovsky > > > > Signed-off-by: Kiran Patil > > > > Signed-off-by: Ranjani Sridharan < > > > > ranjani.sridharan@linux.intel.com> > > > > Signed-off-by: Fred Oh > > > > Signed-off-by: Leon Romanovsky > > > > Signed-off-by: Dave Ertman > > > > Reviewed-by: Pierre-Louis Bossart < > > > > pierre-louis.bossart@linux.intel.com> > > > > Reviewed-by: Shiraz Saleem > > > > Reviewed-by: Parav Pandit > > > > Reviewed-by: Dan Williams > > > > Reviewed-by: Martin Habets > > > > Link: > > > > https://lore.kernel.org/r/20201113161859.1775473-2-david.m.ertman@intel.com > > > > Signed-off-by: Dan Williams > > > > --- > > > > This patch is "To:" the maintainers that have a pending backlog > > > > of > > > > driver updates dependent on this facility, and "Cc:" Greg. > > > > Greg, I > > > > understand you have asked for more time to fully review this > > > > and apply > > > > it to driver-core.git, likely for v5.12, but please consider > > > > Acking it > > > > for v5.11 instead. It looks good to me and several other > > > > stakeholders. > > > > Namely, stakeholders that have pressure building up behind this > > > > facility > > > > in particular Mellanox RDMA, but also SOF, Intel Ethernet, and > > > > later on > > > > Compute Express Link. > > > > > > > > I will take the blame for the 2 months of silence that made > > > > this awkward > > > > to take through driver-core.git, but at the same time I do not > > > > want to > > > > see that communication mistake inconvenience other parties that > > > > reasonably thought this was shaping up to land in v5.11. > > > > > > > > I am willing to host this version at: > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/djbw/linux > > > > tags/auxiliary-bus-for-5.11 > > > > > > > > ...for all the independent drivers to have a common commit > > > > baseline. It > > > > is not there yet pending Greg's Ack. > > > > > > > > For example implementations incorporating this patch, see Dave > > > > Ertman's > > > > SOF series: > > > > > > > > https://lore.kernel.org/r/20201113161859.1775473-2-david.m.ertman@intel.com > > > > > > > > ...and Leon's mlx5 series: > > > > > > > > http://lore.kernel.org/r/20201026111849.1035786-1-leon@kernel.org > > > > > > > > PS: Greg I know I promised some review on newcomer patches to > > > > help with > > > > your queue, unfortunately Intel-internal review is keeping my > > > > plate > > > > full. Again, I do not want other stakeholder to be waiting on > > > > me to > > > > resolve that backlog. > > > > > > Ok, I spent some hours today playing around with this. I wrote > > > up a > > > small test-patch for this (how did anyone test this thing???). > > > > We are running all verifications tests that we have over our > > mlx5 driver. It includes devices reloads, power failures, FW > > reconfiguration to emulate different devices with and without error > > injections and many more. Up till now, no new bugs that are not > > known > > to us were found. > > Yes, sorry, I was implying that the authors here had to create _some_ > code to test this with, it would have been nice to include that as > well > here. We are collecting more and more in-kernel tests, having one > for > this code would be nice to also have so we make sure not to break any > functionality in the future. Hi Greg, Thanks for your patience with this series. The v4 version submitted by Dave included the SOF usage code to demonstrate the usage. We have run all tests for device registration, module reload, PM etc and have not observed any regressions in the SOF audio driver. Thanks, Ranjani 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=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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 F2C4AC4361A for ; Fri, 4 Dec 2020 17:11:42 +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 AAE9D22B30 for ; Fri, 4 Dec 2020 17:11:41 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AAE9D22B30 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com 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 5CDF31691; Fri, 4 Dec 2020 18:10:49 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 5CDF31691 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1607101899; bh=AoWSZ9rATBPrh2CgyRV3pM6GNUWkdHM213tiPQeGUys=; h=Subject:From:To:Date:In-Reply-To:References:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=YqZw+zQKItmtgTbDT3i7ZzcJe9n7qoXXJyLlYlB8+/AfLB7Dj+wxl/0adWz6jO92L 4LhhasHMqT4432ysLEnGS0KzQN6+qXn4OusmirW9x5JcLkZKwvJfeat2TbTTQX176a 1bPNUuLkobklzIL+0+marygOv9P5zofNyUz7+7Vw= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id D4885F800E8; Fri, 4 Dec 2020 18:10:48 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 77504F80278; Fri, 4 Dec 2020 18:10:47 +0100 (CET) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id BE288F800E8 for ; Fri, 4 Dec 2020 18:10:39 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz BE288F800E8 IronPort-SDR: iaB0kxvQSIQ2E76UWMpeYQQYQYvwZHq2X1s0x9WwQNuyhbNFirDe9e1X6haAJ3yIRU+W0TPbPi BJrAhK6OygtA== X-IronPort-AV: E=McAfee;i="6000,8403,9825"; a="160463220" X-IronPort-AV: E=Sophos;i="5.78,393,1599548400"; d="scan'208";a="160463220" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Dec 2020 09:10:35 -0800 IronPort-SDR: Q+kE0BgnPU7r7p9k4wspq1KsSVeTDsFrzsFNtygkwpTILxTTJpT1uFQm7NdtGN+n3JtkEsQNsh z4MleMyQy3BQ== X-IronPort-AV: E=Sophos;i="5.78,393,1599548400"; d="scan'208";a="551015315" Received: from mwalsh7-mobl1.amr.corp.intel.com ([10.212.248.252]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Dec 2020 09:10:34 -0800 Message-ID: Subject: Re: [resend/standalone PATCH v4] Add auxiliary bus support From: Ranjani Sridharan To: Greg KH , Leon Romanovsky Date: Fri, 04 Dec 2020 09:10:34 -0800 In-Reply-To: References: <160695681289.505290.8978295443574440604.stgit@dwillia2-desk3.amr.corp.intel.com> <20201204123207.GH16543@unreal> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.36.3-0ubuntu1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Cc: alsa-devel@alsa-project.org, Kiran Patil , linux-rdma@vger.kernel.org, netdev@vger.kernel.org, Martin Habets , Pierre-Louis Bossart , lgirdwood@gmail.com, Fred Oh , broonie@kernel.org, jgg@nvidia.com, kuba@kernel.org, Dave Ertman , Dan Williams , Shiraz Saleem , davem@davemloft.net, linux-kernel@vger.kernel.org, Parav Pandit 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 Fri, 2020-12-04 at 13:59 +0100, Greg KH wrote: > On Fri, Dec 04, 2020 at 02:32:07PM +0200, Leon Romanovsky wrote: > > On Fri, Dec 04, 2020 at 12:42:46PM +0100, Greg KH wrote: > > > On Wed, Dec 02, 2020 at 04:54:24PM -0800, Dan Williams wrote: > > > > From: Dave Ertman > > > > > > > > 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 > > > > Co-developed-by: Ranjani Sridharan < > > > > ranjani.sridharan@linux.intel.com> > > > > Co-developed-by: Fred Oh > > > > Co-developed-by: Leon Romanovsky > > > > Signed-off-by: Kiran Patil > > > > Signed-off-by: Ranjani Sridharan < > > > > ranjani.sridharan@linux.intel.com> > > > > Signed-off-by: Fred Oh > > > > Signed-off-by: Leon Romanovsky > > > > Signed-off-by: Dave Ertman > > > > Reviewed-by: Pierre-Louis Bossart < > > > > pierre-louis.bossart@linux.intel.com> > > > > Reviewed-by: Shiraz Saleem > > > > Reviewed-by: Parav Pandit > > > > Reviewed-by: Dan Williams > > > > Reviewed-by: Martin Habets > > > > Link: > > > > https://lore.kernel.org/r/20201113161859.1775473-2-david.m.ertman@intel.com > > > > Signed-off-by: Dan Williams > > > > --- > > > > This patch is "To:" the maintainers that have a pending backlog > > > > of > > > > driver updates dependent on this facility, and "Cc:" Greg. > > > > Greg, I > > > > understand you have asked for more time to fully review this > > > > and apply > > > > it to driver-core.git, likely for v5.12, but please consider > > > > Acking it > > > > for v5.11 instead. It looks good to me and several other > > > > stakeholders. > > > > Namely, stakeholders that have pressure building up behind this > > > > facility > > > > in particular Mellanox RDMA, but also SOF, Intel Ethernet, and > > > > later on > > > > Compute Express Link. > > > > > > > > I will take the blame for the 2 months of silence that made > > > > this awkward > > > > to take through driver-core.git, but at the same time I do not > > > > want to > > > > see that communication mistake inconvenience other parties that > > > > reasonably thought this was shaping up to land in v5.11. > > > > > > > > I am willing to host this version at: > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/djbw/linux > > > > tags/auxiliary-bus-for-5.11 > > > > > > > > ...for all the independent drivers to have a common commit > > > > baseline. It > > > > is not there yet pending Greg's Ack. > > > > > > > > For example implementations incorporating this patch, see Dave > > > > Ertman's > > > > SOF series: > > > > > > > > https://lore.kernel.org/r/20201113161859.1775473-2-david.m.ertman@intel.com > > > > > > > > ...and Leon's mlx5 series: > > > > > > > > http://lore.kernel.org/r/20201026111849.1035786-1-leon@kernel.org > > > > > > > > PS: Greg I know I promised some review on newcomer patches to > > > > help with > > > > your queue, unfortunately Intel-internal review is keeping my > > > > plate > > > > full. Again, I do not want other stakeholder to be waiting on > > > > me to > > > > resolve that backlog. > > > > > > Ok, I spent some hours today playing around with this. I wrote > > > up a > > > small test-patch for this (how did anyone test this thing???). > > > > We are running all verifications tests that we have over our > > mlx5 driver. It includes devices reloads, power failures, FW > > reconfiguration to emulate different devices with and without error > > injections and many more. Up till now, no new bugs that are not > > known > > to us were found. > > Yes, sorry, I was implying that the authors here had to create _some_ > code to test this with, it would have been nice to include that as > well > here. We are collecting more and more in-kernel tests, having one > for > this code would be nice to also have so we make sure not to break any > functionality in the future. Hi Greg, Thanks for your patience with this series. The v4 version submitted by Dave included the SOF usage code to demonstrate the usage. We have run all tests for device registration, module reload, PM etc and have not observed any regressions in the SOF audio driver. Thanks, Ranjani