westerndigitalcorporation / zenfs

ZenFS is a storage backend for RocksDB that enables support for ZNS SSDs and SMR HDDs.
GNU General Public License v2.0
235 stars 86 forks source link

Fix Segmentation fault while enable zenfs gc. #250

Closed ywang-wnlo closed 1 year ago

ywang-wnlo commented 1 year ago

When zone become full, the wp_ will be negative value, so change to use max_capacity_.

ywang-wnlo commented 1 year ago

I changed the code, and force-pushed it. Is it OK?

yhr commented 1 year ago

I changed the code, and force-pushed it. Is it OK?

Yes, thanks! Our CI seems to be misbehaving though. I'll run the lint check and merge this manually when i'm back from vacation. Happy holidays @ywang-wnlo

yhr commented 1 year ago

Merged via PR #256 . Thanks @ywang-wnlo !