- Subject: Re: [PATCH 1/3v2] kvm-s390: Implement the directed yield (diag 9c) hypervisor call for KVM
- From: Christian Borntraeger <borntraeger@xxxxxxxxxx>
- Date: Tue, 24 Apr 2012 17:47:34 +0200
- Cc: Marcelo Tossati <mtosatti@xxxxxxxxxx>, Carsten Otte <cotte@xxxxxxxxxx>, Alexander Graf <agraf@xxxxxxx>, Jens Freimann <jfrei@xxxxxxxxxxxxxxxxxx>, Cornelia Huck <cornelia.huck@xxxxxxxxxx>, Heiko Carstens <heiko.carstens@xxxxxxxxxx>, Martin Schwidefsky <schwidefsky@xxxxxxxxxx>, Heinz Graalfs <graalfs@xxxxxxxxxxxxxxxxxx>, KVM <kvm@xxxxxxxxxxxxxxx>, Konstantin Weitz <WEITZKON@xxxxxxxxxx>
- In-reply-to: <4F96A55E.1050703@redhat.com>
- References: <4F96A021.4000704@de.ibm.com> <1335272114-64505-1-git-send-email-borntraeger@de.ibm.com> <4F96A55E.1050703@redhat.com>
- User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.28) Gecko/20120313 Lightning/1.0b2 Thunderbird/3.1.20
> I think the code will me more readable, and less obvious that is was
> copied from kvm_vcpu_on_spin(), if you put all the processing outside
> the loop, except for matching the vpu itself.
>
> So the code reads
>
> find a vcpu
> obtain the task
> do the yield
>
> instead of looking like you're doing the processing for every vcpu. The
> loop is just a slow lookup which might some day be replaced by a table
> lookup.
Ok. We might also have a kvm_vcpu_on_spin_directed in common code,
Would you prefer that?
Christian
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[KVM ARM]
[KVM ia64]
[KVM ppc]
[Spice Development]
[Libvirt]
[Libvirt Users]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Linux Kernel]
[Linux SCSI]
[XFree86]