From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 05993A0C45; Wed, 4 Aug 2021 11:50:27 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id DE32F4014F; Wed, 4 Aug 2021 11:50:26 +0200 (CEST) Received: from mail-108-mta168.mxroute.com (mail-108-mta168.mxroute.com [136.175.108.168]) by mails.dpdk.org (Postfix) with ESMTP id 577694003E for ; Wed, 4 Aug 2021 11:50:25 +0200 (CEST) Received: from filter004.mxroute.com ([149.28.56.236] filter004.mxroute.com) (Authenticated sender: mN4UYu2MZsgR) by mail-108-mta168.mxroute.com (ZoneMTA) with ESMTPSA id 17b10922cea00074ba.001 for (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES128-GCM-SHA256); Wed, 04 Aug 2021 09:50:20 +0000 X-Zone-Loop: 8577e86b5fce3aa92f5e40209b93c03c835d0aec898d X-Originating-IP: [149.28.56.236] DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=ashroe.eu; s=x; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:MIME-Version:Date: Message-ID:From:References:Cc:To:Subject:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=+AUkCEaip1vOmioasCFo1MeQECrYA1aE1MJ0vpNIEpM=; b=lPtCen0S0tjXZbQMXLmshnBbc4 Dgy6pLMnKZFLZn+pgiETJ2EDqfrZV+yHbxoQKONvqraGEpY4MYmtEPMagbwWypRQgWn6UOoWiFDUZ wDfJwBE13I4WRhOgfXFBe1/Ae0POQomNIl3PooInQfPF9pcL9kX3kmN7oRKSlrenXF2JIzTGzrA0E 5lvLaIj38hIu3aVpqzrS7WNbEu2BgRftDRgiFT0Pi4UqbM6/U32tjN7eSnx24Ii+abBvoAdZ9End9 kdgMCVEU6rGdeSzULJmWtfaCH5EzOO3SHKznKVsjGmr21Gur05imbqrrIeql8TgZn5CguTSHmWE1D 3Sn/z06Q==; To: Xueming Li Cc: dev@dpdk.org, Wang Haiyue , Thomas Monjalon , Neil Horman References: <20210623000349.631468-2-xuemingl@nvidia.com> <20210625114726.776425-2-xuemingl@nvidia.com> From: "Kinsella, Ray" Message-ID: <2d518e5f-04c9-a79c-b9b7-a193a38f2aa0@ashroe.eu> Date: Wed, 4 Aug 2021 10:50:17 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 MIME-Version: 1.0 In-Reply-To: <20210625114726.776425-2-xuemingl@nvidia.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-AuthUser: mdr@ashroe.eu X-Zone-Spam-Resolution: no action X-Zone-Spam-Status: No, score=-0.1, required=15, tests=[ARC_NA=0, FROM_HAS_DN=0, TO_DN_SOME=0, MIME_GOOD=-0.1, FROM_EQ_ENVFROM=0, MIME_TRACE=0, RCVD_COUNT_ZERO=0, RCPT_COUNT_FIVE=0, MID_RHS_MATCH_FROM=0, NEURAL_SPAM=0] Subject: Re: [dpdk-dev] [PATCH v6 2/2] bus/auxiliary: introduce auxiliary bus X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 25/06/2021 12:47, Xueming Li wrote: > Auxiliary bus [1] provides a way to split function into child-devices > representing sub-domains of functionality. Each auxiliary device > represents a part of its parent functionality. > > Auxiliary device is identified by unique device name, sysfs path: > /sys/bus/auxiliary/devices/ > > Devargs legacy syntax ofauxiliary device: > -a auxiliary:[,args...] > Devargs generic syntax of auxiliary device: > -a bus=auxiliary,name=,,/class=,,/driver=,, > > [1] kernel auxiliary bus document: > https://www.kernel.org/doc/html/latest/driver-api/auxiliary_bus.html > > Signed-off-by: Xueming Li > Cc: Wang Haiyue > Cc: Thomas Monjalon > Cc: Kinsella Ray > --- > MAINTAINERS | 5 + > doc/guides/rel_notes/release_21_08.rst | 6 + > drivers/bus/auxiliary/auxiliary_common.c | 411 ++++++++++++++++++++++ > drivers/bus/auxiliary/auxiliary_params.c | 59 ++++ > drivers/bus/auxiliary/linux/auxiliary.c | 141 ++++++++ > drivers/bus/auxiliary/meson.build | 16 + > drivers/bus/auxiliary/private.h | 74 ++++ > drivers/bus/auxiliary/rte_bus_auxiliary.h | 201 +++++++++++ > drivers/bus/auxiliary/version.map | 7 + > drivers/bus/meson.build | 1 + > 10 files changed, 921 insertions(+) > create mode 100644 drivers/bus/auxiliary/auxiliary_common.c > create mode 100644 drivers/bus/auxiliary/auxiliary_params.c > create mode 100644 drivers/bus/auxiliary/linux/auxiliary.c > create mode 100644 drivers/bus/auxiliary/meson.build > create mode 100644 drivers/bus/auxiliary/private.h > create mode 100644 drivers/bus/auxiliary/rte_bus_auxiliary.h > create mode 100644 drivers/bus/auxiliary/version.map > [SNIP] > diff --git a/drivers/bus/auxiliary/version.map b/drivers/bus/auxiliary/version.map > new file mode 100644 > index 0000000000..a52260657c > --- /dev/null > +++ b/drivers/bus/auxiliary/version.map > @@ -0,0 +1,7 @@ > +EXPERIMENTAL { > + global: > + > + # added in 21.08 > + rte_auxiliary_register; > + rte_auxiliary_unregister; > +}; Aren't these really internal? > diff --git a/drivers/bus/meson.build b/drivers/bus/meson.build > index 410058de3a..45eab5233d 100644 > --- a/drivers/bus/meson.build > +++ b/drivers/bus/meson.build > @@ -2,6 +2,7 @@ > # Copyright(c) 2017 Intel Corporation > > drivers = [ > + 'auxiliary', > 'dpaa', > 'fslmc', > 'ifpga', >