[FFmpeg-devel] [PATCH v2 1/3] avcodec: make a local copy of executor

[epirat07 at gmail.com]

On 1 Oct 2024, at 9:51, Lynne via ffmpeg-devel wrote:

> On 01/10/2024 08:55, Nuo Mi wrote:
>> We still need several refactors to improve the current VVC decoder's performance,
>> which will frequently break the API/ABI. To mitigate this, we've copied the executor from
>> avutil to avcodec. Once the API/ABI is stable, we will move this class back to avutil
>> ---
>>   libavcodec/Makefile     |   1 +
>>   libavcodec/executor.c   | 221 ++++++++++++++++++++++++++++++++++++++++
>>   libavcodec/executor.h   |  73 +++++++++++++
>>   libavcodec/vvc/thread.c |   2 +-
>>   4 files changed, 296 insertions(+), 1 deletion(-)
>>   create mode 100644 libavcodec/executor.c
>>   create mode 100644 libavcodec/executor.h
>>
>> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
>> index a4fcce3b42..da1a1aa945 100644
>> --- a/libavcodec/Makefile
>> +++ b/libavcodec/Makefile
>> @@ -43,6 +43,7 @@ OBJS = ac3_parser.o                                                     \
>>          dirac.o                                                          \
>>          dv_profile.o                                                     \
>>          encode.o                                                         \
>> +       executor.o                                                       \
>>          get_buffer.o                                                     \
>>          imgconvert.o                                                     \
>>          jni.o                                                            \
>
> You're adding a single field to a structure. This wouldn't break API or ABI. I don't think this is needed.

See patch 2/3 which removes int (*ready)(const AVTask *t, void *user_data); in AVTaskCallbacks.

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".