| Message ID | 1476969615-4421-2-git-send-email-deathsimple@vodafone.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Oct 20, 2016 at 9:20 AM, Christian König <deathsimple@vodafone.de> wrote: > From: Christian König <christian.koenig@amd.com> > > Kernel functions taking a timeout usually return 1 on success even > when they get a zero timeout. > > Signen-off-by: Christian König <christian.koenig@amd.com> > Reviewed-by: Chunming Zhou <david1.zhou@amd.com> Adding Sumit. This series is: Reviewed-by: Alex Deucher <alexander.deucher@amd.com> > --- > drivers/dma-buf/fence.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/dma-buf/fence.c b/drivers/dma-buf/fence.c > index 4d51f9e..fb915ab 100644 > --- a/drivers/dma-buf/fence.c > +++ b/drivers/dma-buf/fence.c > @@ -335,18 +335,20 @@ fence_default_wait_cb(struct fence *fence, struct fence_cb *cb) > * @timeout: [in] timeout value in jiffies, or MAX_SCHEDULE_TIMEOUT > * > * Returns -ERESTARTSYS if interrupted, 0 if the wait timed out, or the > - * remaining timeout in jiffies on success. > + * remaining timeout in jiffies on success. If timeout is zero the value one is > + * returned if the fence is already signaled for consistency with other > + * functions taking a jiffies timeout. > */ > signed long > fence_default_wait(struct fence *fence, bool intr, signed long timeout) > { > struct default_wait_cb cb; > unsigned long flags; > - signed long ret = timeout; > + signed long ret = timeout ? timeout : 1; > bool was_set; > > if (test_bit(FENCE_FLAG_SIGNALED_BIT, &fence->flags)) > - return timeout; > + return ret; > > spin_lock_irqsave(fence->lock, flags); > > -- > 2.5.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Christian, 2016-10-20 Christian König <deathsimple@vodafone.de>: > From: Christian König <christian.koenig@amd.com> > > Kernel functions taking a timeout usually return 1 on success even > when they get a zero timeout. > > Signen-off-by: Christian König <christian.koenig@amd.com> > Reviewed-by: Chunming Zhou <david1.zhou@amd.com> > --- > drivers/dma-buf/fence.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> Gustavo
diff --git a/drivers/dma-buf/fence.c b/drivers/dma-buf/fence.c index 4d51f9e..fb915ab 100644 --- a/drivers/dma-buf/fence.c +++ b/drivers/dma-buf/fence.c @@ -335,18 +335,20 @@ fence_default_wait_cb(struct fence *fence, struct fence_cb *cb) * @timeout: [in] timeout value in jiffies, or MAX_SCHEDULE_TIMEOUT * * Returns -ERESTARTSYS if interrupted, 0 if the wait timed out, or the - * remaining timeout in jiffies on success. + * remaining timeout in jiffies on success. If timeout is zero the value one is + * returned if the fence is already signaled for consistency with other + * functions taking a jiffies timeout. */ signed long fence_default_wait(struct fence *fence, bool intr, signed long timeout) { struct default_wait_cb cb; unsigned long flags; - signed long ret = timeout; + signed long ret = timeout ? timeout : 1; bool was_set; if (test_bit(FENCE_FLAG_SIGNALED_BIT, &fence->flags)) - return timeout; + return ret; spin_lock_irqsave(fence->lock, flags);