linux-kernel - Re: [PATCH] perf/x86/intel/uncore: Add Node ID mask

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: <CDEC00B0-AF34-4F97-AABD-A7FE0E8B7E5F@fb.com>
Date:   Tue, 29 Jan 2019 07:10:16 +0000
From:   Song Liu <songliubraving@...com>
To:     "kan.liang@...ux.intel.com" <kan.liang@...ux.intel.com>
CC:     Thomas Gleixner <tglx@...utronix.de>,
        "peterz@...radead.org" <peterz@...radead.org>,
        "mingo@...hat.com" <mingo@...hat.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "ak@...ux.intel.com" <ak@...ux.intel.com>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>
Subject: Re: [PATCH] perf/x86/intel/uncore: Add Node ID mask



> On Jan 27, 2019, at 6:53 AM, kan.liang@...ux.intel.com wrote:
> 
> From: Kan Liang <kan.liang@...ux.intel.com>
> 
> Some PCI uncore PMUs cannot be registered on a 8-socket system (HPE
> Superdome Flex).
> 
> To understand which Socket the PCI uncore PMUs belong to, perf retrieves
> the local Node ID of the uncore device from CPUNODEID(0xC0) of the PCI
> configuration space, and the mapping between Socket ID and Node ID from
> GIDNIDMAP(0xD4). The Socket ID can be calculated accordingly.
> The local Node ID is only available at bit 2:0, but current code doesn't
> mask it. If a BIOS doesn't clear the rest of the bits, a wrong Node ID
> will be fetched.
> 
> Filter the Node ID by adding a mask.
> 
> Fixes: 7c94ee2e0917 ("perf/x86: Add Intel Nehalem and Sandy Bridge-EP uncore support")
> Reported-by: Song Liu <songliubraving@...com>
> Tested-by: Song Liu <songliubraving@...com>
> Signed-off-by: Kan Liang <kan.liang@...ux.intel.com>
> Cc: stable@...r.kernel.org

Thanks Kan!

Acked-by: Song Liu <songliubraving@...com>


> ---
> arch/x86/events/intel/uncore_snbep.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/intel/uncore_snbep.c
> index c07bee3..b10e043 100644
> --- a/arch/x86/events/intel/uncore_snbep.c
> +++ b/arch/x86/events/intel/uncore_snbep.c
> @@ -1222,6 +1222,8 @@ static struct pci_driver snbep_uncore_pci_driver = {
> 	.id_table	= snbep_uncore_pci_ids,
> };
> 
> +#define NODE_ID_MASK	0x7
> +
> /*
>  * build pci bus to socket mapping
>  */
> @@ -1243,7 +1245,7 @@ static int snbep_pci2phy_map_init(int devid, int nodeid_loc, int idmap_loc, bool
> 		err = pci_read_config_dword(ubox_dev, nodeid_loc, &config);
> 		if (err)
> 			break;
> -		nodeid = config;
> +		nodeid = config & NODE_ID_MASK;
> 		/* get the Node ID mapping */
> 		err = pci_read_config_dword(ubox_dev, idmap_loc, &config);
> 		if (err)
> -- 
> 2.7.4
>