Hi Andrew,

On 16/05/17 18:12, Andrew Cooper wrote:
On 16/05/17 18:06, Tim Deegan wrote:
At 17:51 +0100 on 16 May (1494957116), Andrew Cooper wrote:
c/s 4c5d78a10 was accidentally buggy when handling Protection Keys.
Protection keys applies to all user translations, not just accesses which
originate from user mode.
Reviewed-by: Tim Deegan <t...@xen.org>

Does the test for write-protection just below have the opposite bug?
It seems to check whether the page is writable, when AFAICS it should
be checking whether the action is a write (modulo CR0.WP).

Very good point.  Let me draft a v2.

FIY, I will wait the v2 before considering it for the release.

Cheers,

--
Julien Grall

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

Reply via email to