<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Pengfei,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr"><span class="gmail_default"></span>Pengfei Xu <<a href="mailto:pengfei.xu@intel.com" target="_blank">pengfei.xu@intel.com</a>> wrote:<br></div><div dir="ltr" class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail_default" style="font-size:small">...</span><br>
> <br>
> My concern is that if an LTS distro backports the patch(commit e86c2c8b93),<br>
> then it will break this hardcode kernel-version comparing.<br>
> <br>
> <br>
  Ok, let's wait to confirm umip patch merged into Linux kernel main line,<br>
  and then merge the patch into LTP. :)<br></blockquote><div><br></div><div><div class="gmail_default" style="">Sorry, I'm not talking merge time. I mean we have to consider more kernel distros(RHEL, SLES, Ubuntu) when writing/amending LTP test. </div><div class="gmail_default" style=""><br></div><div class="gmail_default" style="">Imagine that, if RHEL8(kernel base on v4.18) backports the kernel patch(x86/umip: Add emulation (spoofing) for UMIP covered instructions in 64-bit processes) and what will happen after applying your patch in this umip_basic_test.c test? It will fail since that kernel version is less than 5.4. We probably need to find a way to solve this issue.</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> > +               #endif<br>
> > +       }<br>
> > +<br>
> >         if (WIFSIGNALED(status) && WTERMSIG(status) == SIGSEGV) {<br>
> > -               tst_res(TPASS, "Got SIGSEGV");<br>
> > +               tst_res(TPASS, "Got SIGSEGV\n\n");<br>
> ><br>
> <br>
> Why we need two '\n' here?<br>
> <br>
  Because it could help to split umip cases in LTP, there are 5 umip cases<br>
  in LTP tests, and we could check each umip case easily in LTP log,<br></blockquote><div><br></div></div><div class="gmail_default" style="font-size:small">It is unnecessary because each test shows TINFO before testing.</div><div class="gmail_default" style="font-size:small">  e.g  "umip_basic_test.c:41: INFO: TEST sgdt, sgdt result save at [0x7fff930bda8e]"</div><div><br></div>-- <br><div dir="ltr" class="m_-181870375662943215gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>