Re: [qemu-s390x] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support

[Tony Krowiak]

On 04/18/2018 03:40 AM, Cornelia Huck wrote:
> On Tue, 17 Apr 2018 20:21:31 +0200
> Christian Borntraeger <address@hidden> wrote:
>
> > On 04/16/2018 05:44 PM, David Hildenbrand wrote:
> > >    
> > > >   
> > > > diff --git a/target/s390x/gen-features.c b/target/s390x/gen-features.c
> > > > index 0cdbc15..0d5b0f7 100644
> > > > --- a/target/s390x/gen-features.c
> > > > +++ b/target/s390x/gen-features.c
> > > > @@ -447,6 +447,9 @@ static uint16_t full_GEN12_GA1[] = {
> > > >       S390_FEAT_ADAPTER_INT_SUPPRESSION,
> > > >       S390_FEAT_EDAT_2,
> > > >       S390_FEAT_SIDE_EFFECT_ACCESS_ESOP2,
> > > > +    S390_FEAT_AP_QUERY_CONFIG_INFO,
> > > > +    S390_FEAT_AP_FACILITIES_TEST,
> > > > +    S390_FEAT_AP,
> > > >   };
> > > >     
> > > Now I have to ask a very stupid question:
> > >
> > > I heard that the execution controls in the SIE block for AP are one of
> > > the oldest ones we have around. How can it be that the AP feature cannot
> > > be used before zEC12?
> > It was a suggestion from the crypto team due to testability. Nobody has a z196
> > with the older cards.
> Might be worth adding a note to that respect?

I will.

>