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

2016-09-10 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 unused

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

2016-09-10 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 thu

Re: [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 is exactly the reaso

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

2016-09-09 Thread Christian König
@lists.freedesktop.org Cc: dri-de...@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 and bo movement. And that is

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

2016-09-09 Thread Deucher, Alexander
p.org > Cc: dri-de...@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 and > bo mov

RE: [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@lists.freedesktop.org; dri-de...@lists.freedesktop.org Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags In this case please jus

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

2016-09-09 Thread Christian König
istian K?nig Sent: Friday, September 09, 2016 17:07 To: Cui, Flora ; Zhou, David(ChunMing) Cc: amd-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags In this case please just add them back in your tree. That should be a two liner.

RE: [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@lists.freedesktop.org; dri-de...@lists.freedesktop.org Subject: Re: [PATCH 1/2] drm/ttm: remove unused placement flags In this case please jus

Re: [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, zhoucm1

Re: [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 use PRIV

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

2016-09-08 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 removed now