From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 76EC6C4360F for ; Wed, 3 Apr 2019 16:47:27 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 374B42082C for ; Wed, 3 Apr 2019 16:47:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726633AbfDCQr0 (ORCPT ); Wed, 3 Apr 2019 12:47:26 -0400 Received: from mga09.intel.com ([134.134.136.24]:42684 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726155AbfDCQrZ (ORCPT ); Wed, 3 Apr 2019 12:47:25 -0400 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Apr 2019 09:47:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,305,1549958400"; d="scan'208";a="132687041" Received: from hao-dev.bj.intel.com (HELO localhost) ([10.238.157.65]) by orsmga006.jf.intel.com with ESMTP; 03 Apr 2019 09:47:22 -0700 Date: Thu, 4 Apr 2019 00:31:47 +0800 From: Wu Hao To: Moritz Fischer Cc: atull@kernel.org, linux-fpga@vger.kernel.org, linux-kernel@vger.kernel.org, linux-api@vger.kernel.org, Luwei Kang , Russ Weight , Xu Yilun Subject: Re: [PATCH 14/17] fpga: dfl: fme: add thermal management support Message-ID: <20190403163147.GA28570@hao-dev> References: <1553483264-5379-1-git-send-email-hao.wu@intel.com> <1553483264-5379-15-git-send-email-hao.wu@intel.com> <20190402145925.GA15773@archbook> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190402145925.GA15773@archbook> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 02, 2019 at 07:59:25AM -0700, Moritz Fischer wrote: > Hi Wu, > > On Mon, Mar 25, 2019 at 11:07:41AM +0800, Wu Hao wrote: > > This patch adds support to thermal management private feature for DFL > > FPGA Management Engine (FME). As thermal throttling is handled by > > hardware automatically per pre-defined thresholds, this private > > feature driver only provides read-only sysfs interfaces for user > > to read temperature, thresholds, threshold policy and other info. > > > > Signed-off-by: Luwei Kang > > Signed-off-by: Russ Weight > > Signed-off-by: Xu Yilun > > Signed-off-by: Wu Hao > > --- > > Documentation/ABI/testing/sysfs-platform-dfl-fme | 56 +++++++ > > drivers/fpga/dfl-fme-main.c | 202 +++++++++++++++++++++++ > > 2 files changed, 258 insertions(+) > > > > diff --git a/Documentation/ABI/testing/sysfs-platform-dfl-fme b/Documentation/ABI/testing/sysfs-platform-dfl-fme > > index b8327e9..d3aeb88 100644 > > --- a/Documentation/ABI/testing/sysfs-platform-dfl-fme > > +++ b/Documentation/ABI/testing/sysfs-platform-dfl-fme > > @@ -44,3 +44,59 @@ Description: Read-only. It returns socket_id to indicate which socket > > this FPGA belongs to, only valid for integrated solution. > > User only needs this information, in case standard numa node > > can't provide correct information. > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/temperature > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. It returns temperature (in Celsius) of this FPGA > > + device. > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/threshold1 > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. Read this file to get the temperature threshold1 > > + (in Celsius). > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/threshold2 > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. Read this file to get the temperature threshold2 > > + (in Celsius). > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/trip_threshold > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. It returns trip threshold (in Celsius), once FPGA > > + temperature reaches trip threshold, it triggers a fatal event > > + to board management controller (BMC) to shutdown FPGA. > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/threshold1_status > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. It returns 1 if temperature reaches threshold1, > > + otherwise 0. Once temperature reaches threshold1, hardware > > + will automatically enter throttling state (AP1 - 50% > > + or AP2 - 90% throttling, see 'threshold1_policy'). > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/threshold2_status > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. It returns 1 if temperature reaches threshold2, > > + otherwise 0. Once temperature reaches threshold2, hardware > > + will automatically enter the deepest throttling state (AP6 > > + - 100% throttling). > > + > > +What: /sys/bus/platform/devices/dfl-fme.0/thermal_mgmt/threshold1_policy > > +Date: March 2019 > > +KernelVersion: 5.2 > > +Contact: Wu Hao > > +Description: Read-only. Read this file to get the policy of temperature > > + threshold1. It only supports two value (policy): > > + 0 - AP2 state (90% throttling) > > + 1 - AP1 state (50% throttling) > > These look like they could directly map to the linux thermal framework, > any reason you can't use the thermal framework? > > The trip stuff literally maps 1:1 to what a thermal driver does, I think > that's something you'd wanna consider. > Hi Moritz, Thanks a lot for the suggestion, actually I feel that the trip points in thermal zone are used to indicate cooling actions required for thermal software either in kernel or userspace. But in this case, such FPGA hardware handles cooling automatically (yes, driver only expose Read-only sysfs for information), so software doesn't need to take care of this at all. For this purpose, it seems that we don't have to put these thresholds as trip points. And per my understanding, if people use such FPGA device, then they may need to know what's the current hardware throttling behavior, e.g. 50% vs 90%. These information can't be provided by standard thermal zone sysfs, so anyway user needs these sysfs interfaces to know it. But it seems that we still could create a thermal zone without trip points, it could help if user wants to connect some external cooling devices via userspace thermal daemon, they can define whatever trip points they like to activate the external cooling device. I will consider this further more and come up with a new patch in v2 patchset. Thanks Hao > Cheers, > Moritz