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/

Reply via email to