Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 4 Jun 2003 02:18:31 -0400 (EDT)
From:      Jeff Roberson <jroberson@chesapeake.net>
To:        Terry Lambert <tlambert2@mindspring.com>
Cc:        current@freebsd.org
Subject:   Re: umtx/libthr SMP fixes.
Message-ID:  <20030604021752.Y69975-100000@mail.chesapeake.net>
In-Reply-To: <3EDD89F0.C3FED9CE@mindspring.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 3 Jun 2003, Terry Lambert wrote:

> Scott Long wrote:
> > Bryan Liesner wrote:
> > It's very hard to imagine Jeff's patches causing a problem at the point
> > that the PR mentions.  Have you confirmed the problem in a kernel that
> > was build in a totally clean environment?
>
> The changed code is not protecting a traversal of a proc
> struct member with a proc lock in two places.  What's hard
> to imagine?
>

This is no longer the case with the latest revision.  Apparently the
panics in cam continue even after the proc lock issues were fixed.

> Even if it's weren't necessary to protect that FOREACH loop
> (it's necessary; but even if it weren't...), locking the
> proc could easily serialize a number of operations through
> that or other code which could save it from interference.
>
> -- Terry
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
>



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