dma-buf: performance issue when looking up the fence status
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Zesty |
Fix Released
|
Medium
|
Alberto Milone |
Bug Description
SRU Request:
[Impact]
When a timeout of zero is specified, the caller is only interested in
the fence status.
In the current implementation, dma_fence_
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.
[Test Case]
1) Install the new kernel
2) Download amdgpu-
3) Install Unigine Heaven to get performance score
4) Rmeove DKMS, remove this patch http://
5) re-run the benchmark, and check if there's any obvious performance drop.
[Regression Potential]
Low, the patch is very small, and it is already available upstream.
CVE References
Changed in linux (Ubuntu): | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Alberto Milone (albertomilone) |
Changed in linux (Ubuntu Zesty): | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Alberto Milone (albertomilone) |
Changed in linux (Ubuntu): | |
assignee: | Alberto Milone (albertomilone) → nobody |
status: | In Progress → Triaged |
description: | updated |
description: | updated |
description: | updated |
Changed in linux (Ubuntu Zesty): | |
status: | In Progress → Fix Committed |
This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification- needed- zesty' to 'verification- done-zesty' . If the problem still exists, change the tag 'verification- needed- zesty' to 'verification- failed- zesty'.
If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.
See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Thank you!