etnaviv: remove bogus comment about replacing resource storage
authorLucas Stach <l.stach@pengutronix.de>
Fri, 14 Jul 2023 17:01:51 +0000 (19:01 +0200)
committerMarge Bot <emma+marge@anholt.net>
Mon, 17 Jul 2023 09:27:13 +0000 (09:27 +0000)
While we might want to employ this orphaning trick one day to avoid stalls
on busy resources it's certainly not as easy as the comment implies, as
this would involve changing all relocs in cached state objects referencing
the resource.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Christian Gmeiner <cgmeiner@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24164>

src/gallium/drivers/etnaviv/etnaviv_transfer.c

index 6aa2b9a..696e74e 100644 (file)
@@ -108,13 +108,6 @@ etna_transfer_unmap(struct pipe_context *pctx, struct pipe_transfer *ptrans)
    struct etna_resource *rsc = etna_resource(ptrans->resource);
    struct etna_resource_level *res_level = &rsc->levels[ptrans->level];
 
-   /* XXX
-    * When writing to a resource that is already in use, replace the resource
-    * with a completely new buffer
-    * and free the old one using a fenced free.
-    * The most tricky case to implement will be: tiled or supertiled surface,
-    * partial write, target not aligned to 4/64. */
-
    if (rsc->texture && !etna_resource_newer(rsc, etna_resource(rsc->texture)))
       rsc = etna_resource(rsc->texture); /* switch to using the texture resource */