[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Christian König
Am 09.09.2016 um 15:54 schrieb Emil Velikov: > On 9 September 2016 at 12:24, Christian König > wrote: >> Hi Hawking, >> >>> Removing the flag will make ttm_mem_type_from_place skip counting the >>> corresponding placement and thus have impact on mem region create and bo >>> movement. >> And that

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Christian König
stian; Cui, Flora; Zhou, David(ChunMing); >> amd-gfx at lists.freedesktop.org >> Cc: dri-devel at lists.freedesktop.org >> Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags >> >> Hi Hawking, >> >>> Removing the flag will make ttm_mem_type_from_pla

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Emil Velikov
On 9 September 2016 at 15:30, Christian König wrote: > Am 09.09.2016 um 15:54 schrieb Emil Velikov: >> >> On 9 September 2016 at 12:24, Christian König >> wrote: >>> >>> Hi Hawking, >>> Removing the flag will make ttm_mem_type_from_place skip counting the corresponding placement and

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Flora Cui
yes. please don't do this, I need them. On Fri, Sep 09, 2016 at 02:41:16PM +0800, zhoucm1 wrote: > > > On 2016年09月08日 21:41, Christian König wrote: > >From: Christian König > > > >Either never used or not used in quite a while. > No, I remember Flora's Direct GMA is using them like GDS

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Emil Velikov
On 9 September 2016 at 12:24, Christian König wrote: > Hi Hawking, > >> Removing the flag will make ttm_mem_type_from_place skip counting the >> corresponding placement and thus have impact on mem region create and bo >> movement. > > And that is exactly the reason why I want to remove the unuse

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread zhoucm1
On 2016年09月08日 21:41, Christian König wrote: > From: Christian König > > Either never used or not used in quite a while. No, I remember Flora's Direct GMA is using them like GDS use PRIV0-2. And you cannot make sure there isn't no one using them in other closed projects, right? If you

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Deucher, Alexander
rg > Cc: dri-devel at lists.freedesktop.org > Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags > > Hi Hawking, > > > Removing the flag will make ttm_mem_type_from_place skip counting the > corresponding placement and thus have impact on mem region create an

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Christian König
sage- > From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of > Christian K?nig > Sent: Friday, September 09, 2016 17:07 > To: Cui, Flora ; Zhou, David(ChunMing) amd.com> > Cc: amd-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org > Subje

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Christian König
In this case please just add them back in your tree. That should be a two liner. For upstream it certainly doesn't make sense to keep them. Regards, Christian. Am 09.09.2016 um 09:01 schrieb Flora Cui: > yes. please don't do this, I need them. > > On Fri, Sep 09, 2016 at 02:41:16PM +0800, zhouc

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Zhang, Hawking
d-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Friday, September 09, 2016 17:07 To: Cui, Flora ; Zhou, David(ChunMing) Cc: amd-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags In this case please

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-09 Thread Zhang, Hawking
d-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Friday, September 09, 2016 17:07 To: Cui, Flora ; Zhou, David(ChunMing) Cc: amd-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags In this case please

[PATCH 1/2] drm/ttm: remove unused placement flags

2016-09-08 Thread Christian König
From: Christian König Either never used or not used in quite a while. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c| 2 +- include/drm/ttm/ttm_placement.h | 19 --- 2 files changed, 1 insertion(+), 20 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_b