shared memory problem on ARM v5TE using threads

[pavel@ucw.cz (Pavel Machek)]

On Sun 2009-12-20 17:32:08, Nicolas Pitre wrote:
> On Sun, 20 Dec 2009, Pavel Machek wrote:
> 
> > On Fri 2009-12-18 14:00:13, Nicolas Pitre wrote:
> > > On Fri, 18 Dec 2009, Pavel Machek wrote:
> > > 
> > > > On Sun 2009-12-13 12:00:33, Russell King - ARM Linux wrote:
> > > > > On Sun, Dec 13, 2009 at 01:48:48PM +0200, Ronen Shitrit wrote:
> > > > > > Another idea is to change the shared mapping handling, in case of vivt with pipt L2, so it won't remap the shared area as non-cacheable:
> > > > > > - make_coherent won't use adjust_pte and leave only the regular flush.
> > > > > > - Flush L1 for all context switches, also for the case that the new process is using same mm (thread context switch).
> > > > > 
> > > > > That doesn't work.
> > > > > 
> > > > > Well, with a VIVT L1, we flush the L1 on all MM switches anyway.
> > > > > Flushing it on any thread switch is not going to help that much.
> > > > > 
> > > > > The problem with shared mmaps is that if you have multiple within the
> > > > > same thread, it is required that they are _all_ coherent with respect
> > > > > to each other, whether or not a context switch has occurred.
> > > > 
> > > > But that's pretty unusual situation, right?
> > > > 
> > > > So what about...
> > > > 
> > > > a) flush L2 on context switch
> > > 
> > > No.  This is too big a performance killer.
> > 
> > Eek?
> > 
> > Alternative is "disable L2". I guess "use L2, but flush it
> > occasionally" is still better than "disable L2".
> > 
> > Of course, it should be measured. It looks like hardware already is
> > braindead, so maybe logic does not work there.
> > 
> > (And it may depend if you want max throughput or min latency).
> 
> Do you own such hardware?

No.

> If no, why do you care?

Curiosity? You spreading statements that seem strange? Me wanting to
understand VIVT caches?

> For your information, a patch has been proposed already which does not 
> imply disabling L2 nor flushing it on every context switches.  Would you 
> care to familiarize yourself with it before commenting further please?

Really? The patch seemed to solve cached vs. uncached accesses, but
did not seem to solve two (cached) mappings of same file.
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html