Revert "vulkan: Blacklist MSAA path renderer on Qualcomm drivers"
This reverts commit 03616574024a7f321e885e5ab6fc575746573109.
Reason for revert: Caused more bad golds than it fixed
Original change's description:
> vulkan: Blacklist MSAA path renderer on Qualcomm drivers
>
> Bug: skia:7758
> Change-Id: I96b5c259352949d67f5e0263a7164cdce54b3269
> Reviewed-on: https://skia-review.googlesource.com/117152
> Reviewed-by: Derek Sollenberger <djsollen@google.com>
> Reviewed-by: Greg Daniel <egdaniel@google.com>
> Commit-Queue: Chris Dalton <csmartdalton@google.com>
TBR=djsollen@google.com,egdaniel@google.com,csmartdalton@google.com
Change-Id: I80714a4cba7d495bc6ad89616b6c5f6eb8492bdb
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:7758
Reviewed-on: https://skia-review.googlesource.com/117340
Reviewed-by: Chris Dalton <csmartdalton@google.com>
Commit-Queue: Chris Dalton <csmartdalton@google.com>
diff --git a/src/gpu/vk/GrVkCaps.cpp b/src/gpu/vk/GrVkCaps.cpp
index aa93f2e..83546da 100644
--- a/src/gpu/vk/GrVkCaps.cpp
+++ b/src/gpu/vk/GrVkCaps.cpp
@@ -131,10 +131,6 @@
fNewCBOnPipelineChange = true;
}
- if (kIntel_VkVendor == properties.vendorID) {
- fCanUseWholeSizeOnFlushMappedMemory = false;
- }
-
////////////////////////////////////////////////////////////////////////////
// GrCaps workarounds
////////////////////////////////////////////////////////////////////////////
@@ -148,14 +144,8 @@
fMaxVertexAttributes = SkTMin(fMaxVertexAttributes, 32);
}
- if (kQualcomm_VkVendor == properties.vendorID) {
- // http://skbug.com/7758 -- the bots have a rendering issue related to the MSAA path
- // renderer. It doesn't repro locally, and there doesn't seem to be a correlation between it
- // and the driver version. Blacklisting across-the-board for now.
- //
- // properties.driverVersion=93622907, Android 8.1 -> no repro
- // properties.driverVersion=159578699, Android 8.0 -> repros
- fBlacklistMSAAPathRenderer = true;
+ if (kIntel_VkVendor == properties.vendorID) {
+ fCanUseWholeSizeOnFlushMappedMemory = false;
}
////////////////////////////////////////////////////////////////////////////