[Mesa-dev] [PATCH] gbm: add gbm_{bo, surface}_create_with_modifiers2

[Jason Ekstrand]

On Tue, Jun 25, 2019 at 4:04 AM Daniel Stone <daniel at fooishbar.org> wrote:

> Hi,
>
> On Tue, 25 Jun 2019 at 07:26, Simon Ser <contact at emersion.fr> wrote:
> > > I noticed that original patch (v1) for gbm_bo_create_with_modifiers did
> > > have usage at first but it was removed during the review. I'm having
> > > trouble digging what was the reason for this?
> >
> > I'm not sure either. Daniel said it was a mistake.
> >
> > Adding the 63bd2ae7452d4 folks to the discussion. Ben, do you remember
> > the details?
>
> We decided to remove it since we decided that modifiers were a good
> enough proxy for usage; no need to pass SCANOUT or TEXTURE anymore,
> because we already get the scanout modifiers from KMS and the texture
> modifiers from EGL.
>
> In hindsight, I think this was a mistake since it only handles buffer
> layout, and not buffer placement or cache configuration.
>

It's not great but modifiers should be able to handle that as well.  You
can have _CONTIGUOUS versions of the modifiers supported by scanout and
scanout will only advertise those and the caller has to know to place them
in contiguous memory.  That's just an example but I think it would probably
work for a lot of the cases.  If not, I'd love to know why not.

--Jason
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190625/99a07486/attachment.html>