Date: Fri, 12 Jun 2009 00:07:41 -0700 From: Julian Elischer <julian@elischer.org> To: Pawel Jakub Dawidek <pjd@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Marko Zec <zec@FreeBSD.org> Subject: Re: svn commit: r194012 - in head: . sys/netgraph sys/sys Message-ID: <4A31FEBD.6020601@elischer.org> In-Reply-To: <20090611190140.GE2642@garage.freebsd.pl> References: <200906111650.n5BGonnn053446@svn.freebsd.org> <20090611190140.GE2642@garage.freebsd.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
Pawel Jakub Dawidek wrote: > Are you sure Marko that you can't use sys/sys/osd.h instead of adding > yet another field to the thread structure? Netgraph is optional > component and optional components could take advantage of allocating > stuff they need dynamically. The OSD (Object-Specific Data) KPI is > designed for use by optional components - you can add your data to a > thread, you can get it when you want and OSD will call your callback > when thread dies, so you can clean up. > > Maybe you can't, but it's worth checking. > > PS. Currently OSD works for threads and jails, but it is ready to be > extended to work with other object types, eg. vnodes, ifnets, etc. > Even if you can't use it in this particular case, keep it in mind, > as it might be useful for other vimage-related stuff. > BTW Having a quick look around I don't see a man page for osd, and I'd never looked at it until today, nor even really heard of it. (except I saw jamie used it for something). if the include page is the main documentation then it need some comments.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4A31FEBD.6020601>