From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hannes Frederic Sowa Subject: Re: [net-next PATCH 1/5] bpf: introduce new bpf cpu map type BPF_MAP_TYPE_CPUMAP Date: Fri, 29 Sep 2017 11:40:48 +0200 Message-ID: <87r2upn8pr.fsf@stressinduktion.org> References: <150660339205.2808.7084136789768233829.stgit@firesoul> <150660342793.2808.10838498581615265043.stgit@firesoul> <20170929032146.vs5v454wjs4niu4k@ast-mbp> <8737760wg5.fsf@stressinduktion.org> <1506677857.2478.5.camel@redhat.com> Mime-Version: 1.0 Content-Type: text/plain Cc: Alexei Starovoitov , Jesper Dangaard Brouer , netdev@vger.kernel.org, jakub.kicinski@netronome.com, "Michael S. Tsirkin" , Jason Wang , mchan@broadcom.com, John Fastabend , peter.waskiewicz.jr@intel.com, Daniel Borkmann , Andy Gospodarek , edumazet@google.com To: Paolo Abeni Return-path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:35593 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750847AbdI2Jkz (ORCPT ); Fri, 29 Sep 2017 05:40:55 -0400 In-Reply-To: <1506677857.2478.5.camel@redhat.com> (Paolo Abeni's message of "Fri, 29 Sep 2017 11:37:37 +0200") Sender: netdev-owner@vger.kernel.org List-ID: Paolo Abeni writes: > On Fri, 2017-09-29 at 09:56 +0200, Hannes Frederic Sowa wrote: >> [adding Paolo, Eric] >> >> Alexei Starovoitov writes: >> >> > On Thu, Sep 28, 2017 at 02:57:08PM +0200, Jesper Dangaard Brouer wrote: >> >> [...] >> >> > > + wake_up_process(rcpu->kthread); >> > >> > In general the whole thing looks like 'threaded NAPI' that Hannes was >> > proposing some time back. I liked it back then and I like it now. >> > I don't remember what were the objections back then. >> > Something scheduler related? >> > Adding Hannes. > > Beyond the added scheduling complexity, the threaded NAPI > implementation proposed some time ago also possibly introduced OoO > packet delivery, because the NAPI threads were left unbound to any CPU. Right, yes, but that can be resolved. The problem was just in that particular patch.