[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080827233103.GE5927@tesla>
Date: Wed, 27 Aug 2008 16:31:03 -0700
From: "Luis R. Rodriguez" <lrodriguez@...eros.com>
To: Tomas Winkler <tomasw@...il.com>
CC: Michael Buesch <mb@...sch.de>,
"John W. Linville" <linville@...driver.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Bruno Randolf <bruno@...nktube.com>
Subject: Re: pull request: wireless-2.6 2008-08-26
On Wed, Aug 27, 2008 at 04:11:15PM -0700, Tomas Winkler wrote:
> if (beacon_timestamp > rx_timestamp)
> merge
>
> The patch from Assaf just disable reporting RX timestamp to mac and
> thus disabling merging which gives incorrect spec behavior but smooth
> traffic.
We *should follow the spec*.
> Actually we've checked few cards including broadcom and various
> windows NICs
What are windows NICs?
> and non of them implements this correctly so this WA is
> probably the solution.
Disagreed! If there is hardware which is not capable of handling this
we should simply have a HW flag which specifies this to handle this as a
work around (WA). Just because some cards are not capable it doesn't
mean it should impose that on the rest.
> Other solution would be to mark leader with highest TSF and not
> reconnecting to the same station again and again.
>
> Last solution would be to remove this merging all together but then
> I'm not sure if Bruno added this code just implement the spec or
> really tested it with any hardware. I'm not sure if any vendor
> implements PS in IBSS so this merging is probably not important
> anyway.
Absolutely not! IBSS merge is per spec, otherwise you don't really have
a real IBSS.
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists