[Nouveau] [Bug 99913] [G96] Reclocking fails on voltage

2017-03-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99913 Pekka Paalanen changed: What|Removed |Added Status|NEW |NEEDINFO ---

[Nouveau] [Bug 99913] [G96] Reclocking fails on voltage

2017-03-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99913 --- Comment #8 from Karol Herbst --- it would be quite unpleasant if nvidia would indeed use the highest clock level here. And that mmiotrace seems a bit odd, because it accesses a GPIO which shouldn't exist and in any

[Nouveau] [PATCH 0/9] clk subdev updates

2017-03-05 Thread Karol Herbst
This series addresses various issues inside the reclocking code: 1. after resume the set clocks are reset 2. reclocking not possible while GPU is suspended 3. nouveau always does full reclocks even if only a change of the voltage is required Some of the patches were part of the bigger

[Nouveau] [PATCH 6/9] clk: Split out update code to nv40

2017-03-05 Thread Karol Herbst
This code will change for gf100 and newer for partial reclocks. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/nvkm/subdev/clk/base.c | 15 ++- drm/nouveau/nvkm/subdev/clk/g84.c | 1 +

[Nouveau] [PATCH 5/9] clk: We should pass the pstate id around not the index in the list

2017-03-05 Thread Karol Herbst
This makes the code easier, because we can compare the id with pstate->pstate and safe us the trouble iterating over the entire pstate list to match the index. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/nouveau_debugfs.c

[Nouveau] [PATCH 7/9] clk: Only do partial reclocks as required

2017-03-05 Thread Karol Herbst
We don't want to reclock to the same pstate or cstate over and over again, so only do things we actually have to do. This will become usefull later if we have a temperature daemon notifying the clk subdev about a temperature change, because this may only change the voltage or the cstate. And also

[Nouveau] [PATCH 8/9] clk: Set clocks to pre suspend state after suspend

2017-03-05 Thread Karol Herbst
The idea is to clear out the saved state, because after a resume we can't know what the GPU is clocked to. The reclock is triggered by the call to nvkm_clk_update later in nvkm_clk_init. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres ---

[Nouveau] [Bug 99913] [G96] Reclocking fails on voltage

2017-03-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99913 --- Comment #11 from Karol Herbst --- but you can use OpenGL stuff to confirm if it doesn't use the higher clocks -- You are receiving this mail because: You are the assignee for the

[Nouveau] [Bug 99913] [G96] Reclocking fails on voltage

2017-03-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99913 --- Comment #10 from Karol Herbst --- (In reply to Pekka Paalanen from comment #9) > (In reply to Karol Herbst from comment #8) > > it would be quite unpleasant if nvidia would indeed use the highest clock > > level here.

Re: [Nouveau] [PATCH 2/9] clk: Remove dstate

2017-03-05 Thread Ilia Mirkin
On Sun, Mar 5, 2017 at 11:35 AM, Karol Herbst wrote: > We won't need it now, because we will adjust the clocks depending on engine > loads later on anyway or a static lockup table. It also simplifies the Presumably "lookup"? > clocking logic. > > This code was nowhere

[Nouveau] [PATCH 4/9] clk: Hold information about the current cstate status

2017-03-05 Thread Karol Herbst
Later we will have situations where the expected and the current state isn't the same. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/include/nvkm/subdev/clk.h | 2 ++ drm/nouveau/nvkm/subdev/clk/base.c| 32

[Nouveau] [PATCH 3/9] clk: Make pstate a pointer to nvkm_pstate

2017-03-05 Thread Karol Herbst
We will access the current set cstate at least every second and this safes us some CPU cycles looking them up every second. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/include/nvkm/subdev/clk.h | 4 +++-

[Nouveau] [PATCH 9/9] clk: Check pm_runtime status before reclocking

2017-03-05 Thread Karol Herbst
We don't want to change anything on the GPU if it's suspended. Also we need to increase the refcount on the pm_runtime counter so that the GPU won't be suspended while reclocking. Signed-off-by: Karol Herbst --- drm/nouveau/nvkm/subdev/clk/base.c | 10 +- 1 file

[Nouveau] [PATCH 1/9] clk: Rename nvkm_pstate_calc to nvkm_clk_update and export it

2017-03-05 Thread Karol Herbst
This function will be used to update the current clock state. This will happen for various reasons: * Temperature changes * User changes clocking state * Load changes Signed-off-by: Karol Herbst --- drm/nouveau/include/nvkm/subdev/clk.h | 1 +

[Nouveau] [PATCH 2/9] clk: Remove dstate

2017-03-05 Thread Karol Herbst
We won't need it now, because we will adjust the clocks depending on engine loads later on anyway or a static lockup table. It also simplifies the clocking logic. This code was nowhere used anyway and just a mock up. Signed-off-by: Karol Herbst Reviewed-by: Martin Peres

Re: [Nouveau] A request for Virtual GPU based on Nouveau

2017-03-05 Thread Jack Coulter
Hi Jay, On 25/02/17 14:39, Zhengjie (PARC) wrote: > > I know the nouveau project aims to build high-quality, free/libre > software drivers for nVidia cards > . Maybe > the nouveau project developers can do something to help GPU > Virtualization.

[Nouveau] [Bug 99900] [NVC1] nouveau: freeze / crash after kernel update to 4.10

2017-03-05 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99900 --- Comment #11 from Torsten Krah --- Hi Ben, don't you think that https://bugs.freedesktop.org/show_bug.cgi?id=99922 reads like a duplicate of that one here - at least sounds similar to me? -- You are receiving this mail