From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH v6 2/2] decrement static keys on real destroy time Date: Wed, 23 May 2012 13:33:54 -0700 Message-ID: <20120523133354.373f1bb4.akpm@linux-foundation.org> References: <1337682339-21282-1-git-send-email-glommer@parallels.com> <1337682339-21282-3-git-send-email-glommer@parallels.com> <20120522154610.f2f9b78e.akpm@linux-foundation.org> <4FBCAAF4.4030803@parallels.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: , , , , , Tejun Heo , Li Zefan , Johannes Weiner , Michal Hocko , David Miller To: Glauber Costa Return-path: In-Reply-To: <4FBCAAF4.4030803@parallels.com> Sender: owner-linux-mm@kvack.org List-Id: netdev.vger.kernel.org On Wed, 23 May 2012 13:16:36 +0400 Glauber Costa wrote: > On 05/23/2012 02:46 AM, Andrew Morton wrote: > > Here, we're open-coding kinda-test_bit(). Why do that? These flags are > > modified with set_bit() and friends, so we should read them with the > > matching test_bit()? > > My reasoning was to be as cheap as possible, as you noted yourself two > paragraphs below. These aren't on any fast path, are they? Plus: you failed in that objective! The C compiler's internal scalar->bool conversion makes these functions no more efficient than test_bit(). > > So here are suggested changes from*some* of the above discussion. > > Please consider, incorporate, retest and send us a v7? > > How do you want me to do it? Should I add your patch ontop of mine, > and then another one that tweaks whatever else is left, or should I just > merge those changes into the patches I have? A brand new patch, I guess. I can sort out the what-did-he-change view at this end. -- 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