Re: [Spice-devel] Xorg indefinitely hangs in kernelspace

2019-09-08 Thread Gerd Hoffmann
Hi, --verbose please. Do you see the same hang? Does the patch fix it? > --- a/drivers/gpu/drm/ttm/ttm_execbuf_util.c > +++ b/drivers/gpu/drm/ttm/ttm_execbuf_util.c > @@ -97,8 +97,9 @@ int ttm_eu_reserve_buffers(struct ww_acq > struct list_head *dups, bool del_lru) [

Re: [Spice-devel] [spice-gtk v1 4/6] session: gst_deinit() GStreamer if we initialize it

2019-09-08 Thread Frediano Ziglio
> > Hi, > > On 9/2/19 7:04 PM, Victor Toso wrote: > > From: Victor Toso > > > > Let's gst_deinit() if we initialize it for the same rationale pointed out > > in 0381e62 "spicy: Add call of gst_deinit at program exit" in > > 2017-10-20 by Christophe de Dinechin > > > > Signed-off-by: Victor Toso

Re: [Spice-devel] [Nouveau] [Intel-gfx] [PATCH v6 08/17] drm/ttm: use gem vma_node

2019-09-08 Thread Ilia Mirkin
On Wed, Aug 21, 2019 at 7:55 AM Thierry Reding wrote: > > On Wed, Aug 21, 2019 at 04:33:58PM +1000, Ben Skeggs wrote: > > On Wed, 14 Aug 2019 at 20:14, Gerd Hoffmann wrote: > > > > > > Hi, > > > > > > > > Changing the order doesn't look hard. Patch attached (untested, have > > > > > no > > >

Re: [Spice-devel] [spice-gtk v1 6/6] RFC: test: session: gstreamer init by us or client

2019-09-08 Thread Snir Sheriber
HI, On 9/2/19 7:04 PM, Victor Toso wrote: From: Victor Toso Does not work properly just because gst_is_initialized() checks if gst_init() or gst_check_init() was ever called but it does not consider if gst_deinit() was called too. I'm sending this RFC to check if should be added or wait till f

Re: [Spice-devel] [spice-gtk v1 4/6] session: gst_deinit() GStreamer if we initialize it

2019-09-08 Thread Snir Sheriber
Hi, On 9/2/19 7:04 PM, Victor Toso wrote: From: Victor Toso Let's gst_deinit() if we initialize it for the same rationale pointed out in 0381e62 "spicy: Add call of gst_deinit at program exit" in 2017-10-20 by Christophe de Dinechin Signed-off-by: Victor Toso --- src/spice-session.c | 8 +

Re: [Spice-devel] [spice-gtk v1 2/6] display-gst: rely on SpiceSession init of GStreamer

2019-09-08 Thread Snir Sheriber
Hi, On 9/2/19 7:04 PM, Victor Toso wrote: From: Victor Toso This means we can drop gstvideo_init() function and replace its calls with gst_is_initialized(). Signed-off-by: Victor Toso --- src/channel-display-gst.c | 23 +-- 1 file changed, 5 insertions(+), 18 deletion