Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Aug 2001 13:39:33 -0700 (PDT)
From:      John Baldwin <jhb@FreeBSD.org>
To:        Michael Lucas <mwlucas@blackhelicopters.org>
Cc:        current@FreeBSD.org
Subject:   Re: devfs and Vinum (was: any -current && vinum problems?)
Message-ID:  <XFMail.010816133933.jhb@FreeBSD.org>
In-Reply-To: <20010816163617.A52310@blackhelicopters.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On 16-Aug-01 Michael Lucas wrote:
> [cc's trimmed]
> 
> John,
> 
> Thanks for the suggestion, I appreciate it.  I did as you suggested
> (diff below).
> 
> It paniced again, but this time savecore said "dump time is unreasonable."
> 
> The short panic message was:
> 
> panicstr: bremfree: bp 0xcc2a1ae4 not locked
> 
> Looks like the same thing to me, sorry.  Any other suggestions?
> 
> magpire/sys/kern;diff subr_witness.c subr_witness.c-dist 
> 392a393
>>       mtx_lock(&all_mtx);
> 395d395
> <       mtx_lock(&all_mtx);
> magpire/sys/kern;diff -c subr_witness.c subr_witness.c-dist
> *** subr_witness.c      Thu Aug 16 16:16:06 2001
> --- subr_witness.c-dist Thu Aug 16 16:15:20 2001
> ***************
> *** 390,398 ****
>                 mtx_unlock_spin(&w_mtx);
>         }
>   
>         lock_cur_cnt--;
>         STAILQ_REMOVE(&all_locks, lock, lock_object, lo_list);
> -       mtx_lock(&all_mtx);
>         lock->lo_flags &= ~LO_INITIALIZED;
>         mtx_unlock(&all_mtx);
>   }
> --- 390,398 ----
>                 mtx_unlock_spin(&w_mtx);
>         }
>   
> +       mtx_lock(&all_mtx);
>         lock_cur_cnt--;
>         STAILQ_REMOVE(&all_locks, lock, lock_object, lo_list);
>         lock->lo_flags &= ~LO_INITIALIZED;
>         mtx_unlock(&all_mtx);
>   }
> magpire/sys/kern;

No, no. :)  Move the mtx_lock of all_mtx _up_ above the earlier check against
LO_INITALIZED.  The one that does something liek this:

        if ((lock->lo_flags & LO_INITIALIZED) == 0)
                panic();

-- 

John Baldwin <jhb@FreeBSD.org> -- http://www.FreeBSD.org/~jhb/
PGP Key: http://www.baldwin.cx/~john/pgpkey.asc
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.010816133933.jhb>