Re: [PATCH v5 23/44] x86/boot: remove module_map usage from xsm policy loading

["Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>]

On 10/8/24 12:36, Jason Andryuk wrote:
> On 2024-10-06 17:49, Daniel P. Smith wrote:
> > Signed-off-by: Daniel P. Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>
> > ---
> >   xen/arch/x86/setup.c  | 2 +-
> >   xen/include/xsm/xsm.h | 9 +++------
> >   xen/xsm/xsm_core.c    | 6 ++----
> >   xen/xsm/xsm_policy.c  | 5 +----
> >   4 files changed, 7 insertions(+), 15 deletions(-)
> > diff --git a/xen/xsm/xsm_policy.c b/xen/xsm/xsm_policy.c
> > index 921bb254b9d1..a22367a62e93 100644
> > --- a/xen/xsm/xsm_policy.c
> > +++ b/xen/xsm/xsm_policy.c
> > @@ -42,7 +41,6 @@ int __init xsm_multiboot_policy_init(
> >       /*
> >        * Try all modules and see whichever could be the binary policy.
> > -     * Adjust module_map for the module that is the binary policy.
> >        */
> You can collapse to a single line comment /* ... */
Ack.

> With that:
> Reviewed-by: Jason Andryuk <jason.andryuk@xxxxxxx>
Thank you.

v/r,
dps