linux-kernel - RE: [PATCH 1/3] platform/x86: Add GLK PSS Event Table

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5F7315E704FA0841B5DFCE90329B2BB462E9006C@BGSMSX102.gar.corp.intel.com>
Date:   Mon, 17 Jul 2017 05:18:54 +0000
From:   "Chakravarty, Souvik K" <souvik.k.chakravarty@...el.com>
To:     "Bhardwaj, Rajneesh" <rajneesh.bhardwaj@...el.com>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>
CC:     "dvhart@...radead.org" <dvhart@...radead.org>,
        "andy@...radead.org" <andy@...radead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Murthy, Shanth" <shanth.murthy@...el.com>,
        "Bhardwaj, Rajneesh" <rajneesh.bhardwaj@...el.com>
Subject: RE: [PATCH 1/3] platform/x86: Add GLK PSS Event Table

+1 From me.

> -----Original Message-----
> From: platform-driver-x86-owner@...r.kernel.org [mailto:platform-driver-
> x86-owner@...r.kernel.org] On Behalf Of Rajneesh Bhardwaj
> Sent: Friday, July 14, 2017 5:24 PM
> To: platform-driver-x86@...r.kernel.org
> Cc: dvhart@...radead.org; andy@...radead.org; linux-
> kernel@...r.kernel.org; Murthy, Shanth <shanth.murthy@...el.com>;
> Chakravarty, Souvik K <souvik.k.chakravarty@...el.com>; Bhardwaj,
> Rajneesh <rajneesh.bhardwaj@...el.com>
> Subject: [PATCH 1/3] platform/x86: Add GLK PSS Event Table
> 
> Some of the Primary Subsystem events differ on Gemini Lake but the IOSS
> events remain same. This patch adds the updated PSS event table to enable
> Telemetry driver on Gemini Lake.
> 
> Signed-off-by: Shanth Murthy <shanth.murthy@...el.com>
> Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@...el.com>
> ---
>  drivers/platform/x86/intel_telemetry_debugfs.c |  1 +
> drivers/platform/x86/intel_telemetry_pltdrv.c  | 35
> ++++++++++++++++++++++++++
>  2 files changed, 36 insertions(+)
> 
> diff --git a/drivers/platform/x86/intel_telemetry_debugfs.c
> b/drivers/platform/x86/intel_telemetry_debugfs.c
> index 4cc2f4ea0a25..a0e4344b2eec 100644
> --- a/drivers/platform/x86/intel_telemetry_debugfs.c
> +++ b/drivers/platform/x86/intel_telemetry_debugfs.c
> @@ -331,6 +331,7 @@ static struct telemetry_debugfs_conf
> telem_apl_debugfs_conf = {
> 
>  static const struct x86_cpu_id telemetry_debugfs_cpu_ids[] = {
>  	TELEM_DEBUGFS_CPU(INTEL_FAM6_ATOM_GOLDMONT,
> telem_apl_debugfs_conf),
> +	TELEM_DEBUGFS_CPU(INTEL_FAM6_ATOM_GEMINI_LAKE,
> +telem_apl_debugfs_conf),
>  	{}
>  };
> 
> diff --git a/drivers/platform/x86/intel_telemetry_pltdrv.c
> b/drivers/platform/x86/intel_telemetry_pltdrv.c
> index 6ebdbd2b04fc..6393b3b1d5a6 100644
> --- a/drivers/platform/x86/intel_telemetry_pltdrv.c
> +++ b/drivers/platform/x86/intel_telemetry_pltdrv.c
> @@ -153,6 +153,30 @@ static struct telemetry_evtmap
>  	{"PC2_AND_MEM_SHALLOW_IDLE_RES",	0x1D40},
>  };
> 
> +static struct telemetry_evtmap
> +
> 	telemetry_glk_pss_default_events[TELEM_MAX_OS_ALLOCATED_E
> VENTS] = {
> +	{"IA_CORE0_C6_RES",			0x0400},
> +	{"IA_CORE0_C6_CTR",			0x0000},
> +	{"IA_MODULE0_C7_RES",			0x0410},
> +	{"IA_MODULE0_C7_CTR",			0x000C},
> +	{"IA_C0_RES",				0x0805},
> +	{"PCS_LTR",				0x2801},
> +	{"PSTATES",				0x2802},
> +	{"SOC_S0I3_RES",			0x0407},
> +	{"SOC_S0I3_CTR",			0x0008},
> +	{"PCS_S0I3_CTR",			0x0007},
> +	{"PCS_C1E_RES",				0x0414},
> +	{"PCS_IDLE_STATUS",			0x2806},
> +	{"IA_PERF_LIMITS",			0x280B},
> +	{"GT_PERF_LIMITS",			0x280C},
> +	{"PCS_WAKEUP_S0IX_CTR",			0x0025},
> +	{"PCS_IDLE_BLOCKED",			0x2C00},
> +	{"PCS_S0IX_BLOCKED",			0x2C01},
> +	{"PCS_S0IX_WAKE_REASONS",		0x2C02},
> +	{"PCS_LTR_BLOCKING",			0x2C03},
> +	{"PC2_AND_MEM_SHALLOW_IDLE_RES",	0x1D40},
> +};
> +
>  /* APL specific Data */
>  static struct telemetry_plt_config telem_apl_config = {
>  	.pss_config = {
> @@ -163,8 +187,19 @@ static struct telemetry_plt_config
> telem_apl_config = {
>  	},
>  };
> 
> +/* GLK specific Data */
> +static struct telemetry_plt_config telem_glk_config = {
> +	.pss_config = {
> +		.telem_evts = telemetry_glk_pss_default_events,
> +	},
> +	.ioss_config = {
> +		.telem_evts = telemetry_apl_ioss_default_events,
> +	},
> +};
> +
>  static const struct x86_cpu_id telemetry_cpu_ids[] = {
>  	TELEM_CPU(INTEL_FAM6_ATOM_GOLDMONT, telem_apl_config),
> +	TELEM_CPU(INTEL_FAM6_ATOM_GEMINI_LAKE, telem_glk_config),
>  	{}
>  };
> 
> --
> 2.7.4