All of lore.kernel.org
 help / color / mirror / Atom feed
From: John Fastabend <john.r.fastabend@intel.com>
To: David Miller <davem@davemloft.net>, or.gerlitz@gmail.com
Cc: roland@kernel.org, netdev@vger.kernel.org, ali@mellanox.com,
	sean.hefty@intel.com, shlomop@mellanox.com,
	erezsh@mellanox.co.il
Subject: Re: [PATCH net-next V1 1/9] IB/ipoib: Add support for clones / multiple childs on the same partition
Date: Wed, 18 Jul 2012 15:11:16 -0700	[thread overview]
Message-ID: <50073484.9070501@intel.com> (raw)
In-Reply-To: <20120718.143608.2101579052587289420.davem@davemloft.net>

On 7/18/2012 2:36 PM, David Miller wrote:
> From: Or Gerlitz <or.gerlitz@gmail.com>
> Date: Thu, 19 Jul 2012 00:24:58 +0300
>
>> On Wed, Jul 18, 2012 at 9:38 PM, David Miller <davem@davemloft.net> wrote:
>>> From: Or Gerlitz <ogerlitz@mellanox.com>
>>
>>>> All sorts of childs are still created/deleted through sysfs, in a
>>>> similar manner to the way legacy child interfaces are.
>>
>>> Network device instantiation of this type is the domain of
>>> rtnl_link_ops rather than ugly sysfs interfaces.
>>
>> Didn't add any **new** sysfs interfaces in this patch. The IPoIB sysfs
>> entries to create child devices are there from IPoIB's day one, and
>> we're only extending them a tiny bit.
>
> That's extremely unfortunate, having private ways of instantiating
> networking devices leads to an extremely poor user experience.
>
> Would you like to have to train every single user in the case
> where each and every driver author makes his own unique way
> of configuring his hardware?
> --

Or,

I've got a rough patch to use rtnl_link_ops to add what we've been
calling 'virtual machine device queues' or VMDq. This looks a lot
like macvlan with offloaded switching and I believe similar to your
child case above.

Also what is a "pkey"

I'll post it as a use at your own risk shortly although this week I'm
short on time so maybe next week I can get something more "real" out.
Been stealing cycles between other work things today.

If you want to do complete it more power to you.

.John

  reply	other threads:[~2012-07-18 22:11 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-18 10:59 [PATCH net-next V1 0/9] Add Ethernet IPoIB driver Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 1/9] IB/ipoib: Add support for clones / multiple childs on the same partition Or Gerlitz
2012-07-18 18:38   ` David Miller
2012-07-18 21:24     ` Or Gerlitz
2012-07-18 21:36       ` David Miller
2012-07-18 22:11         ` John Fastabend [this message]
2012-07-19  8:11           ` Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 2/9] include/linux: Add private flags for IPoIB interfaces Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 3/9] IB/ipoib: Add support for acting as VIF Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 4/9] net/eipoib: Add private header file Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 5/9] net/eipoib: Add ethtool file support Or Gerlitz
2012-07-18 18:37   ` Ben Hutchings
2012-07-19 15:55     ` Or Gerlitz
2012-07-18 10:59 ` [PATCH net-next V1 6/9] net/eipoib: Add sysfs support Or Gerlitz
2012-07-23 12:55   ` Or Gerlitz
2012-07-18 11:00 ` [PATCH net-next V1 7/9] net/eipoib: Add main driver functionality Or Gerlitz
2012-07-19 13:49   ` Ben Hutchings
2012-07-19 15:46     ` Or Gerlitz
2012-07-19 16:16       ` Ben Hutchings
2012-07-19 16:21         ` Or Gerlitz
2012-07-18 11:00 ` [PATCH net-next V1 8/9] net/eipoib: Add Makefile, Kconfig and MAINTAINERS entries Or Gerlitz
2012-07-18 11:00 ` [PATCH net-next V1 9/9] IB/ipoib: Add support for transmission of skbs w.o dst/neighbour Or Gerlitz

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=50073484.9070501@intel.com \
    --to=john.r.fastabend@intel.com \
    --cc=ali@mellanox.com \
    --cc=davem@davemloft.net \
    --cc=erezsh@mellanox.co.il \
    --cc=netdev@vger.kernel.org \
    --cc=or.gerlitz@gmail.com \
    --cc=roland@kernel.org \
    --cc=sean.hefty@intel.com \
    --cc=shlomop@mellanox.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.