netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Zhu Yi <yi.zhu@intel.com>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: Tomas Winkler <tomasw@gmail.com>,
	"davem@davemloft.net" <davem@davemloft.net>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"sfr@canb.auug.org.au" <sfr@canb.auug.org.au>
Subject: Re: pull request: wireless-2.6 2008-11-18
Date: Mon, 24 Nov 2008 11:34:15 +0800	[thread overview]
Message-ID: <1227497655.2548.257.camel@debian.sh.intel.com> (raw)
In-Reply-To: <20081119143919.GA3485@tuxdriver.com>

On Wed, 2008-11-19 at 22:39 +0800, John W. Linville wrote:
> Well, thanks for your opinion.  Here is mine:

I have to say something here. Although not related to the original
ieee80211_notify_mac change.

> The iwlwifi drivers have been hitting these problems for months,
> and I have seen no effort by your team to address the problems.
> It is possible that your team is working on them behind closed doors
> and will eventually throw something over the wall to us.  I'm tired
> of waiting for that, and I imagine that hordes of iwlagn users are
> tired of waiting as well.

This is not correct. We are tracing this problem in our bugzilla. And it
is the open place. Much more people have used it for a much longer time
even before the kernel bugzilla existed. If you take a look at the bug,
you know how much effort we put on it:
http://www.intellinuxwireless.org/bugzilla/show_bug.cgi?id=1703

> Johannes has presented us with plausible fixes, and people are
> reporting that the patches work for them.  I can merge these patches,
> or wait/hope/pray for some to come from your team.  Experience
> suggests
> that if fixes do come from your team that they will either be buried
> in some monster patch that largely addresses something else, or
> that the patches will arrive with a changelog that is terse and/or
> unintelligble.  Subsequently, I am not optimistic about waiting.
> 
> I think Johannes cited four different bugzilla.kernel.org entries
> between those two patches.  What is your team doing to address
> those bugs?

Johannes' effort is great and should certainly be praised. But looking
down upon Intel's effort is nonsense. We suspected this alignment issue
in the very beginning. But the set of users helped us to debug with this
problem happened to have a different root cause than Johannes. Thus came
with the alignment BUG_ON assert put into the code in case other people
experience with it (unfortunately, it was a wrong one. As I had
appologized for this already.)

The problem should at least have two causes. Johannes had resolved one
of them. The other one (AFAICS) relates to be in an 11n AP environment
(no need to be associated with it) and takes longer time to reproduce.
We will keep tracking it whatever we you think us about it.

Thanks,
-yi


  parent reply	other threads:[~2008-11-24  3:33 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-19  0:07 pull request: wireless-2.6 2008-11-18 John W. Linville
     [not found] ` <20081119000719.GA19756-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2008-11-19  1:33   ` Johannes Berg
     [not found]     ` <1227058418.4014.99.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2008-11-19  1:54       ` John W. Linville
2008-11-20 12:05   ` David Miller
2008-11-19  6:46 ` Tomas Winkler
2008-11-19  9:34   ` Tomas Winkler
     [not found]     ` <1ba2fa240811190134u7974a840i532e8ed0e18f8c72-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-19 14:39       ` John W. Linville
     [not found]         ` <20081119143919.GA3485-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2008-11-19 16:51           ` Tomas Winkler
     [not found]             ` <1ba2fa240811190851i5dda6734w10d38b0f27e5ac97-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-19 18:53               ` John W. Linville
     [not found]                 ` <20081119185340.GC3485-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2008-11-19 21:55                   ` Tomas Winkler
     [not found]                     ` <1ba2fa240811191355p7218dd84j8517d9ff4bab4e0b-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-20  0:37                       ` John W. Linville
     [not found]                         ` <20081120003748.GK3485-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2008-11-20  9:27                           ` Tomas Winkler
2008-11-24  3:34         ` Zhu Yi [this message]
2008-11-24  3:50           ` Marcel Holtmann
2008-11-24  6:33             ` Zhu Yi
     [not found]               ` <1227508394.2548.307.camel-F7Q/YMNgnyhhZnBKgDE8z0EOCMrvLtNR@public.gmane.org>
2008-11-24  8:33                 ` Marcel Holtmann

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=1227497655.2548.257.camel@debian.sh.intel.com \
    --to=yi.zhu@intel.com \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=netdev@vger.kernel.org \
    --cc=sfr@canb.auug.org.au \
    --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;
as well as URLs for NNTP newsgroup(s).