Date: Tue, 01 Aug 2006 10:42:35 -0700 From: Sam Leffler <sam@errno.com> To: Paolo Pisati <piso@freebsd.org> Cc: Perforce Change Reviews <perforce@freebsd.org> Subject: Re: PERFORCE change 102954 for review Message-ID: <44CF928B.7020102@errno.com> In-Reply-To: <200608011725.k71HP4ol019342@repoman.freebsd.org> References: <200608011725.k71HP4ol019342@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Paolo Pisati wrote: > http://perforce.freebsd.org/chv.cgi?CH=102954 > > Change 102954 by piso@piso_newluxor on 2006/08/01 17:24:16 > > Convert ath to use a filter+ithread handler: > > use a spinlock (inside softc) to guard against > races when accessing sc_status or the interrupt registers, > and axe all the taskqueue jobs from ath_intr(). I highly doubt this does the right thing and the spinlock is almost certain to be the wrong thing to do here. We should probably talk privately about how to restructure ath to use your stuff but I expected a very different approach. Sam
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?44CF928B.7020102>