From: "Arend van Spriel" <arend@broadcom.com>
To: "Larry Finger" <Larry.Finger@lwfinger.net>
Cc: "Rafał Miłecki" <zajec5@gmail.com>,
b43-dev <b43-dev@lists.infradead.org>,
wireless <linux-wireless@vger.kernel.org>
Subject: Re: Performance of BCM43224 (14e4:4353)
Date: Sat, 10 Dec 2011 21:28:33 +0100 [thread overview]
Message-ID: <4EE3C0F1.2010307@broadcom.com> (raw)
In-Reply-To: <4EE3B7B9.5010908@lwfinger.net>
On 12/10/2011 08:49 PM, Larry Finger wrote:
> On 12/10/2011 05:04 AM, Arend van Spriel wrote:
>>
>> Obviously it should be a pci_read_config_dword(). I found out that I can
>> reproduce the problem over here so I added the print statements:
>>
>> [61359.893254] brcmsmac bcma0:0: mfg 4bf core 812 rev 23 class 0 irq 17
>> [61359.897801] core[2] id=0x820 addr=18002000 wrap=18102000
>> [61359.897808] core[1] id=0x812 addr=18001000 wrap=18101000
>> [61359.897814] core[0] id=0x800 addr=18000000 wrap=18100000
>> [61359.897822] PCI_BAR0_WIN=18107000
>>
>> Not sure how it got programmed to this value, but it definitely is wrong.
>
> I put pr_info() and dump_stack() at every place where PCI_BAR_WIN0 is changed.
> The one place in drivers/bcma/host_pci.c is never called - only the location in
> drivers/bcma/scan.c.
>
> The logged data is
>
> [ 813.683905] bcma: BCMA_PCI_BAR0_WIN set to 0x18000000
> [ 813.683910] Pid: 8989, comm: work_for_cpu Tainted: G O
> 3.2.0-rc4-wl+ #146
> [ 813.683913] Call Trace:
> [ 813.683926] [<ffffffffa0389af6>]
> bcma_scan_switch_core.isra.9.part.10+0x36/0x40 [bcma]
> [ 813.683933] [<ffffffffa0389bde>] bcma_init_bus+0x6e/0x70 [bcma]
> [ 813.683940] [<ffffffffa0389c04>] bcma_bus_scan+0x24/0x210 [bcma]
> [ 813.683946] [<ffffffff81028f11>] ? __ioremap_caller+0x2b1/0x390
> [ 813.683951] [<ffffffff811dded0>] ? pci_iomap+0x80/0xc0
> [ 813.683958] [<ffffffffa0389209>] bcma_bus_register+0x19/0x2e0 [bcma]
> [ 813.683964] [<ffffffffa038b800>] bcma_host_pci_probe+0x120/0x1c0 [bcma]
> [ 813.683970] [<ffffffff810636b0>] ? cwq_activate_first_delayed+0x110/0x110
> [ 813.683976] [<ffffffff811f17b2>] local_pci_probe+0x12/0x20
> [ 813.683980] [<ffffffff810636c3>] do_work_for_cpu+0x13/0x30
> [ 813.683984] [<ffffffff8106bb07>] kthread+0x87/0x90
> [ 813.683990] [<ffffffff81384f74>] kernel_thread_helper+0x4/0x10
> [ 813.683995] [<ffffffff8138269d>] ? retint_restore_args+0xe/0xe
> [ 813.683999] [<ffffffff8106ba80>] ? __init_kthread_worker+0x70/0x70
> [ 813.684041] [<ffffffff81384f70>] ? gs_change+0xb/0xb
> [ 813.684048] bcma: BCMA_PCI_BAR0_WIN set to 0x18107000
>
> The stack trace is identical for the second entry. It is never remapped, which
> is why it is wrong in brcmsmac.
Great you found it.
> Question: Don't you need to change the R_REG macro to use bcma? It is still
> doing read{b,w,l} operations, and is not checking the mapping.
In the end the R_REG/W_REG macros will be removed and only
bcma_read/bcma_write are used. That will be in the next patch series.
> Larry
>
Gr. AvS
prev parent reply other threads:[~2011-12-10 20:28 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-09 0:46 Performance of BCM43224 (14e4:4353) Larry Finger
2011-12-09 9:16 ` Arend van Spriel
2011-12-09 10:27 ` Arend van Spriel
2011-12-09 16:44 ` Larry Finger
2011-12-09 17:32 ` Arend van Spriel
2011-12-09 17:57 ` Larry Finger
2011-12-09 19:31 ` Arend van Spriel
2011-12-09 20:06 ` Larry Finger
2011-12-09 18:17 ` Larry Finger
2011-12-09 20:04 ` Arend van Spriel
2011-12-09 22:53 ` Larry Finger
2011-12-10 9:18 ` Rafał Miłecki
2011-12-10 11:04 ` Arend van Spriel
2011-12-10 19:49 ` Larry Finger
2011-12-10 20:28 ` Arend van Spriel [this message]
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=4EE3C0F1.2010307@broadcom.com \
--to=arend@broadcom.com \
--cc=Larry.Finger@lwfinger.net \
--cc=b43-dev@lists.infradead.org \
--cc=linux-wireless@vger.kernel.org \
--cc=zajec5@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).