diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2008-04-04 10:03:34 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-04-27 05:01:00 -0400 |
commit | c71799c1f404c6e4f34fa64e6be39cd6149e5019 (patch) | |
tree | cfc9b0d49e02156b0b9931363c21e957ff9caae1 /include | |
parent | f603f0731f43421403160f5f8b12e90f2e51f064 (diff) |
KVM: s390: Improve pgste accesses
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.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Carsten Otte <cotte@de.ibm.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/asm-s390/pgtable.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/include/asm-s390/pgtable.h b/include/asm-s390/pgtable.h index 7fe5c4b6d82d..4c0698c0dda5 100644 --- a/include/asm-s390/pgtable.h +++ b/include/asm-s390/pgtable.h | |||
@@ -553,12 +553,12 @@ static inline void ptep_rcp_copy(pte_t *ptep) | |||
553 | 553 | ||
554 | skey = page_get_storage_key(page_to_phys(page)); | 554 | skey = page_get_storage_key(page_to_phys(page)); |
555 | if (skey & _PAGE_CHANGED) | 555 | if (skey & _PAGE_CHANGED) |
556 | set_bit(RCP_GC_BIT, pgste); | 556 | set_bit_simple(RCP_GC_BIT, pgste); |
557 | if (skey & _PAGE_REFERENCED) | 557 | if (skey & _PAGE_REFERENCED) |
558 | set_bit(RCP_GR_BIT, pgste); | 558 | set_bit_simple(RCP_GR_BIT, pgste); |
559 | if (test_and_clear_bit(RCP_HC_BIT, pgste)) | 559 | if (test_and_clear_bit_simple(RCP_HC_BIT, pgste)) |
560 | SetPageDirty(page); | 560 | SetPageDirty(page); |
561 | if (test_and_clear_bit(RCP_HR_BIT, pgste)) | 561 | if (test_and_clear_bit_simple(RCP_HR_BIT, pgste)) |
562 | SetPageReferenced(page); | 562 | SetPageReferenced(page); |
563 | #endif | 563 | #endif |
564 | } | 564 | } |
@@ -732,8 +732,8 @@ static inline int ptep_test_and_clear_young(struct vm_area_struct *vma, | |||
732 | young = ((page_get_storage_key(physpage) & _PAGE_REFERENCED) != 0); | 732 | young = ((page_get_storage_key(physpage) & _PAGE_REFERENCED) != 0); |
733 | rcp_lock(ptep); | 733 | rcp_lock(ptep); |
734 | if (young) | 734 | if (young) |
735 | set_bit(RCP_GR_BIT, pgste); | 735 | set_bit_simple(RCP_GR_BIT, pgste); |
736 | young |= test_and_clear_bit(RCP_HR_BIT, pgste); | 736 | young |= test_and_clear_bit_simple(RCP_HR_BIT, pgste); |
737 | rcp_unlock(ptep); | 737 | rcp_unlock(ptep); |
738 | return young; | 738 | return young; |
739 | #endif | 739 | #endif |