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 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.