[PATCH 3/7] [v2] zram/xvmalloc: free bit block insertion optimization

Pekka Enberg penberg at cs.helsinki.fi
Sat Jan 29 00:48:18 PST 2011


On Fri, 2011-01-28 at 08:58 -0600, Robert Jennings wrote:
> This change is in a conditional block which is entered only when there is
> an existing data block on the freelist where the insert has taken place.
> 
> The new block is pushed onto the freelist stack and this conditional block
> is updating links in the prior stack head to point to the new stack head.
> After this conditional block the first-/second-level indices are updated
> to indicate that there is a free block at this location.
> 
> This patch adds an immediate return from the conditional block to avoid
> setting bits again to indicate a free block on this freelist. The bits
> would already be set because there was an existing free block on this
> freelist.
> 
> Signed-off-by: Robert Jennings <rcj at linux.vnet.ibm.com>

Reviewed-by: Pekka Enberg <penberg at kernel.org>



More information about the devel mailing list