netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dawid Ciezarkiewicz <dpc@asn.pl>
To: Jay Vosburgh <fubar@us.ibm.com>
Cc: Andy Gospodarek <andy@greyhouse.net>, netdev@vger.kernel.org
Subject: Re: [RFC] wrr (weighted round-robin) bonding
Date: Fri, 20 Oct 2006 22:52:00 +0200	[thread overview]
Message-ID: <200610202252.01390.dpc@asn.pl> (raw)
In-Reply-To: <200610201953.k9KJr2Zt000305@death.nxdomain.ibm.com>

On Friday, 20 October 2006 21:53, Jay Vosburgh wrote:
> Dawid Ciezarkiewicz <dpc@asn.pl> wrote:
> 
> >On Thursday, 19 October 2006 21:04, Andy Gospodarek wrote:
> >> It would seem to me that extending an existing mode would be more
> >> desirable than adding yet another mode to worry about.  I don't even
> >> like the fact that there are as many as there are, but I understand why
> >> they are there.  
> >
> >Ack. I will probably update wrr bonding patch to replace rr mode.
> 
> 	Also, if acceptance into the mainline is your ultimate goal, the
> ioctl control interface will be a very difficult sell.  You'd want to
> look into some other control mechanism, most likely an additional sysfs
> entry.

Oh. I'm quite puzzled here. What is current policy? I'd like sysfs interfaces 
better than ioctl - they are much cleaner etc. - but I thought ioctl will be 
better here because current bonding control uses ioctl and extending it is 
much simpler.

More generally: should I use sysfs always when adding anything to kernel and 
use it even when extending older functionality?

If wrr bonding have chances for inclusion into mainline I'll do any necessary 
changes to let it meet requirements. I'd like to know if functionality from 
Andy's patch is desired to be a part of such round-robin bonding extension. I 
think I could easily integrate it with wrr.

  reply	other threads:[~2006-10-20 20:52 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-16 18:21 [RFC] wrr (weighted round-robin) bonding Dawid Ciezarkiewicz
2006-10-16 18:27 ` Dawid Ciezarkiewicz
2006-10-16 18:50   ` Jay Vosburgh
2006-10-16 19:07     ` Dawid Ciezarkiewicz
2006-10-16 21:30       ` Andy Gospodarek
2006-10-17  8:16         ` Dawid Ciezarkiewicz
2006-10-19 19:04           ` Andy Gospodarek
2006-10-20 19:41             ` Dawid Ciezarkiewicz
2006-10-20 19:53               ` Jay Vosburgh
2006-10-20 20:52                 ` Dawid Ciezarkiewicz [this message]
2006-10-20 21:35                   ` Andy Gospodarek
2006-10-20 21:55                   ` Jay Vosburgh

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=200610202252.01390.dpc@asn.pl \
    --to=dpc@asn.pl \
    --cc=andy@greyhouse.net \
    --cc=fubar@us.ibm.com \
    --cc=netdev@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).