From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752069Ab1AXT5W (ORCPT ); Mon, 24 Jan 2011 14:57:22 -0500 Received: from mx1.redhat.com ([209.132.183.28]:5126 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751374Ab1AXT5V (ORCPT ); Mon, 24 Jan 2011 14:57:21 -0500 Subject: Re: [PATCH 16/16] KVM-GST: adjust scheduler cpu power From: Glauber Costa To: Peter Zijlstra Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, aliguori@us.ibm.com, Rik van Riel , Jeremy Fitzhardinge , Avi Kivity In-Reply-To: <1295898690.28776.472.camel@laptop> References: <1295892397-11354-1-git-send-email-glommer@redhat.com> <1295892397-11354-17-git-send-email-glommer@redhat.com> <1295893920.28776.468.camel@laptop> <1295895083.15920.9.camel@mothafucka.localdomain> <1295898690.28776.472.camel@laptop> Content-Type: text/plain; charset="UTF-8" Organization: Red Hat Date: Mon, 24 Jan 2011 17:57:11 -0200 Message-ID: <1295899031.15920.13.camel@mothafucka.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2011-01-24 at 20:51 +0100, Peter Zijlstra wrote: > On Mon, 2011-01-24 at 16:51 -0200, Glauber Costa wrote: > > > I would really much rather see you change update_rq_clock_task() and > > > subtract your ns resolution steal time from our wall-time, > > > update_rq_clock_task() already updates the cpu_power relative to the > > > remaining time available. > > > > But then we get into the problems we already discussed in previous > > submissions, which is, we don't really want to alter any notion of > > wallclock time. Could you list any more concrete advantages of the > > specific way you proposed? > > clock_task is the time spend on the task, by not taking steal time into > account all steal time is accounted as service to whatever task was > current when the vcpu wasn't running. > > It doesn't change wall-time in the sense of gtod, only the service time > to tasks. Ok, I'll experiment with that and see how it goes.