search for: out_next

...g or NULL when done */ - SwfdecBuffer * in; /* next input buffer or NULL */ - GstBuffer * out; /* available output or NULL */ - int width; /* width of last output buffer */ - int height; /* height of last output buffer */ - GstCaps * srccaps; /* caps to set on buffers */ - gboolean out_next; /* wether the pipeline expects input or output */ - gboolean error; /* we're in an error state */ + gboolean error; + SwfdecGstDecoder dec; /* the decoder element */ + GstBuffer * last; /* last decoded buffer */ }; static void -swfdec_gst_video_unref (gpointer data, GObject *unus...

...nouveau_gem.c b/drivers/gpu/drm/nouveau/nouveau_gem.c index 6454370..d19c7aa 100644 --- a/drivers/gpu/drm/nouveau/nouveau_gem.c +++ b/drivers/gpu/drm/nouveau/nouveau_gem.c @@ -662,7 +662,7 @@ nouveau_gem_ioctl_pushbuf(struct drm_device *dev, void *data, if (unlikely(req->nr_push == 0)) goto out_next; - if (unlikely(req->nr_push > NOUVEAU_GEM_MAX_PUSH)) { + if (unlikely(req->nr_push >= NOUVEAU_GEM_MAX_PUSH)) { NV_ERROR(drm, "pushbuf push count exceeds limit: %d max %d\n", req->nr_push, NOUVEAU_GEM_MAX_PUSH); return nouveau_abi16_put(abi16, -EINVAL); @@ -718...

...ya(req->buffers, req->nr_buffers, sizeof(*bo)); if (IS_ERR(bo)) { - kfree(push); + u_free(push); return nouveau_abi16_put(abi16, PTR_ERR(bo)); } @@ -854,8 +870,8 @@ out: nouveau_fence_unref(&fence); out_prevalid: - kfree(bo); - kfree(push); + u_free(bo); + u_free(push); out_next: if (chan->dma.ib_max) { -- 1.8.3.4

...signal when stuff below changes */ volatile int refcount; /* refcount (d'oh) */ @@ -48,6 +50,8 @@ struct _SwfdecGstVideo { int width; /* width of last output buffer */ int height; /* height of last output buffer */ GstCaps * srccaps; /* caps to set on buffers */ + gboolean out_next; /* wether the pipeline expects input or output */ + gboolean error; /* we're in an error state */ }; static void @@ -68,9 +72,9 @@ swfdec_gst_video_unref (gpointer data, G } static void -swfdec_codec_gst_video_finish (gpointer codec_data) +swfdec_video_decoder_gst_free (SwfdecVideoD...

...gt; return nouveau_abi16_put(abi16, PTR_ERR(bo)); > } > > @@ -854,8 +870,8 @@ out: > nouveau_fence_unref(&fence); > > out_prevalid: > - kfree(bo); > - kfree(push); > + u_free(bo); > + u_free(push); > > out_next: > if (chan->dma.ib_max) { > -- > 1.8.3.4 > > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel

From: Thierry Reding <treding at nvidia.com> Hi, This series implements a new IOCTL to submit push buffers that can optionally return a sync FD or sync object to userspace. This is useful in cases where userspace wants to synchronize operations between the GPU and another driver (such as KMS for display). Among other things this allows extensions such as eglDupNativeFenceFDANDROID to be

Reduces object size ~4kb Joe Perches (2): drm/nouveau: Add new logging function nv_cli_printk drm/nouveau: Convert NV_PRINTK macros and uses to new nv_cli_<level> macros drivers/gpu/drm/nouveau/nouveau_abi16.c | 2 +- drivers/gpu/drm/nouveau/nouveau_chan.c | 12 +++---- drivers/gpu/drm/nouveau/nouveau_drm.c | 21 +++++++++++ drivers/gpu/drm/nouveau/nouveau_drv.h | 44

...== 0) { @@ -225,6 +225,9 @@ nouveau_gem_ioctl_new(struct drm_device *dev, void *data, /* drop reference from allocate - handle holds it now */ drm_gem_object_unreference_unlocked(nvbo->gem); +out: + if (ret == -EIO) + ret = nouveau_reset_device(dev); return ret; } @@ -804,6 +807,9 @@ out_next: } nouveau_channel_put(&chan); + + if (ret == -EIO) + ret = nouveau_reset_device(dev); return ret; } @@ -839,6 +845,9 @@ nouveau_gem_ioctl_cpu_prep(struct drm_device *dev, void *data, ret = ttm_bo_wait(&nvbo->bo, true, true, no_wait); spin_unlock(&nvbo->bo.bdev-&gt...

...== 0) { @@ -225,6 +225,9 @@ nouveau_gem_ioctl_new(struct drm_device *dev, void *data, /* drop reference from allocate - handle holds it now */ drm_gem_object_unreference_unlocked(nvbo->gem); +out: + if (ret == -EIO) + ret = nouveau_reset_device(dev); return ret; } @@ -804,6 +807,9 @@ out_next: } nouveau_channel_put(&chan); + + if (ret == -EIO) + ret = nouveau_reset_device(dev); return ret; } @@ -839,6 +845,9 @@ nouveau_gem_ioctl_cpu_prep(struct drm_device *dev, void *data, ret = ttm_bo_wait(&nvbo->bo, true, true, no_wait); spin_unlock(&nvbo->bo.bdev-&gt...

From: Maarten Lankhorst <maarten.lankhorst at canonical.com> Commit de7b7d59d54852c introduced tiled GART, but a linear copy is still performed. This may result in errors on eviction, fix it by checking tiling from memtype. Signed-off-by: Maarten Lankhorst <maarten.lankhorst at canonical.com> Cc: stable at vger.kernel.org #3.10+ --- drivers/gpu/drm/nouveau/nouveau_bo.c | 33

Hi there, Resending these now that they've had some more polish and testing, and I heard that Ben's vacation is over :-) These patches work as a starting point for more explicit error mechanisms and better robustness. At the moment, when a job hangs or faults, it seems that nouveau doesn't quite know how to handle the situation and often results in a hang. Some of these situations

Hi guys, I'd like to start a new thread about explicit fence synchronization. This time with a Nouveau twist. :-) First, let me define what I understand by implicit/explicit sync: Implicit synchronization * Fences are attached to buffers * Kernel manages fences automatically based on buffer read/write access Explicit synchronization * Fences are passed around independently * Kernel takes