From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with SMTP id B88896B00EE for ; Mon, 12 Sep 2011 11:27:28 -0400 (EDT) Date: Mon, 12 Sep 2011 10:27:25 -0500 (CDT) From: Christoph Lameter Subject: Re: [PATCH 2/2] slub: reduce a variable in __slab_free() In-Reply-To: <1315559521.31737.799.camel@debian> Message-ID: References: <1315559521.31737.799.camel@debian> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: "Alex,Shi" Cc: Pekka Enberg , "linux-mm@kvack.org" , "akpm@linux-foundation.org" , "Chen, Tim C" , "Huang, Ying" On Fri, 9 Sep 2011, Alex,Shi wrote: > After the compxchg, the new.inuse are fixed in __slab_free as a local > variable, so we don't need a extra variable for it. True but its an easier read otherwise and the contents are extracted from the 16 bit value into a long which is easier to handle for the compiler later. Check to see if there is a code difference due to this patch. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org