On Tue, 12 Feb 2019 17:12:33 +0100 David Hildenbrand <da...@redhat.com> wrote:
> 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 ;) I just went with the path of least resistance and removed the 's390' :) Any acks before I queue this?