Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:57584 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750846Ab1IZPHl (ORCPT ); Mon, 26 Sep 2011 11:07:41 -0400 Date: Mon, 26 Sep 2011 20:37:24 +0530 From: Rajkumar Manoharan To: Felix Fietkau CC: , , Subject: Re: [RFC v3] mac80211: Send nullfunc frames at lower rate Message-ID: <20110926150722.GA9947@vmraj-lnx.users.atheros.com> (sfid-20110926_170744_618740_B66F8DEF) References: <1317039804-8521-1-git-send-email-rmanohar@qca.qualcomm.com> <4E807F71.1010809@openwrt.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" In-Reply-To: <4E807F71.1010809@openwrt.org> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, Sep 26, 2011 at 07:34:41AM -0600, Felix Fietkau wrote: > On 2011-09-26 6:23 AM, Rajkumar Manoharan wrote: > >Recently mac80211 was changed to use nullfunc instead of probe > >request for connection monitoring for tx ack status reporting > >hardwares. These nullfunc data frames are being sent at higer > >rates and also as aggregated ones. This could probably delays > >the nullfunc ack so the connection is more frequently getting > >disconnected as max retries are reached. In order to improve > >the connectivity send the nullfunc at lower rate. > I don't think nullfunc frames should be sent at the lowest rate. If > they fail frequently, then the rate control module is doing > something crappy and should be fixed. So far I haven't seen any > nullfunc reliablity issues with minstrel_ht. > Also, as far as I know, mac80211 sends non-QoS nullfunc frames, so > they cannot get aggregated either. > Whatever you mentioned is completely fine, if nullfunc is not used for monitoring purpose. Otherwise the nullfunc can be aggregated and on worst case it will be retried to max and BAR will be exchange. By the time the connection was terminated. As we are using nulldata for controlling the connection, IMHO it is not wrong to send it at min rate. So we can avoid unnecessary disconnections. -- Rajkumar