[libvirt] PATCH: Switch openvz driver to domain APIs

Evgeniy Sokolov evg at openvz.org
Thu Aug 28 08:06:08 UTC 2008


 > The biggest flaw I see currently is that the
> openvz driver doesn't load the existing device config for networks or
> filesystems of existing VMs, so you can't see that info in the XML dump
Yes. Good note. I will implement it.
> 

> -        /*get VCPU*/
> -        ret = openvzReadConfigParam(veid, "CPUS", temp, sizeof(temp));
> -        if (ret < 0) {
> -             openvzError(conn, VIR_ERR_INTERNAL_ERROR,
> -                            _("Cound not read config for container %d"), veid);
> -             goto error;
> -        } else if (ret > 0) {
> -             vmdef->vcpus = strtoI(temp);
> -        }
Why did you remove loading VCPUs and always set = 1 ?
> + 
> +        dom->def->vcpus = 1;





More information about the libvir-list mailing list