Command to mark surface inactive, so gpu process can release resources.

gpu/command_buffer/build_gles2_cmd_buffer.py

 #!/usr/bin/python
 #
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """code generator for GLES2 command buffers."""
 
 import os
 import os.path
@@ skipping 205 matching lines @@
 GL_APICALL void         GL_APIENTRY glResizeCHROMIUM (GLuint width, GLuint height);
 GL_APICALL const GLchar* GL_APIENTRY glGetRequestableExtensionsCHROMIUM (void);
 GL_APICALL void         GL_APIENTRY glRequestExtensionCHROMIUM (const char* extension);
 GL_APICALL void         GL_APIENTRY glRateLimitOffscreenContextCHROMIUM (void);
 GL_APICALL void         GL_APIENTRY glGetMultipleIntegervCHROMIUM (const GLenum* pnames, GLuint count, GLint* results, GLsizeiptr size);
 GL_APICALL void         GL_APIENTRY glGetProgramInfoCHROMIUM (GLidProgram program, GLsizeiNotNegative bufsize, GLsizei* size, void* info);
 GL_APICALL void         GL_APIENTRY glPlaceholder447CHROMIUM (void);
 GL_APICALL void         GL_APIENTRY glPlaceholder451CHROMIUM (void);
 GL_APICALL void         GL_APIENTRY glPlaceholder452CHROMIUM (void);
 GL_APICALL void         GL_APIENTRY glPlaceholder453CHROMIUM (void);
+GL_APICALL void         GL_APIENTRY glSetSurfaceVisibleCHROMIUM (GLboolean visible);

[apatrick_chromium, 2011/09/28 17:33:45]

Is there any reason why changing surface visibility can't be requested using an
IPC message. If the IPC message follows a call to flush() it will be effectively
"in-band" with the GL command stream. 

[mmocny, 2011/09/28 21:37:33]

I originally assumed we would not need the flush, and so I had to use a gl
command.
However, I found that this glSetSurfaceVisibleCHROMIUM command was not always
appropriately sent to the gpu process without a flush.  It wasn't just that it
was delayed by some #of ms, rather it wasn't sent at all until further gl
commands are issued.

I am not sure how the command stream decides when to push, but if I do end up
relying on the flush, then you are right, that same flush allows me to just use
an IPC message anyway.

So, a few questions for you:
1) Is there perhaps a less heavy handed way to make sure the message is
delivered?
2) If I switch to using an IPC message, the glflush guarantees it will be
processed after outstanding gl commands, but how do I guarantee it will be
processed before any subsequent ones? (this needs to be true in the
setVisibility(true) case.)

Thanks.

On 2011/09/28 17:33:45, apatrick_chromium wrote:

[apatrick_chromium, 2011/09/28 22:29:45]

OpenGL does not guarantee that a GL command will be executed until after glFlush
or some other synchronizing event happens. It can be held up indefinitely.
glFlush will work. It is stronger than you need because it will glFlush the
underlying GL backend unnecessarily. GLES2CmdHelper::Flush is cheaper and all
you need.
IPCs, including the async IPC that signals the flush, are not reordered on the
channel so the the flush before the set visibility IPC is a barrier and the set
visibility IPC itself is a barrier wrt to ordering of GL commands.

[mmocny, 2011/10/07 14:26:13]

GLES2CmdHelper::Flush seems to only add a glFlush into the command stream, but
that led me to find CommandBufferHelper::Flush which is I think what you
intended.

On 2011/09/28 22:29:45, apatrick_chromium wrote:

[apatrick_chromium, 2011/10/07 22:45:49]

Yeah that's the one. Sorry about that.

 """
 
 # This is the list of all commmands that will be generated and their Id.
 # If a command is not listed in this table it is an error.
 # This lets us make sure that command ids do not change as the generator
 # generates new variations of commands.
 
 _CMD_ID_TABLE = {
   'ActiveTexture':                                             256,
   'AttachShader':                                              257,
@@ skipping 188 matching lines @@
   'BlitFramebufferEXT':                                        446,
   'Placeholder447CHROMIUM':                                    447,
   'ResizeCHROMIUM':                                            448,
   'GetRequestableExtensionsCHROMIUM':                          449,
   'RequestExtensionCHROMIUM':                                  450,
   'Placeholder451CHROMIUM':                                    451,
   'Placeholder452CHROMIUM':                                    452,
   'Placeholder453CHROMIUM':                                    453,
   'GetMultipleIntegervCHROMIUM':                               454,
   'GetProgramInfoCHROMIUM':                                    455,
+  'SetSurfaceVisibleCHROMIUM':                                 456,
 }
 
 # This is a list of enum names and their valid values. It is used to map
 # GLenum arguments to a specific set of valid values.
 _ENUM_LISTS = {
   'BlitFilter': {
     'type': 'GLenum',
     'valid': [
       'GL_NEAREST',
       'GL_LINEAR',
@@ skipping 1317 matching lines @@
   'Placeholder452CHROMIUM': {
     'type': 'UnknownCommand',
     'extension': True,
     'chromium': True,
   },
   'Placeholder453CHROMIUM': {
     'type': 'UnknownCommand',
     'extension': True,
     'chromium': True,
   },
+  'SetSurfaceVisibleCHROMIUM': {
+      'type': 'Custom',
+      'impl_func': False,
+      'unit_test': False,
+      'extension': True,
+      'chromium': True,
+  },
 }
 
 
 def SplitWords(input_string):
   """Transforms a input_string into a list of lower-case components.
 
   Args:
     input_string: the input string.
 
   Returns:
@@ skipping 4172 matching lines @@
 
     if options.generate_docs:
       gen.WriteDocs("docs/gles2_cmd_format_docs_autogen.h")
 
   if gen.errors > 0:
     print "%d errors" % gen.errors
     sys.exit(1)
 
 if __name__ == '__main__':
   main(sys.argv[1:])