<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 31, 2018 at 9:08 PM, Cyril Hrubis <span dir="ltr"><<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The result pointer wasn't initialized in tst_reinit(), which haven't<br>
allowed for test result propagation to the main test process and there<br>
is actually no reason not to do that. So this commit initializes the<br>
results pointer in tst_reinit() and also adds a needs_ipc_path flag to<br>
tst_test structure, which causes the library not to unlink the IPC file<br>
after it has been mapped and also inserts LTP_IPC_PATH to the<br>
environment.<br>
<br>
This commit also adds a test that also shows the usage of the API.<br><div class="gmail_default" style="font-size:small;display:inline">...</div><br>
        ptr = SAFE_MMAP(NULL, size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);<br>
+       results = ptr;<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">The ptr seems redundant here, I suggest to remove it from tst_reinit().</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Others looks good to me.</div></div><div><br></div>-- <br><div class="m_675180538698900900gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div>
</div></div>