Re: [Xen-devel] [PATCH 2/2] docs/hypervisor-guide: Code Coverage

2019-04-04 Thread Julien Grall
Hi, On 04/04/2019 13:46, Andrew Cooper wrote: On 04/04/2019 13:29, Julien Grall wrote: On 04/04/2019 13:07, Artem Mygaiev wrote: Hello Andrew Hi, Minor note below. On Wed, 2019-04-03 at 16:00 +0100, Andrew Cooper wrote: [snip] + For ARM builds, while Xen will compile with ``CONFIG_COV

Re: [Xen-devel] [PATCH 2/2] docs/hypervisor-guide: Code Coverage

2019-04-04 Thread Andrew Cooper
On 04/04/2019 13:29, Julien Grall wrote: > > On 04/04/2019 13:07, Artem Mygaiev wrote: >> Hello Andrew > Hi, > >> Minor note below. >> >> On Wed, 2019-04-03 at 16:00 +0100, Andrew Cooper wrote: >> >> [snip] >> >>> + For ARM builds, while Xen will compile with ``CONFIG_COVERAGE`` >>> enabled, the >

Re: [Xen-devel] [PATCH 2/2] docs/hypervisor-guide: Code Coverage

2019-04-04 Thread Julien Grall
On 04/04/2019 13:07, Artem Mygaiev wrote: > Hello Andrew Hi, > Minor note below. > > On Wed, 2019-04-03 at 16:00 +0100, Andrew Cooper wrote: > > [snip] > >> + For ARM builds, while Xen will compile with ``CONFIG_COVERAGE`` >> enabled, the >> + resulting binary will not successfully boot if

Re: [Xen-devel] [PATCH 2/2] docs/hypervisor-guide: Code Coverage

2019-04-04 Thread Wei Liu
On Wed, Apr 03, 2019 at 04:00:00PM +0100, Andrew Cooper wrote: > During a discussion in person, it was identified that Coverage doesn't > currently work for ARM yet. Also, there are a number of errors with the > existing coverage document. > > Take the opportunity to rewrite it in RST, making it

Re: [Xen-devel] [PATCH 2/2] docs/hypervisor-guide: Code Coverage

2019-04-04 Thread Artem Mygaiev
Hello Andrew Minor note below. On Wed, 2019-04-03 at 16:00 +0100, Andrew Cooper wrote: [snip] > + For ARM builds, while Xen will compile with ``CONFIG_COVERAGE`` > enabled, the > + resulting binary will not successfully boot if it exceeds 2MB in > size. > + Xen's early memory management code