Page MenuHomeFreeBSD

Drop z_stream zbuf.state->dummy from SDT probe.

Authored by on May 10 2019, 12:15 AM.



First field of deflate.c's private internal_state structure points to an union field and not much useful to monitor.
This zlib internal_state structure with dummy field is only defined in FreeBSD version and for upgrading to zlib version in kernel.

Given one field is removed, use the macro takes 6 arguments.

Test Plan

dtrace -l

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

It looks I can put 2 versions of zlib into kernel.
Let's wait and see.

I think this can go independently with other changes: is there some reason that you would like to wait instead of proceeding? edited the summary of this revision. (Show Details)

Delete comments about usage of STD_PROBE5.

I was able to link both old and new versions of zlib at same time over weekend.
So, this change became no longer immediate blocker.
I thought it would be nicer to address zlib conversion in a single commit instead of two commits once upon time.

On the other hand, I think this change itself is a good improvement and fix for using the same probe for multiple cases.
If the maintainer is happy with this change alone, I'm also happy to go this in.
Based on MAINTAINERS file, jmg is the owner.

Is this something we need to mention in UPDATING file?

+markj and bz@ for the DTrace probe.

@jmg -- do you have any objection with the proposed change?

This revision is now accepted and ready to land.May 14 2019, 6:07 PM
This revision was automatically updated to reflect the committed changes.