Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 06 Apr 2015 17:17:28 -0700
From:      Lawrence Stewart <lstewart@freebsd.org>
To:        Gleb Smirnoff <glebius@FreeBSD.org>
Cc:        svn-src-projects@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r281151 - projects/ifnet/sys/net
Message-ID:  <55232218.5050401@freebsd.org>
In-Reply-To: <20150406223108.GD64665@FreeBSD.org>
References:  <201504061522.t36FMXwX092973@svn.freebsd.org> <5522F214.5060903@freebsd.org> <20150406223108.GD64665@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 04/06/15 15:31, Gleb Smirnoff wrote:
> On Mon, Apr 06, 2015 at 01:52:36PM -0700, Lawrence Stewart wrote:
> L> > Author: glebius
> L> > Date: Mon Apr  6 15:22:32 2015
> L> > New Revision: 281151
> L> > URL: https://svnweb.freebsd.org/changeset/base/281151
> L> > 
> L> > Log:
> L> >   Provide software context store in struct ifnet for any facility. To add
> L> >   a new facility, one needs to grab a value in the ift_feature enum, and
> L> >   not need to modify struct ifnet.
> L> >   
> L> >   The store also has a cache to return frequently requested values,
> L> >   which resembles kobj(9) method cache.
> L> >   
> L> >   The plan is to move into the softc store almost all possible software
> L> >   contexts that hang off the struct ifnet, leaving static only the driver
> L> >   softc, AF_INET, AF_INET6 pointers, and other frequently used pointer.
> L> 
> L> It seems likely that this is partially reinventing OSD(9) (also see the
> L> way I used it for for khelp(9)). Perhaps OSD could be improved with some
> L> useful things like the cache and then it might be a suitable drop in
> L> replacement here as well?
> 
> Yes, osd(9) looks very like. For now I'd prefer to keep the simple stuff
> from r281151, to avoid introducing extra locks. Later we can estimate
> osd(9).
> 

Sounds good.

Cheers,
Lawrence



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