Lock crates to version specified by crosvm when building
to ensure that the build succeeds.

Signed-off-by: Rohan Garg <rohan.garg@collabora.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Rename it to vrend_resource_alloc_texture to match
vrend_resource_alloc_buffer.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Add vrend_resource_fixup_emulated_bgra to fix up emulated bgra.

This is a refactor with no real change.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Add vrend_resource_alloc_buffer to allocate the storage for PIPE_BUFFER.

This is a refactor with no real change.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Add vrend_resource_create helper to allocate and initialize the
vrend_resource struct.

This is a refactor with no real change.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

Most of the work of this MR was done by Tomeu.

Changes:
  - update the Dockerfile to use the new virglrenderer tracing option
  - use command line parsing to pass the parameters when running a trace
  - move some files and rename scripts

v2: - unify gfx-pps build (Tomeu) and use to suggested branch (Fahien)
    - use upstream tagged version for igt-gpu-tools (Tomeu)
    - add parameters to set benchmark loop count (Louis-Francis)
    - add parameter to set perfetto loop count or loopless run

v3: - compile virglrenderer against minigbm (Rohan)
    - compile minigbm with driver i915 (Rohan)

v4: Don't build mesa-gbm (instead of deleting the library and
    header later)

v5: add option to run script whether to wait after each frame

v6: correct gfx-pps version and add gdb

v7: fix tag for gfx-pps

v8: - when building the container use the user ID and group
      id of the user who runs the container build
    - rework passing command line parameters
    - add a command line parameter to record per-frame
      images
    - Use debugoptimized build for gfx-pps (Tomeu)

v9: disable buffer storage, there seems to be a bug in
    context handling triggered by this

v10: move to upstream kernel 5.9.12

v11: - remove writing txt version of host trace
     - add debug flagto enable tracing EGL
     - don't force-remove libgbm1 it is no longer a problem to
       have it

v12: update kernel to 5.20.5

v13: refactor trace merge (Rohan)

v14: - Fix refactoring (Rohan)
     - Add license infro to perfetto merge script

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-By: Rohan Garg <rohan.garg@collabora.com>

It is specified in the tgsi.rst docs (and tgsi_exec.c matches) to only
look at the .x channel.

Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

This is now already checked before we decode the command.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

v2: drop decode state and use local variables to handle keeping
    track of the decode buffer state (Chia-I Wo)

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

With that we don't need to resolve the pointer chain in each decode
function.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

Since each switch case only results is a function call we can also execute
the call directly by using a call table.

v2: Add VIRGL_CCMD_NOP and validate table when a context is
    created (Chai-I Wu).

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

If GL_POINT_SPRITE mode is enabled and a texture has
GL_COORD_REPLACE enabled, the texture will apply coord_replace
to all primitive types, instead of only GL_POINTS as expected.

This change checks the prim_mode for every draw call and selects
the appropriate shader variant to enable coord_replace only when
rendering GL_POINTS primitives.

Closes: virgl/virglrenderer#188



Signed-off-by: Ryan Neph <ryanneph@google.com>
Reviewed-by: Eric Anholt <eric@anholt.net>

We don't know what internal format the GL driver uses and we have to
glFinish and do a gbm transfer unless VREND_STORAGE_GL_IMMUTABLE is set.

This is motivated by Mesa's internal format change from GL_RGB to
GL_RGB8 for VIRGL_FORMAT_B8G8R8X8_UNORM in
mesa/mesa@5e4d69ec

.
That change caused glTexSubImage2D(..., GL_BGRA_EXT, GL_UNSIGNED_BYTE)
to be rejected by _mesa_gles_error_check_format_and_type.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

The bo contents may be undefined when only GBM_BO_TRANSFER_WRITE is set
(e.g., with radeonsi).  When transferring to a subregion, we need to set
GBM_BO_TRANSFER_READ as well.

gbm_bo_map2 does allow us to map a subregion, but not all
implementations work correctly.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

We can now use gbm_bo_map(..) directly to match Mesa.

If minigbm doesn't define the needed use flags, we define it
internally.

This should allow us to build with minigbm allocation enabled and
use the Mesa gbm header.

Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

Closes #186

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

GLES hosts might expose this.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

Fixes the order of params passed to emit_ios() calls
(swaps winsys_adjust_y_emitted and force_color_two_side).

Signed-off-by: Ryan Neph <ryanneph@google.com>
Reviewed-By: David Riley <davidriley@chromium.org>

Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

EGL_ANDROID_native_fence_sync only works on gles profiles.

Signed-off-by: Lepton Wu <lepton@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

v2: Fix artifact copy output path (Tomeu)

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

This simply logs all tagged calls and scopes to stderr.

v2: - Fix option comparsion
    - add trace_init stub
    - update the combo option description

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

Author: Tomeu Vizoso <tomeu.vizoso@collabora.com> (v1)
v2: - Include perfetto_min.h only when perf tracing is enabled
    - Make stubs inline code so it can be optimized away.
    - Add compile with tracing info to configuration status
    - Use static for stubs (Chia-I Wu)
    - pass trace option as a string and add defines so that
      other tracing tools cold be accomodated
      (inspired by Chia-I Wu & Tomeu)
    - Use macros to wrap the trace calls

v3: - Use __attribute__((cleanup)) to handle trace_begin/trace_end
      instead of using macros to wrap the calls (Chia-I Wu)
    - remove cpp from language support

v4: - Fix __attribute__ check in meson.build
    - replace TRACE_FUNC macro to not take a paramter
    - move trace initialization to a seperate function
      (all suggestions by Chia-I Wu)

v5: use vsnprintf to avoid buffer overflow (John Bates)

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com> (v4)

Add definition of GBM_FORMAT_ABGR16161616F when it's missed so
we can compile against mesa version before 20.0

Signed-off-by: Lepton Wu <lepton@chromium.org>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabra.com>

capset 1 has version 0 and 1, and they are the same.  capset 2 has
version 0, 1, and 2, and they are also the same.

Note that Mesa always asks for version 0.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabra.com>

0 is never an invalid capset id.

For an invalid capset id, we have to assume caps points to a buffer of
size 0 (what vrend_renderer_get_cap_set reports).  We can't set
caps->max_version.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabra.com>

Use VIRGL_RENDERER_CAPSET_VIRGL* instead.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabra.com>

So that it can be honored by all renderers.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

So that it can be shared by different renderers (there is only vrend
right now).

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

vrend_free_fences should free fences on both fence_list and
fence_wait_list.  vrend_renderer_fini should call vrend_free_fences.

Since vrend_free_sync_thread is always called before vrend_free_fences,
we can assert(!vrend_state.sync_thread) rather than do the locking.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

It was broken since the switch to meson.

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Gert Wollny <gert.wollny@collabora.com>
Acked-by: David Stevens <stevensd@chromium.org>

Gert Wollny authored 4 years ago and Gert Wollny committed 4 years ago

This extension is needed to get GL 4.4 in the guest when we
run on a GLES host.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Reviewed-by: Lepton Wu <lepton@chromium.org>

This format was used in Android AHardwareBufferNativeTests.

Signed-off-by: Lepton Wu <lepton@chromium.org>
Reviewed-by: Chia-I Wu <olvaffe@gmail.com>