[PATCH] power/amd_uncore: add e-smi installation instructions

[Tummala, Sivaprasad]

[AMD Official Use Only - AMD Internal Distribution Only]

Hi Stephen,

Thanks for your comments. Please find my reply inline.

> -----Original Message-----
> From: Stephen Hemminger <stephen at networkplumber.org>
> Sent: Sunday, March 29, 2026 6:09 AM
> To: Tummala, Sivaprasad <Sivaprasad.Tummala at amd.com>
> Cc: david.hunt at intel.com; anatoly.burakov at intel.com; jerinj at marvell.com;
> radu.nicolau at intel.com; gakhil at marvell.com; cristian.dumitrescu at intel.com; Yigit,
> Ferruh <Ferruh.Yigit at amd.com>; konstantin.ananyev at huawei.com;
> thomas at monjalon.net; dev at dpdk.org
> Subject: Re: [PATCH] power/amd_uncore: add e-smi installation instructions
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> On Fri, 29 Nov 2024 04:32:07 +0000
> Sivaprasad Tummala <sivaprasad.tummala at amd.com> wrote:
>
> > Added section for installing and building the E-SMI library for AMD
> > EPYC Uncore support and version requirements.
> >
> > Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala at amd.com>
>
> A lot has changed since this patch was originally submitted, and apologies for the
> long delay in getting back to it. The E-SMI library is in much better shape now — it's
> packaged in Fedora and EPEL, v4.2+ bundles the amd_hsmp.h header that was
> causing build failures, and upstream now has pkg-config support.
>
> Please revise the patch (and the meson.build) to match current E-SMI upstream.
> The dependency check should use dependency('e_smi', required: false) via pkg-
> config rather than the bare cc.find_library() that's there now.
> The current ESMI_header variable in the build file is defined but never tested — that
> should be cleaned up too.
Updated patch:
https://patches.dpdk.org/project/dpdk/patch/20260331040958.126869-1-sivaprasad.tummala@amd.com/
>
> For the documentation, just state the minimum E-SMI version required and link to
> the upstream project. No need to reproduce build instructions; that's the upstream
> project's job, and distro packages make it even easier.
>
> It would be great to get this updated in time for the 26.11 LTS release.
Thanks for the suggestion. I will follow-up on this and send an updated patch for the same.
It would be great to get this updated in time for the 26.11 LTS release.