Unverified Commit 7dc0c8ae authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!11878 PCI: Add missing bridge lock to pci_bus_lock() 

Merge Pull Request from: @ci-robot 
 
PR sync from: liwei <liwei728@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/ICITJLXLKSGWF2G4IHJ4S5YCW2HJZC74/ 
 
https://gitee.com/src-openeuler/kernel/issues/IARWL9 
 
Link:https://gitee.com/openeuler/kernel/pulls/11878

 

Reviewed-by: default avatarXie XiuQi <xiexiuqi@huawei.com>
Reviewed-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
Signed-off-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
parents 63731a45 157b2fb1

drivers/pci/pci.c

+21 −14
Original line number Diff line number Diff line
@@ -5035,10 +5035,12 @@ static void pci_bus_lock(struct pci_bus *bus) 
{

	struct pci_dev *dev;



	pci_dev_lock(bus->self);

	list_for_each_entry(dev, &bus->devices, bus_list) {

		pci_dev_lock(dev);

		if (dev->subordinate)

			pci_bus_lock(dev->subordinate);

		else

			pci_dev_lock(dev);

	}

}
@@ -5050,8 +5052,10 @@ static void pci_bus_unlock(struct pci_bus *bus) 
	list_for_each_entry(dev, &bus->devices, bus_list) {

		if (dev->subordinate)

			pci_bus_unlock(dev->subordinate);

		else

			pci_dev_unlock(dev);

	}

	pci_dev_unlock(bus->self);

}



/* Return 1 on successful lock, 0 on contention */
@@ -5059,15 +5063,15 @@ static int pci_bus_trylock(struct pci_bus *bus) 
{

	struct pci_dev *dev;



	if (!pci_dev_trylock(bus->self))

		return 0;



	list_for_each_entry(dev, &bus->devices, bus_list) {

		if (!pci_dev_trylock(dev))

			goto unlock;

		if (dev->subordinate) {

			if (!pci_bus_trylock(dev->subordinate)) {

				pci_dev_unlock(dev);

			if (!pci_bus_trylock(dev->subordinate))

				goto unlock;

		} else if (!pci_dev_trylock(dev))

			goto unlock;

			}

		}

	}

	return 1;
@@ -5075,8 +5079,10 @@ static int pci_bus_trylock(struct pci_bus *bus) 
	list_for_each_entry_continue_reverse(dev, &bus->devices, bus_list) {

		if (dev->subordinate)

			pci_bus_unlock(dev->subordinate);

		else

			pci_dev_unlock(dev);

	}

	pci_dev_unlock(bus->self);

	return 0;

}
@@ -5108,9 +5114,10 @@ static void pci_slot_lock(struct pci_slot *slot) 
	list_for_each_entry(dev, &slot->bus->devices, bus_list) {

		if (!dev->slot || dev->slot != slot)

			continue;

		pci_dev_lock(dev);

		if (dev->subordinate)

			pci_bus_lock(dev->subordinate);

		else

			pci_dev_lock(dev);

	}

}
@@ -5136,14 +5143,13 @@ static int pci_slot_trylock(struct pci_slot *slot) 
	list_for_each_entry(dev, &slot->bus->devices, bus_list) {

		if (!dev->slot || dev->slot != slot)

			continue;

		if (!pci_dev_trylock(dev))

			goto unlock;

		if (dev->subordinate) {

			if (!pci_bus_trylock(dev->subordinate)) {

				pci_dev_unlock(dev);

				goto unlock;

			}

		}

		} else if (!pci_dev_trylock(dev))

			goto unlock;

	}

	return 1;
@@ -5154,6 +5160,7 @@ static int pci_slot_trylock(struct pci_slot *slot) 
			continue;

		if (dev->subordinate)

			pci_bus_unlock(dev->subordinate);

		else

			pci_dev_unlock(dev);

	}

	return 0;