HI Boris,
On 9/2/2021 11:28 PM, Borislav Petkov wrote:
[CAUTION: External Email]
On Thu, Sep 02, 2021 at 11:11:53PM +0530, Naveen Krishna Chatradhi wrote:
On Fam19h server CPUs from AMD an HSMP (Host System Management Port)
mailbox interface is added to monitor and manage the CPU parameters.
Mailbox? Hmm, that rings a bell. There's drivers/mailbox/ with a bunch
of drivers in there. Those are
"Mailbox is a framework to control hardware communication between
on-chip processors through queued messages and interrupt driven
signals."
I wonder if that HSMP thing can be wrapped like a mailbox driver...
Sure, I will look into the mailbox framework.
If not, I still don't like it being slapped in amd_nb.c where it would
be built in on *all* AMD hardware which is kinda too much.
In all the future server platforms, AMD's direction is the support HSMP
interface, which exposes system management knobs.
You could make it a separate driver module called amd_hsmp.ko which
loads only on the appropriate hw and uses amd_nb.c for detection only
like the other drivers, for example.
How about, creating a module under drivers/platform/x86/ (lets say
amd_hsmp.c) export an API from here and it can be extended to support
all the knobs that does not fit in an existing frameworks (such as
hwmon, etc) and provide a user space access.
I can see similar references in the drivers/platform/x86/ directory.
--
Regards/Gruss,
Boris.
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpeople.kernel.org%2Ftglx%2Fnotes-about-netiquette&data=04%7C01%7CNaveenKrishna.Chatradhi%40amd.com%7C52aacf76c0d74c526ade08d96e3b3765%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637662022905004162%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DYcj%2BcjedWpUaJYMl7yPDy%2BB%2FIe6YDucA00JcJtfLFk%3D&reserved=0