linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <holtmann@linux.intel.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Zhu Yi <yi.zhu@intel.com>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	Tomas Winkler <tomasw@gmail.com>
Subject: Re: Crash with 2.6.27-rc6 with iwlwifi
Date: Mon, 06 Oct 2008 15:01:30 +0200	[thread overview]
Message-ID: <1223298090.11272.167.camel@violet.holtmann.net> (raw)
In-Reply-To: <1223296783.15196.13.camel@johannes.berg>

Hi Johannes,

> > > > I take this back. It took a couple of hours, but it crashed again.
> > > 
> > > Do you have 64k pages enabled? My issue goes away entirely when I
> > > disable 64k pages.
> > 
> > so the initial issue was a BUG_ON when the ucode reports back an
> > unsupported event. That should have been fixed by now since we just
> > can't crash the whole kernel, because some piece of firmware acts bogus
> > now and then. 
> 
> I have a different interpretation of the original issue, since afaik the
> value it was doing a BUG_ON on is a value that is only passed in by the
> driver and copied by the ucode to the response, hence it shouldn't
> actually crash unless the driver is passing bogus stuff in or, as it
> seems to be the case here, the DMA programming is completely fucked and
> the ucode just goes to write to random memory locations.

that patch wasn't enough, but neverless we shouldn't crash the whole
kernel based on this.

In addition they were working on some IOMMU stuff and that explains what
you are seeing. I never got that far at all. My machine just dies on me.

> > However there seems to be some other issues with me
> > running a 64-bit OS and connecting to an N-capable access point.
> 
> Strange. My test AP isn't N capable though.

We had some discussion about the fact the different firmware versions
might be causing this. Don't have my X61 handy at the moment.

Regards

Marcel



  reply	other threads:[~2008-10-06 13:01 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-11  8:04 Crash with 2.6.27-rc6 with iwlwifi Marcel Holtmann
2008-09-11  9:02 ` Zhu Yi
2008-09-11 15:06   ` Marcel Holtmann
2008-09-12  1:47   ` Marcel Holtmann
2008-09-12  2:10     ` Marcel Holtmann
2008-09-12  7:49       ` Johannes Berg
2008-10-06 12:30       ` Johannes Berg
2008-10-06 12:32         ` Johannes Berg
2008-10-06 12:36           ` Johannes Berg
2008-10-06 13:07           ` Johannes Berg
2008-10-06 13:21             ` Johannes Berg
2008-10-06 12:36         ` Marcel Holtmann
2008-10-06 12:39           ` Johannes Berg
2008-10-06 13:01             ` Marcel Holtmann [this message]
2008-10-06 13:04               ` Johannes Berg
2008-10-06 16:52             ` Luis R. Rodriguez
2008-10-06 16:55               ` Johannes Berg
2008-09-11 10:36 ` Johannes Berg
2008-09-11 15:03   ` Marcel Holtmann
2008-09-11 15:46     ` Johannes Berg
2008-09-24 20:14 ` Johannes Berg
2008-09-24 23:00   ` Ian Schram
2008-09-25  4:28     ` 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=1223298090.11272.167.camel@violet.holtmann.net \
    --to=holtmann@linux.intel.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=tomasw@gmail.com \
    --cc=yi.zhu@intel.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).