HomeFreeBSD

Remove wrong assertion in log spacemap

Description

Remove wrong assertion in log spacemap

It is typical, but not generally true that if log summary has more
blocks it must also have unflushed metaslabs. Normally with metaslabs
flushed in order it works, but there are known exceptions, such as
device removal or metaslab being loaded during its flush attempt.

Before 600a02b8844 if spa_flush_metaslabs() hit loading metaslab it
usually stopped (unless memlimit is also exceeded), but now it may
flush more metaslabs, just skipping that particular one. This
increased chances of assertion to fire when the skipped metaslab is
flushed on next iteration if all other metaslabs in that summary
entry are already flushed out of order.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Alexander Motin <mav@FreeBSD.org>
Sponsored-By: iXsystems, Inc.
Closes #13486
Closes #13513

Details

Provenance
mavAuthored on Jun 1 2022, 4:54 PM
Tony Hutter <hutter2@llnl.gov>Committed on Jun 6 2022, 11:57 PM
Parents
rG1fdd768d7f55: libzfs: Fail making a dataset handle gracefully
Branches
Unknown
Tags
Unknown

Event Timeline