| Message ID | 20200722150416.36426-1-roger.pau@citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [osstest] dom0pvh: assign 1GB of memory to PVH dom0 | expand |
Roger Pau Monne writes ("[osstest PATCH] dom0pvh: assign 1GB of memory to PVH dom0"):
> Current tests use 512MB of memory for dom0, but that's too low for a
> PVH dom0 on some hosts and will cause errors because memory is
> ballooned out in order to obtain physical memory ranges to map foreign
> pages.
>
> Using ballooned out pages for foreign mappings also doesn't seem to
> work properly with the current Linux kernel version, so increase the
> memory assigned to dom0 to 1GB for PVH dom0 tests. We should see about
> reverting this when using ballooned pages is fixed.
>
> The runvar diff is:
>
> +test-amd64-amd64-dom0pvh-xl-amd dom0_mem 1024
> +test-amd64-amd64-dom0pvh-xl-intel dom0_mem 1024
>
> I've done a repro of the failed test on elbling0 with dom0_mem set to
> 1GB and it seems to prevent the issue, the flight is 152111.
>
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
And queued.
Ian.
On Wed, Jul 22, 2020 at 04:11:21PM +0100, Ian Jackson wrote: > Roger Pau Monne writes ("[osstest PATCH] dom0pvh: assign 1GB of memory to PVH dom0"): > > Current tests use 512MB of memory for dom0, but that's too low for a > > PVH dom0 on some hosts and will cause errors because memory is > > ballooned out in order to obtain physical memory ranges to map foreign > > pages. > > > > Using ballooned out pages for foreign mappings also doesn't seem to > > work properly with the current Linux kernel version, so increase the > > memory assigned to dom0 to 1GB for PVH dom0 tests. We should see about > > reverting this when using ballooned pages is fixed. > > > > The runvar diff is: > > > > +test-amd64-amd64-dom0pvh-xl-amd dom0_mem 1024 > > +test-amd64-amd64-dom0pvh-xl-intel dom0_mem 1024 > > > > I've done a repro of the failed test on elbling0 with dom0_mem set to > > 1GB and it seems to prevent the issue, the flight is 152111. > > > > Signed-off-by: Roger Pau Monné <roger.pau@citrix.com> > > Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> > > And queued. Thanks! Forgot to add that I've checked x86 hosts and they all have at least 8GB of RAM, so using 1GB for dom0 should be fine, as I don't think osstest runs guests close to 7GB of RAM. Roger.
diff --git a/make-flight b/make-flight index b8942c1c..85559c68 100755 --- a/make-flight +++ b/make-flight @@ -903,7 +903,7 @@ test_matrix_do_one () { job_create_test test-$xenarch$kern-$dom0arch-dom0pvh-xl-$cpuvendor \ test-debian xl $xenarch $dom0arch $debian_runvars \ all_hostflags=$most_hostflags,hvm-$cpuvendor,iommu \ - xen_boot_append='dom0=pvh,verbose' + xen_boot_append='dom0=pvh,verbose' dom0_mem=1024 done
Current tests use 512MB of memory for dom0, but that's too low for a PVH dom0 on some hosts and will cause errors because memory is ballooned out in order to obtain physical memory ranges to map foreign pages. Using ballooned out pages for foreign mappings also doesn't seem to work properly with the current Linux kernel version, so increase the memory assigned to dom0 to 1GB for PVH dom0 tests. We should see about reverting this when using ballooned pages is fixed. The runvar diff is: +test-amd64-amd64-dom0pvh-xl-amd dom0_mem 1024 +test-amd64-amd64-dom0pvh-xl-intel dom0_mem 1024 I've done a repro of the failed test on elbling0 with dom0_mem set to 1GB and it seems to prevent the issue, the flight is 152111. Signed-off-by: Roger Pau Monné <roger.pau@citrix.com> --- make-flight | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)