Re: [PATCH v2] KVM: s390: add vcpu stat counters for many instruction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 24 Jan 2018 16:50:54 +0100
Christian Borntraeger <borntraeger@xxxxxxxxxx> wrote:

> The overall instruction counter is larger than the sum of the
> single counters. We should try to catch all instruction handlers
> to make this match the summary counter.
> Let us add sck,tb,sske,iske,rrbe,tb,tpi,tsch,lpsw,pswe....
> and remove other unused ones.

This is rebased on top of the diagnose patch, right?

> 
> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
> Acked-by: Janosch Frank <frankja@xxxxxxxxxxxxxxxxxx>
> Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>
> ---
>  arch/s390/include/asm/kvm_host.h | 21 +++++++++++++++++----
>  arch/s390/kvm/diag.c             |  1 +
>  arch/s390/kvm/kvm-s390.c         | 21 +++++++++++++++++----
>  arch/s390/kvm/priv.c             | 32 ++++++++++++++++++++++++++++++--
>  4 files changed, 65 insertions(+), 10 deletions(-)
> 

> @@ -527,6 +544,7 @@ static int handle_io_inst(struct kvm_vcpu *vcpu)
>  		if (vcpu->arch.sie_block->ipa == 0xb235)
>  			return handle_tsch(vcpu);
>  		/* Handle in userspace. */
> +		vcpu->stat.instruction_io_other++;
>  		return -EOPNOTSUPP;
>  	} else {
>  		/*

As an aside, we don't count I/O instructions if userspace did not
enable css support. But nobody does that with modern kernels anyway :)

Reviewed-by: Cornelia Huck <cohuck@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-s390" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux