From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 053446ABD for ; Wed, 26 Jul 2023 04:17:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2708CC433C7; Wed, 26 Jul 2023 04:17:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1690345035; bh=wQstHp+EcrlsHWWQOCouggFaFFoZZW+KsBl9YldoDUk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=pDd51KKK+XBXqygcGzM/kZtZDiJSkS8Q+83ttVkHfEaU96MdfKLIqmsSNrVG5c4FR TUigg0jRpdDSv64rvnPNYEdpRRRNeRfIoAcgiRFCzd/r6HLrKDrzFF4Aakb3/N9JWb z6htCTBbvJP3fVNBbgxVci1muQck/ZJLX3ovfblhUH/jWixRZahe3u8K6IDs370c+f c0Eyx9ZfZiVenV5KCaVTznkvWACkpU9Mrq+dE/cwmYa8lpQPRsY3kc5OX0suInRIWK t9JhFtbYiahI5hK5zjXB0ZZmnFnEDWdzwrHt2BXuK1Aa9vQqRsV2Dm89+E7A4CewnL MWMW6stg7mrDw== Date: Tue, 25 Jul 2023 21:17:13 -0700 From: Jakub Kicinski To: Yinjun Zhang Cc: Louis Peens , David Miller , Paolo Abeni , Simon Horman , Tianyu Yuan , "netdev@vger.kernel.org" , oss-drivers Subject: Re: [PATCH net-next 00/12] nfp: add support for multi-pf configuration Message-ID: <20230725211713.0b603f13@kernel.org> In-Reply-To: References: <20230724094821.14295-1-louis.peens@corigine.com> <20230724170100.14c6493a@kernel.org> <20230725115933.29171e72@kernel.org> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 26 Jul 2023 02:00:30 +0000 Yinjun Zhang wrote: > > > serial_number UKAAMDA2000-100122190023 > > > > > serial_number UKAAMDA2000-100122190023 > > > > Since it's clearly a single ASIC shouldn't it have a single devlink > > instance? > > But there're more than one PCI device now. Isn't it universal implementation > to register a devlink for each PCI device? It's only the prevailing implementation because people are too lazy to implement things correctly, if you ask me. devlink doesn't have the ability to bind to multiple bus devices, that would need to be addressed.