[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/23] qom: Add cpu_exec_enter and cpu_exec_exit
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 01/23] qom: Add cpu_exec_enter and cpu_exec_exit hooks |
Date: |
Thu, 25 Sep 2014 19:03:21 +0100 |
On 17 September 2014 16:22, Richard Henderson <address@hidden> wrote:
> On 09/17/2014 04:54 AM, Andreas Färber wrote:
>> I am fine with adding such hooks, but please let's use a better, active
>> name. CPUClass is the struct name and cpu-exec is a file name; the hook
>> should say what it's doing, not where the code (used to) live(s). Just
>> exec_enter/exec_exit possibly?
>
> cpu_exec is also the function name from which they're invoked, not just the
> file name.
>
> I can make the name change if you really prefer, but (perhaps obviously) I
> kinda like the current naming.
The current name doesn't seem particularly bad to me, and
in the absence of further discussion here I assume you
don't object that strongly, Andreas?
Since this patchset unavoidably touches a pile of the
target-* code (and it will conflict with at least one
patchset I want to fixup-and-queue for target-arm.next),
I'd like to commit it direct to master so we don't end
up having to respin it or fix up collisions. Or do you
have a strong preference for taking it through the
qom tree?
thanks
-- PMM
- [Qemu-devel] [PATCH 00/23] qom hooks to clean up cpu_exec, Richard Henderson, 2014/09/13
- [Qemu-devel] [PATCH 02/23] cpu-exec: Remove do-nothing ifdef chains, Richard Henderson, 2014/09/13
- [Qemu-devel] [PATCH 03/23] target-i386: Use cpu_exec_enter/exit qom hooks, Richard Henderson, 2014/09/13
- [Qemu-devel] [PATCH 04/23] target-m68k: Use cpu_exec_enter/exit qom hooks, Richard Henderson, 2014/09/13
- [Qemu-devel] [PATCH 05/23] target-ppc: Use cpu_exec_enter qom hook, Richard Henderson, 2014/09/13
[Qemu-devel] [PATCH 06/23] qom: Add cpu_exec_interrupt hook, Richard Henderson, 2014/09/13