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

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

 




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.
What about


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]

--
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