Re: [Xen-devel] [PATCH v5 09/14] hvmloader: Check modules whereabouts in perform_tests

2016-06-27 Thread Jan Beulich
>>> On 24.06.16 at 19:14, wrote: > On Fri, Jun 24, 2016 at 01:44:30AM -0600, Jan Beulich wrote: >> >>> On 22.06.16 at 19:15, wrote: >> > +uint64_t cmdline_paddr = modlist[i].cmdline_paddr; >> > + >> > +if ( check_test_overlap(modlist[i].paddr, modlist[i].size) ) >> > +{ >>

Re: [Xen-devel] [PATCH v5 09/14] hvmloader: Check modules whereabouts in perform_tests

2016-06-24 Thread Anthony PERARD
On Fri, Jun 24, 2016 at 01:44:30AM -0600, Jan Beulich wrote: > >>> On 22.06.16 at 19:15, wrote: > > +uint64_t cmdline_paddr = modlist[i].cmdline_paddr; > > + > > +if ( check_test_overlap(modlist[i].paddr, modlist[i].size) ) > > +{ > > +printf("Skipping tests due

Re: [Xen-devel] [PATCH v5 09/14] hvmloader: Check modules whereabouts in perform_tests

2016-06-24 Thread Jan Beulich
>>> On 22.06.16 at 19:15, wrote: > --- a/tools/firmware/hvmloader/tests.c > +++ b/tools/firmware/hvmloader/tests.c > @@ -20,6 +20,7 @@ > */ > > #include "util.h" > +#include "config.h" > > #define TEST_FAIL 0 > #define TEST_PASS 1 > @@ -189,6 +190,15 @@ static int shadow_gs_test(void) >

[Xen-devel] [PATCH v5 09/14] hvmloader: Check modules whereabouts in perform_tests

2016-06-22 Thread Anthony PERARD
As perform_tests() is going to clear memory past 4MB, we check that the memory can be use or we skip the tests. Signed-off-by: Anthony PERARD --- Changes in V5: - also account for the pages table - fix coding style - also check modules cmdline and main cmdline and modlist_paddr - make use of ch