Re: [libvirt-glib] Add gvir_domain_open_graphics_fd()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 19, 2014 at 11:00 AM, Christophe Fergeau
<cfergeau@xxxxxxxxxx> wrote:
> On Wed, Nov 19, 2014 at 01:19:31AM +0000, Zeeshan Ali (Khattak) wrote:
>> Add binding for virDomainOpenGraphicsFD.
>> ---
>>  libvirt-gobject/libvirt-gobject-domain.c | 36 ++++++++++++++++++++++++++++++++
>>  libvirt-gobject/libvirt-gobject-domain.h |  4 ++++
>>  libvirt-gobject/libvirt-gobject.sym      |  5 +++++
>>  3 files changed, 45 insertions(+)
>>
>> diff --git a/libvirt-gobject/libvirt-gobject-domain.c b/libvirt-gobject/libvirt-gobject-domain.c
>> index 8df30d7..9c1aa6e 100644
>> --- a/libvirt-gobject/libvirt-gobject-domain.c
>> +++ b/libvirt-gobject/libvirt-gobject-domain.c
>> @@ -1222,6 +1222,42 @@ cleanup:
>>  }
>>
>>  /**
>> + * gvir_domain_open_graphics_fd:
>> + * @dom: the domain
>> + * @idx: the graphics index
>> + * @flags: extra flags, currently unused
>> + *
>> + * This will create a socket pair connected to the graphics backend of @dom. One
>> + * end of the socket will be returned on success, and the other end is handed to
>> + * the hypervisor. If @dom has multiple graphics backends configured, then @idx
>> + * will determine which one is opened, starting from @idx 0.
>> + *
>> + * Returns: An fd on success, -1 on failure.
>> + */
>> +int gvir_domain_open_graphics_fd(GVirDomain *dom,
>> +                                 guint idx,
>
> Could/should this take a GVirConfigDomainGraphics * instead and infer
> the index from it?
> I know gvir_domain-open_graphics() uses 'idx', but maybe we should offer
> the 2 variants?

Doesn't sound very convenient and also I don't think the API
guarantees the order of devices returned from
gvir_config_domain_get_devices.

-- 
Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]