From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [patch net-next v2 00/10] Add support for resource abstraction Date: Thu, 04 Jan 2018 12:17:57 -0500 (EST) Message-ID: <20180104.121757.1740770992383898707.davem@davemloft.net> References: <49c72225-6437-54d5-a046-96fff5b65ce9@cumulusnetworks.com> <20180103183636.GF2067@nanopsycho.orion> <5c387961-31c2-33a7-15ff-7bdc7a743b5f@cumulusnetworks.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: jiri@resnulli.us, arkadis@mellanox.com, netdev@vger.kernel.org, roopa@cumulusnetworks.com, mlxsw@mellanox.com, andrew@lunn.ch, vivien.didelot@savoirfairelinux.com, f.fainelli@gmail.com, michael.chan@broadcom.com, ganeshgr@chelsio.com, saeedm@mellanox.com, matanb@mellanox.com, leonro@mellanox.com, idosch@mellanox.com, jakub.kicinski@netronome.com, ast@kernel.org, daniel@iogearbox.net, simon.horman@netronome.com, pieter.jansenvanvuuren@netronome.com, john.hurley@netronome.com, alexander.h.duyck@intel.com, linville@tuxdriver.com, gospo@broadcom.com, steven.lin1@broadcom.com, yuvalm@mellanox.com, ogerlitz@mellanox.com To: dsa@cumulusnetworks.com Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:48358 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750770AbeADRSD (ORCPT ); Thu, 4 Jan 2018 12:18:03 -0500 In-Reply-To: <5c387961-31c2-33a7-15ff-7bdc7a743b5f@cumulusnetworks.com> Sender: netdev-owner@vger.kernel.org List-ID: From: David Ahern Date: Thu, 4 Jan 2018 09:58:51 -0700 > This is what I am getting at. Apparently, these resource patches for > devlink require a patched libmnl to work properly. It is wrong for > iproute2 to accept this patch and to build a devlink command that we > know does not work. That means iproute2 needs a dependency on a specific > version of libmnl - a version which does not yet exist because those > changes have not been accepted and libmnl version released. Adding that > dependency is going to inconvenience to all current users of the > iproute2 repo. +1