On Mon, Jul 18, 2016 at 12:11:09PM +0200, Johannes Thumshirn wrote:

bcache's cache_alloc() function currenty has no way freeing memory if one of
the allocations fails. Untangle the if + allocation statement so we have
defined checkpoints to free previous allocations if one fails.

nack. The existing error path handles failure midway through just fine.