drm/nouveau/pm: use hardware signals indexes instead of user-readable names
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Sun, 7 Jun 2015 20:40:20 +0000 (22:40 +0200)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 28 Aug 2015 02:39:58 +0000 (12:39 +1000)
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/include/nvif/class.h
drivers/gpu/drm/nouveau/nvkm/engine/pm/base.c

index 871247c654a6cc6cbacd010d144c2462bc905d3e..9e4db3aaada7de2eba92978f5325d1205f0b7ebf 100644 (file)
@@ -267,7 +267,8 @@ struct nvif_perfmon_query_signal_v0 {
        __u8  version;
        __u8  domain;
        __u16 iter;
-       __u8  pad03[4];
+       __u8  signal;
+       __u8  pad04[3];
        char  name[64];
 };
 
@@ -278,10 +279,12 @@ struct nvif_perfmon_query_signal_v0 {
 
 struct nvif_perfctr_v0 {
        __u8  version;
-       __u8  pad01[1];
+       __u8  domain;
+       __u8  pad02[2];
        __u16 logic_op;
-       __u8  pad04[4];
-       char  name[4][64];
+       __u8  pad04[2];
+       __u8  signal[4];
+       __u8  pad06[4];
 };
 
 #define NVIF_PERFCTR_V0_SAMPLE                                             0x00
index fab05985ba40a5813e0052f04e4ac29795e6b814..71834b96ca9ef1fbba1f8b3763f7e5d701fdb93a 100644 (file)
@@ -73,49 +73,22 @@ nvkm_perfdom_find(struct nvkm_pm *ppm, int di)
        return NULL;
 }
 
-static struct nvkm_perfsig *
-nvkm_perfsig_find_(struct nvkm_perfdom *dom, const char *name, u32 size)
-{
-       char path[64];
-       int i;
-
-       if (name[0] != '/') {
-               for (i = 0; i < dom->signal_nr; i++) {
-                       if ( dom->signal[i].name &&
-                           !strncmp(name, dom->signal[i].name, size))
-                               return &dom->signal[i];
-               }
-       } else {
-               for (i = 0; i < dom->signal_nr; i++) {
-                       snprintf(path, sizeof(path), "/%s/%02x", dom->name, i);
-                       if (!strncmp(name, path, size))
-                               return &dom->signal[i];
-               }
-       }
-
-       return NULL;
-}
-
 struct nvkm_perfsig *
-nvkm_perfsig_find(struct nvkm_pm *ppm, const char *name, u32 size,
+nvkm_perfsig_find(struct nvkm_pm *ppm, uint8_t di, uint8_t si,
                  struct nvkm_perfdom **pdom)
 {
        struct nvkm_perfdom *dom = *pdom;
-       struct nvkm_perfsig *sig;
 
        if (dom == NULL) {
-               list_for_each_entry(dom, &ppm->domains, head) {
-                       sig = nvkm_perfsig_find_(dom, name, size);
-                       if (sig) {
-                               *pdom = dom;
-                               return sig;
-                       }
-               }
-
-               return NULL;
+               dom = nvkm_perfdom_find(ppm, di);
+               if (dom == NULL)
+                       return NULL;
+               *pdom = dom;
        }
 
-       return nvkm_perfsig_find_(dom, name, size);
+       if (!dom->signal[si].name)
+               return NULL;
+       return &dom->signal[si];
 }
 
 /*******************************************************************************
@@ -200,6 +173,7 @@ nvkm_perfmon_mthd_query_signal(struct nvkm_object *object, void *data, u32 size)
                } else {
                        strncpy(args->v0.name, name, sizeof(args->v0.name));
                }
+               args->v0.signal = si;
        }
 
        while (++si < dom->signal_nr) {
@@ -359,11 +333,9 @@ nvkm_perfctr_ctor(struct nvkm_object *parent, struct nvkm_object *engine,
        } else
                return ret;
 
-       for (i = 0; i < ARRAY_SIZE(args->v0.name) && args->v0.name[i][0]; i++) {
-               sig[i] = nvkm_perfsig_find(ppm, args->v0.name[i],
-                                          strnlen(args->v0.name[i],
-                                                  sizeof(args->v0.name[i])),
-                                          &dom);
+       for (i = 0; i < ARRAY_SIZE(args->v0.signal) && args->v0.signal[i]; i++) {
+               sig[i] = nvkm_perfsig_find(ppm, args->v0.domain,
+                                          args->v0.signal[i], &dom);
                if (!sig[i])
                        return -EINVAL;
        }