Re: [PATCH v1 2/2] KVM: s390: vsie: simplify < 8k address checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 05/08/2018 09:30 AM, Christian Borntraeger wrote:
> 
> 
> On 05/02/2018 04:28 PM, David Hildenbrand wrote:
>> On 02.05.2018 14:42, Heiko Carstens wrote:
>>> On Wed, May 02, 2018 at 01:08:14PM +0200, David Hildenbrand wrote:
>>>> This makes it certainly more readable.
>>>>
>>>> Signed-off-by: David Hildenbrand <david@xxxxxxxxxx>
>>>> ---
>>>>  arch/s390/kvm/vsie.c | 10 +++++-----
>>>>  1 file changed, 5 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/arch/s390/kvm/vsie.c b/arch/s390/kvm/vsie.c
>>>> index 969882b54266..7c51a9dc0ec8 100644
>>>> --- a/arch/s390/kvm/vsie.c
>>>> +++ b/arch/s390/kvm/vsie.c
>>>> @@ -557,7 +557,7 @@ static int pin_blocks(struct kvm_vcpu *vcpu, struct vsie_page *vsie_page)
>>>>  	if (test_kvm_cpu_feat(vcpu->kvm, KVM_S390_VM_CPU_FEAT_64BSCAO))
>>>>  		gpa |= (u64) READ_ONCE(scb_o->scaoh) << 32;
>>>>  	if (gpa) {
>>>> -		if (!(gpa & ~0x1fffUL))
>>>> +		if (gpa < 8192)
>>>
>>> 2 * PAGE_SIZE, please. That's how it is done on various other places too.
>>>
>>
>> Christian, does the documentation use the wording "lowcore" or "8192" / 8k?
>>
>> If it is lowcore, I agree to use 2 * PAGE_SIZE. If not, I prefer it to
>> directly match documentation.
>>
>> Anyhow, I leave this decision to Janosch and Christian. Thanks for
>> having alook!
>>
> prefix area and reverse prefix area. 
> In fact what we check here is actually the reversed prefix area so for completeness we
> would need an additional check.

We seem to check both for SCA so we are good for SCA.

--
To unsubscribe from this list: send the line "unsubscribe linux-s390" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux