On 2024-10-06 17:49, Daniel P. Smith wrote:
Signed-off-by: Daniel P. Smith <dpsm...@apertussolutions.com>
---
  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 /* ... */

With that:
Reviewed-by: Jason Andryuk <jason.andr...@amd.com>

      for ( i = bi->nr_modules-1; i >= 1; i-- )
      {

Reply via email to