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. Wonder if this can be squashed with another patch? Or rename to something like "define" ... -- Thanks, David / dhildenb