Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752909Ab1BAS6h (ORCPT ); Tue, 1 Feb 2011 13:58:37 -0500 Received: from casper.infradead.org ([85.118.1.10]:50553 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752553Ab1BAS6f (ORCPT ); Tue, 1 Feb 2011 13:58:35 -0500 Subject: Re: [PATCH v2 5/6] KVM-GST: adjust scheduler cpu power From: Peter Zijlstra To: Glauber Costa Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, aliguori@us.ibm.com, Rik van Riel , Jeremy Fitzhardinge , Avi Kivity In-Reply-To: <1296577358.5081.23.camel@mothafucka.localdomain> References: <1296244340-15173-1-git-send-email-glommer@redhat.com> <1296244340-15173-6-git-send-email-glommer@redhat.com> <1296473120.15234.375.camel@laptop> <1296575996.5081.20.camel@mothafucka.localdomain> <1296577168.26581.240.camel@laptop> <1296577358.5081.23.camel@mothafucka.localdomain> Content-Type: text/plain; charset="UTF-8" Date: Tue, 01 Feb 2011 19:59:31 +0100 Message-ID: <1296586771.26581.298.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1363 Lines: 43 On Tue, 2011-02-01 at 14:22 -0200, Glauber Costa wrote: > > > Which tick accounting? In your other e-mail , you pointed that this only > runs in touch_steal_time, which is fine, will change. That tick ;-), all the account_foo muck is per tick. > But all the rest > here, that is behind the hypervisor specific vs generic code has nothing > to do with ticks at all. But I don't get it, there is no generic code needed, all that's needed is u64 steal_time_clock(int cpu), and the first part of your kvm_account_steal_time() function is exactly that if you add the cpu argument. +static u64 steal_time_clock(int cpu) +{ + u64 steal_time; + struct kvm_steal_time *src; + int version; + + preempt_disable(); + src = &per_cpu_ptr(steal_time, cpu); + do { + version = src->version; + rmb(); + steal_time = src->steal; + rmb(); + } while ((src->version & 1) || (version != src->version)); + preempt_enable(); + + return steal_time +} And you're done.. no need to for any of that steal_time_{read,write} business. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/