[FFmpeg-devel] [PATCH v3 2/2] lavc/vulkan_av1: port to the new stable API

[Dave Airlie]

  >
>  > -    /* Workaround for a spec issue.
>  > -     *Can be removed once no longer needed, and threading can be enabled. */
>  > +    /* TODO: investigate if this can be removed to make decoding completely
>  > +     * independent. */
>  >      FFVulkanDecodeContext          *dec;
>
> Can you explain what the id_alloc_mask thing is doing which needs this?  (The 32 size in particular seems suspicious.)

This is for DPB slot management, 32 is the wrong limit, I think I just
picked uint32_t and bitshifting as a quick option. We should limit
this to maxDpbSlots I suspect, probably still use a uint32_t bitmask
to track it.

Dave.