Re: [PATCH V7 12/18] x86/pks: Add PKS fault callbacks

From: Ira Weiny
Date: Mon Aug 16 2021 - 23:21:32 EST


On Wed, Aug 11, 2021 at 02:18:26PM -0700, Edgecombe, Rick P wrote:
> On Tue, 2021-08-03 at 21:32 -0700, ira.weiny@xxxxxxxxx wrote:
> > +static const pks_key_callback
> > pks_key_callbacks[PKS_KEY_NR_CONSUMERS] = { 0 };
> > +
> > +bool handle_pks_key_callback(unsigned long address, bool write, u16
> > key)
> > +{
> > + if (key > PKS_KEY_NR_CONSUMERS)
> > + return false;
> Good idea, should be >= though?

Yep. Fixed thanks.

>
> > +
> > + if (pks_key_callbacks[key])
> > + return pks_key_callbacks[key](address, write);
> > +
> > + return false;
> > +}
> > +
>
> Otherwise, I've rebased on this series and didn't hit any problems.
> Thanks.

Awesome! I still want Dave and Dan to weigh in prior to me respining with the
changes so far.

Thanks,
Ira