From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753870AbcCHBIM (ORCPT <rfc822;w@1wt.eu>);
	Mon, 7 Mar 2016 20:08:12 -0500
Received: from mga04.intel.com ([192.55.52.120]:63892 "EHLO mga04.intel.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1753436AbcCHBH5 (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Mon, 7 Mar 2016 20:07:57 -0500
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.22,554,1449561600"; 
   d="scan'208";a="904433838"
Date: Mon, 7 Mar 2016 17:06:42 -0800
From: Jacob Pan <jacob.jun.pan@linux.intel.com>
To: Jacob Pan <jacob.jun.pan@linux.intel.com>
Cc: LKML <linux-kernel@vger.kernel.org>, Linux PM <linux-pm@vger.kernel.org>,
        Rafael Wysocki <rafael.j.wysocki@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        "David S. Miller" <davem@davemloft.net>,
        Rusty Russell <rusty@rustcorp.com.au>
Subject: Re: [PATCH v6 0/4] powercap/rapl: reduce ipi calls and misc clean
 up
Message-ID: <20160307170642.31af8b7e@icelake>
In-Reply-To: <1456349498-1527-1-git-send-email-jacob.jun.pan@linux.intel.com>
References: <1456349498-1527-1-git-send-email-jacob.jun.pan@linux.intel.com>
Organization: OTC
X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.23; x86_64-pc-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, 24 Feb 2016 13:31:34 -0800
Jacob Pan <jacob.jun.pan@linux.intel.com> wrote:

> Changes since v5:
> 	- Split changes into seprate patches. No code change.
> Changes since V4:
> 	- Deleted unnecessary cpumask_any_and() call, assign first
> cpu as lead Changes since V3:
> 	- Avoid for_each_online_cpu() search for per package MSR
> access. This is achieved by tracking a per package lead cpu via
> cpumask_xx() calls. (suggested by Thomas Gleixner
> <tglx@linutronix.de>)
> 	    - Add direct reference of RAPL package for each RAPL
> domain.
> 
> Jacob Pan (4):
>   cpumask: export cpumask_any_but
>   powercap/rapl: reduce ipi calls
>   powercap/rapl: add package reference per domain
>   powercap/rapl: track lead cpu per package
> 
>  drivers/powercap/intel_rapl.c | 220
> ++++++++++++++++++++++--------------------
> lib/cpumask.c                 |   1 + 2 files changed, 115
> insertions(+), 106 deletions(-)
> 
Hi Thomas,

Do you have more comments?

Thanks,

Jacob