commit 52b037630b3695962fbceb0911b1aa05d8a79724
author Jason Macnak <natsu@google.com> Wed Mar 29 23:16:08 2023 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Mar 29 23:16:08 2023 +0000
tree 0e70b25f7795c25a09be78fed69085aa445f70ef
parent 1ecad548f6df33d7147e0623c3a18f33189a07d8
parent 60c3a41ec7e37c3b6f16cede630f4dc707a98da1

Merge "Use "gfxstream" namespace"

system/vulkan_enc/ResourceTracker.cpp

diff --git a/system/vulkan_enc/ResourceTracker.cpp b/system/vulkan_enc/ResourceTracker.cpp
index 81f196f..d5e3645 100644
--- a/system/vulkan_enc/ResourceTracker.cpp
+++ b/system/vulkan_enc/ResourceTracker.cpp
@@ -3885,43 +3885,6 @@
         // no-op
     }
 
-uint32_t transformNonExternalResourceMemoryTypeBitsForGuest(
-        uint32_t hostBits) {
-        uint32_t res = 0;
-        for (uint32_t i = 0; i < VK_MAX_MEMORY_TYPES; ++i) {
-            if (hostBits & (1 << i)) {
-                res |= (1 << i);
-            }
-        }
-        return res;
-    }
-
-    uint32_t transformExternalResourceMemoryTypeBitsForGuest(
-        uint32_t normalBits) {
-        uint32_t res = 0;
-        for (uint32_t i = 0; i < VK_MAX_MEMORY_TYPES; ++i) {
-            bool shouldAcceptMemoryIndex = normalBits & (1 << i);
-            if (shouldAcceptMemoryIndex) {
-                res |= (1 << i);
-            }
-        }
-        return res;
-    }
-
-    void transformNonExternalResourceMemoryRequirementsForGuest(
-        VkMemoryRequirements* reqs) {
-        reqs->memoryTypeBits =
-            transformNonExternalResourceMemoryTypeBitsForGuest(
-                reqs->memoryTypeBits);
-    }
-
-    void transformExternalResourceMemoryRequirementsForGuest(
-        VkMemoryRequirements* reqs) {
-        reqs->memoryTypeBits =
-            transformExternalResourceMemoryTypeBitsForGuest(
-                reqs->memoryTypeBits);
-    }
-
     void transformExternalResourceMemoryDedicatedRequirementsForGuest(
         VkMemoryDedicatedRequirements* dedicatedReqs) {
         dedicatedReqs->prefersDedicatedAllocation = VK_TRUE;
@@ -3932,38 +3895,9 @@
         VkImage image,
         VkMemoryRequirements* reqs) {
 
-        auto it = info_VkImage.find(image);
-        if (it == info_VkImage.end()) return;
-
-        auto& info = it->second;
-
-        if (!info.external ||
-            !info.externalCreateInfo.handleTypes) {
-            transformNonExternalResourceMemoryRequirementsForGuest(reqs);
-        } else {
-            transformExternalResourceMemoryRequirementsForGuest(reqs);
-        }
         setMemoryRequirementsForSysmemBackedImage(image, reqs);
     }
 
-    void transformBufferMemoryRequirementsForGuestLocked(
-        VkBuffer buffer,
-        VkMemoryRequirements* reqs) {
-
-        auto it = info_VkBuffer.find(buffer);
-        if (it == info_VkBuffer.end()) return;
-
-        auto& info = it->second;
-
-        if (!info.external ||
-            !info.externalCreateInfo.handleTypes) {
-            transformNonExternalResourceMemoryRequirementsForGuest(reqs);
-            return;
-        }
-
-        transformExternalResourceMemoryRequirementsForGuest(reqs);
-    }
-
     void transformImageMemoryRequirements2ForGuest(
         VkImage image,
         VkMemoryRequirements2* reqs2) {
@@ -3977,14 +3911,10 @@
 
         if (!info.external ||
             !info.externalCreateInfo.handleTypes) {
-            transformNonExternalResourceMemoryRequirementsForGuest(
-                &reqs2->memoryRequirements);
             setMemoryRequirementsForSysmemBackedImage(image, &reqs2->memoryRequirements);
             return;
         }
 
-        transformExternalResourceMemoryRequirementsForGuest(&reqs2->memoryRequirements);
-
         setMemoryRequirementsForSysmemBackedImage(image, &reqs2->memoryRequirements);
 
         VkMemoryDedicatedRequirements* dedicatedReqs =
@@ -4009,13 +3939,9 @@
 
         if (!info.external ||
             !info.externalCreateInfo.handleTypes) {
-            transformNonExternalResourceMemoryRequirementsForGuest(
-                &reqs2->memoryRequirements);
             return;
         }
 
-        transformExternalResourceMemoryRequirementsForGuest(&reqs2->memoryRequirements);
-
         VkMemoryDedicatedRequirements* dedicatedReqs =
             vk_find_struct<VkMemoryDedicatedRequirements>(reqs2);
 
@@ -5269,6 +5195,7 @@
 
         if (supportsCreateResourcesWithRequirements()) {
             info.baseRequirementsKnown = true;
+            info.baseRequirements = memReqs;
         }
 
         if (extBufCiPtr) {
@@ -5282,11 +5209,6 @@
         }
 #endif
 
-        if (info.baseRequirementsKnown) {
-            transformBufferMemoryRequirementsForGuestLocked(*pBuffer, &memReqs);
-            info.baseRequirements = memReqs;
-        }
-
         return res;
     }
 
@@ -5320,8 +5242,6 @@
 
         lock.lock();
 
-        transformBufferMemoryRequirementsForGuestLocked(
-            buffer, pMemoryRequirements);
         info.baseRequirementsKnown = true;
         info.baseRequirements = *pMemoryRequirements;
     }