| Message ID | 20220526081230.3194-1-luca.fancellu@arm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | tools/libxl: optimize domain creation skipping domain cpupool move | expand |
On 26.05.22 10:12, Luca Fancellu wrote: > Commit 92ea9c54fc81 ("arm/dom0less: assign dom0less guests to cpupools") > introduced a way to start a domain directly on a certain cpupool, > adding a "cpupool_id" member to struct xen_domctl_createdomain. > > This was done to be able to start dom0less guests in different pools than > cpupool0, but the toolstack can benefit from it because it can now use > the struct member directly instead of creating the guest in cpupool0 > and then moving it to the target cpupool. > > Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Luca Fancellu <luca.fancellu@arm.com> Reviewed-by: Juergen Gross <jgross@suse.com> Juergen
On Thu, May 26, 2022 at 09:12:30AM +0100, Luca Fancellu wrote: > Commit 92ea9c54fc81 ("arm/dom0less: assign dom0less guests to cpupools") > introduced a way to start a domain directly on a certain cpupool, > adding a "cpupool_id" member to struct xen_domctl_createdomain. > > This was done to be able to start dom0less guests in different pools than > cpupool0, but the toolstack can benefit from it because it can now use > the struct member directly instead of creating the guest in cpupool0 > and then moving it to the target cpupool. > > Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Luca Fancellu <luca.fancellu@arm.com> Acked-by: Anthony PERARD <anthony.perard@citrix.com> Thanks,
diff --git a/tools/libs/light/libxl_create.c b/tools/libs/light/libxl_create.c index 69ec405858a8..2339f09e95a6 100644 --- a/tools/libs/light/libxl_create.c +++ b/tools/libs/light/libxl_create.c @@ -633,6 +633,7 @@ int libxl__domain_make(libxl__gc *gc, libxl_domain_config *d_config, .max_maptrack_frames = b_info->max_maptrack_frames, .grant_opts = XEN_DOMCTL_GRANT_version(b_info->max_grant_version), .vmtrace_size = ROUNDUP(b_info->vmtrace_buf_kb << 10, XC_PAGE_SHIFT), + .cpupool_id = info->poolid, }; if (info->type != LIBXL_DOMAIN_TYPE_PV) { @@ -757,13 +758,6 @@ int libxl__domain_make(libxl__gc *gc, libxl_domain_config *d_config, */ assert(libxl_domid_valid_guest(*domid)); - ret = xc_cpupool_movedomain(ctx->xch, info->poolid, *domid); - if (ret < 0) { - LOGED(ERROR, *domid, "domain move fail"); - rc = ERROR_FAIL; - goto out; - } - dom_path = libxl__xs_get_dompath(gc, *domid); if (!dom_path) { rc = ERROR_FAIL;
Commit 92ea9c54fc81 ("arm/dom0less: assign dom0less guests to cpupools") introduced a way to start a domain directly on a certain cpupool, adding a "cpupool_id" member to struct xen_domctl_createdomain. This was done to be able to start dom0less guests in different pools than cpupool0, but the toolstack can benefit from it because it can now use the struct member directly instead of creating the guest in cpupool0 and then moving it to the target cpupool. Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> Signed-off-by: Luca Fancellu <luca.fancellu@arm.com> --- tools/libs/light/libxl_create.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-)