diff options
author | Serapheim Dimitropoulos <[email protected]> | 2023-08-25 10:28:36 -0700 |
---|---|---|
committer | GitHub <[email protected]> | 2023-08-25 10:28:36 -0700 |
commit | ed39d668ea63e0bbf4e3e4d761a84c7d42ef8f8b (patch) | |
tree | 6999cf71b3d56338ffd6bcb461fdfcd8870ae681 /include | |
parent | eda3fcd56ff9f46dd606c48331cb786856f37fe3 (diff) |
Update outdated assertion from zio_write_compress
As part of some internal gang block testing within Delphix
we hit the assertion removed by this patch. The assertion
was triggered by a ZIO that had two copies and was a gang
block making the following expression equal to 3:
```
MIN(zp->zp_copies + BP_IS_GANG(bp), spa_max_replication(spa))
```
and failing when we expected the above to be equal to
`BP_GET_NDVAS(bp)`.
The assertion is no longer valid since the following commit:
```
commit 14872aaa4f909d72c6b5e4105dadcfa13c7d9d66
Author: Matthew Ahrens <[email protected]>
Date: Mon Feb 6 09:37:06 2023 -0800
EIO caused by encryption + recursive gang
```
The above commit changed gang block headers so they can't
have more than 2 copies but the assertion in question from
this PR was never updated.
Reviewed-by: George Wilson <[email protected]>
Reviewed-by: Matthew Ahrens <[email protected]>
Signed-off-by: Serapheim Dimitropoulos <[email protected]>
Closes #15180
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions