[vlc-devel] [PATCH 00/19] picture cleaning
remi at remlab.net
Thu Jul 30 18:10:09 CEST 2020
Le torstaina 30. heinäkuuta 2020, 15.54.29 EEST Alexandre Janniaux a écrit :
> I'm not at ease with this patchset, it feels like removing the
> picture_resource_t layer and doing it by hand.
Leaving aside my minor detail comments, I think the patchset makes sense.
The whole point of push-buffers was to uniformise picture buffer allocation.
Everything should be allocated as shared memory that can be manipulated
locally, passed onto compositor with zero copy IPC, and eventually exchanged
with sandboxed decoders orf ilters. picture_NewFromFormat() takes care of
that. Pictures resources should not be needed anymore.
Regardless we still need a mean to wrap opaque picture buffers, which cannot
really use picture_NewFromFormat().
> This value previously held vout data in the pull model, if
> I am not wrong. I still use it in scenario where the vout
> binds an object (eg. wl_buffer) to an underlying object in
> the picture (eg. a GBM buffer).
If we need to handle non-opaque pictures coming from an input peripheral (as
opposed to going to an output peripheral), then we need something like similar
to picture_NewFromResource(). I guess that is your implication?
Even if that use case materialises, picture_NewFromResource() cannot handle it
as it stands: it lacks provision for passing the file descriptor, mapping offset
and size. So that would need a whole new picture_New*() function in any case.
More information about the vlc-devel