Enhancing semantics with memremap()

* Enhancing semantics with memremap() - aliasing with memremap()
@ 2016-05-24 23:36 Luis R. Rodriguez
  2016-05-31 16:58 ` Christoph Hellwig
  0 siblings, 1 reply; 5+ messages in thread
From: Luis R. Rodriguez @ 2016-05-24 23:36 UTC (permalink / raw)
  To: Dan Williams, Toshi Kani
  Cc: Paul McKenney, Will Deacon, Christoph Hellwig, X86 ML,
	linux-kernel, linux-arch, Julia Lawall, Benjamin Herrenschmidt,
	Borislav Petkov

Dan, Toshi, Christoph,

I recall a while ago while memremap() was being introduced we
discussed making stronger semantics for memremap() a desirable future
goal [0], along with removal cleanups of ioremap_cache(). This was
when only 2 types were being considered, WB and WT. I see we now have
WC as well for the memremap() family.

Among one of the semantic topics, on the x86 side of things we seem to
have at least concluded we want to at the very least start by
completely discouraging and warning alias uses for different
conflicting types. A recent patch by Toshi helps to find these at run
time, however it may be possible to do this proactively semantically
with Coccinelle as well, but we first need an example test driver with
all known invalid uses cases to write up some grammar rules for the
hunt.

A much much longer term goal would be properly identify only those
valid uses for aliasing, and have a special API for them. This will
take time for the above reaons. During the memremap() review we seemed
to have agreed that making stronger semantics for memremap() was also
desirable, more aimed towards addressing this as a future goal.

Is it a good time for that now? I would hope identifying proper
aliasing uses for memremap() might be a bit easier now than for
ioremap() given its not used as widely. It may be an easier target to
also write some grammar rules for it as well.

[0] http://lkml.kernel.org/r/CAPcyv4g0-CcFRCWPBdKB7h0zZEO5qzT-N9YLYJw9sb=_HYnXiA@mail.gmail.com>

  Luis

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