Page MenuHomeFreeBSD

ZFS: add bookmark renaming

Authored by avg on Sep 25 2019, 8:24 PM.


Group Reviewers
rS353037: ZFS: add bookmark renaming

The feature is implemented as an extension of the existing ZFS_IOC_RENAME
ioctl. Both the userland and the DSL interfaces support renaming only a
single bookmark at a time. As of now, there is no ZCP interface to the new
functionality. I am going to add it once the DSL interface passes a test of

Diff Detail

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

Event Timeline

the interfaces look good to me.

552–562 ↗(On Diff #62570)

I think we should do this in dsl_bookmark_rename_check (assuming it's necessary - I didn't see similar namecheck() calls for renaming snapshots, although maybe that's a bug?). This will help when making the channel program function, which would also want to do the same checks (name length at least).

bcr added a subscriber: bcr.

OK from manpages.

avg marked an inline comment as done.Sep 30 2019, 2:08 PM
avg added inline comments.
552–562 ↗(On Diff #62570)

I added some of these checks before adding ENTITY_NAME check because there was nothing to ensure name validity back then.
I think that now we can be sure that the dataset name and the old bookmark name must be valid.
So, I think that only the new bookmark name and its full length need to be checked now.

I will move the needed checks to the check function and drop the redundant checks.

avg marked an inline comment as done.
  • drop redundant name checks
  • move name checks to the check function
This revision was not accepted when it landed; it landed in state Needs Review.Oct 3 2019, 11:09 AM
Closed by commit rS353037: ZFS: add bookmark renaming (authored by avg). · Explain Why
This revision was automatically updated to reflect the committed changes.