David Malone <[EMAIL PROTECTED]> writes:
> On Mon, May 21, 2001 at 01:44:16AM -0700, Dima Dorfman wrote:
> 
> > exit1 calls shmexit with vm_mtx held on line 228 of kern_exit.c
> > (rev. 1.127).  Actually, shmexit_myhook should always be called with
> > vm_mtx held, so shm_delete_mapping can't assume it isn't held.
> 
> The following seems to work. It's basically your patch, but it
> removes the patch which was originally committed, adds an extra
> assert, expands on one comment and grabs a mutex in one place
> it seemed to be needed after removing the others.

Great!  I can confirm that this works for me.  Alfred, any objections
to me or David committing this?

> +     /* shm_delete_mappings requires this */
> +     mtx_assert(&vm_mtx, MA_OWNED);

That should be 'shm_delete_mapping'; no 's'.  That's my fault.

Thanks,

                                        Dima Dorfman
                                        [EMAIL PROTECTED]

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to