ARM64: PCI: do not enable resources on PROBE_ONLY systems

PATCH New Lorenzo Pieralisi 3 participants 8 messages Jul 30, 2015 – Sep 3, 2015

Subsystems (2)

PCI arm64

lore

Share & tools

lore

Lorenzo Pieralisi

author

2015-07-30 13:13 UTC #0 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.

To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).

As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).

This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.

For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.

Signed-off-by Lorenzo Pieralisi <[email protected]>

Cc Will Deacon <[email protected]>

Cc Bjorn Helgaas <[email protected]>

Cc Catalin Marinas <[email protected]>

---
Bjorn, all,

as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.

It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.

Lorenzo

 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

arch/arm64/kernel/pci.c +13 -0

index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
 	return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+	if (pci_has_flag(PCI_PROBE_ONLY))
+		return 0;
+
+	return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */

Signature · 1 line

2.2.1

Bjorn Helgaas

maintainer

2015-08-28 21:14 UTC #1 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Hi Lorenzo,

Quoted 5 lines · Lorenzo Pieralisi

On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.

Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case? I know it must be there somewhere, but I don't see it.

Quoted 70 lines

To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).
As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).
This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.
For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.
Signed-off-by Lorenzo Pieralisi <[email protected]>
Cc Will Deacon <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc Catalin Marinas <[email protected]>
---
Bjorn, all,
as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.
It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.
Lorenzo
 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)
arch/arm64/kernel/pci.c +13 -0
index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
 	return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+	if (pci_has_flag(PCI_PROBE_ONLY))
+		return 0;
+
+	return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */
Signature · 1 line
2.2.1

Lorenzo Pieralisi

author

2015-08-29 12:44 UTC #2 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Hi Bjorn,

Quoted 10 lines · Bjorn Helgaas

Hi Lorenzo,
Quoted 5 lines · Lorenzo Pieralisi
On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.
Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case?

arm64 (and arm) does not claim resources in arch code, they are always assigned in the respective host controllers (or the pcibios on arm), except for the PCI generic host when in PROBE_ONLY mode, where resources are neither claimed nor assigned (the reason for this patch to exist).

Certainly something to consolidate as part of the rework we talked about at LPC (PCI ACPI probing is yet another reason to consolidate it).

I am working on this, in particular I want to test how claiming the resources by default will impact ARM systems.

For the records, Will kindly provided me with an Alpha box so that I can test on it too.

Quoted 1 line

I know it must be there somewhere, but I don't see it.

It is not there, it is done in the host controllers by assigning the resources (which also "claim" them by assigning the parent pointer).

Thanks,
Lorenzo

Quoted 72 lines

Quoted 71 lines
To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).
As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).
This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.
For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.
Signed-off-by Lorenzo Pieralisi <[email protected]>
Cc Will Deacon <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc Catalin Marinas <[email protected]>
---
Bjorn, all,
as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.
It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.
Lorenzo
 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)
arch/arm64/kernel/pci.c +13 -0
index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
 	return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+	if (pci_has_flag(PCI_PROBE_ONLY))
+		return 0;
+
+	return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */
Signature · 1 line
2.2.1

Bjorn Helgaas

maintainer

2015-08-31 13:28 UTC #3 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Quoted 16 lines · Lorenzo Pieralisi

Quoted 8 lines · Bjorn Helgaas
Quoted 5 lines · Lorenzo Pieralisi
On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.
Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case?
arm64 (and arm) does not claim resources in arch code, they are always assigned in the respective host controllers (or the pcibios on arm), except for the PCI generic host when in PROBE_ONLY mode, where resources are neither claimed nor assigned (the reason for this patch to exist).
...

Quoted 4 lines

Quoted 1 line
I know it must be there somewhere, but I don't see it.
It is not there, it is done in the host controllers by assigning the resources (which also "claim" them by assigning the parent pointer).

The host controller drivers should claim the apertures forwarded to the PCI bus. For pci-host-generic, this happens when gen_pci_parse_request_of_pci_ranges() calls devm_request_resource(). But that doesn't do anything for the BARs of the PCI devices themselves, of course.

If this patch fixes a problem that will occur in the future, when we add arm/arm64 code to claim BARs, let's make this patch part of that series. If it fixes a problem in the current tree, I'll merge it now, but so far I haven't been able to make the connection between this patch and a problem in the current code.

Quoted 71 lines

Quoted 71 lines
Quoted 70 lines
To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).
As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).
This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.
For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.
Signed-off-by Lorenzo Pieralisi <[email protected]>
Cc Will Deacon <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc Catalin Marinas <[email protected]>
---
Bjorn, all,
as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.
It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.
Lorenzo
 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)
arch/arm64/kernel/pci.c +13 -0
index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
 	return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+	if (pci_has_flag(PCI_PROBE_ONLY))
+		return 0;
+
+	return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */
Signature · 1 line
2.2.1

Lorenzo Pieralisi

author

2015-08-31 16:33 UTC #4 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Quoted 17 lines · Bjorn Helgaas

Quoted 16 lines · Lorenzo Pieralisi
Quoted 8 lines · Bjorn Helgaas
Quoted 5 lines · Lorenzo Pieralisi
On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.
Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case?
arm64 (and arm) does not claim resources in arch code, they are always assigned in the respective host controllers (or the pcibios on arm), except for the PCI generic host when in PROBE_ONLY mode, where resources are neither claimed nor assigned (the reason for this patch to exist).
...

Quoted 8 lines

Quoted 4 lines
Quoted 1 line
I know it must be there somewhere, but I don't see it.
It is not there, it is done in the host controllers by assigning the resources (which also "claim" them by assigning the parent pointer).
The host controller drivers should claim the apertures forwarded to the PCI bus. For pci-host-generic, this happens when gen_pci_parse_request_of_pci_ranges() calls devm_request_resource().

Yes, the generic host claims the apertures at DT probing time.

Quoted 8 lines

But that doesn't do anything for the BARs of the PCI devices themselves, of course.
If this patch fixes a problem that will occur in the future, when we add arm/arm64 code to claim BARs, let's make this patch part of that series. If it fixes a problem in the current tree, I'll merge it now, but so far I haven't been able to make the connection between this patch and a problem in the current code.

It fixes a problem in the current tree (I mean when the PCI pull is merged - it contains code that enables PCI host generic on arm64), since without it PROBE_ONLY systems (ie PCI host generic on kvmtool) would try to enable resources and fail, since the resources were not claimed.

It works the same way as arm, I will rework both arches when I am done refactoring the resource claiming implementation.

Will picked it up already I do not know what's the best way to merge it I guess it can go via the arm64 tree if you do not have any objections.

Thank you !
Lorenzo

Quoted 72 lines

Quoted 71 lines
Quoted 71 lines
Quoted 70 lines
To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).
As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).
This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.
For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.
Signed-off-by Lorenzo Pieralisi <[email protected]>
Cc Will Deacon <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc Catalin Marinas <[email protected]>
---
Bjorn, all,
as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.
It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.
Lorenzo
 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)
arch/arm64/kernel/pci.c +13 -0
index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
 	return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+	if (pci_has_flag(PCI_PROBE_ONLY))
+		return 0;
+
+	return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */
Signature · 1 line
2.2.1

Bjorn Helgaas

maintainer

2015-08-31 16:49 UTC #5 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Quoted 43 lines · Lorenzo Pieralisi

Quoted 17 lines · Bjorn Helgaas
Quoted 16 lines · Lorenzo Pieralisi
Quoted 8 lines · Bjorn Helgaas
Quoted 5 lines · Lorenzo Pieralisi
On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.
Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case?
arm64 (and arm) does not claim resources in arch code, they are always assigned in the respective host controllers (or the pcibios on arm), except for the PCI generic host when in PROBE_ONLY mode, where resources are neither claimed nor assigned (the reason for this patch to exist).
...
Quoted 8 lines
Quoted 4 lines
Quoted 1 line
I know it must be there somewhere, but I don't see it.
It is not there, it is done in the host controllers by assigning the resources (which also "claim" them by assigning the parent pointer).
The host controller drivers should claim the apertures forwarded to the PCI bus. For pci-host-generic, this happens when gen_pci_parse_request_of_pci_ranges() calls devm_request_resource().
Yes, the generic host claims the apertures at DT probing time.
Quoted 8 lines
But that doesn't do anything for the BARs of the PCI devices themselves, of course.
If this patch fixes a problem that will occur in the future, when we add arm/arm64 code to claim BARs, let's make this patch part of that series. If it fixes a problem in the current tree, I'll merge it now, but so far I haven't been able to make the connection between this patch and a problem in the current code.
It fixes a problem in the current tree (I mean when the PCI pull is merged - it contains code that enables PCI host generic on arm64), since without it PROBE_ONLY systems (ie PCI host generic on kvmtool) would try to enable resources and fail, since the resources were not claimed.

Quoted 6 lines

It works the same way as arm, I will rework both arches when I am done refactoring the resource claiming implementation.
Will picked it up already I do not know what's the best way to merge it I guess it can go via the arm64 tree if you do not have any objections.

OK, if it fixes a problem in my v4.3 pull request, we need to merge it. I missed the connection, probably because I was browsing the v4.2-rc1 tree, which wouldn't have those changes.

It's OK with me to merge it via the arm64 tree.

Quoted 72 lines

Quoted 72 lines
Quoted 71 lines
Quoted 71 lines
Quoted 70 lines
To solve this issue, resources must be claimed when devices are added on PROBE_ONLY systems, which ensures that the resource hierarchy is validated and the resource tree is sane, but this requires changes in the ARM64 resource management that can affect adversely existing PCI set-ups (claiming resources on !PROBE_ONLY systems might break existing ARM64 PCI platform implementations).
As a temporary solution in preparation for a proper resources claiming implementation in ARM64 core, to enable PCI PROBE_ONLY systems on ARM64, this patch adds a pcibios_enable_device() arch implementation that simply prevents enabling resources on PROBE_ONLY systems (mirroring ARM behaviour).
This is always a safe thing to do because on PROBE_ONLY systems the configuration space set-up can be considered immutable, and it is in preparation of proper resource claiming that would finally validate the PCI resources tree in the ARM64 arch implementation on PROBE_ONLY systems.
For !PROBE_ONLY systems resources enablement in pcibios_enable_device() on ARM64 is implemented as in current PCI core, leaving the behaviour unchanged.
Signed-off-by Lorenzo Pieralisi <[email protected]>
Cc Will Deacon <[email protected]>
Cc Bjorn Helgaas <[email protected]>
Cc Catalin Marinas <[email protected]>
---
Bjorn, all,
as I mention in the commit log, this patch is a temporary solution in preparation for proper resources claiming in ARM64, so that we can safely enable the PCI generic host controller on ARM64 systems.
It mirrors ARM implementation and I will work on changing both ARM and ARM64 to convert them to proper resources claiming in the respective implementations.
Lorenzo
 arch/arm64/kernel/pci.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)
arch/arm64/kernel/pci.c +13 -1
index 4095379..b3d098b 100644
@@ -38,6 +38,19 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
        return res->start;
 }
 
+/**
+ * pcibios_enable_device - Enable I/O and memory.
+ * @dev: PCI device to be enabled
+ * @mask: bitmask of BARs to enable
+ */
+int pcibios_enable_device(struct pci_dev *dev, int mask)
+{
+       if (pci_has_flag(PCI_PROBE_ONLY))
+               return 0;
+
+       return pci_enable_resources(dev, mask);
+}
+
 /*
  * Try to assign the IRQ number from DT when adding a new device
  */
--
2.2.1

Bjorn Helgaas

maintainer

2015-09-03 16:35 UTC #6 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Quoted 52 lines · Bjorn Helgaas

Quoted 43 lines · Lorenzo Pieralisi
Quoted 17 lines · Bjorn Helgaas
Quoted 16 lines · Lorenzo Pieralisi
Quoted 8 lines · Bjorn Helgaas
Quoted 5 lines · Lorenzo Pieralisi
On ARM64 PROBE_ONLY PCI systems resources are not currently claimed, therefore they can't be enabled since they do not have a valid parent pointer; this in turn prevents enabling PCI devices on ARM64 PROBE_ONLY systems, causing PCI devices initialization to fail.
Where does arm64 claim PCI BAR resources for the non-PROBE_ONLY case?
arm64 (and arm) does not claim resources in arch code, they are always assigned in the respective host controllers (or the pcibios on arm), except for the PCI generic host when in PROBE_ONLY mode, where resources are neither claimed nor assigned (the reason for this patch to exist).
...
Quoted 8 lines
Quoted 4 lines
Quoted 1 line
I know it must be there somewhere, but I don't see it.
It is not there, it is done in the host controllers by assigning the resources (which also "claim" them by assigning the parent pointer).
The host controller drivers should claim the apertures forwarded to the PCI bus. For pci-host-generic, this happens when gen_pci_parse_request_of_pci_ranges() calls devm_request_resource().
Yes, the generic host claims the apertures at DT probing time.
Quoted 8 lines
But that doesn't do anything for the BARs of the PCI devices themselves, of course.
If this patch fixes a problem that will occur in the future, when we add arm/arm64 code to claim BARs, let's make this patch part of that series. If it fixes a problem in the current tree, I'll merge it now, but so far I haven't been able to make the connection between this patch and a problem in the current code.
It fixes a problem in the current tree (I mean when the PCI pull is merged - it contains code that enables PCI host generic on arm64), since without it PROBE_ONLY systems (ie PCI host generic on kvmtool) would try to enable resources and fail, since the resources were not claimed.
Quoted 6 lines
It works the same way as arm, I will rework both arches when I am done refactoring the resource claiming implementation.
Will picked it up already I do not know what's the best way to merge it I guess it can go via the arm64 tree if you do not have any objections.

Quoted 5 lines

OK, if it fixes a problem in my v4.3 pull request, we need to merge it. I missed the connection, probably because I was browsing the v4.2-rc1 tree, which wouldn't have those changes.
It's OK with me to merge it via the arm64 tree.

Do you need me to do anything with this? It'd be nice to get it in before -rc1. The PCI changes have already been merged, and it'd be nice to mention the commit that this fixes.

Bjorn

Will Deacon

2015-09-03 16:42 UTC #7 lore raw

PATCH 1/1 ARM64: PCI: do not enable resources on PROBE_ONLY systems

Hi Bjorn,

Quoted 10 lines · Bjorn Helgaas

Quoted 5 lines · Bjorn Helgaas
OK, if it fixes a problem in my v4.3 pull request, we need to merge it. I missed the connection, probably because I was browsing the v4.2-rc1 tree, which wouldn't have those changes.
It's OK with me to merge it via the arm64 tree.
Do you need me to do anything with this? It'd be nice to get it in before -rc1. The PCI changes have already been merged, and it'd be nice to mention the commit that this fixes.

Nope, it's queued in the arm64 tree. I'm just waiting for a dependent branch in -tip to get merged before I send my pull request.

Cheers,

Will