From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chandra Seetharaman Subject: Re: [PATCH 2/3] scsi_dh: Remove the workqueue used foractivate Date: Thu, 07 May 2009 17:35:49 -0700 Message-ID: <1241742949.27327.1.camel@chandra-ubuntu> References: <20090428025230.11108.88559.sendpatchset@chandra-ubuntu> <20090428025242.11108.43024.sendpatchset@chandra-ubuntu> <4A02565B.9060105@cs.wisc.edu> Reply-To: sekharan@linux.vnet.ibm.com, device-mapper development Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4A02565B.9060105@cs.wisc.edu> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com To: device-mapper development Cc: linux-scsi@vger.kernel.org List-Id: linux-scsi@vger.kernel.org On Wed, 2009-05-06 at 22:32 -0500, Mike Christie wrote: > Chandra Seetharaman wrote: > > Since scsi_dh_activate() has become an asynchronous function, > > we do not need a workqueue for submitting scsi_dh_activate(). > > > > This patch just removes the workqueue that submits scsi_dh_activate(). > > > > Before this is merged, we need to convert the other device handlers to > use blk_execute_rq_nowait or use their own workqueue_struct right? If we want those modules also behave async, then yes. But, this set of 3 patches would work fine as I made the relevant changes to those modules so that they work properly. > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel