On Wed, 2023-09-13 at 13:15 -0600, Jens Axboe wrote:
> On 9/13/23 12:59 PM, Nathan Chancellor wrote:
> > Hi Greg,
> >
> > On Fri, Sep 08, 2023 at 10:30:56AM -0500, gjoyce@xxxxxxxxxxxxxxxxxx
> > wrote:
> > > From: Greg Joyce <gjoyce@xxxxxxxxxxxxxxxxxx>
> > >
> > > Define operations for SED Opal to read/write keys
> > > from POWER LPAR Platform KeyStore(PLPKS). This allows
> > > non-volatile storage of SED Opal keys.
> > >
> > > Signed-off-by: Greg Joyce <gjoyce@xxxxxxxxxxxxxxxxxx>
> > > Reviewed-by: Jonathan Derrick <jonathan.derrick@xxxxxxxxx>
> > > Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
> >
> > After this change in -next as commit 9f2c7411ada9
> > ("powerpc/pseries:
> > PLPKS SED Opal keystore support"), I see the following crash when
> > booting some distribution configurations, such as OpenSUSE's [1]
> > (the
> > rootfs is available at [2] if necessary):
>
> I'll drop the series for now - I didn't push out the main branch just
> yet as I don't publish the block next tree until at least at -rc2
> time,
> so it's just in a private branch for now.
>
Agreed. I need to figure out:
1) best place to use plpks_is_available() to prevent a crash when PLPKS
is not present in pseries.
2) Resolve issues compiling with clang
3) declare plpks_init_var() as static
Greg
