Re: [PATCH] drm/radeon: make 64bit fences more robust v3 (3.5 stable)

2012-09-26 Thread Greg KH
On Thu, Sep 20, 2012 at 10:57:01AM -0400, alexdeuc...@gmail.com wrote: > From: Christian König > > Only increase the higher 32bits if we really detect a wrap around. > > v2: instead of increasing the higher 32bits just use the higher > 32bits from the last emitted fence. > v3: also use last

[PATCH] drm/radeon: make 64bit fences more robust v3 (3.5 stable)

2012-09-26 Thread Greg KH
On Thu, Sep 20, 2012 at 10:57:01AM -0400, alexdeucher at gmail.com wrote: > From: Christian K?nig > > Only increase the higher 32bits if we really detect a wrap around. > > v2: instead of increasing the higher 32bits just use the higher > 32bits from the last emitted fence. > v3: also use la

[PATCH] drm/radeon: make 64bit fences more robust v3 (3.5 stable)

2012-09-20 Thread alexdeuc...@gmail.com
From: Christian K?nig Only increase the higher 32bits if we really detect a wrap around. v2: instead of increasing the higher 32bits just use the higher 32bits from the last emitted fence. v3: also use last emitted fence value as upper limit. The intention of this patch is to make fences as

[PATCH] drm/radeon: make 64bit fences more robust v3 (3.5 stable)

2012-09-20 Thread alexdeucher
From: Christian König Only increase the higher 32bits if we really detect a wrap around. v2: instead of increasing the higher 32bits just use the higher 32bits from the last emitted fence. v3: also use last emitted fence value as upper limit. The intention of this patch is to make fences as

[PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Christian König
Only increase the higher 32bits if we really detect a wrap around. v2: instead of increasing the higher 32bits just use the higher 32bits from the last emitted fence. v3: also use last emitted fence value as upper limit. The intention of this patch is to make fences as robust as they where be

[PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 9:54 AM, Alex Deucher wrote: > On Thu, Sep 13, 2012 at 4:33 AM, Christian K?nig > wrote: >> Only increase the higher 32bits if we really detect a wrap around. >> >> v2: instead of increasing the higher 32bits just use the higher >> 32bits from the last emitted fence. >

[PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Alex Deucher
On Thu, Sep 13, 2012 at 4:33 AM, Christian K?nig wrote: > Only increase the higher 32bits if we really detect a wrap around. > > v2: instead of increasing the higher 32bits just use the higher > 32bits from the last emitted fence. > v3: also use last emitted fence value as upper limit. > > The

Re: [PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Jerome Glisse
On Thu, Sep 13, 2012 at 9:54 AM, Alex Deucher wrote: > On Thu, Sep 13, 2012 at 4:33 AM, Christian König > wrote: >> Only increase the higher 32bits if we really detect a wrap around. >> >> v2: instead of increasing the higher 32bits just use the higher >> 32bits from the last emitted fence. >

Re: [PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Alex Deucher
devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel From b6c778e5742da8eb337ef1e0074a07cb0b89d361 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christian=20K=C3=B6nig?= Date: Thu, 13 Sep 2012 10:33:47 +0200 Subject: [PATCH] drm/radeon: m

[PATCH] drm/radeon: make 64bit fences more robust v3

2012-09-13 Thread Christian König
Only increase the higher 32bits if we really detect a wrap around. v2: instead of increasing the higher 32bits just use the higher 32bits from the last emitted fence. v3: also use last emitted fence value as upper limit. The intention of this patch is to make fences as robust as they where be

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Michel Dänzer
On Die, 2012-09-11 at 12:11 +0200, Christian K?nig wrote: > > How about this idea: Instead of increasing the upper 32bits we just use > the upper 32bits of the last emitted fence value? > E.g. see the attached patch. That both should handle random zero and out > of order values more gracefully.

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Christian König
On 10.09.2012 18:07, Jerome Glisse wrote: > On Mon, Sep 10, 2012 at 11:52 AM, Jerome Glisse wrote: >> On Mon, Sep 10, 2012 at 11:38 AM, Michel D?nzer >> wrote: >>> On Mon, 2012-09-10 at 14:02 +0200, Christian K?nig wrote: On 10.09.2012 13:12, Michel D?nzer wrote: > On Mon, 2012-09-10 at

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Jerome Glisse
On Tue, Sep 11, 2012 at 6:23 AM, Michel D?nzer wrote: > On Die, 2012-09-11 at 12:11 +0200, Christian K?nig wrote: >> >> How about this idea: Instead of increasing the upper 32bits we just use >> the upper 32bits of the last emitted fence value? >> E.g. see the attached patch. That both should hand

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Jerome Glisse
On Tue, Sep 11, 2012 at 6:23 AM, Michel Dänzer wrote: > On Die, 2012-09-11 at 12:11 +0200, Christian König wrote: >> >> How about this idea: Instead of increasing the upper 32bits we just use >> the upper 32bits of the last emitted fence value? >> E.g. see the attached patch. That both should hand

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Dave Airlie
>>> >>> It might be related to a hardware bug, or the algorithm is flawed in a way I currently don't see. Anyway the old code we had wasn't so picky about such problems and the patch just tries to make the current code as robust as the old code was, which indeed seems to solve t

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Michel Dänzer
On Die, 2012-09-11 at 12:11 +0200, Christian König wrote: > > How about this idea: Instead of increasing the upper 32bits we just use > the upper 32bits of the last emitted fence value? > E.g. see the attached patch. That both should handle random zero and out > of order values more gracefully.

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-11 Thread Christian König
4fae6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christian=20K=C3=B6nig?= Date: Sun, 9 Sep 2012 11:45:19 +0200 Subject: [PATCH] drm/radeon: make 64bit fences more robust v2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Only increase the higher 32bits if we

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Michel Dänzer
On Mon, 2012-09-10 at 14:02 +0200, Christian K?nig wrote: > On 10.09.2012 13:12, Michel D?nzer wrote: > > On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: > >> Only increase the higher 32bits if we really detect a wrap around. > >> > >> Fixes: > >> https://bugs.freedesktop.org/show_bug.cg

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 5:10 PM, Jerome Glisse wrote: > On Mon, Sep 10, 2012 at 4:13 PM, Dave Airlie wrote: > > >> It might be related to a hardware bug, or the algorithm is flawed in a >> way I currently don't see. Anyway the old code we had wasn't so picky >> about such prob

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 4:13 PM, Dave Airlie wrote: > It might be related to a hardware bug, or the algorithm is flawed in a > way I currently don't see. Anyway the old code we had wasn't so picky > about such problems and the patch just tries to make the current code as

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 5:10 PM, Jerome Glisse wrote: > On Mon, Sep 10, 2012 at 4:13 PM, Dave Airlie wrote: > > >> It might be related to a hardware bug, or the algorithm is flawed in a >> way I currently don't see. Anyway the old code we had wasn't so picky >> about such prob

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 4:13 PM, Dave Airlie wrote: > It might be related to a hardware bug, or the algorithm is flawed in a > way I currently don't see. Anyway the old code we had wasn't so picky > about such problems and the patch just tries to make the current code as

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Christian König
On 10.09.2012 13:12, Michel D?nzer wrote: > On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: >> Only increase the higher 32bits if we really detect a wrap around. >> >> Fixes: >> https://bugs.freedesktop.org/show_bug.cgi?id=54129 >> https://bugs.freedesktop.org/show_bug.cgi?id=54662 >> >>

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Dave Airlie
>>> >>> It might be related to a hardware bug, or the algorithm is flawed in a way I currently don't see. Anyway the old code we had wasn't so picky about such problems and the patch just tries to make the current code as robust as the old code was, which indeed seems to solve t

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Michel Dänzer
On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: > Only increase the higher 32bits if we really detect a wrap around. > > Fixes: > https://bugs.freedesktop.org/show_bug.cgi?id=54129 > https://bugs.freedesktop.org/show_bug.cgi?id=54662 > > Possible fixes: > https://bugzilla.redhat.com/sh

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 11:52 AM, Jerome Glisse wrote: > On Mon, Sep 10, 2012 at 11:38 AM, Michel D?nzer wrote: >> On Mon, 2012-09-10 at 14:02 +0200, Christian K?nig wrote: >>> On 10.09.2012 13:12, Michel D?nzer wrote: >>> > On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: >>> >> Only in

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 11:38 AM, Michel D?nzer wrote: > On Mon, 2012-09-10 at 14:02 +0200, Christian K?nig wrote: >> On 10.09.2012 13:12, Michel D?nzer wrote: >> > On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: >> >> Only increase the higher 32bits if we really detect a wrap around. >>

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 8:02 AM, Christian K?nig wrote: > On 10.09.2012 13:12, Michel D?nzer wrote: >> >> On Mon, 2012-09-10 at 11:13 +0200, Christian K?nig wrote: >>> >>> Only increase the higher 32bits if we really detect a wrap around. >>> >>> Fixes: >>> https://bugs.freedesktop.org/show_bug.cg

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Christian König
Only increase the higher 32bits if we really detect a wrap around. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=54129 https://bugs.freedesktop.org/show_bug.cgi?id=54662 Possible fixes: https://bugzilla.redhat.com/show_bug.cgi?id=846505 https://bugzilla.redhat.com/show_bug.cgi?id=845639 Si

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 11:52 AM, Jerome Glisse wrote: > On Mon, Sep 10, 2012 at 11:38 AM, Michel Dänzer wrote: >> On Mon, 2012-09-10 at 14:02 +0200, Christian König wrote: >>> On 10.09.2012 13:12, Michel Dänzer wrote: >>> > On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: >>> >> Only in

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 11:38 AM, Michel Dänzer wrote: > On Mon, 2012-09-10 at 14:02 +0200, Christian König wrote: >> On 10.09.2012 13:12, Michel Dänzer wrote: >> > On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: >> >> Only increase the higher 32bits if we really detect a wrap around. >>

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Michel Dänzer
On Mon, 2012-09-10 at 14:02 +0200, Christian König wrote: > On 10.09.2012 13:12, Michel Dänzer wrote: > > On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: > >> Only increase the higher 32bits if we really detect a wrap around. > >> > >> Fixes: > >> https://bugs.freedesktop.org/show_bug.cg

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Jerome Glisse
On Mon, Sep 10, 2012 at 8:02 AM, Christian König wrote: > On 10.09.2012 13:12, Michel Dänzer wrote: >> >> On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: >>> >>> Only increase the higher 32bits if we really detect a wrap around. >>> >>> Fixes: >>> https://bugs.freedesktop.org/show_bug.cg

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Christian König
On 10.09.2012 13:12, Michel Dänzer wrote: On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: Only increase the higher 32bits if we really detect a wrap around. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=54129 https://bugs.freedesktop.org/show_bug.cgi?id=54662 Possible fixes: ht

Re: [PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Michel Dänzer
On Mon, 2012-09-10 at 11:13 +0200, Christian König wrote: > Only increase the higher 32bits if we really detect a wrap around. > > Fixes: > https://bugs.freedesktop.org/show_bug.cgi?id=54129 > https://bugs.freedesktop.org/show_bug.cgi?id=54662 > > Possible fixes: > https://bugzilla.redhat.com/sh

[PATCH] drm/radeon: make 64bit fences more robust

2012-09-10 Thread Christian König
Only increase the higher 32bits if we really detect a wrap around. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=54129 https://bugs.freedesktop.org/show_bug.cgi?id=54662 Possible fixes: https://bugzilla.redhat.com/show_bug.cgi?id=846505 https://bugzilla.redhat.com/show_bug.cgi?id=845639 Si