On Wed, 6 Jul 2016, Julien Grall wrote: > (CC Tamas) > > On 06/07/16 11:43, Stefano Stabellini wrote: > > On Tue, 28 Jun 2016, Julien Grall wrote: > > > The parameter 'access' is used by memaccess to restrict temporarily the > > > permission. This parameter should not be used for other purpose (such > > > as restricting permanently the permission). > > > > > > The type p2m_mmio_direct will map the region Read-Write and > > > non-executable. Note that this is already the current behavior with the > > > combination of the type and the access. So there is no functional > > > change. > > > > > > Signed-off-by: Julien Grall <julien.gr...@arm.com> > > > > I could be mistaken, but isn't default_access actually p2m_access_rwx? > > By default, the access is p2m_access_rwx. However this can be changed by > memaccess and the new default value is stored in arch->p2m.default_access. I > have CCed Tamas to confirm that. > > Note that this is how the other calls are done.
This patch replaces p2m_access_rw with default_access, which is p2m_access_rwx by default. Is it actually indended? _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
