Commit 7da90a7c authored by Alexandros Frantzis's avatar Alexandros Frantzis Committed by Chia-I Wu

virgl: Don't allow creating staging pipe_resources

Staging buffers are now created directly by the virgl_staging_mgr. We
don't need to support creating staging pipe_resources.
Signed-off-by: Alexandros Frantzis's avatarAlexandros Frantzis <alexandros.frantzis@collabora.com>
Reviewed-by: Chia-I Wu's avatarChia-I Wu <olvaffe@gmail.com>
parent 5388be03
......@@ -90,12 +90,6 @@ static void virgl_buffer_transfer_unmap(struct pipe_context *ctx,
{
struct virgl_context *vctx = virgl_context(ctx);
struct virgl_transfer *trans = virgl_transfer(transfer);
struct virgl_screen *vs = virgl_screen(ctx->screen);
struct pipe_resource *res = transfer->resource;
/* We don't transfer the contents of staging resources, since they don't
* have any host-side storage. */
assert(pipe_to_virgl_bind(vs, res->bind, res->flags) != VIRGL_BIND_STAGING);
if (trans->base.usage & PIPE_TRANSFER_WRITE) {
if (transfer->usage & PIPE_TRANSFER_FLUSH_EXPLICIT) {
......
......@@ -33,12 +33,6 @@
#include "virgl_screen.h"
#define VR_MAX_TEXTURE_2D_LEVELS 15
/* Indicates that the resource will be used as a staging buffer, not requiring
* dedicated host-side storage. Can only be used with PIPE_BUFFER resources
* that have a PIPE_BIND_CUSTOM bind type.
*/
#define VIRGL_RESOURCE_FLAG_STAGING (PIPE_RESOURCE_FLAG_DRV_PRIV << 0)
struct winsys_handle;
struct virgl_screen;
struct virgl_context;
......@@ -144,12 +138,8 @@ static inline unsigned pipe_to_virgl_bind(const struct virgl_screen *vs,
outbind |= VIRGL_BIND_STREAM_OUTPUT;
if (pbind & PIPE_BIND_CURSOR)
outbind |= VIRGL_BIND_CURSOR;
if (pbind & PIPE_BIND_CUSTOM) {
if (flags & VIRGL_RESOURCE_FLAG_STAGING)
outbind |= VIRGL_BIND_STAGING;
else
outbind |= VIRGL_BIND_CUSTOM;
}
if (pbind & PIPE_BIND_CUSTOM)
outbind |= VIRGL_BIND_CUSTOM;
if (pbind & PIPE_BIND_SCANOUT)
outbind |= VIRGL_BIND_SCANOUT;
if (pbind & PIPE_BIND_SHARED)
......@@ -161,6 +151,12 @@ static inline unsigned pipe_to_virgl_bind(const struct virgl_screen *vs,
if (pbind & PIPE_BIND_COMMAND_ARGS_BUFFER)
if (vs->caps.caps.v2.capability_bits & VIRGL_CAP_BIND_COMMAND_ARGS)
outbind |= VIRGL_BIND_COMMAND_ARGS;
/* Staging resources should only be created directly through the winsys,
* not using pipe_resources.
*/
assert(!(outbind & VIRGL_BIND_STAGING));
return outbind;
}
......
......@@ -313,14 +313,8 @@ static void virgl_texture_transfer_unmap(struct pipe_context *ctx,
{
struct virgl_context *vctx = virgl_context(ctx);
struct virgl_transfer *trans = virgl_transfer(transfer);
struct virgl_screen *vs = virgl_screen(ctx->screen);
struct pipe_resource *res = transfer->resource;
bool queue_unmap = false;
/* We don't transfer the contents of staging resources, since they don't
* have any host-side storage. */
assert(pipe_to_virgl_bind(vs, res->bind, res->flags) != VIRGL_BIND_STAGING);
if (transfer->usage & PIPE_TRANSFER_WRITE &&
(transfer->usage & PIPE_TRANSFER_FLUSH_EXPLICIT) == 0) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment