>>> On 01.06.18 at 16:06, <andrew.coop...@citrix.com> wrote: > c/s "x86/pv: Introduce and use x86emul_write_dr()" fixed a bug with IO shadow > handling, in that it remained stale and visible until %dr7.L/G got set again. > > However, it neglected the -EPERM return inbetween these two hunks, introducing > a different bug in which a write to %dr7 which tries to set IO breakpoints > without %cr4.DE being set clobbers the IO state, rather than leaves it alone. > > Instead, move the zeroing slightly later, which guarentees that the shadow > gets written exactly once, on a successful update to %dr7. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel