Return-path: Received: from mail-qa0-f49.google.com ([209.85.216.49]:49664 "EHLO mail-qa0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755991Ab2EGMBs (ORCPT ); Mon, 7 May 2012 08:01:48 -0400 Received: by qabj40 with SMTP id j40so3647409qab.1 for ; Mon, 07 May 2012 05:01:48 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1319350901-31178-1-git-send-email-arik@wizery.com> <4EA7DA0F.7050603@sipsolutions.net> Date: Mon, 7 May 2012 14:01:48 +0200 Message-ID: (sfid-20120507_140152_158792_B9E4AA27) Subject: Re: [PATCH] mac80211: support adding IV-room in the skb for CCMP keys From: Janusz Dziedzic To: Arik Nemtsov Cc: Johannes Berg , linux-wireless@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: 2011/10/26 Arik Nemtsov : > On Wed, Oct 26, 2011 at 11:59, Johannes Berg wrote: >> On 10/23/2011 8:21 AM, Arik Nemtsov wrote: >>> >>> Some cards can generate CCMP IVs in HW, but require the space for the IV >>> to be pre-allocated in the frame at the correct offset. Add a key flag >>> that allows us to achieve this. >> >> Is it really that expensive to generate the IV and then not use it that this >> is worth the extra complexity? This not just makes it more complex but also >> more expensive in the other case. >> > > Some of the platforms with this chip are pretty weak (host CPU is the > bottleneck). > > We add another "if" for the other case (for a value that's likely in > the cacheline already). I don't think that's too bad. > Hello, Why this is only done for CCMP? Our firmware require such IVs allocation for all modes and currently we have common code that do that in the driver based on: iv_len = tx_info->control.hw_key->iv_len icv_len = tx_info->control.hw_key->icv_len /* cw1200 driver */ skb_push(t->skb, iv_len); memmove(newhdr, newhdr + iv_len, t->hdrlen); skb_put(t->skb, icv_len); .... Isn't better to handle all modes in mac80211 base on IEEE80211_KEY_FLAG_PUT_IV_SPACE or just leave this for the driver? I know this is easy to fix in our driver but still we have to remember that in case of CCMP mac80211 will already do it for us and will not do that in case of other modes. So, my proposal is to remove all changes from net/mac80211/wpa.c file and remember that driver should care about it - in such case PUT_IV_SPACE will be more generic. BR Janusz