<div dir="ltr"><div><div><div><div><div><br></div>Hi...<br><br></div>I think I found a bug in pwrite04.c. I mentioned it on the linux<br></div>filesystem mailing list and it was suggested that I mention it to<br></div>you...<br><br></div>-Mike<br><div><div><div><div><div><br><div class="gmail_quote"><span style="font-size:large;font-weight:bold">Forwarded conversation</span><br>Subject: <b class="gmail_sendername">pwrite04 ltp bug?</b><br>------------------------<br><br><span class="undefined"><font color="#888">From: <b class="undefined">Mike Marshall</b> <span dir="ltr"><<a href="mailto:hubcap@omnibond.com">hubcap@omnibond.com</a>></span><br>Date: Fri, May 20, 2016 at 4:51 PM<br>To: linux-fsdevel <<a href="mailto:linux-fsdevel@vger.kernel.org">linux-fsdevel@vger.kernel.org</a>>, Mike Marshall <<a href="mailto:hubcap@omnibond.com">hubcap@omnibond.com</a>><br></font><br></span><br>Hi everyone...<br>
<br>
While fixing a couple of LTP regressions in the out-of-tree Orangefs<br>
kernel module, I think I found a problem with the pwrite04 test,<br>
here's what I think was meant:<br>
<br>
# git diff testcases/kernel/syscalls/pwrite/pwrite04.c<br>
diff --git a/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
b/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
index b297682..4ca7dc1 100644<br>
--- a/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
+++ b/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
@@ -196,9 +196,9 @@ int main(int ac, char *av[])<br>
<br>
                }<br>
                if (statbuf.st_size != K5) {<br>
-                       tst_resm(TFAIL, "file size is %ld != K4",<br>
+                       tst_resm(TFAIL, "file size is %ld != K5",<br>
                                 statbuf.st_size);<br>
-<br>
+                       cleanup();<br>
                }<br>
                tst_resm(TPASS, "O_APPEND test passed.");<br>
<br>
Should I even be talking about this here, or should I go find<br>
the github maintainers of LTP and ask them about it?<br>
<span class="HOEnZb"><font color="#888888"><br>
-Mike<br>
</font></span><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Zorro Lang</b> <span dir="ltr"><<a href="mailto:zorro.lang@gmail.com">zorro.lang@gmail.com</a>></span><br>Date: Fri, May 20, 2016 at 8:56 PM<br>To: Mike Marshall <<a href="mailto:hubcap@omnibond.com">hubcap@omnibond.com</a>><br></font><br></span><br><span class=""><p dir="ltr"><br>
2016年5月21日 04:52，"Mike Marshall" <<a href="mailto:hubcap@omnibond.com" target="_blank">hubcap@omnibond.com</a>>写道：<br>
><br>
> Hi everyone...<br>
><br>
> While fixing a couple of LTP regressions in the out-of-tree Orangefs<br>
> kernel module, I think I found a problem with the pwrite04 test,<br>
> here's what I think was meant:<br>
><br>
> # git diff testcases/kernel/syscalls/pwrite/pwrite04.c<br>
> diff --git a/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
> b/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
> index b297682..4ca7dc1 100644<br>
> --- a/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
> +++ b/testcases/kernel/syscalls/pwrite/pwrite04.c<br>
> @@ -196,9 +196,9 @@ int main(int ac, char *av[])<br>
><br>
>                 }<br>
>                 if (statbuf.st_size != K5) {<br>
> -                       tst_resm(TFAIL, "file size is %ld != K4",<br>
> +                       tst_resm(TFAIL, "file size is %ld != K5",<br>
>                                  statbuf.st_size);<br>
> -<br>
> +                       cleanup();<br>
>                 }<br>
>                 tst_resm(TPASS, "O_APPEND test passed.");<br>
><br>
> Should I even be talking about this here, or should I go find<br>
> the github maintainers of LTP and ask them about it?</p>
</span><p dir="ltr">Hi Mike,</p>
<p dir="ltr">Generally, we send LTP related patches to: <a href="mailto:ltp@lists.linux.it" target="_blank">ltp@lists.linux.it</a></p>
<p dir="ltr">Thanks,<br>
Zorro</p>
<p dir="ltr">><br>
> -Mike<span class="HOEnZb"><font color="#888888"><br>
> --<br>
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in<br>
> the body of a message to <a href="mailto:majordomo@vger.kernel.org" target="_blank">majordomo@vger.kernel.org</a><br>
> More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
</font></span></p>
<br></div><br></div></div></div></div></div></div>