https://bugs.freedesktop.org/show_bug.cgi?id=90888
--- Comment #1 from Ilia Mirkin imir...@alum.mit.edu ---
Great patch. Please mail it out to nouveau@lists.freedesktop.org, these things
are usually just done over email when there's no real bug to keep track of. One
small thing is I normally use
https://bugs.freedesktop.org/show_bug.cgi?id=90888
Bug ID: 90888
Summary: A trivial fix for a potential crash
Product: Mesa
Version: git
Hardware: All
OS: All
Status: NEW
Severity: trivial
https://bugs.freedesktop.org/show_bug.cgi?id=90887
--- Comment #3 from Ilia Mirkin imir...@alum.mit.edu ---
Gah! The inbound edge ordering having to be the same as in phi nodes has bit me
before as well. I wonder if it wouldn't be simpler to just fix that, i.e. make
phi nodes attach a bb
https://bugs.freedesktop.org/show_bug.cgi?id=90887
--- Comment #1 from jr j...@online.de ---
This is the minified shader I did the debugging with.
translating program of type 4
FRAG
DCL IN[0], GENERIC[9], PERSPECTIVE
DCL OUT[0], COLOR
DCL SAMP[0]
DCL TEMP[0..2], LOCAL
IMM[0] FLT32 {0.,
These functions will be also used by the interface for controlling
performance counters.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/nvkm/engine/sw/nv50.c | 56 ++-
1 file changed, 38 insertions(+), 18 deletions(-)
diff --git
16 bits is large enough to store the maximum number of signals available
for one domain (i.e. 256).
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
bin/nv_perfmon.c | 4 ++--
drm/nouveau/include/nvif/class.h | 7 ---
drm/nouveau/nvkm/engine/pm/base.c | 10
Signed-off-by: Samuel Pitoiset samuel.pitoiset at gmail.com
---
drm/nouveau/include/nvif/class.h | 1 +
drm/nouveau/nvkm/engine/pm/base.c | 102 ++
drm/nouveau/nvkm/engine/pm/priv.h | 2 +
3 files changed, 95 insertions(+), 10 deletions(-)
diff --git
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/include/nvif/class.h | 3 ++-
drm/nouveau/nvkm/engine/pm/base.c | 22 +++---
2 files changed, 21 insertions(+), 4 deletions(-)
diff --git a/drm/nouveau/include/nvif/class.h
This allows to query the ID, the mask and the user-readable name of
sources for each signal.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/include/nvif/class.h | 12 ++
drm/nouveau/nvkm/engine/pm/base.c | 77 +++
2 files
This adds a new method NVIF_PERFCTR_V0_INIT which starts a batch of
hardware counters for sampling. This will allow the userspace to start
a monitoring session using the INIT method and to stop it with SAMPLE,
for example before and after a frame is rendered.
This commit temporarily breaks
https://bugs.freedesktop.org/show_bug.cgi?id=90887
Bug ID: 90887
Summary: PhiMovesPass in register allocator broken
Product: Mesa
Version: git
Hardware: All
OS: All
Status: NEW
Severity: normal
https://bugs.freedesktop.org/show_bug.cgi?id=90887
--- Comment #2 from jr j...@online.de ---
Forgot to add: Interestingly without the additional blocks inserted by TXL in
the if block the bug is not triggered (i.e. other tex operations work fine).
--
You are receiving this mail because:
You are
This commit introduces the NVIF_IOCTL_NEW_V0_PERFMON class which will be
used in order to query domains, signals and sources. This separates the
querying and the counting interface.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
bin/nv_perfmon.c | 12 ++--
This will allow to configure performance counters with hardware signal
indexes instead of user-readable names in an upcoming patch.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
bin/nv_perfmon.c | 185 --
This allows to query the number of available domains, including the
number of hardware counter and the number of signals per domain.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/include/nvif/class.h | 11 -
drm/nouveau/nvkm/engine/pm/base.c | 86
PDAEMON signals don't have to be exposed by the perfmon engine.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
Reviewed-by: Martin Peres martin.pe...@free.fr
---
drm/nouveau/include/nvkm/engine/pm.h | 4 --
drm/nouveau/nvkm/engine/pm/Kbuild| 1 -
This software methods interface will allow the userspace to tie
monitoring of performance counters to the command stream.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/nvkm/engine/sw/nv50.c | 160 ++
drm/nouveau/nvkm/engine/sw/nv50.h
These signals and sources have been reverse engineered from NVIDIA
PerfKit (Windows) and CUPTI (Linux), they will be used to build complex
hardware events from the userspace.
This commit also adds a new class for GT200.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
From: Samuel Pitoiset samuel.pitoisetÅgmail.com
A source (or multiplexer) is a tuple addr+mask+shift which allows to
control a block of signals. The maximum number of sources that a signal
can define is arbitrary limited to 8 and this should be large enough.
This patch allows to define
Configuring counters from the userspace require the kernel to handle some
logic related to performance counters. Basically, it has to find a free
slot to assign a counter, to handle extra counting modes like B4/B6 and it
must return and error when it can't configure a counter.
In my opinion, the
This fixes a crash when multiple PM engine contexts are created.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
Reviewed-by: Martin Peres martin.pe...@free.fr
---
drm/nouveau/nvkm/engine/pm/base.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git
Hello,
This series exposes global performance counters (PCOUNTER) to the userspace
through the nvif interface by reworking most of the code related to the PM
engine.
This interface will allow the userspace to control and monitor complex hardware
events like the proprietary driver already does,
There is always the possiblity that the ppm-context pointer would get
partially updated and accidentally would equal ctx. This would allow two
contexts to co-exist, which is not acceptable. Moving the test to the
critical section takes care of this problem.
Signed-off-by: Samuel Pitoiset
Since a new class has been introduced to query signals, we can now
return an error when the userspace wants to monitor unknown signals.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/nvkm/engine/pm/base.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
Reviewed-by: Martin Peres martin.pe...@free.fr
---
drm/nouveau/nvkm/engine/pm/base.c | 20
drm/nouveau/nvkm/engine/pm/priv.h | 4
2 files changed, 24 deletions(-)
diff --git a/drm/nouveau/nvkm/engine/pm/base.c
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
bin/nv_perfmon.c | 6 +++--
drm/nouveau/include/nvif/class.h | 11 ++---
drm/nouveau/nvkm/engine/pm/base.c | 52 +--
3 files changed, 23 insertions(+), 46 deletions(-)
diff
This signal index must be always allowed even if it's not clearly
defined in a domain in order to monitor a counter like 0x03020100
because it's the default value of signals.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
drm/nouveau/nvkm/engine/pm/base.c | 4 ++--
https://bugs.freedesktop.org/show_bug.cgi?id=83897
freedesk.apri...@spamgourmet.com changed:
What|Removed |Added
CC|
https://bugs.freedesktop.org/show_bug.cgi?id=90883
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
Component|Server/General |Driver/nouveau
29 matches
Mail list logo
