Messages in this thread | | | Subject | Re: [patch] add private data to notifier_block | From | Kristen Accardi <> | Date | Thu, 23 Mar 2006 10:01:29 -0800 |
| |
On Thu, 2006-03-23 at 05:06 -0500, Jes Sorensen wrote: > >>>>> "Kristen" == Kristen Accardi <kristen.c.accardi@intel.com> writes: > > Kristen> On Wed, 2006-03-22 at 14:04 -0500, Alan Stern wrote: > >> I still think this isn't really needed. The same effect can be > >> accomplished by embedding a notifier_block struct within a larger > >> structure that also contains the data pointer. > > Kristen> I thought of this, but felt it would make for less easy to > Kristen> read code. But, that's just my personal style. > > I'd have to vote for Alan's side here. Thats why we have the > containerof() stuff. It also means you can embed more than just a > pointer with the notifier block and it will generate more efficient > code when doing so. > > Cheers, > Jes
Ok, I bow to peer pressure and will do this your way. :) You can ignore this patch.
Thanks, Kristen - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |