[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: ext2fs.static: thread-cancel.c:55: hurd_thread_cancel: Assertion '!
From: |
Samuel Thibault |
Subject: |
Re: ext2fs.static: thread-cancel.c:55: hurd_thread_cancel: Assertion '! __spin_lock_locked (&ss->critical_section_lock)' failed |
Date: |
Sat, 5 Aug 2006 16:24:35 +0200 |
User-agent: |
Mutt/1.5.9i-nntp |
Samuel Thibault, le Sat 05 Aug 2006 02:45:34 +0200, a écrit :
> Samuel Thibault, le Sat 05 Aug 2006 02:27:35 +0200, a écrit :
> > assert (! __spin_lock_locked (&ss->critical_section_lock));
> > __spin_lock (&ss->critical_section_lock);
> >
> > Why should one check that a spinlock is free before taking it?? Here the
> > purpose is to cancel _another_ thread, so it _can_ happen that the lock
> > be taken any time...
> >
> > Shouldn't at least these assertion just be dropped?
>
> Maybe not the ones from other files, which are not trivial, but at least
> this one (thread-cancel), which was reported to actually happen, and
> which to my mind is really arguable.
I got it again this afternoon...
Samuel