On Thu, Feb 04, 2010 at 04:31:13PM +0800, Yuan, Shengquan wrote:
> The patch still needs some refine, just want to check if it make sense.
> 
> >From 104a4800ef8d10fbcc5bd830d0ef0040bd4249e9 Mon Sep 17 00:00:00 2001
> From: Austin Yuan <[email protected]>
> Date: Thu, 4 Feb 2010 16:18:28 +0800
> Subject: [PATCH] DRM/TTM: add NO_EVICTION flag for type manager
> 
> All buffers allocated from this manager will derive this flag, it is
> convenient for two reasons:
> 1) TTM_PL_FLAG_NO_EVICT needs root permission, for non-root X,
> cursor/overlay buffers need NO_EVICT
> 2) Some small devices (camera, video) want to use TTM, but don't want
> to burden driver with heavy
>    swap/relocation stuff because only one or two applications run at one time.
> 
> Signed-off-by: Austin Yuan <[email protected]>

My feeling is NAK, you can do that that by always oring TTM_PL_FLAG_NO_EVICT
to the flags you give a BO allocation time. It's not hard.

Cheers,
Jerome

------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to