mirror of
https://github.com/versity/scoutfs.git
synced 2026-05-01 10:25:43 +00:00
scoutfs_alloc_prepare_commit() is badly named. All it really does is put the references to the two dirty alloc list blocks in the allocator. It must allways be called if allocation was attempted, but it's easier to require that it always be paired with _alloc_init(). If the srch compaction worker in the client sees an error it will send the error back to the server without writing its dirty blocks. In avoiding the write it also avoided putting the two block references, leading to leaked blocks. We've been seeing rare messages with leaked blocks in tests. Signed-off-by: Zach Brown <zab@versity.com>