[PATCH v2 1/5] d3d9: Don't upload unused system memory vertex buffers.

Matteo Bruni matteo.mystral at gmail.com
Tue Feb 26 11:07:22 CST 2019


On Tue, Feb 26, 2019 at 4:54 PM Henri Verbeet <hverbeet at gmail.com> wrote:
>
> On Mon, 25 Feb 2019 at 21:31, Matteo Bruni <mbruni at codeweavers.com> wrote:
> > +    /* Make sure to only upload buffers that are used by the current draw:
> > +     * released buffers might still be bound as stream sources. */
> Thinking about it some more, does this really solve the issue? In
> particular, what about the case where the buffer is released, but also
> referenced by the vertex declaration? Shouldn't we just delay
> releasing the wined3d system memory buffer until
> wined3d_object_destroyed()?

I'll write a test for that case.
I think something like this patch will be useful anyway to avoid
spurious buffer uploads (not strictly for performance reasons, those
extra uploads did look somewhat confusing in the log.)



More information about the wine-devel mailing list