- 25 Jan, 2021 1 commit
-
-
Louis-Francis Ratté-Boulianne authored
Reviewed-by:
Gert Wollny <gert.wollny@collabora.com>
-
- 22 Jan, 2021 5 commits
-
-
Chia-I Wu authored
An untyped resource is a virgl_resource without pipe_resource while vrend_context works with pipe_resources exclusively. When an untyped resource is attached, we defer the insersion into res_hash until VIRGL_CCMD_PIPE_RESOURCE_SET_TYPE is submitted. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Acked-by:
Gert Wollny <gert.wollny@collabora.com> Acked-by: Gurchetan Singh <gurchetansingh@chromium.org> (protocol)
-
Chia-I Wu authored
Track them with a list and a single-slot cache in vrend_context. The cache hit rate is expected to be ~100%. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Acked-by:
Gert Wollny <gert.wollny@collabora.com>
-
Chia-I Wu authored
An untyped virgl_resource does not have a pipe_resource. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Acked-by:
Gert Wollny <gert.wollny@collabora.com>
-
Chia-I Wu authored
egl_image always wraps gbm_bo so far so this is not a real issue. But it will change soon. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Acked-by:
Gert Wollny <gert.wollny@collabora.com>
-
Chia-I Wu authored
Make virgl_egl_image_from_dmabuf a generic helper. Add virgl_egl_image_from_gbm_bo and virgl_egl_aux_plane_image_from_gbm_bo that use the helper. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Acked-by:
Gert Wollny <gert.wollny@collabora.com>
-
- 21 Jan, 2021 15 commits
-
-
Gert Wollny authored
Use helper variables for some pointer derefs and query the shader ID's only once. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
v2: rename ctx to sub_ctx (inspired by comment of Chia-I) Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
This info is only used when creating the shader key, so it can reside in the sub context. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
With that Unigine Heaven goes from around 56 FPS to 58 Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
With that shaders that are used more frequently will remain at the top of the list, and since the search in the shader program list is linear, this should save some time. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
This gives a 6% speed improvement with Unigine Heaven v2: remove the unlikely for dual_src, the whole statement will be changed later (Chia-I Wu) Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
With that only one deref is needed. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
if-chains can usually not be optimized very well, switch/case might be implemented as a jump table. v2: Add a break in bese before default (Chia-I Wu) Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
This makes the code a bit easier to read and may help the complier to avoid needless dereferencing of pointers. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
v2: add comment about that INDEX is not used (Chia-I Wu) Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
- 16 Jan, 2021 7 commits
-
-
Chia-I Wu authored
It gives clients access to virgl_renderer_context_create_with_flags. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
Comparing to VCMD_GET_{CAPS,CAPS2}, it allows any capset id/version to be queried. This is similar to linux's DRM_IOCTL_VIRTGPU_GET_CAPS. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
There is no parameter defined yet, but the command can be used to query optional features. It also allows new features to be introduced without bumping up the protocol version. This is similar to linux's DRM_IOCTL_VIRTGPU_GETPARAM. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
The goal is to allow clients to negoticate protocol versions, capsets, and in the future, parameters and context types before virgl_renderer_context_create is called. Specifically, these commands should not trigger virgl_renderer_context_create VCMD_PING_PROTOCOL_VERSION VCMD_PROTOCOL_VERSION VCMD_GET_CAPS VCMD_GET_CAPS2 VCMD_RESOURCE_BUSY_WAIT when res_id==0 (for legacy reasons) Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
Move virgl_renderer_context_create call from vtest_create_context to the new vtest_lazy_init_context. This defers context initialization until the first command after VCMD_CREATE_RENDERER. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
struct vtest_context is about to get more complex. Use vtest_destroy_context so that we have a single point that does context destruction. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
Comparing to virgl_renderer_context_create, it allows flags to be specified. flags can only be used to specify the capset id in this commit, but that may change in the future. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
- 15 Jan, 2021 10 commits
-
-
Chia-I Wu authored
It should not fail when in_export_fds is false. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
Unlike out_fourcc, where the default value 0 is DRM_FORMAT_INVALID, the default value 0 for out_modifier is DRM_FORMAT_MOD_LINEAR. This changes nothing in practice as the only user of virgl_renderer_execute is crosvm, and crosvm takes the ENABLE_MINIGBM_ALLOCATION path instead. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
Add virgl_resource::map_info and return it in virgl_renderer_resource_get_map_info. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
This allows the callers to modify the returned virgl_resource without looking up. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
VIRGL_RENDERER_MAP_CACHE_NONE is valid internally. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Chia-I Wu authored
When a virgl_resource wraps a pipe_resource, it is said to be typed because pipe_resource contains the type information. Signed-off-by:
Chia-I Wu <olvaffe@gmail.com> Reviewed-by:
Isaac Bosompem <mrisaacb@google.com>
-
Gert Wollny authored
Most of the time we pass a string that can simply be forwarded to the perfetto trace function. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
Gert Wollny authored
This silences a warning about non-static variable not being previously declared. v2: make table const (Chia-I Wu) Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Reviewed-by:
Chia-I Wu <olvaffe@gmail.com>
-
- 14 Jan, 2021 1 commit
-
-
Rohan Garg authored
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>
-
- 11 Jan, 2021 1 commit
-
-
Chia-I Wu authored
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>
-