)]}'
{
  "log": [
    {
      "commit": "ecdb2e257abc33ae6798d3ccba87bdafa40ef6b6",
      "tree": "497db6a95a9f06270506f6a75354d5df855d5a66",
      "parents": [
        "f7b934c8127deebf4eb56fbe4a4ae0da16b6dbcd"
      ],
      "author": {
        "name": "Kiyoshi Ueda",
        "email": "k-ueda@ct.jp.nec.com",
        "time": "Sat Mar 06 02:29:52 2010 +0000"
      },
      "committer": {
        "name": "Alasdair G Kergon",
        "email": "agk@redhat.com",
        "time": "Sat Mar 06 02:29:52 2010 +0000"
      },
      "message": "dm table: remove dm_get from dm_table_get_md\n\nRemove the dm_get() in dm_table_get_md() because dm_table_get_md() could\nbe called from presuspend/postsuspend, which are called while\nmapped_device is in DMF_FREEING state, where dm_get() is not allowed.\n\nJustification for that is the lifetime of both objects: As far as the\ncurrent dm design/implementation, mapped_device is never freed while\ntargets are doing something, because dm core waits for targets to become\nquiet in dm_put() using presuspend/postsuspend.  So targets should be\nable to touch mapped_device without holding reference count of the\nmapped_device, and we should allow targets to touch mapped_device even\nif it is in DMF_FREEING state.\n\nBackgrounds:\nI\u0027m trying to remove the multipath internal queue, since dm core now has\na generic queue for request-based dm.  In the patch-set, the multipath\ntarget wants to request dm core to start/stop queue.  One of such\nstart/stop requests can happen during postsuspend() while the target\nwaits for pg-init to complete, because the target stops queue when\nstarting pg-init and tries to restart it when completing pg-init.  Since\nqueue belongs to mapped_device, it involves calling dm_table_get_md()\nand dm_put().  On the other hand, postsuspend() is called in dm_put()\nfor mapped_device which is in DMF_FREEING state, and that triggers\nBUG_ON(DMF_FREEING) in the 2nd dm_put().\n\nI had tried to solve this problem by changing only multipath not to\ntouch mapped_device which is in DMF_FREEING state, but I couldn\u0027t and I\ncame up with a question why we need dm_get() in dm_table_get_md().\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n"
    },
    {
      "commit": "6076905b5ef39e0ea58db32583c9e0036c05e47b",
      "tree": "918a8da9778d93e8ff6aff6497eb698d7a45edab",
      "parents": [
        "3067e02f8f3ae2f3f02ba76400d03b8bcb4942b0"
      ],
      "author": {
        "name": "Mikulas Patocka",
        "email": "mpatocka@redhat.com",
        "time": "Thu Dec 10 23:51:52 2009 +0000"
      },
      "committer": {
        "name": "Alasdair G Kergon",
        "email": "agk@redhat.com",
        "time": "Thu Dec 10 23:51:52 2009 +0000"
      },
      "message": "dm: avoid _hash_lock deadlock\n\nFix a reported deadlock if there are still unprocessed multipath events\non a device that is being removed.\n\n_hash_lock is held during dev_remove while trying to send the\noutstanding events.  Sending the events requests the _hash_lock\nagain in dm_copy_name_and_uuid.\n\nThis patch introduces a separate lock around regions that modify the\nlink to the hash table (dm_set_mdptr) or the name or uuid so that\ndm_copy_name_and_uuid no longer needs _hash_lock.\n\nAdditionally, dm_copy_name_and_uuid can only be called if md exists\nso we can drop the dm_get() and dm_put() which can lead to a BUG()\nwhile md is being freed.\n\nThe deadlock:\n #0 [ffff8106298dfb48] schedule at ffffffff80063035\n #1 [ffff8106298dfc20] __down_read at ffffffff8006475d\n #2 [ffff8106298dfc60] dm_copy_name_and_uuid at ffffffff8824f740\n #3 [ffff8106298dfc90] dm_send_uevents at ffffffff88252685\n #4 [ffff8106298dfcd0] event_callback at ffffffff8824c678\n #5 [ffff8106298dfd00] dm_table_event at ffffffff8824dd01\n #6 [ffff8106298dfd10] __hash_remove at ffffffff882507ad\n #7 [ffff8106298dfd30] dev_remove at ffffffff88250865\n #8 [ffff8106298dfd60] ctl_ioctl at ffffffff88250d80\n #9 [ffff8106298dfee0] do_ioctl at ffffffff800418c4\n#10 [ffff8106298dff00] vfs_ioctl at ffffffff8002fab9\n#11 [ffff8106298dff40] sys_ioctl at ffffffff8004bdaf\n#12 [ffff8106298dff80] tracesys at ffffffff8005d28d (via system_call)\n\nCc: stable@kernel.org\nReported-by: guy keren \u003cchoo@actcom.co.il\u003e\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n"
    },
    {
      "commit": "e46b272b6608783ed7aa7b0594871550ce20b849",
      "tree": "a65834f60bee7e77f9669a7ff7e95a1ec3cd74ee",
      "parents": [
        "9a7b2b0f36c40beeb252cb5ceff36bb295e88d97"
      ],
      "author": {
        "name": "Harvey Harrison",
        "email": "harvey.harrison@gmail.com",
        "time": "Mon Apr 28 02:15:50 2008 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@linux-foundation.org",
        "time": "Mon Apr 28 08:58:42 2008 -0700"
      },
      "message": "md: replace remaining __FUNCTION__ occurrences\n\n__FUNCTION__ is gcc-specific, use __func__\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n"
    },
    {
      "commit": "7a8c3d3b92883798e4ead21dd48c16db0ec0ff6f",
      "tree": "21a25dc6bd6afa11430e1ab8d997a4b1c0b960f0",
      "parents": [
        "51e5b2bd34ded40ef48cade8a6a8f1baa0b4275e"
      ],
      "author": {
        "name": "Mike Anderson",
        "email": "andmike@linux.vnet.ibm.com",
        "time": "Fri Oct 19 22:48:01 2007 +0100"
      },
      "committer": {
        "name": "Alasdair G Kergon",
        "email": "agk@redhat.com",
        "time": "Sat Oct 20 02:01:26 2007 +0100"
      },
      "message": "dm: uevent generate events\n\nThis patch adds support for the dm_path_event dm_send_event functions which\ncreate and send udev events.\n\nSigned-off-by: Mike Anderson \u003candmike@linux.vnet.ibm.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n"
    },
    {
      "commit": "51e5b2bd34ded40ef48cade8a6a8f1baa0b4275e",
      "tree": "7c1650dcd2f53bed2c0b8c9538e8da2d2d1286c2",
      "parents": [
        "96a1f7dba6e464155c0d1dc69c6c2efa96b644ac"
      ],
      "author": {
        "name": "Mike Anderson",
        "email": "andmike@linux.vnet.ibm.com",
        "time": "Fri Oct 19 22:48:00 2007 +0100"
      },
      "committer": {
        "name": "Alasdair G Kergon",
        "email": "agk@redhat.com",
        "time": "Sat Oct 20 02:01:24 2007 +0100"
      },
      "message": "dm: add uevent to core\n\nThis patch adds a uevent skeleton to device-mapper.\n\nSigned-off-by: Mike Anderson \u003candmike@linux.vnet.ibm.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n"
    }
  ]
}
