Hi,
This is my latest attempt to fix the mapping bug and, so far,
it seems that it is resolved.
This series changes completely how OpenCL buffers are handled
by the r600g driver. Before this, we would add them directly to
a pool, and this pool would grow whenever we needed more space.
But this process implied destroying the pool and creating a new
one. There could be cases where a buffer would be mapped and
the pool would grow, leaving one side of the mapping pointed
to where the item was. This is the 'mapping bug'
Now, Items will have an intermediate resource, where all mappings
can be done, and when a buffer is going to be used with a kernel
it is promoted to the pool. In the case where a promoted item
is going to be mapped, it is previously demoted, so even if
the pool changes its location due to growing, the map remains
valid. In the case of a buffer mapped for reading, and used
by a kernel to read from it, we will duplicate this buffer,
having the intermediate buffer, where the user has its map, and
an item in the pool, which is the one that the kernel is going
to use.
As a summary:
Patches 1-7: These are the main part of the series, and solve
the mapping bug.
Patch 8: Introduces some utils for managing the pool's lists
Patch 9: This is just a proof of concept for avoiding transfers
GPU <-> GPU when using all CL Read/Write functions.
In fact, it CAN'T land as it is because PIPE_TRANSFER_MAP_DIRECTLY
collides with CL_MAP_WRITE_INVALIDATE_REGION
Please review and Thanks :)
Bruno Jiménez (9):
r600g/compute: Add an intermediate resource for OpenCL buffers
r600g/compute: Add statuses to the compute_memory_items
r600g/compute: divide the item list in two
r600g/compute: only move to the pool the buffers marked for promoting
r600g/compute: avoid problems when promoting items mapped for reading
r600g/compute: implement compute_memory_demote_item
r600g/compute: map only against intermediate buffers
r600g/compute: add util functions to add and remove items from lists
r600g/compute: avoid demoting items when reading/writing
src/gallium/drivers/r600/compute_memory_pool.c | 313 +++++++++++++--------
src/gallium/drivers/r600/compute_memory_pool.h | 18 +-
src/gallium/drivers/r600/evergreen_compute.c | 38 ++-
src/gallium/state_trackers/clover/api/transfer.cpp | 4 +-
.../state_trackers/clover/core/resource.cpp | 2 +
5 files changed, 255 insertions(+), 120 deletions(-)
--
2.0.0
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
