From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Ahern Subject: Re: [patch net-next v2 00/10] Add support for resource abstraction Date: Wed, 3 Jan 2018 11:29:46 -0700 Message-ID: <49c72225-6437-54d5-a046-96fff5b65ce9@cumulusnetworks.com> References: <20171226112359.5313-1-jiri@resnulli.us> <977652df-a0ed-d1a5-f299-1dc433ebd337@mellanox.com> <96389ae0-e038-8a26-84ea-0cf1b9fa0a05@cumulusnetworks.com> <0f861e90-63d3-2666-ef2d-0fc91beae957@mellanox.com> <2f0f6007-64a1-0d0d-eff2-2e5ba6fdb701@cumulusnetworks.com> <20180103181744.GE2067@nanopsycho.orion> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Arkadi Sharshevsky , netdev@vger.kernel.org, roopa@cumulusnetworks.com, davem@davemloft.net, 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: Jiri Pirko Return-path: Received: from mail-pf0-f193.google.com ([209.85.192.193]:43409 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750912AbeACS3t (ORCPT ); Wed, 3 Jan 2018 13:29:49 -0500 Received: by mail-pf0-f193.google.com with SMTP id e3so1066207pfi.10 for ; Wed, 03 Jan 2018 10:29:49 -0800 (PST) In-Reply-To: <20180103181744.GE2067@nanopsycho.orion> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 1/3/18 11:17 AM, Jiri Pirko wrote: > Wed, Jan 03, 2018 at 07:14:16PM CET, dsa@cumulusnetworks.com wrote: >> On 1/3/18 11:05 AM, Arkadi Sharshevsky wrote: >>> As I stated this is a user-space bug which I fixed, and updated my repo >>> so please pull. Devlink uses mnl,and currently mnl does not support >>> extended ack. I added support for this in my local ver of libmnl: >>> >>> https://github.com/arkadis/libmnl.git >>> >>> On branch master, so you can check it out. Besides this bugs, which were >>> userspace, can please specify what are the pending problems from your >>> point of view? Thanks! >> >> devlink is in iproute2 package and it has extack support. See 'git log >> lib/libnetlink.c' > > Dave, devlink uses libmnl. > Now I remember. You wrote it independently and but needed iproute2 be a delivery vehicle. It uses none of the common infrastructure from iproute2. Could we make this more difficult .... Sometime in the next day I will jump through the hoops to get a proper devlink command.