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 shengquan.yuan@gmail.com 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 shengquan.yuan@gmail.com Signed-off-by: Elaine Wang elaine.wang@intel.com --- 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