From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH V4 net 2/3] tuntap: disable preemption during XDP processing Date: Mon, 26 Feb 2018 13:50:10 -0500 (EST) Message-ID: <20180226.135010.66707697236959796.davem@davemloft.net> References: <1519443146-9089-1-git-send-email-jasowang@redhat.com> <1519443146-9089-2-git-send-email-jasowang@redhat.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, mst@redhat.com, sergei.shtylyov@cogentembedded.com, christoffer.dall@linaro.org, brouer@redhat.com To: jasowang@redhat.com Return-path: In-Reply-To: <1519443146-9089-2-git-send-email-jasowang@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Jason Wang Date: Sat, 24 Feb 2018 11:32:25 +0800 > Except for tuntap, all other drivers' XDP was implemented at NAPI > poll() routine in a bh. This guarantees all XDP operation were done at > the same CPU which is required by e.g BFP_MAP_TYPE_PERCPU_ARRAY. But > for tuntap, we do it in process context and we try to protect XDP > processing by RCU reader lock. This is insufficient since > CONFIG_PREEMPT_RCU can preempt the RCU reader critical section which > breaks the assumption that all XDP were processed in the same CPU. > > Fixing this by simply disabling preemption during XDP processing. > > Fixes: 761876c857cb ("tap: XDP support") > Signed-off-by: Jason Wang Applied.