On 20/01/14 21:22, Konrad Rzeszutek Wilk wrote:
Zoltan Kiss <zoltan.k...@citrix.com> wrote:
The grant mapping API does m2p_override unnecessarily: only gntdev
needs it,
for blkback and future netback patches it just cause a lock contention,
as
those pages never go to userspace. Therefore this series does the
following:
- the original functions were renamed to __gnttab_[un]map_refs, with a
new
parameter m2p_override
- based on m2p_override either they follow the original behaviour, or
just set
the private flag and call set_phys_to_machine
- gnttab_[un]map_refs are now a wrapper to call __gnttab_[un]map_refs
with
m2p_override false
- a new function gnttab_[un]map_refs_userspace provides the old
behaviour
You don't say anything about the 'return ret' changed to 'return 0'.
Any particular reason for that?
That's the only possible return value there, so it just makes it more
obvious. I'll add a description about that.
Zoli
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/