Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759266AbYABUuK (ORCPT ); Wed, 2 Jan 2008 15:50:10 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753437AbYABUt6 (ORCPT ); Wed, 2 Jan 2008 15:49:58 -0500 Received: from smtp2.linux-foundation.org ([207.189.120.14]:32802 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753408AbYABUt5 (ORCPT ); Wed, 2 Jan 2008 15:49:57 -0500 Date: Wed, 2 Jan 2008 12:49:32 -0800 (PST) From: Linus Torvalds To: Christoph Hellwig cc: Matthew Wilcox , James Bottomley , Ingo Molnar , linux-kernel@vger.kernel.org, Andrew Morton Subject: Re: [patch] scsi: revert "[SCSI] Get rid of scsi_cmnd->done" In-Reply-To: <20080102201734.GA13881@infradead.org> Message-ID: References: <20080102162534.GA4041@elte.hu> <1199292381.3258.32.camel@localhost.localdomain> <20080102194030.GC11638@parisc-linux.org> <20080102201734.GA13881@infradead.org> User-Agent: Alpine 1.00 (LFD 882 2007-12-20) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1475 Lines: 34 On Wed, 2 Jan 2008, Christoph Hellwig wrote: > > I think you misunderstood Matthew here. REQ_TYPE_BLOCK_PC is indeed > used by any kind of SG_IO or similar passthrough no matter where it > originates. And exactly because REQ_TYPE_BLOCK_PC are entirely passthru > the actual driver (sd, sr or sg) is not doing the actual command > completion but it's handled in the scsi layer because it's exactly > the same no matter what driver it came on. You say that, but you then ignore that something *did* change. Maybe it's not that one suspicious test. Maybe it's somethign else. But that commit was confirmed to break something, almost two months ago. You guys seem to be in denial, and saying "it didn't change anything". And no, waiting for more reporters when one reporter has already narrowed it down to the exact (smallish) commit, is simply not good. Either you can fix it by looking at the source, or it gets reverted. I was hoping somebody in SCSI-land would actually look at the commit and try to find out what's wrong, instead of all of you apparently trying to say "nothing is wrong". I hate the excuses of "but, but, but.. it *should* work". It doesn't. Face that, *then* you can argue about why. 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/