From mboxrd@z Thu Jan  1 00:00:00 1970
From: Zach Brown <zach.brown@oracle.com>
Subject: Re: kernel BUG at extent_map.c:275!
Date: Tue, 22 Jul 2008 10:42:31 -0700
Message-ID: <48861C07.3040103@oracle.com>
References: <1216697528.18980.34.camel@shinybook.infradead.org>	 <1216697801.18980.40.camel@shinybook.infradead.org>	 <1216722065.6932.125.camel@think.oraclecorp.com>	 <1216737343.18980.98.camel@shinybook.infradead.org>	 <1216746235.6932.129.camel@think.oraclecorp.com> <1216748498.3019.9.camel@shinybook.infradead.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Cc: Chris Mason <chris.mason@oracle.com>, linux-btrfs@vger.kernel.org
To: David Woodhouse <dwmw2@infradead.org>
Return-path: <linux-btrfs-owner@vger.kernel.org>
In-Reply-To: <1216748498.3019.9.camel@shinybook.infradead.org>
List-ID: <linux-btrfs.vger.kernel.org>

David Woodhouse wrote:
> On Tue, 2008-07-22 at 13:03 -0400, Chris Mason wrote:
>> Well, the test is there to make sure the caller is doing the right
>> thing.  Before we remove it, I'd like to understand why it is failing.
> 
> Because this is a uniprocessor kernel. So spin_lock() and spin_unlock()
> both do absolutely nothing, and spin_trylock() _always_ returns 1.

How about using assert_spin_locked()?

- z