From: gorcunov@gmail.com (Cyrill Gorcunov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm: align shared memory unconditionally to the SHMLBA boundary
Date: Mon, 15 Jul 2013 22:57:39 +0400 [thread overview]
Message-ID: <20130715185739.GK1730@moon> (raw)
In-Reply-To: <20130715180846.GV24642@n2100.arm.linux.org.uk>
On Mon, Jul 15, 2013 at 07:08:46PM +0100, Russell King - ARM Linux wrote:
> On Mon, Jul 15, 2013 at 09:32:38PM +0400, Cyrill Gorcunov wrote:
> > On Tue, Feb 19, 2013 at 10:11:09AM +0400, Alexander Kartashov wrote:
> > > Currently IPC SHM works in a strange way on ARM:
> > > the syscall sys_shmat() requires the argument shmaddr
> > > be SHMLBA-aligned (ARM has the macro __ARCH_FORCE_SHMLBA
> > > unconditionally defined) but allocates memory that
> > > isn't SHMLBA-aligned because the value of memory alignment
> > > depends on presense of certain cache aliases.
> >
> > Hi guys, is there some conclusion on this patch? It has been sent
> > almost 5 months back ;)
>
> It's pointless. The alignment is only required for CPUs which have
> aliasing caches. What the code does in mmap() is correct. However,
> we can't dynamically select this in the SHM code.
Hi Russell, thanks a lot for reply! I must admit I'm not that familiar
with arm hw internals, but if the behaviour is correct than we've a
problem with checkpoint/restore procedure on arm platform :( As far
as I know Alexander pointed me that he find no other way of fixing
the problem except patching the kernel. Alexander, mind to remind
where exactly we're failing in c/r procedure, thus maybe arm guys
would help us to find a way to pass it by with some user-space
trick?
next prev parent reply other threads:[~2013-07-15 18:57 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-19 6:11 [PATCH] arm: align shared memory unconditionally to the SHMLBA boundary Alexander Kartashov
2013-07-15 17:32 ` Cyrill Gorcunov
2013-07-15 18:08 ` Russell King - ARM Linux
2013-07-15 18:57 ` Cyrill Gorcunov [this message]
2013-07-16 5:37 ` Alexander Kartashov
2013-07-16 9:53 ` Cyrill Gorcunov
2013-07-16 10:22 ` Alexander Kartashov
2013-07-16 10:36 ` Russell King - ARM Linux
2013-07-16 10:47 ` Alexander Kartashov
2013-07-16 11:07 ` Russell King - ARM Linux
2013-07-16 13:12 ` Alexander Kartashov
2013-08-16 8:20 ` Alexander Kartashov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130715185739.GK1730@moon \
--to=gorcunov@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.