1. 13 Dec, 2019 5 commits
  2. 12 Dec, 2019 8 commits
  3. 11 Dec, 2019 10 commits
  4. 10 Dec, 2019 8 commits
  5. 09 Dec, 2019 8 commits
  6. 07 Dec, 2019 1 commit
    • Vulkan: fix BufferVk::map() synchronization · 27c89d2b
      Shahbaz Youssefi authored
      4398b2b6 made finishToSerial() in
      BufferVk::map() conditional to whether the serial is in use to work
      around a performance regression.  Notes:
      
      - Prior to 087f1384, finishToSerial
        already did that, but that check was inadvertently removed.
      - finishToSerial waits for the smallest serial that's bigger than or
        equal to the requested serial.
      - The flush() call in BufferVk::map() was conditional to whether the
        serial is in use, but it really meant to check whether the buffer has
        pending commands in the graph.
      
      The end result is that there was an unnecessary flush in BufferVk::map()
      if we had to wait for a previous serial to finish.
      
      This change makes the flush conditional to whether the buffer has
      pending commands in the graph, and the finishToSerial call to whether
      the serial is not yet finished.
      
      Bug: angleproject:3994
      Change-Id: Idca436ef2439bcc8c59396a07b2591c1dfadd669
      Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1932341Reviewed-by: 's avatarBrandon Schade <b.schade@samsung.com>
      Reviewed-by: 's avatarShahbaz Youssefi <syoussefi@chromium.org>
      Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>