1578380 - [Wayland] Implement WaylandDMABUFTextureClientOGL / WaylandDMABUFTextureHostOGL

Status: RESOLVED FIXED

(Core :: Graphics: Layers, enhancement)

Description

[Martin Stránský [:stransky] (ni? me)]

[Wayland] Implement WaylandDMABUFTextureClientOGL / WaylandDMABUFTextureHostOGL.

Comment 1

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Implement WaylandDMABUFTextureClientOGL, r=sotaro

WaylandDMABufSurface is used as a backend for WaylandDMABUFTextureClientOGL.
Pixel data of WaylandDMABufSurface are located at GPU memory and can be
shared by processes.

Comment 2

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Implement WaylandDMABUFTextureHostOGL, r=sotaro

WaylandDMABufSurface is used as a backend for WaylandDMABUFTextureHostOGL.
Pixel data of WaylandDMABufSurface are located at GPU memory and are
binded from WaylandDMABUFTextureClientOGL.

WaylandDMABufSurface can be binded as EGLImage so EGLImageTextureSource is used.

Depends on D46836

Comment 3

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Enable build of WaylandDMABUFTextureClientOGL/WaylandDMABUFTextureHostOGL, r=sotaro

Depends on D46837

Comment 4

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - Export WaylandDMABUF TextureType, r=sotaro

Depends on D46839

Comment 5

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Use WaylandDMABUF texure backend on Wayland when DMABuf is enabled, r=sotaro

Depends on D46840

Comment 6

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Use WaylandDMABufSurface as ref-counted and store GLContext, r=jhorak

Depends on D46841

Comment 7

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Provide gfxPlatformGtk::IsX11Display() and gfxPlatformGtk::UseWaylandDMABufSurfaces(), r=sotaro

Let's use single Wayland configuration point at gfxPlatformGtk instead of various GDK_IS_X11_DISPLAY() calls.
Also provide info about enabled DMABuf surfaces when Wayland backend is used.

Depends on D46842

Comment 8

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1578380 - [Wayland] Use ContentClientDoubleBuffered() on Wayland with DMABuf texture backend enabled, r=sotaro

Depends on D46843

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ca57727c3aad
[Wayland] Implement WaylandDMABUFTextureClientOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/2b52c3d15b7b
[Wayland] Implement WaylandDMABUFTextureHostOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/3a8abb5d7a43
[Wayland] Enable build of WaylandDMABUFTextureClientOGL/WaylandDMABUFTextureHostOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/b80b7e186f02
Export WaylandDMABUF TextureType, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/754865b2810f
[Wayland] Use WaylandDMABUF texure backend on Wayland when DMABuf is enabled, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/3fdbc241d63a
[Wayland] Use WaylandDMABufSurface as ref-counted and store GLContext, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/373c4563e9fd
[Wayland] Provide gfxPlatformGtk::IsX11Display() and gfxPlatformGtk::UseWaylandDMABufSurfaces(), r=sotaro
https://hg.mozilla.org/integration/autoland/rev/28c2e884d20a
[Wayland] Use ContentClientDoubleBuffered() on Wayland with DMABuf texture backend enabled, r=sotaro

Comment 10

[Oana Pop-Rus]

Backed out for causing build bustages in ContentClient.cpp

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=superseded%2Ctestfailed%2Cbusted%2Cexception%2Crunnable&revision=28c2e884d20a1488861f07e9213d7299000fecd8&selectedJob=268123418

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=268123418&repo=autoland&lineNumber=32064

Backout: https://hg.mozilla.org/integration/autoland/rev/b5901b15d50c6744be080c30f7bd2068b40c0e45

Comment 11

[Martin Stránský [:stransky] (ni? me)]

Hm, I don't quite understand how is that possible, I did try and that went fine:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=254143795a2740445e5391d8049a6d932a173dfd

Comment 12

[Martin Stránský [:stransky] (ni? me)]

Ahh I see. UseWaylandDMABufSurfaces() is defined only when MOZ_WAYLAND is defined.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/90e37db667c1
[Wayland] Implement WaylandDMABUFTextureClientOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/d2becde7f177
[Wayland] Implement WaylandDMABUFTextureHostOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/bafb308b2ad1
[Wayland] Enable build of WaylandDMABUFTextureClientOGL/WaylandDMABUFTextureHostOGL, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/2fc6b1a863fd
Export WaylandDMABUF TextureType, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/d286f2b700d7
[Wayland] Use WaylandDMABUF texure backend on Wayland when DMABuf is enabled, r=sotaro
https://hg.mozilla.org/integration/autoland/rev/935af7c7e287
[Wayland] Use WaylandDMABufSurface as ref-counted and store GLContext, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/f5f1a7abdd43
[Wayland] Provide gfxPlatformGtk::IsX11Display() and gfxPlatformGtk::UseWaylandDMABufSurfaces(), r=sotaro
https://hg.mozilla.org/integration/autoland/rev/0fd48261532d
[Wayland] Use ContentClientDoubleBuffered() on Wayland with DMABuf texture backend enabled, r=sotaro

Comment 14

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/90e37db667c1
https://hg.mozilla.org/mozilla-central/rev/d2becde7f177
https://hg.mozilla.org/mozilla-central/rev/bafb308b2ad1
https://hg.mozilla.org/mozilla-central/rev/2fc6b1a863fd
https://hg.mozilla.org/mozilla-central/rev/d286f2b700d7
https://hg.mozilla.org/mozilla-central/rev/935af7c7e287
https://hg.mozilla.org/mozilla-central/rev/f5f1a7abdd43
https://hg.mozilla.org/mozilla-central/rev/0fd48261532d