summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMaciej Sosnowski <maciej.sosnowski@intel.com>2012-05-23 17:27:07 +0200
committerVinod Koul <vinod.koul@intel.com>2013-01-07 22:05:14 -0800
commitc419fcfd071cf34ba00f9f65282583772d2655e7 (patch)
tree76c2586c43b85b63faacd3686220ca9fae85cde5
parent6decffd5f6afaf55722d9c85b8739621dca63d0f (diff)
downloadop-kernel-dev-c419fcfd071cf34ba00f9f65282583772d2655e7.zip
op-kernel-dev-c419fcfd071cf34ba00f9f65282583772d2655e7.tar.gz
dca: check against empty dca_domains list before unregister provider
When providers get blocked unregister_dca_providers() is called ending up with dca_providers and dca_domain lists emptied. Dca should be prevented from trying to unregister any provider if dca_domain list is found empty. Cc: <stable@vger.kernel.org> Reported-by: Jiang Liu <jiang.liu@huawei.com> Tested-by: Gaohuai Han <hangaohuai@huawei.com> Signed-off-by: Maciej Sosnowski <maciej.sosnowski@intel.com> Signed-off-by: Dan Williams <djbw@fb.com>
-rw-r--r--drivers/dca/dca-core.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/dca/dca-core.c b/drivers/dca/dca-core.c
index bc6f5fa..819dfda 100644
--- a/drivers/dca/dca-core.c
+++ b/drivers/dca/dca-core.c
@@ -420,6 +420,11 @@ void unregister_dca_provider(struct dca_provider *dca, struct device *dev)
raw_spin_lock_irqsave(&dca_lock, flags);
+ if (list_empty(&dca_domains)) {
+ raw_spin_unlock_irqrestore(&dca_lock, flags);
+ return;
+ }
+
list_del(&dca->node);
pci_rc = dca_pci_rc_from_dev(dev);
OpenPOWER on IntegriCloud