Differential D17957

netdump: Fix netdumping with INVARIANTS kernels
ClosedPublic
Actions

Authored by cem on Nov 12 2018, 2:37 AM.

Details

Reviewers

markj

Commits

rS340358: netdump: Fix netdumping with INVARIANTS kernels

Summary

Correct boneheaded assertion I added in r339501. Mea culpa.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20762
Build 20163: arc lint + arc unit

Event Timeline

cem created this revision.Nov 12 2018, 2:37 AM

Harbormaster completed remote builds in B20760: Diff 50298.Nov 12 2018, 2:37 AM

markj accepted this revision.Nov 12 2018, 3:53 AM

markj added inline comments.

sys/kern/kern_mbuf.c
422	Really, this function should never get called while !dumping is true. I think you can just remove that test.
462	Ditto.

This revision is now accepted and ready to land.Nov 12 2018, 3:53 AM

cem marked an inline comment as done.Nov 12 2018, 3:57 AM

cem added inline comments.

sys/kern/kern_mbuf.c
422	OK

Removing '!dumping||' portion of assert condition -- it would always be false
anyway. Per markj.

This revision now requires review to proceed.Nov 12 2018, 3:59 AM

Harbormaster completed remote builds in B20762: Diff 50300.Nov 12 2018, 3:59 AM

markj accepted this revision.Nov 12 2018, 4:09 AM

This revision is now accepted and ready to land.Nov 12 2018, 4:09 AM

Closed by commit rS340358: netdump: Fix netdumping with INVARIANTS kernels (authored by cem). · Explain WhyNov 12 2018, 5:24 AM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptNov 12 2018, 5:24 AM

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_mbuf.c

8 lines

Diff 50300

View Options

netdump: Fix netdumping with INVARIANTS kernelsClosedPublicActions