Return-path: Received: from mail-oi0-f53.google.com ([209.85.218.53]:34580 "EHLO mail-oi0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756039AbcECRVS (ORCPT ); Tue, 3 May 2016 13:21:18 -0400 Received: by mail-oi0-f53.google.com with SMTP id k142so34978250oib.1 for ; Tue, 03 May 2016 10:21:17 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1462281243.2251.2.camel@sipsolutions.net> <1462291262.2251.4.camel@sipsolutions.net> Date: Tue, 3 May 2016 19:21:16 +0200 Message-ID: (sfid-20160503_192121_362934_59ACB842) Subject: Re: dynamic ps + offchannel mgmt_tx + HW RoC From: Janusz Dziedzic To: Krishna Chaitanya Cc: Johannes Berg , linux-wireless Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 3 May 2016 at 18:58, Krishna Chaitanya wrote: > On Tue, May 3, 2016 at 9:31 PM, Johannes Berg wrote: >> On Tue, 2016-05-03 at 20:02 +0530, Krishna Chaitanya wrote: >> >>> > i don't see any issues in the powersave w.r.t driver. Isn't it a >>> > valid case? you meant implementing dynamic_ps? >> >> No, I really did mean implementing the entire PS logic in the driver, >> instead of having mac80211 do it. >> >>> If it advertise HW_SUPPORTS_DYNAMIC_PS it works fine. >>> so may be we should queue those frames and send it to >>> HW once it is out of powersave? >> >> We could, but I *really* don't want to patch over the messy and broken >> powersave code in mac80211 now. >> >> I really do think that the only way out of this mess is to implement >> powersave entirely outside of mac80211; perhaps mac80211 could provide >> helpers for it, but tying it into the MLME implementation and having >> all the PS-Poll stuff be global etc. is simply wrong today. > Okay, i understand. This again points to our discussion about mac80211 > ps a while ago. May be we should document this somewhere so that > device driver developers are aware of this? > Why not remove PS code from the mac80211 then, while is broken and should not be used? BR Janusz > > > -- > Thanks, > Regards, > Chaitanya T K. > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html