[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH for-4.7 1/1] xen/arm: Rename map_regions_rw_cache and use p2m.default_access
On Wed, Jun 08, 2016 at 10:36:19AM +0100, Stefano Stabellini wrote: > On Wed, 8 Jun 2016, Julien Grall wrote: > > Hi Wei, > > > > On 08/06/2016 09:17, Wei Liu wrote: > > > On Wed, Jun 08, 2016 at 02:15:27AM +0200, Edgar E. Iglesias wrote: > > > > From: "Edgar E. Iglesias" <edgar.iglesias@xxxxxxxxxx> > > > > > > > > Rename map_regions_rw_cache to map_regions_cache and make it use > > > > p2m.default_access. > > > > > > > > Suggested-by: Julien Grall <julien.grall@xxxxxxx> > > > > Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xxxxxxxxxx> > > > > > > I don't think this is absolutely necessary for 4.7. > > > > > > On the other hand, it is just straight renaming, which should be quite > > > safe. > > > > This patch does not only contain a renaming, it also contain a change to fix > > the default memaccess attribute. > > > > However, I don't see why we should rename the function to map_regions_cache > > given this will always map the region Read-Write (p2m_mmio_direct prevents > > the > > execution of the memory). > > > > > If I can get an ack or review from maintainers and confirmation that it > > > doesn't break ARM build within today, we can shovel this in; otherwise > > > it needs to wait for next version of Xen. > > > > I would wait for a backport here. Stefano, any opinions? > > Indeed, I would also wait for a backport OK, Sounds good. I'm travelling at the moment so it might take a couple of weeks for me to get back to this. Cheers, Edgar _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |