[Xen-changelog] [xen stable-4.7] hvmloader: cater for lack of hvm_start_info

[patchbot@xxxxxxx]

commit e3b4ec6dc77acc13b603497185086cfa96749d19
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Wed Jun 19 09:19:50 2019 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Wed Jun 19 09:19:50 2019 +0200

    hvmloader: cater for lack of hvm_start_info
    
    It wasn't obvious at all that the backport of commit 0d6968635c
    ("hvmloader: avoid tests when they would clobber used memory") has
    further dependencies on changes all having appeared in the 4.8 dev
    cycle. Granted I should have done a build test, yet I routinely do so
    only on the hypervisor side ...
    
    Drop all uses of hvm_start_info again.
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
---
 tools/firmware/hvmloader/tests.c | 61 ----------------------------------------
 1 file changed, 61 deletions(-)

diff --git a/tools/firmware/hvmloader/tests.c b/tools/firmware/hvmloader/tests.c
index 5f7bcc0a43..ed02242ee2 100644
--- a/tools/firmware/hvmloader/tests.c
+++ b/tools/firmware/hvmloader/tests.c
@@ -21,7 +21,6 @@
 
 #include "config.h"
 #include "util.h"
-#include <xen/arch-x86/hvm/start_info.h>
 
 #define TEST_FAIL 0
 #define TEST_PASS 1
@@ -221,66 +220,6 @@ void perform_tests(void)
         printf("Skipping tests due to overlap with base image\n");
         return;
     }
-
-    if ( hvm_start_info->cmdline_paddr &&
-         hvm_start_info->cmdline_paddr < TEST_MEM_BASE + TEST_MEM_SIZE &&
-         ((hvm_start_info->cmdline_paddr +
-           strlen((char *)(uintptr_t)hvm_start_info->cmdline_paddr)) >=
-          TEST_MEM_BASE) )
-    {
-        printf("Skipping tests due to overlap with command line\n");
-        return;
-    }
-
-    if ( hvm_start_info->rsdp_paddr )
-    {
-        printf("Skipping tests due to non-zero RSDP address\n");
-        return;
-    }
-
-    if ( hvm_start_info->nr_modules )
-    {
-        const struct hvm_modlist_entry *modlist =
-            (void *)(uintptr_t)hvm_start_info->modlist_paddr;
-
-        if ( hvm_start_info->modlist_paddr > UINTPTR_MAX ||
-             ((UINTPTR_MAX - (uintptr_t)modlist) / sizeof(*modlist) <
-              hvm_start_info->nr_modules) )
-        {
-            printf("Skipping tests due to inaccessible module list\n");
-            return;
-        }
-
-        if ( TEST_MEM_BASE < (uintptr_t)(modlist +
-                                         hvm_start_info->nr_modules) &&
-             (uintptr_t)modlist < TEST_MEM_BASE + TEST_MEM_SIZE )
-        {
-            printf("Skipping tests due to overlap with module list\n");
-            return;
-        }
-
-        for ( i = 0; i < hvm_start_info->nr_modules; ++i )
-        {
-            if ( TEST_MEM_BASE < modlist[i].paddr + modlist[i].size &&
-                 modlist[i].paddr < TEST_MEM_BASE + TEST_MEM_SIZE )
-            {
-                printf("Skipping tests due to overlap with module %u\n", i);
-                return;
-            }
-
-            if ( modlist[i].cmdline_paddr &&
-                 modlist[i].cmdline_paddr < TEST_MEM_BASE + TEST_MEM_SIZE &&
-                 ((modlist[i].cmdline_paddr +
-                   strlen((char *)(uintptr_t)modlist[i].cmdline_paddr)) >=
-                  TEST_MEM_BASE) )
-            {
-                printf("Skipping tests due to overlap with module %u 
cmdline\n",
-                       i);
-                return;
-            }
-        }
-    }
-
     passed = skipped = 0;
     for ( i = 0; tests[i].test; i++ )
     {
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.7

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog