[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: Missing singlestep for already-translated code?
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] Re: Missing singlestep for already-translated code? |
Date: |
Thu, 15 Apr 2010 12:13:46 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Aurelien Jarno wrote:
> Jun Koi a écrit :
>> On Wed, Apr 14, 2010 at 12:28 AM, Jan Kiszka <address@hidden> wrote:
>>> Alexander Graf wrote:
>>>> On 13.04.2010, at 15:36, Jan Kiszka wrote:
>>>>
>>>>> Jun Koi wrote:
>>>>>> Hi,
>>>>>>
>>>>>> I am looking into the singlestep command in monitor interface, and it
>>>>>> seems that we only take into account the singlestep flag when we are
>>>>>> translating code.
>>>>>> So for the already-translated code, we will miss singlestep?
>>>>> This feature is broken. For TCG, it should at least flush the
>>>>> translation buffer, and for KVM it has to enable single-stepping in the
>>>>> kernel. That's what happens automatically when you call cpu_single_step.
>>>>> I guess 'singlestep' wants to be somehow orthogonal to this. But this is
>>>>> the wrong approach.
>>>>>
>>>>> Does anyone actually used this feature or still does so? It looks fairly
>>>>> redundant to me, kind of a poor-man's gdb front-end as part of the
>>>>> monitor console.
>>>> Not sure what it does, but I use -singlestep quite a lot to get register
>>>> dumps for instructions when using -d cpu.
>>> Ah, "singlestep" is not about stopping the VM after each instruction but
>>> about limiting the TB length to a single instruction. Badly named and
>>> poorly documented.
>>>
>>> In that case, the dynamic switch should already be fine by adding a
>>> tb_flush() on enable. Still, someone should also patch at least the docs.
>>>
>
> What's the real point of flushing the tb to get it retranslated again?
> It will be retranslated in the exact same way.
Nope. AFAIU, 'singlestep' will enforce single-instruction TBs.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] Missing singlestep for already-translated code?, Jun Koi, 2010/04/13
- [Qemu-devel] Re: Missing singlestep for already-translated code?, takasi-y, 2010/04/13
- [Qemu-devel] Re: Missing singlestep for already-translated code?, Jan Kiszka, 2010/04/13
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Alexander Graf, 2010/04/13
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Jan Kiszka, 2010/04/13
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Jun Koi, 2010/04/15
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Jan Kiszka, 2010/04/15
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Aurelien Jarno, 2010/04/15
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?,
Jan Kiszka <=
- Re: [Qemu-devel] Re: Missing singlestep for already-translated code?, Aurelien Jarno, 2010/04/15