Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp653685pxa; Wed, 19 Aug 2020 11:05:40 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyp3IHZMskPE5gl/OqbxCNH76adYerpWJDeWX+c6S0FfSZCA6Ui6bK2DTu9DZVGDTiv/vJz X-Received: by 2002:a05:6402:1a54:: with SMTP id bf20mr26184366edb.217.1597860340162; Wed, 19 Aug 2020 11:05:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597860340; cv=none; d=google.com; s=arc-20160816; b=00lqQeuIm78dyPvKVq1D+dnE4lSI9vZNZF1PH15XQ6AC//ruyB0NV0zOo070MetkIx 8OvHCRqW5ElIqqJhKmmHmh6bdmchuclrRl69qwhjM7iFykRhcfYNETv+2ho7ffOCdaza O8FHZLVHs0XK1tY2Bn6lUiQRpYn8uZUW7L3+SiUi6N2qDeEKTANFUeYO2QUgZRYNQNIS f6plRRPzYFCl+q3BxDl04rjP3jcRXH1HdbtzwlQON2La6jCAjqPAUYcV5x5GjwWyfmny J017iFQuPJYwkvFACeS2PP+5dLlbujTembHQwo+8knetskGkoGdD+VtJVQd770tZtVnR iOqg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:ironport-sdr:ironport-sdr; bh=im3VkW9Um9O1fwVgnHTgyjz6soGYvbQtQfYrwzB6qCk=; b=TzXe7Rs8pJFHqaggcQUbIBjMSQMX0NM4paGS3XIP+dIeCar/Jk8BFQ3AJeRRJ0oAfO H2t+c217blD6sCfoisYks5EE30MY4Kh6eDrgbhNu8bOIEdIfm7RrWWNQ1FfMsJQg9H/Z gG5ThGqUGUsNABgbWgddDqWH+3KQYICOO/kRUQHfiOlIf8yrfIGVM8Y7d5xjRNI1eleo oPF1rX0I6uo4YWjXAWDLiSy4wpwWsxUPjpeNtiiJqA2RQ4Icaha9XbZNULxpBusFSAtX GswGvjJOqElVqE7+r1yDhv/mHzfpUDvXT4lEPKGTnFpF1TqSIZnK75RkQ6wTbRU3Fn0J 9Tmw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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. [23.128.96.18]) by mx.google.com with ESMTP id v17si15498197ejd.324.2020.08.19.11.05.16; Wed, 19 Aug 2020 11:05:40 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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 S1726976AbgHSSCF (ORCPT + 99 others); Wed, 19 Aug 2020 14:02:05 -0400 Received: from mga09.intel.com ([134.134.136.24]:57877 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726995AbgHSSBZ (ORCPT ); Wed, 19 Aug 2020 14:01:25 -0400 IronPort-SDR: ztJEyCBVxs4t874OOzrkVh8DILdb/7ra72YbzjIadG0uR4YCyiAIvL6GAbBaqaqPBa2RsaaMGE MEGcvw+6Sbmw== X-IronPort-AV: E=McAfee;i="6000,8403,9718"; a="156245738" X-IronPort-AV: E=Sophos;i="5.76,332,1592895600"; d="scan'208";a="156245738" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Aug 2020 11:01:20 -0700 IronPort-SDR: zwX/EqTTFzfSHKqs0SMu8p445imnUrFLeltHbvGv3DjIe7reGDumsFI2qBS+a0bGijYba4d0g+ /MDAlXUxs2tQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.76,332,1592895600"; d="scan'208";a="279762367" Received: from linux.intel.com ([10.54.29.200]) by fmsmga008.fm.intel.com with ESMTP; 19 Aug 2020 11:01:18 -0700 Received: from debox1-desk2.jf.intel.com (debox1-desk2.jf.intel.com [10.54.75.16]) by linux.intel.com (Postfix) with ESMTP id 11C86580812; Wed, 19 Aug 2020 11:01:20 -0700 (PDT) From: "David E. Box" To: lee.jones@linaro.org, david.e.box@linux.intel.com, dvhart@infradead.org, andy@infradead.org, bhelgaas@google.com, alexander.h.duyck@linux.intel.com Cc: linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, linux-pci@vger.kernel.org Subject: [RESEND PATCH V5 0/3] Intel Platform Monitoring Technology Date: Wed, 19 Aug 2020 11:02:52 -0700 Message-Id: <20200819180255.11770-1-david.e.box@linux.intel.com> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Intel Platform Monitoring Technology (PMT) is an architecture for enumerating and accessing hardware monitoring capabilities on a device. With customers increasingly asking for hardware telemetry, engineers not only have to figure out how to measure and collect data, but also how to deliver it and make it discoverable. The latter may be through some device specific method requiring device specific tools to collect the data. This in turn requires customers to manage a suite of different tools in order to collect the differing assortment of monitoring data on their systems. Even when such information can be provided in kernel drivers, they may require constant maintenance to update register mappings as they change with firmware updates and new versions of hardware. PMT provides a solution for discovering and reading telemetry from a device through a hardware agnostic framework that allows for updates to systems without requiring patches to the kernel or software tools. PMT defines several capabilities to support collecting monitoring data from hardware. All are discoverable as separate instances of the PCIE Designated Vendor extended capability (DVSEC) with the Intel vendor code. The DVSEC ID field uniquely identifies the capability. Each DVSEC also provides a BAR offset to a header that defines capability-specific attributes, including GUID, feature type, offset and length, as well as configuration settings where applicable. The GUID uniquely identifies the register space of any monitor data exposed by the capability. The GUID is associated with an XML file from the vendor that describes the mapping of the register space along with properties of the monitor data. This allows vendors to perform firmware updates that can change the mapping (e.g. add new metrics) without requiring any changes to drivers or software tools. The new mapping is confirmed by an updated GUID, read from the hardware, which software uses with a new XML. The current capabilities defined by PMT are Telemetry, Watcher, and Crashlog. The Telemetry capability provides access to a continuous block of read only data. The Watcher capability provides access to hardware sampling and tracing features. Crashlog provides access to device crash dumps. While there is some relationship between capabilities (Watcher can be configured to sample from the Telemetry data set) each exists as stand alone features with no dependency on any other. The design therefore splits them into individual, capability specific drivers. MFD is used to create platform devices for each capability so that they may be managed by their own driver. The PMT architecture is (for the most part) agnostic to the type of device it can collect from. Devices nodes are consequently generic in naming, e.g. /dev/telem and /dev/smplr. Each capability driver creates a class to manage the list of devices supporting it. Software can determine which devices support a PMT feature by searching through each device node entry in the sysfs class folder. It can additionally determine if a particular device supports a PMT feature by checking for a PMT class folder in the device folder. This patch set provides support for the PMT framework, along with support for Telemetry on Tiger Lake. Changes from V4: - Replace MFD with PMT in driver title - Fix commit tags in chronological order - Fix includes in alphabetical order - Use 'raw' string instead of defines for device names - Add an error message when returning an error code for unrecognized capability id - Use dev_err instead of dev_warn for messages when returning an error - Change while loop to call pci_find_next_ext_capability once - Add missing continue in while loop - Keep PCI platform defines using PCI_DEVICE_DATA magic tied to the pci_device_id table - Comment and kernel message cleanup Changes from V3: - Write out full acronym for DVSEC in PCI patch commit message and add 'Designated' to comments - remove unused variable caught by kernel test robot - Add required Co-developed-by signoffs, noted by Andy - Allow access using new CAP_PERFMON capability as suggested by Alexey Bundankov - Fix spacing in Kconfig, noted by Randy - Other style changes and fixups suggested by Andy Changes from V2: - In order to handle certain HW bugs from the telemetry capability driver, create a single platform device per capability instead of a device per entry. Add the entry data as device resources and let the capability driver manage them as a set allowing for cleaner HW bug resolution. - Handle discovery table offset bug in intel_pmt.c - Handle overlapping regions in intel_pmt_telemetry.c - Add description of sysfs class to testing ABI. - Don't check size and count until confirming support for the PMT capability to avoid bailing out when we need to skip it. - Remove unneeded header file. Move code to the intel_pmt.c, the only place where it's needed. - Remove now unused platform data. - Add missing header files types.h, bits.h. - Rename file name and build options from telem to telemetry. - Code cleanup suggested by Andy S. - x86 mailing list added. Changes from V1: - In the telemetry driver, set the device in device_create() to the parent PCI device (the monitoring device) for clear association in sysfs. Was set before to the platform device created by the PCI parent. - Move telem struct into driver and delete unneeded header file. - Start telem device numbering from 0 instead of 1. 1 was used due to anticipated changes, no longer needed. - Use helper macros suggested by Andy S. - Rename class to pmt_telemetry, spelling out full name - Move monitor device name defines to common header - Coding style, spelling, and Makefile/MAINTAINERS ordering fixes David E. Box (3): PCI: Add defines for Designated Vendor-Specific Extended Capability mfd: Intel Platform Monitoring Technology support platform/x86: Intel PMT Telemetry capability driver .../ABI/testing/sysfs-class-pmt_telemetry | 46 ++ MAINTAINERS | 6 + drivers/mfd/Kconfig | 10 + drivers/mfd/Makefile | 1 + drivers/mfd/intel_pmt.c | 220 +++++++++ drivers/platform/x86/Kconfig | 10 + drivers/platform/x86/Makefile | 1 + drivers/platform/x86/intel_pmt_telemetry.c | 448 ++++++++++++++++++ include/uapi/linux/pci_regs.h | 5 + 9 files changed, 747 insertions(+) create mode 100644 Documentation/ABI/testing/sysfs-class-pmt_telemetry create mode 100644 drivers/mfd/intel_pmt.c create mode 100644 drivers/platform/x86/intel_pmt_telemetry.c -- 2.20.1