From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Schwidefsky Subject: Re: [PATCH 03/04] kvm-s390: Improve pgste accesses Date: Fri, 04 Apr 2008 15:55:37 +0200 Message-ID: <1207317337.15093.7.camel@localhost> References: <1207314468.18908.12.camel@cotte.boeblingen.de.ibm.com> <1207314757.18908.20.camel@cotte.boeblingen.de.ibm.com> Reply-To: schwidefsky@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 , Avi Kivity , Christian Borntraeger , 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 On Fri, 2008-04-04 at 15:12 +0200, Carsten Otte wrote: > Index: kvm/include/asm-s390/pgtable.h > =================================================================== > --- kvm.orig/include/asm-s390/pgtable.h > +++ kvm/include/asm-s390/pgtable.h > @@ -553,12 +553,12 @@ static inline void ptep_rcp_copy(pte_t * > > 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); > if (skey & _PAGE_REFERENCED) > - set_bit(RCP_GR_BIT, pgste); > - if (test_and_clear_bit(RCP_HC_BIT, pgste)) > + set_bit_simple(RCP_GR_BIT, pgste); > + if (test_and_clear_bit_simple(RCP_HC_BIT, pgste)) > SetPageDirty(page); > - if (test_and_clear_bit(RCP_HR_BIT, pgste)) > + if (test_and_clear_bit_simple(RCP_HR_BIT, pgste)) > SetPageReferenced(page); > #endif > } > @@ -732,8 +732,8 @@ static inline int ptep_test_and_clear_yo > young = ((page_get_storage_key(physpage) & _PAGE_REFERENCED) != 0); > rcp_lock(ptep); > if (young) > - set_bit(RCP_GR_BIT, pgste); > - young |= test_and_clear_bit(RCP_HR_BIT, pgste); > + set_bit_simple(RCP_GR_BIT, pgste); > + young |= test_and_clear_bit_simple(RCP_HR_BIT, pgste); > rcp_unlock(ptep); > return young; > #endif Major formatting accident ? -- blue skies, Martin. "Reality continues to ruin my life." - Calvin. ------------------------------------------------------------------------- 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