Return-path: Received: from mail-we0-f179.google.com ([74.125.82.179]:59628 "EHLO mail-we0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750816AbaAEHVk (ORCPT ); Sun, 5 Jan 2014 02:21:40 -0500 Received: by mail-we0-f179.google.com with SMTP id q59so14651815wes.38 for ; Sat, 04 Jan 2014 23:21:38 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <1388870653-3507-1-git-send-email-chaitanya.mgit@gmail.com> From: Krishna Chaitanya Date: Sun, 5 Jan 2014 12:51:18 +0530 Message-ID: (sfid-20140105_082157_885277_A5AB7571) Subject: Re: [PATCH] mac80211: SMPS: Fix the overwrite of rx_chains in the interface loop. To: Emmanuel Grumbach Cc: Johannes Berg , linux-wireless Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sun, Jan 5, 2014 at 12:12 PM, Emmanuel Grumbach wrote: > > On Sat, Jan 4, 2014 at 11:24 PM, Chaitanya T K wrote: > > The interface loop identifies the no of static and dynamic > > smps chains, but overwrites their values across the VIF's > > (assuming there are more than one, in my case its AP and STA > > loopback), so the driver might not intimated about this update > > of SMPS mode, is the value is same. > > > > So inform driver for every VIF. > > > > For Ex: STA's SMPS state is changed, but in the loop AP appears > > at the last, SMPS state of AP is unchanged hence not intimated > > to the driver. > > > > Signed-off-by: Chaitanya T K > > --- > > But the dynamic / static chains number is a property of the channel > context. If you have several vifs on the same channel context, the > maximum number of dynamic / static chains will be assigned to the > channel context. In short, I think that mac80211 already does the job > of taking all the parameters into account and putting the right value > into the channel context? > I don't really see what bug you are trying to solve - could you please > tell us exactly what is your configuration? > What SMPS do you have for each interface and what dynamic / static > chains settings does mac80211 end up with? Agree this looks good in channel context angle. Lets say we have two VIF's one AP and one STA and the driver doesn't support channel context's so in that case the config will happen through hw_config but the problem is hw_config is out side the loop so the config will happen with the parameters of the last VIF, which is incorrect.