ACK: [PATCH][V4] klog.json: add a "Unable ... IOMMU perf counter" message

[Colin Ian King]

On 28/04/2020 19:41, Alex Hung wrote:
> This message shows performance counters are not supported on AMD IOMMU
> hardware.
> 
> More info @ https://lore.kernel.org/lkml/20180727102710.GA6738@8bytes.org/
> 
> Reported-by: Paul Menzel <pmenzel at molgen.mpg.de>
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
> 
> [V2] update advise and commit comments
> [V3] change spec -> specifications
> [V4] should not be -> should be
> 
>  data/klog.json | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/data/klog.json b/data/klog.json
> index 15043658..cdb93554 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -2697,6 +2697,13 @@ atform Communications Channel Table (PCCT) and find the PCC subspace communicati
>      "advice": "BIOS setup is configured to enable IOMMU, but BIOS lacks IVRS table that is describing which is the address of IOMMU",
>      "label": "KlogAmdIommuNoIVRS"
>    },
> +  {
> +    "compare_mode": "string",
> +    "log_level": "LOG_LEVEL_MEDIUM",
> +    "pattern": "Unable to read/write to IOMMU perf counter",
> +    "advice": "AMD IOMMU performance counter is inaccessible and public specifications do not have information on how it should be supported",
> +    "label": "KlogAmdIommuPCNotSupported"
> +  },
>    {
>      "compare_mode": "string",
>      "log_level": "LOG_LEVEL_HIGH",
> 

Thanks for the V4 fix up.

Acked-by: Colin Ian King <colin.king at canonical.com>