Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755075AbZCZAAv (ORCPT ); Wed, 25 Mar 2009 20:00:51 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752734AbZCZAAk (ORCPT ); Wed, 25 Mar 2009 20:00:40 -0400 Received: from cantor.suse.de ([195.135.220.2]:34850 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751070AbZCZAAj (ORCPT ); Wed, 25 Mar 2009 20:00:39 -0400 Date: Thu, 26 Mar 2009 01:00:36 +0100 From: Jan Kara To: Alexander Beregalov Cc: Theodore Tso , "linux-next@vger.kernel.org" , linux-ext4@vger.kernel.org, LKML , sparclinux@vger.kernel.org Subject: Re: next-20090310: ext4 hangs Message-ID: <20090326000035.GB11024@duck.suse.cz> References: <20090325151122.GA14881@atrey.karlin.mff.cuni.cz> <20090325151516.GB14881@atrey.karlin.mff.cuni.cz> <20090325152234.GN23439@duck.suse.cz> <20090325161556.GP23439@duck.suse.cz> <20090325194316.GQ23439@duck.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1001 Lines: 31 Hi, thanks for the updated disassembly. > 55bab0: 82 10 60 01 or %g1, 1, %g1 > spin_unlock(&journal->j_list_lock); > 55bab4: 40 06 4b 20 call 6ee734 <_spin_unlock> > 55bab8: c2 24 e0 28 st %g1, [ %l3 + 0x28 ] OK, so it really seems that: jinode->i_flags |= JI_COMMIT_RUNNING; spin_unlock(&journal->j_list_lock); has been compiled to ld [ %l3 + 0x28 ], %g1 or %g1, 1, %g1 call 6ee734 <_spin_unlock> st %g1, [ %l3 + 0x28 ] Which seems like a bug in the compiler or in the way implement compiler barriers in spin_unlock() on UP sparc. Or is there some sparc magic by which this is correct code? Any clever sparc guy? Honza -- Jan Kara SUSE Labs, CR -- 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/