commit cbc503c5bc7a7e22f187b394e900f4b8719644d5
author Lingfeng Yang <lfy@google.com> Fri Jan 10 09:16:21 2020 -0800
committer Lingfeng Yang <lfy@google.com> Fri Jan 10 09:23:37 2020 -0800
tree c670cfac5b56e0e9369d848fe0306cb3bb263ad9
parent 8d00806a4d62e812c4a97de116754e1f1311d30f

VirtioGpuPipeStream: no need to skip page at a time

that was a bug on the host side

bug: 146066070
Change-Id: Ib38b4dc566f642088650a5bf24acb3750827efd8

system/OpenglSystemCommon/VirtioGpuPipeStream.cpp

diff --git a/system/OpenglSystemCommon/VirtioGpuPipeStream.cpp b/system/OpenglSystemCommon/VirtioGpuPipeStream.cpp
index 3718571..d3618f3 100644
--- a/system/OpenglSystemCommon/VirtioGpuPipeStream.cpp
+++ b/system/OpenglSystemCommon/VirtioGpuPipeStream.cpp
@@ -370,15 +370,7 @@
         done += toXfer;
         readPtr += toXfer;
 		todo -= toXfer;
-
-        // Note; One does not simply increment |m_writtenPos| by exactly
-        // |toXfer| because the way the underlying memory works (esp going
-        // through TTM/GEM), we have no guarantee that things will stay
-        // coherent byte-by-byte as it gets to the host. To be safe, skip a
-        // page each time.
-        const size_t kWrittenPosIncrement = PAGE_SIZE;
-        m_writtenPos += ((toXfer + kWrittenPosIncrement - 1) /
-                kWrittenPosIncrement) * kWrittenPosIncrement;
+        m_writtenPos += toXfer;
     }
 
     return len;