Page MenuHomeFreeBSD
Differential D2352

replace a comment about zfs recv -F corner case with a longer, more detailed one
ClosedPublic
Actions

Authored by avg on Apr 23 2015, 10:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 20, 3:37 PM
Unknown Object (File)
Thu, Feb 20, 8:01 AM
Unknown Object (File)
Thu, Feb 20, 8:00 AM
Unknown Object (File)
Thu, Feb 20, 7:54 AM
Unknown Object (File)
Mon, Feb 10, 6:01 AM
Unknown Object (File)
Jan 18 2025, 10:59 AM
Unknown Object (File)
Jan 8 2025, 6:26 AM
Unknown Object (File)
Dec 20 2024, 7:03 PM
View All 73 Files
Subscribers
delphij
imp

Details

Reviewers
delphij
pjd
smh
Group Reviewers
ZFS
Commits
rS282131: replace a comment about zfs recv -F corner case with a longer, more detailed one
Summary

The old comment in zfs_rezget explains what situation the code handles,
the new comment also describes how the situation can arise.

Diff Detail

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

Event Timeline

avg updated this revision to Diff 4963.Apr 23 2015, 10:40 AM
avg retitled this revision from to replace a comment about zfs recv -F corner case with a longer, more detailed one.
avg updated this object.
avg edited the test plan for this revision. (Show Details)
Herald added subscribers: imp, delphij. · View Herald TranscriptApr 23 2015, 10:40 AM
avg added reviewers: smh, pjd, delphij.Apr 23 2015, 10:42 AM
avg added a reviewer: ZFS.Apr 23 2015, 10:54 AM
delphij accepted this revision.Apr 27 2015, 8:53 PM
delphij edited edge metadata.

Looks fine, please consider making the style change a separate commit though.

sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_znode.c
1341 ↗(On Diff #4963)

I'd recommend this portion of change be in a different commit.

This revision is now accepted and ready to land.Apr 27 2015, 8:53 PM
smh accepted this revision.Apr 27 2015, 11:11 PM
smh edited edge metadata.

Nicely explains it to me :)

sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_znode.c
1341 ↗(On Diff #4963)

Personally as they are both no-op changes I wouldn't bother splitting.

Closed by commit rS282131: replace a comment about zfs recv -F corner case with a longer, more detailed one (authored by avg). · Explain WhyApr 28 2015, 9:19 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
cddl/
contrib/
opensolaris/
uts/
common/
fs/
zfs/
24 lines

Diff 5052

View Options

head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_znode.c

Loading...