linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: OMAP2+: TEMP: Round of the carve out memory requested to section_size
@ 2012-09-12  5:51 R Sricharan
  2012-09-12  5:58 ` Shilimkar, Santosh
  0 siblings, 1 reply; 3+ messages in thread
From: R Sricharan @ 2012-09-12  5:51 UTC (permalink / raw)
  To: linux-arm-kernel

memblock_steal tries to reserve physical memory during boot.
When the requested size is not aligned on the section size
then, the remaining memory available for lowmem becomes
unaligned on the section boundary. There is a issue with this,
which is discussed in the thread below.

https://lkml.org/lkml/2012/6/28/112

The final conclusion from the thread seems to
be align the memblock_steal calls on the SECTION boundary.

Signed-off-by: R Sricharan <r.sricharan@ti.com>
---
 arch/arm/mach-omap2/omap-secure.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-omap2/omap-secure.c b/arch/arm/mach-omap2/omap-secure.c
index d9ae4a5..26edfec 100644
--- a/arch/arm/mach-omap2/omap-secure.c
+++ b/arch/arm/mach-omap2/omap-secure.c
@@ -61,8 +61,8 @@ int __init omap_secure_ram_reserve_memblock(void)
 {
 	u32 size = OMAP_SECURE_RAM_STORAGE;
 
-	size = ALIGN(size, SZ_1M);
-	omap_secure_memblock_base = arm_memblock_steal(size, SZ_1M);
+	size = ALIGN(size, SECTION_SIZE);
+	omap_secure_memblock_base = arm_memblock_steal(size, SECTION_SIZE);
 
 	return 0;
 }
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH] ARM: OMAP2+: TEMP: Round of the carve out memory requested to section_size
  2012-09-12  5:51 [PATCH] ARM: OMAP2+: TEMP: Round of the carve out memory requested to section_size R Sricharan
@ 2012-09-12  5:58 ` Shilimkar, Santosh
  2012-09-12  8:42   ` Sricharan R
  0 siblings, 1 reply; 3+ messages in thread
From: Shilimkar, Santosh @ 2012-09-12  5:58 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Sep 12, 2012 at 11:21 AM, R Sricharan <r.sricharan@ti.com> wrote:
> memblock_steal tries to reserve physical memory during boot.
> When the requested size is not aligned on the section size
> then, the remaining memory available for lowmem becomes
> unaligned on the section boundary. There is a issue with this,
> which is discussed in the thread below.
>
> https://lkml.org/lkml/2012/6/28/112
>
> The final conclusion from the thread seems to
> be align the memblock_steal calls on the SECTION boundary.
>
> Signed-off-by: R Sricharan <r.sricharan@ti.com>
Why is the TEMP in the subject line. From patch it doesn't be
temporary version. Can you please clarify that ?

Regards
Santosh

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] ARM: OMAP2+: TEMP: Round of the carve out memory requested to section_size
  2012-09-12  5:58 ` Shilimkar, Santosh
@ 2012-09-12  8:42   ` Sricharan R
  0 siblings, 0 replies; 3+ messages in thread
From: Sricharan R @ 2012-09-12  8:42 UTC (permalink / raw)
  To: linux-arm-kernel

[..]
> > memblock_steal tries to reserve physical memory during boot.
> > When the requested size is not aligned on the section size
> > then, the remaining memory available for lowmem becomes
> > unaligned on the section boundary. There is a issue with this,
> > which is discussed in the thread below.
> >
> > https://lkml.org/lkml/2012/6/28/112
> >
> > The final conclusion from the thread seems to
> > be align the memblock_steal calls on the SECTION boundary.
> >
> > Signed-off-by: R Sricharan <r.sricharan@ti.com>
> Why is the TEMP in the subject line. From patch it doesn't be
> temporary version. Can you please clarify that ?
>
 Oops, that was a mistake. Reposted by correcting it.

Thanks,
 Sricharan

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-09-12  8:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-12  5:51 [PATCH] ARM: OMAP2+: TEMP: Round of the carve out memory requested to section_size R Sricharan
2012-09-12  5:58 ` Shilimkar, Santosh
2012-09-12  8:42   ` Sricharan R

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).