lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 18 May 2022 15:36:59 +0530 From: Abhishek Sahu <abhsahu@...dia.com> To: Alex Williamson <alex.williamson@...hat.com> Cc: Cornelia Huck <cohuck@...hat.com>, Yishai Hadas <yishaih@...dia.com>, Jason Gunthorpe <jgg@...dia.com>, Shameer Kolothum <shameerali.kolothum.thodi@...wei.com>, Kevin Tian <kevin.tian@...el.com>, "Rafael J . Wysocki" <rafael@...nel.org>, Max Gurtovoy <mgurtovoy@...dia.com>, Bjorn Helgaas <bhelgaas@...gle.com>, linux-kernel@...r.kernel.org, kvm@...r.kernel.org, linux-pm@...r.kernel.org, linux-pci@...r.kernel.org Subject: Re: [PATCH v4 4/4] vfio/pci: Move the unused device into low power state with runtime PM On 5/18/2022 1:32 AM, Alex Williamson wrote: > On Tue, 17 May 2022 15:32:19 +0530 > Abhishek Sahu <abhsahu@...dia.com> wrote: >> diff --git a/drivers/vfio/pci/vfio_pci_core.c b/drivers/vfio/pci/vfio_pci_core.c >> index 4fe9a4efc751..5ea1b3099036 100644 >> --- a/drivers/vfio/pci/vfio_pci_core.c >> +++ b/drivers/vfio/pci/vfio_pci_core.c >> @@ -156,7 +156,7 @@ static void vfio_pci_probe_mmaps(struct vfio_pci_core_device *vdev) >> } >> >> struct vfio_pci_group_info; >> -static bool vfio_pci_dev_set_try_reset(struct vfio_device_set *dev_set); >> +static void vfio_pci_dev_set_try_reset(struct vfio_device_set *dev_set); >> static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set, >> struct vfio_pci_group_info *groups); >> >> @@ -275,6 +275,19 @@ void vfio_pci_lock_and_set_power_state(struct vfio_pci_core_device *vdev, >> up_write(&vdev->memory_lock); >> } >> >> +#ifdef CONFIG_PM > > Neither of the CONFIG_PM checks added are actually needed afaict, both > struct dev_pm_ops and the pm pointer on struct device_driver are defined > regardless. Thanks, > > Alex > Yes. These are not needed for build. I will remove these explicit CONFIG_PM checks. Regards, Abhishek
Powered by blists - more mailing lists