Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Andrew Cooper
On 03/03/16 16:00, Anthony PERARD wrote: > On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: > On 25.02.16 at 15:56, wrote: >>> --- a/tools/firmware/hvmloader/hvmloader.c >>> +++ b/tools/firmware/hvmloader/hvmloader.c >>> @@ -303,6 +303,15 @@ int main(void) >>> >>> smp_initia

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 17:00, wrote: > On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: >> >>> On 25.02.16 at 15:56, wrote: >> > --- a/tools/firmware/hvmloader/hvmloader.c >> > +++ b/tools/firmware/hvmloader/hvmloader.c >> > @@ -303,6 +303,15 @@ int main(void) >> > >> > smp_initial

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Anthony PERARD
On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: > >>> On 25.02.16 at 15:56, wrote: > > --- a/tools/firmware/hvmloader/hvmloader.c > > +++ b/tools/firmware/hvmloader/hvmloader.c > > @@ -303,6 +303,15 @@ int main(void) > > > > smp_initialise(); > > > > +/* Check that tests

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-02-29 Thread Jan Beulich
>>> On 25.02.16 at 15:56, wrote: > --- a/tools/firmware/hvmloader/hvmloader.c > +++ b/tools/firmware/hvmloader/hvmloader.c > @@ -303,6 +303,15 @@ int main(void) > > smp_initialise(); > > +/* Check that tests does not use memory where modules are stored */ > +BUG_ON( ((uint32_t)hvm