Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 4 Dec 2009 23:13:03 +0300
From:      Igor Sysoev <is@rambler-co.ru>
To:        freebsd-net@freebsd.org
Subject:   Re: hw.bge.forced_collapse
Message-ID:  <20091204201303.GD76992@rambler-co.ru>
In-Reply-To: <20091204195140.GH16491@michelle.cdnetworks.com>
References:  <20091204075440.GH14822@rambler-co.ru> <20091204173243.GC16491@michelle.cdnetworks.com> <20091204191114.GB76992@rambler-co.ru> <20091204195140.GH16491@michelle.cdnetworks.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Dec 04, 2009 at 11:51:40AM -0800, Pyun YongHyeon wrote:

> On Fri, Dec 04, 2009 at 10:11:14PM +0300, Igor Sysoev wrote:
> > On Fri, Dec 04, 2009 at 09:32:43AM -0800, Pyun YongHyeon wrote:
> > 
> > > On Fri, Dec 04, 2009 at 10:54:40AM +0300, Igor Sysoev wrote:
> > > > I saw commit introducing hw.bge.forced_collapse loader tunable.
> > > > Just intresting, why it can not be a sysctl ?
> > > 
> > > I didn't think the sysctl variable would be frequently changed
> > > in runtime except debugging driver so I took simple path.
> > 
> > I do not think it's worth to reboot server just to look how various
> > values affect on bandwidth and CPU usage, expecially in production.
> > 
> > As I understand the change is trivial:
> > 
> > -          CTLFLAG_RD
> > +          CTLFLAG_RW
> > 
> > since bge_forced_collapse is used atomically.
> > 
> 
> I have no problem changing it to RW but that case I may have to
> create actual sysctl node(e.g. dev.bge.0.forced_collapse) instead
> of hw.bge.forced_collapse which may affect all bge(4) controllers
> on system. Attached patch may be what you want. You can change the
> value at any time.

Thank you for the patch. Can it be installed on 8-STABLE ?


-- 
Igor Sysoev
http://sysoev.ru/en/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20091204201303.GD76992>