[Bugme-new] [Bug 21332] New: Regression in single stepping over hardware breakpoint (Wine)
bugzilla-daemon at bugzilla.kernel.org
bugzilla-daemon at bugzilla.kernel.org
Wed Oct 27 16:41:38 PDT 2010
https://bugzilla.kernel.org/show_bug.cgi?id=21332
URL: http://git.kernel.org/?p=linux/kernel/git/torvalds/lin
ux-2.6.git;a=commit;h=0c4519e825c9e2b6a8310deff8582f8c
35bfbba9
Summary: Regression in single stepping over hardware breakpoint
(Wine)
Product: Process Management
Version: 2.5
Kernel Version: 2.6.36
Platform: All
OS/Version: Linux
Tree: Mainline
Status: NEW
Severity: normal
Priority: P1
Component: Other
AssignedTo: process_other at kernel-bugs.osdl.org
ReportedBy: mstefani at redhat.com
Regression: Yes
There is a regression between 2.6.35 and 2.6.36 for 32bit Wine in the Wine test
checking the single stepping over hardware breakpoints:
/home/michi/work/wine/tools/runtest -q -P wine -M ntdll.dll -T ../../.. -p
ntdll_test.exe.so /home/michi/work/wine/dlls/ntdll/tests/exception.c && touch
exception.ok
exception.c:585: Test failed: eip is wrong: 340002 instead of 340001
exception.c:587: Test failed: B0 flag is not set in Dr6
exception.c:588: Test failed: BS flag is set in Dr6
exception.c:593: Test failed: eip is wrong: 7ed569d3 instead of 340002
make: *** [exception.ok] Error 4
All those tests are in bpx_handler()
http://source.winehq.org/git/wine.git/?a=blob;f=dlls/ntdll/tests/exception.c;h=701b6bba091dddce724075dc41dd2ab407f28ac4;hb=HEAD#l559
A regression test gives:
0c4519e825c9e2b6a8310deff8582f8c35bfbba9 is the first bad commit
commit 0c4519e825c9e2b6a8310deff8582f8c35bfbba9
Author: Frederic Weisbecker <fweisbec at gmail.com>
Date: Thu Jun 24 21:21:27 2010 +0200
x86: Set resume bit before returning from breakpoint exception
Instruction breakpoints trigger before the instruction executes,
and returning back from the breakpoint handler brings us again
to the instruction that breakpointed. This naturally bring to
a breakpoint recursion.
To solve this, x86 has the Resume Bit trick. When the cpu flags
have the RF flag set, the next instruction won't trigger any
instruction breakpoint, and once this instruction is executed,
RF is cleared back.
This let's us jump back to the instruction that triggered the
breakpoint without recursion.
Use this when an instruction breakpoint triggers.
Signed-off-by: Frederic Weisbecker <fweisbec at gmail.com>
Cc: Will Deacon <will.deacon at arm.com>
Cc: Prasad <prasad at linux.vnet.ibm.com>
Cc: Mahesh Salgaonkar <mahesh at linux.vnet.ibm.com>
Cc: Paul Mackerras <paulus at samba.org>
Cc: Ingo Molnar <mingo at elte.hu>
Cc: Jason Wessel <jason.wessel at windriver.com>
:040000 040000 2eae1fa5f90b141d60fe28a971a618e9c0b1a232
e27fd94a1beb10e3688d555bd74c888b6a310293 M arch
Reverting this patch on top of 2.6.36 makes the problem go away.
The problem is 100% reproducible and I see it in 32bit Wine on both 32bit and
64bit Linux. The regression might affect also 64bit Wine but those exception
tests aren't compiled for 64bit Wine due to the lack of compiler support for
win64 exceptions; so we cannot test it.
Steps to reproduce:
-------------------
Any Wine version wine-1.2 or newer will do (either a tarball from
http://sourceforge.net/projects/wine/files/Source/ or from git
git clone git://source.winehq.org/git/wine.git
./configure
make
cd dlls/ntdll/tests/
make exception.ok
There should be no errors and exception.ok should have been created.
For builds on a x86_64 machine a pretty complete 32bit setup is needed too, but
configure will complain. If configure doesn't errors out but produces warnings,
those can be safely ignored. It means the dependencies are optional and those
aren't needed to reproduce this bug. http://wiki.winehq.org/WineOn64bit has a
lot more info on setting this up.
--
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the Bugme-new
mailing list