From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751942AbcFNL5H (ORCPT ); Tue, 14 Jun 2016 07:57:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56478 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751035AbcFNL5E (ORCPT ); Tue, 14 Jun 2016 07:57:04 -0400 Date: Tue, 14 Jun 2016 13:56:59 +0200 From: Jesper Dangaard Brouer To: "Michael S. Tsirkin" Cc: linux-kernel@vger.kernel.org, Jason Wang , Eric Dumazet , davem@davemloft.net, netdev@vger.kernel.org, Steven Rostedt , kvm@vger.kernel.org, brouer@redhat.com Subject: Re: [PATCH v8 0/5] skb_array: array based FIFO for skbs Message-ID: <20160614135659.0cb9f2ff@redhat.com> In-Reply-To: <20160613235426-mutt-send-email-mst@redhat.com> References: <20160613235426-mutt-send-email-mst@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Tue, 14 Jun 2016 11:57:04 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 13 Jun 2016 23:54:26 +0300 "Michael S. Tsirkin" wrote: > > Jesper, Jason, I know that both of you tested this, > please post Tested-by tags for whatever was tested. Yes, I tested this via "external" kernel modules, not your framework. I have verified that this V8 patchset is equivalent, to what I have in: https://github.com/netoptimizer/prototype-kernel/ (at 81ca09c2a945) > changes since v7 > fix typos noticed by Jesper Brouer You also fixed a missing _bh ... I verified you also included this fix ;-) https://github.com/netoptimizer/prototype-kernel/commit/24da4f495bb > changes since v6 > resize implemented. peek/full calls are no longer lockless > > replaced _FIELD macros with _CALL which invoke a function > on the pointer rather than just returning a value > > destroy now scans the array and frees all queued skbs > > changes since v5 > implemented a generic ptr_ring api, and > made skb_array a type-safe wrapper > apis for taking the spinlock in different contexts > following expected usecase in tun Tested-by: Jesper Dangaard Brouer -- Best regards, Jesper Dangaard Brouer MSc.CS, Principal Kernel Engineer at Red Hat Author of http://www.iptv-analyzer.org LinkedIn: http://www.linkedin.com/in/brouer