[PATCH] vfio/pci: Also demote hiding standard cap messages

[PATCH] vfio/pci: Also demote hiding standard cap messages

[Alex Williamson]

Apply the same logic as commit 912b625b4dcf ("vfio/pci: demote hiding
ecap messages to debug level") for the less common case of hiding
standard capabilities.

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
---
 drivers/vfio/pci/vfio_pci_config.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c
index 1d95fe435f0e..7e2e62ab0869 100644
--- a/drivers/vfio/pci/vfio_pci_config.c
+++ b/drivers/vfio/pci/vfio_pci_config.c
@@ -1566,8 +1566,8 @@ static int vfio_cap_init(struct vfio_pci_core_device *vdev)
 		}
 
 		if (!len) {
-			pci_info(pdev, "%s: hiding cap %#x@%#x\n", __func__,
-				 cap, pos);
+			pci_dbg(pdev, "%s: hiding cap %#x@%#x\n", __func__,
+				cap, pos);
 			*prev = next;
 			pos = next;
 			continue;
-- 
2.39.2

Re: [PATCH] vfio/pci: Also demote hiding standard cap messages

[Cédric Le Goater]

On 5/24/23 00:52, Alex Williamson wrote:
> Apply the same logic as commit 912b625b4dcf ("vfio/pci: demote hiding
> ecap messages to debug level") for the less common case of hiding
> standard capabilities.
> 
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>

Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.

> ---
>   drivers/vfio/pci/vfio_pci_config.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c
> index 1d95fe435f0e..7e2e62ab0869 100644
> --- a/drivers/vfio/pci/vfio_pci_config.c
> +++ b/drivers/vfio/pci/vfio_pci_config.c
> @@ -1566,8 +1566,8 @@ static int vfio_cap_init(struct vfio_pci_core_device *vdev)
>   		}
>   
>   		if (!len) {
> -			pci_info(pdev, "%s: hiding cap %#x@%#x\n", __func__,
> -				 cap, pos);
> +			pci_dbg(pdev, "%s: hiding cap %#x@%#x\n", __func__,
> +				cap, pos);
>   			*prev = next;
>   			pos = next;
>   			continue;

Re: [PATCH] vfio/pci: Also demote hiding standard cap messages

[Oleksandr Natalenko]

[-- Attachment #1: Type: text/plain, Size: 1078 bytes --]

On středa 24. května 2023 0:52:50 CEST Alex Williamson wrote:
> Apply the same logic as commit 912b625b4dcf ("vfio/pci: demote hiding
> ecap messages to debug level") for the less common case of hiding
> standard capabilities.
> 
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> ---
>  drivers/vfio/pci/vfio_pci_config.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c
> index 1d95fe435f0e..7e2e62ab0869 100644
> --- a/drivers/vfio/pci/vfio_pci_config.c
> +++ b/drivers/vfio/pci/vfio_pci_config.c
> @@ -1566,8 +1566,8 @@ static int vfio_cap_init(struct vfio_pci_core_device *vdev)
>  		}
>  
>  		if (!len) {
> -			pci_info(pdev, "%s: hiding cap %#x@%#x\n", __func__,
> -				 cap, pos);
> +			pci_dbg(pdev, "%s: hiding cap %#x@%#x\n", __func__,
> +				cap, pos);
>  			*prev = next;
>  			pos = next;
>  			continue;

Reviewed-by: Oleksandr Natalenko <oleksandr@natalenko.name>

Thank you.

-- 
Oleksandr Natalenko (post-factum)

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]