On Thu, Jan 11, 2007 at 11:29:49AM -0600, Anthony Liguori wrote:
> Zhai, Edwin wrote:
> >+
> >+/* set info to hvm guest for restore */
> >+int xc_domain_hvm_setcontext(int xc_handle,
> >+ uint32_t domid,
> >+ hvm_domain_context_t *hvm_ctxt)
> >+{
> >+ int rc;
> >+ DECLARE_DOMCTL;
> >+
> >+ domctl.cmd = XEN_DOMCTL_sethvmcontext;
> >+ domctl.domain = domid;
> >+ set_xen_guest_handle(domctl.u.hvmcontext.ctxt, hvm_ctxt);
> >+
> >+ if ( (rc = mlock(hvm_ctxt, sizeof(*hvm_ctxt))) != 0 )
> >+ return rc;
> >+
> >+ rc = do_domctl(xc_handle, &domctl);
> >+
> >+ safe_munlock(hvm_ctxt, sizeof(*hvm_ctxt));
> >+
> >+ return rc;
> > }
>
> Perhaps these should be lock_pages instead of calling mlock() directly?
yes, you are right. i forgot to refresh old patch when rebase.
attached minor fix is okay.
thanks,
>
> Regards,
>
> Anthony Liguori
>
-- 
best rgds,
edwin

Attachment: hyper-call-fix.patch
Description: Text document

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel