Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-07 Thread Christian König
work with new version KMS/kmd? With above question addressed, we can together discuss how to modify CONTEXT_CONTROL patch BR Monk -Original Message- From: Koenig, Christian Sent: Monday, September 05, 2016 7:57 PM To: Liu, Monk Cc: brahma_hybrid_dev Subject: Re: [PATCH] drm/amdgpu:im

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-06 Thread Liu, Monk
--Original Message- From: Christian König [mailto:deathsim...@vodafone.de] Sent: Tuesday, September 06, 2016 5:39 PM To: Liu, Monk ; Koenig, Christian ; Bas Nieuwenhuizen Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) > 1) Is that my patch

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-06 Thread Christian König
essed, we can together discuss how to modify CONTEXT_CONTROL patch BR Monk -Original Message- From: Koenig, Christian Sent: Monday, September 05, 2016 7:57 PM To: Liu, Monk Cc: brahma_hybrid_dev Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Another possible solution which

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-06 Thread Liu, Monk
ONTEXT_CONTROL patch BR Monk -Original Message- From: Koenig, Christian Sent: Monday, September 05, 2016 7:57 PM To: Liu, Monk Cc: brahma_hybrid_dev Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Another possible solution which just came to my mind: Completely ignore

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-05 Thread Christian König
wenhuizen [mailto:b...@basnieuwenhuizen.nl] Sent: Friday, September 02, 2016 12:09 AM To: Liu, Monk Cc: Christian König ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) On Thu, Sep 1, 2016 at 12:55 PM, Liu, Monk wrote: Why does that makes a differen

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-04 Thread Liu, Monk
uizen.nl] Sent: Friday, September 02, 2016 12:09 AM To: Liu, Monk Cc: Christian König ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) On Thu, Sep 1, 2016 at 12:55 PM, Liu, Monk wrote: >> Why does that makes a difference if it is seen for the first time? >&

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-04 Thread Liu, Monk
sktop.org] On Behalf Of Christian K?nig Sent: Thursday, September 01, 2016 10:10 PM To: Liu, Monk ; Bas Nieuwenhuizen Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Am 01.09.2016 um 12:55 schrieb Liu, Monk: >> Why does that makes a difference

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Bas Nieuwenhuizen
ot correct. >>> Besides, CONTEXT_CONTROL not only deals CE but also deal DE. >> >> >>> BR Monk >>> >>> >>> -Original Message- >>> From: Bas Nieuwenhuizen [mailto:b...@basnieuwenhuizen.nl] >>> Sent: Thursday, S

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Christian König
Nieuwenhuizen [mailto:b...@basnieuwenhuizen.nl] Sent: Thursday, September 01, 2016 4:19 PM To: Liu, Monk Cc: Christian König ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) On Thu, Sep 1, 2016 at 9:37 AM, Liu, Monk wrote: -Original Message

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Bas Nieuwenhuizen
s not correct. > Besides, CONTEXT_CONTROL not only deals CE but also deal DE. > > BR Monk > > > -Original Message- > From: Bas Nieuwenhuizen [mailto:b...@basnieuwenhuizen.nl] > Sent: Thursday, September 01, 2016 4:19 PM > To: Liu, Monk > Cc: Christian König ; amd-g

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Liu, Monk
rg] On Behalf Of Christian K?nig Sent: Thursday, September 01, 2016 10:10 PM To: Liu, Monk ; Bas Nieuwenhuizen Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Am 01.09.2016 um 12:55 schrieb Liu, Monk: >> Why does that makes a difference if it

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Liu, Monk
sday, September 01, 2016 10:10 PM To: Liu, Monk ; Bas Nieuwenhuizen Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Am 01.09.2016 um 12:55 schrieb Liu, Monk: >> Why does that makes a difference if it is seen for the first time? >> >>

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Christian König
Monk Cc: Christian König ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) On Thu, Sep 1, 2016 at 9:37 AM, Liu, Monk wrote: -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Wednes

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Liu, Monk
so deal DE. BR Monk -Original Message- From: Bas Nieuwenhuizen [mailto:b...@basnieuwenhuizen.nl] Sent: Thursday, September 01, 2016 4:19 PM To: Liu, Monk Cc: Christian König ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) On T

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Bas Nieuwenhuizen
Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) > > Looks good to me in general, a few nit picks and sugegstions below. > > Am 31.08.2016 um 05:49 schrieb Monk Liu: >> v1: >> for gfx8, use CONTEXT_CONTROL package to dynamically skip preamble >> CEIB and ot

[PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Monk Liu
v1: for gfx8, use CONTEXT_CONTROL package to dynamically skip preamble CEIB and other load_xxx command in sequence. v2: support GFX7 as well, and bump up version. remove cntxcntl in compute ring funcs because CPC doesn't support this packet. v3: fix reduntant judgement in cntxcntl. v4: some clean

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-09-01 Thread Liu, Monk
-Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Wednesday, August 31, 2016 7:53 PM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) Looks good to me in general

Re: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-08-31 Thread Christian König
Looks good to me in general, a few nit picks and sugegstions below. Am 31.08.2016 um 05:49 schrieb Monk Liu: v1: for gfx8, use CONTEXT_CONTROL package to dynamically skip preamble CEIB and other load_xxx command in sequence. v2: support GFX7 as well, and bump up version. remove cntxcntl in comp

RE: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-08-31 Thread Liu, Monk
Monk Subject: [PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3) v1: for gfx8, use CONTEXT_CONTROL package to dynamically skip preamble CEIB and other load_xxx command in sequence. v2: support GFX7 as well, and bump up version. remove cntxcntl in compute ring funcs because CPC doesn't su

[PATCH] drm/amdgpu:implement CONTEXT_CONTROL (v3)

2016-08-30 Thread Monk Liu
v1: for gfx8, use CONTEXT_CONTROL package to dynamically skip preamble CEIB and other load_xxx command in sequence. v2: support GFX7 as well, and bump up version. remove cntxcntl in compute ring funcs because CPC doesn't support this packet. v3: fix reduntant judgement in cntxcntl. Change-Id: I4