On 29/08/17 15:55, Konrad Rzeszutek Wilk wrote: > On Tue, Aug 29, 2017 at 11:46:35AM +0000, Yang Zhang wrote: >> So far, pv_idle_ops.poll is the only ops for pv_idle. .poll is called in >> idle path which will polling 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 come from the vmexit which is a >> hardware context switch between VM 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 <[email protected]> >> Signed-off-by: Quan Xu <[email protected]> >> Cc: Jeremy Fitzhardinge <[email protected]> >> Cc: Chris Wright <[email protected]> >> Cc: Alok Kataria <[email protected]> >> Cc: Rusty Russell <[email protected]> >> Cc: Thomas Gleixner <[email protected]> >> Cc: Ingo Molnar <[email protected]> >> Cc: "H. Peter Anvin" <[email protected]> >> Cc: [email protected] >> Cc: Peter Zijlstra <[email protected]> >> Cc: Andy Lutomirski <[email protected]> >> Cc: "Kirill A. Shutemov" <[email protected]> >> Cc: Pan Xinhui <[email protected]> >> Cc: Kees Cook <[email protected]> >> Cc: [email protected] >> Cc: [email protected] > > Adding xen-devel. > > Juergen, we really should replace Jeremy's name with xen-devel or > your name..
I wouldn't mind being added. What does Jeremy think of being removed? > Wasn't there an patch by you that took some of the > mainternship over it? I added include/linux/hypervisor.h to the PARAVIRT section and offered to maintain it in case the PARAVIRT maintainers didn't want to. Juergen
