Linux wireless drivers development
 help / color / mirror / Atom feed
From: Sujith <Sujith.Manoharan@atheros.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Tomas Winkler <tomasw@gmail.com>,
	"linville@tuxdriver.com" <linville@tuxdriver.com>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	Luis Rodriguez <Luis.Rodriguez@Atheros.com>
Subject: Re: [RFC] mac80211: Re-enable aggregation
Date: Thu, 23 Oct 2008 14:27:08 +0530	[thread overview]
Message-ID: <18688.15460.336059.400706@gargle.gargle.HOWL> (raw)
In-Reply-To: <1224696170.30459.12.camel@johannes.berg>

Johannes Berg wrote:
> On Wed, 2008-10-22 at 13:59 +0200, Tomas Winkler wrote:
> 
> > It answers BA i n RX path and interpret BA in TX Scheduling
> > retransmission of not acked frames is done in HW (not firmware) no
> > need to requeue them. In corner case when retries are exhausted  BA
> > request is issued from mac80211 to advance RX side window.
> 
> Ok, but if we do both of that in mac80211, it wouldn't really matter
> since we never see the relevant frames, right? Or we can add a flag to
> ignore them.
> 
> > > Like I said, from what I can tell bcom is really similar, and I'd like
> > > to not have to re-implement all this first. mac80211 used to have a more
> > > push-model, which we've deviated from with the HT stuff you did, but for
> > > most hardware the push-model is still appropriate, so imho we should try
> > > to go to one even with HT and then see.
> > 
> > Works for me. What I stumble upon is retransmission from within TX
> > response  path
> > how this go together with context of regular TX path. Even in iwlwifi
> > we need to be able to kick/start the internal aggregation queue?
> 
> Yeah, this is something we definitely need to work out. We haven't even
> managed to decide yet how aggregated packets are given to the driver :)
> I think for atheros and bcom a model where mac80211 decides pretty much
> everything and hands the driver a list of skbs to aggregate would work
> best, but then I think that wouldn't work with your hw at all.
> 

Retransmission of unacked frames is done within the driver, in ath9k.
In case of retry exhaustion (failed transmission), we do what Intel does,
i.e, request for a BAR to be sent out.

Sujith

  reply	other threads:[~2008-10-23  8:59 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-20  8:22 [RFC] mac80211: Re-enable aggregation Sujith
2008-10-20  8:31 ` Johannes Berg
2008-10-20  8:35   ` Luis R. Rodriguez
2008-10-20  8:45     ` Johannes Berg
2008-10-20  8:47       ` Sujith
2008-10-20  8:55         ` Luis R. Rodriguez
2008-10-20  8:57           ` Johannes Berg
2008-10-20  9:01             ` Luis R. Rodriguez
2008-10-20  9:04               ` Johannes Berg
2008-10-20  9:05                 ` Luis R. Rodriguez
2008-10-20  9:11                 ` Sujith
2008-10-20  8:57       ` Luis R. Rodriguez
2008-10-20  8:58   ` Sujith
2008-10-20  9:12     ` Johannes Berg
2008-10-20  9:31       ` Sujith
2008-10-20  9:38         ` Johannes Berg
2008-10-20 10:35           ` Sujith
2008-10-20 12:22             ` Johannes Berg
2008-10-20 18:03               ` Sujith
2008-10-20 21:46                 ` Tomas Winkler
2008-10-21  6:46                   ` Johannes Berg
2008-10-21  7:18                   ` Sujith
2008-10-22 10:03                   ` Johannes Berg
2008-10-22 11:59                     ` Tomas Winkler
2008-10-22 17:22                       ` Johannes Berg
2008-10-23  8:57                         ` Sujith [this message]
2008-10-23 14:46                           ` Johannes Berg
2008-10-27 14:25                             ` Sujith
2008-10-27 15:04                               ` Johannes Berg
2008-10-27 15:56                                 ` Tomas Winkler
2008-10-27 16:04                                   ` Johannes Berg
2008-10-28 14:24                                     ` Tomas Winkler
2008-10-22 10:00                 ` Johannes Berg
2008-10-22 11:41                   ` Tomas Winkler
2008-10-22 17:24                     ` Johannes Berg
2008-10-23  9:31                   ` Sujith
2008-10-23 14:45                     ` Johannes Berg
2008-10-23 17:23                       ` Luis R. Rodriguez
2008-10-23 18:31                         ` Tomas Winkler
2008-10-23 20:18                           ` Luis R. Rodriguez
2008-10-23 21:17                             ` Tomas Winkler
2008-10-31  1:26                               ` Luis R. Rodriguez
2008-10-27 14:11                       ` Sujith
2008-10-27 15:07                         ` Johannes Berg
2008-10-20  9:14     ` Luis R. Rodriguez
2008-10-20  9:24       ` Johannes Berg
2008-10-20 10:07         ` Sujith
2008-10-20 10:15           ` Johannes Berg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18688.15460.336059.400706@gargle.gargle.HOWL \
    --to=sujith.manoharan@atheros.com \
    --cc=Luis.Rodriguez@Atheros.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=tomasw@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox