Re: [PATCH v8 6/6] AMD/IOMMU: expose errors and warnings unconditionally

2021-09-28 Thread Jan Beulich
On 28.09.2021 09:42, Durrant, Paul wrote: > On 22/09/2021 15:38, Jan Beulich wrote: >> Making these dependent upon "iommu=debug" isn't really helpful in the >> field. Where touching respective code anyway also make use of %pp and >> %pd. >> >> Requested-by: Andrew Cooper >> Signed-off-by: Jan

Re: [PATCH v8 6/6] AMD/IOMMU: expose errors and warnings unconditionally

2021-09-28 Thread Durrant, Paul
On 22/09/2021 15:38, Jan Beulich wrote: Making these dependent upon "iommu=debug" isn't really helpful in the field. Where touching respective code anyway also make use of %pp and %pd. Requested-by: Andrew Cooper Signed-off-by: Jan Beulich Reviewed-by: Paul Durrant ... with one nit

[PATCH v8 6/6] AMD/IOMMU: expose errors and warnings unconditionally

2021-09-22 Thread Jan Beulich
Making these dependent upon "iommu=debug" isn't really helpful in the field. Where touching respective code anyway also make use of %pp and %pd. Requested-by: Andrew Cooper Signed-off-by: Jan Beulich --- While I'm adding AMD_IOMMU_VERBOSE(), there aren't any uses for now. It's not really clear