Re: [Mesa-dev] [PATCH 01/13] intel/blorp: Assert on subresource in surface_info_init

2017-06-20 Thread Nanley Chery
On Mon, Jun 19, 2017 at 04:05:04PM -0700, Jason Ekstrand wrote: > General comment: "Subresource" is not a term that we've historically used > outside of Vulkan. Maybe better to just say "intel/blorp: Assert that > levels and layers are in range" > Looks like I'll have to send out a V2, so I'll

Re: [Mesa-dev] [PATCH 01/13] intel/blorp: Assert on subresource in surface_info_init

2017-06-19 Thread Jason Ekstrand
General comment: "Subresource" is not a term that we've historically used outside of Vulkan. Maybe better to just say "intel/blorp: Assert that levels and layers are in range" On Tue, Jun 13, 2017 at 11:41 AM, Nanley Chery wrote: > Signed-off-by: Nanley Chery

[Mesa-dev] [PATCH 01/13] intel/blorp: Assert on subresource in surface_info_init

2017-06-13 Thread Nanley Chery
Signed-off-by: Nanley Chery --- src/intel/blorp/blorp.c | 4 1 file changed, 4 insertions(+) diff --git a/src/intel/blorp/blorp.c b/src/intel/blorp/blorp.c index 9c88658e8a..993b42fa85 100644 --- a/src/intel/blorp/blorp.c +++ b/src/intel/blorp/blorp.c @@ -66,6