Re: [PATCH 05/12] s390/css: expose the AIV facility

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

 




On 01/25/2018 01:00 PM, Cornelia Huck wrote:
> On Thu, 25 Jan 2018 12:42:28 +0100
> Christian Borntraeger <borntraeger@xxxxxxxxxx> wrote:
> 
>> On 01/18/2018 06:54 PM, Cornelia Huck wrote:
>>> On Thu, 18 Jan 2018 13:02:45 +0100
>>> Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx> wrote:
>>>   
>>>> On 17.01.18 16:19, David Hildenbrand wrote:  
>>>>> On 16.01.2018 21:02, Christian Borntraeger wrote:    
>>>>>> From: Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx>
>>>>>>
>>>>>> The patch exposes the Adapter Interruption Virtualization facility (AIV)
>>>>>> of the general channel subsystem characteristics.
>>>>>>
>>>>>> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx>
>>>>>> Reviewed-by: Halil Pasic <pasic@xxxxxxxxxxxxxxxxxx>
>>>>>> Reviewed-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
>>>>>> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
>>>>>> ---
>>>>>>   arch/s390/include/asm/css_chars.h | 4 +++-
>>>>>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>>>>>
>>>>>> diff --git a/arch/s390/include/asm/css_chars.h b/arch/s390/include/asm/css_chars.h
>>>>>> index a478eb61aaf7..fb56fa3283a2 100644
>>>>>> --- a/arch/s390/include/asm/css_chars.h
>>>>>> +++ b/arch/s390/include/asm/css_chars.h
>>>>>> @@ -20,7 +20,9 @@ struct css_general_char {
>>>>>>   	u32 aif_tdd : 1; /* bit 56 */
>>>>>>   	u32 : 1;
>>>>>>   	u32 qebsm : 1;	 /* bit 58 */
>>>>>> -	u32 : 8;
>>>>>> +	u32 : 2;
>>>>>> +	u32 aiv : 1;     /* bit 61 */
>>>>>> +	u32 : 5;
>>>>>>   	u32 aif_osa : 1; /* bit 67 */
>>>>>>   	u32 : 12;
>>>>>>   	u32 eadm_rf : 1; /* bit 80 */
>>>>>>    
>>>>> "Expose" sounds like actually forwarding something / enabling a bit.    
>>>> Expose here basically means to make sth. visible what was already available.  
>>>>> Wonder if this can be squashed with another patch?
>>>>>
>>>>> Or rename to something like "define" ...    
>>>> I don't plan to change anything.  
>>>
>>> I'd prefer to simply squash this with the next patch, where the bit is
>>> actually checked.  
>>
>> Its a separate code (not KVM) so I keep it separate.
> 
> I would just have collected an ack from the relevant maintainers, but
> fair enough.

I need that anyway and its easier if I just have this hunk.

Heiko Martin, can any of you ack

this patch ([PATCH 05/12] s390/css: expose the AIV facility)
as well as
[PATCH 03/12] s390/bitops: add test_and_clear_bit_inv()
and
[PATCH 11/12] s390/sclp: expose the GISA format facility

for handling these patches via the KVM tree.


>>
>>
>> s390/css: indicate the availability of the AIV facility
>>   
>> The patch adds an indication for the presence Adapter Interruption
>> Virtualization facility (AIV) of the general channel subsystem
>> characteristics.
>>
>> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx>
>> Reviewed-by: Halil Pasic <pasic@xxxxxxxxxxxxxxxxxx>
>> Reviewed-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
>> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
>> [change wording]
>>
> 
> Also works for me.
> 
> Acked-by: Cornelia Huck <cohuck@xxxxxxxxxx>
Thanks.




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux