Re: [PATCH 4.4 08/40] ARM: dts: mvebu: pl310-cache disable double-linefill

2017-11-08 Thread Greg Kroah-Hartman
On Wed, Nov 08, 2017 at 12:06:48AM +0100, Sebastian Gottschall wrote:
> what about 4.9?

Ok, that was odd, I don't know what happened there, now added to the 4.9
tree, thanks for catching that.

greg k-h


Re: [PATCH 4.4 08/40] ARM: dts: mvebu: pl310-cache disable double-linefill

2017-11-08 Thread Greg Kroah-Hartman
On Wed, Nov 08, 2017 at 12:06:48AM +0100, Sebastian Gottschall wrote:
> what about 4.9?

Ok, that was odd, I don't know what happened there, now added to the 4.9
tree, thanks for catching that.

greg k-h


Re: [PATCH 4.4 08/40] ARM: dts: mvebu: pl310-cache disable double-linefill

2017-11-07 Thread Sebastian Gottschall

what about 4.9?

Am 06.11.2017 um 10:44 schrieb Greg Kroah-Hartman:

4.4-stable review patch.  If anyone has any objections, please let me know.

--

From: Yan Markman 

commit cda80a82ac3e89309706c027ada6ab232be1d640 upstream.

Under heavy system stress mvebu SoC using Cortex A9 sporadically
encountered instability issues.

The "double linefill" feature of L2 cache was identified as causing
dependency between read and write which lead to the deadlock.

Especially, it was the cause of deadlock seen under heavy PCIe traffic,
as this dependency violates PCIE overtaking rule.

Fixes: c8f5a878e554 ("ARM: mvebu: use DT properties to fine-tune the L2 
configuration")
Signed-off-by: Yan Markman 
Signed-off-by: Igal Liberman 
Signed-off-by: Nadav Haklai 
[gregory.clem...@free-electrons.com: reformulate commit log, add Armada
375 and add Fixes tag]
Signed-off-by: Gregory CLEMENT 
Signed-off-by: Greg Kroah-Hartman 

---
  arch/arm/boot/dts/armada-375.dtsi |4 ++--
  arch/arm/boot/dts/armada-38x.dtsi |4 ++--
  arch/arm/boot/dts/armada-39x.dtsi |4 ++--
  3 files changed, 6 insertions(+), 6 deletions(-)

--- a/arch/arm/boot/dts/armada-375.dtsi
+++ b/arch/arm/boot/dts/armada-375.dtsi
@@ -176,9 +176,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  
--- a/arch/arm/boot/dts/armada-38x.dtsi

+++ b/arch/arm/boot/dts/armada-38x.dtsi
@@ -143,9 +143,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  
--- a/arch/arm/boot/dts/armada-39x.dtsi

+++ b/arch/arm/boot/dts/armada-39x.dtsi
@@ -104,9 +104,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  






--
Mit freundlichen Grüssen / Regards

Sebastian Gottschall / CTO

NewMedia-NET GmbH - DD-WRT
Firmensitz:  Stubenwaldallee 21a, 64625 Bensheim
Registergericht: Amtsgericht Darmstadt, HRB 25473
Geschäftsführer: Peter Steinhäuser, Christian Scheele
http://www.dd-wrt.com
email: s.gottsch...@dd-wrt.com
Tel.: +496251-582650 / Fax: +496251-5826565



Re: [PATCH 4.4 08/40] ARM: dts: mvebu: pl310-cache disable double-linefill

2017-11-07 Thread Sebastian Gottschall

what about 4.9?

Am 06.11.2017 um 10:44 schrieb Greg Kroah-Hartman:

4.4-stable review patch.  If anyone has any objections, please let me know.

--

From: Yan Markman 

commit cda80a82ac3e89309706c027ada6ab232be1d640 upstream.

Under heavy system stress mvebu SoC using Cortex A9 sporadically
encountered instability issues.

The "double linefill" feature of L2 cache was identified as causing
dependency between read and write which lead to the deadlock.

Especially, it was the cause of deadlock seen under heavy PCIe traffic,
as this dependency violates PCIE overtaking rule.

Fixes: c8f5a878e554 ("ARM: mvebu: use DT properties to fine-tune the L2 
configuration")
Signed-off-by: Yan Markman 
Signed-off-by: Igal Liberman 
Signed-off-by: Nadav Haklai 
[gregory.clem...@free-electrons.com: reformulate commit log, add Armada
375 and add Fixes tag]
Signed-off-by: Gregory CLEMENT 
Signed-off-by: Greg Kroah-Hartman 

---
  arch/arm/boot/dts/armada-375.dtsi |4 ++--
  arch/arm/boot/dts/armada-38x.dtsi |4 ++--
  arch/arm/boot/dts/armada-39x.dtsi |4 ++--
  3 files changed, 6 insertions(+), 6 deletions(-)

--- a/arch/arm/boot/dts/armada-375.dtsi
+++ b/arch/arm/boot/dts/armada-375.dtsi
@@ -176,9 +176,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  
--- a/arch/arm/boot/dts/armada-38x.dtsi

+++ b/arch/arm/boot/dts/armada-38x.dtsi
@@ -143,9 +143,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  
--- a/arch/arm/boot/dts/armada-39x.dtsi

+++ b/arch/arm/boot/dts/armada-39x.dtsi
@@ -104,9 +104,9 @@
reg = <0x8000 0x1000>;
cache-unified;
cache-level = <2>;
-   arm,double-linefill-incr = <1>;
+   arm,double-linefill-incr = <0>;
arm,double-linefill-wrap = <0>;
-   arm,double-linefill = <1>;
+   arm,double-linefill = <0>;
prefetch-data = <1>;
};
  






--
Mit freundlichen Grüssen / Regards

Sebastian Gottschall / CTO

NewMedia-NET GmbH - DD-WRT
Firmensitz:  Stubenwaldallee 21a, 64625 Bensheim
Registergericht: Amtsgericht Darmstadt, HRB 25473
Geschäftsführer: Peter Steinhäuser, Christian Scheele
http://www.dd-wrt.com
email: s.gottsch...@dd-wrt.com
Tel.: +496251-582650 / Fax: +496251-5826565