Log

Author Commit Date CI Message
Kimmo Kinnunen 0d02f857 2024-11-07T09:32:28 Test SeparateDeclarations This makes it easier to fix issues with SeparateDeclarations Bug: angleproject:377330017 Change-Id: I53b9ca06971e0272563b6b3c752e0438c5802193 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6000533 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com> Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com>
angle-autoroll c9407fec 2024-11-12T07:00:51 Roll Chromium from dcfc04e3c072 to 75279b66ac13 (575 revisions) https://chromium.googlesource.com/chromium/src.git/+log/dcfc04e3c072..75279b66ac13 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,solti@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/827e4e2f1e..8a810c637f * testing: https://chromium.googlesource.com/chromium/src/testing/+log/c448863360..319e286ea3 * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/5a79ab5495..af142f1e69 * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/1063637350..41d43a2a22 * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/8ba0a768d6..cec7f47835 * third_party/llvm-libc/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libc.git/+log/ccaf1e62d0..ca74a72e2b * third_party/r8/cipd: uYapBJQiyZps8iko9FUxKUIPKRvOfk4a3umRDg9TpdgC..6tS5W0W6vy5TPDxAc-wTObNng6QBVt7cQr3lazvrbpEC * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/05cdff4728..4bd500b6f2 * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/644714c582..e7abceab37 * tools/skia_goldctl/linux: IlHPGzbUfx7oVzokTZJFxwi8WlNeBa4rBMtyFdiZgt4C..AiY7CO223oVpSMe9b7SwHVhQrF-ijgHEDzjsAV8SUKoC * tools/skia_goldctl/mac_amd64: L7lW1QzG1t7Sx50nS77H6FJxV7HyorteX0aL7QiRd1AC..aWw1JVWpKckecy3rBuN-5Y_osHIckC0ganu7xQ28f4UC * tools/skia_goldctl/mac_arm64: OTCCrMReyAqtbtGf3MMr15PrWMvHQNwZ7tt-CGp64JUC..WnSB4H9RkKT6MKFpLZ9yfWPnBUeECncOVa3XuRnyTEEC * tools/skia_goldctl/win: 9_wlbFkVOPWD2yCsq4B7K45Wd8UAGtlKOPrZ3_iWDJkC..EK_tQRk4h7ikvzyk5kFX_pkGw3gjHkP4QPmEjN8D54EC No update to Clang. Bug: None Tbr: solti@google.com Change-Id: I896ddabb33922eb5d77a13d8d0391bd2011f1fb2 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6014315 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Yuxin Hu e5619a5c 2024-10-15T18:27:00 Use EGL sync global lock for all EGL*sync entrypoint calls To free EGL sync operations from waiting for other EGL calls to finish, we use a separate global lock for EGL*sync entrypoints. Below angle::SimpleMutex are added to protect resources that may have race condition due to being accessed by EGL*sync calls and non EGL*sync calls at the same time: 1. Display::mContextMapMutex that protects Display::mState.contextMap 2. static angle::base::NoDestructor<angle::SimpleMutex> anglePlatformDisplayMapMutex that protects static angle::base::NoDestructor<ANGLEPlatformDisplayMap> displays 3. static angle::base::NoDestructor<angle::SimpleMutex> devicePlatformDisplayMapMutex that protects static angle::base::NoDestructor<DevicePlatformDisplayMap> displays EGL_Terminate() entry point takes both global lock and global egl sync lock. This is to protect Display::mSyncMap, Display::mSyncPools, and Display::mSyncHandleAllocator being get cleared by thread 1 calling eglTerminate, while they are still accessed by thread 2 through a call such as eglCreateSync. So that we won't have thread 2 finish validating the sync object with syncID exists in Display::mSyncMap, but then find the mSyncMap.find(syncID) returns a nullptr due to the mSyncMap is cleared by thread 1. Same applies to EGL_LabelObjectKHR(), EGL_ReleaseThread(), ThreadCleanupCallback(). EGL_Initialize() writes to Display::mInitialized. This is read by EGL Sync API validation functions. EGL_Initialize() also takes both global lock and global sync lock to prevent race conditions between EGL_Initialize() and EGL Sync APIs. When ANGLE_CAPTURE_ENABLED is enabled, fall back to global lock, as the CaptureEGLCallToFrameCapture() touches many resources (e.g. mMaxAccessedResourceIDs, mFrameCalls) that could lead to race conditions without a global lock. Bug: b/362604439 Change-Id: Ic0d54a4cd66743bcd0f48f41f247dd223cff2f5e Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5933570 Reviewed-by: Roman Lavrov <romanl@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Igor Nazarov <i.nazarov@samsung.com> Commit-Queue: Yuxin Hu <yuxinhu@google.com>
Yuxin Hu 1c70fb79 2024-11-11T10:54:26 Fix dependency metadata invalid date warnings According to the latest third-party dependency metadata template: https://source.chromium.org/chromium/chromium/src/+/main:third_party/README.chromium.template, Date field is OPTIONAL if Version or Revision is supplied. If Date field is supplied, it needs to be in the format of YYYY-MM-DD. Since the Revision field is supplied, we can remove the N/A Date field. Bug: chromium:378273799 Change-Id: I30af7f462f0c53d0bbdb65a94f38218cb61e597f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6013040 Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com> Auto-Submit: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Amirali Abdolrashidi <abdolrashidi@google.com>
Yuxin Hu f0d8a820 2024-11-11T11:35:24 Fix dependency metadata invalid date warnings According to the latest third-party dependency metadata template: https://source.chromium.org/chromium/chromium/src/+/main:third_party/README.chromium.template, Date field is OPTIONAL if Version or Revision is supplied. If Date field is supplied, it needs to be in the format of YYYY-MM-DD. Since the Revision field is supplied, we can remove the N/A Date field. Bug: chromium:378273779 Change-Id: I58b5699fed8cbab2d2bf6d0662dd4173eaa3e18f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6013366 Auto-Submit: Yuxin Hu <yuxinhu@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Amirali Abdolrashidi <abdolrashidi@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Yuxin Hu 1f82ca25 2024-11-08T12:19:32 Enable KHR-NoContext* and KHR-Single* tests on Linux Nvidia Bug: angleproject:40644911 Change-Id: I9a37aa742cc8af237d653535e51439b899757aaf Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6005396 Reviewed-by: Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuxin Hu <yuxinhu@google.com>
Austin Annestrand f198e807 2024-10-16T17:47:17 CL/VK: Serialize cmds when queue profiling enabled When queue profiling is enabled by user (opt-in) we should serialize/finish each cmd. Bug: angleproject:377942756 Change-Id: I7809df073c0a6de0e4d5338bec2fa2032ebc1577 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003807 Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com>
Austin Annestrand ac41a84a 2024-10-10T17:35:20 CL/VK: Fix reflection parsing out-of-order cases Some clspv generated binaries cause the SPIR-V Tools parser to parse reflection instructions out-of-order (with respect to what we see in disassembler dump). This fix addresses kernel and kernel args so that order does not matter anymore. Bug: angleproject:377941117 Change-Id: I75da428b6f0a9f155f3faf5db7a0408485d4ebdc Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003806 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Austin Annestrand a4172290 2024-10-08T13:20:40 CL/VK: Initial impl for migrateMemObjects Defer migration handling in multi-device setups. Allow API to return error-free on 1 device cases (nop) for now. Bug: angleproject:377942759 Change-Id: I9bcc238ad4d2965e1281a99ec2782e8cfe6c05a6 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003805 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Austin Annestrand 1071079a 2024-10-23T17:32:37 CL/VK: Fix frontend cl_mem_flags default access Additionally, if neither of these flags are specified: - CL_MEM_READ_WRITE - CL_MEM_READ_ONLY - CL_MEM_WRITE_WRITE Then we need to bitwise OR the default kernel access flag (according to spec): CL_MEM_READ_WRITE Bug: angleproject:377942757 Change-Id: I5decd05096a834469fdb6bca941284a400b2c69c Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003804 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com>
Austin Annestrand 152f8035 2024-10-29T14:14:03 CL/VK: Fix missing default device on contextFromType Bug: angleproject:377941119 Change-Id: Ia75184233a5b0b422425626cbce46a2105b08249 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003463 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Austin Annestrand de287909 2024-08-30T18:38:40 CL/VK: Generalize host transfer sync utility Rework host transfer sync utility to handle multiple non-blocking enqueue transfer ops for buffer/image reads. Bug: angleproject:377545840 Change-Id: Ic382162ded15cb15c900baf724e02592bfd846c9 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6001611 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com>
Kimmo Kinnunen efcb94ab 2024-11-06T15:55:02 Output the type of constant union Output the type of the constant union node in OutputTree output. This makes it easier to understand typing bugs related to the constant union. Adds a node nesting level for constant unions. This makes it easier to understand consecutive constant unions. Bug: angleproject:377330017 Change-Id: Ia83072b78c4d1713f839fd136692b254f6d2c517 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6000532 Commit-Queue: Geoff Lang <geofflang@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com>
Shahbaz Youssefi e3011d96 2024-11-08T16:17:07 Translator: Optimize size calculation for variable arrays Bug: angleproject:373924024 Change-Id: Id3a8b844b8274c19c0c2c833842fb8b501f50174 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6004338 Commit-Queue: Geoff Lang <geofflang@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org>
Kimmo Kinnunen 6359ec11 2024-11-11T13:24:35 Metal: Avoid leaking library and binary sources Fix leak of dispatch_data during binary load. The dispatch object was not released. Dispatch objects are Obj-C objects, so hold with AutoObjCPtr. Fix leak of MTLLibrary objects when creating the objects. The method newLibraryWithData returns +1 due to being "new" type method. This ref must be adopted. Avoid using autoreleased objects, use AutoObjCPtr. Some leaks regressed in c5ab1cebccaab0f8de466e5f797faa3d5a84bea5. Bug: angleproject:351165323 Change-Id: If23859fbddd21693d923f66f6fb015eed494f06e Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6011208 Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com> Commit-Queue: Geoff Lang <geofflang@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>
angle-autoroll ce13a00a 2024-11-11T10:01:58 Roll vulkan-deps from 12e843b4aad1 to a2dfb2276ea5 (5 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/12e843b4aad1..a2dfb2276ea5 Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/76cc41d26f6902de543773023611e40fbcdde58b..9c644fcb5b9a1a9c975c50a790fd14c5451292b0 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/1cf0cede5f779a1f9bca38ad5ad632c7d50b3aad..6cde840752a4510dfaa45de753b0be0694bc8c4b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,solti@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: solti@google.com Change-Id: I9802c2d8158f93e435f04c25d1cce304783e28b9 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6007531 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll 65288796 2024-11-11T07:01:24 Roll Chromium from 4201d3dd229a to dcfc04e3c072 (845 revisions) https://chromium.googlesource.com/chromium/src.git/+log/4201d3dd229a..dcfc04e3c072 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,solti@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/83dc15583b..827e4e2f1e * testing: https://chromium.googlesource.com/chromium/src/testing/+log/e47bfb0661..c448863360 * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/e9b2687111..5a79ab5495 * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/46ade108f8..1063637350 * third_party/fuchsia-sdk/sdk: version:25.20241102.1.1..version:25.20241109.3.1 * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/3fdba6f550..8ba0a768d6 * third_party/libunwind/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind.git/+log/93a837d836..5b01ea4a6f * third_party/llvm-libc/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libc.git/+log/846cb08622..ccaf1e62d0 * third_party/r8/cipd: Vz8vXNhg5R5IAETlzCdV1ZZTBHqr108_RhW0meOJXTcC..uYapBJQiyZps8iko9FUxKUIPKRvOfk4a3umRDg9TpdgC * third_party/siso/cipd: git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b..git_revision:aa31236f9e208039b6196cbae4318d56d4c30e3d * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/c656cea22c..05cdff4728 * tools/luci-go: git_revision:aa31236f9e208039b6196cbae4318d56d4c30e3d..git_revision:a52658ecc54d0fa4ad0065918588c890982634b2 * tools/luci-go: git_revision:aa31236f9e208039b6196cbae4318d56d4c30e3d..git_revision:a52658ecc54d0fa4ad0065918588c890982634b2 * tools/mb: https://chromium.googlesource.com/chromium/src/tools/mb/+log/37d1642b0d..45239f8c24 * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/9c9896ada1..644714c582 * tools/skia_goldctl/linux: Vd2g1IexnjJOyeYJEnBauprtDO72lnM7KoV7KH8_pyMC..IlHPGzbUfx7oVzokTZJFxwi8WlNeBa4rBMtyFdiZgt4C * tools/skia_goldctl/mac_amd64: 2lbVntmxhc-PqX6iK1PNyXMBxSNA36UXTfDQKNkxsFcC..L7lW1QzG1t7Sx50nS77H6FJxV7HyorteX0aL7QiRd1AC * tools/skia_goldctl/mac_arm64: 8Mdyp7IxqQT8bs3V_9W9nMgt0ZtRY7FpJuSYk6aoK68C..OTCCrMReyAqtbtGf3MMr15PrWMvHQNwZ7tt-CGp64JUC * tools/skia_goldctl/win: NE6rhxS2e-F58MAathE-L_OuUVdzbPXFhze5SlpUIpQC..9_wlbFkVOPWD2yCsq4B7K45Wd8UAGtlKOPrZ3_iWDJkC No update to Clang. Bug: None Tbr: solti@google.com Change-Id: I98577146e196c790e5cdb36c02eacbf8fc7c72fe Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6010466 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Amirali Abdolrashidi 7fea539c 2024-10-23T11:40:51 Vulkan: Remove extra non-conformant flag checks It may be possible to remove the flag check and simply rely on the checks performed in CanSupportGLES32() to determine support for ES 3.2. The non-conformant flag can remain utilized for certain features until the relevant issues are resolved and this flag is no longer needed. * Removed the checks from getMaxConformantESVersion(). * Removed the flag check in Renderer::getMaxSupportedESVersion(). * Added exception to the blend extension when using the non-conformant flag for Win/Intel. * Skipped the related tests on this platform that now fail. * Added a new feature flag to force-enable ES 3.2 on certain platforms for testing. * exposeES32ForTesting * If the bots are updated to support ES 3.2, it can be removed from using this flag. * Skipped some VVL errors when the flag is enabled. * Skipped end2end ES 3.2 tests on SwS. * Skipped some dEQP tests on P4 and SwS that would use ES 3.2 features that are not available on those platforms. Bug: b/374841339 Bug: b/376899587 Change-Id: Ifd405457ab8bf2076eb8fb5c92c5316f4d520595 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5958147 Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Yuxin Hu <yuxinhu@google.com>
Ho Cheung d30d81d0 2024-11-06T16:08:28 [code health] Remove underscores from test names in ANGLE (1/N) In the GoogleTest framework, underscores are not allowed. [1] ANGLE has underscores in many test names, and those should be changed. At the same time, add a comment to a test case according to the requirements of the Presubmit check. ```` *************** ANGLE requires a comment describing what a test does. *************** ```` [1] https://github.com/google/googletest/blob/main/docs/faq.md#why-should-test-suite-names-and-test-names-not-contain-underscore Bug: angleproject:377543127 Change-Id: I90e634bd985ca518eca43e2c919a3a8bca91a55b Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5997835 Reviewed-by: Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Auto-Submit: Ho Cheung <hocheung@chromium.org>
Igor Nazarov 11d73f1d 2024-11-04T20:31:57 Revert "spirv::Print without ANGLE_ENABLE_ASSERTS -> compile error" This reverts commit be9e63ad090dc4b86eae08ad8f9f4c9f1a18a6a4. Reason for revert is to fix build error in "CLProgramVk.cpp" if build with `dcheck_always_on = false`. Since https://crrev.com/c/5898609, `spirv::Print()` is now always defined, so no liker error if `ANGLE_ENABLE_ASSERTS` is not defined. Bug: angleproject:370557215 Change-Id: I983b57987eb63f601a75e90646c24b9cd16170a7 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003012 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Roman Lavrov <romanl@google.com> Commit-Queue: Roman Lavrov <romanl@google.com>
Yuxin Hu 094a0b11 2024-11-07T17:45:50 Skip KHR-GLES31.core.tessellation_shader.single.primitive* on win Bug: angleproject:377993076 Change-Id: I7d7d99b87b51911bbb3046b4e7353afb18060d5d Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003811 Commit-Queue: Roman Lavrov <romanl@google.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Yuxin Hu <yuxinhu@google.com>
angle-autoroll a0866346 2024-11-08T10:01:06 Roll vulkan-deps from 12b75c58255f to 12e843b4aad1 (4 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/12b75c58255f..12e843b4aad1 Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/58b4fcad7be86c8f60baae3cfb2fb45096b437e8..1cf0cede5f779a1f9bca38ad5ad632c7d50b3aad If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: yuxinhu@google.com Change-Id: I76ae965005287116aa9f444d4f7e8295c3f98d38 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6004057 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll fec8786e 2024-11-08T07:59:09 Roll Chromium from 369ac471ea04 to 4201d3dd229a (705 revisions) https://chromium.googlesource.com/chromium/src.git/+log/369ac471ea04..4201d3dd229a If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/0fc767f93a..83dc15583b * testing: https://chromium.googlesource.com/chromium/src/testing/+log/cb21bddb7a..e47bfb0661 * third_party/android_build_tools/error_prone/cipd: P4ar2IVWgipWzMglFQGb45LE-VKychgjOUmbssZK7qcC..DBKfaigj6AuTYPdN1_6jkOlQUuJa1szDRIMvHbtZrD8C * third_party/android_build_tools/lint/cipd: ZtEs7jgil5yykPwZynLNdQuisSTtHKp4uUi7aZHTc-YC..vBoRPSPyoHi8GYvjtidLlKAtqp5a-2CaHgpEZb16dAoC * third_party/android_build_tools/manifest_merger/cipd: -Yqd1ysKh8HXbxIODxAAUrZ6hSS9NCVF1fceEtPMCPsC..1gDDnuAjB1-mlxlnLFAfx4h9f_hG3lEIIbwyYr86i1wC * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/31beb497fc..e9b2687111 * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/0ff74cfe9f..46ade108f8 * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/171c504c4b..3fdba6f550 * third_party/libunwind/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind.git/+log/c3b4ce8edd..93a837d836 * third_party/llvm-libc/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libc.git/+log/194ae301ad..846cb08622 * third_party/r8/cipd: YpJxENNZqq3mMt24rEGk3QC1-tzRwn6Tu5HD2vsxzGIC..Vz8vXNhg5R5IAETlzCdV1ZZTBHqr108_RhW0meOJXTcC * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/3850115d14..9b898fa7cc * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/830220aa08..c656cea22c * tools/luci-go: git_revision:17620695ace472852acc003b74bf240c94a07cbc..git_revision:aa31236f9e208039b6196cbae4318d56d4c30e3d * tools/luci-go: git_revision:17620695ace472852acc003b74bf240c94a07cbc..git_revision:aa31236f9e208039b6196cbae4318d56d4c30e3d * tools/mb: https://chromium.googlesource.com/chromium/src/tools/mb/+log/8fe44e5d21..37d1642b0d * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/bd85c54890..9c9896ada1 * tools/skia_goldctl/linux: C0CwX5cN6vEo8BX2QpfBB0hMQ6Yg_BpTgtW6kED6a0QC..Vd2g1IexnjJOyeYJEnBauprtDO72lnM7KoV7KH8_pyMC * tools/skia_goldctl/mac_amd64: xmAykgoYIP74ev2nA3hQCJv5t-THH1oVqgTA8O4Sfu4C..2lbVntmxhc-PqX6iK1PNyXMBxSNA36UXTfDQKNkxsFcC * tools/skia_goldctl/mac_arm64: B_QRS0ZdBtI8oL8MDGOn6KQaMsdqpc-nV2lKWRGlOKAC..8Mdyp7IxqQT8bs3V_9W9nMgt0ZtRY7FpJuSYk6aoK68C * tools/skia_goldctl/win: 2J10fvw0AgR3Dpcg6wIiMTbhXIqL2J_QDOSXlOTBfpwC..NE6rhxS2e-F58MAathE-L_OuUVdzbPXFhze5SlpUIpQC No update to Clang. Bug: None Tbr: yuxinhu@google.com Change-Id: I218871cb6d9cfadc56558fd751d7bd17cd49b2cf Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6003821 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Yuxin Hu 2a61126b 2024-10-30T14:42:08 Remove GLES 3.2 tests from swiftshader backend GLES 3.2 is not supported with swiftshader backend. Remove the GLES 3.2 tests from swiftshader bots. Bug: b/374841339 Change-Id: I5ab51c139550ee3ecec4e06292bcfdd04f403ff5 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5980409 Commit-Queue: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Cody Northrop 2c7e9830 2024-11-04T10:05:25 restricted_trace_perf: Setup and wrapper support This CL adds support for installing and setting up traces. Also allows running via the wrapper in http://crrev/c/5985570 With this change, if you run without using the wrapper, you must provide the --build-dir flag. It allows running the script from any directory: --build-dir ../../../out/Android Bug: b/376300037 Change-Id: I2259ee120b2af5e1917a2a57a27fa3dcfca761e8 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5983806 Reviewed-by: Roman Lavrov <romanl@google.com> Commit-Queue: Cody Northrop <cnorthrop@google.com>
Roman Lavrov bdebee8c 2024-10-29T17:08:11 Tests: add repro for running out of outside RP serials Bug: b/375661776 Change-Id: I2cd82710bdf5b00a6165ddad6ef21f30150aa5bc Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5977123 Commit-Queue: Roman Lavrov <romanl@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Yuxin Hu 750d9a24 2024-11-06T17:28:19 Skip SourceAHBTarget2DGenerateMipmap* tests on S22 There is a vulkan driver bug on S22 device: https://chromium-review.googlesource.com/c/angle/angle/+/5966623/comments/44b72fb9_3ba112c5, causing ImageTestES3.SourceAHBTarget2DGenerateMipmap* to fail. Skip these tests on S22 until the vulkan driver of the device is updated. Bug: angleproject:40644776 Change-Id: Id5e68aa4ad8f2172c5cdda9adb26af60da87b905 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6001577 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Yuxin Hu <yuxinhu@google.com>
Roman Lavrov d01f5101 2024-11-01T18:45:55 Tests: add wrappers for restricted_trace scripts Example use: out/Android/restricted_trace_gold_tests \ --bypass-skia-gold-functionality \ --filter=dota_underlords:dragon_raja out/Android/restricted_trace_perf --flags Bug: b/376300037 Change-Id: Ic3e8cb241af25a770f5d1888538d13482f877b6b Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5985570 Commit-Queue: Roman Lavrov <romanl@google.com> Reviewed-by: Cody Northrop <cnorthrop@google.com>
Roman Lavrov 6cda99d8 2024-11-07T09:53:01 Tests: fix run_angle_android_test.py and trace bundle The output_directory arg is provided when invoked by the wrapper, but not when running with run_angle_android_test.py, in which case cwd has been historically used such as: cd out/AndroidPerformance; ../../src/tests/run_angle_android_test.py ... Move the import added in https://crrev.com/c/5985232, as 1) it now imports protobufs which is not included with python3 and requires an additional package 2) build/ is excluded from trace bundles where running traces with the Chromium runner isn't supported. Bug: b/377875190 Change-Id: Ie092dc6382e74ac7413a42acffaf507c05b73514 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6002496 Reviewed-by: Cody Northrop <cnorthrop@google.com> Commit-Queue: Roman Lavrov <romanl@google.com>
angle-autoroll 7ac10ebe 2024-11-07T10:01:42 Roll vulkan-deps from 155bbe2e1429 to 12b75c58255f (31 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/155bbe2e1429..12b75c58255f Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/d81aeef9f82050bd5940ce1c475d8316ad41d747..76cc41d26f6902de543773023611e40fbcdde58b https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Headers/+log/cb6b2c32dbfc3257c1e9142a116fe9ee3d9b80a2..996c728cf7dcfb29845cfa15222822318f047810 https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools/+log/9aaa2601400a29ce39ce1111aa9f790e5c9d4c1b..9117e042b93d4ff08d2406542708170f77aaa2a3 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Loader/+log/9276d73f29651c92402f0f0389108d69bb51b9ca..b0177a972b8d47e823a4500cf88df88a8c27add7 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Tools/+log/e94085e2167e33b6b18474da24ff18f46bc56976..15f2de809304aba619ee327f3273425418ca83de https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Utility-Libraries/+log/9123b8c04b378c1b81899314e3060a80f255a24b..87ab6b39a97d084a2ef27db85e3cbaf5d2622a09 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/7d88badaa519c125a2a7a4c041634acb9baba52d..58b4fcad7be86c8f60baae3cfb2fb45096b437e8 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: yuxinhu@google.com Change-Id: Ie5f103164dc63689b681f9102a731254a5480ed5 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6001579 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll a04cedac 2024-11-07T07:00:09 Roll Chromium from aba74d57387d to 369ac471ea04 (477 revisions) https://chromium.googlesource.com/chromium/src.git/+log/aba74d57387d..369ac471ea04 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/5534012c41..0fc767f93a * testing: https://chromium.googlesource.com/chromium/src/testing/+log/9958df0c62..cb21bddb7a * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/03d405a15f..0ff74cfe9f * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/b1a4d63356..171c504c4b * third_party/r8/cipd: YyLscJSIeKgVII-9Cs-Apvzq0aP3NL1pFHvlh4ZXDNwC..YpJxENNZqq3mMt24rEGk3QC1-tzRwn6Tu5HD2vsxzGIC * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/4f4782e7c7..3850115d14 * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/e0c1633a7a..830220aa08 * tools/mb: https://chromium.googlesource.com/chromium/src/tools/mb/+log/a4a07db8ee..8fe44e5d21 * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/ee092511cb..bd85c54890 * tools/skia_goldctl/linux: ab2tOr3TlFy5RVGJNCDNzx7GN0z5CmqjNjihGpMiJuoC..C0CwX5cN6vEo8BX2QpfBB0hMQ6Yg_BpTgtW6kED6a0QC * tools/skia_goldctl/mac_amd64: x7CeAkfbwULBde4pP6MdPN62gDQ-9F310_BaLecUP_4C..xmAykgoYIP74ev2nA3hQCJv5t-THH1oVqgTA8O4Sfu4C * tools/skia_goldctl/mac_arm64: x7HadrOySXOgPFJ2HqfBPetsWzNrSqufiw_1TqdrbZkC..B_QRS0ZdBtI8oL8MDGOn6KQaMsdqpc-nV2lKWRGlOKAC * tools/skia_goldctl/win: UsIbI7R1j3k4UfymAMNHHW2qV_tFb1pC9FjE8d7cpoUC..2J10fvw0AgR3Dpcg6wIiMTbhXIqL2J_QDOSXlOTBfpwC No update to Clang. Bug: None Tbr: yuxinhu@google.com Change-Id: I8019c49a937012c452e89ecabacf27e8652a4a43 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6001578 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll 026ba848 2024-11-06T16:14:21 Manual roll Chromium from 64d6e30da907 to aba74d57387d (848 revisions) Manual roll requested by ynovikov@google.com https://chromium.googlesource.com/chromium/src.git/+log/64d6e30da907..aba74d57387d If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,ynovikov@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/4e8de165cd..5534012c41 * buildtools/reclient: re_client_version:0.170.0.08051991-gomaip..re_client_version:0.171.0.1869ad5c-gomaip * testing: https://chromium.googlesource.com/chromium/src/testing/+log/ce678c02c7..9958df0c62 * third_party/abseil-cpp: https://chromium.googlesource.com/chromium/src/third_party/abseil-cpp/+log/78991980f5..1ffd70268e * third_party/android_build_tools/error_prone/cipd: XiEOJCA7q-Q8YABdSSaYOW06nR1GvKOoSAxrsNzxEBYC..P4ar2IVWgipWzMglFQGb45LE-VKychgjOUmbssZK7qcC * third_party/android_build_tools/lint/cipd: hSFon_JjQvR4BuNkKM_aUTmw20ucssOQ2oFB2eXiw2EC..ZtEs7jgil5yykPwZynLNdQuisSTtHKp4uUi7aZHTc-YC * third_party/android_build_tools/manifest_merger/cipd: WkTffFZv5M3W9L_yy3Z8ZQLMagWNQgAN_QmLR8BzCHEC..-Yqd1ysKh8HXbxIODxAAUrZ6hSS9NCVF1fceEtPMCPsC * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/3a454956bc..31beb497fc * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/37c0cf9100..03d405a15f * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/53f8886e48..b1a4d63356 * third_party/r8/d8/cipd: zNYp2KVrHI9KDsLQCdJIFzXTvkqx3Dav9o5vR1UlMxQC..MnTG4zEzUB1GchkIsnkYCRziw_tkkWGJ7v_T6NUJHksC * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/1d14a955d9..4f4782e7c7 * tools/luci-go: git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b..git_revision:17620695ace472852acc003b74bf240c94a07cbc * tools/luci-go: git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b..git_revision:17620695ace472852acc003b74bf240c94a07cbc * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/751a4f4982..ee092511cb * tools/skia_goldctl/linux: Xe04Orb62kNw7G2RahBXWQgnaFaSVnePM49IX9xxOZQC..ab2tOr3TlFy5RVGJNCDNzx7GN0z5CmqjNjihGpMiJuoC * tools/skia_goldctl/mac_amd64: yW207_i1jDYnbWVrWjhE71mVOkcoovgS05scBJCeJzMC..x7CeAkfbwULBde4pP6MdPN62gDQ-9F310_BaLecUP_4C * tools/skia_goldctl/mac_arm64: 7AVueq6vBSRyrlg5DE8fULb1Nv8b8LnSgfLkUh_t9uUC..x7HadrOySXOgPFJ2HqfBPetsWzNrSqufiw_1TqdrbZkC * tools/skia_goldctl/win: 4Tr5RFJLbgif5NawL-lkD1cSxwBi2-xjqufcRBMG-4kC..UsIbI7R1j3k4UfymAMNHHW2qV_tFb1pC9FjE8d7cpoUC No update to Clang. Bug: None Tbr: yuxinhu@google.com,ynovikov@google.com Change-Id: Ifcd46248190e10bf1c88a4796ffaf0cc954f40d8 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/6000614 Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Reviewed-by: Yuxin Hu <yuxinhu@google.com>
Shahbaz Youssefi f44427b5 2024-11-05T15:18:59 Vulkan: Fix MSAA glReadPixels into PBOs The temporary image used to resolve the MSAA framebuffer during glReadPixels did not have the SAMPLED usage bit. Additionally, the image was supposed to be garbage collected afterwards but ImageHelper's release() function was accidentally immediately destroy()ing it. This was not an issue with blocking glReadPixels paths, because the command buffer was immediately flushed and the GPU work was waited on before the image was destroyed in RendererScoped's destructor. Bug: b/377437834 Change-Id: I1dca47172d6f363277059a848fe9446ac2a872d4 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5995530 Commit-Queue: Charlie Lao <cclao@google.com> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Charlie Lao <cclao@google.com>
angle-autoroll d399841a 2024-11-06T10:01:52 Roll SwiftShader from 1495532f997f to 76855a9baecc (1 revision) https://swiftshader.googlesource.com/SwiftShader.git/+log/1495532f997f..76855a9baecc 2024-11-05 swiftshader.regress@gmail.com Regres: Update test lists @ 1495532f If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in SwiftShader: https://bugs.chromium.org/p/swiftshader/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: yuxinhu@google.com Use-Permissive-Angle-Pixel-Comparison: True Change-Id: Idf15cd2ed21d3b4c026cfdbbbd77386dc32380fe Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5997998 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Roman Lavrov cc2edfd1 2024-11-05T15:37:21 Fix getPerfMonitorCounterData maxResults, skip AsyncCommandQueue test Skips TextureDataInLoopManyTimes on SwS AsyncCommandQueue. Add more debug info to the test although without the AsyncCommandQueue variant we likely won't be see flakes. Bug: angleproject:377503738 Change-Id: I372531451c280b04ffed84b020463657440fca08 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5995531 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Roman Lavrov <romanl@google.com>
Gowtham Tammana 644b91f7 2024-02-29T16:37:37 CL/Vulkan: Implement buffer map/unmap Implement the buffer map, for now mapping is performed at the call site. Bug: angleproject:42267077 Change-Id: I16929920be8a49863de514b541ff0a57b6ac4453 Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5971713 Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Lichen Liu d4420489 2024-10-25T08:53:17 Add check for iOS simulator when initializing caps in metal On Intel-arch MacBook, the system API [supportsFamily:MTLGPUFamilyApple3] returns false on the iOS simulator, causing maxVertexOutputComponents to be set to 60, which is below than the required value of 64, resulting in WebGL2 initialization failure on chromium iOS. Add a TARGET_OS_SIMULATOR check to ensure the values of caps can be set correctly in the siumulator environment. Bug: angleproject:375244081 Change-Id: Iafc44a2aa62130b09127bbec7ea6940506a4165c Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5961039 Reviewed-by: Kimmo Kinnunen <kkinnunen@apple.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Austin Annestrand 6df20e5f 2024-08-21T13:29:36 CL/VK: Add missing HostPtr-BufferVk sync on unmap Missing extra copy/sync for enqueueUnmapMemObject routine for USE_HOST_PTR scenario. Bug: angleproject:377366930 Change-Id: Ifcb0ed58beabf5870e00ed47b36cfd08767eba70 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5990104 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Austin Annestrand <a.annestrand@samsung.com> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Shahbaz Youssefi 2a569b2b 2024-11-04T11:41:29 Vulkan: Document that hex can be viewed with spirv-dis Since https://github.com/KhronosGroup/SPIRV-Tools/pull/5870, spirv-dis is able to disassemble SPIR-V given a hex stream. This change indicates this when retrieving shader debug info (a WebGL extension). Bug: angleproject:42266948 Change-Id: Ifab4668d6608093700d3e03ddd7ea675fd2bcf42 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5988173 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org>
Mohan Maiya 17a01469 2024-10-26T06:07:31 Vulkan: Bugfix TextureVk::generateMipmap Add support for generating mipmaps of textures that are EGLImage texture targets with colorspace overrides Bug: angleproject:40644776 Tests: ImageTestES3.SourceAHBTarget2DGenerateMipmap* Change-Id: I9b4ff802f4118a42d54dc8d80ab30e2f9958bfee Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5966623 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Charlie Lao <cclao@google.com> Commit-Queue: mohan maiya <m.maiya@samsung.com>
Mohan Maiya e2cd9082 2024-11-05T04:07:17 Vulkan: Bugfix in setCurrentImageLayout Make sure to update mLastNonShaderReadOnlyLayout and mCurrentShaderReadStageMask when updating the current layout. Bug: angleproject:40644776 Change-Id: Ie8652099a0d4caca9f9aea5bac38256a513b08e7 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5992020 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Charlie Lao <cclao@google.com> Commit-Queue: mohan maiya <m.maiya@samsung.com>
angle-autoroll 5a9f361c 2024-11-05T08:39:33 Roll Chromium from 69b5e685119d to 64d6e30da907 (601 revisions) https://chromium.googlesource.com/chromium/src.git/+log/69b5e685119d..64d6e30da907 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/679ccfd7d2..4e8de165cd * testing: https://chromium.googlesource.com/chromium/src/testing/+log/10d0c29aa1..ce678c02c7 * third_party/android_build_tools/error_prone/cipd: O1OVfpqbI-kjqiCSF27xOQ2CmfQMGSY-Lq4qH671sdMC..XiEOJCA7q-Q8YABdSSaYOW06nR1GvKOoSAxrsNzxEBYC * third_party/android_build_tools/lint/cipd: 9quVbgwnFSKuZJ3QNXnfzy0iokvyHFOL1Q7qkgafP1MC..hSFon_JjQvR4BuNkKM_aUTmw20ucssOQ2oFB2eXiw2EC * third_party/android_build_tools/manifest_merger/cipd: rdroZtdhMUv8gACwfRmE_Gycwkb8DTSDjw-PpVSx1nIC..WkTffFZv5M3W9L_yy3Z8ZQLMagWNQgAN_QmLR8BzCHEC * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/39b2e4efd6..37c0cf9100 * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/191356bd99..53f8886e48 * third_party/libunwind/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind.git/+log/bf062897f1..c3b4ce8edd * third_party/siso/cipd: git_revision:bf523d95598ccdc9003361bf8a4142414af474f1..git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/6a846861ae..1d14a955d9 * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/c7e4aee0f5..e0c1633a7a * tools/luci-go: git_revision:4797ba66f62a28b1bedf6790134a0cc6d1fe89ea..git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b * tools/luci-go: git_revision:4797ba66f62a28b1bedf6790134a0cc6d1fe89ea..git_revision:f0f43ce899b4da811f906ab3557c9b4b5418769b * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/a4a0ab9ea5..751a4f4982 * tools/skia_goldctl/linux: TFJRb4k3TwngoVhErGjZ2UWUBCpvMrPlLBPpA0vhw_AC..Xe04Orb62kNw7G2RahBXWQgnaFaSVnePM49IX9xxOZQC * tools/skia_goldctl/mac_amd64: Qdi4Rk6BKCq4Nh1ayE1--3i1ZGo5RdOkCMcD3NC9StAC..yW207_i1jDYnbWVrWjhE71mVOkcoovgS05scBJCeJzMC * tools/skia_goldctl/mac_arm64: 9hJI5SrV6oIvn3p0aspVvHaI3MYOB2C_T9v3epRey60C..7AVueq6vBSRyrlg5DE8fULb1Nv8b8LnSgfLkUh_t9uUC * tools/skia_goldctl/win: gLMz4_WFdf2HxxnlVjNztw4Dcl1tfttWR1bHdsFxdssC..4Tr5RFJLbgif5NawL-lkD1cSxwBi2-xjqufcRBMG-4kC No update to Clang. Bug: None Tbr: yuxinhu@google.com Change-Id: Ie03484a23118a7599486afb73383990a36eab690 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5993374 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Gowtham Tammana 84a24a1e 2024-07-02T11:56:25 CL: Implement clone for kernel object clCloneKernel() can be used to make a shallow copy of a kernel object. The returned object is an exact copy of the source kernel, with latest calls to arguments sets applied to the kernel. Implement a clone version at the front-end with replicating the `clSetKernelArg` called on the source kernel. The `clSetKernelArgsSVMPointer` and `clSetKernelExecInfo` are ignored as they are currently unsupported. Bug: angleproject:376023087 Change-Id: I2e21931d9ff799e35b153cffe91fd4a2adbbf88d Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5971715 Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Gowtham Tammana 8dae26c6 2024-06-26T11:35:24 CL: Add missing validation checks -1- Check on device enqueue support Add a check for the device enqueue support before proceeding with the device enqueue info queries. -2- Validate the non uniform work groups The spec requires reporting CL_INVALID_WORK_GROUP_SIZE when non-uniform workgroups are not supported and supplied work sizes dont evenly distribute the work items. Adding a check in the validation. -3- Check for alignment on sub-buffer creation The spec requires the sub-buffer origin to be aligned to CL_DEVICE_MEM_BASE_ADDR_ALIGN value. Check for this in validation layer. Bug: angleproject:42267011 Change-Id: I9dc086fea16b200cc05c0fca29e3608403099a2c Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5975419 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Roman Lavrov 1daf17b5 2024-11-04T11:05:46 Tests: restore angle_end2end_tests --help on Android https://crrev.com/c/5985232 added support for choosing the test runner, but that inadvertently changed behavior with --help (or with certain flags). This CL restores that behavior. parse_known_args() exits the process by default, so add manual --help handling. Bug: b/344849382 Change-Id: I87f197f90a161ec141a6e1f46ba81cfbf01a2fec Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5987387 Commit-Queue: Roman Lavrov <romanl@google.com> Reviewed-by: Amirali Abdolrashidi <abdolrashidi@google.com>
angle-autoroll 924f2079 2024-11-04T13:55:18 Manual roll Chromium from 601f829c4935 to 69b5e685119d (60 revisions) Manual roll requested by ynovikov@google.com https://chromium.googlesource.com/chromium/src.git/+log/601f829c4935..69b5e685119d If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,ynovikov@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/43a2115b4e..679ccfd7d2 * testing: https://chromium.googlesource.com/chromium/src/testing/+log/cda399271a..10d0c29aa1 * third_party/turbine/cipd: YDjqj5VA9_8SsjmFf_gJG3tSTb6BdOLqQnr8SabZmFEC..1iVaPZ0ts_R6ymFUftgOKVE50dXSx6duVZEOP891WCMC No update to Clang. Bug: None Tbr: yuxinhu@google.com,ynovikov@google.com Change-Id: I434225a7b07653e005df9cdccbab7348618b15d0 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5987707 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Roman Lavrov 47fafdb9 2024-11-04T10:05:12 Disable tracegz (trace interpreter) by default, remove from CI Trace interpreter is incomplete and we've been punting on it. Can be re-enabled with angle_enable_tracegz=true in gn args. This avoids accidentally pulling trace sources into builds. Bug: b/376300037 Change-Id: Ib755fa9f57885216b02a481fd670a1bb0266f0db Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5987382 Reviewed-by: Cody Northrop <cnorthrop@google.com> Commit-Queue: Roman Lavrov <romanl@google.com>
Roman Lavrov e43d3591 2024-11-01T16:23:27 Tests: allow choosing Chromium/our test runner + screen checks Chromium test runner (build/android/test_runner.py) is currently the default runner for most suites. One exception is angle_trace_tests where we're always using our scripts instead (android_helper.py etc). We do that by setting android_test_runner_script to our runner for that angle_test instance. This CL supports setting android_test_runner_script to our script but then choose whether to run with the Chromium runner or android_helper depending on the command-line arg (--angle-test-runner) This CL also adds a check for the screen state which will be performed regardless of the runner. This is to have a fast and clean failure when the device wasn't properly initialized by the infra (see bug). The check is based on the output of `dumpsys deviceidle | grep mScreen` (this is the most generic check I found, I previously used nfc but that depends on nfc and turns out the output is device-dependent) This CL enables this mode for angle_end2end_tests. Bots will continue to run with Chromium runner as before (but with the added screen checks). Locally, we will now be able to switch to our scripts directly: out/Android/angle_end2end_tests \ --gtest_filter='*ClearTextureEXT2DMSStencil*' \ --angle-test-runner Now when running end2end tests locally (regardless of the runner) with a locked device screen, we'll see the following error right away instead of sporadic failures or stuckness during execution: Exception: Unexpected device screen state: ['mScreenOn=false', 'mScreenLocked=true']; expected: ['mScreenOn=true', 'mScreenLocked=false'] The plan is to enable this mode for all suites, after which the current local script run_angle_android_test.py will no longer be needed. Bug: b/344849382 Change-Id: I77e7733804ab37e2f3d26bf3574a52a9553e4274 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5985232 Reviewed-by: Cody Northrop <cnorthrop@google.com> Commit-Queue: Roman Lavrov <romanl@google.com>
angle-autoroll 7ce8b268 2024-11-04T10:01:36 Roll vulkan-deps from a52547961655 to 155bbe2e1429 (7 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/a52547961655..155bbe2e1429 Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools/+log/e5c1498afa1daa8cf3a48af7a74e3b7429afb05d..9aaa2601400a29ce39ce1111aa9f790e5c9d4c1b https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Headers/+log/f2eb740f3f0340ad941e563484d5ba45a5ec6af0..cbcad3c0587dddc768d76641ea00f5c45ab5a278 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/b8093e3a1854285cbd183fa98b410090889fa500..7d88badaa519c125a2a7a4c041634acb9baba52d If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: yuxinhu@google.com Change-Id: I44727ebef8f9161749620187f471daa9bb971846 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5987530 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll d06410da 2024-11-04T09:04:56 Roll Chromium from d8c3950b24a5 to 601f829c4935 (447 revisions) https://chromium.googlesource.com/chromium/src.git/+log/d8c3950b24a5..601f829c4935 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,yuxinhu@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * testing: https://chromium.googlesource.com/chromium/src/testing/+log/8faf4d87eb..cda399271a * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/77ab8f9211..3a454956bc * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b832657a0c..39b2e4efd6 * third_party/fuchsia-sdk/sdk: version:25.20241025.6.1..version:25.20241102.1.1 * tools/luci-go: git_revision:4a7755a0f40eb3aa20ff4dcecde6bab0f6a24889..git_revision:4797ba66f62a28b1bedf6790134a0cc6d1fe89ea * tools/luci-go: git_revision:4a7755a0f40eb3aa20ff4dcecde6bab0f6a24889..git_revision:4797ba66f62a28b1bedf6790134a0cc6d1fe89ea * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/715bc5a0af..a4a0ab9ea5 * tools/skia_goldctl/linux: VabgQkFnMfqcuiR6DAjGRzgo-flvZDQ3ZOqJSRnONWwC..TFJRb4k3TwngoVhErGjZ2UWUBCpvMrPlLBPpA0vhw_AC * tools/skia_goldctl/mac_amd64: 6LbtEUb4VFflDPP_gLkZm69GPz_kldTfr0GKZqTcY10C..Qdi4Rk6BKCq4Nh1ayE1--3i1ZGo5RdOkCMcD3NC9StAC * tools/skia_goldctl/mac_arm64: ZBwJMfIfaokP3eCIzjf9r2JF1lxyFoWutFyAIMJIkAkC..9hJI5SrV6oIvn3p0aspVvHaI3MYOB2C_T9v3epRey60C * tools/skia_goldctl/win: V37TmFdJ6NMKXBOncF_8W9vtnJJRfLQIVmoHmDsH4LkC..gLMz4_WFdf2HxxnlVjNztw4Dcl1tfttWR1bHdsFxdssC No update to Clang. Bug: None Tbr: yuxinhu@google.com Change-Id: Ie2eb3f160f202beb95f92d3c9de3188f672207ed Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5986476 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Kimmo Kinnunen eccfec93 2024-11-01T11:49:11 Metal: gl_ClipDistance fails validation Failure is "Found child with two parents". Result from reusing gl_ClipDistance in: float gl_ClipDistance[8]; ClipDistance_0 = gl_ClipDistance[0]; ... Fix by creating new node for each symbol access. Bug: angleproject:376718272 Change-Id: I7a866a10e389cc25d0cff4e6f18ae9c70ab376e0 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979781 Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com>
Gowtham Tammana 7483897c 2024-10-01T17:23:37 CL/Vulkan: Add numeric versioning for extensions With extended versioning support, the version number of extensions need to be reported as well. Using the numeric versioning mode for adding supported extensions. Also, fix the version number for cl_khr_icd and cl_khr_extended_versioning extensions. Bug: angleproject:372085147 Change-Id: I7bf3157227f9579c69c351a63a88b92be7f07d71 Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5916159 Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Gowtham Tammana 9ce9e678 2024-04-16T10:07:02 CL/Vulkan: Set storage buffer usage for cl buffers clspv is setting all the buffer objects as storage buffers, so removing the distinction on the CL MEM flags for now. Bug: angleproject:365694588 Change-Id: I3bd65457d53a6a2fb5fd5c8f2a0b8c25094ed563 Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5916158 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Gowtham Tammana 2f8ad9c1 2024-09-26T20:34:34 CL/Vulkan: Add support for sub-buffer creation Sub-buffer is setup to point to offset location of the parent buffer. The copy buffer interface is updated to account for copies from the same sub-buffer. Bug: angleproject:372157566 Change-Id: I668c7182a8c4c12e265115f1c28d9bfbfe562bb0 Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5916156 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Rafay Khurram 3c1e98a3 2024-05-22T01:46:52 CL/Vulkan: Fix clEnqueueMapImage/clEnqueueUnmapMemObject * Updates slice_pitch and row_pitch parameters with relevant values according to the spec. * For mapping, image is copied to staging buffer, then copied to dynamically allocated array * For unmapping, the reverse is done with the array being freed Tests-Passing: OCLCTS.test_cl_copy_images 2D, test_cl_copy_images small_images 2D, test_cl_copy_images max_images 2D, test_cl_copy_images 3D, test_cl_copy_images small_images 3D, test_cl_copy_images max_images 3D, Bug: angleproject:42266936 Change-Id: I0cd778244832ce66e72a58a228ed889a9002171e Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5961948 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Rafay Khurram <r.khurram@samsung.com> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Shahbaz Youssefi bd9d0285 2024-10-26T01:12:23 Remove feature description / condition enums Since ANGLE no longer produces anything for these strings, Chromium no longer queries them and they can be removed. Bug: chromium:371512561 Change-Id: I921cbd91a3328df99b15262d31d52986d4a7b806 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5966697 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
angle-autoroll 5242386b 2024-11-01T20:29:53 Manual roll Chromium from 61c4298a5d2e to d8c3950b24a5 (286 revisions) Manual roll requested by syoussefi@google.com https://chromium.googlesource.com/chromium/src.git/+log/61c4298a5d2e..d8c3950b24a5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,romanl@google.com,syoussefi@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/a378269311..43a2115b4e * testing: https://chromium.googlesource.com/chromium/src/testing/+log/8aa2471595..8faf4d87eb * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/c73dc115ba..b832657a0c * third_party/protobuf: https://chromium.googlesource.com/chromium/src/third_party/protobuf/+log/29d8a79ce5..0cbab48ca5 * third_party/r8/cipd: RzOonm113-m5_HZcKgMupfMghkDOTHWOJGYHjaaWfRIC..YyLscJSIeKgVII-9Cs-Apvzq0aP3NL1pFHvlh4ZXDNwC * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/c9e8288d5f..6a846861ae * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/c67de0cebb..715bc5a0af No update to Clang. Bug: None Tbr: romanl@google.com,syoussefi@google.com Change-Id: Ib7dc6f375c6ccc69dc3e38629157538b9392bf96 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5985113 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Charlie Lao 2b8d6bbe 2024-11-01T11:23:35 Vulkan: Use UpdateFullTexturesDescriptorSet when cache missed DescriptorSetDescBuilder::updateDescriptorSet() relies on the cache key to build descriptorSet. UpdateFullTexturesDescriptorSet() builds descriptorSet directly from state, it does not use cache key. Test shows UpdateFullTexturesDescriptorSet is much faster than updateActiveTexturesForCacheMiss and updateDescriptorSet pair. This CL removes updateActiveTexturesForCacheMiss() function and uses UpdateFullTexturesDescriptorSet for cache miss case. The timing code is added around the cache miss functions to measure the time. Old: asphalt_9 average 7,554 nanosec gl_driver2_off: 20,354 nanosec batman_telltale: 12,992 nanosec New: asphalt_9 average 916 nanosec gl_driver2_off: 1,839 nanosec batman_telltale: 3,437 nanosec Bug: angleproject:372268711 Change-Id: I176d67ed732c3fe3a18a079df7c4973aa926087a Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5984893 Reviewed-by: Yuxin Hu <yuxinhu@google.com> Commit-Queue: Charlie Lao <cclao@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Charlie Lao fbe34df7 2024-10-29T16:19:29 Vulkan: More texture descriptorSet code cleanup Removed unused argument `pipelineType` from updateActiveTexturesForCacheMiss(). Removed unused argument `context` from getReadImageView() Rename getBufferViewAndRecordUse() to getBufferView() since there is no "record use" happening. Moved UpdateFullActiveTexturesDescriptorSet() function from vk_cache_utils.cpp to ProgramExecutableVk.cpp anonymous name space, since it is only used in this file. Bug: angleproject:372268711 Change-Id: Ib7240c1063f727fb52588234e79fba349f9aff9e Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5977481 Reviewed-by: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Charlie Lao <cclao@google.com>
Gowtham Tammana 79b6c7ab 2023-10-09T13:29:07 CL/Vulkan: Add fillWithPattern interface In CL, the buffer can be requested to filled with a pattern. Adding a pattern fill helper routine that fills up the buffer from CPU side. Bug: angleproject:42267074 Change-Id: I144e9b7c6f4d1263f21cabc2491c46e8951e604f Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5916157 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org>
Charlie Lao 1a3fadbf 2024-11-01T13:19:43 Vulkan: Enable imagelessFB for recent QualComm drivers Qualcomm recent drivers has the fix for imageless framebuffer bug. Re-enable supportsImagelessFramebuffer for drivers 512.802.0 or newer. Bug: b/369693310 Change-Id: I33a69b8abfefed572880636ff7af2c788cee5688 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5984959 Commit-Queue: Charlie Lao <cclao@google.com> Reviewed-by: Cody Northrop <cnorthrop@google.com>
Rafay Khurram c0a28403 2024-05-06T12:42:28 CL/Vulkan: Enable clEnqueueNDRangeKernel for Images and Samplers * Performs clspv reflection for ArgumentSampler, ArgumentStorageImage, and ArgumentSampledImage * Creates descriptor sets for performing enqueue Tests-Passing: OCLCTS.test_basic readimage, readimage_fp32, readimage_int16, writeimage, writeimage_fp32, writeimage_int16, mri_one, mri_multiple, imagenpot Bug: angleproject:42266936 Change-Id: I2b6c631e76556870c4342d2046c267ff5cf5105d Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5955597 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Rafay Khurram 0baeb12e 2024-10-22T19:24:09 CL/Vulkan: Fix ImageDescriptor constructor * Constructor should set the depth to 1 for 2D images rather than 3D Tests-Passing: OCLCTS.test_basic imagecopy3d, imagearraycopy3d Bug: angleproject:42266936 Change-Id: Id6fe927260a808732c29b3b051562274ee9bf50d Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5955542 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Rafay Khurram c3ff2bbe 2024-05-16T11:31:54 CL/Vulkan: Enable clGetSupportedImageFormats * Returns formats that are supported by physical device from the minimum list of image formats Tests-Passing: OCLCTS.test_api min_image_formats, OCLCTS.test_basic imagearraycopy, arrayimagecopy, OCLCTS.test_computeinfo computeinfo Bug: angleproject:42266936 Change-Id: I2bdb9a793b45f21c12c7c7d6a42e7a63295c2708 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5800266 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Rafay Khurram 9db2e88b 2024-05-14T11:44:01 CL/Vulkan: Add support for required image formats * Included adding cases for CL_DEPTH, CL_DEPTH_STENCIL, CL_sRGBA, and CL_BGRA Bug: angleproject:42266936 Change-Id: Ie7bd3ee6c6413d8fa89357c81d19f4c807883f36 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5800265 Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Chris Dalton a05a0e15 2024-09-25T22:33:36 Validate PLS shaders against context state Add shader introspection for PLS uniforms and validate that they match context state during draw calls. Bug: angleproject:40096838 Change-Id: I76cdf8add03de8f8b0b3e772c15c0087c1d97e98 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5893962 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Kenneth Russell <kbr@chromium.org>
Rafay Khurram a21b7ad0 2024-04-24T02:11:42 CL/Vulkan: Add skeleton for CLSamplerVk * It is setup to be a wrapper for the SamplerHelper interface Bug: angleproject:42266936 Change-Id: Iac7e80c4d5262687d98a8188a60a24a9be190dc2 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5801184 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Kimmo Kinnunen b03f0148 2024-11-01T10:26:04 Metal: interpolateAtOffset fails validation Fix validation error Found function prototype with an invalid qualifier. The offset parameter was created with "global" qualifier. Bug: angleproject:376718268 Change-Id: I6f314a42fe1d35d54bb673cfa7d58eba526c9ea5 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979778 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Shahbaz Youssefi c0c541da 2024-10-28T11:54:26 Remove the gl+d3d-only build of the translator This was done at a time where the spir-v generator used glslang and added significant binary size. Bug: chromium:371512561 Change-Id: I6cc733e51f8af555d20181a328b4fbeba6c053c3 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5971194 Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Gowtham Tammana 57ce489f 2024-07-02T10:25:35 CL: Check that arguments are set at enqueue call On clEnqueueNDRangeKernel, all of the kernel arguments have to be set by appropriate clSetKernelArg or clSetKernelArgsSVMPointer or setKernelExecInfo calls. Add a validation check for the same. For now, only clSetKernelArg are checked as the others are not supported yet. Bug: angleproject:42267011 Change-Id: I15be8efab80f475ae6093ab7e1798e7a04182ac0 Signed-off-by: Gowtham Tammana <g.tammana@samsung.com> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5971714 Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Charlie Lao 5c26ffea 2024-10-29T11:11:38 Vulkan: Optmize descriptorSet cache disable code path Right now the way it works is that it first computes the cache key and then use the cache key to look in the cache. If cache misses, then it builds descriptorSet out of the cache Key. This might make sense if cache is enabled. If cache is disabled then no need to go through the middle man. This CL skip all the cache key build up entirely and directly build descriptorSet out of context state. In this CL, updateFullActiveTextures() and updateDescriptorSet() are merged into one function UpdateFullActiveTexturesDescriptorSet() which updates VkWriteDescriptorSet directly. Bug: angleproject:372268711 Change-Id: I7ba0c60a23b967d1ac903020d04022405c29e354 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5972508 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Yuxin Hu <yuxinhu@google.com> Commit-Queue: Charlie Lao <cclao@google.com>
angle-autoroll f4a239bc 2024-11-01T10:01:44 Roll vulkan-deps from 977783acab11 to a52547961655 (6 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/977783acab11..a52547961655 Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools/+log/5f1e86cb55c48949d8a25eb8e03e6930c4179ed9..e5c1498afa1daa8cf3a48af7a74e3b7429afb05d https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/2e3b9e9d7ba562187bff4b92b6a9dae6b1824008..b8093e3a1854285cbd183fa98b410090889fa500 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,romanl@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: romanl@google.com Change-Id: Ib0ffb9ba33866be060d730a211521f75dbc5c917 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5984150 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
angle-autoroll 2b6e5b2e 2024-11-01T07:00:49 Roll Chromium from c0123634901e to 61c4298a5d2e (219 revisions) https://chromium.googlesource.com/chromium/src.git/+log/c0123634901e..61c4298a5d2e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,romanl@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/933340a792..a378269311 * testing: https://chromium.googlesource.com/chromium/src/testing/+log/103f6af5f0..8aa2471595 * third_party/android_build_tools/lint/cipd: c7Q2GAMqK00nzjHPowtl0at22vglh6T3CrvXLOhpXbEC..9quVbgwnFSKuZJ3QNXnfzy0iokvyHFOL1Q7qkgafP1MC * third_party/android_build_tools/manifest_merger/cipd: Pb7MFARSTl0ZjwRKHTt3BHBXnNL4WkxZcE6CYILj_zgC..rdroZtdhMUv8gACwfRmE_Gycwkb8DTSDjw-PpVSx1nIC * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/61356f2ec3..77ab8f9211 * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/9bf2a2011a..c73dc115ba * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/88c655feba..191356bd99 * tools/luci-go: git_revision:e76a7fa7e1d656b1d6a484e3fcf3b02d0717f87a..git_revision:4a7755a0f40eb3aa20ff4dcecde6bab0f6a24889 * tools/luci-go: git_revision:e76a7fa7e1d656b1d6a484e3fcf3b02d0717f87a..git_revision:4a7755a0f40eb3aa20ff4dcecde6bab0f6a24889 No update to Clang. Bug: None Tbr: romanl@google.com Change-Id: Id3ea8ac1887f857505b47adb34138b12a69d3b10 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5984149 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Kimmo Kinnunen bf29a047 2024-10-28T11:07:57 Metal: Remove uniform struct decl separation code Remove uniform specific struct declaration separation and anonymous struct naming code. This is already done by SeparateDeclarations. Removes code for NameEmbeddedUniformStructsMetal. Does not remove SeparateStructFromUniformDeclarations code as it is used for SPIRV. Bug: angleproject:375937551 Change-Id: I943d96897d97b23fd0f54155dc215a4490f63937 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5972994 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com>
Kimmo Kinnunen 2156cd6e 2024-10-31T14:22:31 Metal: Fix rewritten array variables clashes It was possible to generate MSL name clashes by declaring arrays, such as in: mat3 a[1] mat2 a_0; Complex GLSL outputs need to be linearized into primitive MSL types. This would happen for struct outputs, arrays and matrices. For these new MSL variables, the translator needs to invent new names that match, generated from both VS and FS. The clashes were due to the encoding scheme: For arrays, the MSL vertex output variable was named with _0 suffix for the array and _0 for the matrix row 0. In the example above, this would create a clash since also non-array mat would be linearized row-wise. For structs, field `a.b` would be encoded as `a_b`. By selecting different a and b, clashes could be generated trivially. It is redundant to encode named semantics in the out variable names. The only needed element is the discriminator which associates VS variables to FS variables. Currently this is done by using the user-provided root field name. Fix by encoding only the root variable name to the MSL variable name. The rest of the field discriminators are just a running number. Each GLSL name `a` is unique. MSL names get fixed suffix form `a_X`, and thus all of them are unique too. This is a continuation of following fixes that turned out to be incomplete: 6fe8a399dd Metal: Fix rewritten out variables with underscores 27423bffff Metal: Generate names for rewritten inputs Bug: angleproject:376417347 Change-Id: I407db373d201b3e321c6d8414bafdbecfd82cf19 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979774 Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com>
Rafay Khurram 7c99c225 2024-04-03T01:48:57 CL/Vulkan: Implement clEnqueue APIs involving images Review enables the functionality of: - clEnqueueReadImage - clEnqueueWriteImage - clEnqueueCopyImage - clEnqueueMapImage - clEnqueueCopyImageToBuffer - clEnqueueCopyBufferToImage Tests-Passing: OCLCTS.test_basic imagecopy, imagerandomcopy, OCLCTS.test_profiling copy_array_to_image Bug: angleproject:42266936 Change-Id: Ifd4ccd0a9e16bac11dbff74d4634d9ca4c5b4284 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5796530 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Rafay Khurram <r.khurram@samsung.com>
Kimmo Kinnunen 0624b4fb 2024-10-31T20:59:20 Metal: Make ToposortStructs compile on c++17 Use .find() != .end() instead of .contains() for std::unordered_map. Bug: angleproject:375352601 Change-Id: I2e550354e1df3b390b74fdea29427fd3a0326fe8 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979775 Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
angle-autoroll 20de3a8a 2024-10-31T22:06:42 Manual roll Chromium from ada7221c4738 to c0123634901e (3970 revisions) Manual roll requested by romanl@google.com https://chromium.googlesource.com/chromium/src.git/+log/ada7221c4738..c0123634901e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromium-angle-autoroll Please CC angle-team@google.com,romanl@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Chromium: https://bugs.chromium.org/p/chromium/issues/entry To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Changed dependencies * build: https://chromium.googlesource.com/chromium/src/build.git/+log/774b608643..933340a792 * buildtools: https://chromium.googlesource.com/chromium/src/buildtools.git/+log/09ea565f4a..c61bf9c942 * buildtools/reclient: re_client_version:0.168.0.c46e68bc-gomaip..re_client_version:0.170.0.08051991-gomaip * testing: https://chromium.googlesource.com/chromium/src/testing/+log/546ad56e8a..103f6af5f0 * third_party/abseil-cpp: https://chromium.googlesource.com/chromium/src/third_party/abseil-cpp/+log/437172bab2..78991980f5 * third_party/android_build_tools/error_prone/cipd: n77J6kWie5cSD7pmkTyni9GjOYh2lyebCU6Eyr6GxXkC..O1OVfpqbI-kjqiCSF27xOQ2CmfQMGSY-Lq4qH671sdMC * third_party/android_build_tools/lint/cipd: 4LvjVBJ-tE7OOEwn-wr9aM6zYpZjl2bZ6jvbfAdCo3wC..c7Q2GAMqK00nzjHPowtl0at22vglh6T3CrvXLOhpXbEC * third_party/android_build_tools/manifest_merger/cipd: 92AO-IC4QP7pBCfygcFNL6xN7xiJLTSWdiS0q-pq4ukC..Pb7MFARSTl0ZjwRKHTt3BHBXnNL4WkxZcE6CYILj_zgC * third_party/android_deps: https://chromium.googlesource.com/chromium/src/third_party/android_deps/+log/fdebd3d15a..b8ace7091f * third_party/catapult: https://chromium.googlesource.com/catapult.git/+log/940a36cedf..61356f2ec3 * third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/cc924d1ab6..9bf2a2011a * third_party/fuchsia-sdk/sdk: version:25.20241022.0.1..version:25.20241025.6.1 * third_party/ijar: https://chromium.googlesource.com/chromium/src/third_party/ijar/+log/3bd143299e..fda85bf1f2 * third_party/libc++/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxx.git/+log/c8eec3629a..8e31ad4256 * third_party/libc++abi/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libcxxabi.git/+log/5bfc713304..88c655feba * third_party/libunwind/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libunwind.git/+log/59d05179e3..bf062897f1 * third_party/llvm-libc/src: https://chromium.googlesource.com/external/github.com/llvm/llvm-project/libc.git/+log/98ed093183..194ae301ad * third_party/r8/cipd: zNYp2KVrHI9KDsLQCdJIFzXTvkqx3Dav9o5vR1UlMxQC..RzOonm113-m5_HZcKgMupfMghkDOTHWOJGYHjaaWfRIC * third_party/siso/cipd: git_revision:4237c4b45d2accad9955693251a99f0b862a0a03..git_revision:bf523d95598ccdc9003361bf8a4142414af474f1 * third_party/turbine/cipd: dODwcAXI7V4M-fHO0rZvcMEcW3qgfNnkoPeuwxzO4DsC..YDjqj5VA9_8SsjmFf_gJG3tSTb6BdOLqQnr8SabZmFEC * tools/android: https://chromium.googlesource.com/chromium/src/tools/android/+log/4b9a28a9b7..c9e8288d5f * tools/clang: https://chromium.googlesource.com/chromium/src/tools/clang.git/+log/53554bf3da..c7e4aee0f5 * tools/luci-go: git_revision:d212e392e327834378013806f0263ab263600db2..git_revision:e76a7fa7e1d656b1d6a484e3fcf3b02d0717f87a * tools/luci-go: git_revision:d212e392e327834378013806f0263ab263600db2..git_revision:e76a7fa7e1d656b1d6a484e3fcf3b02d0717f87a * tools/mb: https://chromium.googlesource.com/chromium/src/tools/mb/+log/3793b7f7b4..a4a07db8ee * tools/perf: https://chromium.googlesource.com/chromium/src/tools/perf/+log/752c7a08fc..c67de0cebb * tools/rust: https://chromium.googlesource.com/chromium/src/tools/rust.git/+log/ed0fe5c0e0..e4fd8fdeea * tools/skia_goldctl/linux: CXeEV6BLm5q9KGe-brhM2NLdmEgjyMyTutd_o3ioqMwC..VabgQkFnMfqcuiR6DAjGRzgo-flvZDQ3ZOqJSRnONWwC * tools/skia_goldctl/mac_amd64: XrxKbvaH3xwOzOdudnSzNTwqJhocETDPtsYNQNWHeIwC..6LbtEUb4VFflDPP_gLkZm69GPz_kldTfr0GKZqTcY10C * tools/skia_goldctl/mac_arm64: oZ0XmjQl2uVb7JQS4TNGfkr4O_ZGRDCZK7VuTwAdZsoC..ZBwJMfIfaokP3eCIzjf9r2JF1lxyFoWutFyAIMJIkAkC * tools/skia_goldctl/win: fuydFP_nMX2mXcsO2vqFYHv2H1jdpvF95t8AGCcH7wIC..V37TmFdJ6NMKXBOncF_8W9vtnJJRfLQIVmoHmDsH4LkC Clang version changed llvmorg-20-init-8527-g923566a6:llvmorg-20-init-9764-gb81d8e90 Details: https://chromium.googlesource.com/chromium/src/tools/clang/+/53554bf3da41153f2e01f9ff234c194c156b7a93..c7e4aee0f5a2104de4307c016ca57637d506b2fc/scripts/update.py Bug: angleproject:2516 Tbr: romanl@google.com Change-Id: I9e177b89f58db5ddba97ea32c4978001088688fe Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5983810 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Amirali Abdolrashidi 6b9d3762 2024-08-22T15:29:00 Vulkan: Optimize full texture clears Currently, a full texture clear (glClearTexImageEXT()) is treated as a special case of a partial clear (glClearTexSubImageEXT() with image dims as the input). However, it can be further optimized by treating it as a clear update. * For full clears from EXT_clear_texture, the clear update path is taken. * It leads to a more optimized path, including the usage of the following APIs: * vkCmdClearColorImage() * vkCmdClearDepthStencilImage() * It uses the following enum: ClearTextureMode * If a partial clear uses the extents for the entire image, it is treated as a full clear. * Updated the method to determine if a texture is renderable in clearSubImageImpl(). * Added perf counter: fullImageClears * Added new unit tests * Single 3D texture full clear (Clear3DSingleFull) * 2D RGB SNORM clear (Clear2DRGB8Snorm) * Added Vulkan perf counter test for 2D and 3D color image clear. * Updated the related skipped tests on Pineapple. Bug: angleproject:42266869 Bug: angleproject:375425839 Change-Id: I12ef3002dee190d7f8f43204f7d3f76e05d0b54f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5806207 Reviewed-by: Charlie Lao <cclao@google.com> Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Cody Northrop 91ea8aef 2024-10-31T12:33:35 scripts: Add restricted_trace_perf as data dep This gets it in to angle_trace_bundle.py. Bug: b/376300037 Change-Id: Iebffb105d24bd45c346d6d7085051c432039e2ef Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5980801 Reviewed-by: Roman Lavrov <romanl@google.com> Commit-Queue: Cody Northrop <cnorthrop@google.com>
Amirali Abdolrashidi 66701c98 2024-10-31T11:37:44 Vulkan: Remove extra non-conformant flag check In Vulkan, getMaxConformantESVersion() calls another function which already checks which ES version to use (getMaxSupportedESVersion()). It also uses the non-conformance flag to check if it should bypass the ES 3.2 check if the flag is enabled. However, the former uses the flag to cap the version to ES 3.1 if the flag is disabled. This check seems unnecessary, and can cap the version for a device that does have the ability to use ES 3.2. * Removed the non-conformant flag check from getMaxConformant*(). Bug: b/374841339 Change-Id: I377e6ba2fe174aae7e2e1b19407b2acd89749157 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979643 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Amirali Abdolrashidi <abdolrashidi@google.com> Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Solti 02f88b31 2024-10-30T21:49:19 Improve CanSupportAEP Error Reporting This change improves the error reporting for the `CanSupportAEP` function. Previously, if the function returned false, it was difficult to determine which requirement was not met. This change adds logging that will list all unmet requirements, making it easier to debug AEP support issues. For Chromium logs, this logging is too much. Therefore, a switch is used and by default the logging is off. ref: https://crrev.com/c/5979208?tab=comments Test: the default presubmit Bug: b/268108262 Change-Id: I7f633cdb63f9cfeb501cd5f134b0ea7874465aed Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979292 Reviewed-by: Yuxin Hu <yuxinhu@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Kimmo Kinnunen 8d12b278 2024-03-15T15:39:39 Make separated anonymous in/out structs work better Fixes failing case for other backends than GLSL output: VS: out struct { .. } a, b; FS: in struct { .. } a, b Make the ANGLE VS-FS interface matching work by using the anonymous name as the variable structOrBlock name. Bug: angleproject:42267047 Change-Id: If8107387c02b5520134857fb210a27a8f92a2db0 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5372727 Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com> Reviewed-by: Kenneth Russell <kbr@chromium.org>
Roman Lavrov eaa3d4a9 2024-10-31T16:05:34 Add llvm-libc BUILD.gn Not part of llvm-libc, needs to be copied from Chromium Missed in https://crrev.com/c/5980611 Bug: chromium:370535840 Change-Id: Iaf6a042593574715b75b44bc1daf99fd4a6193b9 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5981291 Reviewed-by: Cody Northrop <cnorthrop@google.com> Commit-Queue: Roman Lavrov <romanl@google.com>
Kimmo Kinnunen b8d546b2 2024-10-30T15:40:58 Fix immutable string concats with ints `BuildConcatenatedImmutableString("a", 10, "b")` would construct "a\nb" because the implementation lacked int overloads and ints would be promoted to chars implicitly. Fix by implementing simpler way to calculate digits for numbers and then add useful overloads. Remove ImmutableStringBuilder::appendDecimal() since the code is already expecting << for all other types, and the bug was due to this expectation. Bug: angleproject:376417347 Change-Id: Iea5e14a6e2fede068b704754b9203db794dd5bf0 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5972641 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com>
Shahbaz Youssefi d4a9fa51 2024-10-31T09:37:28 Vulkan: Re-enable dynamic rendering on newer ARM drivers Bug: b/356051947 Change-Id: I03fbed2dbdb8f454ee1c429db4669d13e719b9bd Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5980610 Commit-Queue: Yuxin Hu <yuxinhu@google.com> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Yuxin Hu <yuxinhu@google.com>
Cody Northrop 7bb1e0f6 2024-10-29T13:18:49 restricted_trace_perf: Allow use of system ANGLE libs This change allows the perf script to use built in ANGLE libs, including from the system partition, instead of requiring a separate package. By default, the script will now use the ANGLE libs packaged in the test APK (com.android.angle.test). OpenGL/EGL calls will still go though the platform EGL loader, but you no longer need to install a separate ANGLE APK containing libs. To use the system libs, add the following flag: --angle-package system You can use it to point to *any* package that is queryable and contains ANGLE libraries. To get the previous behavior, you'd use: --angle-package org.chromium.angle Also two small unrelated fixes: * Fix typo when clearing shader cache * Broaden the 'MODEL' skip in thermal throttling Bug: b/376300037 Change-Id: I762cfd7516026e903a6a5de2949d50e96e766d48 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5976899 Reviewed-by: Roman Lavrov <romanl@google.com> Commit-Queue: Cody Northrop <cnorthrop@google.com>
Neil Zhang 236e0f48 2024-10-22T18:33:23 Add fix for create multi-window surfaces cause crash This patch fix crash when created multiple window surfaces, and not call corresponding eglDestroySurface, also added related test. Bug: angleproject:374797737 Change-Id: I58941660130e84e213cf9d78806027d56d6efc8c Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5981430 Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Roman Lavrov 42bfb554 2024-10-31T10:26:54 Add llvm-libc dependency, now required for libc++ Bug: chromium:370535840 Change-Id: I98c9dbf90d35a19a4f7fbef68efd2f9221e89b9f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5980611 Commit-Queue: Roman Lavrov <romanl@google.com> Reviewed-by: Cody Northrop <cnorthrop@google.com>
angle-autoroll 0d914d46 2024-10-31T10:02:00 Roll vulkan-deps from a5edfbb83552 to 977783acab11 (10 revisions) https://chromium.googlesource.com/vulkan-deps.git/+log/a5edfbb83552..977783acab11 Also rolling transitive DEPS: https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/63ebbe367c8c219e262a2bf78b7395b74cc9d9af..d81aeef9f82050bd5940ce1c475d8316ad41d747 https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Headers/+log/22c4d1b1e9d1c7d9aa5086c93e6491f21080019b..cb6b2c32dbfc3257c1e9142a116fe9ee3d9b80a2 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Headers/+log/ab1ea9059d75b42a5717c7ab55713bdf194ccf21..f2eb740f3f0340ad941e563484d5ba45a5ec6af0 https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-Loader/+log/7a83084ef84cc0d47c31d2872a4939c4c59186a1..9276d73f29651c92402f0f0389108d69bb51b9ca https://chromium.googlesource.com/external/github.com/KhronosGroup/Vulkan-ValidationLayers/+log/154691a92a8e9733ab20854d2405ae3a19c13f67..2e3b9e9d7ba562187bff4b92b6a9dae6b1824008 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vulkan-deps-angle-autoroll Please CC angle-team@google.com,romanl@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: romanl@google.com Change-Id: I78f31ee5fe63477957d5bcf6a97967acc466f0d2 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5982438 Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>
Charlie Lao 2df8d32b 2024-10-25T13:49:40 Vulkan: Tag DescriptorSets properly with every command buffer When descriptorSetCache is disabled, there is a bug that the current descriptorSet is not properly tagged with current ResourceUse. Right now when we get a descriptorSet (from cache or reused or allocated new), we retain to the current command buffer. But if we submit command buffer and get a new command buffer, and draw with the same program/buffer/textures, we will be reusing the current bound descriptorSets, but it is not retained with new command buffer. In theory, we have the same bug for pool eviction as well when cache is enabled. It's just very hard to hit due to pool eviction occur very rare. But with cache disabled, this is very easy to hit with multiple tests. In this CL, the retainResource call is moved from ProgramExecutableVk::getOrAllocateDescriptorSet() call to ProgramExecutableVk::bindDescriptorSets() call. Since bindDescriptorSets is always get called when we get a new descriptorSet, and is always get called when a new command buffer is allocated, this covers all usage case. And even better, with this change we are able to remove commandBufferHelper from arguments of quite a few functions. Bug: angleproject:372268711 Change-Id: I1f21a3e7e9ea34e2842e54025b5eb930dbf6c593 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/4743599 Commit-Queue: Charlie Lao <cclao@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Yuxin Hu <yuxinhu@google.com>
Amirali Abdolrashidi 913251aa 2024-09-25T17:42:59 Add clear tests related to layered image * Added tests to bind a layered framebuffer to a 3D image and clear it. Bug: angleproject:42266869 Change-Id: I488a26883d3e05daf0cc746ccf4b409ba470cb4a Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5889409 Reviewed-by: Charlie Lao <cclao@google.com> Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Shahbaz Youssefi 63f5a328 2024-10-30T14:40:40 Revert "Improve CanSupportAEP Error Reporting" This reverts commit fb655e43f4a86fbdf35bf0cb5d2978d728b949d9. Reason for revert: Causes excessive logging and leads to flakiness in Chromium Original change's description: > Improve CanSupportAEP Error Reporting > > This change improves the error reporting for the `CanSupportAEP` > function. Previously, if the function returned false, it was difficult > to determine which requirement was not met. This change adds logging > that will list all unmet requirements, making it easier to debug AEP > support issues. > > Test: the default presubmit > Bug: b/268108262 > Change-Id: I48eb6b26358ca4991d760523f44568d97e8c35a6 > Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5972497 > Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> > Commit-Queue: Solti Ho <solti@google.com> > Reviewed-by: Yuxin Hu <yuxinhu@google.com> Bug: b/268108262 Bug: chromium:376380430 Bug: chromium:40266306 Change-Id: Ic5b1bbce16fe5c98af10ca0554d1035206d5c592 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5979208 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Charlie Lao 1652f8ed 2024-10-17T13:35:39 Vulkan: end2end tests when descriptorSetCache is disabled Some end2end tests are testing specific descriptorSet cache behavior. When cache is disabled, these tests failed. In this CL these perfCounter based tests haven been modified to check total allocation to ensure the descriptorSets are properly reused instead of cache hit/miss. Bug: angleproject:372268711 Change-Id: I1d2f4cfcf622b05cdcb3317c8804416a80e72c48 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/3735732 Commit-Queue: Charlie Lao <cclao@google.com> Reviewed-by: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Shahbaz Youssefi 0a372f29 2024-10-28T22:59:53 Vulkan: Remove docs about OpenGL line rasterization emulation The code that emulated OpenGL line rasterization has been removed. Bug: angleproject:42265836 Change-Id: Ib1c1d8745b1790e0291a691a6a1c8e9438c34351 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5973239 Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com> Auto-Submit: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Amirali Abdolrashidi <abdolrashidi@google.com>
Solti fb655e43 2024-10-28T21:45:41 Improve CanSupportAEP Error Reporting This change improves the error reporting for the `CanSupportAEP` function. Previously, if the function returned false, it was difficult to determine which requirement was not met. This change adds logging that will list all unmet requirements, making it easier to debug AEP support issues. Test: the default presubmit Bug: b/268108262 Change-Id: I48eb6b26358ca4991d760523f44568d97e8c35a6 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5972497 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Solti Ho <solti@google.com> Reviewed-by: Yuxin Hu <yuxinhu@google.com>
Charlie Lao d0a0fd1a 2024-10-17T14:50:51 Vulkan: Skip pool eviction when cache is disabled When cache is disabled, every time a new descriptorSet is allocated and bind to program, the previous descriptorSet will be added to the tail of garbage list, and the new descriptorSet is retrieved from the head of the garbage list, if its GPU usage has been completed. This means there will never have a situation that significant amount of descriptorSets are needed, therefore no need for pool eviction. One possible situation is that at one point you need a lot of descriptorSets and then after a while you only need small amount of descriptorSets. In that case we could delete the excessive unused descriptorSets. But since they are all allocated from a pool, as long as there is still one descriptorSet in the pool is used, you still can't destroy the pool. This means eviction is really not much useful when cache is disabled. Bug: angleproject:372268711 Change-Id: Id77b181b64e122f576ee43d11c39dc75bd681dcf Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5941126 Reviewed-by: Yuxin Hu <yuxinhu@google.com> Commit-Queue: Charlie Lao <cclao@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Roman Lavrov 9c0fc663 2024-10-29T16:34:52 Set use_custom_libcxx=false in MSVC builds Temporary workaround, support is being removed upstream: https://crrev.com/c/5963336 Before that CL, there was a condition (is_win && is_clang) which disabled it for MSVC builds. Now the flag defaults to true and is deprecated. Bug: angleproject:376074941 Change-Id: I23b4a59157d3a333f78e5af2f44447cf270a9671 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5973072 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Roman Lavrov <romanl@google.com> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Charlie Lao af73a7eb 2024-10-18T17:19:29 Vulkan: Add WeakPtr to mirror std::weak_ptr This is a follow up from previous CL crrev.com/c/5938947. Because of pool eviction is based on reference count, there is need to have a weak pointer to the reference counted pool that does not add an extra reference count. This CL adds WeakPtr that works similarly to std::weak_ptr, and replaced direct RefCountedDescriptorPool pointer with WeakPtr wrapper. This is safer than RefCountedDescriptorPool in a way that it does not allow modification of underline reference count. Also the use of WeakPtr has been reduced to minimum in this CL. Bug: angleproject:372268711 Change-Id: Idd6fa77432a9351269c968c961785a7cf5fab50c Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5944061 Reviewed-by: Yuxin Hu <yuxinhu@google.com> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Charlie Lao <cclao@google.com>
angle-autoroll 629dc14f 2024-10-29T13:55:40 Manual roll VK-GL-CTS from cfd0b16e7b5e to f674555ab03e (12 revisions) Manual roll requested by romanl@google.com https://chromium.googlesource.com/external/github.com/KhronosGroup/VK-GL-CTS.git/+log/cfd0b16e7b5e..f674555ab03e 2024-10-28 lorenzo@khronosgroup.org Merge vk-gl-cts/opengl-es-cts-3.2.11 into vk-gl-cts/main 2024-10-26 piotr.byszewski@mobica.com Update json parsing headers for vksc 2024-10-25 kamil.goras@mobica.com Re-enable dEQP-GLES2.functional.polygon_offset.*render_with_units 2024-10-25 javed@igalia.com Add tests for device scope operations 2024-10-24 mateusz.bahyrycz@mobica.com Tests for sparse descriptor buffers 2024-10-24 piotr.byszewski@mobica.com Expand test suite for generic profile support 2024-10-24 ziga@lunarg.com Remove shader object variants of sample locations tests 2024-10-24 ziga@lunarg.com Add tests for VkAndroidHardwareBufferFormatProperties2ANDROID 2024-10-24 mateusz.bahyrycz@mobica.com Fix bug while selecting transfer queue for sparse resource tests 2024-10-24 kamil.goras@mobica.com Re-enable tests excluded due to Tracker/vk-gl-cts/issues/373 2024-10-24 gleese@broadcom.com Remove autogenerated struct indices 2024-10-24 lorenzo@khronosgroup.org Merge vk-gl-cts/vulkan-cts-1.3.10 into vk-gl-cts/main If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/vk-gl-cts-angle-autoroll Please CC angle-team@google.com,romanl@google.com on the revert to ensure that a human is aware of the problem. To file a bug in ANGLE: https://bugs.chromium.org/p/angleproject/issues/entry To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md Bug: None Tbr: romanl@google.com Change-Id: Idded64c9dc64fe9f1d4a040048ffb2199fbc8105 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5976422 Bot-Commit: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: angle-autoroll <angle-autoroll@skia-public.iam.gserviceaccount.com>