From: "Christian König" <deathsimple@vodafone.de>
To: Andres Rodriguez <andresx7@gmail.com>, dri-devel@lists.freedesktop.org
Cc: linaro-mm-sig@lists.linaro.org, sumit.semwal@linaro.org,
linux-media@vger.kernel.org
Subject: Re: [PATCH] dma-buf: avoid scheduling on fence status query
Date: Wed, 26 Apr 2017 09:20:31 +0200 [thread overview]
Message-ID: <6a3b44f0-bc9f-462c-9b0f-96ae15712b8b@vodafone.de> (raw)
In-Reply-To: <d555eb6a-e975-b025-6ed0-c458b1c71f34@gmail.com>
NAK, I'm wondering how often I have to reject that change. We should
probably add a comment here.
Even with a zero timeout we still need to enable signaling, otherwise
some fence will never signal if userspace just polls on them.
If a caller is only interested in the fence status without enabling the
signaling it should call dma_fence_is_signaled() instead.
Regards,
Christian.
Am 26.04.2017 um 04:50 schrieb Andres Rodriguez:
> CC a few extra lists I missed.
>
> Regards,
> Andres
>
> On 2017-04-25 09:36 PM, Andres Rodriguez wrote:
>> When a timeout of zero is specified, the caller is only interested in
>> the fence status.
>>
>> In the current implementation, dma_fence_default_wait will always call
>> schedule_timeout() at least once for an unsignaled fence. This adds a
>> significant overhead to a fence status query.
>>
>> Avoid this overhead by returning early if a zero timeout is specified.
>>
>> Signed-off-by: Andres Rodriguez <andresx7@gmail.com>
>> ---
>>
>> This heavily affects the performance of the Source2 engine running on
>> radv.
>>
>> This patch improves dota2(radv) perf on a i7-6700k+RX480 system from
>> 72fps->81fps.
>>
>> drivers/dma-buf/dma-fence.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
>> index 0918d3f..348e9e2 100644
>> --- a/drivers/dma-buf/dma-fence.c
>> +++ b/drivers/dma-buf/dma-fence.c
>> @@ -380,6 +380,9 @@ dma_fence_default_wait(struct dma_fence *fence,
>> bool intr, signed long timeout)
>> if (test_bit(DMA_FENCE_FLAG_SIGNALED_BIT, &fence->flags))
>> return ret;
>>
>> + if (!timeout)
>> + return 0;
>> +
>> spin_lock_irqsave(fence->lock, flags);
>>
>> if (intr && signal_pending(current)) {
>>
next prev parent reply other threads:[~2017-04-26 7:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-26 1:36 [PATCH] dma-buf: avoid scheduling on fence status query Andres Rodriguez
2017-04-26 2:50 ` Andres Rodriguez
2017-04-26 7:20 ` Christian König [this message]
2017-04-26 9:59 ` Dave Airlie
2017-04-26 9:59 ` Dave Airlie
2017-04-26 10:13 ` Christian König
2017-04-26 10:13 ` Christian König
2017-04-26 14:28 ` Andres Rodriguez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6a3b44f0-bc9f-462c-9b0f-96ae15712b8b@vodafone.de \
--to=deathsimple@vodafone.de \
--cc=andresx7@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=linux-media@vger.kernel.org \
--cc=sumit.semwal@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.