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: Tue, 3 Apr 2018 17:42:10 +0200 Message-ID: <20180403154210.GK3313@nanopsycho> References: <1522573990-5242-1-git-send-email-si-wei.liu@oracle.com> <1522573990-5242-3-git-send-email-si-wei.liu@oracle.com> <8b589cd2-1abc-59c2-99f1-96df8174bb6b@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Si-Wei Liu , mst@redhat.com, stephen@networkplumber.org, alexander.h.duyck@intel.com, davem@davemloft.net, 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 Ahern Return-path: Received: from mail-wm0-f68.google.com ([74.125.82.68]:55027 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751696AbeDCPmM (ORCPT ); Tue, 3 Apr 2018 11:42:12 -0400 Received: by mail-wm0-f68.google.com with SMTP id r191so3916028wmg.4 for ; Tue, 03 Apr 2018 08:42:11 -0700 (PDT) Content-Disposition: inline In-Reply-To: <8b589cd2-1abc-59c2-99f1-96df8174bb6b@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: Sun, Apr 01, 2018 at 06:11:29PM CEST, dsahern@gmail.com wrote: >On 4/1/18 3:13 AM, Si-Wei Liu wrote: >> Hidden netdevice is not visible to userspace such that >> typical network utilites e.g. ip, ifconfig and et al, >> cannot sense its existence or configure it. Internally >> hidden netdev may associate with an upper level netdev >> that userspace has access to. Although userspace cannot >> manipulate the lower netdev directly, user may control >> or configure the underlying hidden device through the >> upper-level netdev. For identification purpose, the >> kobject for hidden netdev still presents in the sysfs >> hierarchy, however, no uevent message will be generated >> when the sysfs entry is created, modified or destroyed. >> >> For that end, a separate namescope needs to be carved >> out for IFF_HIDDEN netdevs. As of now netdev name that >> starts with colon i.e. ':' is invalid in userspace, >> since socket ioctls such as SIOCGIFCONF use ':' as the >> separator for ifname. The absence of namescope started >> with ':' can rightly be used as the namescope for >> the kernel-only IFF_HIDDEN netdevs. >> >> Signed-off-by: Si-Wei Liu >> --- >> include/linux/netdevice.h | 12 ++ >> include/net/net_namespace.h | 2 + >> net/core/dev.c | 281 ++++++++++++++++++++++++++++++++++++++------ >> net/core/net_namespace.c | 1 + >> 4 files changed, 263 insertions(+), 33 deletions(-) >> > >There are other use cases that want to hide a device from userspace. I What usecases do you have in mind? >would prefer a better solution than playing games with name prefixes and >one that includes an API for users to list all devices -- even ones >hidden by default. Netdevice hiding feels a bit scarry for me. This smells like a workaround for userspace issues. Why can't the netdevice be visible always and userspace would know what is it and what should it do with it? Once we start with hiding, there are other things related to that which appear. Like who can see what, levels of visibility etc... > >https://github.com/dsahern/linux/commit/48a80a00eac284e58bae04af10a5a932dd7aee00 > >https://github.com/dsahern/iproute2/commit/7563f5b26f5539960e99066e34a995d22ea908ed > >Also, why are you suggesting that the device should still be visible via >/sysfs? That leads to inconsistent views of networking state - /sys >shows a device but a link dump does not.