Age | Commit message (Collapse) | Author | Files | Lines |
|
See https://github.com/01org/libva
Signed-off-by: Sean V Kelley <seanvk@posteo.de>
|
|
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99375
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Instead the user must call vaDestroyBuffer() to destroy a buffer explicitly.
If following the previous API specification,
1. Violate "who allocate who release" principle
2. The user cannot re-use VA buffer flexibly
3. The user still has to call vaDestroyBuffer() to destroy the buffers which
are not going to be passed to vaRenderPicture()
We discussed the change at https://bugs.freedesktop.org/show_bug.cgi?id=97970
v2: bump version to 0.40 because this is a incompatible change to VA-API
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Conflicts:
configure.ac
va/va.h
|
|
Signed-off-by: Mark Thompson <sw@jkqxz.net>
|
|
If we specify compiling option with "-Werror=strict-prototypes", we
will get the following error:
va/va.h:294:1: error: function declaration isn't a prototype [-Werror=strict-prototypes]
typedef int (*VAPrivFunc)();
This patch declares the VAPrivFunc with parameter 'void', to make the
compiler happy.
Signed-off-by: Li Zhijian <zhijianx.li@intel.com>
|
|
I010 is another kind of 10-bit format. Its layout is similar to IYUV(I420).
And the lower 10-bit is used.
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Reviewed-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
If we do not use a render node we must authenticate. Doing the extra
GetClient calls/ioctls does not help much, so don't bother.
Cc: David Herrmann <dh.herrmann@gmail.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
By passing divisor=0, we imply we do not care about synchronisation of
this request to the vertical refresh - the spec says that if we miss the
target frame, the swap will be presented as quickly as possible and may
forgo waiting until the next vblank. By using divisor=1, we request that
the swap be presented upon the vertical refresh immediately following
recipe, enforcing the synchronisation to vblank and avoiding tearing.
Reported-and-tested-by: Lukas Hejtmanek <xhejtman@ics.muni.cz>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
(cherry picked from commit 3b7e4999950a04fabd42edbead8c2f24c6cdf3cf)
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
(cherry picked from commit a55ea7cb3143d57c8dba1b76ccea3511ea69adf2)
|
|
Update wayland-drm-client-protocol.h as well
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
(cherry picked from commit d8719fb8b2d999c3de64def3a801778021c71f84)
|
|
It is based on VA-API
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
(cherry picked from commit 7aa2dd979a3758e1f6378344b235595c5028a9e7)
|
|
To express a frame rate, such as 7.5, we have to use 'numerator / denominator'.
The high 2 bytes will be used as denominator and the low 2 bytes will be used
as numerator.
To keep backward compatible, the low 2 bytes is the real frame rate when the high
2 bytes is 0.
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
(cherry picked from commit 1ca62cedd41ed2061c3a9ddafadc59a4692a07c9)
|
|
Cherry-pick'ed from:
e6f92ae: Added geneirc support for encoding bit-rate control per temporal layer
753ec17: va.h: add data structure for the structure of temporal layer encoding
v2: Update the comment that user must send VAEncMiscParameterTemporalLayerStructure before
VAEncMiscParameterRateControl and VAEncMiscParameterFrameRate and the condition for error return
Signed-off-by: Jonathan Bian <jonathan.bian@intel.com>
Signed-off-by: Austin Yuan <shengquan.yuan@intel.com>
Signed-off-by: Zhangfei Zhang <zhangfei.zhang@intel.com>
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
(cherry picked from commit d12fc9823f3c09a834a071c2327ef8f9c1352649)
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
(cherry picked from commit 2ebf897d51f1ac4e6620a830a402af0faa8df3ae)
Conflicts:
configure.ac
|
|
This lets any application using libva choose the best way to report
info and error messages to the user, for example graphical application
can open a popup on errors and write info messages in the toolbar.
Signed-off-by: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Update wayland-drm-client-protocol.h as well
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Conflicts:
configure.ac
|
|
It is based on VA-API
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
To express a frame rate, such as 7.5, we have to use 'numerator / denominator'.
The high 2 bytes will be used as denominator and the low 2 bytes will be used
as numerator.
To keep backward compatible, the low 2 bytes is the real frame rate when the high
2 bytes is 0.
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
Cherry-pick'ed from:
e6f92ae: Added geneirc support for encoding bit-rate control per temporal layer
753ec17: va.h: add data structure for the structure of temporal layer encoding
v2: Update the comment that user must send VAEncMiscParameterTemporalLayerStructure before
VAEncMiscParameterRateControl and VAEncMiscParameterFrameRate and the condition for error return
Signed-off-by: Jonathan Bian <jonathan.bian@intel.com>
Signed-off-by: Austin Yuan <shengquan.yuan@intel.com>
Signed-off-by: Zhangfei Zhang <zhangfei.zhang@intel.com>
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
|
|
v1:
add --roi-test for test only for ROI. default only one region(0,0,120,120) has been test.
v2:
add ROI region size check to ensure region width/height < frame_width/4 or frame_height/4 if fram width/height < 120
add ROI attrib check
v3:
check the attrib return value for ROI
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: ceciliapeng <cecilia.peng@intel.com>
Signed-off-by: Pengfei Qu <Pengfei.Qu@intel.com>
(cherry picked from commit e5f8db3434667a044122ad1973459aca0d44903c)
|
|
v1:
add --roi-test for test only for ROI. default only one region(0,0,120,120) has been test.
v2:
add ROI region size check to ensure region width/height < frame_width/4 or frame_height/4 if fram width/height < 120
add ROI attrib check
v3:
check the attrib return value for ROI
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: ceciliapeng <cecilia.peng@intel.com>
Signed-off-by: Pengfei Qu <Pengfei.Qu@intel.com>
|
|
add trace information for VAEncPictureParameterBufferVP9
and VAEncSequenceParameterBufferVP9
also add the case for VAEncMiscParameterBufferType on
va_TraceVP9Buf
Signed-off-by: Daniel Charles <daniel.charles@intel.com>
(cherry picked from commit b27feb9b8e62725fed959c090feb2f1c853f74b3)
|
|
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
(cherry picked from commit 695f99ef0405cf4255e7767b44effb0da2fe706e)
|
|
non-CQP mode
Currently the roi_value in VAEncROI is used as the qp priority when
VAConfigAttribRateControl != VA_RC_CQP. But some customers hope to use
it as qp_delta like CQP mode. In such case one flag is added to indicate
whether it is qp_priority or not. At the same time driver can export it
in VAConfigAttribValEncROI to indicate whether the qp_delta is supported
when VAConfigAttribRateControl != VA_RC_CQP.
v2: update comments in va.h
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: ceciliapeng <cecilia.peng@intel.com>
Signed-off-by: Pengfei Qu <Pengfei.Qu@intel.com>
Signed-off-by: Xiang, Haihao <haihao.xiang@intel.com>
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
(cherry picked from commit 66c63e6eef3836e2243881f111553344edce54a5)
|
|
The function strncpy() does not guarantee to nul terminate the
destination. In most cases, this cannot be triggered, but it is also
used to parse user environment variables. These are allowed to be longer
than 1023 characters, effectively resulting in an unterminated string.
I've adjusted other places as well, because it won't hurt.
https://bugs.freedesktop.org/show_bug.cgi?id=96677
(cherry picked from commit 1517fd276e12cac14c018d5a30792177eb6c59de)
|
|
The getopt_long func is return integer data type is more than 1 bytes.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit def5691f789b897b47af7f2a7ffa3b135a93950d)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit f7e22630bd18ed80d22da20a9dc18487938b6c8a)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 6bf712ad26b11d2caa5d806c1bb31eaef12d382c)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 1f8aa1546182667cf70484b0c08959b08edb5a84)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 3a617e995435eafb87ea87dd1afc647d51d49582)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit c0fa72c00aa661faf51d7c1fb1275ef11c5a9667)
|
|
If the !d->display is true, will exit without free memory
block allocate for 'd'.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 5b185d937f354c462ddfe3dd754f7337dc6538c4)
|
|
Add in missing profile and entrypoint get from va.h
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit de8e9edcf110355d7733f970ff0205e4970465fb)
|
|
The size of array huffman_table only 2 in VAHuffmanTableBufferJPEGBaseline
default_huffman_table_param. The index in 2..3 in huffman_table[x] will
be access invalid memory location in for loop that looping 4 times.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit c56ebc89d19efdbf879a86ef5f0407ac9aad45af)
|
|
The size of array surface_1 is 2, but it passed to test_unique_surfaces
func is 16 or 6 to do compare for surface_16. The index for surface_1
not in the array index range for surface_1 and access invalid memory
location.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit e4535e404d1dc33fb1fddf99f057c341fc652bfd)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 1199b16ebd6488734d43c8d962d1d2d7eb42a37a)
|
|
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 56f90c0dee7de287f9557c59d65f870240b3ada2)
|
|
v2:
Add in second free(NULL) for pic_y and third free(NULL) for pic_u.
Put back 'allocated' missing consider about the pic_y, pic_u and pic_v
will be contains pic_y_old, pic_u_old and pic_v_old will be assign to
pic_y, pic_u and pic_v.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 1c2997307e007c9a999b33da9c237dda370d1990)
|
|
v2:
Remove not necessary check dst_Y against NULL for sencond free(NULL).
Remove not necessary check dst_Y and dst_U against NULL for second free(NULL)
and third free(NULL).
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit 8c0973050d825fb6a0734dd62fdf8b209aee8ad0)
|
|
The dc_values only have 12 bytes and ac_value only 162 bytes but the
memcpy did it for 16 bytes and 256 bytes copying thru hard code value.
To avoid the array index out of bound again, recommend move to use sizeof.
v2:
Fix commit message typo from 265 bytes to 256 bytes.
Signed-off-by: Lim Siew Hoon <siew.hoon.lim@intel.com>
(cherry picked from commit c36778ff264b3c45b538db4bbfe6aea38fcb165e)
|