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

[Cornelia Huck <cohuck@redhat.com>]

On Thu, 25 Jan 2018 12:42:28 +0100
Christian Borntraeger <borntraeger@de.ibm.com> wrote:

> On 01/18/2018 06:54 PM, Cornelia Huck wrote:
> > On Thu, 18 Jan 2018 13:02:45 +0100
> > Michael Mueller <mimu@linux.vnet.ibm.com> wrote:
> >   
> >> On 17.01.18 16:19, David Hildenbrand wrote:  
> >>> On 16.01.2018 21:02, Christian Borntraeger wrote:    
> >>>> From: Michael Mueller <mimu@linux.vnet.ibm.com>
> >>>>
> >>>> The patch exposes the Adapter Interruption Virtualization facility (AIV)
> >>>> of the general channel subsystem characteristics.
> >>>>
> >>>> Signed-off-by: Michael Mueller <mimu@linux.vnet.ibm.com>
> >>>> Reviewed-by: Halil Pasic <pasic@linux.vnet.ibm.com>
> >>>> Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
> >>>> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> >>>> ---
> >>>>   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.

> 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@linux.vnet.ibm.com>
> Reviewed-by: Halil Pasic <pasic@linux.vnet.ibm.com>
> Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> [change wording]
> 

Also works for me.

Acked-by: Cornelia Huck <cohuck@redhat.com>