From: Claudio Fontana <claudio.fontana@huawei.com>
To: Richard Henderson <rth@twiddle.net>
Cc: Peter Maydell <peter.maydell@linaro.org>,
proljc@gmail.com, Claudio Fontana <claudio.fontana@gmail.com>,
qemu-devel@nongnu.org, Max Filippov <jcmvbkbc@gmail.com>,
gxt@mprc.pku.edu.cn
Subject: Re: [Qemu-devel] [PATCH 0/2] Disassembly with external objdump
Date: Tue, 3 Sep 2013 15:18:20 +0200 [thread overview]
Message-ID: <5225E19C.1040708@huawei.com> (raw)
In-Reply-To: <520698E5.4080409@twiddle.net>
Hi, resuming this conversation about external objdump,
On 10.08.2013 21:47, Richard Henderson wrote:
> On 08/10/2013 06:53 AM, Max Filippov wrote:
>> On Sat, Aug 10, 2013 at 8:42 PM, Richard Henderson <rth@twiddle.net> wrote:
>>> On 08/10/2013 05:45 AM, Peter Maydell wrote:
>>>> Well, it depends. If we're going to dump all the in-tree
>>>> disassemblers and always use an external objdump, that's fine.
>>>
>>> It's tempting, given that the only internal disassemblers that
>>> are not missing opcodes are for the extinct cpus.
>>
>> Will that leave those CPUs without x/i support from monitor?
>
> Yes.
>
> One could always have qemu fork to objdump too. Though finding the
> right one could be tricky without help.
I lost track of what you mean here.. that we could implement x/i support also using an external objdump?
I got my first actual run of a qemu with integrated libvixl from ARM for disassembly output of aarch64 code, and the results are substandard.
Many instructions are missing, in particular all PC relative instructions, and the README warns about multiple other problems.
It also introduces dependencies from scons, C++, and can only build and run on 64bit LP64 hosts.
Using external objdump would be very much better for aarch64.
What about making the external objdump the first class citizen here, and removing all the old stuff? Or just keeping it as fallback?
>
>> Does anybody use it?
>
> That I can't answer.
As I started the aarch64 tcg task I just used gdb's disassembly feature for achieving that goal.
However, since the feature is already there and people rely on it, maybe it could be done with external objdump as well?
>
>
> r~
Ciao,
Claudio
next prev parent reply other threads:[~2013-09-03 13:19 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-09 19:19 [Qemu-devel] [PATCH 0/2] Disassembly with external objdump Richard Henderson
2013-08-09 19:19 ` [Qemu-devel] [PATCH 1/2] disas: Implement fallback to dump object code as hex Richard Henderson
2013-08-09 19:19 ` [Qemu-devel] [PATCH 2/2] disas: Add disas-objdump.pl Richard Henderson
2013-08-10 4:08 ` Max Filippov
2013-08-10 10:39 ` Richard Henderson
2013-08-10 1:54 ` [Qemu-devel] [PATCH 0/2] Disassembly with external objdump Jia Liu
2013-08-10 9:16 ` Peter Maydell
2013-08-10 10:22 ` Claudio Fontana
2013-08-10 15:45 ` Peter Maydell
2013-08-10 16:42 ` Richard Henderson
2013-08-10 16:53 ` Max Filippov
2013-08-10 19:47 ` Richard Henderson
2013-09-03 13:18 ` Claudio Fontana [this message]
2013-08-11 8:59 ` Blue Swirl
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=5225E19C.1040708@huawei.com \
--to=claudio.fontana@huawei.com \
--cc=claudio.fontana@gmail.com \
--cc=gxt@mprc.pku.edu.cn \
--cc=jcmvbkbc@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=proljc@gmail.com \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).