From patchwork Wed Aug 19 12:59:41 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Grumbach, Emmanuel" X-Patchwork-Id: 508725 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id B2D21140784 for ; Wed, 19 Aug 2015 23:00:15 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932093AbbHSNAJ (ORCPT ); Wed, 19 Aug 2015 09:00:09 -0400 Received: from mga03.intel.com ([134.134.136.65]:11337 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754146AbbHSNAF (ORCPT ); Wed, 19 Aug 2015 09:00:05 -0400 Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga103.jf.intel.com with ESMTP; 19 Aug 2015 06:00:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,709,1432623600"; d="scan'208";a="771721199" Received: from egrumbacbox.jer.intel.com ([10.12.127.30]) by fmsmga001.fm.intel.com with ESMTP; 19 Aug 2015 06:00:02 -0700 From: Emmanuel Grumbach To: linux-wireless@vger.kernel.org Cc: ido@wizery.com, netdev@vger.kernel.org, Emmanuel Grumbach Subject: [RFC v2 3/3] iwlwifi: mvm: transfer the truesize to the last TSO segment Date: Wed, 19 Aug 2015 15:59:41 +0300 Message-Id: <1439989181-4444-4-git-send-email-emmanuel.grumbach@intel.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1439989181-4444-1-git-send-email-emmanuel.grumbach@intel.com> References: <1439989181-4444-1-git-send-email-emmanuel.grumbach@intel.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org This allows to release the backpressure on the socket only when the last segment is released. Now the truesize looks like this: if the truesize of the original skb is 65420, all the segments will have a truesize of 704 (skb itself) and the last one will have 65420. Change-Id: I3c894cf2afc0aedfe7b2a5b992ba41653ff79c0e Signed-off-by: Emmanuel Grumbach --- drivers/net/wireless/iwlwifi/mvm/tx.c | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/iwlwifi/mvm/tx.c b/drivers/net/wireless/iwlwifi/mvm/tx.c index 5046833..046e50d 100644 --- a/drivers/net/wireless/iwlwifi/mvm/tx.c +++ b/drivers/net/wireless/iwlwifi/mvm/tx.c @@ -764,7 +764,7 @@ static int iwl_mvm_tx_tso(struct iwl_mvm *mvm, struct sk_buff *skb_gso, bool ipv6 = skb_shinfo(skb_gso)->gso_type & SKB_GSO_TCPV6; struct iwl_lso_splitter s = {}; struct page *hdr_page; - unsigned int mpdu_sz; + unsigned int mpdu_sz, sum_truesize = 0; u8 *hdr_page_pos, *qc, tid; int i, ret; @@ -898,6 +898,7 @@ static int iwl_mvm_tx_tso(struct iwl_mvm *mvm, struct sk_buff *skb_gso, mpdu_sz, tcp_hdrlen(skb_gso)); __skb_queue_tail(mpdus_skb, skb_gso); + sum_truesize += skb_gso->truesize; /* mss bytes have been consumed from the data */ s.gso_payload_pos = s.mss; @@ -1034,6 +1035,20 @@ static int iwl_mvm_tx_tso(struct iwl_mvm *mvm, struct sk_buff *skb_gso, } __skb_queue_tail(mpdus_skb, skb); + sum_truesize += skb->truesize; + } + + /* Release the backpressure on the socket only when + * the last segment is released. + */ + if (skb_gso->destructor == sock_wfree) { + struct sk_buff *tail = mpdus_skb->prev; + + swap(tail->truesize, skb_gso->truesize); + swap(tail->destructor, skb_gso->destructor); + swap(tail->sk, skb_gso->sk); + atomic_add(sum_truesize - skb_gso->truesize, + &skb_gso->sk->sk_wmem_alloc); } ret = 0;