From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 3 of 5] virtio: support unlocked queue kick Date: Thu, 3 Nov 2011 03:52:11 -0400 Message-ID: <20111103075211.GD6993@infradead.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: MichaelS.Tsirkist@redhat.com, Christoph Hellwig , virtualization , kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Rusty Russell Return-path: Received: from 173-166-109-252-newengland.hfc.comcastbusiness.net ([173.166.109.252]:55762 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755065Ab1KCHwN (ORCPT ); Thu, 3 Nov 2011 03:52:13 -0400 Content-Disposition: inline In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On Thu, Nov 03, 2011 at 06:12:51PM +1030, Rusty Russell wrote: > Based on patch by Christoph for virtio_blk speedup: Please credit it to Stefan - he also sent a pointer to his original version in reply to the previous thread. Also shouldn't virtqueue_kick have kerneldoc comments? I also notices that you documented the functions bother here and in the first patch in the headers. At least historically the kerneldoc tools didn't parse comments at declarations, but only at the function defintions. Did you check these actually get picked up?