Jason Ekstrand <[email protected]> writes:

> From: Scott D Phillips <[email protected]>
>
> These will be used to assign virtual addresses to soft pinned
> buffers in a later patch.
>
> Two allocators are added for separate 'low' and 'high' virtual
> memory areas. Another alternative would have been to add a
> double-sided allocator, which wasn't done here just because it
> didn't appear to give any code complexity advantages.
>
> v2 (Scott Phillips):
>  - rename has_exec_softpin to use_softpin (Jason)
>  - Only remove bottom one page and top 4 GiB from virt (Jason)
>  - refer to comment in anv_allocator about state address + size
>    overflowing 48 bits (Jason)
>  - Mention hi/lo allocators vs double-sided allocator in
>    commit message (Chris)
>  - assign state pool memory ranges statically (Jason)
>
> v3 (Jason Ekstrand):
>  - Use (LOW|HIGH)_HEAP_(MIN|MAX)_ADDRESS rather than (1 << 31) for
>    determining which heap to use in anv_vma_free
>  - Only return de-canonicalized addresses to the heap
>
> Reviewed-by: Jordan Justen <[email protected]>
> Reviewed-by: Jason Ekstrand <[email protected]>

Reviewed-by: Scott D Phillips <[email protected]>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to