)]}'
{
  "commit": "0565d317768cc66b13e37184f29d9f270c2886dc",
  "tree": "9618e46da0eac37472cde829d9574a98a1d28bbb",
  "parents": [
    "5b990546e33477c34ee6fbc20fad6584386b46c3"
  ],
  "author": {
    "name": "Tejun Heo",
    "email": "tj@kernel.org",
    "time": "Tue Jan 10 15:08:26 2012 -0800"
  },
  "committer": {
    "name": "Linus Torvalds",
    "email": "torvalds@linux-foundation.org",
    "time": "Tue Jan 10 16:30:45 2012 -0800"
  },
  "message": "mempool: drop unnecessary and incorrect BUG_ON() from mempool_destroy()\n\nmempool_destroy() is a thin wrapper around free_pool().  The only thing it\nadds is BUG_ON(pool-\u003ecurr_nr !\u003d pool-\u003emin_nr).  The intention seems to be\nto enforce that all allocated elements are freed; however, the BUG_ON()\ncan\u0027t achieve that (it doesn\u0027t know anything about objects above min_nr)\nand incorrect as mempool_resize() is allowed to leave the pool extended\nbut not filled.  Furthermore, panicking is way worse than any memory leak\nand there are better debug tools to track memory leaks.\n\nDrop the BUG_ON() from mempool_destory() and as that leaves the function\nidentical to free_pool(), replace it.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\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": "11f0d0a5e0f853687744a0430ae6b8259daab87c",
      "old_mode": 33188,
      "old_path": "mm/mempool.c",
      "new_id": "e3a802a0cea096811bf911ba271f2b302d9c110d",
      "new_mode": 33188,
      "new_path": "mm/mempool.c"
    }
  ]
}
