<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 26, 2019 at 4:20 PM Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>> wrote:<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>
----- Original Message -----<br>
> +<br>
> +static void setup(void)<br>
> +{<br>
> +     int i, fd;<br>
> +<br>
> +     if (access(PATH_VM_NRHPS, F_OK)) {<br>
> +             tst_res(TINFO, "Huge page is not supported");<br>
> +             size = getpagesize();<br>
> +             no_hugepage = 1;<br>
> +     } else {<br>
> +             int val;<br>
> +<br>
> +             SAFE_FILE_PRINTF(PATH_VM_NRHPS, "%d", 1);<br>
<br>
This is still SAFE write, which may trigger TBROK:<br>
<br>
# ./pkey01 <br>
tst_test.c:1100: INFO: Timeout per run is 0h 05m 00s<br>
safe_file_ops.c:301: BROK: Failed to close <span class="gmail_default" style="font-size:small"></span>FILE '/proc/sys/vm/nr_hugepages' at pkey01.c:67: EOPNOTSUPP<br>
safe_macros.c:773: WARN: pkey01.c:91: umount(tmp_pkey) failed: ENOENT<br>
safe_macros.c:184: WARN: pkey01.c:92: rmdir(tmp_pkey) failed: ENOENT<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Er, sorry about still not working here.</div></div><div><br><div class="gmail_default">If a system(e.g PowerKVM guest) configured with NO huge page support, then the <span class="gmail_default"></span>file '/proc/sys/vm/nr_hugepages' exist but EOPNOTSUPP to read/write. In this key01, perhaps I shouldn't use the "/proc/.../nr_hugepages" to detect that at the beginning. </div><div class="gmail_default"><br></div><div class="gmail_default">Seems the correct way is to use "/sys/kernel/mm/hugepages/" for huge page detecting and leave the SAFE_FILE_* still in setup(). Because we need to catch the "/proc/.../nr_hugepages" open/close or read/write issue in testing.</div><div class="gmail_default"><br></div><div class="gmail_default">What do you think? Should we mask the EOPNOTSUPP as a TCONF skipping or others?</div></div></div><div><br></div>-- <br><div dir="ltr" class="m_-4029781209951391135gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>