[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110115164507.GE1240@cormyr>
Date: Sat, 15 Jan 2011 18:45:07 +0200
From: Vlad Dogaru <ddvlad@...edu.org>
To: Stephen Hemminger <shemminger@...tta.com>
Cc: netdev@...r.kernel.org, jamal <hadi@...erus.ca>,
Octavian Purdila <opurdila@...acom.com>
Subject: Re: [PATCH v3 0/2] net: add device groups
On Fri, Jan 14, 2011 at 08:08:10AM -0800, Stephen Hemminger wrote:
> On Fri, 14 Jan 2011 11:38:29 +0200
> Vlad Dogaru <ddvlad@...edu.org> wrote:
>
> > This patchset implements network device grouping and simple manipulation
> > of groups. Netlink has been updated to provide group information and
> > means of applying changes to members of a specific group via a single
> > message.
> >
> > The patchset has a corresponding one for iproute2, which implements the
> > new functionality in userspace.
>
> What about a read/write sysfs interface as well?
> /sys/class/net/eth0/devgroup
Oops, I overlooked that, probably because I've been testing in a virtual
machine where the kernel has sysfs turned off -- less waiting for simple
tests.
> Not sure if numeric devgroup is best choice. Since this is more of
> a human interface parameter maybe it should be a string? Or have
> a translation in the utilities /etc/iproute2/devgroup?
I didn't know about /etc/iproute2. I could add the devgroup file as you
mention, seems like a good choice for userspace.
I'll update the patches with these, but it will probably take a few days,
I have a few exams next week :)
Thanks for the suggestions,
Vlad
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists