Re: [PATCH] drm/ttm: remove unneeded preempt_disable/enable

2012-11-14 Thread Thomas Hellstrom
On 11/09/2012 01:10 PM, Akinobu Mita wrote: It is unnecessary to disable preemption explicitly while calling copy_highpage(). Because copy_highpage() will do it again through kmap_atomic/kunmap_atomic. Signed-off-by: Akinobu Mita Cc: dri-de...@lists.freedesktop.org Cc: David Airlie ---

Re: [PATCH] drm/ttm: remove unneeded preempt_disable/enable

2012-11-14 Thread Thomas Hellstrom
On 11/09/2012 01:10 PM, Akinobu Mita wrote: It is unnecessary to disable preemption explicitly while calling copy_highpage(). Because copy_highpage() will do it again through kmap_atomic/kunmap_atomic. Signed-off-by: Akinobu Mita akinobu.m...@gmail.com Cc: dri-de...@lists.freedesktop.org Cc:

[PATCH] drm/ttm: remove unneeded preempt_disable/enable

2012-11-09 Thread Akinobu Mita
It is unnecessary to disable preemption explicitly while calling copy_highpage(). Because copy_highpage() will do it again through kmap_atomic/kunmap_atomic. Signed-off-by: Akinobu Mita Cc: dri-de...@lists.freedesktop.org Cc: David Airlie --- drivers/gpu/drm/ttm/ttm_tt.c | 4 1 file

[PATCH] drm/ttm: remove unneeded preempt_disable/enable

2012-11-09 Thread Akinobu Mita
It is unnecessary to disable preemption explicitly while calling copy_highpage(). Because copy_highpage() will do it again through kmap_atomic/kunmap_atomic. Signed-off-by: Akinobu Mita akinobu.m...@gmail.com Cc: dri-de...@lists.freedesktop.org Cc: David Airlie airl...@linux.ie ---