diff options
| author | Ezequiel Garcia <ezequiel.garcia@free-electrons.com> | 2014-07-08 10:37:37 -0300 |
|---|---|---|
| committer | Jason Cooper <jason@lakedaemon.net> | 2014-07-08 13:53:53 +0000 |
| commit | a728b977429383b3fe92b6e3bff9e69365609e0f (patch) | |
| tree | 114e418d5816073e9eff0f7fd9fcd6926512fd5e | |
| parent | 0d461e1b087048b0cc37c9d7b351649578c507b4 (diff) | |
| download | op-kernel-dev-a728b977429383b3fe92b6e3bff9e69365609e0f.zip op-kernel-dev-a728b977429383b3fe92b6e3bff9e69365609e0f.tar.gz | |
ARM: mvebu: Fix coherency bus notifiers by using separate notifiers
Currently, the coherency fabric support registers two bus notifiers;
one for platform, one for pci bus types, with the same notifier block.
However, this is illegal and can cause serious issues: the notifier
block is also a link in the notifier list and cannot be inserted twice.
This commit fixes this by using different notifier blocks (with the same
notifier callback) to set the platform and pci bus types notifiers.
Fixes: b0063aad5dd8 ("ARM: mvebu: use hardware I/O coherency also for PCI devices")
Reported-by: Paolo Pisati <p.pisati@gmail.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Link: https://lkml.kernel.org/r/1404826657-6977-1-git-send-email-ezequiel.garcia@free-electrons.com
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
| -rw-r--r-- | arch/arm/mach-mvebu/coherency.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/arch/arm/mach-mvebu/coherency.c b/arch/arm/mach-mvebu/coherency.c index 477202f..2bdc323 100644 --- a/arch/arm/mach-mvebu/coherency.c +++ b/arch/arm/mach-mvebu/coherency.c @@ -292,6 +292,10 @@ static struct notifier_block mvebu_hwcc_nb = { .notifier_call = mvebu_hwcc_notifier, }; +static struct notifier_block mvebu_hwcc_pci_nb = { + .notifier_call = mvebu_hwcc_notifier, +}; + static void __init armada_370_coherency_init(struct device_node *np) { struct resource res; @@ -427,7 +431,7 @@ static int __init coherency_pci_init(void) { if (coherency_available()) bus_register_notifier(&pci_bus_type, - &mvebu_hwcc_nb); + &mvebu_hwcc_pci_nb); return 0; } |
