From: Julien Grall <julien.grall@arm.com>
To: Stefano Stabellini <sstabellini@kernel.org>,
Steve Capper <steve.capper@linaro.org>
Cc: xen-devel@lists.xenproject.org, Jun Sun <jsun@junsun.net>
Subject: Re: [PATCH] Don't clear HCR_VM bit when updating VTTBR.
Date: Tue, 1 Nov 2016 14:38:40 +0000 [thread overview]
Message-ID: <ab403476-6557-8b36-f21c-65ed4d1bde1f@arm.com> (raw)
In-Reply-To: <alpine.DEB.2.10.1610311516350.19454@sstabellini-ThinkPad-X260>
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@junsun.net>
>>>
>>> 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@linaro.org>
>
> 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?
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-11-01 14:38 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-10 19:27 [PATCH] Don't clear HCR_VM bit when updating VTTBR Jun Sun
2016-10-12 0:21 ` Jun Sun
2016-10-19 19:59 ` Stefano Stabellini
2016-10-31 11:25 ` Steve Capper
2016-10-31 22:18 ` Stefano Stabellini
2016-11-01 14:38 ` Julien Grall [this message]
2016-11-01 18:10 ` Stefano Stabellini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ab403476-6557-8b36-f21c-65ed4d1bde1f@arm.com \
--to=julien.grall@arm.com \
--cc=jsun@junsun.net \
--cc=sstabellini@kernel.org \
--cc=steve.capper@linaro.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).