Re: Spice bug62033, Gnome bug 680195 rework: new inhibitors for desktop effects

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

 



Hi all,

I've sent a preview patch of this work. Please confirm
1) the names of the bus, interface, object path
2) the interface itself (currently 3 read-only properties: Connected,
Type and Speed)
3) I'm not sure whether lazy generation of GDBusNodeInfo from
introspection XML (as static char *) is acceptable - or should I use
gdbus-codegen?

My next steps include:
1. Implement export of actual properties, i.e. connection status,
connection type (local/remote) and speed (in kbps). The idea is that
clients are expected to use Speed to judge upon disabling some
features like animations.
2. Implement signal on properties changed
3. Other TODOs (GIO 2.32 thread API change, error handling, doc & comments etc.)


On Mon, Dec 16, 2013 at 2:12 PM, Marc-André Lureau <mlureau@xxxxxxxxxx> wrote:
> Hi
>
> ----- Original Message -----
>> Hi everyone,
>>
>> As we've decided a while ago (ITT), I'm looking into implementing a
>> DBus interface for spice-vdagent (session-level) using GDbus API from
>> GIO, and have run into following shortcomings:
>>
>> 1. vdagent is single-threaded and blocks on select() for udscs I/O
>> 2. vdagent isn't GLib-based
>>
>> A simplest solution I'm going to try first:
>> 1. Add GLib main loop in the vdagent main thread
>> 2. Create a separate GThread for all current I/O
>>
>> Will such change be acceptable?
>>
>> Next steps I can think of:
>> 1. Use GSource to attach current udscs I/O to GMainLoop
>> 2. Move most of current udscs messages to DBus - probably need to keep
>> using udscs for xfer for performance reasons.
>>
>> What do you think?
>
> Using GLib for main loop would be a good step forward.
>
> I am not sure using a separate thread for current IO is needed, but it might make it easier for a first step?
>
> thanks!



-- 
Best regards,
Fedor
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]