linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Kumar Gala <galak@kernel.crashing.org>
To: Kevin Hao <haokexin@gmail.com>
Cc: linuxppc <linuxppc-dev@lists.ozlabs.org>
Subject: Re: [PATCH] powerpc/fsl-pci: don't unmap the PCI SoC controller registers in setup_pci_atmu
Date: Mon, 29 Apr 2013 14:46:46 -0500	[thread overview]
Message-ID: <1E5EAD9C-0E2F-4B3D-9258-BC5865C46D76@kernel.crashing.org> (raw)
In-Reply-To: <1365837281-27002-1-git-send-email-haokexin@gmail.com>


On Apr 13, 2013, at 2:14 AM, Kevin Hao wrote:

> In patch 34642bbb (powerpc/fsl-pci: Keep PCI SoC controller registers =
in
> pci_controller) we choose to keep the map of the PCI SoC controller
> registers. But we missed to delete the unmap in setup_pci_atmu
> function. This will cause the following call trace once we access
> the PCI SoC controller registers later.
>=20
> Unable to handle kernel paging request for data at address =
0x8000080080040f14
> Faulting instruction address: 0xc00000000002ea58
> Oops: Kernel access of bad area, sig: 11 [#1]
> SMP NR_CPUS=3D24 T4240 QDS
> Modules linked in:
> NIP: c00000000002ea58 LR: c00000000002eaf4 CTR: c00000000002eac0
> REGS: c00000017e10b4a0 TRAP: 0300   Not tainted  =
(3.9.0-rc1-00052-gfa3529f-dirty)
> MSR: 0000000080029000 <CE,EE,ME>  CR: 28adbe22  XER: 00000000
> SOFTE: 0
> DEAR: 8000080080040f14, ESR: 0000000000000000
> TASK =3D c00000017e100000[1] 'swapper/0' THREAD: c00000017e108000 CPU: =
2
> GPR00: 0000000000000000 c00000017e10b720 c0000000009928d8 =
c00000017e578e00
> GPR04: 0000000000000000 000000000000000c 0000000000000001 =
c00000017e10bb40
> GPR08: 0000000000000000 8000080080040000 0000000000000000 =
0000000000000016
> GPR12: 0000000088adbe22 c00000000fffa800 c000000000001ba0 =
0000000000000000
> GPR16: 0000000000000000 0000000000000000 0000000000000000 =
0000000000000000
> GPR20: 0000000000000000 0000000000000000 0000000000000000 =
c0000000008a5b70
> GPR24: c0000000008af938 c0000000009a28d8 c0000000009bb5dc =
c00000017e10bb40
> GPR28: c00000017e32a400 c00000017e10bc00 c00000017e32a400 =
c00000017e578e00
> NIP [c00000000002ea58] .fsl_pcie_check_link+0x88/0xf0
> LR [c00000000002eaf4] .fsl_indirect_read_config+0x34/0xb0
> Call Trace:
> [c00000017e10b720] [c00000017e10b7a0] 0xc00000017e10b7a0 (unreliable)
> [c00000017e10ba30] [c00000000002eaf4] =
.fsl_indirect_read_config+0x34/0xb0
> [c00000017e10bad0] [c00000000033aa08] =
.pci_bus_read_config_byte+0x88/0xd0
> [c00000017e10bb90] [c00000000088d708] =
.pci_apply_final_quirks+0x9c/0x18c
> [c00000017e10bc40] [c0000000000013dc] .do_one_initcall+0x5c/0x1f0
> [c00000017e10bcf0] [c00000000086ebac] =
.kernel_init_freeable+0x180/0x26c
> [c00000017e10bdb0] [c000000000001bbc] .kernel_init+0x1c/0x460
> [c00000017e10be30] [c000000000000880] =
.ret_from_kernel_thread+0x64/0xe4
> Instruction dump:
> 38210310 2b800015 4fdde842 7c600026 5463fffe e8010010 7c0803a6 =
4e800020
> 60000000 60000000 e92301d0 7c0004ac <80690f14> 0c030000 4c00012c =
38210310
> ---[ end trace 7a8fe0cbccb7d992 ]---
>=20
> Kernel panic - not syncing: Attempted to kill init! =
exitcode=3D0x0000000b
>=20
> Signed-off-by: Kevin Hao <haokexin@gmail.com>
> ---
> This is based on Kumar's next branch.
>=20
> arch/powerpc/sysdev/fsl_pci.c | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)

applied to next

- k=

      parent reply	other threads:[~2013-04-29 19:47 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-13  7:14 [PATCH] powerpc/fsl-pci: don't unmap the PCI SoC controller registers in setup_pci_atmu Kevin Hao
2013-04-23 17:37 ` Roy Zang
2013-04-29 19:46 ` Kumar Gala [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=1E5EAD9C-0E2F-4B3D-9258-BC5865C46D76@kernel.crashing.org \
    --to=galak@kernel.crashing.org \
    --cc=haokexin@gmail.com \
    --cc=linuxppc-dev@lists.ozlabs.org \
    /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).