<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Martin,</div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Martin Doucha <<a href="mailto:mdoucha@suse.cz">mdoucha@suse.cz</a>> wrote:<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>> +    /* Create VM */<br>
> <br>
> These comments are pretty redundant when we have ioctl's like<br>
> KVM_CREATE_VM and KVM_CREATE_VCPU. There are much harder things to<br>
> understand in this patchset.<br>
<br>
I know that the code is straightforward. But it's quite dense so I've<br>
added the comments as section headers for easier navigation when you<br>
need to change something.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I have no dissent on the comments.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">But I'm wondering do we need a check if the SUT support</div><div class="gmail_default" style="font-size:small">creating VM before doing that? Or, should we guarantee</div><div class="gmail_default" style="font-size:small">the KVM module is enabled.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I simply built these codes and got TBROK on my KVM guest.</div><div class="gmail_default" style="font-size:small">Probably we should avoid this but I hoping it can skip gracefully.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">[root@local-guest-rhel83 ~]# virt-what<br>kvm<br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"># ./kvm_pagefault01<br>tst_sys_conf.c:64: TINFO: Path not found: '/sys/module/kvm/parameters/tdp_mmu'<br>tst_test.c:1458: TINFO: Timeout per run is 0h 05m 00s<br>lib_host.c:133: TBROK: open(/dev/kvm,2,0000) failed: ENOENT (2)<br><br>Summary:<br>passed   0<br>failed   0<br>broken   1<br>skipped  0<br>warnings 0<br><br></div></div><div class="gmail_default" style="font-size:small"><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>