|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|