Re: [PATCH v2 14/16] x86/amd_node, platform/x86/amd/hsmp: Have HSMP use SMN through AMD_NODE

From: Yazen Ghannam
Date: Wed Dec 11 2024 - 11:14:14 EST


On Mon, Dec 09, 2024 at 03:32:02PM +0200, Ilpo Järvinen wrote:
> On Fri, 6 Dec 2024, Yazen Ghannam wrote:

[...]

> > --- a/drivers/platform/x86/amd/hsmp/acpi.c
> > +++ b/drivers/platform/x86/amd/hsmp/acpi.c
> > @@ -10,7 +10,6 @@
> > #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> >
> > #include <asm/amd_hsmp.h>
> > -#include <asm/amd_nb.h>
> >
> > #include <linux/acpi.h>
> > #include <linux/device.h>
> > @@ -24,6 +23,8 @@
> >
> > #include <uapi/asm-generic/errno-base.h>
> >
> > +#include <asm/amd_node.h>
> > +
> > #include "hsmp.h"
> >
> > #define DRIVER_NAME "amd_hsmp"
> > @@ -321,8 +322,8 @@ static int hsmp_acpi_probe(struct platform_device *pdev)
> > return -ENOMEM;
> >
> > if (!hsmp_pdev->is_probed) {
> > - hsmp_pdev->num_sockets = amd_nb_num();
> > - if (hsmp_pdev->num_sockets == 0 || hsmp_pdev->num_sockets > MAX_AMD_SOCKETS)
> > + hsmp_pdev->num_sockets = amd_num_nodes();
> > + if (hsmp_pdev->num_sockets == 0 || hsmp_pdev->num_sockets > MAX_AMD_NUM_NODES)
>
> Hi,
>
> Now that this define moves outside of hsmp files, it would be useful to
> add static_assert() and a comment next to the attributes which are always
> created for 0-7 sockets. That way, it can be detected on the build time if
> the define changes and more entires would need to be added there.
>
> Other than that, this seemed okay.
>

Okay, will do.

Thanks,
Yazen