Re: [Nouveau] [PATCH 1/2] Revert "bus: remove cpu_coherent flag"

2016-09-18 Thread Martin Peres
On 19/09/16 06:50, Alexandre Courbot wrote: On Mon, Sep 19, 2016 at 12:43 PM, Ilia Mirkin wrote: On Sun, Sep 18, 2016 at 11:39 PM, Alexandre Courbot wrote: On Sun, Sep 18, 2016 at 7:21 PM, Karol Herbst wrote: This reverts commit 01bbcb69f80e1058395b737ae399c6f4ef48691b. I think you meant a

Re: [Nouveau] [PATCH 1/2] Revert "bus: remove cpu_coherent flag"

2016-09-18 Thread Alexandre Courbot
On Mon, Sep 19, 2016 at 12:43 PM, Ilia Mirkin wrote: > On Sun, Sep 18, 2016 at 11:39 PM, Alexandre Courbot wrote: >> On Sun, Sep 18, 2016 at 7:21 PM, Karol Herbst wrote: >>> This reverts commit 01bbcb69f80e1058395b737ae399c6f4ef48691b. > > I think you meant aff51175cdbf345740ec9203eff88e772af880

Re: [Nouveau] [PATCH 1/2] Revert "bus: remove cpu_coherent flag"

2016-09-18 Thread Ilia Mirkin
On Sun, Sep 18, 2016 at 11:39 PM, Alexandre Courbot wrote: > On Sun, Sep 18, 2016 at 7:21 PM, Karol Herbst wrote: >> This reverts commit 01bbcb69f80e1058395b737ae399c6f4ef48691b. I think you meant aff51175cdbf345740ec9203eff88e772af88059 - that's the commit id that matters, not the one in Ben's

Re: [Nouveau] [PATCH 1/2] Revert "bus: remove cpu_coherent flag"

2016-09-18 Thread Alexandre Courbot
On Sun, Sep 18, 2016 at 7:21 PM, Karol Herbst wrote: > This reverts commit 01bbcb69f80e1058395b737ae399c6f4ef48691b. > > The commit caused fence timeouts within nvc0_screen_destroy and most likely > other places as well. > > The most obvious effect is, that userspace processes take minutes to actu