search for: p_load_target

...b/nvkm/include/subdev/dvfs.h > @@ -0,0 +1,30 @@ > +#ifndef __NOUVEAU_DVFS_H__ > +#define __NOUVEAU_DVFS_H__ > + > +#include <core/subdev.h> > +#include <core/device.h> > + > +#include <subdev/timer.h> > + > +struct nouveau_dvfs_data { > + int p_load_target; > + int p_load_max; > + int p_smooth; > + unsigned int avg_load; > +}; > + > +struct nouveau_dvfs { > + struct nouveau_subdev base; > + struct nouveau_alarm alarm; > + struct nouveau_dvfs_data *data; > +}; > + > +static inline...

...AU_DVFS_H__ >>> +#define __NOUVEAU_DVFS_H__ >>> + >>> +#include <core/subdev.h> >>> +#include <core/device.h> >>> + >>> +#include <subdev/timer.h> >>> + >>> +struct nouveau_dvfs_data { >>> + int p_load_target; >>> + int p_load_max; >>> + int p_smooth; >>> + unsigned int avg_load; >>> +}; >>> + >>> +struct nouveau_dvfs { >>> + struct nouveau_subdev base; >>> + struct nouveau_alarm alarm; >>> +...

Hi, This is a try to have some simple DVFS (Dynamic Voltage and Frequency Scaling) support for GK20A. Instead of relying on other existing frequency scaling framework, we create a simple subdev in Nouveau for the same purpose. That's because we don't want to make the DVFS implementation for GK20A far more than enough in the beginning and hinder the implementation for dGPU in the future.

...de 100644 index 000000000000..3851629a0de4 --- /dev/null +++ b/nvkm/include/subdev/dvfs.h @@ -0,0 +1,30 @@ +#ifndef __NOUVEAU_DVFS_H__ +#define __NOUVEAU_DVFS_H__ + +#include <core/subdev.h> +#include <core/device.h> + +#include <subdev/timer.h> + +struct nouveau_dvfs_data { + int p_load_target; + int p_load_max; + int p_smooth; + unsigned int avg_load; +}; + +struct nouveau_dvfs { + struct nouveau_subdev base; + struct nouveau_alarm alarm; + struct nouveau_dvfs_data *data; +}; + +static inline struct nouveau_dvfs * +nouveau_dvfs(void *obj) +{ + return (void *)nv_device(obj)->subdev[NV...

...0 +1,30 @@ >> +#ifndef __NOUVEAU_DVFS_H__ >> +#define __NOUVEAU_DVFS_H__ >> + >> +#include <core/subdev.h> >> +#include <core/device.h> >> + >> +#include <subdev/timer.h> >> + >> +struct nouveau_dvfs_data { >> + int p_load_target; >> + int p_load_max; >> + int p_smooth; >> + unsigned int avg_load; >> +}; >> + >> +struct nouveau_dvfs { >> + struct nouveau_subdev base; >> + struct nouveau_alarm alarm; >> + struct nouveau_dvfs_data *data;...

...e __NOUVEAU_DVFS_H__ >>>> + >>>> +#include <core/subdev.h> >>>> +#include <core/device.h> >>>> + >>>> +#include <subdev/timer.h> >>>> + >>>> +struct nouveau_dvfs_data { >>>> + int p_load_target; >>>> + int p_load_max; >>>> + int p_smooth; >>>> + unsigned int avg_load; >>>> +}; >>>> + >>>> +struct nouveau_dvfs { >>>> + struct nouveau_subdev base; >>>> + struct nouve...

There might be some callers of nouveau_clock_astate(), and they are from inetrrupt context. So we must ensure that this function can be atomic in that condition. This patch adds one parameter which is subsequently passed to nouveau_pstate_calc(). Therefore we can choose whether we want to wait for the pstate work's completion or not. Signed-off-by: Vince Hsu <vinceh at nvidia.com> ---

nvkm_mm_fini() was not called when exiting the driver, resulting in a memory leak. Fix this. Signed-off-by: Alexandre Courbot <acourbot at nvidia.com> --- drm/nouveau/nouveau_platform.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drm/nouveau/nouveau_platform.c b/drm/nouveau/nouveau_platform.c index 3691982452a9..f83aa12ee5c6 100644 --- a/drm/nouveau/nouveau_platform.c +++

...type; + u8 unit_id; +}; + +struct pmu_rc_msg { + u8 msg_type; + struct pmu_rc_msg_unhandled_cmd unhandled_cmd; +}; + +/*pmu generic msg format*/ +struct pmu_msg { + struct pmu_hdr hdr; + union { + struct pmu_init_msg init; + struct pmu_rc_msg rc; + } msg; +}; struct gk20a_pmu_dvfs_data { int p_load_target; @@ -39,8 +211,19 @@ struct gk20a_pmu_priv { struct nvkm_pmu base; struct nvkm_alarm alarm; struct gk20a_pmu_dvfs_data *data; + struct pmu_ucode_desc *desc; + struct pmu_buf_desc ucode; + struct pmu_buf_desc trace_buf; + struct mutex pmu_copy_lock; + bool pmu_ready; + int pmu_state; + struct...

...mat*/ >> +struct pmu_msg { >> + struct pmu_hdr hdr; >> + union { >> + struct pmu_init_msg init; >> + struct pmu_rc_msg rc; >> + } msg; >> +}; >> >> struct gk20a_pmu_dvfs_data { >> int p_load_target; >> @@ -39,8 +211,19 @@ struct gk20a_pmu_priv { >> struct nvkm_pmu base; >> struct nvkm_alarm alarm; >> struct gk20a_pmu_dvfs_data *data; >> + struct pmu_ucode_desc *desc; >> + struct pmu_buf_desc ucode; >> + struct...

...u_rc_msg_unhandled_cmd unhandled_cmd; +}; + +struct pmu_msg { + struct pmu_hdr hdr; + union { + struct pmu_init_msg init; + struct pmu_rc_msg rc; + } msg; +}; + #define BUSY_SLOT 0 #define CLK_SLOT 7 +#define GK20A_PMU_UCODE_IMAGE "gpmu_ucode.bin" struct gk20a_pmu_dvfs_data { int p_load_target; @@ -39,8 +206,22 @@ struct gk20a_pmu_priv { struct nvkm_pmu base; struct nvkm_alarm alarm; struct gk20a_pmu_dvfs_data *data; + struct pmu_ucode_desc *desc; + struct pmu_buf_desc ucode; + struct pmu_buf_desc trace_buf; + struct mutex pmu_copy_lock; + bool pmu_ready; + int pmu_state; + struct...

Last update here: https://lists.freedesktop.org/archives/nouveau/2017-September/028848.html Basically big cleanup, reordering, simplifications and some renaming to make the code easier to read and to review. I also moved some bugfixes to the front so they can be merged prior the other patches. There was also a bug related to the therm daemon triggering a pstate change leading to PMU lockups,

...struct nvkm_pmu *ppmu); > +static int gk20a_init_pmu_setup_hw1(struct nvkm_pmu *ppmu, struct nvkm_mc *pmc); > +static void gk20a_pmu_intr(struct nvkm_subdev *subdev); > > +static void gk20a_pmu_pgob(struct nvkm_pmu *ppmu, bool enable); > struct gk20a_pmu_dvfs_data { > int p_load_target; > int p_load_max; > int p_smooth; > unsigned int avg_load; > }; > - > struct gk20a_pmu_priv { > struct nvkm_pmu base; > struct nvkm_alarm alarm; > @@ -46,7 +92,30 @@ struct gk20a_pmu_dvfs_dev_status { > unsigned long b...

...tatic int gk20a_init_pmu_setup_sw(struct nvkm_pmu *ppmu); +static int gk20a_init_pmu_setup_hw1(struct nvkm_pmu *ppmu, struct nvkm_mc *pmc); +static void gk20a_pmu_intr(struct nvkm_subdev *subdev); +static void gk20a_pmu_pgob(struct nvkm_pmu *ppmu, bool enable); struct gk20a_pmu_dvfs_data { int p_load_target; int p_load_max; int p_smooth; unsigned int avg_load; }; - struct gk20a_pmu_priv { struct nvkm_pmu base; struct nvkm_alarm alarm; @@ -46,7 +92,30 @@ struct gk20a_pmu_dvfs_dev_status { unsigned long busy; int cur_state; }; - +int gk20a_pmu_debugfs_init(struct nvkm_pmu *ppmu) +{ + st...

...struct nvkm_pmu *ppmu); > +static int gk20a_init_pmu_setup_hw1(struct nvkm_pmu *ppmu, struct nvkm_mc *pmc); > +static void gk20a_pmu_intr(struct nvkm_subdev *subdev); > > +static void gk20a_pmu_pgob(struct nvkm_pmu *ppmu, bool enable); > struct gk20a_pmu_dvfs_data { > int p_load_target; > int p_load_max; > int p_smooth; > unsigned int avg_load; > }; > - > struct gk20a_pmu_priv { > struct nvkm_pmu base; > struct nvkm_alarm alarm; > @@ -46,7 +92,30 @@ struct gk20a_pmu_dvfs_dev_status { > unsigned long b...