* [PATCH v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-03-25 19:43 ` Elijah Ragozin
0 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-03-25 19:43 UTC (permalink / raw)
To: linux-arm-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-03-25 19:43 ` Elijah Ragozin
0 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-03-25 19:43 UTC (permalink / raw)
To: Russell King, Jason Gunthorpe, Jason Cooper, Andrew Lunn
Cc: linux-arm-kernel, linux-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* Fwd: [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
2013-03-25 19:43 ` Elijah Ragozin
@ 2013-04-08 19:31 ` Elijah Ragozin
-1 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-08 19:31 UTC (permalink / raw)
To: linux-arm-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Fwd: [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-04-08 19:31 ` Elijah Ragozin
0 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-08 19:31 UTC (permalink / raw)
To: Russell King, Jason Gunthorpe, Jason Cooper, Andrew Lunn
Cc: linux-arm-kernel, linux-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
2013-04-08 19:31 ` Elijah Ragozin
@ 2013-04-08 19:34 ` Elijah Ragozin
-1 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-08 19:34 UTC (permalink / raw)
To: linux-arm-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-04-08 19:34 ` Elijah Ragozin
0 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-08 19:34 UTC (permalink / raw)
To: Russell King, Jason Cooper, Jason Gunthorpe, Andrew Lunn
Cc: linux-arm-kernel, linux-kernel
On Feroceon the L2 cache becomes non-coherent with the CPU
when the L1 caches are disabled. Thus the L2 needs to be invalidated
after both L1 caches are disabled.
On kexec before the starting the code for relocation the kernel,
the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
but after L2 cache is never invalidated, because inv_all is not set
in cache-feroceon-l2.c.
So kernel relocation and decompression may has (and usually has) errors.
Setting the function enables L2 invalidation and fixes the issue.
Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
---
arch/arm/mm/cache-feroceon-l2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mm/cache-feroceon-l2.c
b/arch/arm/mm/cache-feroceon-l2.c
index e0b0e7a..09f8851 100644
--- a/arch/arm/mm/cache-feroceon-l2.c
+++ b/arch/arm/mm/cache-feroceon-l2.c
@@ -342,6 +342,7 @@ void __init feroceon_l2_init(int __l2_wt_override)
outer_cache.inv_range = feroceon_l2_inv_range;
outer_cache.clean_range = feroceon_l2_clean_range;
outer_cache.flush_range = feroceon_l2_flush_range;
+ outer_cache.inv_all = l2_inv_all;
enable_l2();
--
1.8.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
2013-04-08 19:34 ` Elijah Ragozin
@ 2013-04-10 16:48 ` Jason Cooper
-1 siblings, 0 replies; 10+ messages in thread
From: Jason Cooper @ 2013-04-10 16:48 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Apr 08, 2013 at 10:34:07PM +0300, Elijah Ragozin wrote:
> On Feroceon the L2 cache becomes non-coherent with the CPU
> when the L1 caches are disabled. Thus the L2 needs to be invalidated
> after both L1 caches are disabled.
>
> On kexec before the starting the code for relocation the kernel,
> the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
> but after L2 cache is never invalidated, because inv_all is not set
> in cache-feroceon-l2.c.
> So kernel relocation and decompression may has (and usually has) errors.
> Setting the function enables L2 invalidation and fixes the issue.
>
> Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
> ---
> arch/arm/mm/cache-feroceon-l2.c | 1 +
> 1 file changed, 1 insertion(+)
fwiw,
Acked-by: Jason Cooper <jason@lakedaemon.net>
Please put this in Russell King's patch tracker so he can pull it in.
thx,
Jason.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-04-10 16:48 ` Jason Cooper
0 siblings, 0 replies; 10+ messages in thread
From: Jason Cooper @ 2013-04-10 16:48 UTC (permalink / raw)
To: Elijah Ragozin
Cc: Russell King, Jason Gunthorpe, Andrew Lunn, linux-kernel,
linux-arm-kernel
On Mon, Apr 08, 2013 at 10:34:07PM +0300, Elijah Ragozin wrote:
> On Feroceon the L2 cache becomes non-coherent with the CPU
> when the L1 caches are disabled. Thus the L2 needs to be invalidated
> after both L1 caches are disabled.
>
> On kexec before the starting the code for relocation the kernel,
> the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
> but after L2 cache is never invalidated, because inv_all is not set
> in cache-feroceon-l2.c.
> So kernel relocation and decompression may has (and usually has) errors.
> Setting the function enables L2 invalidation and fixes the issue.
>
> Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
> ---
> arch/arm/mm/cache-feroceon-l2.c | 1 +
> 1 file changed, 1 insertion(+)
fwiw,
Acked-by: Jason Cooper <jason@lakedaemon.net>
Please put this in Russell King's patch tracker so he can pull it in.
thx,
Jason.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
2013-04-10 16:48 ` Jason Cooper
@ 2013-04-10 18:42 ` Elijah Ragozin
-1 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-10 18:42 UTC (permalink / raw)
To: linux-arm-kernel
On 10.04.2013 19:48, Jason Cooper wrote:
> On Mon, Apr 08, 2013 at 10:34:07PM +0300, Elijah Ragozin wrote:
>> On Feroceon the L2 cache becomes non-coherent with the CPU
>> when the L1 caches are disabled. Thus the L2 needs to be invalidated
>> after both L1 caches are disabled.
>>
>> On kexec before the starting the code for relocation the kernel,
>> the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
>> but after L2 cache is never invalidated, because inv_all is not set
>> in cache-feroceon-l2.c.
>> So kernel relocation and decompression may has (and usually has) errors.
>> Setting the function enables L2 invalidation and fixes the issue.
>>
>> Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
>> ---
>> arch/arm/mm/cache-feroceon-l2.c | 1 +
>> 1 file changed, 1 insertion(+)
> fwiw,
>
> Acked-by: Jason Cooper <jason@lakedaemon.net>
>
> Please put this in Russell King's patch tracker so he can pull it in.
>
> thx,
>
> Jason.
>
Jason,
thank you for reviewing and acking,
I just put the patch to the Russell King's patch tracker.
--
Best Regards,
Illia Ragozin.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RESEND v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all
@ 2013-04-10 18:42 ` Elijah Ragozin
0 siblings, 0 replies; 10+ messages in thread
From: Elijah Ragozin @ 2013-04-10 18:42 UTC (permalink / raw)
To: Jason Cooper
Cc: Russell King, Jason Gunthorpe, Andrew Lunn, linux-kernel,
linux-arm-kernel
On 10.04.2013 19:48, Jason Cooper wrote:
> On Mon, Apr 08, 2013 at 10:34:07PM +0300, Elijah Ragozin wrote:
>> On Feroceon the L2 cache becomes non-coherent with the CPU
>> when the L1 caches are disabled. Thus the L2 needs to be invalidated
>> after both L1 caches are disabled.
>>
>> On kexec before the starting the code for relocation the kernel,
>> the L1 caches are disabled in cpu_froc_fin (cpu_v7_proc_fin for Feroceon),
>> but after L2 cache is never invalidated, because inv_all is not set
>> in cache-feroceon-l2.c.
>> So kernel relocation and decompression may has (and usually has) errors.
>> Setting the function enables L2 invalidation and fixes the issue.
>>
>> Signed-off-by: Illia Ragozin <illia.ragozin@grapecom.com>
>> ---
>> arch/arm/mm/cache-feroceon-l2.c | 1 +
>> 1 file changed, 1 insertion(+)
> fwiw,
>
> Acked-by: Jason Cooper <jason@lakedaemon.net>
>
> Please put this in Russell King's patch tracker so he can pull it in.
>
> thx,
>
> Jason.
>
Jason,
thank you for reviewing and acking,
I just put the patch to the Russell King's patch tracker.
--
Best Regards,
Illia Ragozin.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2013-04-10 18:43 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-25 19:43 [PATCH v2 1/1] ARM Feroceon: fix kexec by setting outer_cache.inv_all Elijah Ragozin
2013-03-25 19:43 ` Elijah Ragozin
2013-04-08 19:31 ` Fwd: [PATCH RESEND " Elijah Ragozin
2013-04-08 19:31 ` Elijah Ragozin
2013-04-08 19:34 ` Elijah Ragozin
2013-04-08 19:34 ` Elijah Ragozin
2013-04-10 16:48 ` Jason Cooper
2013-04-10 16:48 ` Jason Cooper
2013-04-10 18:42 ` Elijah Ragozin
2013-04-10 18:42 ` Elijah Ragozin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.