From: Hans de Goede <hdegoede@redhat.com>
To: Jorge Lopez <jorgealtxwork@gmail.com>,
platform-driver-x86@vger.kernel.org,
linux-kernel@vger.kernel.org, thomas@t-8ch.de,
ilpo.jarvinen@linux.intel.com
Subject: Re: [PATCH v17 00/13] hp-bioscfg driver
Date: Tue, 4 Jul 2023 12:43:07 +0200 [thread overview]
Message-ID: <43369522-5e39-a90c-5263-cd4eacba3e24@redhat.com> (raw)
In-Reply-To: <20230608163319.18934-1-jorge.lopez2@hp.com>
Hi,
On 6/8/23 18:33, Jorge Lopez wrote:
> HP BIOS Configuration driver purpose is to provide a driver supporting
> the latest sysfs class firmware attributes framework allowing the user
> to change BIOS settings and security solutions on HP Inc.’s commercial
> notebooks.
>
> Many features of HP Commercial notebooks can be managed using Windows
> Management Instrumentation (WMI). WMI is an implementation of Web-Based
> Enterprise Management (WBEM) that provides a standards-based interface
> for changing and monitoring system settings. HP BIOSCFG driver provides
> a native Linux solution and the exposed features facilitates the
> migration to Linux environments.
>
> The Linux security features to be provided in hp-bioscfg driver enables
> managing the BIOS settings and security solutions via sysfs, a virtual
> filesystem that can be used by user-mode applications. The new
> documentation cover HP-specific firmware sysfs attributes such Secure
> Platform Management and Sure Start. Each section provides security
> feature description and identifies sysfs directories and files exposed
> by the driver.
>
> Many HP Commercial notebooks include a feature called Secure Platform
> Management (SPM), which replaces older password-based BIOS settings
> management with public key cryptography. PC secure product management
> begins when a target system is provisioned with cryptographic keys
> that are used to ensure the integrity of communications between system
> management utilities and the BIOS.
>
> HP Commercial notebooks have several BIOS settings that control its
> behaviour and capabilities, many of which are related to security.
> To prevent unauthorized changes to these settings, the system can
> be configured to use a cryptographic signature-based authorization
> string that the BIOS will use to verify authorization to modify the
> setting.
>
> Linux Security components are under development and not published yet.
> The only linux component is the driver (hp bioscfg) at this time.
> Other published security components are under Windows.
>
> Signed-off-by: Jorge Lopez <jorge.lopez2@hp.com>
Thank you for your patch, I've applied this patch to my review-hans
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans
Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.
Patches which are added to review-hans now are intended for
the next kernel cycle. This branch will get rebased to the next
rc1 when it is out and after the rebasing the contents of review-hans
will be pushed to the platform-drivers-x86/for-next branch.
Regards,
Hans
>
> ---
> Based on the latest platform-drivers-x86.git/for-next
>
> History
>
> Version 17
> Only patches marked [update] changed between version 17 and 16
> Added helper routine to handle all common data for all attributes.
> Corrected spaces found prior and after a '"'.
> Update commit message to reflect comments from reviewers
>
>
> Patches
> Documentation [update]
> biosattr-interface
> bioscfg [update]
> bioscfg-h [update]
> enum-attributes [update]
> int-attributes [update]
> order-list-attributes [update]
> passwdattr-interface [update]
> spmobj-attributes
> string-attributes [update]
> surestart-attributes
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
> Version 16
> Only patches marked [update] changed between version 16 and 15
>
> Patches
> Documentation
> biosattr-interface
> bioscfg [update]
> bioscfg-h [update]
> enum-attributes [update]
> int-attributes [update]
> order-list-attributes [update]
> passwdattr-interface
> spmobj-attributes
> string-attributes [update]
> surestart-attributes
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
>
> Version 15
> Only patches marked [update] changed between version 15 and 14
>
> Patches
> Documentation [update]
> biosattr-interface
> bioscfg
> bioscfg-h
> enum-attributes
> int-attributes
> order-list-attributes
> passwdattr-interface
> spmobj-attributes [update]
> string-attributes
> surestart-attributes
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
> Version 14
> Only patches marked [update] changed between version 14 and 13
> Sorted commit patches alphabetically
> Rename ordered-attributes to order-list-attributes
>
> Patches
> Documentation
> biosattr-interface [update]
> bioscfg
> bioscfg-h
> enum-attributes [update]
> int-attributes
> order-list-attributes
> passwdattr-interface
> spmobj-attributes
> string-attributes
> surestart-attributes
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
>
> Version 13
> Only patches marked [update] changed between version 12 and 13
> Sorted commit patches alphabetically
> Rename ordered-attributes to order-list-attributes
>
> Patches
> Documentation [update]
> biosattr-interface [update]
> bioscfg [update]
> bioscfg-h [update]
> enum-attributes [update]
> int-attributes [update]
> order-list-attributes [update]
> passwdattr-interface [update]
> spmobj-attributes [update]
> string-attributes [update]
> surestart-attributes [update]
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
> Version 12
> Only patches marked [update] changed between version 11 and 12
>
> Patches
> Documentation [update]
> biosattr-interface [update]
> bioscfg [update]
> int-attributes [update]
> ordered-attributes [update]
> passwdobj-attributes [deleted]
> string-attributes [update]
> bioscfg-h [update]
> enum-attributes [update]
> passwdattr-interface [update]
> spmobj-attributes [update]
> surestart-attributes [update]
> Makefile ../hp/Makefile ../hp/Kconfig [update]
> MAINTAINERS
>
>
> Version 11
> Only patches marked [update] changed between version 10 and 11
>
> Patches
> Documentation
> biosattr-interface [update]
> bioscfg
> int-attributes
> ordered-attributes
> passwdobj-attributes [update]
> string-attributes
> bioscfg-h
> enum-attributes
> passwdattr-interface
> spmobj-attributes [update]
> surestart-attributes [update]
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
> Version 10
> Break down changes to single files per patch
> Removed SPM/statusbin support
> Patches
> Documentation
> biosattr-interface
> bioscfg
> int-attributes
> ordered-attributes
> passwdobj-attributes
> string-attributes
> bioscfg-h
> enum-attributes
> passwdattr-interface
> spmobj-attributes
> surestart-attributes
> Makefile ../hp/Makefile ../hp/Kconfig
> MAINTAINERS
>
> Version 9
> Includes only sysfs-class-firmware-attributes documentation
>
> Version 8
> Includes only sysfs-class-firmware-attributes documentation
>
> Version 7
> Includes only sysfs-class-firmware-attributes documentation
>
>
> Jorge Lopez (13):
> hp-bioscfg: Documentation
> hp-bioscfg: bioscfg-h
> hp-bioscfg: bioscfg
> hp-bioscfg: biosattr-interface
> hp-bioscfg: enum-attributes
> hp-bioscfg: int-attributes
> hp-bioscfg: order-list-attributes
> hp-bioscfg: passwdobj-attributes
> hp-bioscfg: spmobj-attributes
> hp-bioscfg: string-attributes
> hp-bioscfg: surestart-attributes
> hp-bioscfg: Makefile
> hp-bioscfg: MAINTAINERS
>
> .../testing/sysfs-class-firmware-attributes | 101 +-
> MAINTAINERS | 6 +
> drivers/platform/x86/hp/Kconfig | 16 +
> drivers/platform/x86/hp/Makefile | 1 +
> drivers/platform/x86/hp/hp-bioscfg/Makefile | 11 +
> .../x86/hp/hp-bioscfg/biosattr-interface.c | 312 +++++
> drivers/platform/x86/hp/hp-bioscfg/bioscfg.c | 1055 +++++++++++++++++
> drivers/platform/x86/hp/hp-bioscfg/bioscfg.h | 487 ++++++++
> .../x86/hp/hp-bioscfg/enum-attributes.c | 447 +++++++
> .../x86/hp/hp-bioscfg/int-attributes.c | 409 +++++++
> .../x86/hp/hp-bioscfg/order-list-attributes.c | 436 +++++++
> .../x86/hp/hp-bioscfg/passwdobj-attributes.c | 543 +++++++++
> .../x86/hp/hp-bioscfg/spmobj-attributes.c | 386 ++++++
> .../x86/hp/hp-bioscfg/string-attributes.c | 390 ++++++
> .../x86/hp/hp-bioscfg/surestart-attributes.c | 132 +++
> 15 files changed, 4730 insertions(+), 2 deletions(-)
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/Makefile
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/biosattr-interface.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/bioscfg.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/bioscfg.h
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/int-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/order-list-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/spmobj-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/string-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/surestart-attributes.c
>
prev parent reply other threads:[~2023-07-04 10:44 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-08 16:33 Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 01/13] hp-bioscfg: Documentation Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 02/13] hp-bioscfg: bioscfg-h Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 03/13] hp-bioscfg: bioscfg Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 04/13] hp-bioscfg: biosattr-interface Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 05/13] hp-bioscfg: enum-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 06/13] hp-bioscfg: int-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 07/13] hp-bioscfg: order-list-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 08/13] hp-bioscfg: passwdobj-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 09/13] hp-bioscfg: spmobj-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 10/13] hp-bioscfg: string-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 11/13] hp-bioscfg: surestart-attributes Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 12/13] hp-bioscfg: Makefile Jorge Lopez
2023-06-08 16:33 ` [PATCH v17 13/13] hp-bioscfg: MAINTAINERS Jorge Lopez
2023-06-18 18:53 ` [PATCH v17 00/13] hp-bioscfg driver Thomas Weißschuh
2023-07-04 10:43 ` Hans de Goede [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=43369522-5e39-a90c-5263-cd4eacba3e24@redhat.com \
--to=hdegoede@redhat.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=jorgealtxwork@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=thomas@t-8ch.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®