)]}'
{
  "commit": "9d8aa4ea855e0d64bb6926acb5618e6d1e2ed344",
  "tree": "a21357f21084169e0a322f8fa8ce8abe343d4f06",
  "parents": [
    "c033a93c0d961fc7ec5b0872649143e061d97dd4"
  ],
  "author": {
    "name": "Hugh Dickins",
    "email": "hughd@google.com",
    "time": "Tue Mar 22 16:33:06 2011 -0700"
  },
  "committer": {
    "name": "Linus Torvalds",
    "email": "torvalds@linux-foundation.org",
    "time": "Tue Mar 22 17:44:04 2011 -0700"
  },
  "message": "mm: remove worrying dead code from find_get_pages()\n\nThe radix_tree_deref_retry() case in find_get_pages() has a strange little\nexcrescence, not seen in the other gang lookups: it looks like the start\nof an abandoned attempt to guarantee forward progress in a case that\ncannot arise.\n\nret should always be 0 here: if it isn\u0027t, then going back to restart will\nleak references to pages already gotten.  There used to be a comment\nsaying nr_found is necessarily 1 here: that\u0027s not quite true, but the\nradix_tree_deref_retry() case is peculiar to the entry at index 0, when we\nrace with it being moved out of the radix_tree root or back.\n\nRemove the worrisome two lines, add a brief comment here and in\nfind_get_pages_contig() and find_get_pages_tag(), and a WARN_ON in\nfind_get_pages() should it ever be seen elsewhere than at 0.\n\nSigned-off-by: Hugh Dickins \u003chughd@google.com\u003e\nCc: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nAcked-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nCc: Wu Fengguang \u003cfengguang.wu@intel.com\u003e\nCc: Salman Qazi \u003csqazi@google.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "cb476e70cf197b061048aad164be64d926e7bff7",
      "old_mode": 33188,
      "old_path": "mm/filemap.c",
      "new_id": "a29318147365831ce12d8e86dd41e1644b2f8628",
      "new_mode": 33188,
      "new_path": "mm/filemap.c"
    }
  ]
}
