[PATCH v2 07/11] qemu: use virQEMUDriverGetEmbedRoot() to access embeddedRoot
Daniel P. Berrangé
berrange at redhat.com
Mon Mar 30 11:20:03 UTC 2020
On Thu, Mar 26, 2020 at 04:15:11PM +0100, Michal Privoznik wrote:
> Now that we have shiny accessor for driver->embeddedRoot use it
> instead of accessing the struct member directly.
Is this really better ?
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> src/qemu/qemu_driver.c | 6 +++---
> src/qemu/qemu_process.c | 2 +-
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 5cbf2f2478..664ea4eeda 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -1173,7 +1173,7 @@ static virDrvOpenStatus qemuConnectOpen(virConnectPtr conn,
> return VIR_DRV_OPEN_ERROR;
> }
>
> - if (qemu_driver->embeddedRoot) {
> + if (virQEMUDriverGetEmbedRoot(qemu_driver)) {
> const char *root = virURIGetParam(conn->uri, "root");
> if (!root)
> return VIR_DRV_OPEN_ERROR;
> @@ -1184,11 +1184,11 @@ static virDrvOpenStatus qemuConnectOpen(virConnectPtr conn,
> return VIR_DRV_OPEN_ERROR;
> }
>
> - if (STRNEQ(root, qemu_driver->embeddedRoot)) {
> + if (STRNEQ(root, virQEMUDriverGetEmbedRoot(qemu_driver))) {
> virReportError(VIR_ERR_INTERNAL_ERROR,
> _("Cannot open embedded driver at path '%s', "
> "already open with path '%s'"),
> - root, qemu_driver->embeddedRoot);
> + root, virQEMUDriverGetEmbedRoot(qemu_driver));
> return VIR_DRV_OPEN_ERROR;
> }
> } else {
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index fcb47e8596..b0b2258850 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -6715,7 +6715,7 @@ qemuProcessLaunch(virConnectPtr conn,
> _("Domain autodestroy requires a connection handle"));
> return -1;
> }
> - if (driver->embeddedRoot) {
> + if (virQEMUDriverGetEmbedRoot(driver)) {
> virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
> _("Domain autodestroy not supported for embedded drivers yet"));
> return -1;
> --
> 2.24.1
>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list