Re: [PATCH] bcache: untagle cache_aolloc

From: Kent Overstreet
Date: Mon Jul 18 2016 - 06:13:41 EST


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.