<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 3, 2020 at 3:00 PM Paul Menzel <<a href="mailto:pmenzel@molgen.mpg.de">pmenzel@molgen.mpg.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Dear Alex,<br>
<br>
<br>
Am 31.03.20 um 03:21 schrieb Alex Hung:<br>
> This message shows performance counters are not supported on AMD IOMMU<br>
> hardware.<br>
> <br>
> Reported-by: Paul Menzel <<a href="mailto:pmenzel@molgen.mpg.de" target="_blank">pmenzel@molgen.mpg.de</a>><br>
> Signed-off-by: Alex Hung <<a href="mailto:alex.hung@canonical.com" target="_blank">alex.hung@canonical.com</a>><br>
> ---<br>
>   data/klog.json | 7 +++++++<br>
>   1 file changed, 7 insertions(+)<br>
> <br>
> diff --git a/data/klog.json b/data/klog.json<br>
> index 15043658..eb78fe47 100644<br>
> --- a/data/klog.json<br>
> +++ b/data/klog.json<br>
> @@ -2697,6 +2697,13 @@ atform Communications Channel Table (PCCT) and find the PCC subspace communicati<br>
>       "advice": "BIOS setup is configured to enable IOMMU, but BIOS lacks IVRS table that is describing which is the address of IOMMU",<br>
>       "label": "KlogAmdIommuNoIVRS"<br>
>     },<br>
> +  {<br>
> +    "compare_mode": "string",<br>
> +    "log_level": "LOG_LEVEL_MEDIUM",<br>
> +    "pattern": "Unable to read/write to IOMMU perf counter",<br>
> +    "advice": "AMD IOMMU performance counter is not accessible and not supported due to firmware limitations.",<br>
<br>
By the way,<br>
<br>
<a href="https://lore.kernel.org/lkml/20180727102710.GA6738@8bytes.org/" rel="noreferrer" target="_blank">https://lore.kernel.org/lkml/20180727102710.GA6738@8bytes.org/</a><br>
<br>
lists the maintainers answer. Maybe the advice can be more direct.<br></blockquote><div><br></div><div><div style="font-family:verdana,sans-serif" class="gmail_default">It says the public version of AMD's BKDG has no references. <br></div><div style="font-family:verdana,sans-serif" class="gmail_default"><br></div><div style="font-family:verdana,sans-serif" class="gmail_default">Do you prefer the advice to be something like " AMD IOMMU performance counter is not accessible and not supported due to limitation of AMD's public document" instead?<br></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> AMD IOMMU performance counter is not accessible due to a bug in the system firmware.<br>
<br>
> +    "label": "KlogAmdIommuPCNotSupported"<br>
> +  },<br>
>     {<br>
>       "compare_mode": "string",<br>
>       "log_level": "LOG_LEVEL_HIGH",<br>
<br>
Tested-by: Paul Menzel <<a href="mailto:pmenzel@molgen.mpg.de" target="_blank">pmenzel@molgen.mpg.de</a>><br>
<br>
<br>
Kind regards,<br>
<br>
Paul<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Cheers,<br>Alex Hung<br></div></div></div>