[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kvm-ppc
Subject:    Re: [PATCH V2 3/5] kvm/ppc/book3s_hv: Implement halt polling in the kvm_hv kernel module
From:       Suraj Jitindar Singh <sjitindarsingh () gmail ! com>
Date:       2016-07-12 6:33:10
Message-ID: 57848F26.3060505 () gmail ! com
[Download RAW message or body]



On 12/07/16 03:26, David Matlack wrote:
> On Mon, Jul 11, 2016 at 10:07 AM, Paolo Bonzini <pbonzini@redhat.com> wrote:
>>
>> On 11/07/2016 18:57, David Matlack wrote:
>>> On Mon, Jul 11, 2016 at 12:08 AM, Suraj Jitindar Singh
>>> <sjitindarsingh@gmail.com> wrote:
>>>> This patch introduces new halt polling functionality into the kvm_hv kernel
>>>> module. When a vcore is idle it will poll for some period of time before
>>>> scheduling itself out.
>>> Is there any way to reuse the existing halt-polling code? Having two
>>> copies risks them diverging over time.
>> s/risks/guarantees/ :(
>>
>> Unfortunately, handling of the hardware threads in KVM PPC is a mess,
>> and I don't think it's possible to remove the duplication.
> Ah, ok. That's a shame.

It's definitely not ideal having this code duplicated, although we have
the issue that on PPC we only poll once all of the vcpus on a vcore have
ceded and need to retain a reference to that vcore.

Additionally we only actually do this in HV code, on the KVM PR
version we call the generic halt-polling code which doesn't know
about vcores.

I don't see an easy way to use the existing function.

>
>> Paolo

--
To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]