All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Brook <paul@codesourcery.com>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>, qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH 4/4] gdbstub: x86: Switch 64/32 bit registers dynamically
Date: Mon, 29 Jun 2009 14:01:59 +0100	[thread overview]
Message-ID: <200906291402.00530.paul@codesourcery.com> (raw)
In-Reply-To: <20090627075351.13376.20106.stgit@mchn012c.ww002.siemens.net>

On Saturday 27 June 2009, Jan Kiszka wrote:
> Commit 56aebc891674cd2d07b3f64183415697be200084 changed gdbstub in way
> that debugging 32 or 16-bit guest code is no longer possible with qemu
> for x86_64 guest CPUs. Since that commit, qemu only provides registers
> sets for 64-bit, forcing current and foreseeable gdb to also switch its
> architecture to 64-bit. And this breaks if the inferior is 32 or 16 bit.
>
> No question, this is a gdb issue. But, as it was confirmed in several
> discusssions with gdb people, it is a non-trivial thing to fix. So until
> qemu finds a gdb version attach with a rework x86 support, we have to
> work around it by switching the register layout as the guest switches
> its execution mode between 16/32 and 64 bit.

I still object to this patch. Especially as there's no indication that it's a 
harroble hack to workaround broken GDB.

Paul

  reply	other threads:[~2009-06-29 13:02 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-27  7:53 [Qemu-devel] [RESEND][PATCH 0/4] Long pending gdbstub patches Jan Kiszka
2009-06-27  7:53 ` [Qemu-devel] [PATCH 3/4] gdbstub: x86: Support for setting segment registers Jan Kiszka
2009-06-27  7:53 ` [Qemu-devel] [PATCH 4/4] gdbstub: x86: Switch 64/32 bit registers dynamically Jan Kiszka
2009-06-29 13:01   ` Paul Brook [this message]
2009-06-29 13:42     ` [Qemu-devel] " Jan Kiszka
2009-06-29 14:07       ` Paul Brook
2009-06-29 14:22         ` Jan Kiszka
2009-06-29 14:43           ` Paul Brook
2009-06-29 14:53             ` Jan Kiszka
2009-06-29 15:16               ` Daniel Jacobowitz
2009-06-29 15:36                 ` Jan Kiszka
2009-06-29 22:00                   ` Jamie Lokier
2009-06-30 11:54                     ` Jan Kiszka
2009-06-30  7:15             ` Gerd Hoffmann
2009-06-30 12:00               ` Jan Kiszka
2009-06-29 14:51           ` Paul Brook
2009-06-27  7:53 ` [Qemu-devel] [PATCH 2/4] gdbstub: x86: Refactor register access Jan Kiszka
2009-06-27  7:53 ` [Qemu-devel] [PATCH 1/4] gdbstub: Add vCont support Jan Kiszka

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=200906291402.00530.paul@codesourcery.com \
    --to=paul@codesourcery.com \
    --cc=aliguori@us.ibm.com \
    --cc=jan.kiszka@siemens.com \
    --cc=qemu-devel@nongnu.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.