Page MenuHomeFreeBSD
Differential D18521

gmirror: Fix a bug introduced in r341674
ClosedPublic
Actions

Authored by cem on Dec 12 2018, 1:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 6:48 AM
Unknown Object (File)
Tue, Nov 19, 10:20 PM
Unknown Object (File)
Oct 21 2024, 9:49 PM
Unknown Object (File)
Oct 8 2024, 2:38 PM
Unknown Object (File)
Sep 18 2024, 11:33 PM
Unknown Object (File)
Sep 18 2024, 11:11 PM
Unknown Object (File)
Sep 17 2024, 3:43 PM
Unknown Object (File)
Sep 16 2024, 8:38 AM
View All 34 Files
Subscribers
imp

Details

Reviewers
markj
cindy.yang_isilon.com
Commits
rS341840: gmirror: Fix a bug introduced in r341674
Summary

r341674 inadvertently introduced a bug where newer mirror components being
tasted would clear the high sc_flags that are not controlled by component
metadata, such as G_MIRROR_DEVICE_FLAG_TASTING. This could plausibly expose
a small window of time during STARTING where device destruction might race
with mirror component addition, probably resulting in a crash.

X-MFC-With: r341674

Diff Detail

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

Event Timeline

cem created this revision.Dec 12 2018, 1:09 AM
cem edited the summary of this revision. (Show Details)Dec 12 2018, 1:10 AM
cem added reviewers: markj, cindy.yang_isilon.com.
Harbormaster completed remote builds in B21505: Diff 51883.Dec 12 2018, 1:11 AM
markj accepted this revision.Dec 12 2018, 2:12 AM
markj added inline comments.
sys/geom/mirror/g_mirror.c
3072 ↗(On Diff #51883)

While here, perhaps assert that the softc is xlocked at the beginning of this routine.

This revision is now accepted and ready to land.Dec 12 2018, 2:12 AM
cem marked an inline comment as done.Dec 12 2018, 2:19 AM
cem added inline comments.
sys/geom/mirror/g_mirror.c
3072 ↗(On Diff #51883)

Sure, that sounds like a good idea. Will do.

cem updated this revision to Diff 51888.Dec 12 2018, 2:22 AM
cem marked an inline comment as done.

Add softc xlock assertion

This revision now requires review to proceed.Dec 12 2018, 2:22 AM
Harbormaster completed remote builds in B21509: Diff 51888.Dec 12 2018, 2:24 AM
markj accepted this revision.Dec 12 2018, 2:25 AM
This revision is now accepted and ready to land.Dec 12 2018, 2:25 AM
Closed by commit rS341840: gmirror: Fix a bug introduced in r341674 (authored by cem). · Explain WhyDec 12 2018, 5:49 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptDec 12 2018, 5:49 AM

Revision Contents
Changeset List

PathSize
head/
sys/
geom/
mirror/
5 lines

Diff 51896

View Options

head/sys/geom/mirror/g_mirror.c

Loading...