[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI devic
From: |
Alexander Graf |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment |
Date: |
Sat, 8 Sep 2012 14:13:05 +0200 |
On 08.09.2012, at 12:16, Blue Swirl <address@hidden> wrote:
> On Sat, Sep 8, 2012 at 9:28 AM, Alexander Graf <address@hidden> wrote:
>>
>>
>> On 08.09.2012, at 10:06, Blue Swirl <address@hidden> wrote:
>>
>>> On Thu, Sep 6, 2012 at 8:44 AM, Avi Kivity <address@hidden> wrote:
>>>> On 09/05/2012 10:04 PM, Blue Swirl wrote:
>>>>>
>>>>> Reinventing a disassembler for ever growing x86 assembly is
>>>>> no fun.
>>>>
>>>> We can try linking to a disassembler library. I use udis86 to
>>>> disassemble instructions in kvm tracepoints
>>>> (http://udis86.git.sourceforge.net/git/gitweb.cgi?p=udis86/udis86;a=shortlog),
>>>> it's maintained but not heavily so.
>>>
>>> I think commonality with KVM would be preferred. The library looks
>>> neat and based on changelog, more actively developed than BSD DDB.
>>>
>>>>
>>>> Of course for non-x86 we'd need to continue using binutils; this is
>>>> about copying code vs. libraries, not about licensing.
>>>
>>> For most architectures, pre-GPLv3 binutils is good enough since the
>>> instruction set does not change anymore. Maybe only PPC and Sparc64
>>> still change besides x86. New CPUs types more recent than 2007 will
>>> have problems.
>>
>> Alternatively we could try to run the disassembler in a different process,
>> right?
>
> For qemu.log this would be doable and even improve performance since
> only binary data would be transferred.
>
> But for monitor disassembly command x/i it may be too clumsy.
Why would it be clumsy? We'd have to make sure we are communicating
synchronously with the daemon, but apart from that it shouldn't be too
different from the log, no?
> There's
> some overlap with GDB support, so maybe we could deprecate monitor
> disassembly.
I really like the way the monitor goes through special v->p lookup, as it's a
lot easier to debug...
Alex
>
>>
>> Alex
>>
>>>
>>>>
>>>>
>>>> --
>>>> error compiling committee.c: too many arguments to function
>>>
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, (continued)
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Alexander Graf, 2012/09/05
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Blue Swirl, 2012/09/08
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Eric Blake, 2012/09/05
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Blue Swirl, 2012/09/05
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Avi Kivity, 2012/09/06
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Blue Swirl, 2012/09/08
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Alexander Graf, 2012/09/08
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Blue Swirl, 2012/09/08
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment,
Alexander Graf <=
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Blue Swirl, 2012/09/08
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 4/4] kvm: i386: Add classic PCI device assignment, Alexander Graf, 2012/09/08