From: Borislav Petkov <bp@alien8.de>
To: Dave Hansen <dave@sr71.net>
Cc: linux-kernel@vger.kernel.org, hpa@linux.intel.com,
penguin-kernel@i-love.sakura.ne.jp, x86@kernel.org,
akpm@linux-foundation.org, suresh.b.siddha@intel.com,
khlebnikov@openvz.org, bhelgaas@google.com
Subject: Re: [PATCH] do not try to sync identity map for non-mapped pages
Date: Sun, 7 Apr 2013 15:33:46 +0200 [thread overview]
Message-ID: <20130407133346.GD31299@pd.tnic> (raw)
In-Reply-To: <20130307163151.D9B58C4E@kernel.stglabs.ibm.com>
Hey Dave,
On Thu, Mar 07, 2013 at 08:31:51AM -0800, Dave Hansen wrote:
>
> The original bug reporter says this fixes it for him, so I'm
> broadening the cc list a bit. I assume this should just get
> sucked in to the x86 tree.
looks like we haven't whacked all the moles - I keep seeing this when
testing 32-bit builds in qemu on latest Linus + tip. I'd guess this is
still that /dev/mem accessing thing called wdm.
I'm still wondering though whether we should BUG_ON on a /dev/mem
access?
I've added debug output to show why we're triggering:
[ 471.102902] slow_virt_to_phys((void *)x): 0x0, phys_addr: 0x37bfe000
[ 471.119500] ------------[ cut here ]------------
[ 471.119500] kernel BUG at arch/x86/mm/physaddr.c:85!
[ 471.119500] invalid opcode: 0000 [#1] PREEMPT SMP
[ 471.119500] Modules linked in:
[ 471.119500] Pid: 1571, comm: wdm Not tainted 3.9.0-rc5+ #42 Bochs Bochs
[ 471.119500] EIP: 0060:[<c1032f56>] EFLAGS: 00000206 CPU: 0
[ 471.119500] EIP is at __phys_addr+0x86/0xb0
[ 471.119500] EAX: 37bfe000 EBX: 37bfe000 ECX: 00000001 EDX: 37bfe000
[ 471.119500] ESI: f7bfe000 EDI: 00002000 EBP: f67f1f3c ESP: f67f1f28
[ 471.119500] DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0068
[ 471.119500] CR0: 8005003b CR2: bfeb12d4 CR3: 35edd000 CR4: 000006f0
[ 471.119500] DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000
[ 471.119500] DR6: 00000000 DR7: 00000000
[ 471.119500] Process wdm (pid: 1571, ti=f67f0000 task=f5c90000 task.ti=f67f0000)
[ 471.119500] Stack:
[ 471.119500] c16b1f2c 00000000 37bfe000 00000000 00002000 f67f1f64 c131d074 00002000
[ 471.119500] 00000246 bfeb12ec 00000000 00000000 f67a6c40 c131d040 00002000 f67f1f8c
[ 471.119500] c1129c85 f67f1f98 f67f0000 f5ebe864 c131d040 00000020 f67a6c40 00000000
[ 471.119500] Call Trace:
[ 471.119500] [<c131d074>] read_mem+0x34/0x100
[ 471.119500] [<c131d040>] ? write_mem+0x110/0x110
[ 471.119500] [<c1129c85>] vfs_read+0x85/0x130
[ 471.119500] [<c131d040>] ? write_mem+0x110/0x110
[ 471.119500] [<c1129e87>] sys_read+0x47/0xa0
[ 471.119500] [<c1546e5e>] sysenter_do_call+0x12/0x36
[ 471.119500] Code: 0b a1 88 ae 0c c2 05 00 00 80 00 39 c6 72 bb a1 ac 1a 76 c1 2d 00 a0 3e 00 25 00 00 e0 ff 2d 00 20 00 00 39 c6 73 a3 0f 0b 0f 0b <0f> 0b 89 f0 e8 41 ca ff ff 89 5c 24 08 89 44 24 04 c7 04 24 2c
[ 471.119500] EIP: [<c1032f56>] __phys_addr+0x86/0xb0 SS:ESP 0068:f67f1f28
[ 471.508967] ---[ end trace 5fc00ac35d61284a ]---
Hmmm.
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is fine.
--
next prev parent reply other threads:[~2013-04-07 13:33 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-07 16:31 [PATCH] do not try to sync identity map for non-mapped pages Dave Hansen
2013-03-07 21:59 ` [tip:x86/urgent] x86: Do " tip-bot for Dave Hansen
2013-03-07 22:05 ` [PATCH] do " Tetsuo Handa
2013-03-07 22:13 ` H. Peter Anvin
2013-04-07 13:33 ` Borislav Petkov [this message]
2013-04-07 16:34 ` H. Peter Anvin
2013-04-07 17:25 ` Borislav Petkov
2013-04-08 20:33 ` Dave Hansen
-- strict thread matches above, loose matches on Subject: below --
2013-03-06 23:10 Dave Hansen
2013-03-07 10:19 ` Tetsuo Handa
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=20130407133346.GD31299@pd.tnic \
--to=bp@alien8.de \
--cc=akpm@linux-foundation.org \
--cc=bhelgaas@google.com \
--cc=dave@sr71.net \
--cc=hpa@linux.intel.com \
--cc=khlebnikov@openvz.org \
--cc=linux-kernel@vger.kernel.org \
--cc=penguin-kernel@i-love.sakura.ne.jp \
--cc=suresh.b.siddha@intel.com \
--cc=x86@kernel.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