Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

2016-09-30 Thread Nicolai Hähnle
Well, it depends on both the size and the type. Sub-allocation can be used up to 16KB with the current setting, but the override is clearly the right solution here :) Cheers, Nicolai On 30.09.2016 12:24, Christian König wrote: Oh, so you use the allocation type instead of the size to determin

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

2016-09-30 Thread Christian König
Oh, so you use the allocation type instead of the size to determine if a buffer can be sub-allocated or not? I wasn't expecting that, in this case even the DPB would have ended up in the sub-allocator and that would clearly blow up immediately. Good that we stumbled over this. Regards, Chris

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

2016-09-30 Thread Christian König
Patches #1-#3 of this series are Reviewed-by: Christian König . Regards, Christian. Am 30.09.2016 um 11:52 schrieb Nicolai Hähnle: From: Nicolai Hähnle Cc: Christian König Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=979

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

2016-09-30 Thread Nicolai Hähnle
I should add that textures are already never sub-allocated, so with this patch all the buffers used with UVD and VCE should be covered. Nicolai On 30.09.2016 11:52, Nicolai Hähnle wrote: From: Nicolai Hähnle Cc: Christian König Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976 Bu

[Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

2016-09-30 Thread Nicolai Hähnle
From: Nicolai Hähnle Cc: Christian König Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97969 --- src/gallium/drivers/radeon/r600_buffer_common.c | 3 +++ src/gallium/drivers/radeon/radeon_video.c | 8 +++- 2 files c