From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subash Abhinov Kasiviswanathan Subject: Re: [RFC PATCH v2 06/10] udp: cope with UDP GRO packet misdirection Date: Tue, 23 Oct 2018 18:55:31 -0600 Message-ID: <42bdf1dc2422cf02b763acef7843b88f@codeaurora.org> References: <63e4ceb238db122d3d831f0809285243701b2284.1539957909.git.pabeni@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, Willem de Bruijn , Steffen Klassert To: Paolo Abeni Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:58644 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726240AbeJXLuF (ORCPT ); Wed, 24 Oct 2018 07:50:05 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: >> Is the "likely" required here? > > Not required, but currently helpful IMHO, as we should hit the above > only on unlikey and really unwonted configuration. > > Note that only SKB_GSO_UDP_L4 GSO packets will not match the above > likely condition. > >> HW can coalesce all incoming streams of UDP and may not know the >> socket >> state. >> In that case, a socket not having UDP GRO option might see a penalty >> here. > > Really? Is there any HW creating SKB_GSO_UDP_L4 packets on RX? if the > HW is doing that, without this patch, I think it's breaking existing > applications (which may expext that the read UDP frame length > implicitly describe the application level message length). > Hi Yes, I agree that existing HW would not work without the patch. My question was based on how UDP GRO packets from any future HW would interact with this code path and if they might be potentially have any side effects due to socket option not being set. We do not have control over the application and the socket options being used on systems like Android. The packet count reduction due to UDP GRO would help when there are multiple firewall rules present even if we do not take advantage of the reduced recvmsg() calls. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project