All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: "David S. Miller" <davem@davemloft.net>,
	Network Development <netdev@vger.kernel.org>,
	andrew@lunn.ch, Guenter Roeck <linux@roeck-us.net>,
	Jiri Pirko <jiri@resnulli.us>,
	sfeldma@gmail.com
Subject: Re: [PATCH RFC 0/5] net: L2 only interfaces
Date: Wed, 26 Aug 2015 10:56:09 -0700	[thread overview]
Message-ID: <BC7F5FFB-65E3-47A6-BB52-658DA32E1D48@holtmann.org> (raw)
In-Reply-To: <55DDF954.1000903@gmail.com>

Hi Florian,

>>>> This patch series implements a L2 only interface concept which
>>>> basically denies any kind of IP address configuration on these
>>>> interfaces, but still allows them to be used as configuration
>>>> end-points to keep using ethtool and friends.
>>>> 
>>>> A cleaner approach might be to finally come up with the concept of
>>>> net_port which a net_device would be a superset of, but this still
>>>> raises tons of questions as to whether we should be modifying
>>>> userland tools to be able to configure/query these
>>>> interfaces. During all the switch talks/discussions last year, it
>>>> seemed to me like th L2-only interface is closest we have to a
>>>> "network port".
>>>> 
>>>> Comments, flames, flying tomatoes welcome!
>>> 
>>> Interesting, indeed.
>>> 
>>> Do you plan to extend this to defining a more minimal network device
>>> sub-type as well?
>>> 
>>> Then we can pass "net_device_common" or whatever around as a common
>>> base type of actual net device "implementations".
>>> 
>>> Or is you main goal just getting the L2-only semantic?
>> 
>> the other end of this could be also an IP only net_device where we do not have ethtool semantics.
>> 
>> We do have a need for a IPv6 only net_device when utilizing ARPHRD_6LOWPAN for 802.15.4 and Bluetooth LE. Skipping in_dev initialization there might be an interesting step towards that. Not sure how much entangled in_dev and in6_dev still are. If it works for IFF_L2_ONLY, it might work also in the other direction.
> 
> Just out of curiosity, is the aim for IPv6 only net_device to be denying
> any kind of IPv4 configuration/tools, or is it for performance purposes?

when you have 6LoWPAN, then it would be actually good to forbid IPv4 configuration on these interface since they have no mapping whatsoever. Eventually it might allow us to decrease the size of the network stack for embedded sensor devices.

> The IFF_L2_ONLY flag would probably need to mean something like
> (IFF_NO_IPV4 | IFF_NO_IPV6) such that you could decide which one of the
> two IP stacks you want to use, or none.

I think IFF_NO_IPV4 and IFF_NO_IPV6 instead of IFF_L2_ONLY sounds like a good idea.

Regards

Marcel

  reply	other threads:[~2015-08-26 17:56 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-25 22:50 [PATCH RFC 0/5] net: L2 only interfaces Florian Fainelli
2015-08-25 22:50 ` [PATCH RFC 1/5] net: add IFF_L2_ONLY flag Florian Fainelli
2015-08-25 22:50 ` [PATCH RFC 2/5] net: ipv4: Skip in_dev initialization for IFF_L2_ONLY interfaces Florian Fainelli
2015-08-25 22:50 ` [PATCH RFC 3/5] net: ipv6: Skip in6_dev " Florian Fainelli
2015-08-25 22:50 ` [PATCH RFC 4/5] net: dsa: Flag slave network devices with IFF_L2_ONLY Florian Fainelli
2015-08-25 22:50 ` [PATCH RFC 5/5] net: dsa: bcm_sf2: Allow disabling tagging protocol Florian Fainelli
2015-08-26  0:09   ` David Miller
2015-08-25 23:20 ` [PATCH RFC 0/5] net: L2 only interfaces Alexei Starovoitov
2015-08-25 23:24   ` Florian Fainelli
2015-08-25 23:33   ` David Ahern
2015-09-01 17:07   ` Vivien Didelot
2015-08-25 23:24 ` Stephen Hemminger
2015-08-25 23:23   ` Florian Fainelli
2015-08-25 23:44 ` Sowmini Varadhan
2015-08-25 23:52   ` David Ahern
2015-08-26  0:05     ` Sowmini Varadhan
2015-08-26  0:12 ` David Miller
2015-08-26  4:24   ` Marcel Holtmann
2015-08-26 17:37     ` Florian Fainelli
2015-08-26 17:56       ` Marcel Holtmann [this message]
2015-08-26 17:32   ` Florian Fainelli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BC7F5FFB-65E3-47A6-BB52-658DA32E1D48@holtmann.org \
    --to=marcel@holtmann.org \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=f.fainelli@gmail.com \
    --cc=jiri@resnulli.us \
    --cc=linux@roeck-us.net \
    --cc=netdev@vger.kernel.org \
    --cc=sfeldma@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.