Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759295AbXLLLkG (ORCPT ); Wed, 12 Dec 2007 06:40:06 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754357AbXLLLjv (ORCPT ); Wed, 12 Dec 2007 06:39:51 -0500 Received: from brick.kernel.dk ([87.55.233.238]:27221 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750856AbXLLLju (ORCPT ); Wed, 12 Dec 2007 06:39:50 -0500 Date: Wed, 12 Dec 2007 12:36:25 +0100 From: Jens Axboe To: Boaz Harrosh Cc: James Bottomley , Andrew Morton , Gabriel C , linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, Hannes Reinecke Subject: Re: 2.6.24-rc3-mm1 Message-ID: <20071212113625.GA22544@kernel.dk> References: <20071120204525.ff27ac98.akpm@linux-foundation.org> <47462F3C.3040700@googlemail.com> <20071122201250.f957e280.akpm@linux-foundation.org> <47466B5D.90607@googlemail.com> <20071126221509.8f437b61.akpm@linux-foundation.org> <1197390783.3812.19.camel@localhost.localdomain> <475FB316.6040804@panasas.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <475FB316.6040804@panasas.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2429 Lines: 66 On Wed, Dec 12 2007, Boaz Harrosh wrote: > On Tue, Dec 11 2007 at 18:33 +0200, James Bottomley wrote: > > On Mon, 2007-11-26 at 22:15 -0800, Andrew Morton wrote: > >> OK, thanks. I'll assume that James and Hannes have this in hand (or will > >> have, by mid-week) and I won't do anything here. > > > > Just to confirm what I think I'm going to be doing: rebasing the > > scsi-misc tree to remove this commit: > > > > commit 8655a546c83fc43f0a73416bbd126d02de7ad6c0 > > Author: Hannes Reinecke > > Date: Tue Nov 6 09:23:40 2007 +0100 > > > > [SCSI] Do not requeue requests if REQ_FAILFAST is set > > > > And its allied fix ups: > > > > commit 983289045faa96fba8841d3c51b98bb8623d9504 > > Author: James Bottomley > > Date: Sat Nov 24 19:47:25 2007 +0200 > > > > [SCSI] fix up REQ_FASTFAIL not to fail when state is QUIESCE > > > > commit 9dd15a13b332e9f5c8ee752b1ccd9b84cb5bdf17 > > Author: James Bottomley > > Date: Sat Nov 24 19:55:53 2007 +0200 > > > > [SCSI] fix domain validation to work again > > > > James > > > > The problems caused by this patch where nagging me at the back of my head > from the begging. Why should we fail on a check of FAIL_FAST in all kind > of weird places like boots, when the only place that should ever set the > flag should be one of the multi-path drivers. finally it struck me: > > It might be a bug in ll_rw_blk at blk_rq_bio_prep() there is this: > > static void blk_rq_bio_prep(struct request_queue *q, struct request *rq, > struct bio *bio) > { > /* first two bits are identical in rq->cmd_flags and bio->bi_rw */ > rq->cmd_flags |= (bio->bi_rw & 3); > ... > > Now this is no longer true and is a bug. > Second bit of bio->bi_rw defined in bio.h is: > #define BIO_RW_AHEAD 1 > but > Second bit of rq->cmd_flags is __REQ_FAILFAST > > so maybe we are getting FAILFAST in the wrong places? But that's actually on purpose, though the comment is pretty much crap. We don't want to be retrying readahead requests, those should always just be tossable. -- Jens Axboe -- 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/