[DTrace-devel] [PATCH 2/2] test: Remove XFAIL from many fbt stress tests
Kris Van Hees
kvanhees at kvh-deb-bpf.us.oracle.com
Tue Jan 30 16:18:33 UTC 2024
On Fri, Jan 12, 2024 at 01:31:00PM -0500, eugene.loh at oracle.com wrote:
>
> These tests started to pass with commit bcf5f82d
> "cc, bpf: separate final BPF program creation from linking and loading".
>
> Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewedoby: Kris Van Hees <kris.van.hees at oracle.com>
> ---
> test/stress/fbtprovider/tst.basic.d | 1 -
> test/stress/fbtprovider/tst.return.d | 1 -
> test/stress/fbtsafety/tst.basename.d | 1 -
> test/stress/fbtsafety/tst.caller.d | 1 -
> test/stress/fbtsafety/tst.dirname.d | 1 -
> test/stress/fbtsafety/tst.errno.d | 1 -
> test/stress/fbtsafety/tst.execname.d | 1 -
> test/stress/fbtsafety/tst.gid.d | 1 -
> test/stress/fbtsafety/tst.hton.d | 1 -
> test/stress/fbtsafety/tst.index.d | 1 -
> test/stress/fbtsafety/tst.null.d | 1 -
> test/stress/fbtsafety/tst.pid.d | 1 -
> test/stress/fbtsafety/tst.ppid.d | 1 -
> test/stress/fbtsafety/tst.progenyof.d | 1 -
> test/stress/fbtsafety/tst.random.d | 1 -
> test/stress/fbtsafety/tst.rw.d | 1 -
> test/stress/fbtsafety/tst.stack.d | 1 -
> test/stress/fbtsafety/tst.stackdepth.d | 1 -
> test/stress/fbtsafety/tst.stddev.d | 1 -
> test/stress/fbtsafety/tst.strchr.d | 1 -
> test/stress/fbtsafety/tst.strjoin.d | 1 -
> test/stress/fbtsafety/tst.strstr.d | 1 -
> test/stress/fbtsafety/tst.strtok.d | 1 -
> test/stress/fbtsafety/tst.substr.d | 1 -
> test/stress/fbtsafety/tst.ucaller.d | 1 -
> test/stress/fbtsafety/tst.uid.d | 1 -
> test/stress/fbtsafety/tst.unalign.d | 1 -
> test/stress/fbtsafety/tst.uregs.d | 1 -
> test/stress/fbtsafety/tst.ustack.d | 1 -
> test/stress/fbtsafety/tst.ustackdepth.d | 1 -
> 30 files changed, 30 deletions(-)
>
> diff --git a/test/stress/fbtprovider/tst.basic.d b/test/stress/fbtprovider/tst.basic.d
> index d4702509..2c2d1c4e 100644
> --- a/test/stress/fbtprovider/tst.basic.d
> +++ b/test/stress/fbtprovider/tst.basic.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION: simple fbt arguments test.
> diff --git a/test/stress/fbtprovider/tst.return.d b/test/stress/fbtprovider/tst.return.d
> index 7f4b51a4..7df30897 100644
> --- a/test/stress/fbtprovider/tst.return.d
> +++ b/test/stress/fbtprovider/tst.return.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 - requires a high limit on locked pages */
>
> /*
> * ASSERTION: simple fbt provider return test.
> diff --git a/test/stress/fbtsafety/tst.basename.d b/test/stress/fbtsafety/tst.basename.d
> index 59dbbd7d..af3ef8bf 100644
> --- a/test/stress/fbtsafety/tst.basename.d
> +++ b/test/stress/fbtsafety/tst.basename.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.caller.d b/test/stress/fbtsafety/tst.caller.d
> index 9850d5bb..78552289 100644
> --- a/test/stress/fbtsafety/tst.caller.d
> +++ b/test/stress/fbtsafety/tst.caller.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.dirname.d b/test/stress/fbtsafety/tst.dirname.d
> index 9bd0bf83..0220fed7 100644
> --- a/test/stress/fbtsafety/tst.dirname.d
> +++ b/test/stress/fbtsafety/tst.dirname.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.errno.d b/test/stress/fbtsafety/tst.errno.d
> index ff4599ba..092ceb5d 100644
> --- a/test/stress/fbtsafety/tst.errno.d
> +++ b/test/stress/fbtsafety/tst.errno.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.execname.d b/test/stress/fbtsafety/tst.execname.d
> index 2367efb4..a41f032d 100644
> --- a/test/stress/fbtsafety/tst.execname.d
> +++ b/test/stress/fbtsafety/tst.execname.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.gid.d b/test/stress/fbtsafety/tst.gid.d
> index 7db783de..633e4c67 100644
> --- a/test/stress/fbtsafety/tst.gid.d
> +++ b/test/stress/fbtsafety/tst.gid.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.hton.d b/test/stress/fbtsafety/tst.hton.d
> index 7bcd69e3..f11fdb1a 100644
> --- a/test/stress/fbtsafety/tst.hton.d
> +++ b/test/stress/fbtsafety/tst.hton.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.index.d b/test/stress/fbtsafety/tst.index.d
> index 4719297d..e30b94c0 100644
> --- a/test/stress/fbtsafety/tst.index.d
> +++ b/test/stress/fbtsafety/tst.index.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.null.d b/test/stress/fbtsafety/tst.null.d
> index cdfd8c89..798b3225 100644
> --- a/test/stress/fbtsafety/tst.null.d
> +++ b/test/stress/fbtsafety/tst.null.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.pid.d b/test/stress/fbtsafety/tst.pid.d
> index 13a17fc2..1f940872 100644
> --- a/test/stress/fbtsafety/tst.pid.d
> +++ b/test/stress/fbtsafety/tst.pid.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.ppid.d b/test/stress/fbtsafety/tst.ppid.d
> index 8354b258..85be56ff 100644
> --- a/test/stress/fbtsafety/tst.ppid.d
> +++ b/test/stress/fbtsafety/tst.ppid.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.progenyof.d b/test/stress/fbtsafety/tst.progenyof.d
> index 486455bd..479704f2 100644
> --- a/test/stress/fbtsafety/tst.progenyof.d
> +++ b/test/stress/fbtsafety/tst.progenyof.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.random.d b/test/stress/fbtsafety/tst.random.d
> index 309b22b6..1495c804 100644
> --- a/test/stress/fbtsafety/tst.random.d
> +++ b/test/stress/fbtsafety/tst.random.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.rw.d b/test/stress/fbtsafety/tst.rw.d
> index e571acc1..245b532f 100644
> --- a/test/stress/fbtsafety/tst.rw.d
> +++ b/test/stress/fbtsafety/tst.rw.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.stack.d b/test/stress/fbtsafety/tst.stack.d
> index 1c5269be..243ba27a 100644
> --- a/test/stress/fbtsafety/tst.stack.d
> +++ b/test/stress/fbtsafety/tst.stack.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.stackdepth.d b/test/stress/fbtsafety/tst.stackdepth.d
> index 8b7f10b2..f0cb91b0 100644
> --- a/test/stress/fbtsafety/tst.stackdepth.d
> +++ b/test/stress/fbtsafety/tst.stackdepth.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.stddev.d b/test/stress/fbtsafety/tst.stddev.d
> index 1d3a84a2..db926da6 100644
> --- a/test/stress/fbtsafety/tst.stddev.d
> +++ b/test/stress/fbtsafety/tst.stddev.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.strchr.d b/test/stress/fbtsafety/tst.strchr.d
> index 920f9b45..a07edace 100644
> --- a/test/stress/fbtsafety/tst.strchr.d
> +++ b/test/stress/fbtsafety/tst.strchr.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.strjoin.d b/test/stress/fbtsafety/tst.strjoin.d
> index a1e12424..a22cc794 100644
> --- a/test/stress/fbtsafety/tst.strjoin.d
> +++ b/test/stress/fbtsafety/tst.strjoin.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.strstr.d b/test/stress/fbtsafety/tst.strstr.d
> index c8317e54..13f4a924 100644
> --- a/test/stress/fbtsafety/tst.strstr.d
> +++ b/test/stress/fbtsafety/tst.strstr.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.strtok.d b/test/stress/fbtsafety/tst.strtok.d
> index 878ce42e..bc0a707e 100644
> --- a/test/stress/fbtsafety/tst.strtok.d
> +++ b/test/stress/fbtsafety/tst.strtok.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.substr.d b/test/stress/fbtsafety/tst.substr.d
> index 929c65aa..8cd02117 100644
> --- a/test/stress/fbtsafety/tst.substr.d
> +++ b/test/stress/fbtsafety/tst.substr.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> #pragma D option bufsize=1000
> #pragma D option bufpolicy=ring
> diff --git a/test/stress/fbtsafety/tst.ucaller.d b/test/stress/fbtsafety/tst.ucaller.d
> index 59f76161..5ef1127f 100644
> --- a/test/stress/fbtsafety/tst.ucaller.d
> +++ b/test/stress/fbtsafety/tst.ucaller.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.uid.d b/test/stress/fbtsafety/tst.uid.d
> index 71d4823b..aceedd9d 100644
> --- a/test/stress/fbtsafety/tst.uid.d
> +++ b/test/stress/fbtsafety/tst.uid.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.unalign.d b/test/stress/fbtsafety/tst.unalign.d
> index 788c0fe6..70497644 100644
> --- a/test/stress/fbtsafety/tst.unalign.d
> +++ b/test/stress/fbtsafety/tst.unalign.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * We set our buffer size absurdly low to prevent a flood of errors that we
> diff --git a/test/stress/fbtsafety/tst.uregs.d b/test/stress/fbtsafety/tst.uregs.d
> index 16a13b30..dd64df45 100644
> --- a/test/stress/fbtsafety/tst.uregs.d
> +++ b/test/stress/fbtsafety/tst.uregs.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.ustack.d b/test/stress/fbtsafety/tst.ustack.d
> index 8c6cd8f7..426d2eef 100644
> --- a/test/stress/fbtsafety/tst.ustack.d
> +++ b/test/stress/fbtsafety/tst.ustack.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> diff --git a/test/stress/fbtsafety/tst.ustackdepth.d b/test/stress/fbtsafety/tst.ustackdepth.d
> index 3cc4540c..67ba0597 100644
> --- a/test/stress/fbtsafety/tst.ustackdepth.d
> +++ b/test/stress/fbtsafety/tst.ustackdepth.d
> @@ -4,7 +4,6 @@
> * Licensed under the Universal Permissive License v 1.0 as shown at
> * http://oss.oracle.com/licenses/upl.
> */
> -/* @@xfail: dtv2 */
>
> /*
> * ASSERTION:
> --
> 2.18.4
>
>
More information about the DTrace-devel
mailing list