Re: [PATCH] drm/syncobj: remove boring message

2019-08-02 Thread Jason Ekstrand
On Fri, Aug 2, 2019 at 12:33 PM Koenig, Christian wrote: > > > Am 02.08.2019 18:28 schrieb Jason Ekstrand : > > On Thu, Aug 1, 2019 at 9:05 AM Koenig, Christian > wrote: > > Am 01.08.19 um 15:45 schrieb Lionel Landwerlin: > > Just had a few exchanges with Chris about this. > > Chris suggests

Re: [PATCH] drm/syncobj: remove boring message

2019-08-02 Thread Koenig, Christian
Am 02.08.2019 18:28 schrieb Jason Ekstrand : On Thu, Aug 1, 2019 at 9:05 AM Koenig, Christian mailto:christian.koe...@amd.com>> wrote: Am 01.08.19 um 15:45 schrieb Lionel Landwerlin: > Just had a few exchanges with Chris about this. > Chris suggests that if we're about to add a point to the

Re: [PATCH] drm/syncobj: remove boring message

2019-08-02 Thread Jason Ekstrand
On Thu, Aug 1, 2019 at 9:05 AM Koenig, Christian wrote: > Am 01.08.19 um 15:45 schrieb Lionel Landwerlin: > > Just had a few exchanges with Chris about this. > > Chris suggests that if we're about to add a point to the timeline in > > an unordered fashion, actually better not add it at all. > >

Re: [PATCH] drm/syncobj: remove boring message

2019-08-01 Thread Koenig, Christian
Am 01.08.19 um 15:45 schrieb Lionel Landwerlin: > Just had a few exchanges with Chris about this. > Chris suggests that if we're about to add a point to the timeline in > an unordered fashion, actually better not add it at all. > > What's your take on this? That is a clear NAK. See not adding a

Re: [PATCH] drm/syncobj: remove boring message

2019-08-01 Thread Lionel Landwerlin
Just had a few exchanges with Chris about this. Chris suggests that if we're about to add a point to the timeline in an unordered fashion, actually better not add it at all. What's your take on this? I'm fine with this, but rather than add another variant of add_point() maybe we change change

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread zhoucm1
On 2019年07月30日 17:40, Lionel Landwerlin wrote: On 30/07/2019 12:36, Daniel Vetter wrote: On Tue, Jul 30, 2019 at 05:31:26PM +0800, zhoucm1 wrote: On 2019年07月30日 17:27, Daniel Vetter wrote: On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: It is normal that binary syncobj

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread Lionel Landwerlin
On 30/07/2019 12:36, Daniel Vetter wrote: On Tue, Jul 30, 2019 at 05:31:26PM +0800, zhoucm1 wrote: On 2019年07月30日 17:27, Daniel Vetter wrote: On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: It is normal that binary syncobj replaces the underlying fence. Signed-off-by:

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread Daniel Vetter
On Tue, Jul 30, 2019 at 05:31:26PM +0800, zhoucm1 wrote: > > > On 2019年07月30日 17:27, Daniel Vetter wrote: > > On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: > > > It is normal that binary syncobj replaces the underlying fence. > > > > > > Signed-off-by: Chunming Zhou > > Do we

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread zhoucm1
On 2019年07月30日 17:27, Daniel Vetter wrote: On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: It is normal that binary syncobj replaces the underlying fence. Signed-off-by: Chunming Zhou Do we hit this with one of the syncobj igts? Unforturnately, No, It's only hit in AMDGPU

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread Lionel Landwerlin
On 30/07/2019 12:27, Daniel Vetter wrote: On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: It is normal that binary syncobj replaces the underlying fence. Signed-off-by: Chunming Zhou Do we hit this with one of the syncobj igts? -Daniel With one of the tests sitting on the

Re: [PATCH] drm/syncobj: remove boring message

2019-07-30 Thread Daniel Vetter
On Mon, Jul 29, 2019 at 04:20:39PM +0800, Chunming Zhou wrote: > It is normal that binary syncobj replaces the underlying fence. > > Signed-off-by: Chunming Zhou Do we hit this with one of the syncobj igts? -Daniel > --- > drivers/gpu/drm/drm_syncobj.c | 3 --- > 1 file changed, 3

[PATCH] drm/syncobj: remove boring message

2019-07-29 Thread Chunming Zhou
It is normal that binary syncobj replaces the underlying fence. Signed-off-by: Chunming Zhou --- drivers/gpu/drm/drm_syncobj.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c index 929f7c64f9a2..bc7ec1679e4d 100644 ---