From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Hlo1J-0003IY-H5 for qemu-devel@nongnu.org; Wed, 09 May 2007 11:24:01 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Hlo1H-0003II-Vk for qemu-devel@nongnu.org; Wed, 09 May 2007 11:24:00 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Hlo1H-0003IF-Q8 for qemu-devel@nongnu.org; Wed, 09 May 2007 11:23:59 -0400 Received: from nz-out-0506.google.com ([64.233.162.239]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Hlnu4-00063w-An for qemu-devel@nongnu.org; Wed, 09 May 2007 11:16:32 -0400 Received: by nz-out-0506.google.com with SMTP id i11so491378nzi for ; Wed, 09 May 2007 08:16:32 -0700 (PDT) Message-ID: <1fa17f810705090816h3b3a5e2o1c3bc1036a859faa@mail.gmail.com> Date: Wed, 9 May 2007 23:16:31 +0800 From: MingyanGuo In-Reply-To: <4641D0AD.8060500@web.de> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_70796_27774950.1178723791242" References: <1fa17f810705090611o3355056hd9782dcca3f89ef8@mail.gmail.com> <4641D0AD.8060500@web.de> Subject: [Qemu-devel] Re: Could any body fix the bug in gdbstub.c for x86_64? Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: qemu-devel@nongnu.org ------=_Part_70796_27774950.1178723791242 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline On 5/9/07, Jan Kiszka wrote: > > MingyanGuo wrote: > > Hi, > > The -s option for qemu-system-x86_64 is not useable. > > I've already post a patch, but it is ignored. > > Have you seen my patch [1] on this already? Does it work for you? It's > running fine here, but there are still some open questions [2], and I > unfortunately didn't received feedback on my last revisions as well. > > I haven't looked in details at your approach, but you stumbled over the > same thing as I did, that redefining the register constants would > simplify things significantly - if this is fine with the rest of qemu. I read your patch now, our approaches are the same I think, just in different forms. I don't know if there are any adverse side effects to redefine the register constants (see my comments in the patch), so a simple register mapping is a quick fix :-) > http://lists.gnu.org/archive/html/qemu-devel/2007-05/msg00062.html > > This patch is for version 0.9.0, could some body merge it > > in the cvs tree? > > > > However the solution will finally look like, a fix for the next release > would be welcome here as well! One just need to know which way is > preferred... > > Jan > > > [1] http://lists.gnu.org/archive/html/qemu-devel/2007-04/msg00554.html > [2] http://lists.gnu.org/archive/html/qemu-devel/2007-04/msg00524.html > > > I should cancel my patch now, as I am debugging an OS kernel, no xmms/fpu registers considered, and your patch is better. :D Regards, MingyanGuo -- Three passions, simple but overwhelmingly strong, have governed my life: the longing for love, the search for knowledge, and unbearable pity for the suffering of mankind. ---------Bertrand Russell ------=_Part_70796_27774950.1178723791242 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline On 5/9/07, Jan Kiszka <jan.kiszka@web.de> wrote:
MingyanGuo wrote:
> Hi,
> The -s option for qemu-system-x86_64 is not useable.
> I've already post a patch, but it is ignored.

Have you seen my patch [1] on this already? Does it work for you? It's
running fine here, but there are still some open questions [2], and I
unfortunately didn't received feedback on my last revisions as well.

I haven't looked in details at your approach, but you stumbled over the
same thing as I did, that redefining the register constants would
simplify things significantly - if this is fine with the rest of qemu.

I read your patch now, our approaches are the same I think, just in different
forms. I don't know if there are any adverse side effects to redefine the register
constants (see my comments in the patch), so a simple register mapping is a
quick fix :-)

> http://lists.gnu.org/archive/html/qemu-devel/2007-05/msg00062.html
> This patch is for version 0.9.0, could some body merge it
> in the cvs tree?
>

However the solution will finally look like, a fix for the next release
would be welcome here as well! One just need to know which way is
preferred...

Jan


[1] http://lists.gnu.org/archive/html/qemu-devel/2007-04/msg00554.html
[2] http://lists.gnu.org/archive/html/qemu-devel/2007-04/msg00524.html



I should cancel my patch now, as I am debugging an OS kernel, no xmms/fpu
registers considered, and your patch is better. :D

Regards,
MingyanGuo

--
Three passions, simple but overwhelmingly strong, have governed my life:
the longing for love, the search for knowledge, and unbearable pity for
the suffering of mankind.
                         ---------Bertrand Russell
------=_Part_70796_27774950.1178723791242--