Subject: Re: [PATCH 1/2] x86/idle: add halt poll for halt idle
To: Thomas Gleixner <tglx@linutronix.de>
References: <1498130534-26568-1-git-send-email-root@ip-172-31-39-62.us-west-2.compute.internal>
 <1498130534-26568-2-git-send-email-root@ip-172-31-39-62.us-west-2.compute.internal>
 <alpine.DEB.2.20.1706221620120.1885@nanos>
Cc: mingo@redhat.com, hpa@zytor.com, pbonzini@redhat.com, x86@kernel.org,
        corbet@lwn.net, tony.luck@intel.com, bp@alien8.de,
        peterz@infradead.org, mchehab@kernel.org, akpm@linux-foundation.org,
        krzk@kernel.org, jpoimboe@redhat.com, luto@kernel.org,
        borntraeger@de.ibm.com, thgarnie@google.com, rgerst@gmail.com,
        minipli@googlemail.com, douly.fnst@cn.fujitsu.com, nicstange@gmail.com,
        fweisbec@gmail.com, dvlasenk@redhat.com, bristot@redhat.com,
        yamada.masahiro@socionext.com, mika.westerberg@linux.intel.com,
        yu.c.chen@intel.com, aaron.lu@intel.com, rostedt@goodmis.org,
        me@kylehuey.com, len.brown@intel.com, prarit@redhat.com,
        hidehiro.kawai.ez@hitachi.com, fengtiantian@huawei.com,
        pmladek@suse.com, jeyu@redhat.com, Larry.Finger@lwfinger.net,
        zijun_hu@htc.com, luisbg@osg.samsung.com, johannes.berg@intel.com,
        niklas.soderlund+renesas@ragnatech.se, zlpnobody@gmail.com,
        adobriyan@gmail.com, fgao@48lvckh6395k16k5.yundunddos.com,
        ebiederm@xmission.com, subashab@codeaurora.org, arnd@arndb.de,
        matt@codeblueprint.co.uk, mgorman@techsingularity.net,
        linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
        linux-edac@vger.kernel.org, kvm@vger.kernel.org
From: Yang Zhang <yang.zhang.wz@gmail.com>
Message-ID: <b5b650b0-921c-7f1d-8dfc-c4aad5e28bd4@gmail.com>
Date: Fri, 23 Jun 2017 12:05:35 +0800
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.20.1706221620120.1885@nanos>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1506
Lines: 57

On 2017/6/22 22:23, Thomas Gleixner wrote:
> On Thu, 22 Jun 2017, root wrote:
>> --- a/arch/x86/kernel/process.c
>> +++ b/arch/x86/kernel/process.c
>> @@ -39,6 +39,10 @@
>>  #include <asm/desc.h>
>>  #include <asm/prctl.h>
>>
>> +#ifdef CONFIG_HYPERVISOR_GUEST
>> +unsigned long poll_threshold_ns;
>> +#endif
>> +
>>  /*
>>   * per-CPU TSS segments. Threads are completely 'soft' on Linux,
>>   * no more per-task TSS's. The TSS size is kept cacheline-aligned
>> @@ -313,6 +317,23 @@ static inline void play_dead(void)
>>  }
>>  #endif
>>
>> +#ifdef CONFIG_HYPERVISOR_GUEST
>> +void arch_cpu_idle_poll(void)
>> +{
>> +	ktime_t start, cur, stop;
>> +
>> +	if (poll_threshold_ns) {
>> +		start = cur = ktime_get();
>> +		stop = ktime_add_ns(ktime_get(), poll_threshold_ns);
>> +		do {
>> +			if (need_resched())
>> +				break;
>> +			cur = ktime_get();
>> +		} while (ktime_before(cur, stop));
>> +	}
>> +}
>> +#endif
>
> Aside of the whole approach being debatable, what's the reason to make this
> depend on CONFIG_HYPERVISOR_GUEST and to move it into x86. If that
> mechanism is worthwhile then it should go into the generic code and not
> into x86. There is absolutely nothing x86 specific in that patch.
>
> Also the CONFIG_HYPERVISOR_GUEST dependency is silly. Distro kernels ship
> with CONFIG_HYPERVISOR_GUEST=y so this also gets into affect on bare metal.

You are right. As Paolo suggested, i will integrate it with 
paravalization code.

>
> Thanks,
>
> 	tglx
>


-- 
Yang
Alibaba Cloud Computing