On 12.02.19 16:58, Cornelia Huck wrote:
> On Tue, 12 Feb 2019 16:52:09 +0100
> Philippe Mathieu-Daudé <phi...@redhat.com> wrote:
>
>> Hi Cornelia,
>>
>> On 2/12/19 4:30 PM, Cornelia Huck wrote:
>>> Trace when assigning/unassigning.
>>>
>>> Signed-off-by: Cornelia Huck <coh...@redhat.com>
>>> ---
>>> target/s390x/kvm.c | 2 ++
>>> target/s390x/trace-events | 1 +
>>> 2 files changed, 3 insertions(+)
>>>
>>> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
>>> index 15fdc168e1c5..7d61bd109092 100644
>>> --- a/target/s390x/kvm.c
>>> +++ b/target/s390x/kvm.c
>>> @@ -1886,6 +1886,8 @@ int kvm_s390_assign_subch_ioeventfd(EventNotifier
>>> *notifier, uint32_t sch,
>>> .addr = sch,
>>> .len = 8,
>>> };
>>> + trace_kvm_s390_assign_subch_ioeventfd(kick.fd, kick.addr, assign,
>>> + kick.datamatch);
>>> if (!kvm_check_extension(kvm_state, KVM_CAP_IOEVENTFD)) {
>>> return -ENOSYS;
>>> }
>>> diff --git a/target/s390x/trace-events b/target/s390x/trace-events
>>> index a84e316e4937..bdc22f42cdbb 100644
>>> --- a/target/s390x/trace-events
>>> +++ b/target/s390x/trace-events
>>> @@ -14,6 +14,7 @@ ioinst_chsc_cmd(uint16_t cmd, uint16_t len) "IOINST: chsc
>>> command 0x%04x, len 0x
>>> kvm_enable_cmma(int rc) "CMMA: enabling with result code %d"
>>> kvm_clear_cmma(int rc) "CMMA: clearing with result code %d"
>>> kvm_failed_cpu_state_set(int cpu_index, uint8_t state, const char *msg)
>>> "Warning: Unable to set cpu %d state %" PRIu8 " to KVM: %s"
>>> +kvm_s390_assign_subch_ioeventfd(int fd, uint32_t addr, bool assign, int
>>> datamatch) "fd: %d sch: @0x%x assign: %d vq: %d"
>>
>> I noticed all s390x related trace events don't specify 's390' in the
>> event name, maybe you can simply strip it. If you agree, feel free to
>> apply that change directly yourself, no need for v2 ;)
>
> Yeah; not exactly sure why the other events do that, though... I can
> strip it if others agree.
Yes, let's keep it consistent. Or if you feel like crafting patches,
rename the others ;)
>
>>
>> Regardless the name used:
>> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>
> Thanks!
>
>>
>>>
>>> # target/s390x/cpu.c
>>> cpu_set_state(int cpu_index, uint8_t state) "setting cpu %d state to %"
>>> PRIu8
>>>
>
--
Thanks,
David / dhildenb
