diff options
| author | Alan Somers <[email protected]> | 2023-07-11 15:13:57 -0600 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2023-07-21 11:49:26 -0700 |
| commit | 6fd87e1d8df1cab8d6087026cbc361a0886a2a97 (patch) | |
| tree | ee115630d8cb4630be50f8b3dfa0d71fb436494e /module/zfs/zio_checksum.c | |
| parent | cf2a225b2481534e313029da1fec248b6d2ad297 (diff) | |
Don't emit cksum_{actual_expected} in ereport.fs.zfs.checksum events
With anything but fletcher-4, even a tiny change in the input will cause
the checksum value to change completely. So knowing the actual and
expected checksums doesn't provide much more information than "they
don't match". The harm in sending them is simply that they bloat the
event. In particular, on FreeBSD the event must fit into a 1016 byte
buffer.
Fixes #14717 for mirrored pools.
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Rich Ercolani <[email protected]>
Signed-off-by: Alan Somers <[email protected]>
Sponsored-by: Axcient
Closes #14717
Closes #15052
Diffstat (limited to 'module/zfs/zio_checksum.c')
| -rw-r--r-- | module/zfs/zio_checksum.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/module/zfs/zio_checksum.c b/module/zfs/zio_checksum.c index 6090959c5..9de515e87 100644 --- a/module/zfs/zio_checksum.c +++ b/module/zfs/zio_checksum.c @@ -515,8 +515,6 @@ zio_checksum_error_impl(spa_t *spa, const blkptr_t *bp, } if (info != NULL) { - info->zbc_expected = expected_cksum; - info->zbc_actual = actual_cksum; info->zbc_checksum_name = ci->ci_name; info->zbc_byteswapped = byteswap; info->zbc_injected = 0; |