[PATCH 1/2] drm/ttm: fix stupid parameter inversion in the pipeline code

[Alex Deucher]

On Fri, Jun 24, 2016 at 3:51 PM, Christian König
 wrote:
> From: Christian König 
>
> We want to keep the newest fence, not the oldest one.
>
> Signed-off-by: Christian König 

Reviewed-by: Alex Deucher 

> ---
>  drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c 
> b/drivers/gpu/drm/ttm/ttm_bo_util.c
> index 0c389a5..4da0e78 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo_util.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
> @@ -753,7 +753,7 @@ int ttm_bo_pipeline_move(struct ttm_buffer_object *bo,
>  */
>
> spin_lock(>move_lock);
> -   if (!from->move || fence_is_later(from->move, fence)) {
> +   if (!from->move || fence_is_later(fence, from->move)) {
> fence_put(from->move);
> from->move = fence_get(fence);
> }
> --
> 2.5.0
>
> ___
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[PATCH 1/2] drm/ttm: fix stupid parameter inversion in the pipeline code

[Christian König]

From: Christian König 

We want to keep the newest fence, not the oldest one.

Signed-off-by: Christian König 
---
 drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c 
b/drivers/gpu/drm/ttm/ttm_bo_util.c
index 0c389a5..4da0e78 100644
--- a/drivers/gpu/drm/ttm/ttm_bo_util.c
+++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
@@ -753,7 +753,7 @@ int ttm_bo_pipeline_move(struct ttm_buffer_object *bo,
 */

spin_lock(>move_lock);
-   if (!from->move || fence_is_later(from->move, fence)) {
+   if (!from->move || fence_is_later(fence, from->move)) {
fence_put(from->move);
from->move = fence_get(fence);
}
-- 
2.5.0