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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1A86CC433F5 for ; Tue, 5 Oct 2021 04:08:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F3A84613E6 for ; Tue, 5 Oct 2021 04:08:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230097AbhJEEKd (ORCPT ); Tue, 5 Oct 2021 00:10:33 -0400 Received: from mga17.intel.com ([192.55.52.151]:5183 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229446AbhJEEKb (ORCPT ); Tue, 5 Oct 2021 00:10:31 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10127"; a="206462198" X-IronPort-AV: E=Sophos;i="5.85,347,1624345200"; d="scan'208";a="206462198" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Oct 2021 21:08:41 -0700 X-IronPort-AV: E=Sophos;i="5.85,347,1624345200"; d="scan'208";a="487835896" Received: from yilai-mobl1.ccr.corp.intel.com (HELO chenyu5-mobl1) ([10.249.172.101]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 04 Oct 2021 21:08:35 -0700 Date: Tue, 5 Oct 2021 12:08:31 +0800 From: Chen Yu To: "Rafael J. Wysocki" Cc: Greg Kroah-Hartman , ACPI Devel Maling List , Linux Kernel Mailing List , Len Brown , Dan Williams , Andy Shevchenko , Aubrey Li , Ashok Raj , Mike Rapoport , Ard Biesheuvel , Jonathan Corbet , Hans de Goede , Ben Widawsky , "open list:DOCUMENTATION" Subject: Re: [PATCH v3 3/5] drivers/acpi: Introduce Platform Firmware Runtime Update device driver Message-ID: <20211005040831.GA7134@chenyu5-mobl1> References: <90d270c031401430445cb2c4ba1b9b0c265cf9d4.1631802163.git.yu.c.chen@intel.com> <20210922090442.GA16963@chenyu5-mobl1> <20210922163321.GA31068@chenyu5-mobl1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 27, 2021 at 07:40:39PM +0200, Rafael J. Wysocki wrote: > On Wed, Sep 22, 2021 at 7:28 PM Greg Kroah-Hartman > wrote: > > > > On Thu, Sep 23, 2021 at 12:33:21AM +0800, Chen Yu wrote: > > > On Wed, Sep 22, 2021 at 11:10:02AM +0200, Greg Kroah-Hartman wrote: > > > > On Wed, Sep 22, 2021 at 05:04:42PM +0800, Chen Yu wrote: > > > > > Hi Greg, > > > > > On Tue, Sep 21, 2021 at 05:59:05PM +0200, Greg Kroah-Hartman wrote: > > > > > > On Fri, Sep 17, 2021 at 12:02:18AM +0800, Chen Yu wrote: > > > > > > > Introduce the pfru_update driver which can be used for Platform Firmware > > > > > > > Runtime code injection and driver update. The user is expected to provide > > > > > > > the update firmware in the form of capsule file, and pass it to the driver > > > > > > > via ioctl. Then the driver would hand this capsule file to the Platform > > > > > > > Firmware Runtime Update via the ACPI device _DSM method. At last the low > > > > > > > level Management Mode would do the firmware update. > > > > > > > > > > > > > > Signed-off-by: Chen Yu > > > > > > > > > > > > Where is the userspace code that uses this ioctl and has tested it out > > > > > > to verify it works properly? A link to that in the changelog would be > > > > > > great to have. > > > > > > > > > > > The patch [5/5] is a self testing tool to test the whole feature. I'll send a > > > > > new version and Cc you too. > > > > > > > > That tests it, but does not answer the question of who will actually use > > > > this. What userspace tool needs this new api? > > > > > > > Currently there is no dedicated userspace tool developed to use this > > > feature AFAIK. > > > > Wonderful, then it is not needed to be added to the kernel :) > > [snip] > > > It was expected that the end users > > > could refer to the self test tool to customize their tools. I'm not sure if > > > this is the proper way to propose the feature, may I have your suggestion on > > > this, should I create a separate git repository for this tool, or put it in > > > tools/selftestings as it is now? > > > > No, do not add this to the kernel unless you have a real need and user > > for this. After revisiting this patch set, I'll revise it to better describe the background and usage model, and also propose the user space tool to fully demonstrate how to use this feature. thanks, Chenyu