Re: [PATCH 02/21] include/xen/slr_table.h: Secure Launch Resource Table definitions

[ross.philipson@xxxxxxxxxx]

On 4/23/25 7:47 AM, Sergii Dmytruk wrote:
> On Tue, Apr 22, 2025 at 01:46:14PM -0700, ross.philipson@xxxxxxxxxx wrote:
> > > +
> > > +/*
> > > + * ARM DRTM Info table
> > > + */
> > > +struct slr_entry_arm_info
> > > +{
> > > +    struct slr_entry_hdr hdr;
> > > +} __packed;
> > You can probably ditch this for now.
> Right, it has no value at this point.
>
> > I am surprised you did not need the slr_add_entry() function. How do you add
> > entries to the SLRT?
> Xen doesn't add any SLRT entries.  It's also the final consumer of the
> SLRT, at least at the moment, so no need to update something that won't
> be used again.
Ahh right. The Linux version allows the policy to be updated by the EFI 
stub but you are not doing that.
Thanks
Ross

> > Thanks
> > Ross