On Tue, Apr 22, 2008 at 09:37:57AM -0400, Lytvyn, Oleksandr (IT) wrote:
> That's an interesting analysis. Hope we can have a fix soon.
I forgot to mention that you can work around the issue by replacing
curlwpsinfo->pr_stype with:
(((this->tmp = curthread->t_sobj_ops) != NULL) ? this->tmp->sobj_type : 0)
> It also makes me wonder: how many races of that kind are there? And is
> it possible to have a comprehensive fix or approach to handle stuff like
> this?
There may be other problems like this, but the same fix can apply to all of
them. If we go with a compiler change that will cause the DIF to only load
the member once in a given probe firing, then that will clean up similarly
affected situations. If we create a language construct to make a new local
scope, then we'll need to vet our translators and fix them individually.
Adam
--
Adam Leventhal, Fishworks http://blogs.sun.com/ahl
_______________________________________________
dtrace-discuss mailing list
dtrace-discuss@opensolaris.org
