[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH] Fix save/restore for HVM domains with viridian=1
On 25/11/2011 14:58, "Jan Beulich" <JBeulich@xxxxxxxx> wrote: >> xc_domain_save/restore currently pay no attention to HVM_PARAM_VIRIDIAN >> which >> results in an HVM domain running a recent version on Windows (post-Vista) >> locking up on a domain restore due to EOIs (done via a viridian MSR write) >> being silently dropped. >> This patch adds an extra save entry for the viridian parameter and also >> adds code in the viridian kernel module to catch attempted use of viridian >> functionality when the HVM parameter has not been set. > > Assuming this means the changes to {rd,wr}msr_viridian_regs(), isn't > this going to needlessly spam the log? I.e. printing this just once (per > VM) would seem to suffice. Or alternatively I would think that > HVM_DBG_LOG() would be a better choice here. It's unlikely to print more than once before the VM wedges. We could go further and inject #GP, which we ought to be doing anyway. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |