From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v5 0/7] virtio ring layout optimization and simple rx/tx processing Date: Tue, 27 Oct 2015 10:17:23 +0000 Message-ID: <20151027101723.GB9176@bricha3-MOBL3> References: <1443537953-23917-1-git-send-email-huawei.xie@intel.com> <1445787304-18267-1-git-send-email-huawei.xie@intel.com> <20151027021501.GA3115@yliu-dev.sh.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "dev@dpdk.org" To: Yuanhan Liu Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 66B41374E for ; Tue, 27 Oct 2015 11:17:27 +0100 (CET) Content-Disposition: inline In-Reply-To: <20151027021501.GA3115@yliu-dev.sh.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Oct 27, 2015 at 10:15:01AM +0800, Yuanhan Liu wrote: > On Tue, Oct 27, 2015 at 01:44:09AM +0000, Tan, Jianfeng wrote: > > > > > > > -----Original Message----- > > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Huawei Xie > > > Sent: Sunday, October 25, 2015 11:35 PM > > > To: dev@dpdk.org > > > Subject: [dpdk-dev] [PATCH v5 0/7] virtio ring layout optimization and simple > > > rx/tx processing > > > > > > Changes in v5: > > > - Call __rte_pktmbuf_prefree_seg to check refcnt when free mbufs > > > > > > Changes in v4: > > > - Fix the error in virtio tx ring layout ascii chart in the commit message > > > - Move virtio_xmit_cleanup ahead to free descriptors earlier > > > - Test merge-able feature when select simple rx/tx functions > > [...] > > > > > Acked-by Jianfeng Tan > > Jianfeng, > > I often see a reply like this, just dropping an ACK at the end of > long email, and no more, which takes me (as well as others) some > time to scroll it many times to the bottom till see that. > > TBH, it's always a bit frustrating that, after the scroll effort, > I just see such a reply that could have been put on the top of > the email so that I can see it with a glimpse only. > > So, top reply would be good for this case, or you could reply like > what I did, removing other context to make your reply fit in one > screen. > > --yliu +1 When ack'ing patches, please place the ack on the line underneath the signoff and delete the rest of the email below, as it's unneeded. /Bruce