Page MenuHomeFreeBSD

netdump: Add ddb 'show netdumpconf' command
Changes PlannedPublic

Authored by cem on Nov 21 2018, 11:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 19 2023, 9:53 PM
Unknown Object (File)
Dec 19 2023, 3:38 PM
Unknown Object (File)
Nov 11 2023, 4:46 AM
Unknown Object (File)
Oct 21 2023, 5:26 PM
Unknown Object (File)
Oct 12 2023, 12:48 AM
Unknown Object (File)
Oct 10 2023, 3:48 AM
Unknown Object (File)
Sep 30 2023, 1:42 PM
Subscribers
None

Details

Reviewers
markj

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 21156
Build 20514: arc lint + arc unit

Event Timeline

sys/netinet/netdump/netdump_client.c
1330

The casts to unsigned here and below shouldn't be needed.

1336

I was going to suggest this too. Maybe have a generic "show dumpdev" or whatever which calls a function in the netdump code to print additional parameters.

cem marked 2 inline comments as done.Nov 22 2018, 5:59 PM
cem added inline comments.
sys/netinet/netdump/netdump_client.c
1330

Why not? nd_server_port is a uint16_t; the kda casts below are uint8_t.

nd_seqno is uint32_t, which I guess is the same size as unsigned on all supported FreeBSD architectures; that one could be dropped.

1336

That seems reasonable. I probably won't have time to work on that until at least Sunday — three different family gatherings this week 🥳🤯😱.

sys/netinet/netdump/netdump_client.c
1330

Sorry, I didn't think that through. I think they're unneeded if you also change the format specifier to %d, which should be fine since nd_server_port and the kda fields fit in an int. It's not important. :)

cem planned changes to this revision.Sep 28 2019, 9:34 PM

I won't necessarily have time to work on this soon (this diff is from like a year ago), but annotate the differential such that it isn't in any "waiting on reviewers" queue. The right db command, discussed above, is probably a generic show dumperinfo.

Also, poachable.