Page MenuHomeFreeBSD
Differential D17306

netdump: Zone mbufs should be allocated before dump
ClosedPublic
Actions

Authored by cem on Sep 24 2018, 11:24 PM.
Tags
None
Referenced Files
F166238377: D17306.id49369.diff
Wed, Aug 12, 12:53 PM
Unknown Object (File)
Tue, Aug 11, 5:01 AM
Unknown Object (File)
Mon, Aug 10, 8:27 PM
Unknown Object (File)
Mon, Aug 10, 8:26 PM
Unknown Object (File)
Mon, Aug 10, 8:25 PM
Unknown Object (File)
Sat, Aug 8, 8:00 PM
Unknown Object (File)
Fri, Aug 7, 5:11 AM
Unknown Object (File)
Jun 2 2026, 5:09 AM
View All Files
Subscribers
imp

Details

Reviewers
markj
Commits
rS339501: netdump: Zone mbufs should be allocated before dump

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cem created this revision.Sep 24 2018, 11:24 PM
Harbormaster completed remote builds in B19785: Diff 48413.Sep 24 2018, 11:24 PM
markj added inline comments.Sep 25 2018, 1:27 AM
sys/kern/kern_mbuf.c
412 ↗(On Diff #48413)

I'd add a comment or a KASSERT message to the effect of "ran out of pre-allocated buffers".

447 ↗(On Diff #48413)

We want a similar assertion here.

cem updated this revision to Diff 48421.Sep 25 2018, 2:39 AM
cem marked 2 inline comments as done.

Use a more descriptive kassert, and also check dumping in a spot I missed :-)

Harbormaster completed remote builds in B19790: Diff 48421.Sep 25 2018, 2:39 AM
markj accepted this revision.Sep 25 2018, 3:38 PM
This revision is now accepted and ready to land.Sep 25 2018, 3:38 PM
Closed by commit rS339501: netdump: Zone mbufs should be allocated before dump (authored by cem). · Explain WhyOct 20 2018, 10:25 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptOct 20 2018, 10:25 PM

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
6 lines

Diff 49369

View Options

head/sys/kern/kern_mbuf.c

Loading...