)]}'
{
  "commit": "26934d9503af96b2ae3579a0384afdca45fcf816",
  "tree": "1605b4c8e5c757b20292887efcd9595400136cb1",
  "parents": [
    "a98d774cd6ea792abc414bbbdfd940aaaa4ffa88"
  ],
  "author": {
    "name": "Yunchao He",
    "email": "yunchao.he@intel.com",
    "time": "Thu May 19 16:35:09 2022 +0000"
  },
  "committer": {
    "name": "Dawn LUCI CQ",
    "email": "dawn-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Thu May 19 16:35:09 2022 +0000"
  },
  "message": "D3D12: set correct data layout of dst texture for B2T copy\n\nWhen we initialize the dst texture before T2T copy, we clear a\nbuffer with zeros and do a B2T copy in order to initialize the\ntexture. The T2T copy may only copy a subrect of the texture\u0027s\nsubresource, so does the initialization via B2T copy. As a result,\nthe buffer can store compact zeros for that exact subrect and then\ndo an exact B2T copy to initialize that area.\n\nHowever, the current B2T copy uses incorrect data layout for the\nbuffer, making some part of the dst texture uninitialized, and\neven leading to D3D12 runtime validation errors sometimes if it\ngoes beyond the boundary of staging buffer pool (4MB).\n\nThis change fixes this bug via using the exact height of copy area\nto replace dataLayout.rowsPerImage (unnecessary padding rows included)\nfor the height of each depth image for 3D textures.\n\nBug: dawn:1288\nChange-Id: I303e0d363039a6a87e352a8445898031e673cf4e\nReviewed-on: https://dawn-review.googlesource.com/c/dawn/+/90780\nCommit-Queue: Yunchao He \u003cyunchao.he@intel.com\u003e\nReviewed-by: Corentin Wallez \u003ccwallez@chromium.org\u003e\nReviewed-by: Austin Eng \u003cenga@chromium.org\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "59e27a516a7ee8c82c20754d6c4b4bc200493a1e",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d12/TextureD3D12.cpp",
      "new_id": "c7258a5b1881fd7a95f2e36e5e15571481663dcd",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d12/TextureD3D12.cpp"
    }
  ]
}