Received: by 2002:ac0:946b:0:0:0:0:0 with SMTP id j40csp3664778imj; Tue, 12 Feb 2019 02:36:05 -0800 (PST) X-Google-Smtp-Source: AHgI3Ia/St1w1w4XcvXdJWz9giTFaGA3T/EsSE5FGdIHS5wrWr24z2NMJz1/O0wTPOKNzyJNmyBF X-Received: by 2002:a17:902:2a0a:: with SMTP id i10mr3225764plb.323.1549967765599; Tue, 12 Feb 2019 02:36:05 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1549967765; cv=none; d=google.com; s=arc-20160816; b=UutEem58mFadJqsKlkaJQKAoahQUkR4SBO7VUJsP6GmbfD2r25ytW+4v2gUXowtLbg s7rwxT7A732HQcJKG2NkYiYWnMmk3NJUZ2h0T0sDAcZL3sWDnLn1pdrtDcUAaa8THxLT JCC1JW16Jyy1n3WEDr7PbjOk1p5090EBp8vSn2bR5JRUSKrnjum8qGJ1dsoeb2ePEC0P U1t67uYnrc76F8nXEXpN5EB7Eyai0LI9VSbxZUB5kMawLPDcec/+C4vu6cBP62Ih/4/s xQHUJIhCCp/+PesOv7P1qixML0sq1zbf+R4NVUxzzRv48nu7juVpmxwmYyjqmq1Aty7c t9nQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-language :content-transfer-encoding:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=qRnaWHnh6dskUYYKryXKHXMRyKjOCk0YwC43PiwSuOM=; b=vvqFn/ib08f868VD0VL3gXSVCe9JZoKAiEgsWT1DwaqZ/hyjb+ri5NO+3HogQstnKu 8B1Z4pC7ORRHOPxkS4kAP4qLZmVlY20UT+6948ZZxtYfy9FDOnku/WE7z3XjxEbb25PM +BEkL/mq8EIdXl0ZpaJYfsZiObQ86uqEjI7V7U9TeFzjy5E7IXc15+d2AXKmG5wj7cgY NcN5hwyi/Jm+eTKVW7UxA3FLoDMKl+GVzzZdf0eENLwv9zPuNKpYOMv5cTVHM3uq6t5h JRWO91o2GtHPHfdB8g4ZPSIn4HU0VMoRz1O4uPH5R+vlADQT38W6F2QK2SRTruLjw2kR VzGg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c12si2518223pgd.39.2019.02.12.02.35.47; Tue, 12 Feb 2019 02:36:05 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728848AbfBLKe5 (ORCPT + 99 others); Tue, 12 Feb 2019 05:34:57 -0500 Received: from mga01.intel.com ([192.55.52.88]:64778 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726756AbfBLKe5 (ORCPT ); Tue, 12 Feb 2019 05:34:57 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Feb 2019 02:34:56 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,361,1544515200"; d="scan'208";a="117246137" Received: from rbhardw1-mobl.gar.corp.intel.com (HELO [10.67.170.184]) ([10.67.170.184]) by orsmga008.jf.intel.com with ESMTP; 12 Feb 2019 02:34:53 -0800 Subject: Re: [PATCH 09/10] platform/x86: intel_pmc_core: Add Package cstates residency info To: Andy Shevchenko Cc: Anshuman Gupta , Platform Driver , Darren Hart , Andy Shevchenko , Linux Kernel Mailing List , Arjan van de Ven , "David E. Box" , Srinivas Pandruvada , Len Brown , "Rafael J . Wysocki" References: <20190201073234.13280-1-rajneesh.bhardwaj@linux.intel.com> <20190201073234.13280-10-rajneesh.bhardwaj@linux.intel.com> <20190211164118.GA23796@genxfsim-desktop> <78e5e6b3-5712-6ef5-283c-83b0f6d3dcc7@linux.intel.com> From: "Bhardwaj, Rajneesh" Message-ID: <38444053-cbab-ae6d-9504-18ca3cd84d15@linux.intel.com> Date: Tue, 12 Feb 2019 16:04:51 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12-Feb-19 3:55 PM, Andy Shevchenko wrote: > On Mon, Feb 11, 2019 at 8:32 PM Bhardwaj, Rajneesh > wrote: >> On 11-Feb-19 10:11 PM, Anshuman Gupta wrote: >>> On Fri, Feb 01, 2019 at 01:02:33PM +0530, Rajneesh Bhardwaj wrote: >>>> This patch introduces a new debugfs entry to read current Package >>>> cstate residency counters. A similar variant of this patch was discussed >>>> earlier "https://patchwork.kernel.org/patch/9908563/" but didn't make it >>>> into mainline for various reasons. Current version only adds debugfs >>>> entry which is quite useful for S0ix debug but excludes the exported API >>>> that was there in initial version. Though there are tools like turbostat >>>> and socwatch which can also show this info but sometimes its more >>>> practical to have it here as it's hard to switch between various tools for >>>> S0ix debug when pmc_core driver is the primary debug tool. Internal and >>>> external customers have requested for this patch to be included in the >>>> PMC driver on many occasions and Google Chrome OS team has already included >>>> it in their builds. This becomes handy when requesting logs from external >>>> customers who may not always have above mentioned tools in their integrated >>>> kernel builds. >>> Tested-by: Anshuman Gupta >>> Acked-by: Anshuman Gupta >> Hi Andy, Darren - Can i used Acked-and-tested-by instead of these two? > Better to split. I didn't see in common practice acked-and-tested. > Rather reviewed-and-tested, or reported-and-tested. Sure Andy, I can do as you suggest but i am confused now because on another patch i've got "Acked-and-tested-by" only i.e. just one single tag. So for consistency, what should i do? I also noticed that this tag is used about 59 times in Linux kernel and i even found one "Reported-Acked-and-Tested-by" too :) though i understand these may not be acceptable by all Maintainers. So, please guide. >