Re: [PATCH] lock assertion macros for 2.5.30

Joshua MacDonald (jmacd@namesys.com)
Thu, 8 Aug 2002 21:43:17 +0400


On Thu, Aug 08, 2002 at 02:31:40PM -0300, Rik van Riel wrote:
> On Thu, 8 Aug 2002, Jesse Barnes wrote:
> > On Thu, Aug 08, 2002 at 08:00:45AM +0200, Jens Axboe wrote:
>
> > > For MUST_NOT_HOLD to work, you need to take into account which processor
> > > took the lock etc.
>
> [snip]
>
> > Agreed. I'll post another patch that doesn't mess with the scsi
> > stuff. Maybe later I can put together a useful
> > 'lock-not-held-on-this-cpu' macro.
>
> You don't need to put this in a macro. This test is valid
> for ALL spinlocks in the kernel and can be done from inside
> the spin_lock() macro itself, when spinlock debugging is on.
>

This is just not true. When you make this assertion, it doesn't mean you
intend to take the lock. It could have to do with lock ordering, or it could
be testing that some lock is properly released. Why do you seem to be against
more assertions? They won't get in your way.

-josh
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/