Re: Patch "mac80211: send control port protocol frames to the VO queue" has been added to the 3.4-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 20, 2014 at 10:29:13AM +0000, Luís Henriques wrote:
> On Wed, Mar 19, 2014 at 11:45:02PM +0000, Greg Kroah-Hartman wrote:
> > 
> > This is a note to let you know that I've just added the patch titled
> > 
> >     mac80211: send control port protocol frames to the VO queue
> > 
> > to the 3.4-stable tree which can be found at:
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > The filename of the patch is:
> >      mac80211-send-control-port-protocol-frames-to-the-vo-queue.patch
> > and it can be found in the queue-3.4 subdirectory.
> > 
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@xxxxxxxxxxxxxxx> know about it.
> > 
> > 
> > From 1bf4bbb4024dcdab5e57634dd8ae1072d42a53ac Mon Sep 17 00:00:00 2001
> > From: Felix Fietkau <nbd@xxxxxxxxxxx>
> > Date: Tue, 11 Feb 2014 16:02:47 +0100
> > Subject: mac80211: send control port protocol frames to the VO queue
> > 
> > From: Felix Fietkau <nbd@xxxxxxxxxxx>
> > 
> > commit 1bf4bbb4024dcdab5e57634dd8ae1072d42a53ac upstream.
> > 
> > Improves reliability of wifi connections with WPA, since authentication
> > frames are prioritized over normal traffic and also typically exempt
> > from aggregation.
> > 
> > Signed-off-by: Felix Fietkau <nbd@xxxxxxxxxxx>
> > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > 
> > ---
> >  net/mac80211/wme.c |    5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > --- a/net/mac80211/wme.c
> > +++ b/net/mac80211/wme.c
> > @@ -132,6 +132,11 @@ u16 ieee80211_select_queue(struct ieee80
> >  		return IEEE80211_AC_BE;
> >  	}
> >  
> > +	if (skb->protocol == sdata->control_port_protocol) {
> > +		skb->priority = 7;
> > +		return ieee80211_downgrade_queue(sdata, skb);
> 
> This doesn't look correct to me.  In 3.4 kernel the 1st parameter for
> function ieee80211_downgrade_queue() is a struct ieee80211_local, not a
> struct ieee80211_sub_if_data.

Good catch, I've dropped it from there now, thanks.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]