From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 03/04] kvm-s390: Improve pgste accesses Date: Sat, 05 Apr 2008 00:08:40 +0300 Message-ID: <47F698D8.7080808@qumranet.com> References: <1207314468.18908.12.camel@cotte.boeblingen.de.ibm.com> <1207314757.18908.20.camel@cotte.boeblingen.de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: sfr@canb.auug.org.au, "kvm-devel@lists.sourceforge.net" , Heiko Carstens , Christian Borntraeger , Martin Schwidefsky , Olaf Schnapper To: Carsten Otte Return-path: In-Reply-To: <1207314757.18908.20.camel@cotte.boeblingen.de.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces@lists.sourceforge.net Errors-To: kvm-devel-bounces@lists.sourceforge.net List-Id: kvm.vger.kernel.org Carsten Otte wrote: > From: Heiko Carstens > > There is no need to use interlocked updates when the rcp > lock is held. Therefore the simple bitops variants can be > used. This should improve performance. > > skey = page_get_storage_key(page_to_phys(page)); > if (skey & _PAGE_CHANGED) > - set_bit(RCP_GC_BIT, pgste); > + set_bit_simple(RCP_GC_BIT, pgste); > This looks whitespace damaged. -- Any sufficiently difficult bug is indistinguishable from a feature. ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace