diff options
author | Christian Borntraeger <borntraeger@de.ibm.com> | 2008-08-29 07:29:45 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-10-15 04:15:24 -0400 |
commit | a0046b6db1c514149585e11895cd8434e0eafa79 (patch) | |
tree | df02a1764fcae0bd94b0db8fa3463558cc690bde | |
parent | a89c1ad270ca7ad0eec2667bc754362ce7b142be (diff) |
KVM: s390: Make facility bits future-proof
Heiko Carstens pointed out, that its safer to activate working facilities
instead of disabling problematic facilities. The new code uses the host
facility bits and masks it with known good ones.
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
-rw-r--r-- | arch/s390/kvm/priv.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/s390/kvm/priv.c b/arch/s390/kvm/priv.c index d1faf5c54405..cce40ff2913b 100644 --- a/arch/s390/kvm/priv.c +++ b/arch/s390/kvm/priv.c | |||
@@ -157,8 +157,8 @@ static int handle_stfl(struct kvm_vcpu *vcpu) | |||
157 | int rc; | 157 | int rc; |
158 | 158 | ||
159 | vcpu->stat.instruction_stfl++; | 159 | vcpu->stat.instruction_stfl++; |
160 | facility_list &= ~(1UL<<24); /* no stfle */ | 160 | /* only pass the facility bits, which we can handle */ |
161 | facility_list &= ~(1UL<<23); /* no large pages */ | 161 | facility_list &= 0xfe00fff3; |
162 | 162 | ||
163 | rc = copy_to_guest(vcpu, offsetof(struct _lowcore, stfl_fac_list), | 163 | rc = copy_to_guest(vcpu, offsetof(struct _lowcore, stfl_fac_list), |
164 | &facility_list, sizeof(facility_list)); | 164 | &facility_list, sizeof(facility_list)); |