From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [RFC PATCH 2/3] netdev: kernel-only IFF_HIDDEN netdevice Date: Wed, 4 Apr 2018 20:20:51 +0200 Message-ID: <20180404182051.GD2209@nanopsycho> References: <3bdfc39f-4935-2433-7982-9ce28c3aa166@gmail.com> <54accf73-e6cc-e03f-6a1c-34e1bbd78047@gmail.com> <20180404.133749.1802514210170809419.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dsahern@gmail.com, loseweigh@gmail.com, si-wei.liu@oracle.com, mst@redhat.com, stephen@networkplumber.org, alexander.h.duyck@intel.com, jesse.brandeburg@intel.com, kubakici@wp.pl, jasowang@redhat.com, sridhar.samudrala@intel.com, netdev@vger.kernel.org, virtualization@lists.linux-foundation.org, virtio-dev@lists.oasis-open.org To: David Miller Return-path: Received: from mail-wm0-f45.google.com ([74.125.82.45]:35441 "EHLO mail-wm0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751281AbeDDSUy (ORCPT ); Wed, 4 Apr 2018 14:20:54 -0400 Received: by mail-wm0-f45.google.com with SMTP id r82so45399734wme.0 for ; Wed, 04 Apr 2018 11:20:54 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180404.133749.1802514210170809419.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: Wed, Apr 04, 2018 at 07:37:49PM CEST, davem@davemloft.net wrote: >From: David Ahern >Date: Wed, 4 Apr 2018 11:21:54 -0600 > >> It is a netdev so there is no reason to have a separate ip command to >> inspect it. 'ip link' is the right place. > >I agree on this. > >What I really don't understand still is the use case... really. > >So there are control netdevs, what exactly is the problem with that? > >Are we not exporting enough information for applications to handle >these devices sanely? If so, then's let add that information. > >We can set netdev->type to ETH_P_LINUXCONTROL or something like that. > >Another alternative is to add an interface flag like IFF_CONTROL or >similar, and that probably is much nicer. > >Hiding the devices means that we acknowledge that applications are >currently broken with control netdevs... and we want them to stay >broken! > >That doesn't sound like a good plan to me. > >So let's fix handling of control netdevs instead of hiding them. Exactly. Don't workaround userspace issues by kernel patches.