Add av1 decode interfaces

[ZefuLI]

add av1 decoding api

[xuguangxin]

whey we need start_tile_idx, it's possible calc if from tile_fow tile_column?

[ZefuLI]

It is the index of the first tile in the tile group of which the current tile belongs to. The index is in raster scan order of all tiles in frame. For the first tile group, this value should be 0. This parameter can be used to uniquely identify the tile group.

[xuguangxin]

could you explain more about this? Is equals to tg_start and tg_end?
If yes, why we need tile_row and tile_column?

[ZefuLI]

they are tg_start and tg_end, and will use these names.

[xuguangxin]

so why we need tile_row and tile_column? can we calc it from tg_start and tg_end?

thanks

[ZefuLI]

tile_row/tile_column is tile row/column index, which is frame level , we need it since AV1 is tile based decoding, we use tile index info to verify which the current tile it is.

[rosetta-jpn]

Sorry for interrupting.
When is the estimated time of checking in this patch?

[sreerenjb]

As Haihao already mentioned, I would like to see the use of VASliceParameterBufferXXX instead of VABitStreamParameterBufferXXX and also
%s/bit_stream_data_size/slice_data_size
%s/bit_stream_data_offset/slice_data_offset
%s/bit_stream_data_flag/slice_data_flag
Which will make life easier for the middleware implementations .Especially GStreamer using common interface for all codecs.

[ZefuLI]

As Haihao already mentioned, I would like to see the use of VASliceParameterBufferXXX instead of VABitStreamParameterBufferXXX and also
%s/bit_stream_data_size/slice_data_size
%s/bit_stream_data_offset/slice_data_offset
%s/bit_stream_data_flag/slice_data_flag
Which will make life easier for the middleware implementations .Especially GStreamer using common interface for all codecs.

Hi, you can see we are using VASliceParameterBufferAV1 in my latest update.

[XinfengZhang]

what's the difference between these fields of cr and cb, why cb_luma_mult is uint8, but cr_luma_mult is uint16? and I dont like uint8 uint16 uint8, it is better uint8 uint8 uint16

[ZefuLI]

@XinfengZhang
The latest version I uploaded used uint8 uint8 uint16, like,
uint8_t cb_mult;
uint8_t cb_luma_mult;
uint16_t cb_offset;
uint8_t cr_mult;
uint8_t cr_luma_mult;
uint16_t cr_offset;

[HeJunyan]

Need to add va_dec_av1.h to va/meson.build file's libva_headers. We still miss that header file in system include dir when apply this patch.

[uartie]

This breaks builds that include installed va/va.h: va/va.h:4630:10: fatal error: va/va_dec_av1.h: No such file or directory. Did you forget to add an install target for va_dec_av1.h?

[HeJunyan]

yes, this patch forget adding va_dec_av1.h to va/meson.build, please help

[uartie]

yes, this patch forget adding va_dec_av1.h to va/meson.build, please help

Actually, the header is added to meson.build in this patch, afaict. But It was not added to Makefile.am (we still use automake in CI). I fixed the automake issue in #383. Does the meson build not work for you?