Page MenuHomeFreeBSD
Differential D56400

glabel(8): Fix debug message for already existing labels
ClosedPublic
Actions

Authored by michaelo on Apr 15 2026, 8:13 AM.
Tags
None
Referenced Files
F167896781: D56400.id175536.diff
Tue, Aug 25, 7:55 AM
Unknown Object (File)
Tue, Aug 25, 2:21 AM
Unknown Object (File)
Tue, Aug 25, 12:20 AM
Unknown Object (File)
Mon, Aug 24, 1:09 PM
Unknown Object (File)
Mon, Aug 24, 8:47 AM
Unknown Object (File)
Mon, Aug 24, 8:26 AM
Unknown Object (File)
Mon, Aug 24, 6:57 AM
Unknown Object (File)
Mon, Aug 24, 5:37 AM
View All 71 Files
Subscribers
None

Details

Reviewers
mav
markj
imp
Commits
rG0fda88fa010f: glabel(8): Fix debug message for already existing labels
rG5131d206877c: glabel(8): Fix debug message for already existing labels
rGc93e8b63d62b: glabel(8): Fix debug message for already existing labels
rG92c69c791c43: glabel(8): Fix debug message for already existing labels
Summary

Remove the duplicate (incorrect) name.

PR: 294542
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

michaelo requested review of this revision.Apr 15 2026, 8:13 AM
michaelo created this revision.
Harbormaster completed remote builds in B72212: Diff 175536.Apr 15 2026, 8:13 AM
markj added inline comments.Apr 15 2026, 1:23 PM
sys/geom/label/g_label.c
255

Just don't print pp2->name at all? We already know it's the same as name.

michaelo updated this revision to Diff 175585.Apr 15 2026, 4:05 PM

Remove the duplicate (incorrect) name.

Harbormaster completed remote builds in B72232: Diff 175585.Apr 15 2026, 4:05 PM
michaelo edited the summary of this revision. (Show Details)Apr 15 2026, 4:06 PM
michaelo marked an inline comment as done.
michaelo added inline comments.
sys/geom/label/g_label.c
255

Totally right!

markj accepted this revision.Apr 15 2026, 4:07 PM
This revision is now accepted and ready to land.Apr 15 2026, 4:07 PM
Closed by commit rG92c69c791c43: glabel(8): Fix debug message for already existing labels (authored by michaelo). · Explain WhyApr 15 2026, 6:19 PM
This revision was automatically updated to reflect the committed changes.
michaelo marked an inline comment as done.
michaelo added a commit: rG92c69c791c43: glabel(8): Fix debug message for already existing labels.
michaelo added a commit: rGc93e8b63d62b: glabel(8): Fix debug message for already existing labels.Apr 22 2026, 9:31 AM
michaelo added a commit: rG5131d206877c: glabel(8): Fix debug message for already existing labels.
michaelo added a commit: rG0fda88fa010f: glabel(8): Fix debug message for already existing labels.Apr 22 2026, 9:35 AM

Revision Contents
Changeset List

PathSize
sys/
geom/
label/
4 lines

Diff 175594

View Options

sys/geom/label/g_label.c

Loading...