diff mbox series

vfio/pci: Also demote hiding standard cap messages

Message ID 20230523225250.1215911-1-alex.williamson@redhat.com (mailing list archive)
State New, archived
Headers show
Series vfio/pci: Also demote hiding standard cap messages | expand

Commit Message

Alex Williamson May 23, 2023, 10:52 p.m. UTC
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(-)

Comments

Cédric Le Goater May 24, 2023, 7:52 a.m. UTC | #1
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;
Oleksandr Natalenko May 24, 2023, 8:01 a.m. UTC | #2
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.
diff mbox series

Patch

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;