From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: [PATCH 0/3]: blk-iopoll, a polled completion API for block devices Date: Tue, 11 Aug 2009 19:14:27 +0200 Message-ID: <20090811171427.GV12579@kernel.dk> References: <1249588685-4662-1-git-send-email-jens.axboe@oracle.com> <20090811143919.GU12579@kernel.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from brick.kernel.dk ([93.163.65.50]:37762 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753719AbZHKRO0 (ORCPT ); Tue, 11 Aug 2009 13:14:26 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Bart Van Assche Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, Eric.Moore@lsi.com, jeff@garzik.org On Tue, Aug 11 2009, Bart Van Assche wrote: > On Tue, Aug 11, 2009 at 4:39 PM, Jens Axboe wrote: > > On Tue, Aug 11 2009, Bart Van Assche wrote: > >> On Thu, Aug 6, 2009 at 9:58 PM, Jens Axboe wrote: > >> > Anyway, YMMV, I would appreciate some test results (and as usual, that > >> > even includes just saying that it boots and functions for you). If > >> > people feel adventurous, patches for other controllers will be happily > >> > queued up for testing. I may even be convinced to implement support > >> > for your controller of choice, if you have some fast storage hooked up > >> > and would like to experiment. Generally, adding support to a driver is > >> > not very hard and the two conversions included were also meant to serve > >> > as an inspiration. > >> > >> Sounds very interesting. Have you already considered patching the SRP > >> initiator ? During the SRP performance tests I ran CPU usage on the > >> initiator was more than 95% and on the target less than 10%. > > > > No I haven't, if you point me at which srp files, I can take a look. > > The relevant source files are: > include/scsi/srp.h > include/scsi/scsi_transport_srp.h > drivers/infiniband/ulp/srp/ib_srp.h > drivers/infiniband/ulp/srp/ib_srp.c > drivers/scsi/scsi_transport_srp.c > drivers/scsi/libsrp.c > drivers/scsi/scsi_transport_srp_internal.h I can find | grep too :-) Did you profile this? Where did it burn all the CPU time on the initiator side? -- Jens Axboe