Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-17 Thread Paolo Bonzini
Il 13/06/2014 23:27, Eric Blake ha scritto: +# @RTC_CHANGE +# +# Emitted when the guest changes the RTC time. +# +# @offset: offset between base RTC clock (as specified by -rtc base), and +# new RTC clock value +# +# Since: 2.1 0.14.0 0.13.0 :)

Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-17 Thread Eric Blake
On 06/17/2014 03:21 AM, Paolo Bonzini wrote: Il 13/06/2014 23:27, Eric Blake ha scritto: +# @RTC_CHANGE +# +# Emitted when the guest changes the RTC time. +# +# @offset: offset between base RTC clock (as specified by -rtc base), and +# new RTC clock value +# +# Since:

Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-15 Thread Paolo Bonzini
Il 13/06/2014 23:27, Eric Blake ha scritto: visit_start_struct(v, NULL, , RTC_CHANGE, 0, local_err); if (local_err) { goto clean; } Hmm, qmp_output_start_struct() never sets errp. visit_type_int(v, offset, offset, local_err); if (local_err) { goto

Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-15 Thread Paolo Bonzini
Il 15/06/2014 02:38, Wenchao Xia ha scritto: Once again, all callers of qapi_event_send_rtc_change() are passing a NULL errp to silently ignore errors; and I just audited that no errors happen anyways. Fixing it. No, please don't. I prefer the way you did it in v6. Paolo

Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-14 Thread Wenchao Xia
于 2014/6/14 5:27, Eric Blake 写道: On 06/05/2014 06:22 AM, Wenchao Xia wrote: This patch also eliminates build time warning caused by no caller of monitor_qapi_event_throttle(). Again, my suggestion on 6/29 could avoid that warning; if you use that workaround, don't clean it until 29/29, but

Re: [Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-13 Thread Eric Blake
On 06/05/2014 06:22 AM, Wenchao Xia wrote: This patch also eliminates build time warning caused by no caller of monitor_qapi_event_throttle(). Again, my suggestion on 6/29 could avoid that warning; if you use that workaround, don't clean it until 29/29, but you can drop this paragraph of this

[Qemu-devel] [PATCH V6 16/29] qapi event: convert RTC_CHANGE

2014-06-05 Thread Wenchao Xia
This patch also eliminates build time warning caused by no caller of monitor_qapi_event_throttle(). Signed-off-by: Wenchao Xia wenchaoq...@gmail.com --- docs/qmp/qmp-events.txt | 16 hw/ppc/spapr_rtas.c |3 ++- hw/timer/mc146818rtc.c |3 ++-