Re: [Spice-devel] [PATCH spice-streaming-agent 5/5] Provide helper to give information to daemon

2018-02-02 Thread Frediano Ziglio
> > On Thu, 2018-02-01 at 16:40 +, Frediano Ziglio wrote: > > This is required so the daemon can manage the various sessions. > > At first glance, this looks like a really big hack. At second glance, I > understand why you did it... (doesn't mean I like it now ;)) But in any > case this deser

Re: [Spice-devel] [PATCH spice-streaming-agent 5/5] Provide helper to give information to daemon

2018-02-02 Thread Lukáš Hrázký
On Thu, 2018-02-01 at 16:40 +, Frediano Ziglio wrote: > This is required so the daemon can manage the various sessions. At first glance, this looks like a really big hack. At second glance, I understand why you did it... (doesn't mean I like it now ;)) But in any case this deserves a much bett

Re: [Spice-devel] [PATCH spice-streaming-agent 4/5] Implement a daemon/agent separation

2018-02-02 Thread Lukáš Hrázký
On Thu, 2018-02-01 at 16:40 +, Frediano Ziglio wrote: > This allows to manage properly multiple servers (currently only Xorg). > The executable will run as a service forking the proper agent. > The agent will then manage the active server. > The server receive just minimal information for the v

Re: [Spice-devel] [PATCH spice-streaming-agent 3/5] Starts/stops the agent based on VT status

2018-02-02 Thread Lukáš Hrázký
On Thu, 2018-02-01 at 16:40 +, Frediano Ziglio wrote: > Check if the current server is active or not and stream only if active. > This will allow to support multiple servers running. > When multiple servers are running only one have the GPU associated > and is writing to the screen. > Stop capt

Re: [Spice-devel] [PATCH spice-streaming-agent 2/5] Add file with utilities for Xorg

2018-02-02 Thread Lukáš Hrázký
On Thu, 2018-02-01 at 16:40 +, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio > --- > src/Makefile.am | 2 ++ > src/spice-streaming-agent.cpp | 1 + > src/xorg-utils.cpp| 33 + > src/xorg-utils.hpp| 8 >