Page MenuHomeFreeBSD

pfsync: Insert static trace points
AbandonedPublic

Authored by kp on Nov 14 2018, 9:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 30 2024, 10:24 PM
Unknown Object (File)
Sep 28 2024, 6:06 PM
Unknown Object (File)
Sep 25 2024, 7:33 PM
Unknown Object (File)
Sep 5 2024, 11:34 PM
Unknown Object (File)
Aug 12 2024, 5:58 AM
Unknown Object (File)
Aug 4 2024, 2:00 PM
Unknown Object (File)
Aug 4 2024, 2:00 PM
Unknown Object (File)
Aug 4 2024, 12:44 PM

Details

Reviewers
eri
glebius
Group Reviewers
network
Summary

Make it easier to debug & intrument pfsync.
The fbt provider isn't very helpful, because many important functions
are static. Introduce a handful of interesting probe points.

Sponsored by: Orange Business Services

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 20816

Event Timeline

eri added a subscriber: eri.

Can you add some text to the manual pages for documenting the feature? Possibly linking to some example?

This revision is now accepted and ready to land.Nov 26 2018, 1:09 AM
In D17994#389260, @eri wrote:

Can you add some text to the manual pages for documenting the feature? Possibly linking to some example?

I've got 'write a dtrace man page for pf/pfsync' on my todo list, but I'm waiting until I commit the pfsync changes. There's still some discussion going on, and if the implementation ends up changing the probe points will too.

glebius added a subscriber: glebius.

Thanks.

P.S. In pf itself, the dtrace probing should just substitute existing printf(9) debugging.

Abandoned in favour of D18373.

These trace points are closely tied to the pfsync improvement approach that has been abandoned, so there's no point inserting these.
I do hope to add trace points to pf at some point, but I'll create a new review then.