From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753337AbdKNIP6 (ORCPT ); Tue, 14 Nov 2017 03:15:58 -0500 Received: from mail-ot0-f194.google.com ([74.125.82.194]:43754 "EHLO mail-ot0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751114AbdKNIPt (ORCPT ); Tue, 14 Nov 2017 03:15:49 -0500 X-Google-Smtp-Source: AGs4zMaJrQ0680fN3LcnPLBz08gJ8DdZbJ767KIIjxcih8wfuEN7f6F1ZvKuUkp/GirSH5zrVKvPmQ== Subject: Re: [PATCH RFC v3 1/6] x86/paravirt: Add pv_idle_ops to paravirt ops To: Wanpeng Li Cc: Juergen Gross , Quan Xu , kvm , linux-doc@vger.kernel.org, "open list:FILESYSTEMS (VFS and infrastructure)" , "linux-kernel@vger.kernel.org" , virtualization@lists.linux-foundation.org, the arch/x86 maintainers , xen-devel , Yang Zhang , Alok Kataria , Rusty Russell , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" References: <1510567565-5118-1-git-send-email-quan.xu0@gmail.com> <1510567565-5118-2-git-send-email-quan.xu0@gmail.com> <07fac696-e3d4-8f35-8f3d-764d7ab41204@suse.com> <902da704-1e4f-583b-91c3-1a62ccd6e73d@gmail.com> From: Quan Xu Message-ID: Date: Tue, 14 Nov 2017 16:15:39 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017/11/14 15:12, Wanpeng Li wrote: > 2017-11-14 15:02 GMT+08:00 Quan Xu : >> >> On 2017/11/13 18:53, Juergen Gross wrote: >>> On 13/11/17 11:06, Quan Xu wrote: >>>> From: Quan Xu >>>> >>>> So far, pv_idle_ops.poll is the only ops for pv_idle. .poll is called >>>> in idle path which will poll for a while before we enter the real idle >>>> state. >>>> >>>> In virtualization, idle path includes several heavy operations >>>> includes timer access(LAPIC timer or TSC deadline timer) which will >>>> hurt performance especially for latency intensive workload like message >>>> passing task. The cost is mainly from the vmexit which is a hardware >>>> context switch between virtual machine and hypervisor. Our solution is >>>> to poll for a while and do not enter real idle path if we can get the >>>> schedule event during polling. >>>> >>>> Poll may cause the CPU waste so we adopt a smart polling mechanism to >>>> reduce the useless poll. >>>> >>>> Signed-off-by: Yang Zhang >>>> Signed-off-by: Quan Xu >>>> Cc: Juergen Gross >>>> Cc: Alok Kataria >>>> Cc: Rusty Russell >>>> Cc: Thomas Gleixner >>>> Cc: Ingo Molnar >>>> Cc: "H. Peter Anvin" >>>> Cc: x86@kernel.org >>>> Cc: virtualization@lists.linux-foundation.org >>>> Cc: linux-kernel@vger.kernel.org >>>> Cc: xen-devel@lists.xenproject.org >>> Hmm, is the idle entry path really so critical to performance that a new >>> pvops function is necessary? >> Juergen, Here is the data we get when running benchmark netperf: >> 1. w/o patch and disable kvm dynamic poll (halt_poll_ns=0): >> 29031.6 bit/s -- 76.1 %CPU >> >> 2. w/ patch and disable kvm dynamic poll (halt_poll_ns=0): >> 35787.7 bit/s -- 129.4 %CPU >> >> 3. w/ kvm dynamic poll: >> 35735.6 bit/s -- 200.0 %CPU > Actually we can reduce the CPU utilization by sleeping a period of > time as what has already been done in the poll logic of IO subsystem, > then we can improve the algorithm in kvm instead of introduing another > duplicate one in the kvm guest. We really appreciate upstream's kvm dynamic poll mechanism, which is really helpful for a lot of scenario.. However, as description said, in virtualization, idle path includes several heavy operations includes timer access (LAPIC timer or TSC deadline timer) which will hurt performance especially for latency intensive workload like message passing task. The cost is mainly from the vmexit which is a hardware context switch between virtual machine and hypervisor. for upstream's kvm dynamic poll mechanism, even you could provide a better algorism, how could you bypass timer access (LAPIC timer or TSC deadline timer), or a hardware context switch between virtual machine and hypervisor. I know these is a tradeoff. Furthermore, here is the data we get when running benchmark contextswitch to measure the latency(lower is better): 1. w/o patch and disable kvm dynamic poll (halt_poll_ns=0):   3402.9 ns/ctxsw -- 199.8 %CPU 2. w/ patch and disable kvm dynamic poll:   1163.5 ns/ctxsw -- 205.5 %CPU 3. w/ kvm dynamic poll:   2280.6 ns/ctxsw -- 199.5 %CPU so, these tow solution are quite similar, but not duplicate.. that's also why to add a generic idle poll before enter real idle path. When a reschedule event is pending, we can bypass the real idle path. Quan Alibaba Cloud > Regards, > Wanpeng Li > >> 4. w/patch and w/ kvm dynamic poll: >> 42225.3 bit/s -- 198.7 %CPU >> >> 5. idle=poll >> 37081.7 bit/s -- 998.1 %CPU >> >> >> >> w/ this patch, we will improve performance by 23%.. even we could improve >> performance by 45.4%, if we use w/patch and w/ kvm dynamic poll. also the >> cost of CPU is much lower than 'idle=poll' case.. >> >>> Wouldn't a function pointer, maybe guarded >>> by a static key, be enough? A further advantage would be that this would >>> work on other architectures, too. >> >> I assume this feature will be ported to other archs.. a new pvops makes code >> clean and easy to maintain. also I tried to add it into existed pvops, but >> it >> doesn't match. >> >> >> >> Quan >> Alibaba Cloud >>> >>> Juergen >>>