[PATCH v2 05/11] qemu: Introduce virQEMUDriverGetEmbedRoot

Michal Prívozník mprivozn at redhat.com
Mon Mar 30 16:20:28 UTC 2020


On 30. 3. 2020 13:16, Daniel P. Berrangé wrote:
> On Thu, Mar 26, 2020 at 04:15:09PM +0100, Michal Privoznik wrote:
>> This function returns embeddedRoot member of the driver
>> structure.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> Reviewed-by: Andrea Bolognani <abologna at redhat.com>
>> Reviewed-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
>> ---
>>  src/qemu/qemu_conf.c | 12 ++++++++++++
>>  src/qemu/qemu_conf.h |  1 +
>>  2 files changed, 13 insertions(+)
>>
>> diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
>> index 9786e19f8f..fdc6f53ad3 100644
>> --- a/src/qemu/qemu_conf.c
>> +++ b/src/qemu/qemu_conf.c
>> @@ -1230,6 +1230,18 @@ virQEMUDriverIsPrivileged(virQEMUDriverPtr driver)
>>      return driver->privileged;
>>  }
>>  
>> +/* virQEMUDriverGetEmbedRoot:
>> + * @driver: the QEMU driver
>> + *
>> + * Returns root directory specified in connection URI for embed
>> + * mode, NULL otherwise.
>> + */
>> +const char *
>> +virQEMUDriverGetEmbedRoot(virQEMUDriverPtr driver)
>> +{
>> +    return driver->embeddedRoot;
>> +}
> 
> I don't really see the benefit in this method. The embeddedRoot
> field is immutable so we can just be accessing it directly with
> no need for APIs wrappers, as we often do when accesing the
> privileged field.

I just wanted to follow what virQEMUDriverIsPrivileged() is doing. I
don't care honestly.

Michal




More information about the libvir-list mailing list