Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759412AbcDEQGh (ORCPT ); Tue, 5 Apr 2016 12:06:37 -0400 Received: from smtp-out6.electric.net ([192.162.217.184]:57462 "EHLO smtp-out6.electric.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752663AbcDEQGg (ORCPT ); Tue, 5 Apr 2016 12:06:36 -0400 From: David Laight To: "'Amitkumar Karwar'" , Eric Dumazet , Wei-Ning Huang CC: Kalle Valo , Linux Wireless , LKML , Nishant Sarmukadam , Sameer Nanda , "netdev@vger.kernel.org" , Sonny Rao , Douglas Anderson Subject: RE: [PATCH] mwifiex: add __GFP_REPEAT to skb allocation call Thread-Topic: [PATCH] mwifiex: add __GFP_REPEAT to skb allocation call Thread-Index: AQHRiXYoHVEQswN050uRfddbG1brt59wGbN2gACDFwCAADsQAIAKDmvAgACwTfA= Date: Tue, 5 Apr 2016 16:05:16 +0000 Message-ID: <063D6719AE5E284EB5DD2968C1650D6D54DC5395@AcuExch.aculab.com> References: <1459226840-36287-1-git-send-email-wnhuang@chromium.org> <87vb45brxc.fsf@kamboji.qca.qualcomm.com> <1459256320.6473.160.camel@edumazet-glaptop3.roam.corp.google.com> <4fa3762986eb4b848dbe7daf6871d7a8@SC-EXCH04.marvell.com> In-Reply-To: <4fa3762986eb4b848dbe7daf6871d7a8@SC-EXCH04.marvell.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.202.99.200] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-Outbound-IP: 213.249.233.130 X-Env-From: David.Laight@ACULAB.COM X-PolicySMART: 3396946, 3397078 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u35G6hgZ028564 Content-Length: 457 Lines: 11 From: Amitkumar Karwar > Sent: 05 April 2016 06:48 ... > Our one time allocated 64k buffer read from firmware contains multiple data chunks. We have a feature > called single port aggregation in which firmware attaches an aggregated buffer to single port. So > sometimes a single data chunk can exceed 32k. dev_kfree_skb_any() is called to free those data chunks. Ah yes, which particular problem does aggregating data into a single buffer solve? David