dtrace: fix signature mismatch

[aokblast]

Fix with declaration

[markj]

sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c

13002

This function is called directly in some places, so the patch doesn't compile.

For the taskq_dispatch case, I'd introduce a wrapper like:

static void
dtrace_enabling_matchall_task(void *args __unused)
{
    dtrace_enabling_matchall();
}

[aokblast]

Fix compile error

[aokblast]

Sorry. Too much patch at the same time.

I forgot to cherry-pick the patch from my kcfi work branch.

[markj]

sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c

12998

[aokblast]

Remove redundant return statement

[aokblast]

BTW, a question is that it is put inside the opensolaris directory. Should I upstream to illumos? I don't find the counterpart on illumos.

[markj]

In D49112#1120051, @aokblast wrote:

BTW, a question is that it is put inside the opensolaris directory. Should I upstream to illumos? I don't find the counterpart on illumos.

It's not necessary from my point of view. Our copy of dtrace has diverged from illumos quite a lot over time. It is also not particularly easy to upstream such small patches to illumos unless they fix some concrete bug, in my (limited) experience. You can try of course, I just would not say that it's necessary.

[aokblast]

I see. Thanks.

Here is the patch for merging.