kc3-lang/angle/extensions/ANGLE_external_objects_flags.txt
Branch
-
Show log
Commit
-
Hash : 6fe13477
Author :
Date : 2021-11-11T00:47:14
Vulkan: Add external's pNext to ANGLE_external_objects_flags ANGLE was chaining VkImageFormatListCreateInfoKHR to VkImageCreateInfo::pNext to support sRGB extensions. For external images, it was unknown whether that was valid because there was no way to know if external used an identical chain of pNexts. This was causing a discrepancy between images created by Chrome and those created by ANGLE as part of an import. This change updates ANGLE_external_objects_flags to take in the pNext chain external has used to create the image so ANGLE could create the image identically. Bug: chromium:1266094 Change-Id: I479b9e7ff39d437425dc91c79834880749766f99 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/3274177 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org> Reviewed-by: Peng Huang <penghuang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
-
extensions/ANGLE_external_objects_flags.txt
-
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333
Name ANGLE_external_objects_flags Name Strings GL_ANGLE_memory_object_flags Contributors Shahbaz Youssefi, Google Contact Shahbaz Youssefi, Google (syoussefi 'at' google.com) Status Draft Version Last Modified Date: Nov 10, 2021 Revision: 2 Number TBD Dependencies Written against the OpenGL 4.5 and OpenGL ES 3.2 specifications GL_ANGLE_memory_object_flags requires GL_EXT_memory_object. Overview Building upon the OpenGL memory object framework defined in EXT_external_objects, this extension introduces new Texture entry points for specifying create and usage flags parameters. These parameters match an external Texture's corresponding Vulkan create and usage flags. This allows applications to drop usage flags that are detrimental to performance, or add create flags as necessary. Additionally, these entry points take the pointer to the chain of extension structures appended to VkImageCreateInfo, so that the OpenGL implementation can recreate the image identically. New Procedures and Functions The following commands are added: void TexStorageMemFlags2DANGLE(enum target, sizei levels, enum internalFormat, sizei width, sizei height, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TexStorageMemFlags2DMultisampleANGLE(enum target, sizei samples, enum internalFormat, sizei width, sizei height, boolean fixedSampleLocations, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TexStorageMemFlags3DANGLE(enum target, sizei levels, enum internalFormat, sizei width, sizei height, sizei depth, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TexStorageMemFlags3DMultisampleANGLE(enum target, sizei samples, enum internalFormat, sizei width, sizei height, sizei depth, boolean fixedSampleLocations, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); [[ The following are added if direct state access is supported ]] void TextureStorageMemFlags2DANGLE(uint texture, sizei levels, enum internalFormat, sizei width, sizei height, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TextureStorageMemFlags2DMultisampleANGLE(uint texture, sizei samples, enum internalFormat, sizei width, sizei height, boolean fixedSampleLocations, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TextureStorageMemFlags3DANGLE(uint texture, sizei levels, enum internalFormat, sizei width, sizei height, sizei depth, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TextureStorageMemFlags3DMultisampleANGLE(uint texture, sizei samples, enum internalFormat, sizei width, sizei height, sizei depth, boolean fixedSampleLocations, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); [[ The following are available in OpenGL only ]] void TexStorageMemFlags1DANGLE(enum target, sizei levels, enum internalFormat, sizei width, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); [[ The following are availble in OpenGL only, and only when direct state access is available ]] void TextureStorageMemFlags1DANGLE(uint texture, sizei levels, enum internalFormat, sizei width, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); New Tokens The following tokens are added: Accepted by the <createFlags> parameter of TexStorageMemFlags*ANGLE() and TextureStorageMemFlags*ANGLE(). CREATE_SPARSE_BINDING_BIT_ANGLE 0x00000001 CREATE_SPARSE_RESIDENCY_BIT_ANGLE 0x00000002 CREATE_SPARSE_ALIASED_BIT_ANGLE 0x00000004 CREATE_MUTABLE_FORMAT_BIT_ANGLE 0x00000008 CREATE_CUBE_COMPATIBLE_BIT_ANGLE 0x00000010 CREATE_ALIAS_BIT_ANGLE 0x00000400 CREATE_SPLIT_INSTANCE_BIND_REGIONS_BIT_ANGLE 0x00000040 CREATE_2D_ARRAY_COMPATIBLE_BIT_ANGLE 0x00000020 CREATE_BLOCK_TEXEL_VIEW_COMPATIBLE_BIT_ANGLE 0x00000080 CREATE_EXTENDED_USAGE_BIT_ANGLE 0x00000100 CREATE_PROTECTED_BIT_ANGLE 0x00000800 CREATE_DISJOINT_BIT_ANGLE 0x00000200 CREATE_CORNER_SAMPLED_BIT_ANGLE 0x00002000 CREATE_SAMPLE_LOCATIONS_COMPATIBLE_DEPTH_BIT_ANGLE 0x00001000 CREATE_SUBSAMPLED_BIT_ANGLE 0x00004000 Accepted by the <usageFlags> parameter of TexStorageMemFlags*ANGLE() and TextureStorageMemFlags*ANGLE(). USAGE_TRANSFER_SRC_BIT_ANGLE 0x00000001 USAGE_TRANSFER_DST_BIT_ANGLE 0x00000002 USAGE_SAMPLED_BIT_ANGLE 0x00000004 USAGE_STORAGE_BIT_ANGLE 0x00000008 USAGE_COLOR_ATTACHMENT_BIT_ANGLE 0x00000010 USAGE_DEPTH_STENCIL_ATTACHMENT_BIT_ANGLE 0x00000020 USAGE_TRANSIENT_ATTACHMENT_BIT_ANGLE 0x00000040 USAGE_INPUT_ATTACHMENT_BIT_ANGLE 0x00000080 USAGE_SHADING_RATE_IMAGE_BIT_ANGLE 0x00000100 USAGE_FRAGMENT_DENSITY_MAP_BIT_ANGLE 0x00000200 Additions to Chapter 8 of the OpenGL 4.5 Specification (Textures and Samplers) For each list of TexStorage* commands in the 1D, 2D, 3D, 2DMultisample, and 3DMultisample families, add the following variants: void TexStorageMemFlags*ANGLE(<existing parameters>, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); void TextureStorageMemFlags*ANGLE(<existing parameters>, uint memory, uint64 offset, bitfield createFlags, bitfield usageFlags, const void *imageCreateInfoPNext); For each of the TexStorageMemFlags*ANGLE commands, move the description from the corresponding TexStorageMem*EXT from EXT_external_objects under that command and add the following language: "The implementation should apply the same semantics to the internal storage of the texture based on <createFlags>, <usageFlags> and <imageCreateInfoPNext> as the corresonding VkImageCreateFlags, VkImageUsageFlags and chained structs to VkImageCreateInfo would have on a Vulkan image respectively. See tables 8.17 and 8.18 for the mapping between the GL and Vulkan flags. It is undefined behavior if the application provides create flags, usage flags or chained structs that are not identical to those used to create the Vulkan image." Table 8.17: Create flags and corresponding Vulkan Image Create Flags | Create Flag | Equivalent Vulkan Create Flag | +----------------------------------------------------+-----------------------------------------------------------+ | CREATE_SPARSE_BINDING_BIT_ANGLE | VK_IMAGE_CREATE_SPARSE_BINDING_BIT | | CREATE_SPARSE_RESIDENCY_BIT_ANGLE | VK_IMAGE_CREATE_SPARSE_RESIDENCY_BIT | | CREATE_SPARSE_ALIASED_BIT_ANGLE | VK_IMAGE_CREATE_SPARSE_ALIASED_BIT | | CREATE_MUTABLE_FORMAT_BIT_ANGLE | VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT | | CREATE_CUBE_COMPATIBLE_BIT_ANGLE | VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT | | CREATE_ALIAS_BIT_ANGLE | VK_IMAGE_CREATE_ALIAS_BIT | | CREATE_SPLIT_INSTANCE_BIND_REGIONS_BIT_ANGLE | VK_IMAGE_CREATE_SPLIT_INSTANCE_BIND_REGIONS_BIT | | CREATE_2D_ARRAY_COMPATIBLE_BIT_ANGLE | VK_IMAGE_CREATE_2D_ARRAY_COMPATIBLE_BIT | | CREATE_BLOCK_TEXEL_VIEW_COMPATIBLE_BIT_ANGLE | VK_IMAGE_CREATE_BLOCK_TEXEL_VIEW_COMPATIBLE_BIT | | CREATE_EXTENDED_USAGE_BIT_ANGLE | VK_IMAGE_CREATE_EXTENDED_USAGE_BIT | | CREATE_PROTECTED_BIT_ANGLE | VK_IMAGE_CREATE_PROTECTED_BIT | | CREATE_DISJOINT_BIT_ANGLE | VK_IMAGE_CREATE_DISJOINT_BIT | | CREATE_CORNER_SAMPLED_BIT_ANGLE | VK_IMAGE_CREATE_CORNER_SAMPLED_BIT_NV | | CREATE_SAMPLE_LOCATIONS_COMPATIBLE_DEPTH_BIT_ANGLE | VK_IMAGE_CREATE_SAMPLE_LOCATIONS_COMPATIBLE_DEPTH_BIT_EXT | | CREATE_SUBSAMPLED_BIT_ANGLE | VK_IMAGE_CREATE_SUBSAMPLED_BIT_EXT | +----------------------------------------------------------------------------------------------------------------+ Table 8.18: Usage flags and corresponding Vulkan Image Usage Flags | Usage Flag | Equivalent Vulkan Usage Flag | +------------------------------------------+---------------------------------------------+ | USAGE_TRANSFER_SRC_BIT_ANGLE | VK_IMAGE_USAGE_TRANSFER_SRC_BIT | | USAGE_TRANSFER_DST_BIT_ANGLE | VK_IMAGE_USAGE_TRANSFER_DST_BIT | | USAGE_SAMPLED_BIT_ANGLE | VK_IMAGE_USAGE_SAMPLED_BIT | | USAGE_STORAGE_BIT_ANGLE | VK_IMAGE_USAGE_STORAGE_BIT | | USAGE_COLOR_ATTACHMENT_BIT_ANGLE | VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT | | USAGE_DEPTH_STENCIL_ATTACHMENT_BIT_ANGLE | VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT | | USAGE_TRANSIENT_ATTACHMENT_BIT_ANGLE | VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT | | USAGE_INPUT_ATTACHMENT_BIT_ANGLE | VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT | | USAGE_SHADING_RATE_IMAGE_BIT_ANGLE | VK_IMAGE_USAGE_SHADING_RATE_IMAGE_BIT_NV | | USAGE_FRAGMENT_DENSITY_MAP_BIT_ANGLE | VK_IMAGE_USAGE_FRAGMENT_DENSITY_MAP_BIT_EXT | +----------------------------------------------------------------------------------------+ Add the following errors for each of the TexStorageMemFlags*ANGLE commands: "An INVALID_VALUE error is generated if <createFlags> has a bit set other than those specified in table 8.17." "An INVALID_VALUE error is generated if <usageFlags> has a bit set other than those specified in table 8.18." For each of the TexStorageMem*EXT commands, change the description of TexStorageMem*EXT and TextureStorageMem*EXT as such: "Calling TexStorageMem*EXT is equivalent to calling TexStorageMemFlags*ANGLE with 0 as <createFlags>, all Vulkan-supported usage flags as <usageFlags>, even if there are no corresponding GL flags specified in this extension, and NULL as <imageCreateInfoPNext>." Issues 1) Can the GL and Vulkan create and usage flag values diverge? RESOLVED: No. It is expected for the application to provide the flags used to create the Vulkan image verbatim. 2) What if Vulkan adds new create and usage flags? RESOLVED: If the implementation is too old to be aware of those flags, the functions from GL_EXT_memory_object must be used instead. New flags can be added through new extensions that build on this extension. 3) Should the GL implementation apply the same validation as Vulkan based on the create and usage flags? RESOLVED: No. The application has already created an image with these flags in Vulkan, so the GL implementation can assume they are valid. Validating this is expensive and unnecessary in almost all applications. 4) How can extension structs applied to VkImageCreateInfo be communicated to the GL implementation? RESOLVED: The chain of structs itself is directly passed to the implementation. Revision History Revision 2, 2021-11-10 (Shahbaz Youssefi) - Added the imageCreateInfoPNext parameter. Revision 1, 2020-08-03 (Shahbaz Youssefi) - Initial draft based on EXT_external_objects.