Re: [PATCH v2 2/3] etnaviv: add permon support
Am Dienstag, den 12.12.2017, 22:27 +0100 schrieb Christian Gmeiner: > Query all domains and their signals and provide it this information > via struct etna_perfmon and the corresponding api functions. > > v2: > - code style changes > - etna_perfmon_create(..): add missing clean up in error case > > Signed-off-by: Christian Gmeiner Reviewed-by: Lucas Stach > --- > etnaviv/Makefile.sources | 1 + > etnaviv/etnaviv-symbol-check | 4 + > etnaviv/etnaviv_drmif.h | 11 +++ > etnaviv/etnaviv_perfmon.c| 189 > +++ > etnaviv/etnaviv_priv.h | 21 + > 5 files changed, 226 insertions(+) > create mode 100644 etnaviv/etnaviv_perfmon.c > > diff --git a/etnaviv/Makefile.sources b/etnaviv/Makefile.sources > index 52580567..0eb73783 100644 > --- a/etnaviv/Makefile.sources > +++ b/etnaviv/Makefile.sources > @@ -3,6 +3,7 @@ LIBDRM_ETNAVIV_FILES := \ > etnaviv_gpu.c \ > etnaviv_bo.c \ > etnaviv_bo_cache.c \ > + etnaviv_perfmon.c \ > etnaviv_pipe.c \ > etnaviv_cmd_stream.c \ > etnaviv_drm.h \ > diff --git a/etnaviv/etnaviv-symbol-check b/etnaviv/etnaviv-symbol- > check > index 0e2030e4..bd95b459 100755 > --- a/etnaviv/etnaviv-symbol-check > +++ b/etnaviv/etnaviv-symbol-check > @@ -42,6 +42,10 @@ etna_cmd_stream_flush > etna_cmd_stream_flush2 > etna_cmd_stream_finish > etna_cmd_stream_reloc > +etna_perfmon_create > +etna_perfmon_del > +etna_perfmon_get_dom_by_name > +etna_perfmon_get_sig_by_name > EOF > done) > > diff --git a/etnaviv/etnaviv_drmif.h b/etnaviv/etnaviv_drmif.h > index 87704acd..949b9b62 100644 > --- a/etnaviv/etnaviv_drmif.h > +++ b/etnaviv/etnaviv_drmif.h > @@ -35,6 +35,9 @@ struct etna_pipe; > struct etna_gpu; > struct etna_device; > struct etna_cmd_stream; > +struct etna_perfmon; > +struct etna_perfmon_domain; > +struct etna_perfmon_signal; > > enum etna_pipe_id { > ETNA_PIPE_3D = 0, > @@ -190,4 +193,12 @@ struct etna_reloc { > > void etna_cmd_stream_reloc(struct etna_cmd_stream *stream, const > struct etna_reloc *r); > > +/* performance monitoring functions: > + */ > + > +struct etna_perfmon *etna_perfmon_create(struct etna_pipe *pipe); > +void etna_perfmon_del(struct etna_perfmon *perfmon); > +struct etna_perfmon_domain *etna_perfmon_get_dom_by_name(struct > etna_perfmon *pm, const char *name); > +struct etna_perfmon_signal *etna_perfmon_get_sig_by_name(struct > etna_perfmon_domain *dom, const char *name); > + > #endif /* ETNAVIV_DRMIF_H_ */ > diff --git a/etnaviv/etnaviv_perfmon.c b/etnaviv/etnaviv_perfmon.c > new file mode 100644 > index ..aa5130a6 > --- /dev/null > +++ b/etnaviv/etnaviv_perfmon.c > @@ -0,0 +1,189 @@ > +/* > + * Copyright (C) 2017 Etnaviv Project > + * Copyright (C) 2017 Zodiac Inflight Innovations > + * > + * Permission is hereby granted, free of charge, to any person > obtaining a > + * copy of this software and associated documentation files (the > "Software"), > + * to deal in the Software without restriction, including without > limitation > + * the rights to use, copy, modify, merge, publish, distribute, > sublicense, > + * and/or sell copies of the Software, and to permit persons to whom > the > + * Software is furnished to do so, subject to the following > conditions: > + * > + * The above copyright notice and this permission notice (including > the next > + * paragraph) shall be included in all copies or substantial > portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF > MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO > EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES > OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, > ARISING FROM, > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > DEALINGS IN THE > + * SOFTWARE. > + * > + * Authors: > + *Christian Gmeiner > + */ > + > +#ifdef HAVE_CONFIG_H > +# include > +#endif > + > +#include "etnaviv_priv.h" > + > +static int etna_perfmon_query_signals(struct etna_perfmon *pm, > struct etna_perfmon_domain *dom) > +{ > + struct etna_device *dev = pm->pipe->gpu->dev; > + struct drm_etnaviv_pm_signal req = { > + .pipe = pm->pipe->id, > + .domain = dom->id > + }; > + > + do { > + struct etna_perfmon_signal *sig; > + int ret; > + > + ret = drmCommandWriteRead(dev->fd, > DRM_ETNAVIV_PM_QUERY_SIG, &req, sizeof(req)); > + if (ret) > + break; > + > + sig = calloc(1, sizeof(*sig)); > + if (!sig) > + return -ENOMEM; > + > + INFO_MSG("perfmon signal:"); > + INFO_MSG("id = %d", req.id); > + INFO_MSG("name = %s", req.name); > + > + sig->domain
[PATCH v2 2/3] etnaviv: add permon support
Query all domains and their signals and provide it this information via struct etna_perfmon and the corresponding api functions. v2: - code style changes - etna_perfmon_create(..): add missing clean up in error case Signed-off-by: Christian Gmeiner --- etnaviv/Makefile.sources | 1 + etnaviv/etnaviv-symbol-check | 4 + etnaviv/etnaviv_drmif.h | 11 +++ etnaviv/etnaviv_perfmon.c| 189 +++ etnaviv/etnaviv_priv.h | 21 + 5 files changed, 226 insertions(+) create mode 100644 etnaviv/etnaviv_perfmon.c diff --git a/etnaviv/Makefile.sources b/etnaviv/Makefile.sources index 52580567..0eb73783 100644 --- a/etnaviv/Makefile.sources +++ b/etnaviv/Makefile.sources @@ -3,6 +3,7 @@ LIBDRM_ETNAVIV_FILES := \ etnaviv_gpu.c \ etnaviv_bo.c \ etnaviv_bo_cache.c \ + etnaviv_perfmon.c \ etnaviv_pipe.c \ etnaviv_cmd_stream.c \ etnaviv_drm.h \ diff --git a/etnaviv/etnaviv-symbol-check b/etnaviv/etnaviv-symbol-check index 0e2030e4..bd95b459 100755 --- a/etnaviv/etnaviv-symbol-check +++ b/etnaviv/etnaviv-symbol-check @@ -42,6 +42,10 @@ etna_cmd_stream_flush etna_cmd_stream_flush2 etna_cmd_stream_finish etna_cmd_stream_reloc +etna_perfmon_create +etna_perfmon_del +etna_perfmon_get_dom_by_name +etna_perfmon_get_sig_by_name EOF done) diff --git a/etnaviv/etnaviv_drmif.h b/etnaviv/etnaviv_drmif.h index 87704acd..949b9b62 100644 --- a/etnaviv/etnaviv_drmif.h +++ b/etnaviv/etnaviv_drmif.h @@ -35,6 +35,9 @@ struct etna_pipe; struct etna_gpu; struct etna_device; struct etna_cmd_stream; +struct etna_perfmon; +struct etna_perfmon_domain; +struct etna_perfmon_signal; enum etna_pipe_id { ETNA_PIPE_3D = 0, @@ -190,4 +193,12 @@ struct etna_reloc { void etna_cmd_stream_reloc(struct etna_cmd_stream *stream, const struct etna_reloc *r); +/* performance monitoring functions: + */ + +struct etna_perfmon *etna_perfmon_create(struct etna_pipe *pipe); +void etna_perfmon_del(struct etna_perfmon *perfmon); +struct etna_perfmon_domain *etna_perfmon_get_dom_by_name(struct etna_perfmon *pm, const char *name); +struct etna_perfmon_signal *etna_perfmon_get_sig_by_name(struct etna_perfmon_domain *dom, const char *name); + #endif /* ETNAVIV_DRMIF_H_ */ diff --git a/etnaviv/etnaviv_perfmon.c b/etnaviv/etnaviv_perfmon.c new file mode 100644 index ..aa5130a6 --- /dev/null +++ b/etnaviv/etnaviv_perfmon.c @@ -0,0 +1,189 @@ +/* + * Copyright (C) 2017 Etnaviv Project + * Copyright (C) 2017 Zodiac Inflight Innovations + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + * + * Authors: + *Christian Gmeiner + */ + +#ifdef HAVE_CONFIG_H +# include +#endif + +#include "etnaviv_priv.h" + +static int etna_perfmon_query_signals(struct etna_perfmon *pm, struct etna_perfmon_domain *dom) +{ + struct etna_device *dev = pm->pipe->gpu->dev; + struct drm_etnaviv_pm_signal req = { + .pipe = pm->pipe->id, + .domain = dom->id + }; + + do { + struct etna_perfmon_signal *sig; + int ret; + + ret = drmCommandWriteRead(dev->fd, DRM_ETNAVIV_PM_QUERY_SIG, &req, sizeof(req)); + if (ret) + break; + + sig = calloc(1, sizeof(*sig)); + if (!sig) + return -ENOMEM; + + INFO_MSG("perfmon signal:"); + INFO_MSG("id = %d", req.id); + INFO_MSG("name = %s", req.name); + + sig->domain = dom; + sig->signal = req.id; + strncpy(sig->name, req.name, sizeof(sig->name)); + list_addtail(&sig->head, &dom->signals); + } while (req.iter != 0x); + + return 0; +} + +static int etna_perfmon_query_domains(struct etna_perfmon *pm) +{ + struct etna_device *dev = pm->pipe->gpu->dev;