ttm_tt_set_user has "ttm->state = tt_unbound" which will cause the
following ttm_tt_populate return directly because of bellow check:
if (ttm->state != tt_unpopulated)
return 0;
Does it mean we don't need to populate user pages? Here is the patch
if it is not the case
Thanks
Austin
>From 91ca6f8a5bb84a9375033d7df1cf7c0d1568e6ce Mon Sep 17 00:00:00 2001
From: Austin Yuan <[email protected]>
Date: Tue, 29 Jun 2010 14:15:34 +0800
Subject: [PATCH] TTM: fix the issue that user buffers are not
populuated due to the wrong state
Signed-off-by: Austin Yuan <[email protected]>
Signed-off-by: Elaine Wang <[email protected]>
---
drivers/gpu/drm/ttm/ttm_tt.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c
index a7bab87..4ea44c2 100644
--- a/drivers/gpu/drm/ttm/ttm_tt.c
+++ b/drivers/gpu/drm/ttm/ttm_tt.c
@@ -367,7 +367,7 @@ int ttm_tt_set_user(struct ttm_tt *ttm,
ttm->tsk = tsk;
ttm->start = start;
- ttm->state = tt_unbound;
+ ttm->state = tt_unpopulated;
return 0;
}
--
1.6.0.4
_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel
