Page MenuHomeFreeBSD
Differential D3903

Change improper locking assertion in mge
ClosedPublic
Actions

Authored by bsz_semihalf.com on Oct 15 2015, 12:41 PM.
Tags
Referenced Files
F109305159: D3903.diff
Mon, Feb 3, 8:06 AM
Unknown Object (File)
Oct 29 2024, 2:40 PM
Unknown Object (File)
Oct 22 2024, 4:27 PM
Unknown Object (File)
Oct 20 2024, 7:12 AM
Unknown Object (File)
Sep 27 2024, 12:28 PM
Unknown Object (File)
Sep 24 2024, 11:15 PM
Unknown Object (File)
Sep 22 2024, 10:10 AM
Unknown Object (File)
Sep 20 2024, 5:58 AM
View All 54 Files
Subscribers
imp

Details

Reviewers
zbb
adrian
• ian
Commits
rS289948: Change improper locking assertion in mge
Summary

Assertion used here was not atomic (locks' state could change between consecutive 'if' conditions), what could lead to unwanted panic(). Moreover, it was wrong itself: it is *not* a problem if one of the locks is held during MGE_GLOBAL_LOCK.

Diff Detail

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

Event Timeline

bsz_semihalf.com updated this revision to Diff 9413.Oct 15 2015, 12:41 PM
bsz_semihalf.com retitled this revision from to Change improper locking assertion in mge.
bsz_semihalf.com updated this object.
bsz_semihalf.com edited the test plan for this revision. (Show Details)
bsz_semihalf.com added reviewers: adrian, • ian, zbb.
bsz_semihalf.com set the repository for this revision to rS FreeBSD src repository - subversion.
bsz_semihalf.com added a project: network.
Herald added a subscriber: imp. · View Herald TranscriptOct 15 2015, 12:41 PM
Closed by commit rS289948: Change improper locking assertion in mge (authored by zbb). · Explain WhyOct 25 2015, 10:17 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
mge/
6 lines

Diff 9702

View Options

head/sys/dev/mge/if_mgevar.h

Loading...