fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-01-10 01:40:14 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

tc: do a GPU->CPU copy to initialize cpu_storage

Browse source 
If the GPU-side storage has been written to without using cpu_storage,
then we have to initialize the CPU-side storage correctly.

This requires a sync + copy but it's a one time operation so it shouldn't
affect performance much.

I don't think it fixes any existing bug, but the next commit will need
this to behave correctly.

cc: mesa-stable

Reviewed-by: Rob Clark <robdclark@chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18774>
This commit is contained in:
Pierre-Eric Pelloux-Prayer 2022-09-23 12:16:59 +02:00 • committed by Marge Bot
parent 582bf4d9f7
commit 8af8dc97bc
1 changed files with 23 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
src/gallium/auxiliary/util/u_threaded_context.c
View file

@ -2282,9 +2282,31 @@ tc_buffer_map(struct pipe_context *_pipe,
/* We can't let resource_copy_region disable the CPU storage. */
assert(!(tres->b.flags & PIPE_RESOURCE_FLAG_DONT_MAP_DIRECTLY));
if (!tres->cpu_storage)
if (!tres->cpu_storage) {
tres->cpu_storage = align_malloc(resource->width0, tc->map_buffer_alignment);
if (tres->cpu_storage && tres->valid_buffer_range.end) {
/* The GPU buffer contains valid data. Copy them to the CPU storage. */
struct pipe_box box2;
struct pipe_transfer *transfer2;
unsigned valid_range_len = tres->valid_buffer_range.end - tres->valid_buffer_range.start;
u_box_1d(tres->valid_buffer_range.start, valid_range_len, &box2);
tc_sync_msg(tc, "cpu storage GPU -> CPU copy");
tc_set_driver_thread(tc);
void *ret = pipe->buffer_map(pipe, tres->latest ? tres->latest : resource,
0, PIPE_MAP_READ, &box2, &transfer2);
memcpy(&((uint8_t*)tres->cpu_storage)[tres->valid_buffer_range.start],
ret,
valid_range_len);
pipe->buffer_unmap(pipe, transfer2);
tc_clear_driver_thread(tc);
}
}
if (tres->cpu_storage) {
struct threaded_transfer *ttrans = slab_zalloc(&tc->pool_transfers);
ttrans->b.resource = resource;