Re: [PATCH 1/5] sysemu: add set_virtual_time to accel ops

[Pierrick Bouvier <pierrick.bouvier@linaro.org>]

On 5/29/24 05:29, Philippe Mathieu-Daudé wrote:
> On 17/5/24 00:20, Pierrick Bouvier wrote:
>> From: Alex Bennée <alex.bennee@linaro.org>
>>
>> We are about to remove direct calls to individual accelerators for
>> this information and will need a central point for plugins to hook
>> into time changes.
>>
>> From: Alex Bennée <alex.bennee@linaro.org>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> ---
>>    include/sysemu/accel-ops.h                     | 18 +++++++++++++++++-
>>    include/sysemu/cpu-timers.h                    |  3 ++-
>>    ...et-virtual-clock.c => cpus-virtual-clock.c} |  5 +++++
>>    system/cpus.c                                  | 11 +++++++++++
>>    stubs/meson.build                              |  6 +++++-
>>    5 files changed, 40 insertions(+), 3 deletions(-)
> 
> 
>> diff --git a/stubs/meson.build b/stubs/meson.build
>> index 3b9d42023cb..672213b7482 100644
>> --- a/stubs/meson.build
>> +++ b/stubs/meson.build
>> @@ -3,6 +3,11 @@
>>    # below, so that it is clear who needs the stubbed functionality.
>>    
>>    stub_ss.add(files('cpu-get-clock.c'))
>> +stub_ss.add(files('cpus-virtual-clock.c'))
> 
> Are these lines <...
> 
>> +stub_ss.add(files('qemu-timer-notify-cb.c'))
>> +stub_ss.add(files('icount.c'))
>> +stub_ss.add(files('dump.c'))
>> +stub_ss.add(files('error-printf.c'))
> 
> ...> due to a failed git-rebase?
> 

You're right, fixed this!

>>    stub_ss.add(files('fdset.c'))
>>    stub_ss.add(files('iothread-lock.c'))
>>    stub_ss.add(files('is-daemonized.c'))
>> @@ -28,7 +33,6 @@ endif
>>    if have_block or have_ga
>>      stub_ss.add(files('replay-tools.c'))
>>      # stubs for hooks in util/main-loop.c, util/async.c etc.
>> -  stub_ss.add(files('cpus-get-virtual-clock.c'))
>>      stub_ss.add(files('icount.c'))
>>      stub_ss.add(files('graph-lock.c'))
>>      if linux_io_uring.found()
>