<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=Windows-1252"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">On 09/28/2017 10:25 PM, Joonas Lahtinen wrote: </div> <blockquote type="cite" cite="mid:1506608701.9186.44.camel@linux.intel.com"> <pre wrap="">On Thu, 2017-09-28 at 10:09 +0800, Xiaolin Zhang wrote: </pre> <blockquote type="cite"> <pre wrap="">if vgpu active, the page table entry should be initialized after allocation and then the hypersivor can ping pages succesuffly, otherwise hypervisor will ping pages failed and the host will print a lot of annoying errors such as “ERROR gvt: guest page write error -22, gfn 0x7ada8, pa 0x7ada89a8, var 0x6, len 1” when create linux guest. Signed-off-by: Xiaolin Zhang <a class="moz-txt-link-rfc2396E" href="mailto:xiaolin.zhang@intel.com"><xiaolin.zhang@intel.com></a> </pre> </blockquote> <pre wrap="">Why does the hypervisor try to access the entries prior to them being made valid for hardware? Regards, Joonas </pre> </blockquote> Hi Joonas, thanks your comment. I think what you ask is the point we got the error message in gvt since the current gvt implementation is that page under write protection and trapped should be valid with correct shadow page setup and p2m translation. Actually, to work with “initialize partially filled pagetables" , there is a certain refine work to do in gvt side (maybe less or maybe large). but before refine work done, this patch is trying to bring back gvt behavior as before “initialize partially filled pagetables"patch. BRs, Xiaolin </body> </html>