From: Frederic Weisbecker <fweisbec@gmail.com>
To: Michael Stefaniuc <mstefani@redhat.com>
Cc: "K.Prasad" <prasad@linux.vnet.ibm.com>,
Alan Stern <stern@rowland.harvard.edu>,
linux-kernel@vger.kernel.org,
Maneesh Soni <maneesh@linux.vnet.ibm.com>,
Alexandre Julliard <julliard@winehq.org>,
"Rafael J. Wysocki" <rjw@sisk.pl>,
Maciej Rutecki <maciej.rutecki@gmail.com>
Subject: Re: Regression in ptrace (Wine) starting with 2.6.33-rc1
Date: Thu, 11 Feb 2010 19:22:26 +0100 [thread overview]
Message-ID: <20100211182224.GC4915@nowhere> (raw)
In-Reply-To: <4B743149.4000707@redhat.com>
On Thu, Feb 11, 2010 at 05:33:13PM +0100, Michael Stefaniuc wrote:
> Hello!
>
> 2.6.33-rc1 broke ptrace for Wine, specifically the setting of the debug
> registers. This is visible in the Wine ntdll exception tests failing on
> 2.6.33-rcX while they work just fine in 2.6.32.
>
> A regression test resulted in:
> 72f674d203cd230426437cdcf7dd6f681dad8b0d is the first bad commit
> commit 72f674d203cd230426437cdcf7dd6f681dad8b0d
> Author: K.Prasad <prasad@linux.vnet.ibm.com>
> Date: Mon Jun 1 23:45:48 2009 +0530
>
> hw-breakpoints: modify Ptrace routines to access breakpoint registers
>
> This patch modifies the ptrace code to use the new wrapper routines
> around
> the
> debug/breakpoint registers.
>
> [ Impact: adapt x86 ptrace to the new breakpoint Api ]
>
> Original-patch-by: Alan Stern <stern@rowland.harvard.edu>
> Signed-off-by: K.Prasad <prasad@linux.vnet.ibm.com>
> Signed-off-by: Maneesh Soni <maneesh@linux.vnet.ibm.com>
> Reviewed-by: Alan Stern <stern@rowland.harvard.edu>
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
>
> :040000 040000 f72ff4760c3fa1dffcd72494e77bee2c76039505
> b60d5fe2088ff635568e800d5759a0b373b5e439 M arch
>
>
> The first ntdll exception test in test_exceptions()
> http://source.winehq.org/git/wine.git/?a=blob;f=dlls/ntdll/tests/exception.c;h=9149b6961764dec31a0af5cd3b93ab3072703dbb;hb=312e4f6b235a468f8bf764101a5b97cf34dd4143#l594
> run_exception_test(dreg_handler, NULL, &segfault_code,
> sizeof(segfault_code),
> 0);
> produces (make exception.ok) the output:
> err:seh:setup_exception_record stack overflow 932 bytes in thread 0009 eip
> 7bc3c97f esp 00240f8c stack 0x240000-0x241000-0x340000
> The stack overflow is detected by the ntdll internal function
> setup_exception_record()
> http://source.winehq.org/git/wine.git/?a=blob;f=dlls/ntdll/signal_i386.c;h=4eccb61954c43d75144575411313d59405decfc3;hb=312e4f6b235a468f8bf764101a5b97cf34dd4143#l1495
> which aborts the thread.
> The problem happens on both i386 (Intel Atom CPU) as well as on x86_64
> (Intel Q9450); the stack overflow bytes differ though but are always the
> same for each box.
>
> All the ntdll exception tests run just fine with 2.6.32 and older
> kernels. For a summary of the ntdll exception tests please see
> http://test.winehq.org/data/tests/ntdll:exception.html in the Wine
> column.
>
> I have opened also http://bugzilla.kernel.org/show_bug.cgi?id=15273 for
> this.
>
> thanks
> bye
> michael
Thanks a lot for your report. Is there an easy way to reproduce
this?
next prev parent reply other threads:[~2010-02-11 18:22 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-11 16:33 Regression in ptrace (Wine) starting with 2.6.33-rc1 Michael Stefaniuc
2010-02-11 18:22 ` Frederic Weisbecker [this message]
2010-02-11 19:49 ` Michael Stefaniuc
2010-02-12 18:15 ` Frederic Weisbecker
2010-02-13 17:33 ` K.Prasad
2010-02-13 21:29 ` Michael Stefaniuc
2010-02-14 17:15 ` Frederic Weisbecker
2010-02-14 20:13 ` Michael Stefaniuc
2010-02-14 20:41 ` Frederic Weisbecker
2010-02-14 23:05 ` Michael Stefaniuc
2010-02-15 11:57 ` K.Prasad
2010-02-15 15:57 ` Alexandre Julliard
2010-02-15 19:37 ` Michael Stefaniuc
2010-02-15 19:47 ` Roland McGrath
2010-02-17 16:03 ` Frederic Weisbecker
2010-02-17 17:06 ` Frederic Weisbecker
2010-02-18 17:59 ` Regression in ptrace (Wine) starting with 2.6.33-rc1, fixes Frederic Weisbecker
2010-02-18 19:27 ` Michael Stefaniuc
2010-02-18 19:41 ` Alexandre Julliard
2010-02-19 17:19 ` Frederic Weisbecker
2010-02-19 17:17 ` Frederic Weisbecker
2010-02-18 18:00 ` [PATCH 1/2] hw-breakpoints: Accept breakpoints on NULL address Frederic Weisbecker
2010-02-18 21:16 ` Roland McGrath
2010-02-19 17:38 ` Frederic Weisbecker
2010-02-19 8:51 ` K.Prasad
2010-02-18 18:00 ` [PATCH 2/2] hw-breakpoint: Keep track of dr7 local enable bits Frederic Weisbecker
2010-02-19 8:45 ` K.Prasad
2010-02-19 15:34 ` Frederic Weisbecker
2010-02-19 17:58 ` K.Prasad
2010-02-19 18:03 ` Frederic Weisbecker
2010-02-19 8:58 ` K.Prasad
2010-02-19 15:49 ` Frederic Weisbecker
2010-02-19 17:41 ` Frederic Weisbecker
2010-02-19 18:04 ` K.Prasad
2010-02-19 18:12 ` [GIT PULL] hw-breakpoint regression fixes Frederic Weisbecker
2010-02-22 9:56 ` Ingo Molnar
2010-02-19 18:12 ` [PATCH 1/2] hw-breakpoints: Accept breakpoints on NULL address Frederic Weisbecker
2010-02-19 18:12 ` [PATCH 2/2] hw-breakpoint: Keep track of dr7 local enable bits Frederic Weisbecker
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=20100211182224.GC4915@nowhere \
--to=fweisbec@gmail.com \
--cc=julliard@winehq.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maciej.rutecki@gmail.com \
--cc=maneesh@linux.vnet.ibm.com \
--cc=mstefani@redhat.com \
--cc=prasad@linux.vnet.ibm.com \
--cc=rjw@sisk.pl \
--cc=stern@rowland.harvard.edu \
/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.