Re: [Spice-devel] [PATCH spice-streaming-agent 8/9] Encapsulate the stream port fd and locking

2018-05-03 Thread Lukáš Hrázký
On Thu, 2018-05-03 at 07:35 -0400, Frediano Ziglio wrote: > > Wrap the streaming virtio port along with the mutex to lock it in a > > class. Pass the class temporarily around to functions that need it until > > the functions too are consolidated into the class. > > > > Signed-off-by: Lukáš Hrázký

Re: [Spice-devel] [PATCH spice-streaming-agent 8/9] Encapsulate the stream port fd and locking

2018-05-03 Thread Frediano Ziglio
> Wrap the streaming virtio port along with the mutex to lock it in a > class. Pass the class temporarily around to functions that need it until > the functions too are consolidated into the class. > > Signed-off-by: Lukáš Hrázký The mutex is supposed to avoid situations like: 1- data header 2-

[Spice-devel] [PATCH spice-streaming-agent 8/9] Encapsulate the stream port fd and locking

2018-04-27 Thread Lukáš Hrázký
Wrap the streaming virtio port along with the mutex to lock it in a class. Pass the class temporarily around to functions that need it until the functions too are consolidated into the class. Signed-off-by: Lukáš Hrázký --- src/spice-streaming-agent.cpp | 104 +---