[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] Don't clear HCR_VM bit when updating VTTBR.



On Tue, 1 Nov 2016, Julien Grall wrote:
> Hi Stefano,
> 
> On 31/10/2016 22:18, Stefano Stabellini wrote:
> > On Mon, 31 Oct 2016, Steve Capper wrote:
> > > On Wed, Oct 19, 2016 at 12:59:45PM -0700, Stefano Stabellini wrote:
> > > > On Mon, 10 Oct 2016, Jun Sun wrote:
> > > > > Currently function p2m_restore_state() would clear HCR_VM bit, i.e.,
> > > > > disabling stage2 translation, before updating VTTBR register. After
> > > > > some research and talking to ARM support, I got confirmed that this is
> > > > > not
> > > > > necessary. We are currently working on a new platform that would need
> > > > > this
> > > > > to be removed.
> > > > > 
> > > > > The patch is tested on FVP foundation model.
> > > > > 
> > > > > Signed-off-by: Jun Sun <jsun@xxxxxxxxxx>
> > > > 
> > > > Hello Jun,
> > > > 
> > > > thanks for the patch and sorry for the late reply. I would appreciate
> > > > feedback from Julien, but he is current AFK.
> > > > 
> > > > Steve,
> > > > can I have your Acked-by on this patch?
> > > > 
> > > 
> > > Apologies for my very late reply on this.
> > > 
> > > I've taken a look and I think that this is okay, so please feel free to
> > > add:
> > > Acked-by: Steve Capper <steve.capper@xxxxxxxxxx>
> > 
> > Thanks Steve. Given the state of the release, I am going to push the fix
> > to Xen 4.9 unless you or Jun have a strong argument for committing this
> > sooner.
> 
> Not related to this patch. Do you plan to have a branch for queuing all Xen
> 4.9 patches?

Hi Julien, thanks a good suggestion (I am already starting to lose
patches). I created:

git://xenbits.xen.org/people/sstabellini/xen-unstable.git xen-arm-next

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.