Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760555AbXIMPne (ORCPT ); Thu, 13 Sep 2007 11:43:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757637AbXIMPnZ (ORCPT ); Thu, 13 Sep 2007 11:43:25 -0400 Received: from smtp2.linux-foundation.org ([207.189.120.14]:34860 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756751AbXIMPnY (ORCPT ); Thu, 13 Sep 2007 11:43:24 -0400 Date: Thu, 13 Sep 2007 08:42:29 -0700 (PDT) From: Linus Torvalds To: Jens Axboe cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, orgis@agnld.uni-potsdam.de, arekm@maven.pl, ed.lin@promise.com, Andrew Morton , James.Bottomley@SteelEye.com Subject: Re: [PATCH] Fix race with shared tag queue maps In-Reply-To: <20070913152222.GS25592@kernel.dk> Message-ID: References: <20070913122652.GK25592@kernel.dk> <20070913152222.GS25592@kernel.dk> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 902 Lines: 26 On Thu, 13 Sep 2007, Jens Axboe wrote: > > My bad, I think I added the smp_mb__before_clear_bit() when it was > __test_and_set_bit() like in the first hunk. Ahh, that wouldn't work at all. The "__test_and_set_bit()" thing isn't atomic at all, and no amount of memory barriers around it would help (you'd need to use real locking, but at that point the memory barriers are pointless anyway). > + /* > + * Ensure ordering between ->tag_index[tag] clear and tag clear > + */ > + smp_mb__after_clear_bit(); You still left this one. But never mind - I already edited your original patch and it's in my tree with both of those things removed. Linus - 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/