From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261227AbVCaKMr (ORCPT ); Thu, 31 Mar 2005 05:12:47 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261226AbVCaKMj (ORCPT ); Thu, 31 Mar 2005 05:12:39 -0500 Received: from pentafluge.infradead.org ([213.146.154.40]:4067 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S261227AbVCaKMN (ORCPT ); Thu, 31 Mar 2005 05:12:13 -0500 Date: Thu, 31 Mar 2005 11:12:11 +0100 From: Christoph Hellwig To: Tejun Heo Cc: James.Bottomley@steeleye.com, axboe@suse.de, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH scsi-misc-2.6 01/13] scsi: don't use blk_insert_request() for requeueing Message-ID: <20050331101211.GB13842@infradead.org> Mail-Followup-To: Christoph Hellwig , Tejun Heo , James.Bottomley@steeleye.com, axboe@suse.de, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org References: <20050331090647.FEDC3964@htj.dyndns.org> <20050331090647.BA0001D5@htj.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050331090647.BA0001D5@htj.dyndns.org> User-Agent: Mutt/1.4.1i X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 31, 2005 at 06:07:55PM +0900, Tejun Heo wrote: > 01_scsi_no_REQ_SPECIAL_on_requeue.patch > > blk_insert_request() has 'reinsert' argument, which, when set, > turns on REQ_SPECIAL and REQ_SOFTBARRIER and requeues the > request. SCSI midlayer was the only user of this feature and > all requeued requests become special requests defeating > quiesce state. This patch makes scsi midlayer use > blk_requeue_request() for requeueing and removes 'reinsert' > feature from blk_insert_request(). > > Note: In drivers/scsi/scsi_lib.c, scsi_single_lun_run() and > scsi_run_queue() are moved upward unchanged. That lest part doesn't belong into this patch at all.