There were some minor changes not so recently, and as a result some loops
that unrolled in the old version don't unroll now, so add the new shader
versions from latest Xorg.

Acked-by: Timothy Arceri <tarceri@itsqueeze.com>

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

This statistic is a hash to find shaders back from a runtime trace.
It's not a value we can apply int() on.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com>

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

Mike Blumenkrantz authored 10 months ago and Rhys Perry committed 8 months ago

sometimes shaders are passed which don't specify a context type,
and these should be retried to ensure that they actually cannot be
compiled

"python" doesn't exist on Ubuntu 22.04.

Konstantin Seurer authored 1 year ago and Rhys Perry committed 1 year ago

This patch prints the replay duration after the "Replaying $db" output.
It should be useful for spotting slow fossils.

Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>

Ray tracing pipelines can have multiple shaders of the same stage type.
This means that changes to ray tracing shaders may not show up. This
patch solves the issue by adding an index that makes those shaders
unique.

Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>

Konstantin Seurer authored 1 year ago and Rhys Perry committed 1 year ago

When running ./fossil_replay.sh with --graphics-pipeline-range 0 0
--compute-pipeline-range 0 0, the script will append lines which don't
contain any driver specific columns. Read rows until we are able to guess
the driver instead of aborting after the first row.

Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com>

Displaying information about multiple tests being skipped due to missing GL level compliance
from the driver side is unnecessary when verifying drivers in CI.

To enhance clarity, a verbose mode is added, and skips related to driver support are hidden in this update.

Signed-off-by: David Heidelberg <david.heidelberg@collabora.com>

Reviewed-by: Felix DeGrood <felix.j.degrood@intel.com>

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Felix DeGrood <felix.j.degrood@intel.com>

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Felix DeGrood <felix.j.degrood@intel.com>

https://github.com/Themaister/parallel-rdp



These fossils contain very large and complex shaders. The small_*.foz
files use 8/16-bit arithmetic.

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>

Konstantin Seurer authored 1 year ago and Samuel Pitoiset committed 1 year ago

Aligns RADV stage names to the common ones, including combined stages.

Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>

Konstantin Seurer authored 1 year ago and Samuel Pitoiset committed 1 year ago

This patch adds handling for all possible stage names returned by
_mesa_shader_stage_to_string.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>

This adds a property to indicate that a statistic change makes all
other statistics uncomparable. This is quite useful on Intel where
shaders are compiled in different variants and a SIMD8 compute shader
cannot be compared to a SIMD16 variant. All that should be visible is
the change of SIMD width.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>

reported by pycodestyle

also change the text

The per-app table is better and the biggest changes are printed
with shader file names.

It only printed shaders that also had regressions.

This reverts commit 47ded5b3.

Mistake on my part, `V3D_DEBUG=shaderdb` is actually for manually
dumping stats from random apps in the shader-db format so that they can
be read using shader-db's `report.py`.

Having `V3D_DEBUG=shaderdb` in shader-db makes it dump the information
twice, causing issues.

Signed-off-by: Eric Engestrom <eric@igalia.com>
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>

This fixes ./report-fossil.py --help, which previously just croaked
with an error about not being able to detect the driver, because no .csv
files were specified on the command line.  Which isn't a great error
message when you're...trying to figure out the command line.

While at it, we also drop the requirement that filenames must end with
.csv.  The code to check for that was basically to try and distinguish
filenames from other arguments...which we don't need to do now that we
properly do argument parsing beforehand.  They still have to be .csv
files, of course, but mandating extensions is a bit unusual.

Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>

Almost all the time ;)

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>

We don't want to compare a FS16 to a FS8.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>

Signed-off-by: Eric Engestrom <eric@engestrom.ch>
Acked-by: David Heidelberg <david.heidelberg@collabora.com>

Ian Romanick authored 3 years ago and Matt Turner committed 2 years ago

Discovered when an experiment resulted in a driver that only gave OpenGL
3.0 without GL_ARB_uniform_buffer_object.

All changes made with this script:

for ext in GL_ARB_shader_image_load_store \
           GL_ARB_shader_storage_buffer_object \
	   GL_ARB_shading_language_420pack \
	   GL_ARB_uniform_buffer_object \
	   GL_ARB_explicit_attrib_location
do
    egrep -lr "extension[[:space:]]+${ext}.*enable" . |\
        while read s; do
            # If the shader already has the requirement, skip
            if grep -A10 '[[]require]' $s | grep -q ^$ext ; then
                continue
            fi

            # If the shader has ifdefs for the extension, skip
            if egrep -q "#[[:space:]]*if.*${ext}" $s; then
                continue
            fi

            sed --in-place -e "s/\(^GLSL >.*$\)/\1\n${ext}/" $s
        done
done

These are OpenCL kernels compiled through clvk-0.0.1-r20 on ChromeOS, to
be used by Google Meet.

Filip Gawin authored 2 years ago and Lionel Landwerlin committed 2 years ago

Acked-by: Alejandro Piñeiro <apinheiro@igalia.com>