From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH v2 15/16] qla2xxx: Add bulk send for atio & ctio completion paths. Date: Tue, 5 Jan 2016 22:07:10 -0800 Message-ID: <20160106060710.GA17359@infradead.org> References: <1450382231-4259-1-git-send-email-himanshu.madhani@qlogic.com> <1450382231-4259-16-git-send-email-himanshu.madhani@qlogic.com> <1450682033.25674.44.camel@haakon3.risingtidesystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: target-devel-owner@vger.kernel.org To: Quinn Tran Cc: "Nicholas A. Bellinger" , Himanshu Madhani , "target-devel@vger.kernel.org" , Giridhar Malavali , linux-scsi List-Id: linux-scsi@vger.kernel.org Hi Quinn , On Mon, Jan 04, 2016 at 06:27:58PM +0000, Quinn Tran wrote: > >in kernel/workqueue.c to be used beyond qla2xxx, or: > > QT> For every work element, the pool->lock is grabbed. Unless the lock > can be grab 1 time and the rest of the work elements piggy back on it then > it?s worth it to have a new kernel service. Can you remove these 'QT>' markers? They really mess up display in most common mailers and make every first line of your text look like you quoted it from the previous mail. This made your two mails almost unreadable for me.