linux-kernel - Re: [PATCH v4 0/9] misc: Add AMD side band interface(SBI) functionality

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] [day] [month] [year] [list]

Message-ID: <2024092116-dental-chamber-e978@gregkh>
Date: Sat, 21 Sep 2024 11:01:47 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: "Gupta, Akshay" <Akshay.Gupta@....com>
Cc: linux-hwmon@...r.kernel.org, linux-kernel@...r.kernel.org,
	arnd@...db.de, linux@...ck-us.net, naveenkrishna.chatradhi@....com
Subject: Re: [PATCH v4 0/9] misc: Add AMD side band interface(SBI)
 functionality

On Fri, Sep 20, 2024 at 12:15:37PM +0530, Gupta, Akshay wrote:
> 
> On 9/12/2024 12:38 PM, Akshay Gupta wrote:
> > AMD's Advanced Platform Management Link (APML) interface provides system
> > management functionality accessed by the baseboard management controller (BMC).
> > sbrmi driver under hwmon subsystem, which is probed as an i2c driver and
> > reports power using APML specified protocol.
> > However, APML interface defines few other protocols to support
> > full system management functionality out-of-band.
> > Out-of-band management is term used for BMC talking to system management unit
> > (IP in the processor). AMD's documentation called this link as side band interface.
> > 
> > This patchset is an attempt to add all APML core functionality in one place
> > and provide hwmon and user space interface
> > 1. [Patch 1] Move the i2c client probe, hwmon sensors and sbrmi core functionality
> >     from drivers/hwmon to drivers/misc/
> > 2. [Patch 2] Convert i2c to regmap which provides multiple benefits
> >     over direct smbus APIs.
> >      a. i2c/i3c support and
> >      b. 1 byte/2 byte RMI register size addressing
> > 3. [Patch 3] Optimize wait condition with regmap API regmap_read_poll_timeout as per
> >     suggestion from Arnd
> > 4. [Patch 4, 5] Register a misc device which provides
> >      a. An ioctl interface through node /dev/sbrmiX
> >      b. Register sets is common across APML protocols. IOCTL is providing
> >         synchronization among protocols as transactions may create
> >         race condition.
> > 5. [Subsequent patches 6, 7 and 8] add support for AMD custom protocols
> >      a. CPUID
> >      b. MCAMSR
> >      c. Register xfer
> > 6. [Patch 9] AMD side band description document
> > 
> > Open-sourced and widely used https://github.com/amd/esmi_oob_library
> > will continue to provide user-space programmable API.
> > 
> > Akshay Gupta (9):
> >    hwmon/misc: amd-sbi: Move core sbrmi from hwmon to misc
> >    misc: amd-sbi: Use regmap subsystem
> >    misc: amd-sbi: Optimize the wait condition for mailbox command
> >      completion
> >    misc: amd-sbi: Add support for AMD_SBI IOCTL
> >    misc: amd-sbi: Add support for mailbox error codes
> >    misc: amd-sbi: Add support for CPUID protocol
> >    misc: amd-sbi: Add support for MCA register protocol
> >    misc: amd-sbi: Add supoort for register xfer
> >    misc: amd-sbi: Add document for AMD SB IOCTL description
> 
> Hi Greg, Arnd,
> 
> You have previously reviewed v3 of patch set and I have addressed the review
> comments in v4.
> 
> Can you please take review v4 patch set?

Please wait until after the merge window is over (i.e. -rc1 is out) as
we can't do anything until then as you know.  Also remember that most of
us are traveling all of this week (and some of us next week) for
conferences, and will get to the backlog when we return.

To help us out, please go through the lists and review other submissions
for these subsystems so that your changes move to the top of unreviewed
ones.

thanks,

greg k-h