qemu-s390x
[Top][All Lists]
Advanced

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

Re: [kvm-unit-tests PATCH v3] s390x: Add stsi 3.2.2 tests


From: David Hildenbrand
Subject: Re: [kvm-unit-tests PATCH v3] s390x: Add stsi 3.2.2 tests
Date: Tue, 31 Mar 2020 19:24:31 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0

On 31.03.20 11:35, Cornelia Huck wrote:
> On Tue, 31 Mar 2020 03:14:56 -0400
> Janosch Frank <address@hidden> wrote:
> 
>> Subcode 3.2.2 is handled by KVM/QEMU and should therefore be tested
>> a bit more thorough.
> 
> s/thorough/thoroughly/ ?

I fixed that up for now.

[...]

>> +    if (data->vm[0].ext_name_encoding != 2) {
>> +            report_skip("Extended VM names are not UTF-8.");
> 
> Do we expect this to be anything other than UTF-8?

I'm not a fan of hard-coding things that can change to other valid
values. No strong feelings though. I'll let Janosch decide :)


-- 
Thanks,

David / dhildenb




reply via email to

[Prev in Thread] Current Thread [Next in Thread]