On 3/20/23 16:34, Philippe Mathieu-Daudé wrote:
> On 20/3/23 16:23, Claudio Fontana wrote:
>> Hi Alex, all,
>>
>> again, this moves TCG-only code to common code, no?
>
> Oh, good point.
>
>> Even if this happens to work, the idea is to avoid adding unneeded accel TCG
>> code to a KVM-only binary
Claudio Fontana writes:
> Hi Alex, all,
>
> again, this moves TCG-only code to common code, no?
>
> Even if this happens to work, the idea is to avoid adding unneeded accel TCG
> code to a KVM-only binary.
>
> We need to keep in mind all dimensions when we do refactorings:
>
> user-mode vs sys
Philippe Mathieu-Daudé writes:
> On 20/3/23 11:10, Alex Bennée wrote:
>> We don't want to be polluting the core run loop code with target
>> specific handling, punt it to sysemu_ops where it belongs.
>> Signed-off-by: Alex Bennée
>> ---
>> include/hw/core/sysemu-cpu-ops.h | 5 +
>> tar
On 20/3/23 16:23, Claudio Fontana wrote:
Hi Alex, all,
again, this moves TCG-only code to common code, no?
Oh, good point.
Even if this happens to work, the idea is to avoid adding unneeded accel TCG
code to a KVM-only binary.
Could yet another AccelSysemuCPUOps *accel struct in SysemuCPU
Hi Alex, all,
again, this moves TCG-only code to common code, no?
Even if this happens to work, the idea is to avoid adding unneeded accel TCG
code to a KVM-only binary.
We need to keep in mind all dimensions when we do refactorings:
user-mode vs sysemu,
the architecture,
the accel, in particu
On 20/3/23 11:10, Alex Bennée wrote:
We don't want to be polluting the core run loop code with target
specific handling, punt it to sysemu_ops where it belongs.
Signed-off-by: Alex Bennée
---
include/hw/core/sysemu-cpu-ops.h | 5 +
target/i386/cpu-internal.h | 1 +
accel/tcg/cpu