OpenGL performance on Graphic_Update branch

[degasus]

Hello

I made a cpu usage profile of the gpu thread in SMG:

uploading vertices: 1461 usecs per frame
uploading intices: 347 usecs per frame
DrawVertexBufferObjectBase: 2672 usecs per frame
DrawVertexBufferObjectBase_AlphaPass: 674 usecs per frame

I see, uploading needs time, but shouldn't the draw command be asynchronous?

At all, this 4 commands would need 31% of the cpu at 60 fps ...

btw, there are 429 indexed draws (177 with alpha pass), so every draw command needed 6.2 usecs. Is this usual?

[neobrain]

That's why I said built-in profilers suck; GPU drivers delay any operation as long as they can, and this includes RAM->VRAM transfers.

What you're probably seeing is that a vertex buffer was still in use when we tried writing new data into it. The GPU driver likely delayed the actual RAM->VRAM upload to the next DrawPrimitive() call, which is why you see so much CPU time spent there.

.. Or it's just the typical case that the CPU says "Yo GPU, here's some data to draw" and the GPU answers "Yo dawg, I'm still busy so wait a minute with that data".

[degasus]

What you're probably seeing is that a vertex buffer was still in use when we tried writing new data into it. The GPU driver likely delayed the actual RAM->VRAM upload to the next DrawPrimitive() call, which is why you see so much CPU time spent there.

But the draw command should also be delayed? In fact, adding much sleeps between uploading and drawing doesn't change this timings.

.. Or it's just the typical case that the CPU says "Yo GPU, here's some data to draw" and the GPU answers "Yo dawg, I'm still busy so wait a minute with that data".

A secound identical draw command will return directly, so I guess, this is not the reason.

btw, after switching the shader, the draw command use double the time. So it is perhaps changing the state of the pipeline?

Edit: enabling USE_DUAL_SOURCE_BLEND, the secound draw command isn't used anymore and (more importend) the shader won't be changed for this. So the shader sometimes don't need to be changed befor the regular call. So the total time spend on draw commands (1856usec) is now lower than the time spend on the frist draw command without alphapass.