Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix slop space calculation with block cloning #15874

Merged
merged 1 commit into from Feb 12, 2024

Conversation

Bi11
Copy link
Contributor

@Bi11 Bi11 commented Feb 10, 2024

Similar to deduplication, the size of data duplicated by block cloning should not be included in the slop space calculation.

Motivation and Context

After block cloning operations, we observe an unintended decrease in the reported available space.

Description

Similar to deduplication, the size of data duplicated by block cloning should not be included in the slop space calculation.

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@mmatuska mmatuska mentioned this pull request Feb 10, 2024
13 tasks
@Bi11 Bi11 force-pushed the patch-slop-brt branch 2 times, most recently from 8e119a8 to c1fb6ac Compare February 10, 2024 14:46
Similar to deduplication, the size of data duplicated by block cloning
should not be included in the slop space calculation.

Signed-off-by: Yuxin Wang <yuxinwang9999@gmail.com>
@behlendorf behlendorf merged commit 6cc93cc into openzfs:master Feb 12, 2024
24 of 26 checks passed
tonyhutter pushed a commit that referenced this pull request Feb 22, 2024
Similar to deduplication, the size of data duplicated by block cloning
should not be included in the slop space calculation.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Yuxin Wang <yuxinwang9999@gmail.com>
Closes #15874
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Mar 13, 2024
Similar to deduplication, the size of data duplicated by block cloning
should not be included in the slop space calculation.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Yuxin Wang <yuxinwang9999@gmail.com>
Closes openzfs#15874
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Mar 13, 2024
Similar to deduplication, the size of data duplicated by block cloning
should not be included in the slop space calculation.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Yuxin Wang <yuxinwang9999@gmail.com>
Closes openzfs#15874
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Mar 13, 2024
Similar to deduplication, the size of data duplicated by block cloning
should not be included in the slop space calculation.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Signed-off-by: Yuxin Wang <yuxinwang9999@gmail.com>
Closes openzfs#15874
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants