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 12:36 PM, Christophe Fergeau
<cfergeau@xxxxxxxxxx> wrote:
> On Wed, Nov 19, 2014 at 12:08:21PM +0000, Zeeshan Ali (Khattak) wrote:
>> On Wed, Nov 19, 2014 at 11:00 AM, Christophe Fergeau
>> <cfergeau@xxxxxxxxxx> wrote:
>> > 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.
>
> Hmm if it's not guaranteed now, we'll have to make sure it is at some
> point.

Well, I guess its guaranteed if we say that it is since we create list
of devices by parsing xml..

However, there is still the issue of this API not being convenient for
app if it takes a GVirConfigDomainGraphics *. They'll have to get the
GVirConfigDomain pointer and get list of devices and and find the
graphics device themselves while most domains have just one graphics
device typically.

> Or is it possible to find out the index to pass to that function
> with the current API when there are multiple graphics devices?

No, AFAIK.

-- 
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]