When a buffer with a GEM handle already existing in our context is
(re-)imported from a PRIME FD, the resulting GEM handle is exactly the
same as the original one. Since the GEM handles are not reference
counted, we need to detect duplicate imports and reference count our
internal buffer structs ou
From: Emil Velikov
As the spec allows for {server,client}_wait_sync to be called without
currently bound context, while our implementation requires context
pointer.
v2: Add a mutex and acquire it for the duration of
brw_fence_client_wait() and brw_fence_is_completed() as suggested
by Cha
[Used wrong CC list originally, sorry ;/]
On Fri, Jul 15, 2016 at 4:27 PM, Tomasz Figa wrote:
> From: Emil Velikov
>
> As the spec allows for {server,client}_wait_sync to be called without
> currently bound context, while our implementation requires context
> pointer.
>
> v2: Add a mutex and acq
It might return NULL if specific config variant is unsupported.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/egl/drivers/dri2/platform_android.c
b/src/egl/drivers/dri2/platform_android.c
index f42febc..a695e00 1
Drivers can request different set of buffers depending on the buffer
mask they pass to the get_buffers callback. This patch makes
droid_image_get_buffers() respect this mask.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 15 +++
1 file changed, 11 insertion
It is much easier to debug issues when the application gives some
meaningful error messages. This patch adds few to the EGL Android
platform backend.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff
Hi,
This series is a collection of various fixes and extensions we came up
with during our attempt to use Mesa for Android.
Fixes included in this series:
- added mandatory EGL_MAX_PBUFFER_WIDTH and _HEIGHT attributes to EGL
configs,
- fixed multiple issues with handling pbuffers in the back
From: Haixia Shi
Set config attributes EGL_MAX_PBUFFER_WIDTH and EGL_MAX_PBUFFER_HEIGHT to
hard-coded non-zero values. These two attributes are required on Android.
Signed-off-by: Tomasz Figa
---
src/egl/drivers/dri2/platform_android.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/
Current implementation of the DRI image loader does not free the images
created in get_back_bo() and so leaks memory. Moreover, it creates a new
image every time the DRI driver queries for buffers, even if the backing
native buffer has not changed. leaking memory again.
This patch adds missing cal
There are DRI_IMAGE_FOURCC macros, for which there are no corresponding
DRI_IMAGE_FORMAT macros. To support such formats we need to make the
lookup function take the native format directly. As a side effect, it
simplifies all existing calls to this function, because they all called
get_format() fir
From: Nicolas Boichat
Existing image loader code supports creating images only for window
surfaces. Moreover droid_create_surface() passes wrong surface type to
dri2_get_dri_config(), resulting in incorrect configs being returned for
pbuffers. This patch fixes these issues.
In addition, the conf
We can support render nodes alone without any private headers, so let's
make support for control nodes depend on presence of private drm_gralloc
headers.
Signed-off-by: Tomasz Figa
---
src/egl/Android.mk | 1 +
src/egl/drivers/dri2/egl_dri2.h | 2 +
src/egl/drive
This patch adds support for YV12 pixel format to the Android platform
backend. Only creating EGL images is supported, it is not added to the
list of available visuals.
Signed-off-by: Tomasz Figa
Signed-off-by: Kalyan Kondapally
---
src/egl/drivers/dri2/platform_android.c | 82 ++
If no hardware driver is present, it is possible to fall back to
the kms_swrast driver with any DRI node that supports dumb GEM create
and mmap IOCTLs with softpipe/llvmpipe drivers. This patch makes the
Android EGL platform code retry probe with kms_swrast if hardware-only
probe fails.
Signed-off
Please ignore this patch, I'll resend along with some slightly
improved/annotated EGL error handling.
On Thu, Jul 14, 2016 at 12:22 PM, Nicolas Boichat wrote:
> From: Nicolas Boichat
>
> Without this, if a configuration is, say, available only on GLES2/3, but
> not on GLES1, and is rejected by t
From: Nicolas Boichat
Without this, if a configuration is, say, available only on GLES2/3, but
not on GLES1, and is rejected by the dri module's bindContext call,
eglMakeCurrent fails with error "EGL_SUCCESS".
In this patch, we set error to EGL_BAD_MATCH, which is what CTS/dEQP
dEQP-EGL.function
android.opengl.cts.WrapperTest#testGetIntegerv1 CTS test calls
eglTerminate, followed by eglReleaseThread. In that case, the
display will not be initialized when eglReleaseThread calls
MakeCurrent with NULL parameters, to unbind the context, which
causes a a segfault in drv->API.MakeCurrent (dri2_m
On 14/07/16 21:24, Francisco Jerez wrote:
> Alejandro Piñeiro writes:
>
>> Without this commit, a image is considered valid if the level of the
>> texture bound to the image is complete, something we can check as mesa
>> save independently if it is "base incomplete" of "mipmap incomplete".
>>
>> B
From the OpenGL 4.3 Core Specification, section 8.25 ("Texture
Image Loads and Stores"):
"An access is considered invalid if:
the texture bound to the selected image unit is incomplete;"
This fixes:
GL44-CTS.shader_image_load_store.incomplete_textures
v2: use _mesa_is_texture_complete, mai
Our indirect URB read messages take both a direct and an indirect offset
so when we emit the second message for a 64-bit input load we can just
always incremement the immediate offset, even for the indirect case.
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 8 +---
1 file changed, 1 insertio
We totally ignored this before because there were no piglit tests for
indirect loads in tessellation stages with doubles.
Cc: "12.0"
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 86
1 file changed, 64 insertions(+), 22 deletions(-)
diff --git a/src/mesa/driver
Hi,
I am developing an application which uses vaapi and egl. My goal is to use
vaDeriveImage and vaAcquireBufferHandle to get the drm buffer id which then can
be used to create a EGLImageKHR. My problem is, that on amd hardware (AMD
GX-212ZC SOC with Radeon(TM) R1E Graphics and AMD GX-424CC SO
On Thursday, July 14, 2016 9:20:05 PM PDT Jason Ekstrand wrote:
> This renames BLEND_STATE to BLEND_STATE_ENTRY and adds an new struct
> BLEND_STATE which is just an array of 8 BLEND_STATE_ENTRYs. This will make
> it much easier to write gen-agnostic blend handling code.
>
> Signed-off-by: Jason
On Fri, 2016-07-15 at 11:04 +0200, Iago Toral Quiroga wrote:
> We totally ignored this before because there were no piglit tests for
> indirect loads in tessellation stages with doubles.
>
> Cc: "12.0"
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 86
>
> 1
On Fri, 2016-07-15 at 20:59 +1000, Timothy Arceri wrote:
> On Fri, 2016-07-15 at 11:04 +0200, Iago Toral Quiroga wrote:
> > We totally ignored this before because there were no piglit tests
> > for
> > indirect loads in tessellation stages with doubles.
> >
> > Cc: "12.0"
> > ---
> > src/mesa/dr
Whilst I don't know that code as well as I'd like. It does get my
Tested-by: Rhys Kidd
for fixing that piglit on i965 ILK. Thanks for the two patches!
On 15 July 2016 at 04:28, Nicolas Boichat wrote:
> android.opengl.cts.WrapperTest#testGetIntegerv1 CTS test calls
> eglTerminate, followed by
On Fri, 2016-07-15 at 20:59 +1000, Timothy Arceri wrote:
> On Fri, 2016-07-15 at 11:04 +0200, Iago Toral Quiroga wrote:
> >
> > We totally ignored this before because there were no piglit tests
> > for
> > indirect loads in tessellation stages with doubles.
> >
> > Cc: "12.0"
> > ---
> > src/me
On Fri, 2016-07-15 at 13:16 +0200, Iago Toral wrote:
> On Fri, 2016-07-15 at 20:59 +1000, Timothy Arceri wrote:
> > On Fri, 2016-07-15 at 11:04 +0200, Iago Toral Quiroga wrote:
> > >
> > > We totally ignored this before because there were no piglit tests
> > > for
> > > indirect loads in tessellat
On Fri, 2016-07-15 at 21:20 +1000, Timothy Arceri wrote:
> On Fri, 2016-07-15 at 13:16 +0200, Iago Toral wrote:
> >
> > On Fri, 2016-07-15 at 20:59 +1000, Timothy Arceri wrote:
> > >
> > > On Fri, 2016-07-15 at 11:04 +0200, Iago Toral Quiroga wrote:
> > > >
> > > >
> > > > We totally ignored th
On 15 July 2016 at 09:28, Nicolas Boichat wrote:
> android.opengl.cts.WrapperTest#testGetIntegerv1 CTS test calls
> eglTerminate, followed by eglReleaseThread. In that case, the
> display will not be initialized when eglReleaseThread calls
> MakeCurrent with NULL parameters, to unbind the context,
Christian König wrote:
Am 05.07.2016 um 13:17 schrieb Christian König:
Am 01.07.2016 um 18:18 schrieb Andy Furniss:
Christian König wrote:
Am 01.07.2016 um 18:02 schrieb Andy Furniss:
Christian König wrote:
Am 01.07.2016 um 16:42 schrieb Andy Furniss:
Boyuan Zhang wrote:
Signed-off-by: Boy
Hi Andy,
Thanks for the testing. I will look into the memory issue. For the cbr test,
what was the bitrate you set? And by saying testing high rates, how high was
the rate approximately?
Regards,
Boyuan
-Original Message-
From: Andy Furniss [mailto:adf.li...@gmail.com]
Sent: July-15-1
On Fri, Jul 15, 2016 at 3:38 AM, Kenneth Graunke
wrote:
> On Thursday, July 14, 2016 9:20:05 PM PDT Jason Ekstrand wrote:
> > This renames BLEND_STATE to BLEND_STATE_ENTRY and adds an new struct
> > BLEND_STATE which is just an array of 8 BLEND_STATE_ENTRYs. This will
> make
> > it much easier t
On Thu, Jul 14, 2016 at 09:20:08PM -0700, Jason Ekstrand wrote:
> The old calculation, which used view->offset, encorporated buffer->offset
> into the size calculation where it doesn't belong. This meant that, if
> buffer->offset > buffer->size, you would always get a negative size. This
> fixes
https://bugs.freedesktop.org/show_bug.cgi?id=96943
Bug ID: 96943
Summary: [gallium] glCopyPixels is affected by enabled texture
state
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NE
So we can use it for computing size of proxy textures.
---
src/mesa/main/mipmap.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/mipmap.c b/src/mesa/main/mipmap.c
index 5ff53f4..996e3f8 100644
--- a/src/mesa/main/mipmap.c
+++ b/src/mesa/main/mipmap.c
@@
So that the function can work properly with glTexStorage(), where we know
how many mipmap levels there are. And so we can compute storage for MSAA
textures.
Also, remove the obsolete texture border parameter.
A subsequent patch will update _mesa_test_proxy_teximage() to use these
new parameters.
This avoids a failed assert(img->_BaseFormat != -1) in
init_teximage_fields_ms() because the internalFormat argument is GL_NONE.
This was hit when using glTexStorage() to do a proxy texture test.
Fixes a failure with the updated Piglit tex3d-maxsize test.
Cc:
---
src/mesa/main/texstorage.c | 4
If numSamples > 0, we can compute the size of the whole mipmapped texture.
That's the case for glTexStorage(GL_PROXY_TEXTURE_x).
Also, multiply the texture size by numSamples for MSAA textures.
---
src/mesa/main/teximage.c | 48 +---
1 file changed, 45
On Thu, Jul 14, 2016 at 04:18:33PM -0700, Jason Ekstrand wrote:
> On Thu, Jul 14, 2016 at 4:13 PM, Nanley Chery wrote:
>
> > On Thu, Jul 14, 2016 at 03:57:09PM -0700, Jason Ekstrand wrote:
> > > On Thu, Jul 14, 2016 at 3:45 PM, Nanley Chery
> > wrote:
> > >
> > > > On Sat, Jul 09, 2016 at 12:17:
Zhang, Boyuan wrote:
Hi Andy,
Thanks for the testing. I will look into the memory issue. For the
cbr test, what was the bitrate you set? And by saying testing high
rates, how high was the rate approximately?
Testing more shows that I get the same size file independent of
framerate. An example
Tomasz Figa writes:
> It might return NULL if specific config variant is unsupported.
>
> Signed-off-by: Tomasz Figa
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://
Tomasz Figa writes:
> From: Haixia Shi
>
> Set config attributes EGL_MAX_PBUFFER_WIDTH and EGL_MAX_PBUFFER_HEIGHT to
> hard-coded non-zero values. These two attributes are required on Android.
>
> Signed-off-by: Tomasz Figa
> ---
> src/egl/drivers/dri2/platform_android.c | 2 ++
> 1 file chang
Tomasz Figa writes:
> It is much easier to debug issues when the application gives some
> meaningful error messages. This patch adds few to the EGL Android
> platform backend.
>
> Signed-off-by: Tomasz Figa
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
On Fri, Jul 15, 2016 at 9:57 AM, Nanley Chery wrote:
> On Thu, Jul 14, 2016 at 09:20:08PM -0700, Jason Ekstrand wrote:
> > The old calculation, which used view->offset, encorporated buffer->offset
> > into the size calculation where it doesn't belong. This meant that, if
> > buffer->offset > buf
[Adding Haixia to the thread.]
On Sat, Jul 16, 2016 at 2:52 AM, Eric Anholt wrote:
> Tomasz Figa writes:
>
>> From: Haixia Shi
>>
>> Set config attributes EGL_MAX_PBUFFER_WIDTH and EGL_MAX_PBUFFER_HEIGHT to
>> hard-coded non-zero values. These two attributes are required on Android.
>>
>> Signe
Tomasz Figa writes:
> Current implementation of the DRI image loader does not free the images
> created in get_back_bo() and so leaks memory. Moreover, it creates a new
> image every time the DRI driver queries for buffers, even if the backing
> native buffer has not changed. leaking memory again
https://bugs.freedesktop.org/show_bug.cgi?id=96943
--- Comment #1 from Roland Scheidegger ---
(In reply to Ilia Mirkin from comment #0)
> The recently rewritten copy-pixels test has exposed some failures in
> st/mesa. When there's an overlapping copy (among other conditions), the
> operation beco
Hi Eric,
On Sat, Jul 16, 2016 at 3:05 AM, Eric Anholt wrote:
> Tomasz Figa writes:
>
>> Current implementation of the DRI image loader does not free the images
>> created in get_back_bo() and so leaks memory. Moreover, it creates a new
>> image every time the DRI driver queries for buffers, even
https://bugs.freedesktop.org/show_bug.cgi?id=96943
--- Comment #2 from Ilia Mirkin ---
(In reply to Roland Scheidegger from comment #1)
> The piglit test is probably wrong (I didn't really look), at least different
> results based on enabled texturing don't surprise me. This is one of the
> more
Tomasz Figa writes:
> From: Nicolas Boichat
>
> Existing image loader code supports creating images only for window
> surfaces. Moreover droid_create_surface() passes wrong surface type to
> dri2_get_dri_config(), resulting in incorrect configs being returned for
> pbuffers. This patch fixes the
Tomasz Figa writes:
> There are DRI_IMAGE_FOURCC macros, for which there are no corresponding
> DRI_IMAGE_FORMAT macros. To support such formats we need to make the
> lookup function take the native format directly. As a side effect, it
> simplifies all existing calls to this function, because th
Tomasz Figa writes:
> This patch adds support for YV12 pixel format to the Android platform
> backend. Only creating EGL images is supported, it is not added to the
> list of available visuals.
>
> Signed-off-by: Tomasz Figa
> Signed-off-by: Kalyan Kondapally
> ---
> src/egl/drivers/dri2/platf
https://bugs.freedesktop.org/show_bug.cgi?id=96943
--- Comment #3 from Roland Scheidegger ---
(In reply to Ilia Mirkin from comment #2)
> (In reply to Roland Scheidegger from comment #1)
> > The piglit test is probably wrong (I didn't really look), at least different
> > results based on enabled
Tomasz Figa writes:
> If no hardware driver is present, it is possible to fall back to
> the kms_swrast driver with any DRI node that supports dumb GEM create
> and mmap IOCTLs with softpipe/llvmpipe drivers. This patch makes the
> Android EGL platform code retry probe with kms_swrast if hardware
Tomasz Figa writes:
> Hi Eric,
>
> On Sat, Jul 16, 2016 at 3:05 AM, Eric Anholt wrote:
>> Tomasz Figa writes:
>>
>>> Current implementation of the DRI image loader does not free the images
>>> created in get_back_bo() and so leaks memory. Moreover, it creates a new
>>> image every time the DRI
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_state.h| 9 ++
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 184
> +++
> 2 files changed, 193 insertions(+)
> +void
> +brw_update_texture_su
Andy Furniss wrote:
omx doesn't have this issue, for 30mbit the files produced are 53M
and 26M.
Actually by default omx will make the same size files, I lucked into that
behavior playing around with the control-rate param.
___
mesa-dev mailing list
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_state.h | 7 -
> src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 194
> +-
> 2 files changed, 1 insertion(+), 200 deletions(-)
More --'s !
Patch
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_binding_tables.c| 2 +-
> src/mesa/drivers/dri/i965/brw_context.h | 8 --
> src/mesa/drivers/dri/i965/brw_state.h | 9 +++
> src/mesa/drivers/dri/i965/brw_w
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 11 ++-
> src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 9 +
> src/mesa/drivers/dri/i965/gen8_surface_state.c| 9 +
> 3 files c
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 94
> +---
> 1 file changed, 1 insertion(+), 93 deletions(-)
Patches 28 and 29 are
Reviewed-by: Chad Versace
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_state.h | 16 --
> src/mesa/drivers/dri/i965/gen8_surface_state.c | 249
> +
> 2 files changed, 2 insertions(+), 263 deletions(-)
That diff has a lot
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> The only useful thing left was gen6_init_vtable_surface_functions which we
> can easily put in brw_wm_surface_state.c.
>
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/Makefile.sources | 1 -
> src/mesa/drivers/dri/i965/brw_wm
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_context.h | 17 -
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 3 +--
> src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 1 -
> src/mesa/drivers/
On Thu 14 Jul 2016, Jason Ekstrand wrote:
> On Thu, Jul 14, 2016 at 2:26 PM, Chad Versace
> wrote:
>
> > On Wed 13 Jul 2016, Jason Ekstrand wrote:
> > > Reviewed-by: Topi Pohjolainen
> > > ---
> > > src/mesa/drivers/dri/i965/brw_blorp.c | 157
> > ++
> > > src/me
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/gen6_blorp.c | 76
> +-
> 1 file changed, 2 insertions(+), 74 deletions(-)
Patches 14-18 are
Reviewed-by: Chad Versace
That's the whole series, now. Did
Hi Mauro,
On 14 July 2016 at 04:33, Mauro Rossi wrote:
> Sending patches to implement DRI2_Fence extension for i915,
> to enable EGL_KHR_fence_sync and EGL_KHR_wait_sync.
>
> It is the step-by-step porting to i915 of i965/sync,
> plus patch to support {server,client}_wait_sync without bound conte
On Fri, Jul 15, 2016 at 10:14 AM, Brian Paul wrote:
> If numSamples > 0, we can compute the size of the whole mipmapped texture.
> That's the case for glTexStorage(GL_PROXY_TEXTURE_x).
>
> Also, multiply the texture size by numSamples for MSAA textures.
> ---
> src/mesa/main/teximage.c | 48
> ++
On 15 July 2016 at 18:57, Tomasz Figa wrote:
> [Adding Haixia to the thread.]
>
> On Sat, Jul 16, 2016 at 2:52 AM, Eric Anholt wrote:
>> Tomasz Figa writes:
>>
>>> From: Haixia Shi
>>>
>>> Set config attributes EGL_MAX_PBUFFER_WIDTH and EGL_MAX_PBUFFER_HEIGHT to
>>> hard-coded non-zero values.
On 07/15/2016 01:19 PM, Anuj Phogat wrote:
On Fri, Jul 15, 2016 at 10:14 AM, Brian Paul wrote:
If numSamples > 0, we can compute the size of the whole mipmapped texture.
That's the case for glTexStorage(GL_PROXY_TEXTURE_x).
Also, multiply the texture size by numSamples for MSAA textures.
---
2016-07-15 21:18 GMT+02:00 Emil Velikov :
> Hi Mauro,
>
> On 14 July 2016 at 04:33, Mauro Rossi wrote:
> > Sending patches to implement DRI2_Fence extension for i915,
> > to enable EGL_KHR_fence_sync and EGL_KHR_wait_sync.
> >
> > It is the step-by-step porting to i915 of i965/sync,
> > plus patc
On 15 July 2016 at 08:53, Tomasz Figa wrote:
> Drivers can request different set of buffers depending on the buffer
> mask they pass to the get_buffers callback. This patch makes
> droid_image_get_buffers() respect this mask.
>
> Signed-off-by: Tomasz Figa
> ---
> src/egl/drivers/dri2/platform_a
On Fri, Jul 15, 2016 at 12:34 PM, Brian Paul wrote:
> On 07/15/2016 01:19 PM, Anuj Phogat wrote:
>>
>> On Fri, Jul 15, 2016 at 10:14 AM, Brian Paul wrote:
>>>
>>> If numSamples > 0, we can compute the size of the whole mipmapped
>>> texture.
>>> That's the case for glTexStorage(GL_PROXY_TEXTURE_x
Some copyrights are not correct, please look at the log of
https://github.com/chrisbmr/Mesa-3D/tree/gallium-nine/src/gallium/state_trackers/nine
to find out.
Axel
On 14/07/2016 20:17, Vedran Miletić wrote :
Few files were missing the copyright headers. This patch adds correct
copyright headers
Alejandro Piñeiro writes:
> On 14/07/16 21:24, Francisco Jerez wrote:
>> Alejandro Piñeiro writes:
>>
>>> Without this commit, a image is considered valid if the level of the
>>> texture bound to the image is complete, something we can check as mesa
>>> save independently if it is "base incomple
On Thu 14 Jul 2016, Chad Versace wrote:
> On Wed 13 Jul 2016, Jason Ekstrand wrote:
> > Reviewed-by: Topi Pohjolainen
> > ---
> > src/mesa/drivers/dri/i965/brw_state.h| 8 +++
> > src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 79
> >
> > 2 files changed
On Wed 13 Jul 2016, Jason Ekstrand wrote:
> Reviewed-by: Topi Pohjolainen
> ---
Reviewed-by: Chad Versace
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 15 July 2016 at 08:27, Tomasz Figa wrote:
> When a buffer with a GEM handle already existing in our context is
> (re-)imported from a PRIME FD, the resulting GEM handle is exactly the
> same as the original one. Since the GEM handles are not reference
> counted, we need to detect duplicate impo
Add entrypoint to distinguish H.264 decode and encode. For example, in patch
5/11 when is calling "VaCreateContext", "pps" and "sps" shouldn't be allocated
for H.264 encoding. So we need to use the entry_point to determine this is
H.264 decode or H.264 encode. We can use config to determine the
For putimage call, if image format is yv12 (or IYUV with U V field swap) and
surface format is nv12, then we need to convert yv12 to nv12 and then copy the
converted data from image to surface. We can't use the existing logic where
surface is destroyed and re-created with yv12 format.
Signed-of
Add function to copy from yv12 image to nv12 surface for VAAPI putimage call.
We need this function in VaPutImage call where copying from yv12 image to nv12
surface for encoding. Existing function can't be used because it only work for
copying from yv12 surface to nv12 image in Vaapi.
Signed-of
VAAPI passes PIPE_VIDEO_ENTRYPOINT_ENCODE as entry point for encoding case. We
will save this encode entry point in config. config_id was used as profile
previously. Now, config has both profile and entrypoint field, and config_id is
used to get the config object. Later on, we pass this entrypoi
Add necessary functions/changes for VAAPI encoding to buffer and picture. These
changes will allow driver to handle all Vaapi encode related operations. This
patch doesn't change the Vaapi decode behaviour.
Signed-off-by: Boyuan Zhang
---
src/gallium/state_trackers/va/buffer.c | 6 ++
sr
Rate control method is passed from app to driver through config attrib list.
That is why we need to store this rate control method to config. And later on,
we will pass this value to context->desc.h264enc.rate_ctrl.rate_ctrl_method.
Signed-off-by: Boyuan Zhang
---
src/gallium/state_trackers/va
Enable H.264 VAAPI encoding through config. Currently only H.264 baseline is
supported.
Signed-off-by: Boyuan Zhang
---
src/gallium/state_trackers/va/config.c | 32 ++--
1 file changed, 22 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/va/con
Add some hardcoded values hardware needs mainly for rate control purpose. With
previously hardcoded values for OMX, the rate control result is not correct.
This change fixed the rate control result by setting correct values for Vaapi.
Signed-off-by: Boyuan Zhang
---
src/gallium/state_trackers/
Hi Andy,
The memory issue is fixed. Please try the new patch set I just submitted.
Should NOT have hard lock anymore.
And thanks for providing the rate control info, I will do some test about it.
Regards,
Boyuan
-Original Message-
From: Andy Furniss [mailto:adf.li...@gmail.com]
Sent:
Signed-off-by: Jason Ekstrand
Cc: "12.0"
Cc: Connor Abbott
---
src/compiler/nir/nir.c | 107 +++
src/compiler/nir/nir.h | 4 ++
src/compiler/nir/nir_lower_returns.c | 9 +--
3 files changed, 116 insertions(+), 4 deletions(-)
diff
Signed-off-by: Jason Ekstrand
Cc: "12.0"
Cc: Connor Abbott
---
src/compiler/nir/nir_inline_functions.c | 41 +++--
1 file changed, 39 insertions(+), 2 deletions(-)
diff --git a/src/compiler/nir/nir_inline_functions.c
b/src/compiler/nir/nir_inline_functions.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=96949
Bug ID: 96949
Summary: [regression] Piglit numSamples assertion failures with
9a23a177b90
Product: Mesa
Version: git
Hardware: Other
OS: All
Sta
On Fri, Jul 15, 2016 at 2:30 PM, Chad Versace
wrote:
> On Thu 14 Jul 2016, Chad Versace wrote:
> > On Wed 13 Jul 2016, Jason Ekstrand wrote:
> > > Reviewed-by: Topi Pohjolainen
> > > ---
> > > src/mesa/drivers/dri/i965/brw_state.h| 8 +++
> > > src/mesa/drivers/dri/i965/brw_wm_surf
On Wed, Jul 13, 2016 at 10:50 AM, Vedran Miletić wrote:
> On 07/13/2016 05:19 AM, Timothy Arceri wrote:
>>
>> So I finally got around to setting up my new polaris card on fedora. I
>> was curious to see how Talos performed compared to i965 as I was pretty
>> sure the compiling/linking was not just
https://bugs.freedesktop.org/show_bug.cgi?id=96949
--- Comment #1 from Brian Paul ---
I'd appreciate it if you could debug this a bit further. I didn't see any of
those regressions here.
I'm on vacation for the next week so I won't be able to do anything. Feel free
to revert the patch if you c
https://bugs.freedesktop.org/show_bug.cgi?id=96950
Bug ID: 96950
Summary: Another regression from bc4e0c486: vbo: Use a bitmask
to track the active arrays in vbo_exec*.
Product: Mesa
Version: unspecified
Hardware: Other
https://bugs.freedesktop.org/show_bug.cgi?id=96950
Brian Paul changed:
What|Removed |Added
CC||mathias.froehl...@web.de
--
You are receiv
https://bugs.freedesktop.org/show_bug.cgi?id=96950
--- Comment #1 from Rob Clark ---
Do you still see the issue after 64d35f81 "vbo: fix attr reset"?
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=96950
Rob Clark changed:
What|Removed |Added
CC||robcl...@freedesktop.org
--
You are receivi
https://bugs.freedesktop.org/show_bug.cgi?id=96950
--- Comment #2 from Brian Paul ---
(In reply to Rob Clark from comment #1)
> Do you still see the issue after 64d35f81 "vbo: fix attr reset"?
Yes.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee
Zhang, Boyuan wrote:
Hi Andy,
The memory issue is fixed. Please try the new patch set I just submitted.
Should NOT have hard lock anymore.
Seems good, no leak and no lock :-)
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.fr
1 - 100 of 114 matches
Mail list logo