glTexSubImage2D not updating texture as expected

[diamond-msc]

Calling glTexSubImage2D doesn't always add to the existing texture data. With double buffering, e. g. calling from a cococ2d-x update, glTexSubImage2D appears to write to some kind of texture back buffer. This texture back buffer is initially empty, and becomes visible on the next global buffer flip after the glTexSubImage2D call.

The result, when displayed, is an empty texture with only the subimage set. (Tested with CCTexture2D::drawAtPoint).

It should be the current texture plus the given subimage data.

[diamond-msc]

The texture back buffer may be the cause for https://github.com/MSOpenTech/cocos2d-x/issues/28

[pwang08]

The angle project is developed and maintained by google, I have read the reference of angle, it looks as if the texture back buffer is caused by the difference of the way in which opengl and directx store the texutures.

[diamond-msc]

I tested glTexSubImage2D on Win32 with a modified Simple_Texture2D sample from gles2_book. It works properly there. That's why I assume it's a bug in the WinRT port. But I'm not sure since the gles2_book samples aren't part of the WinRT solution.

[stammen]

Thanks for the updates. We will be looking at why glTexSubImage2D is broken in the WinRT/WP8 versions.

[JingxuOuyang]

It may be cause by the 'updateSubresourceLevel' function in TextureStorage11.cpp line 166. The data in texture is cleared by 'mTexture->release'?

[hujian]

Has anybody made any progress on this question?

[yihuang]

We've met the same problem, is there any progress on this, or is there work around for this?

[steven-chith]

I'm not able to repo this. Would you be able to put up some sample code for us to verify?

[diamond-msc]

https://gist.github.com/diamond-msc/7167ba71776e48cdb150

took the cube renderer and replaced GL stuff with Simple_Texture2D sample. (Base is fd780fcf527c6d6707683a4f7bad5077c98b1cc3 on winrt branch)

Expected: Only the top left one of the 4 texture pixels changes color. Actual: The other 3 pixels also change color (they alternate to/from black).

[steven-chith]

Looks like it only works when the texture is RGBA instead of RGB on both internal format and passed in format. Will investigate why. If you need to get it done quickly just convert to RGBA for the time being.

[stammen]

For now add glPixelStorei(GL_UNPACK_ALIGNMENT, 1) before the call to glTexSubImage2D. We are investigating why the glPixelStorei state is not staying set.

void CubeRenderer::Update(float timeTotal, float timeDelta) { pixels[2] = rand() % 2 * 255; glBindTexture(GL_TEXTURE_2D, textureId); glPixelStorei(GL_UNPACK_ALIGNMENT, 1); glTexSubImage2D(GL_TEXTURE_2D, 0, 0, 0, 1, 1, GL_RGB, GL_UNSIGNED_BYTE, pixels); // expected: only the top left color flickers red/purple }

[steven-chith]

This is fixed in the winrt-es3proto branch. We'll merge these changes over to the winrt branch soon.