Page MenuHomeFreeBSD
Differential D51910

libsa/zfs: ignore labels with zero txg
ClosedPublic
Actions

Authored by glebius on Aug 15 2025, 5:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 1, 5:57 AM
Unknown Object (File)
Sun, Mar 29, 2:09 AM
Unknown Object (File)
Wed, Mar 18, 10:05 AM
Unknown Object (File)
Tue, Mar 17, 4:07 AM
Unknown Object (File)
Wed, Mar 11, 9:43 AM
Unknown Object (File)
Wed, Mar 11, 6:08 AM
Unknown Object (File)
Mon, Mar 9, 3:28 PM
Unknown Object (File)
Feb 26 2026, 8:43 PM
View All 55 Files
Subscribers
dab
delphij

Details

Reviewers
mav
imp
tsoome
manu
Commits
rGde055556a0d1: libsa/zfs: ignore labels with zero txg
Summary

Zero txg means this vdev has been correctly removed from the pool. For
reference see module/zfs/vdev_removal.c in OpenZFS. It calls
vdev_label_init(vd, 0, VDEV_LABEL_REMOVE), and second argument there is
the txg to write.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 66253
Build 63136: arc lint + arc unit

Event Timeline

glebius created this revision.Aug 15 2025, 5:42 AM
Herald added a reviewer: manu. · View Herald TranscriptAug 15 2025, 5:42 AM
Herald added subscribers: dab, delphij. · View Herald Transcript
Harbormaster completed remote builds in B66253: Diff 160384.Aug 15 2025, 5:42 AM
glebius requested review of this revision.Aug 15 2025, 5:42 AM
glebius added a parent revision: D51909: libsa/zfs: remove unused variable.
glebius added a child revision: D51911: libsa/zfs: simplify vdev_find_previous().
imp accepted this revision.Aug 15 2025, 9:27 PM
This revision is now accepted and ready to land.Aug 15 2025, 9:27 PM
mav accepted this revision.Aug 18 2025, 2:51 PM

Searching for ZPOOL_CONFIG_POOL_TXG in OpenZFS I indeed see plenty of similar txg == 0 checks.

tsoome accepted this revision.Aug 18 2025, 4:04 PM

thanks!

Closed by commit rGde055556a0d1: libsa/zfs: ignore labels with zero txg (authored by glebius). · Explain WhyAug 20 2025, 2:54 PM
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rGde055556a0d1: libsa/zfs: ignore labels with zero txg.

Revision Contents
Changeset List

PathSize
stand/
libsa/
zfs/
3 lines

Diff 160384

View Options

stand/libsa/zfs/zfsimpl.c

Loading...