btrfs-progs: optionally restore metadata
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest posting 2/20 of 1 patch 1 replyIn review
- v2 posting 1/21 patch 2 repliesApplied
Changes in v2 from the cover letter
* Documented in the manpage
* Added to usage() for btrfs restore
* Made it an optional flag (-m/--restore-metadata)
* Use endian-safe macros to access the on-disk data.
* Restore the proper mtime instead of atime twice.
* Restore owner and mode
* Restore metadata for directories as well as plain files.
* Since it's now explicitly requested, errors are fatal unless ignore_errors is requested.
Patches in v2 posting 2/2 No applies recorded yet
The patch the cover letter announces is not in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [PATCH v2 0/1] btrfs-progs: optionally restore metadata
Changes since v1:
* Documented in the manpage
* Added to usage() for btrfs restore
* Made it an optional flag (-m/--restore-metadata)
* Use endian-safe macros to access the on-disk data.
* Restore the proper mtime instead of atime twice.
* Restore owner and mode
* Restore metadata for directories as well as plain files.
* Since it's now explicitly requested, errors are fatal unless ignore_errors is requested.
I tested this on the array I'm restoring, it looks sane to me.
Thanks to Noah Massey for the patch review, and Duncan for the prompt to add owner/permissions to the patch.
Symlinks and hardlinks are beyond the scope of these changes, I'll look into them if this looks good to everyone.