Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-12-23 Thread Dmitry Osipenko
On 21.12.2018 21:27, Christian König wrote: > Am 19.12.18 um 18:53 schrieb Dmitry Osipenko: >> [SNIP] >>> @@ -931,9 +718,6 @@ static signed long >>> drm_syncobj_array_wait_timeout(struct drm_syncobj **syncobjs, >>>     if (flags & DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT) { >>>   for (i

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-12-23 Thread Dmitry Osipenko
On 21.12.2018 21:45, Koenig, Christian wrote: > Am 21.12.18 um 19:35 schrieb Dmitry Osipenko: >> On 21.12.2018 21:27, Christian König wrote: >>> Am 19.12.18 um 18:53 schrieb Dmitry Osipenko: [SNIP] > @@ -931,9 +718,6 @@ static signed long > drm_syncobj_array_wait_timeout(struct drm_sy

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-12-21 Thread Koenig, Christian
Am 21.12.18 um 19:35 schrieb Dmitry Osipenko: > On 21.12.2018 21:27, Christian König wrote: >> Am 19.12.18 um 18:53 schrieb Dmitry Osipenko: >>> [SNIP] @@ -931,9 +718,6 @@ static signed long drm_syncobj_array_wait_timeout(struct drm_syncobj **syncobjs,     if (flags & DRM_SYNCO

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-12-21 Thread Christian König
Am 19.12.18 um 18:53 schrieb Dmitry Osipenko: [SNIP] @@ -931,9 +718,6 @@ static signed long drm_syncobj_array_wait_timeout(struct drm_syncobj **syncobjs, if (flags & DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT) { for (i = 0; i < count; ++i) { - if (entrie

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-12-20 Thread Dmitry Osipenko
On 08.11.2018 19:04, Eric Anholt wrote: > Daniel suggested I submit this, since we're still seeing regressions > from it. This is a revert to before 48197bc564c7 ("drm: add syncobj > timeline support v9") and its followon fixes. > > Fixes this on first V3D testcase execution: > > [ 48.767088]

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-15 Thread Eric Anholt
Eric Anholt writes: > [ Unknown signature status ] > "Koenig, Christian" writes: > >> Hi Eric, >> >> thanks for your time to test this. >> >> Am 14.11.18 um 08:07 schrieb Eric Anholt: >>> zhoucm1 writes: >>> Ping.. Eric, Could you help try attached patch for your memory issue? If

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread zhoucm1
On 2018年11月12日 18:16, Christian König wrote: Am 09.11.18 um 23:26 schrieb Eric Anholt: Eric Anholt writes: [ Unknown signature status ] zhoucm1 writes: On 2018年11月09日 00:52, Christian König wrote: Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt:

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread zhoucm1
On 2018年11月12日 18:48, Chris Wilson wrote: Quoting Christian König (2018-11-12 10:16:01) Am 09.11.18 um 23:26 schrieb Eric Anholt: Eric Anholt writes: [ Unknown signature status ] zhoucm1 writes: On 2018年11月09日 00:52, Christian König wrote:

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread Koenig, Christian
Am 12.11.18 um 11:48 schrieb Chris Wilson: > Quoting Christian König (2018-11-12 10:16:01) >> Am 09.11.18 um 23:26 schrieb Eric Anholt: >> >> Eric Anholt writes: >> >> >> [ Unknown signature status ] >> zhoucm1 writes: >> >> >> On 2018年11月09日 00:52, Christian K

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread Chris Wilson
Quoting Christian König (2018-11-12 10:16:01) > Am 09.11.18 um 23:26 schrieb Eric Anholt: > > Eric Anholt writes: > > > [ Unknown signature status ] > zhoucm1 writes: > > > On 2018年11月09日 00:52, Christian König wrote: > > Am 08.11.18 um 17:07

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread zhoucm1
On 2018年11月12日 18:16, Christian König wrote: Am 09.11.18 um 23:26 schrieb Eric Anholt: Eric Anholt writes: [ Unknown signature status ] zhoucm1 writes: On 2018年11月09日 00:52, Christian König wrote: Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt:

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-12 Thread Christian König
Am 09.11.18 um 23:26 schrieb Eric Anholt: Eric Anholt writes: [ Unknown signature status ] zhoucm1 writes: On 2018年11月09日 00:52, Christian König wrote: Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt: Daniel suggested I submit this, since we're st

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-09 Thread Eric Anholt
Eric Anholt writes: > [ Unknown signature status ] > zhoucm1 writes: > >> On 2018年11月09日 00:52, Christian König wrote: >>> Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt: > Daniel suggested I submit this, since we're still seeing regressions >>>

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-09 Thread Eric Anholt
zhoucm1 writes: > On 2018年11月09日 00:52, Christian König wrote: >> Am 08.11.18 um 17:07 schrieb Koenig, Christian: >>> Am 08.11.18 um 17:04 schrieb Eric Anholt: Daniel suggested I submit this, since we're still seeing regressions from it.  This is a revert to before 48197bc564c7 ("drm: a

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-08 Thread zhoucm1
On 2018年11月09日 00:52, Christian König wrote: Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt: Daniel suggested I submit this, since we're still seeing regressions from it.  This is a revert to before 48197bc564c7 ("drm: add syncobj timeline support v9"

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-08 Thread Christian König
Am 08.11.18 um 17:07 schrieb Koenig, Christian: Am 08.11.18 um 17:04 schrieb Eric Anholt: Daniel suggested I submit this, since we're still seeing regressions from it. This is a revert to before 48197bc564c7 ("drm: add syncobj timeline support v9") and its followon fixes. This is a harmless fa

Re: [PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-08 Thread Koenig, Christian
Am 08.11.18 um 17:04 schrieb Eric Anholt: > Daniel suggested I submit this, since we're still seeing regressions > from it. This is a revert to before 48197bc564c7 ("drm: add syncobj > timeline support v9") and its followon fixes. This is a harmless false positive from lockdep, Chouming and I are

[PATCH 2/2] drm: Revert syncobj timeline changes.

2018-11-08 Thread Eric Anholt
Daniel suggested I submit this, since we're still seeing regressions from it. This is a revert to before 48197bc564c7 ("drm: add syncobj timeline support v9") and its followon fixes. Fixes this on first V3D testcase execution: [ 48.767088] [ 48.77