On 2015-12-07 11:00, Marek Olšák wrote:
Hi,

This series addresses the weirdness in radeon drivers that every
buffer has 2 handles:
- pb_buffer (== pb_cache_buffer)
- radeon_winsys_cs_handle (winsys-specific pb_buffer)

The inefficiency of converting pb_cache_buffer into the
winsys-specific buffer made me introduce radeon_winsys_cs_handle a few
years ago, which has been used for almost everything
(map/unmap/command submission/etc.) and pb_buffer has only held the
reference keeping the buffer alive.

Now it's time to do this right.

This series moves the pb_cache_manager logic into an independent
module "pb_cache". Next, all dependencies on pb_manager are removed
from both winsyses and the new module is used instead. The result is
that pb_buffer is equal to radeon_winsys_cs_handle, and the latter can
be removed.

Please review.

Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Nice cleanup.

This series is,

Reviewed-by: Edward O'Callaghan <eocallag...@alterapraxis.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to