All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felix Janda <felix.janda@posteo.de>
To: Larry Finger <Larry.Finger@lwfinger.net>
Cc: John W Linville <linville@tuxdriver.com>,
	b43-dev <b43-dev@lists.infradead.org>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: [RFC/RFT] b43: Load initial firmware file asynchronously
Date: Sat, 8 Dec 2012 15:30:32 +0100	[thread overview]
Message-ID: <20121208143032.GA1903@gauss> (raw)
In-Reply-To: <50C01558.8050102@lwfinger.net>

On 12/05/12 at 09:47pm, Larry Finger wrote:
> Felix,
> 
> Please test this with b43 built into the kernel.
> 
> Thanks,
> 
> Larry

Thanks, for coming back to this.

It now gives me a kernel panic after 30 seconds.

I did some debugging using ssleep and printk. b43_fw_cb seems
to be called several times and on the last time the "firmware"
argument seems to be zero. The execution then returns to
b43_do_request_fw, which tries to deference a zero pointer in
the line

> +	if (ctx->blob->size < sizeof(struct b43_fw_header))


It is another machine then last time, has B4306 (rev 3) and a
kernel without modules. The firmware is in /lib/firmware/b43,
which is on the root partition. There is no udev installed.

Felix

WARNING: multiple messages have this Message-ID (diff)
From: Felix Janda <felix.janda@posteo.de>
To: Larry Finger <Larry.Finger@lwfinger.net>
Cc: John W Linville <linville@tuxdriver.com>,
	b43-dev <b43-dev@lists.infradead.org>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [RFC/RFT] b43: Load initial firmware file asynchronously
Date: Sat, 8 Dec 2012 15:30:32 +0100	[thread overview]
Message-ID: <20121208143032.GA1903@gauss> (raw)
In-Reply-To: <50C01558.8050102@lwfinger.net>

On 12/05/12 at 09:47pm, Larry Finger wrote:
> Felix,
> 
> Please test this with b43 built into the kernel.
> 
> Thanks,
> 
> Larry

Thanks, for coming back to this.

It now gives me a kernel panic after 30 seconds.

I did some debugging using ssleep and printk. b43_fw_cb seems
to be called several times and on the last time the "firmware"
argument seems to be zero. The execution then returns to
b43_do_request_fw, which tries to deference a zero pointer in
the line

> +	if (ctx->blob->size < sizeof(struct b43_fw_header))


It is another machine then last time, has B4306 (rev 3) and a
kernel without modules. The firmware is in /lib/firmware/b43,
which is on the root partition. There is no udev installed.

Felix

  reply	other threads:[~2012-12-08 14:30 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-06  3:47 [RFC/RFT] b43: Load initial firmware file asynchronously Larry Finger
2012-12-08 14:30 ` Felix Janda [this message]
2012-12-08 14:30   ` Felix Janda
2012-12-08 17:15   ` Larry Finger
2012-12-08 17:15     ` Larry Finger
2012-12-08 17:56     ` Felix Janda
2012-12-08 17:56       ` Felix Janda
2012-12-08 19:48       ` Larry Finger
2012-12-08 19:48         ` Larry Finger
     [not found]         ` <20121208202435.GA14973@gauss>
     [not found]           ` <50C3AF0C.8000607@lwfinger.net>
2012-12-08 21:41             ` Felix Janda
2012-12-08 21:41               ` Felix Janda
2012-12-08 22:19               ` Larry Finger
2012-12-08 22:19                 ` Larry Finger
2012-12-08 22:35                 ` Felix Janda
2012-12-08 22:35                   ` Felix Janda
2012-12-08 23:12                   ` Larry Finger
2012-12-08 23:12                     ` Larry Finger
2012-12-09  7:44                     ` Felix Janda
2012-12-09  7:44                       ` Felix Janda

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=20121208143032.GA1903@gauss \
    --to=felix.janda@posteo.de \
    --cc=Larry.Finger@lwfinger.net \
    --cc=b43-dev@lists.infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.