dav1d split MAX_THREADS

[LigH]

Summary of the bug:
The AV1 decoder dav1d now uses DAV1D_MAX_TILE_THREADS and DAV1D_MAX_FRAME_THREADS instead of a common DAV1D_MAX_THREADS, so compiling mplayer fails.
How to reproduce:

libavcodec/libdav1d.c: At top level:
libavcodec/libdav1d.c:462:94: error: 'DAV1D_MAX_TILE_THREADS' undeclared here (not in a function); did you mean 'DAV1D_MAX_THREADS'?
  462 |     { "tilethreads", "Tile threads", OFFSET(tile_threads), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, DAV1D_MAX_TILE_THREADS, VD },
      |                                                                                              ^~~~~~~~~~~~~~~~~~~~~~
      |                                                                                              DAV1D_MAX_THREADS
libavcodec/libdav1d.c:463:97: error: 'DAV1D_MAX_FRAME_THREADS' undeclared here (not in a function); did you mean 'DAV1D_MAX_THREADS'?
  463 |     { "framethreads", "Frame threads", OFFSET(frame_threads), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, DAV1D_MAX_FRAME_THREADS, VD },
      |                                                                                                 ^~~~~~~~~~~~~~~~~~~~~~~
      |                                                                                                 DAV1D_MAX_THREADS
make[1]: *** [ffbuild/common.mak:67: libavcodec/libdav1d.o] Error 1

[LigH]

log of "make install" recorded by media-autobuild suite

[beastd]

Thanks for reporting this build failure.

Could you paste the contents of dav1d/version.h ?

[LigH]

Without comment header:

#ifndef DAV1D_VERSION_H
#define DAV1D_VERSION_H

#define DAV1D_API_VERSION_MAJOR 6
#define DAV1D_API_VERSION_MINOR 0
#define DAV1D_API_VERSION_PATCH 0

#endif /* DAV1D_VERSION_H */

[beastd]

AFAICT the problem is that your libavcodec version is too low for your libdav1d version.

For your libdav1d version, at least libavcodec from FFmpeg 4.4.1 is required.

Regarding threaded decoding with libdav1d, the current development version of FFmpeg and the upcoming 4.4.2 will be better because it contains a fix and improvements regarding libdav1d threading.

Hope that helps and if possible report back if compiling with newer FFmpeg fixes the build.

[LigH]

I, personally, am not able to change anything; I am not a developer. I only let the media-autobuild suite run and hope that it passes. The best I can do in case of issues is sharing educated guesses with real developers.

The media-autobuild suite maintainers may not take the burden of maintaining the mplayer/mencoder building process, their main goal is building ffmpeg with a lot of modules, and that works with a current libdav1d. I can tell them about it but doubt that there is an easy solution from their side...

In general, the suite builds the most recent modules, it always retrieves HEAD (or known good) sources to build ffmpeg from. But I don't know if the ffmpeg version mplayer will be based on is currently frozen to an older version, maybe...

​M-AB-S issue 2072

[beastd]

I assume this isn't a problem anymore, right?

According to my analysis it isn't an MPlayer problem anyway. Thus closing this ticket with resolution invalid.

Thanks for taking the time to report!