)]}' { "log": [ { "commit": "32aaeffbd4a7457bf2f7448b33b5946ff2a960eb", "tree": "faf7ad871d87176423ff9ed1d1ba4d9c688fc23f", "parents": [ "208bca0860406d16398145ddd950036a737c3c9d", "67b84999b1a8b1af5625b1eabe92146c5eb42932" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Nov 06 19:44:47 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Nov 06 19:44:47 2011 -0800" }, "message": "Merge branch \u0027modsplit-Oct31_2011\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/paulg/linux\n\n* \u0027modsplit-Oct31_2011\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/paulg/linux: (230 commits)\n Revert \"tracing: Include module.h in define_trace.h\"\n irq: don\u0027t put module.h into irq.h for tracking irqgen modules.\n bluetooth: macroize two small inlines to avoid module.h\n ip_vs.h: fix implicit use of module_get/module_put from module.h\n nf_conntrack.h: fix up fallout from implicit moduleparam.h presence\n include: replace linux/module.h with \"struct module\" wherever possible\n include: convert various register fcns to macros to avoid include chaining\n crypto.h: remove unused crypto_tfm_alg_modname() inline\n uwb.h: fix implicit use of asm/page.h for PAGE_SIZE\n pm_runtime.h: explicitly requires notifier.h\n linux/dmaengine.h: fix implicit use of bitmap.h and asm/page.h\n miscdevice.h: fix up implicit use of lists and types\n stop_machine.h: fix implicit use of smp.h for smp_processor_id\n of: fix implicit use of errno.h in include/linux/of.h\n of_platform.h: delete needless include \u003clinux/module.h\u003e\n acpi: remove module.h include from platform/aclinux.h\n miscdevice.h: delete unnecessary inclusion of module.h\n device_cgroup.h: delete needless include \u003clinux/module.h\u003e\n net: sch_generic remove redundant use of \u003clinux/module.h\u003e\n net: inet_timewait_sock doesnt need \u003clinux/module.h\u003e\n ...\n\nFix up trivial conflicts (other header files, and removal of the ab3550 mfd driver) in\n - drivers/media/dvb/frontends/dibx000_common.c\n - drivers/media/video/{mt9m111.c,ov6650.c}\n - drivers/mfd/ab3550-core.c\n - include/linux/dmaengine.h\n" }, { "commit": "b4fdcb02f1e39c27058a885905bd0277370ba441", "tree": "fd4cfd1994f21f44afe5e7904681fb5ac09f81b8", "parents": [ "044595d4e448305fbaec472eb7d22636d24e7d8c", "6dd9ad7df2019b1e33a372a501907db293ebcd0d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Nov 04 17:06:58 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Nov 04 17:06:58 2011 -0700" }, "message": "Merge branch \u0027for-3.2/core\u0027 of git://git.kernel.dk/linux-block\n\n* \u0027for-3.2/core\u0027 of git://git.kernel.dk/linux-block: (29 commits)\n block: don\u0027t call blk_drain_queue() if elevator is not up\n blk-throttle: use queue_is_locked() instead of lockdep_is_held()\n blk-throttle: Take blkcg-\u003elock while traversing blkcg-\u003epolicy_list\n blk-throttle: Free up policy node associated with deleted rule\n block: warn if tag is greater than real_max_depth.\n block: make gendisk hold a reference to its queue\n blk-flush: move the queue kick into\n blk-flush: fix invalid BUG_ON in blk_insert_flush\n block: Remove the control of complete cpu from bio.\n block: fix a typo in the blk-cgroup.h file\n block: initialize the bounce pool if high memory may be added later\n block: fix request_queue lifetime handling by making blk_queue_cleanup() properly shutdown\n block: drop @tsk from attempt_plug_merge() and explain sync rules\n block: make get_request[_wait]() fail if queue is dead\n block: reorganize throtl_get_tg() and blk_throtl_bio()\n block: reorganize queue draining\n block: drop unnecessary blk_get/put_queue() in scsi_cmd_ioctl() and blk_get_tg()\n block: pass around REQ_* flags instead of broken down booleans during request alloc/free\n block: move blk_throtl prototypes to block/blk.h\n block: fix genhd refcounting in blkio_policy_parse_and_set()\n ...\n\nFix up trivial conflicts due to \"mddev_t\" -\u003e \"struct mddev\" conversion\nand making the request functions be of type \"void\" instead of \"int\" in\n - drivers/md/{faulty.c,linear.c,md.c,md.h,multipath.c,raid0.c,raid1.c,raid10.c,raid5.c}\n - drivers/staging/zram/zram_drv.c\n" }, { "commit": "056075c76417b112b4924e7b6386fdc6dfc9ac03", "tree": "7326cbdb0f54f975b140e548bd4f4d6f4b3486cc", "parents": [ "ec37d321b96621906337c4279c490e1b5893ecae" ], "author": { "name": "Paul Gortmaker", "email": "paul.gortmaker@windriver.com", "time": "Sun Jul 03 13:58:33 2011 -0400" }, "committer": { "name": "Paul Gortmaker", "email": "paul.gortmaker@windriver.com", "time": "Mon Oct 31 19:31:18 2011 -0400" }, "message": "md: Add module.h to all files using it implicitly\n\nA pending cleanup will mean that module.h won\u0027t be implicitly\neverywhere anymore. Make sure the modular drivers in md dir\nare actually calling out for \u003cmodule.h\u003e explicitly in advance.\n\nSigned-off-by: Paul Gortmaker \u003cpaul.gortmaker@windriver.com\u003e\n" }, { "commit": "5c04b426f2e8b46cfc7969a35b2631063a3c646c", "tree": "2d27d9f5d2fe5d5e8fbc01a467ec58bcb50235c1", "parents": [ "499337bb6511e665a236a6a947f819d98ea340c6", "899e3ee404961a90b828ad527573aaaac39f0ab1" ], "author": { "name": "Jens Axboe", "email": "axboe@kernel.dk", "time": "Wed Oct 19 14:30:42 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "axboe@kernel.dk", "time": "Wed Oct 19 14:30:42 2011 +0200" }, "message": "Merge branch \u0027v3.1-rc10\u0027 into for-3.2/core\n\nConflicts:\n\tblock/blk-core.c\n\tinclude/linux/blkdev.h\n\nSigned-off-by: Jens Axboe \u003caxboe@kernel.dk\u003e\n" }, { "commit": "751e67ca2e2790e9b9b187557d5ba6f96b6c4a7e", "tree": "029525eb09d78e63b97d151a0d2e1871d6766885", "parents": [ "d70ed2e4fafdbef0800e73942482bb075c21578b" ], "author": { "name": "Chris Dunlop", "email": "chris@onthe.net.au", "time": "Wed Oct 19 16:48:26 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Oct 19 17:15:15 2011 +1100" }, "message": "md.c: trivial comment fix\n\nTrivial comment fix\n\nSigned-off-by: Chris Dunlop \u003cchris@onthe.net.au\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d70ed2e4fafdbef0800e73942482bb075c21578b", "tree": "a0fb7f9b6993b44e37dc2f724df251bd6fcffae3", "parents": [ "d30519fc59c5cc2f7772fa67b16b1a2426d36c95" ], "author": { "name": "Andrei Warkentin", "email": "andreiw@vmware.com", "time": "Tue Oct 18 12:16:48 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 18 12:16:48 2011 +1100" }, "message": "MD: Allow restarting an interrupted incremental recovery.\n\nIf an incremental recovery was interrupted, a subsequent\nre-add will result in a full recovery, even though an\nincremental should be possible (seen with raid1).\n\nSolve this problem by not updating the superblock on the\nrecovering device until array is not degraded any longer.\n\nCc: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrei Warkentin \u003candreiw@vmware.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d30519fc59c5cc2f7772fa67b16b1a2426d36c95", "tree": "3aae0a21c16816e1989ea90db23a5b123a5a6a12", "parents": [ "34db0cd60f8a1f4ab73d118a8be3797c20388223" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 18 12:13:47 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 18 12:13:47 2011 +1100" }, "message": "md: clear In_sync bit on devices added to an active array.\n\nWhen we add a device to an active array it can be meaningful to set\nthe \u0027insync\u0027 flag. This indicates that the device is in-sync with the\narray except for locations recorded in the bitmap.\nA bitmap-based recovery can then bring it completely in-sync.\n\nInternally we move that flag to \u0027saved_raid_disk\u0027 but forgot to clear\nIn_sync like we do in add_new_disk.\n\nSo clear In_sync after moving its value to saved_raid_disk.\n\nReported-by: Andrei Warkentin \u003candreiw@vmware.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "84fc4b56db85cb9e05326424049973a2036c9940", "tree": "b8f942766ad620d63da92c24df56032168879253", "parents": [ "29d3247ea2274bca0d0b7f41fcf3b82e0a8bc44c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:49:58 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:49:58 2011 +1100" }, "message": "md: rename \"mdk_personality\" to \"md_personality\"\n\n\"mdk\" doesn\u0027t mean anything any more.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b8bf3451d1e3133ebc3998721d14013a6c27114", "tree": "6ad9afb9124d5eb865d513ec3f30259231075a8f", "parents": [ "fd01b88c75a718020ff77e7f560d33835e9b58de" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:48:23 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:48:23 2011 +1100" }, "message": "md: remove typedefs: mdk_thread_t -\u003e struct md_thread\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "fd01b88c75a718020ff77e7f560d33835e9b58de", "tree": "c455d5adefd58f3263dcf265bb8ba2024523b106", "parents": [ "3cb03002000f133f9f97269edefd73611eafc873" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:47:53 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:47:53 2011 +1100" }, "message": "md: remove typedefs: mddev_t -\u003e struct mddev\n\nHaving mddev_t and \u0027struct mddev_s\u0027 is ugly and not preferred\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3cb03002000f133f9f97269edefd73611eafc873", "tree": "77fdb146666298b33aaeef0c05f082b6f82840fe", "parents": [ "50de8df4abca1b27dbf7b2f81a56451bd8b5a7d8" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:45:26 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 11 16:45:26 2011 +1100" }, "message": "md: removing typedefs: mdk_rdev_t -\u003e struct md_rdev\n\nThe typedefs are just annoying. \u0027mdk\u0027 probably refers to \u0027md_k.h\u0027\nwhich used to be an include file that defined this thing.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "36a4e1fe0f454146724c174bf7c1e8e76297a212", "tree": "57f0de83fda98e5ae1b4f66d5e993ca27386a793", "parents": [ "bdc04e6b15f70a8f96d8cdfe21df159a6466b49a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 07 14:23:17 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 07 14:23:17 2011 +1100" }, "message": "md: remove PRINTK and dprintk debugging and use pr_debug\n\nBeing able to dynamically enable these make them much more useful.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2dba6a911c284603d2085fec1e2d3c142f58a010", "tree": "8db93ad2131afd519b9d618f12841ec5d981c082", "parents": [ "7e841526263b3e0042a423513147dfd06c8e998d" ], "author": { "name": "Daniel P. Berrange", "email": "berrange@redhat.com", "time": "Fri Sep 23 10:40:45 2011 +0100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Sep 23 19:54:04 2011 +1000" }, "message": "md: don\u0027t delay reboot by 1 second if no MD devices exist\n\nThe md_notify_reboot() method includes a call to mdelay(1000),\nto deal with \"exotic SCSI devices\" which are too volatile on\nreboot. The delay is unconditional. Even if the machine does\nnot have any block devices, let alone MD devices, the kernel\nshutdown sequence is slowed down.\n\n1 second does not matter much with physical hardware, but with\ncertain virtualization use cases any wasted time in the bootup\n\u0026 shutdown sequence counts for alot.\n\n* drivers/md/md.c: md_notify_reboot() - only impose a delay if\n there was at least one MD device to be stopped during reboot\n\nSigned-off-by: Daniel P. Berrange \u003cberrange@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "01f96c0a9922cd9919baf9d16febdf7016177a12", "tree": "a877fe509c4ef0db5252b7192df56009c1d06d6f", "parents": [ "27a7b260f71439c40546b43588448faac01adb93" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 21 15:30:20 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 21 15:30:20 2011 +1000" }, "message": "md: Avoid waking up a thread after it has been freed.\n\nTwo related problems:\n\n1/ some error paths call \"md_unregister_thread(mddev-\u003ethread)\"\n without subsequently clearing -\u003ethread. A subsequent call\n to mddev_unlock will try to wake the thread, and crash.\n\n2/ Most calls to md_wakeup_thread are protected against the thread\n disappeared either by:\n - holding the -\u003emutex\n - having an active request, so something else must be keeping\n the array active.\n However mddev_unlock calls md_wakeup_thread after dropping the\n mutex and without any certainty of an active request, so the\n -\u003ethread could theoretically disappear.\n So we need a spinlock to provide some protections.\n\nSo change md_unregister_thread to take a pointer to the thread\npointer, and ensure that it always does the required locking, and\nclears the pointer properly.\n\nReported-by: \"Moshe Melnikov\" \u003cmoshe@zadarastorage.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\ncc: stable@kernel.org\n" }, { "commit": "5a7bbad27a410350e64a2d7f5ec18fc73836c14f", "tree": "3447cd62dbcbd77b4071e2eb7576f1d7632ef2d3", "parents": [ "c20e8de27fef9f59869c81c288ad6cf28200e00c" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Mon Sep 12 12:12:01 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Sep 12 12:12:01 2011 +0200" }, "message": "block: remove support for bio remapping from -\u003emake_request\n\nThere is very little benefit in allowing to let a -\u003emake_request\ninstance update the bios device and sector and loop around it in\n__generic_make_request when we can archive the same through calling\ngeneric_make_request from the driver and letting the loop in\ngeneric_make_request handle it.\n\nNote that various drivers got the return value from -\u003emake_request and\nreturned non-zero values for errors.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: NeilBrown \u003cneilb@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "27a7b260f71439c40546b43588448faac01adb93", "tree": "80fdd39a446f93fa84a355cf4d0118244b4d7fd6", "parents": [ "079fa166a2874985ae58b2e21e26e1cbc91127d4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sat Sep 10 17:21:28 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sat Sep 10 17:21:28 2011 +1000" }, "message": "md: Fix handling for devices from 2TB to 4TB in 0.90 metadata.\n\n0.90 metadata uses an unsigned 32bit number to count the number of\nkilobytes used from each device.\nThis should allow up to 4TB per device.\nHowever we multiply this by 2 (to get sectors) before casting to a\nlarger type, so sizes above 2TB get truncated.\n\nAlso we allow rdev-\u003esectors to be larger than 4TB, so it is possible\nfor the array to be resized larger than the metadata can handle.\nSo make sure rdev-\u003esectors never exceeds 4TB when 0.90 metadata is in\nused.\n\nAlso the sanity check at the end of super_90_load should include level\n1 as it used -\u003esize too. (RAID0 and Linear don\u0027t use -\u003esize at all).\n\nReported-by: Pim Zandbergen \u003cP.Zandbergen@macroscoop.nl\u003e\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7da64a0abc3b2c6cbd3521672e9bb74dd560bb89", "tree": "1b17033fe40a2b29c7e4d6a2f200f54f39514a4d", "parents": [ "1b6afa17581027218088a18a9ceda600e0ddba7a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 30 16:20:17 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 30 16:20:17 2011 +1000" }, "message": "md: fix clearing of \u0027blocked\u0027 flag in the presence of bad blocks.\n\nWhen the \u0027blocked\u0027 flag on a device is cleared while there are\nunacknowledged bad blocks we must fail the device. This is needed for\nbackwards compatability of the interface.\n\nThe code currently uses the wrong test for \"unacknowledged bad blocks\nexist\". Change it to the right test.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a5bf4df0c88b88d34b6f0e3bc8a402dac7d14611", "tree": "40296244aa28ddff0360a1a57878a99926243620", "parents": [ "aeb9b211849621f592288ed5ad694de9eeaae87a" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Thu Aug 25 14:43:34 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 25 14:43:34 2011 +1000" }, "message": "md: use REQ_NOIDLE flag in md_super_write()\n\nQueue idling is used for the anticipation of immediate\nsequencial I/O\u0027s but md_super_write() is a kind of one-\nshot operation, coupled with md_super_wait(), so the\nidling in this case will be just a waste of time.\n\nSpecifying REQ_NOIDLE prevents it. Instead of adding\nthe flag to submit_bio() directly, use pre-defined\nmacro WRITE_FLUSH_FUA.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "aeb9b211849621f592288ed5ad694de9eeaae87a", "tree": "affa418deafe2f218dc84e4aeb6ed4bbaf85d3e5", "parents": [ "5ef56c8fecedf403a346d02140e52a072d693d6b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 25 14:43:08 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 25 14:43:08 2011 +1000" }, "message": "md: ensure changes to \u0027write-mostly\u0027 are reflected in metadata.\n\nThe \u0027write-mostly\u0027 flag can be changed through sysfs.\nWith 0.90 metadata, those changes are reflected in the metadata.\nFor 1.x metadata, they aren\u0027t.\n\nSo fix super_1_sync to record \u0027write-mostly\u0027 status.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5ef56c8fecedf403a346d02140e52a072d693d6b", "tree": "1f121fe5e4db61364c352e7ffe2293a1cc20e9a9", "parents": [ "14c62e78dc1379185515be41903c4a667efc6d54" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 25 14:42:51 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 25 14:42:51 2011 +1000" }, "message": "md: report failure if a \u0027set faulty\u0027 request doesn\u0027t.\n\nSometimes a device will refuse to be set faulty. e.g. RAID1 will\nnever let the last working device become faulty.\n\nSo check if \"md_error()\" did manage to set the faulty flag and fail\nwith EBUSY if it didn\u0027t.\n\nResolves-Debian-Bug: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug\u003d601198\nReported-by: Mike Hommey \u003cmh+reportbug@glandium.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "6140333d3656f62ac7e6a5af87e7fe92cfb8d655", "tree": "d96f7ad2196b4383f5ca4396c956e24c82b2952c", "parents": [ "6f56c218666b5c7eff354364357307d18c10058b", "58c54fcca3bac5bf9290cfed31c76e4c4bfbabaf" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jul 28 05:50:27 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jul 28 05:50:27 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md: (75 commits)\n md/raid10: handle further errors during fix_read_error better.\n md/raid10: Handle read errors during recovery better.\n md/raid10: simplify read error handling during recovery.\n md/raid10: record bad blocks due to write errors during resync/recovery.\n md/raid10: attempt to fix read errors during resync/check\n md/raid10: Handle write errors by updating badblock log.\n md/raid10: clear bad-block record when write succeeds.\n md/raid10: avoid writing to known bad blocks on known bad drives.\n md/raid10 record bad blocks as needed during recovery.\n md/raid10: avoid reading known bad blocks during resync/recovery.\n md/raid10 - avoid reading from known bad blocks - part 3\n md/raid10: avoid reading from known bad blocks - part 2\n md/raid10: avoid reading from known bad blocks - part 1\n md/raid10: Split handle_read_error out from raid10d.\n md/raid10: simplify/reindent some loops.\n md/raid5: Clear bad blocks on successful write.\n md/raid5. Don\u0027t write to known bad block on doubtful devices.\n md/raid5: write errors should be recorded as bad blocks if possible.\n md/raid5: use bad-block log to improve handling of uncorrectable read errors.\n md/raid5: avoid reading from known bad blocks.\n ...\n" }, { "commit": "e875ecea266a543e643b19e44cf472f1412708f9", "tree": "b602d08f7aa4a743d3c27ad55e347d36991f0814", "parents": [ "40c356ce5ad1a6be817825e1da1bc7494349cc6d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:39:24 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:39:24 2011 +1000" }, "message": "md/raid10 record bad blocks as needed during recovery.\n\nWhen recovering one or more devices, if all the good devices have\nbad blocks we should record a bad block on the device being rebuilt.\n\nIf this fails, we need to abort the recovery.\n\nTo ensure we don\u0027t think that we aborted later than we actually did,\nwe need to move the check for MD_RECOVERY_INTR earlier in md_do_sync,\nin particular before mddev-\u003ecurr_resync is updated.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "de393cdea66cbd63c90725663f400c76faf1b255", "tree": "6a2bf37bee98bf7de42856f904bd23c81e082f8e", "parents": [ "d7a9d443bc8a75a24873c0506f50051edfedc714" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "message": "md: make it easier to wait for bad blocks to be acknowledged.\n\nIt is only safe to choose not to write to a bad block if that bad\nblock is safely recorded in metadata - i.e. if it has been\n\u0027acknowledged\u0027.\n\nIf it hasn\u0027t we need to wait for the acknowledgement.\n\nWe support that using rdev-\u003eblocked wait and\nmd_wait_for_blocked_rdev by introducing a new device flag\n\u0027BlockedBadBlock\u0027.\n\nThis flag is only advisory.\nIt is cleared whenever we acknowledge a bad block, so that a waiter\ncan re-check the particular bad blocks that it is interested it.\n\nIt should be set by a caller when they find they need to wait.\nThis (set after test) is inherently racy, but as\nmd_wait_for_blocked_rdev already has a timeout, losing the race will\nhave minimal impact.\n\nWhen we clear \"Blocked\" was also clear \"BlockedBadBlocks\" incase it\nwas set incorrectly (see above race).\n\nWe also modify the way we manage \u0027Blocked\u0027 to fit better with the new\nhandling of \u0027BlockedBadBlocks\u0027 and to make it consistent between\nexternally managed and internally managed metadata. This requires\nthat each raidXd loop checks if the metadata needs to be written and\ntriggers a write (md_check_recovery) if needed. Otherwise a queued\nwrite request might cause raidXd to wait for the metadata to write,\nand only that thread can write it.\n\nBefore writing metadata, we set FaultRecorded for all devices that\nare Faulty, then after writing the metadata we clear Blocked for any\ndevice for which the Fault was certainly Recorded.\n\nThe \u0027faulty\u0027 device flag now appears in sysfs if the device is faulty\n*or* it has unacknowledged bad blocks. So user-space which does not\nunderstand bad blocks can continue to function correctly.\nUser space which does, should not assume a device is faulty until it\nsees the \u0027faulty\u0027 flag, and then sees the list of unacknowledged bad\nblocks is empty.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d7a9d443bc8a75a24873c0506f50051edfedc714", "tree": "6255771f6cca87698dcc8eb11d62e7fc217b8d0b", "parents": [ "06f603851fa90bcd236328438278d4dc8b655495" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "message": "md: add \u0027write_error\u0027 flag to component devices.\n\nIf a device has ever seen a write error, we will want to handle\nknown-bad-blocks differently.\nSo create an appropriate state flag and export it via sysfs.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReviewed-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\n" }, { "commit": "d2eb35acfdccbe2a3622ed6cc441a5482148423b", "tree": "77600cab29fc9e1fd39d612773086a456fc32d88", "parents": [ "9f2f3830789a4c9c1af2d1437d407c43e05136e6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:48 2011 +1000" }, "message": "md/raid1: avoid reading from known bad blocks.\n\nNow that we have a bad block list, we should not read from those\nblocks.\nThere are several main parts to this:\n 1/ read_balance needs to check for bad blocks, and return not only\n the chosen device, but also how many good blocks are available\n there.\n 2/ fix_read_error needs to avoid trying to read from bad blocks.\n 3/ read submission must be ready to issue multiple reads to\n different devices as different bad blocks on different devices\n could mean that a single large read cannot be served by any one\n device, but can still be served by the array.\n This requires keeping count of the number of outstanding requests\n per bio. This count is stored in \u0027bi_phys_segments\u0027\n 4/ retrying a read needs to also be ready to submit a smaller read\n and queue another request for the rest.\n\nThis does not yet handle bad blocks when reading to perform resync,\nrecovery, or check.\n\n\u0027md_trim_bio\u0027 will also be used for RAID10, so put it in md.c and\nexport it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "9f2f3830789a4c9c1af2d1437d407c43e05136e6", "tree": "8e20afbfca82acd7346fb93d35170f44ee01a807", "parents": [ "2699b67223aca6b1450fc2f72e40fada952afc85" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "message": "md: Disable bad blocks and v0.90 metadata.\n\nv0.90 metadata cannot record bad blocks, so when loading metadata\nfor such a device, set shift to -1.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2699b67223aca6b1450fc2f72e40fada952afc85", "tree": "c84d0cbb763ad03cd770a218a37f0f1ec31118af", "parents": [ "34b343cff4354ab9864be83be88405fd53d928a0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "message": "md: load/store badblock list from v1.x metadata\n\nSpace must have been allocated when array was created.\nA feature flag is set when the badblock list is non-empty, to\nensure old kernels don\u0027t load and trust the whole device.\n\nWe only update the on-disk badblocklist when it has changed.\nIf the badblocklist (or other metadata) is stored on a bad block, we\ndon\u0027t cope very well.\n\nIf metadata has no room for bad block, flag bad-blocks as disabled,\nand do the same for 0.90 metadata.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "16c791a5af3e50d0c11760485fd68e5829f3be4d", "tree": "24182e37b278a55a9b05b19154ff52597d220714", "parents": [ "2230dfe4ccc3add340dc6d437965b2de1d269fde" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:47 2011 +1000" }, "message": "md/bad-block-log: add sysfs interface for accessing bad-block-log.\n\nThis can show the log (providing it fits in one page) and\nallows bad blocks to be \u0027acknowledged\u0027 meaning that they\nhave safely been recorded in metadata.\n\nClearing bad blocks is not allowed via sysfs (except for\ncode testing). A bad block can only be cleared when\na write to the block succeeds.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReviewed-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\n" }, { "commit": "2230dfe4ccc3add340dc6d437965b2de1d269fde", "tree": "fc45b727ad2e1a148e7d20f327b45a3afc474e9d", "parents": [ "a519b26dbe6533416d21b552053b0bf687f878d7" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:46 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 11:31:46 2011 +1000" }, "message": "md: beginnings of bad block management.\n\nThis the first step in allowing md to track bad-blocks per-device so\nthat we can fail individual blocks rather than the whole device.\n\nThis patch just adds a data structure for recording bad blocks, with\nroutines to add, remove, search the list.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReviewed-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\n" }, { "commit": "a519b26dbe6533416d21b552053b0bf687f878d7", "tree": "bef24eab8604ef2275eb8f1b591ea52bd4b89194", "parents": [ "768e587e18c1c4ce0403b9532115752189422408" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 07:56:24 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jul 28 07:56:24 2011 +1000" }, "message": "md: remove suspicious size_of()\n\nWhen calling bioset_create we pass the size of the front_pad as\n sizeof(mddev)\nwhich looks suspicious as mddev is a pointer and so it looks like a\ncommon mistake where\n sizeof(*mddev)\nwas intended.\nThe size is actually correct as we want to store a pointer in the\nfront padding of the bios created by the bioset, so make the intent\nmore explicit by using\n sizeof(mddev_t *)\n\nReported-by: Zdenek Kabelac \u003czdenek.kabelac@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "768e587e18c1c4ce0403b9532115752189422408", "tree": "1b0d65fc839aa562c409a46984e1e15b1ae629d6", "parents": [ "3520fa4db7fc4ae1b0373dcecdaf720f620dab2d" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Wed Jul 27 11:00:37 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:37 2011 +1000" }, "message": "MD: generate an event when array sync is complete\n\nThis patch causes MD to generate an event (for device-mapper) when the\nsynchronization thread is reaped. This is expected behavior for device-mapper.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "65a06f06744d3deae61f3596566952a8da92199f", "tree": "5befc4ef2e1965c55a112d436bd980ff6c0bef55", "parents": [ "700c72138938cf428c74379806886c6b017d6295" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "message": "md: get rid of unnecessary casts on page_address()\n\npage_address() returns void pointer, so the casts can be removed.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5389042ffa36976caa45a79af16081d759001fa7", "tree": "eb303de141a50f41ccac438e734b65dbfe25825e", "parents": [ "a478a069b6adf186373a6648c54fc5f1d3c2f8eb" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "message": "md: change managed of recovery_disabled.\n\nIf we hit a read error while recovering a mirror, we want to abort the\nrecovery without necessarily failing the disk - as having a disk this\na read error is better than not having an array at all.\n\nCurrently this is managed with a per-array flag \"recovery_disabled\"\nand is only implemented for RAID1. For RAID10 we will need finer\ngrained control as we might want to disable recovery for individual\ndevices separately.\n\nSo push more of the decision making into the personality.\n\u0027recovery_disabled\u0027 is now a \u0027cookie\u0027 which is copied when the\npersonality want to disable recovery and is changed when a device is\nadded to the array as this is used as a trigger to \u0027try recovery\nagain\u0027.\n\nThis will allow RAID10 to get the control that it needs.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a478a069b6adf186373a6648c54fc5f1d3c2f8eb", "tree": "ac10604e2b0e3525c25c6fb23128ab1fc858a550", "parents": [ "36fad858a7404a9656122a9e560a224ae2a00979" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "message": "md: remove ro check in md_check_recovery()\n\nCommit c89a8eee6154 (\"Allow faulty devices to be removed from a\nreadonly array.\") added some work on ro array in the function,\nbut it couldn\u0027t be done since we didn\u0027t allow the ro array to be\nhandled from the beginning. Fix it.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "36fad858a7404a9656122a9e560a224ae2a00979", "tree": "95f8a7a6b8dad6bb8d7c7735f7845888b5e702db", "parents": [ "8bda470e8ebde35f9349e98ecbce4dfb508a60fa" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 27 11:00:36 2011 +1000" }, "message": "md: introduce link/unlink_rdev() helpers\n\nThere are places where sysfs links to rdev are handled\nin a same way. Add the helper functions to consolidate\nthem.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f15146380d28b746df3c8b81b392812eb982382a", "tree": "bf43b38b60c21bd01b007c9636062783d406eb29", "parents": [ "72c5052ddc3956d847f21c2b8d55c93664a51b2c" ], "author": { "name": "Kay Sievers", "email": "kay.sievers@vrfy.org", "time": "Tue Jul 12 20:48:39 2011 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Jul 20 20:47:50 2011 -0400" }, "message": "fs: seq_file - add event counter to simplify poll() support\n\nMoving the event counter into the dynamically allocated \u0027struc seq_file\u0027\nallows poll() support without the need to allocate its own tracking\nstructure.\n\nAll current users are switched over to use the new counter.\n\nRequested-by: Andrew Morton akpm@linux-foundation.org\nAcked-by: NeilBrown \u003cneilb@suse.de\u003e\nTested-by: Lucas De Marchi lucas.demarchi@profusion.mobi\nSigned-off-by: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "4274215d24633df7302069e51426659d4759c5ed", "tree": "c21fff5f11201eaaea0e44cf81a38df21dd63ffd", "parents": [ "2992c4bd5742b31a0ee00a76eee9c1c284507418" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 28 16:59:42 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 28 16:59:42 2011 +1000" }, "message": "md: avoid endless recovery loop when waiting for fail device to complete.\n\nIf a device fails in a way that causes pending request to take a while\nto complete, md will not be able to immediately remove it from the\narray in remove_and_add_spares.\nIt will then incorrectly look like a spare device and md will try to\nrecover it even though it is failed.\nThis leads to a recovery process starting and instantly aborting over\nand over again.\n\nWe should check if the device is faulty before considering it to be a\nspare. This will avoid trying to start a recovery that cannot\nproceed.\n\nThis bug was introduced in 2.6.26 so that patch is suitable for any\nkernel since then.\n\nCc: stable@kernel.org\nReported-by: Jim Paradis \u003cjames.paradis@stratus.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "01393f3d5836b7d62e925e6f4658a7eb22b83a11", "tree": "38c6b3856adc22ef5bcada7855a6601d6e2e9654", "parents": [ "9864c0053d3da4c5731ac8a6c4835179310bd40a" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Thu Jun 09 11:42:54 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 09 11:42:54 2011 +1000" }, "message": "md: check -\u003ehot_remove_disk when removing disk\n\nCheck pers-\u003ehot_remove_disk instead of pers-\u003ehot_add_disk in slot_store()\nduring disk removal. The linear personality only has -\u003ehot_add_disk and\nno -\u003ehot_remove_disk, so that removing disk in the array resulted to\nfollowing kernel bug:\n\n$ sudo mdadm --create /dev/md0 --level\u003dlinear --raid-devices\u003d4 /dev/loop[0-3]\n$ echo none | sudo tee /sys/block/md0/md/dev-loop2/slot\n BUG: unable to handle kernel NULL pointer dereference at (null)\n IP: [\u003c (null)\u003e] (null)\n PGD c9f5d067 PUD 8575a067 PMD 0\n Oops: 0010 [#1] SMP\n CPU 2\n Modules linked in: linear loop bridge stp llc kvm_intel kvm asus_atk0110 sr_mod cdrom sg\n\n Pid: 10450, comm: tee Not tainted 3.0.0-rc1-leonard+ #173 System manufacturer System Product Name/P5G41TD-M PRO\n RIP: 0010:[\u003c0000000000000000\u003e] [\u003c (null)\u003e] (null)\n RSP: 0018:ffff880085757df0 EFLAGS: 00010282\n RAX: ffffffffa00168e0 RBX: ffff8800d1431800 RCX: 000000000000006e\n RDX: 0000000000000001 RSI: 0000000000000002 RDI: ffff88008543c000\n RBP: ffff880085757e48 R08: 0000000000000002 R09: 000000000000000a\n R10: 0000000000000000 R11: ffff88008543c2e0 R12: 00000000ffffffff\n R13: ffff8800b4641000 R14: 0000000000000005 R15: 0000000000000000\n FS: 00007fe8c9e05700(0000) GS:ffff88011fa00000(0000) knlGS:0000000000000000\n CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b\n CR2: 0000000000000000 CR3: 00000000b4502000 CR4: 00000000000406e0\n DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000\n DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400\n Process tee (pid: 10450, threadinfo ffff880085756000, task ffff8800c9f08000)\n Stack:\n ffffffff8138496a ffff8800b4641000 ffff88008543c268 0000000000000000\n ffff8800b4641000 ffff88008543c000 ffff8800d1431868 ffffffff81a78a90\n ffff8800b4641000 ffff88008543c000 ffff8800d1431800 ffff880085757e98\n Call Trace:\n [\u003cffffffff8138496a\u003e] ? slot_store+0xaa/0x265\n [\u003cffffffff81384bae\u003e] rdev_attr_store+0x89/0xa8\n [\u003cffffffff8115a96a\u003e] sysfs_write_file+0x108/0x144\n [\u003cffffffff81106b87\u003e] vfs_write+0xb1/0x10d\n [\u003cffffffff8106e6c0\u003e] ? trace_hardirqs_on_caller+0x111/0x135\n [\u003cffffffff81106cac\u003e] sys_write+0x4d/0x77\n [\u003cffffffff814fe702\u003e] system_call_fastpath+0x16/0x1b\n Code: Bad RIP value.\n RIP [\u003c (null)\u003e] (null)\n RSP \u003cffff880085757df0\u003e\n CR2: 0000000000000000\n ---[ end trace ba5fc64319a826fb ]---\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "9864c0053d3da4c5731ac8a6c4835179310bd40a", "tree": "4aea4c5a457f9e2c4058abfb2d6b69b843c721c5", "parents": [ "d744540cd39e93976c4c8401e140232444ef3b0b" ], "author": { "name": "马建朋", "email": "majianpeng@gmail.com", "time": "Thu Jun 09 11:42:48 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 09 11:42:48 2011 +1000" }, "message": "md: Using poll /proc/mdstat can monitor the events of adding a spare disks\n\nSigned-off-by: majianpeng \u003cmajianpeng@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "076f968b37f0232d883749da8f5031df5dea7ade", "tree": "aa1c58756c3754abe657be36b4720a16d5c41858", "parents": [ "1ed7242e591af7e233234d483f12d33818b189d9" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Tue Jun 07 17:51:30 2011 -0500" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:11:31 2011 +1000" }, "message": "MD: add sync_super to mddev_t struct\n\nAdd the \u0027sync_super\u0027 function pointer to MD array structure (struct mddev_s)\n\nIf device-mapper (dm-raid.c) is to define its own on-disk superblock and be\nable to load it, there must still be a way for MD to initiate superblock\nupdates. The simplest way to make this happen is to provide a pointer in\nthe MD array structure that can be set by device-mapper (or other module)\nwith a function to do this. If the function has been set, it will be used;\notherwise, the method with be looked up via \u0027super_types\u0027 as usual.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0fd018af37dadbb7826850883ad8abfecdb1a00b", "tree": "f95ebb9ec9111dc87ad8037ff7e07da802d924e3", "parents": [ "ac42450c7c814769bee963ae4b897c149bb0ab53" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Tue Jun 07 17:49:36 2011 -0500" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:11:31 2011 +1000" }, "message": "MD: move thread wakeups into resume\n\nMove personality and sync/recovery thread starting outside md_run.\n\nMoving the wakeup\u0027s of the personality and sync/recovery threads out of\nmd_run and into do_md_run and mddev_resume solves two issues:\n1) It allows bitmap_load to be called before the sync_thread is run and\n2) when MD personalities are used by device-mapper (dm-raid.c), the start-up\nof the array is better alligned with device-mapper primatives\n(CTR/resume/suspend/DTR). I/O - in this case, recovery operations - should\nnot happen until after a resume has taken place.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ac42450c7c814769bee963ae4b897c149bb0ab53", "tree": "582225e700c0a9eb0f99959f4f797a418a47ddc3", "parents": [ "68866e425be2ef2664aa5c691bb3ab789736acf5" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Tue Jun 07 17:48:35 2011 -0500" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:11:31 2011 +1000" }, "message": "MD: possible typo\n\nMake message a bit clearer by s/blocks/k/\n\nI chose \u0027k\u0027 vs \u0027kiB\u0027 or \u0027kB\u0027 because it is what is used earlier in the\nmessage. \u0027k\u0027 may be a bit ambigous, but I think it\u0027s better than \"blocks\"\nwhich normally means 512, but means 1024 in MD.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "68866e425be2ef2664aa5c691bb3ab789736acf5", "tree": "03cc5c452dcf70228a467031ae3f1989755cd1c1", "parents": [ "629acb6abac0ef217ee579e14084af2ce7381dbc" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@f14.redhat.com", "time": "Wed Jun 08 15:10:08 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:10:08 2011 +1000" }, "message": "MD: no sync IO while suspended\n\nDisallow resync I/O while the RAID array is suspended.\n\nRecovery, resync, and metadata I/O should not be allowed while a device is\nsuspended.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "629acb6abac0ef217ee579e14084af2ce7381dbc", "tree": "75963147cec3ff2e16aa775d103cb41706e503b0", "parents": [ "59c5f46fbe01a00eedf54a23789634438bb80603" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@f14.redhat.com", "time": "Wed Jun 08 15:10:08 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:10:08 2011 +1000" }, "message": "MD: no integrity register if no gendisk\n\nDon\u0027t attempt md_integrity_register if there is no gendisk struct available.\n\nWhen MD arrays are built via device-mapper, the gendisk structure is not\navailable via mddev.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b098636cf04c89db4036fedc778da0acc666ad1a", "tree": "11a970bdc482b4e5cb2211eefb9a71ae34355bd3", "parents": [ "ab9d47e990c12c11cc95ed1247a3782234a7e33a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 15:52:21 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 15:52:21 2011 +1000" }, "message": "md: allow resync_start to be set while an array is active.\n\nThe sysfs attribute \u0027resync_start\u0027 (known internally as recovery_cp),\nrecords where a resync is up to. A value of 0 means the array is\nnot known to be in-sync at all. A value of MaxSector means the array\nis believed to be fully in-sync.\n\nWhen the size of member devices of an array (RAID1,RAID4/5/6) is\nincreased, the array can be increased to match. This process sets\nresync_start to the old end-of-device offset so that the new part of\nthe array gets resynced.\n\nHowever with RAID1 (and RAID6) a resync is not technically necessary\nand may be undesirable. So it would be good if the implied resync\nafter the array is resized could be avoided.\n\nSo: change \u0027resync_start\u0027 so the value can be changed while the array\nis active, and as a precaution only allow it to be changed while\nresync/recovery is \u0027frozen\u0027. Changing it once resync has started is\nnot going to be useful anyway.\n\nThis allows the array to be resized without a resync by:\n write \u0027frozen\u0027 to \u0027sync_action\u0027\n write new size to \u0027component_size\u0027 (this will set resync_start)\n write \u0027none\u0027 to \u0027resync_start\u0027\n write \u0027idle\u0027 to \u0027sync_action\u0027.\n\nAlso slightly improve some tests on recovery_cp when resizing\nraid1/raid5. Now that an arbitrary value could be set we should be\nmore careful in our tests.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "bedd86b7773fd97f0d708cc0c371c8963ba7ba9a", "tree": "acc508209cdc418fcfe9a1493915dfa88c22bc29", "parents": [ "b0140891a8cea36469f58d23859e599b1122bd37" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:26:20 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:26:20 2011 +1000" }, "message": "md: reject a re-add request that cannot be honoured.\n\nThe \u0027add_new_disk\u0027 ioctl can be used to add a device either as a\nspare, or as an active disk that just needs to be resynced based on\nwrite-intent-bitmap information (re-add)\n\nCurrently if a re-add is requested but fails we add as a spare\ninstead. This makes it impossible for user-space to check for\nfailure.\n\nSo change to require that a re-add attempt will either succeed or\ncompletely fail. User-space can then decide what to do next.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b0140891a8cea36469f58d23859e599b1122bd37", "tree": "01f378d9964c1d24683a3c42bfd06b1da7d985b6", "parents": [ "693d92a1bbc9e42681c42ed190bd42b636ca876f" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 10 17:49:01 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:26:17 2011 +1000" }, "message": "md: Fix race when creating a new md device.\n\nThere is a race when creating an md device by opening /dev/mdXX.\n\nIf two processes do this at much the same time they will follow the\ncall path\n __blkdev_get -\u003e get_gendisk -\u003e kobj_lookup\n\nThe first will call\n -\u003e md_probe -\u003e md_alloc -\u003e add_disk -\u003e blk_register_region\n\nand the race happens when the second gets to kobj_lookup after\nadd_disk has called blk_register_region but before it returns to\nmd_alloc.\n\nIn the case the second will not call md_probe (as the probe is already\ndone) but will get a handle on the gendisk, return to __blkdev_get\nwhich will then call md_open (via the -\u003eopen) pointer.\n\nAs mddev-\u003egendisk hasn\u0027t been set yet, md_open will think something is\nwrong an return with ERESTARTSYS.\n\nThis can loop endlessly while the first thread makes no progress\nthrough add_disk. Nothing is blocking it, but due to scheduler\nbehaviour it doesn\u0027t get a turn.\nSo this is essentially a live-lock.\n\nWe fix this by simply moving the assignment to mddev-\u003egendisk before\nthe call the add_disk() so md_open doesn\u0027t get confused.\nAlso move blk_queue_flush earlier because add_disk should be as late\nas possible.\n\nTo make sure that md_open doesn\u0027t complete until md_alloc has done all\nthat is needed, we take mddev-\u003eopen_mutex during the last part of\nmd_alloc. md_open will wait for this.\n\nThis can cause a lock-up on boot so Cc:ing for stable.\nFor 2.6.36 and earlier a different patch will be needed as the\n\u0027blk_queue_flush\u0027 call isn\u0027t there.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReported-by: Thomas Jarosch \u003cthomas.jarosch@intra2net.com\u003e\nTested-by: Thomas Jarosch \u003cthomas.jarosch@intra2net.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "fee68723cf6ae00082f70f3eff17fceab2a4f7d7", "tree": "2ff52c9cce0115bc745b60bd9c0faffed8dd46f0", "parents": [ "3b71bd9337b404baab5c894e066be6b6bf51b1c3" ], "author": { "name": "Krzysztof Wojcik", "email": "krzysztof.wojcik@intel.com", "time": "Wed Apr 20 15:39:53 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Apr 20 15:39:53 2011 +1000" }, "message": "md: Cleanup after raid45-\u003eraid0 takeover\n\nProblem:\nAfter raid4-\u003eraid0 takeover operation, another takeover operation\n(e.g raid0-\u003eraid10) results \"kernel oops\".\nRoot cause:\nVariables \u0027degraded\u0027 in mddev structure is not cleared\non raid45-\u003eraid0 takeover.\n\nThis patch reset this variable.\n\nSigned-off-by: Krzysztof Wojcik \u003ckrzysztof.wojcik@intel.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "97658cdd3af7d01461874c93b89afa4a2465e7c6", "tree": "d4e2f930850eb3981307a3eec8c86399dcebf68a", "parents": [ "482c083492ddaa32ef5864bae3d143dc8bcdf7d1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:42 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:42 2011 +1000" }, "message": "md: provide generic support for handling unplug callbacks.\n\nWhen an md device adds a request to a queue, it can call\nmddev_check_plugged.\nIf this succeeds then we know that the md thread will be woken up\nshortly, and -\u003eplug_cnt will be non-zero until then, so some\nprocessing can be delayed.\n\nIf it fails, then no unplug callback is expected and the make_request\nfunction needs to do whatever is required to make the request happen.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "482c083492ddaa32ef5864bae3d143dc8bcdf7d1", "tree": "75017fd1a51fe945c65b3600442ddaa20b636b1a", "parents": [ "af1db72d8b340f97ad12b60175afdef43e6f0e60" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:42 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:42 2011 +1000" }, "message": "md - remove old plugging code.\n\nmd has some plugging infrastructure for RAID5 to use because the\nnormal plugging infrastructure required a \u0027request_queue\u0027, and when\ncalled from dm, RAID5 doesn\u0027t have one of those available.\n\nThis relied on the -\u003eunplug_fn callback which doesn\u0027t exist any more.\n\nSo remove all of that code, both in md and raid5. Subsequent patches\nwith restore the plugging functionality.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "25985edcedea6396277003854657b5f3cb31a628", "tree": "f026e810210a2ee7290caeb737c23cb6472b7c38", "parents": [ "6aba74f2791287ec407e0f92487a725a25908067" ], "author": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Wed Mar 30 22:57:33 2011 -0300" }, "committer": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Thu Mar 31 11:26:23 2011 -0300" }, "message": "Fix common misspellings\n\nFixes generated by \u0027codespell\u0027 and manually reviewed.\n\nSigned-off-by: Lucas De Marchi \u003clucas.demarchi@profusion.mobi\u003e\n" }, { "commit": "89078d572eb9ce8d4c04264b8b0ba86de0d74c8f", "tree": "d01104f97740c19daf01f49ae170df821a43252c", "parents": [ "7de8e5737df31a6826015a65135ed8f0a32bb2c4" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Mon Mar 28 20:09:12 2011 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Mar 28 17:53:29 2011 -0700" }, "message": "md: Fix integrity registration error when no devices are capable\n\nWe incorrectly returned -EINVAL when none of the devices in the array\nhad an integrity profile. This in turn prevented mdadm from starting\nthe metadevice. Fix this so we only return errors on mismatched\nprofiles and memory allocation failures.\n\nReported-by: Giacomo Catenazzi \u003ccate@cateee.net\u003e\nReported-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6c5103890057b1bb781b26b7aae38d33e4c517d8", "tree": "e6e57961dcddcb5841acb34956e70b9dc696a880", "parents": [ "3dab04e6978e358ad2307bca563fabd6c5d2c58b", "9d2e157d970a73b3f270b631828e03eb452d525e" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 24 10:16:26 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 24 10:16:26 2011 -0700" }, "message": "Merge branch \u0027for-2.6.39/core\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.39/core\u0027 of git://git.kernel.dk/linux-2.6-block: (65 commits)\n Documentation/iostats.txt: bit-size reference etc.\n cfq-iosched: removing unnecessary think time checking\n cfq-iosched: Don\u0027t clear queue stats when preempt.\n blk-throttle: Reset group slice when limits are changed\n blk-cgroup: Only give unaccounted_time under debug\n cfq-iosched: Don\u0027t set active queue in preempt\n block: fix non-atomic access to genhd inflight structures\n block: attempt to merge with existing requests on plug flush\n block: NULL dereference on error path in __blkdev_get()\n cfq-iosched: Don\u0027t update group weights when on service tree\n fs: assign sb-\u003es_bdi to default_backing_dev_info if the bdi is going away\n block: Require subsystems to explicitly allocate bio_set integrity mempool\n jbd2: finish conversion from WRITE_SYNC_PLUG to WRITE_SYNC and explicit plugging\n jbd: finish conversion from WRITE_SYNC_PLUG to WRITE_SYNC and explicit plugging\n fs: make fsync_buffers_list() plug\n mm: make generic_writepages() use plugging\n blk-cgroup: Add unaccounted time to timeslice_used.\n block: fixup plugging stubs for !CONFIG_BLOCK\n block: remove obsolete comments for blkdev_issue_zeroout.\n blktrace: Use rq-\u003ecmd_flags directly in blk_add_trace_rq.\n ...\n\nFix up conflicts in fs/{aio.c,super.c}\n" }, { "commit": "a91a2785b200864aef2270ed6a3babac7a253a20", "tree": "3d21d7ef82ddabbdbfb74499d68a73f15adc819f", "parents": [ "82f04ab47e1d94d78503591a7460b2cad9601ede" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Thu Mar 17 11:11:05 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 17 11:11:05 2011 +0100" }, "message": "block: Require subsystems to explicitly allocate bio_set integrity mempool\n\nMD and DM create a new bio_set for every metadevice. Each bio_set has an\nintegrity mempool attached regardless of whether the metadevice is\ncapable of passing integrity metadata. This is a waste of memory.\n\nInstead we defer the allocation decision to MD and DM since we know at\nmetadevice creation time whether integrity passthrough is needed or not.\n\nAutomatic integrity mempool allocation can then be removed from\nbioset_create() and we make an explicit integrity allocation for the\nfs_bio_set.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nReported-by: Zdenek Kabelac \u003czkabelac@redhat.com\u003e\nAcked-by: Mike Snitzer \u003csnizer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "bd2895eeade5f11f3e5906283c630bbdb4b57454", "tree": "4d98f4fcd80c7d062afce28823d08aee53e66f82", "parents": [ "016aa2ed1cc9cf704cf76d8df07751b6daa9750f", "24d51add7438f9696a7205927bf9de3c5c787a58" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 16 08:20:19 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 16 08:20:19 2011 -0700" }, "message": "Merge branch \u0027for-2.6.39\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq\n\n* \u0027for-2.6.39\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq:\n workqueue: fix build failure introduced by s/freezeable/freezable/\n workqueue: add system_freezeable_wq\n rds/ib: use system_wq instead of rds_ib_fmr_wq\n net/9p: replace p9_poll_task with a work\n net/9p: use system_wq instead of p9_mux_wq\n xfs: convert to alloc_workqueue()\n reiserfs: make commit_wq use the default concurrency level\n ocfs2: use system_wq instead of ocfs2_quota_wq\n ext4: convert to alloc_workqueue()\n scsi/scsi_tgt_lib: scsi_tgtd isn\u0027t used in memory reclaim path\n scsi/be2iscsi,qla2xxx: convert to alloc_workqueue()\n misc/iwmc3200top: use system_wq instead of dedicated workqueues\n i2o: use alloc_workqueue() instead of create_workqueue()\n acpi: kacpi*_wq don\u0027t need WQ_MEM_RECLAIM\n fs/aio: aio_wq isn\u0027t used in memory reclaim path\n input/tps6507x-ts: use system_wq instead of dedicated workqueue\n cpufreq: use system_wq instead of dedicated workqueues\n wireless/ipw2x00: use system_wq instead of dedicated workqueues\n arm/omap: use system_wq in mailbox\n workqueue: use WQ_MEM_RECLAIM instead of WQ_RESCUER\n" }, { "commit": "4c63f5646e405b5010cc9499419060bf2e838f5b", "tree": "df91ba315032c8ec4aafeb3ab96fdfa7c6c656e1", "parents": [ "cafb0bfca1a73efd6d8a4a6a6a716e6134b96c24", "69d60eb96ae8a73cf9b79cf28051caf973006011" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "message": "Merge branch \u0027for-2.6.39/stack-plug\u0027 into for-2.6.39/core\n\nConflicts:\n\tblock/blk-core.c\n\tblock/blk-flush.c\n\tdrivers/md/raid1.c\n\tdrivers/md/raid10.c\n\tdrivers/md/raid5.c\n\tfs/nilfs2/btnode.c\n\tfs/nilfs2/mdt.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "721a9602e6607417c6bc15b18e97a2f35266c690", "tree": "4987991e43f35b8b3b685fea0040c5265b578996", "parents": [ "cf15900e1209d5b46ec2d24643adbf561830935f" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 09 11:56:30 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:27 2011 +0100" }, "message": "block: kill off REQ_UNPLUG\n\nWith the plugging now being explicitly controlled by the\nsubmitter, callers need not pass down unplugging hints\nto the block layer. If they want to unplug, it\u0027s because they\nmanually plugged on their own - in which case, they should just\nunplug at will.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7eaceaccab5f40bbfda044629a6298616aeaed50", "tree": "33954d12f63e25a47eb6d86ef3d3d0a5e62bf752", "parents": [ "73c101011926c5832e6e141682180c4debe2cf45" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "message": "block: remove per-queue plugging\n\nCode has been converted over to the new explicit on-stack plugging,\nand delay users have been converted to use the new API for that.\nSo lets kill off the old plugging along with aops-\u003esync_page().\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f0b4f7e2f29af678bd9af43422c537dcb6008603", "tree": "dfc2aa18bbf8411a499ef8117a5e15490eb44728", "parents": [ "93b270f76e7ef3b81001576860c2701931cdc78b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Feb 24 17:26:41 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Feb 24 17:26:41 2011 +1100" }, "message": "md: Fix - again - partition detection when array becomes active\n\nRevert\n b821eaa572fd737faaf6928ba046e571526c36c6\nand\n f3b99be19ded511a1bf05a148276239d9f13eefa\n\nWhen I wrote the first of these I had a wrong idea about the\nlifetime of \u0027struct block_device\u0027. It can disappear at any time that\nthe block device is not open if it falls out of the inode cache.\n\nSo relying on the \u0027size\u0027 recorded with it to detect when the\ndevice size has changed and so we need to revalidate, is wrong.\n\nRather, we really do need the \u0027changed\u0027 attribute stored directly in\nthe mddev and set/tested as appropriate.\n\nWithout this patch, a sequence of:\n mknod / open / close / unlink\n\n(which can cause a block_device to be created and then destroyed)\nwill result in a rescan of the partition table and consequence removal\nand addition of partitions.\nSeveral of these in a row can get udev racing to create and unlink and\nother code can get confused.\n\nWith the patch, the rescan is only performed when needed and so there\nare no races.\n\nThis is suitable for any stable kernel from 2.6.35.\n\nReported-by: \"Wojcik, Krzysztof\" \u003ckrzysztof.wojcik@intel.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nCc: stable@kernel.org\n" }, { "commit": "43d133c18b44e7d82d82ef0dcc2bddd55d5dfe81", "tree": "8de75c837b55874cc8a81a29bdedbc62668d4481", "parents": [ "4149efb22da66e326fc48baf80d628834509f7f0", "6f576d57f1fa0d6026b495d8746d56d949989161" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Feb 21 09:43:56 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Feb 21 09:43:56 2011 +0100" }, "message": "Merge branch \u0027master\u0027 into for-2.6.39\n" }, { "commit": "8f5f02c460b7ca74ce55ce126ce0c1e58a3f923d", "tree": "de4f933fe250bd2ba71727f535432a82ef68883b", "parents": [ "cbe6ef1d2622e08e272600b3cb6040bed60f0450" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 16 13:58:51 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 16 13:58:51 2011 +1100" }, "message": "md: correctly handle probe of an \u0027mdp\u0027 device.\n\n\u0027mdp\u0027 devices are md devices with preallocated device numbers\nfor partitions. As such it is possible to mknod and open a partition\nbefore opening the whole device.\n\nthis causes md_probe() to be called with a device number of a\npartition, which in-turn calls mddev_find with such a number.\n\nHowever mddev_find expects the number of a \u0027whole device\u0027 and\ndoes the wrong thing with partition numbers.\n\nSo add code to mddev_find to remove the \u0027partition\u0027 part of\na device number and just work with the \u0027whole device\u0027.\n\nThis patch addresses https://bugzilla.kernel.org/show_bug.cgi?id\u003d28652\n\nReported-by: hkmaly@bigfoot.com\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nCc: \u003cstable@kernel.org\u003e\n" }, { "commit": "cbe6ef1d2622e08e272600b3cb6040bed60f0450", "tree": "03d1ae0fe73404a3bd7850d89c415f3412977693", "parents": [ "f7bee80945155ad0326916486dabc38428c6cdef" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 16 13:58:38 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 16 13:58:38 2011 +1100" }, "message": "md: don\u0027t set_capacity before array is active.\n\nIf the desired size of an array is set (via sysfs) before the array is\nactive (which is the normal sequence), we currrently call set_capacity\nimmediately.\nThis means that a subsequent \u0027open\u0027 (as can be caused by some\nudev-triggers program) will notice the new size and try to probe for\npartitions. However as the array isn\u0027t quite ready yet the read will\nfail. Then when the array is read, as the size doesn\u0027t change again\nwe don\u0027t try to re-probe.\n\nSo when setting array size via sysfs, only call set_capacity if the\narray is already active.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e91ece5590b3c728624ab57043fc7a05069c604a", "tree": "4c5fd4ad374caf5bd9c5f575b0a228b5cdf2f191", "parents": [ "c6751b2bde477f56ceef67aa1d298ce44e8e2e23" ], "author": { "name": "Chris Mason", "email": "chris.mason@oracle.com", "time": "Mon Feb 07 19:21:48 2011 -0500" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Feb 08 09:53:28 2011 +1100" }, "message": "md_make_request: don\u0027t touch the bio after calling make_request\n\nmd_make_request was calling bio_sectors() for part_stat_add\nafter it was calling the make_request function. This is\nbad because the make_request function can free the bio and\nbecause the bi_size field can change around.\n\nThe fix here was suggested by Jens Axboe. It saves the\nsector count before the make_request call. I hit this\nwith CONFIG_DEBUG_PAGEALLOC turned on while trying to break\nhis pretty fusionio card.\n\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Chris Mason \u003cchris.mason@oracle.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c6751b2bde477f56ceef67aa1d298ce44e8e2e23", "tree": "59916ec74802396ec4b3f871d27232550848bc8c", "parents": [ "7281f8129c362436237b82c8c026494dd36479dc" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 02 11:57:13 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 02 11:57:13 2011 +1100" }, "message": "md: Don\u0027t allow slot_store while resync/recovery is happening.\n\nActivating a spare in an array while resync/recovery is already\nhappening can lead the that spare being marked in-sync when it isn\u0027t\nreally.\nSo don\u0027t allow the \u0027slot\u0027 to be set (this activating the device)\nwhile resync/recovery is happening.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7281f8129c362436237b82c8c026494dd36479dc", "tree": "c12196b42c807d4ee80d5f6e1657f31fa3ea4d7b", "parents": [ "a8c42c7f476b5bb39bb3a5b32d5473b9a46cadb9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 14:30:27 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 14:30:27 2011 +1100" }, "message": "md: don\u0027t clear curr_resync_completed at end of resync.\n\nThere is no need to set this to zero at this point. It will be\nset to zero by remove_and_add_spares or at the start of\nmd_do_sync at the latest.\nAnd setting it to zero before MD_RECOVERY_RUNNING is cleared can\nmake a \u0027zero\u0027 appear briefly in the \u0027sync_completed\u0027 sysfs attribute\njust as resync is finishing.\n\nSo simply remove this setting to zero.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a8c42c7f476b5bb39bb3a5b32d5473b9a46cadb9", "tree": "59b82042476e6b92ee293570d30a2580dba56d09", "parents": [ "fc3a08b85b7a4f6c1069e5f71f6ad40d925ff55b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 13:47:13 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 13:47:13 2011 +1100" }, "message": "md: Don\u0027t use remove_and_add_spares to remove failed devices from a read-only array\n\nremove_and_add_spares is called in two places where the needs really\nare very different.\nremove_and_add_spares should not be called on an array which is about\nto be reshaped as some extra devices might have been manually added\nand that would remove them. However if the array is \u0027read-auto\u0027,\nthat will currently happen, which is bad.\n\nSo in the \u0027ro !\u003d 0\u0027 case don\u0027t call remove_and_add_spares but simply\nremove the failed devices as the comment suggests is needed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f21e9ff7f77d41ceca4e1e5ee5a4efa5ad7a5e40", "tree": "c6bf9c0457ae891505983c1493bf15856e815c5b", "parents": [ "50da08409654e036c4c964a473567a61a654cb83" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 12:10:09 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 12:10:09 2011 +1100" }, "message": "md: Remove the AllReserved flag for component devices.\n\nThis flag is not needed and is used badly.\n\nDevices that are included in a native-metadata array are reserved\nexclusively for that array - and currently have AllReserved set.\nThey all are bd_claimed for the rdev and so cannot be shared.\n\nDevices that are included in external-metadata arrays can be shared\namong multiple arrays - providing there is no overlap.\nThese are bd_claimed for md in general - not for a particular rdev.\n\nWhen changing the amount of a device that is used in an array we need\nto check for overlap. This currently includes a check on AllReserved\nSo even without overlap, sharing with an AllReserved device is not\nallowed.\nHowever the bd_claim usage already precludes sharing with these\ndevices, so the test on AllReserved is not needed. And in fact it is\nwrong.\n\nAs this is the only use of AllReserved, simply remove all usage and\ndefinition of AllReserved.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "de171cb9a52598cc023adceafc6c166112401386", "tree": "56fc899c45b538dedba0fc139e3213a0f47f9a59", "parents": [ "1f0324caefd39985e9fe052fac97da31694db31e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 11:57:42 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jan 31 11:57:42 2011 +1100" }, "message": "md: revert change to raid_disks on failure.\n\nIf we try to update_raid_disks and it fails, we should put\n\u0027delta_disks\u0027 back to zero. This is important because some code,\nsuch as slot_store, assumes that delta_disks has been validated.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ada609ee2ac2e03bd8abb07f9b3e92cd2e650f19", "tree": "ccc1051a961b12448420428b5a400a6333d3e821", "parents": [ "c723fdab8aa728dc2bf0da6a0de8bb9c3f588d84" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 25 14:35:54 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 25 14:35:54 2011 +0100" }, "message": "workqueue: use WQ_MEM_RECLAIM instead of WQ_RESCUER\n\nWQ_RESCUER is now an internal flag and should only be used in the\nworkqueue implementation proper. Use WQ_MEM_RECLAIM instead.\n\nThis doesn\u0027t introduce any functional difference.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: dm-devel@redhat.com\nCc: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "49731baa41df404c2c3f44555869ab387363af43", "tree": "5d3476368fa546aebb1c223e9cf1bab5ad80f698", "parents": [ "c553f8e335c00a7cff3ab3f13e793b13d3f2207f" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jan 14 18:43:57 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 14 18:44:22 2011 +0100" }, "message": "block: restore multiple bd_link_disk_holder() support\n\nCommit e09b457b (block: simplify holder symlink handling) incorrectly\nassumed that there is only one link at maximum. dm may use multiple\nlinks and expects block layer to track reference count for each link,\nwhich is different from and unrelated to the exclusive device holder\nidentified by @holder when the device is opened.\n\nRemove the single holder assumption and automatic removal of the link\nand revive the per-link reference count tracking. The code\nessentially behaves the same as before commit e09b457b sans the\nunnecessary kobject reference count dancing.\n\nWhile at it, note that this facility should not be used by anyone else\nthan the current ones. Sysfs symlinks shouldn\u0027t be abused like this\nand the whole thing doesn\u0027t belong in the block layer at all.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Milan Broz \u003cmbroz@redhat.com\u003e\nCc: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: linux-raid@vger.kernel.org\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "509e4aef44eb10e4aef1f81c3c3ff1214671503b", "tree": "b90daecbc3d05787b0dd141e524b67d8579a6dfd", "parents": [ "375b6f5a40a3d8373561a3dbeed440a845e6d379", "bf2cb0dab8c97f00a71875d9b13dbac17a2f47ca" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 17:30:20 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 17:30:20 2011 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md:\n md: Fix removal of extra drives when converting RAID6 to RAID5\n md: range check slot number when manually adding a spare.\n md/raid5: handle manually-added spares in start_reshape.\n md: fix sync_completed reporting for very large drives (\u003e2TB)\n md: allow suspend_lo and suspend_hi to decrease as well as increase.\n md: Don\u0027t let implementation detail of curr_resync leak out through sysfs.\n md: separate meta and data devs\n md-new-param-to_sync_page_io\n md-new-param-to-calc_dev_sboffset\n md: Be more careful about clearing flags bit in -\u003erecovery\n md: md_stop_writes requires mddev_lock.\n md/raid5: use sysfs_notify_dirent_safe to avoid NULL pointer\n md: Ensure no IO request to get md device before it is properly initialised.\n md: Fix single printks with multiple KERN_\u003clevel\u003es\n md: fix regression resulting in delays in clearing bits in a bitmap\n md: fix regression with re-adding devices to arrays with no metadata\n" }, { "commit": "bf2cb0dab8c97f00a71875d9b13dbac17a2f47ca", "tree": "68587eaff5c7cb85628c1f68aecb19f2a18c8f6e", "parents": [ "ba1b41b6b4e30cb66ae2775faadea05cae3ce61c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: Fix removal of extra drives when converting RAID6 to RAID5\n\nWhen a RAID6 is converted to a RAID5, the extra drive should\nbe discarded. However it isn\u0027t due to a typo in a comparison.\n\nThis bug was introduced in commit e93f68a1fc6 in 2.6.35-rc4\nand is suitable for any -stable since than.\n\nAs the extra drive is not removed, the \u0027degraded\u0027 counter is wrong and\nso the RAID5 will not respond correctly to a subsequent failure.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ba1b41b6b4e30cb66ae2775faadea05cae3ce61c", "tree": "02da80f72095418dd1a08661f59c24607029a8e7", "parents": [ "1a940fcee31ec6c18c2f24dbdad31d54e4c35048" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: range check slot number when manually adding a spare.\n\nWhen adding a spare to an active array, we should check the slot\nnumber, but allow it to be larger than raid_disks if a reshape\nis being prepared.\n\nApply the same test when adding a device to an\narray-under-construction. It already had most of the test in place,\nbut not quite all.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "13ae864bc86ff65547ffe7e966b6433a0d0edb8a", "tree": "6e5ca3a6736b75fa0db8415c1e3a2506af6ab9cc", "parents": [ "23ddff3792f61193695114c68d6ebd57e974c4f8" ], "author": { "name": "Rémi Rérolle", "email": "rrerolle@lacie.com", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: fix sync_completed reporting for very large drives (\u003e2TB)\n\nThe values exported in the sync_completed file are unsigned long, which\noverflows with very large drives, resulting in wrong values reported.\n\nSince sync_completed uses sectors as unit, we\u0027ll start getting wrong\nvalues with components larger than 2TB.\n\nThis patch simply replaces the use of unsigned long by unsigned long long.\n\nSigned-off-by: Rémi Rérolle \u003crrerolle@lacie.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "23ddff3792f61193695114c68d6ebd57e974c4f8", "tree": "5b16efbc900648c8ef402a41cb308759a2706c61", "parents": [ "75d3da43cb74d2e5fb87816dbfecb839cd97c7f4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: allow suspend_lo and suspend_hi to decrease as well as increase.\n\nThe sysfs attributes \u0027suspend_lo\u0027 and \u0027suspend_hi\u0027 describe a region\nto which read/writes are suspended so that the under lying data can be\nmanipulated without user-space noticing.\nCurrently the window they describe can only move forwards along the\ndevice. However this is an unnecessary restriction which will cause\nproblems with planned developments.\nSo relax this restriction and allow these endpoints to move\narbitrarily.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "75d3da43cb74d2e5fb87816dbfecb839cd97c7f4", "tree": "27074aceb801ada315329591f32c938b36150ed9", "parents": [ "a6ff7e089c7fca813c956ccbed824087e89a3a49" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: Don\u0027t let implementation detail of curr_resync leak out through sysfs.\n\nmddev-\u003ecurr_resync has artificial values of \u00271\u0027 and \u00272\u0027 which are used\nby the code which ensures only one resync is happening at a time on\nany given device.\n\nThese values are internal and should never be exposed to user-space\n(except when translated appropriately as in the \u0027pending\u0027 status in\n/proc/mdstat).\n\nUnfortunately they are as -\u003ecurr_resync is assigned to\n-\u003ecurr_resync_completed and that value is directly visible through\nsysfs.\n\nSo change the assignments to -\u003ecurr_resync_completed to get the same\nvalued from elsewhere in a form that doesn\u0027t have the magic \u00271\u0027 or \u00272\u0027\nvalues.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a6ff7e089c7fca813c956ccbed824087e89a3a49", "tree": "6537f72a2431c30601736676e9b1f89a7f1fe52e", "parents": [ "ccebd4c4159462c96397ae9af9c667bb394d7b70" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:34 2011 +1100" }, "message": "md: separate meta and data devs\n\nAllow the metadata to be on a separate device from the\ndata.\n\nThis doesn\u0027t mean the data and metadata will by on separate\nphysical devices - it simply gives device-mapper and userspace\ntools more flexibility.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ccebd4c4159462c96397ae9af9c667bb394d7b70", "tree": "04554f0e7607d906c6b09f31a995fca0dd40c5b5", "parents": [ "57b2caa394393f8870ed41bdcc38a7542593018f" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md-new-param-to_sync_page_io\n\nAdd new parameter to \u0027sync_page_io\u0027.\n\nThe new parameter allows us to distinguish between metadata and data\noperations. This becomes important later when we add the ability to\nuse separate devices for data and metadata.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\n" }, { "commit": "57b2caa394393f8870ed41bdcc38a7542593018f", "tree": "22579497603304e9694986a4cac2cd360c993fa9", "parents": [ "7ebc0be7fff4146e87b4078f054977b72998abd3" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md-new-param-to-calc_dev_sboffset\n\nWhen we allow for separate devices for data and metadata\nin a later patch, we will need to be able to calculate\nthe superblock offset based on more than the bdev.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\n" }, { "commit": "7ebc0be7fff4146e87b4078f054977b72998abd3", "tree": "f5b5b40674ec4e221e77eb1f9e4f36138f75df3c", "parents": [ "defad61a5b16352d3e22a04d4c930a5b5a7fd1f0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md: Be more careful about clearing flags bit in -\u003erecovery\n\nSetting -\u003erecovery to 0 is generally not a good idea as it could clear\nbits that shouldn\u0027t be cleared. In particular, MD_RECOVERY_FROZEN\nshould only be cleared on explicit request from user-space.\n\nSo when we need to clear things, just clear the bits that need\nclearing.\n\nAs there are a few different places which reap a resync process - and\nsome do an incomplte job - factor out the code for doing the from\nmd_check_recovery and call that function instead of open coding part\nof it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReported-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\n" }, { "commit": "defad61a5b16352d3e22a04d4c930a5b5a7fd1f0", "tree": "c353bd036c12011d0eee7f7924c230888fd34b0a", "parents": [ "43c73ca43b3e03bb228ff9350b6b44d0e560f262" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md: md_stop_writes requires mddev_lock.\n\nAs md_stop_writes manipulates the sync_thread and calls md_update_sb,\nit need to be called with mddev_lock held.\n\nIn all internal cases it is, but the symbol is exported for dm-raid to\ncall and in that case the lock won\u0027t be help.\nDo make an exported version which takes the lock, and an internal\nversion which does not.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0ca69886a8273ac1350143d562280bfcbe4760dc", "tree": "98acbe1e5682e4455bf7d2e7a9413b5a3fd43b2a", "parents": [ "067032bc628598606056412594042564fcf09e22" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md: Ensure no IO request to get md device before it is properly initialised.\n\nWhen an md device is in the process of coming on line it is possible\nfor an IO request (typically a partition table probe) to get through\nbefore the array is fully initialised, which can cause unexpected\nbehaviour (e.g. a crash).\n\nSo explicitly record when the array is ready for IO and don\u0027t allow IO\nthrough until then.\n\nThere is no possibility for a similar problem when the array is going\noff-line as there must only be one \u0027open\u0027 at that time, and it is busy\noff-lining the array and so cannot send IO requests. So no memory\nbarrier is needed in md_stop()\n\nThis has been a bug since commit 409c57f3801 in 2.6.30 which\nintroduced md_make_request. Before then, each personality would\nregister its own make_request_fn when it was ready.\nThis is suitable for any stable kernel from 2.6.30.y onwards.\n\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReported-by: \"Hawrylewicz Czarnowski, Przemyslaw\" \u003cprzemyslaw.hawrylewicz.czarnowski@intel.com\u003e\n" }, { "commit": "6c9879101442b08581e8a0e3ae6b7f643a78fd63", "tree": "bfe780d4e09896988fe367aa329a94daf55163c9", "parents": [ "bf572541ab44240163eaa2d486b06f306a31d45a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:13:53 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:13:53 2011 +1100" }, "message": "md: fix regression resulting in delays in clearing bits in a bitmap\n\ncommit 589a594be1fb (2.6.37-rc4) fixed a problem were md_thread would\nsometimes call the -\u003erun function at a bad time.\n\nIf an error is detected during array start up after the md_thread has\nbeen started, the md_thread is killed. This resulted in the -\u003erun\nfunction being called once. However the array may not be in a state\nthat it is safe to call -\u003erun.\n\nHowever the fix imposed meant that -\u003erun was not called on a timeout.\nThis means that when an array goes idle, bitmap bits do not get\ncleared promptly. While the array is busy the bits will still be\ncleared when appropriate so this is not very serious. There is no\nrisk to data.\n\nChange the test so that we only avoid calling -\u003erun when the thread\nis being stopped. This more explicitly addresses the problem situation.\n\nThis is suitable for 2.6.37-stable and any -stable kernel to which\n589a594be1fb was applied.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "275220f0fcff1adf28a717076e00f575edf05fda", "tree": "d249bccc80c64443dab211639050c4fb14332648", "parents": [ "fe3c560b8a22cb28e54fe8950abef38e88d75831", "81c5e2ae33c4b19e53966b427e33646bf6811830" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 10:45:01 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 10:45:01 2011 -0800" }, "message": "Merge branch \u0027for-2.6.38/core\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.38/core\u0027 of git://git.kernel.dk/linux-2.6-block: (43 commits)\n block: ensure that completion error gets properly traced\n blktrace: add missing probe argument to block_bio_complete\n block cfq: don\u0027t use atomic_t for cfq_group\n block cfq: don\u0027t use atomic_t for cfq_queue\n block: trace event block fix unassigned field\n block: add internal hd part table references\n block: fix accounting bug on cross partition merges\n kref: add kref_test_and_get\n bio-integrity: mark kintegrityd_wq highpri and CPU intensive\n block: make kblockd_workqueue smarter\n Revert \"sd: implement sd_check_events()\"\n block: Clean up exit_io_context() source code.\n Fix compile warnings due to missing removal of a \u0027ret\u0027 variable\n fs/block: type signature of major_to_index(int) to major_to_index(unsigned)\n block: convert !IS_ERR(p) \u0026\u0026 p to !IS_ERR_NOR_NULL(p)\n cfq-iosched: don\u0027t check cfqg in choose_service_tree()\n fs/splice: Pull buf-\u003eops-\u003econfirm() from splice_from_pipe actors\n cdrom: export cdrom_check_events()\n sd: implement sd_check_events()\n sr: implement sr_check_events()\n ...\n" }, { "commit": "bf572541ab44240163eaa2d486b06f306a31d45a", "tree": "ef9630d692ce3681bf226877056622afd070f1c0", "parents": [ "3c0eee3fe6a3a1c745379547c7e7c904aa64f6d5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jan 12 09:03:35 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jan 12 09:03:35 2011 +1100" }, "message": "md: fix regression with re-adding devices to arrays with no metadata\n\nCommit 1a855a0606 (2.6.37-rc4) fixed a problem where devices were\nre-added when they shouldn\u0027t be but caused a regression in a less\ncommon case that means sometimes devices cannot be re-added when they\nshould be.\n\nIn particular, when re-adding a device to an array without metadata\nwe should always access the device, but after the above commit we\ndidn\u0027t.\n\nThis patch sets the In_sync flag in that case so that the re-add\nsucceeds.\n\nThis patch is suitable for any -stable kernel to which 1a855a0606 was\napplied.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7f8635cc9e66a26d7280ba680b044fa2f65104af", "tree": "0d8506e86d07e15c473aca1a09af7ad6ff7d8b49", "parents": [ "3cb50ddf97a0a1ca4c68bc12fa1e727a6b45fbf2", "0fc13c8995cd96f4123de400c71c223d80400ed9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 20 09:19:46 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 20 09:19:46 2010 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block:\n cciss: fix cciss_revalidate panic\n block: max hardware sectors limit wrapper\n block: Deprecate QUEUE_FLAG_CLUSTER and use queue_limits instead\n blk-throttle: Correct the placement of smp_rmb()\n blk-throttle: Trim/adjust slice_end once a bio has been dispatched\n block: check for proper length of iov entries earlier in blk_rq_map_user_iov()\n drbd: fix for spin_lock_irqsave in endio callback\n drbd: don\u0027t recvmsg with zero length\n" }, { "commit": "e692cb668fdd5a712c6ed2a2d6f2a36ee83997b4", "tree": "accf682fe5e1388f305b5fc364a931dfda5f3fb9", "parents": [ "04a6b516cdc6efc2500b52a540cf65be8c5aaf9e" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Dec 01 19:41:49 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 08:35:53 2010 +0100" }, "message": "block: Deprecate QUEUE_FLAG_CLUSTER and use queue_limits instead\n\nWhen stacking devices, a request_queue is not always available. This\nforced us to have a no_cluster flag in the queue_limits that could be\nused as a carrier until the request_queue had been set up for a\nmetadevice.\n\nThere were several problems with that approach. First of all it was up\nto the stacking device to remember to set queue flag after stacking had\ncompleted. Also, the queue flag and the queue limits had to be kept in\nsync at all times. We got that wrong, which could lead to us issuing\ncommands that went beyond the max scatterlist limit set by the driver.\n\nThe proper fix is to avoid having two flags for tracking the same thing.\nWe deprecate QUEUE_FLAG_CLUSTER and use the queue limit directly in the\nblock layer merging functions. The queue_limit \u0027no_cluster\u0027 is turned\ninto \u0027cluster\u0027 to avoid double negatives and to ease stacking.\nClustering defaults to being enabled as before. The queue flag logic is\nremoved from the stacking function, and explicitly setting the cluster\nflag is no longer necessary in DM and MD.\n\nReported-by: Ed Lin \u003ced.lin@promise.com\u003e\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "589a594be1fb8815b3f18e517be696c48664f728", "tree": "9de6f6b4f9244802ccf77312f7fba8612dd13a75", "parents": [ "1a855a0606653d2d82506281e2c686bacb4b2f45" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 17:02:14 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 17:02:14 2010 +1100" }, "message": "md: protect against NULL reference when waiting to start a raid10.\n\nWhen we fail to start a raid10 for some reason, we call\nmd_unregister_thread to kill the thread that was created.\n\nUnfortunately md_thread() will then make one call into the handler\n(raid10d) even though md_wakeup_thread has not been called. This is\nnot safe and as md_unregister_thread is called after mddev-\u003eprivate\nhas been set to NULL, it will definitely cause a NULL dereference.\n\nSo fix this at both ends:\n - md_thread should only call the handler if THREAD_WAKEUP has been\n set.\n - raid10 should call md_unregister_thread before setting things\n to NULL just like all the other raid modules do.\n\nThis is applicable to 2.6.35 and later.\n\nCc: stable@kernel.org\nReported-by: \"Citizen\" \u003ccitizen_lee@thecus.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1a855a0606653d2d82506281e2c686bacb4b2f45", "tree": "7eeca410e738491318c0d80bd0a0940c392e513f", "parents": [ "a035fc3e2531703b539f23bec4ca7943cfc69349" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:36:28 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:36:28 2010 +1100" }, "message": "md: fix bug with re-adding of partially recovered device.\n\nWith v0.90 metadata, a hot-spare does not become a full member of the\narray until recovery is complete. So if we re-add such a device to\nthe array, we know that all of it is as up-to-date as the event count\nwould suggest, and so it a bitmap-based recovery is possible.\n\nHowever with v1.x metadata, the hot-spare immediately becomes a full\nmember of the array, but it record how much of the device has been\nrecovered. If the array is stopped and re-assembled recovery starts\nfrom this point.\n\nWhen such a device is hot-added to an array we currently lose the \u0027how\nmuch is recovered\u0027 information and incorrectly included it as a full\nin-sync member (after bitmap-based fixup).\nThis is wrong and unsafe and could corrupt data.\n\nSo be more careful about setting saved_raid_disk - which is what\nguides the re-adding of devices back into an array.\nThe new code matches the code in slot_store which does a similar\nthing, which is encouraging.\n\nThis is suitable for any -stable kernel.\n\nReported-by: \"Dailey, Nate\" \u003cNate.Dailey@stratus.com\u003e\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a035fc3e2531703b539f23bec4ca7943cfc69349", "tree": "f10e160ce65745fd7ac56838c491a45ae180cb1e", "parents": [ "a7a07e69653acf8540daa1da053cd84bf86e8e66" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:17:51 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:17:51 2010 +1100" }, "message": "md: fix possible deadlock in handling flush requests.\n\nAs recorded in\n https://bugzilla.kernel.org/show_bug.cgi?id\u003d24012\n\nit is possible for a flush request through md to hang. This is due to\nan interaction between the recursion avoidance in\ngeneric_make_request, the insistence in md of only having one flush\nactive at a time, and the possibility of dm (or md) submitting two\nflush requests to a device from the one generic_make_request.\n\nIf a generic_make_request call into dm causes two flush requests to be\nqueued (as happens if the dm table has two targets - they get one\neach), these two will be queued inside generic_make_request.\n\nAssume they are for the same md device.\nThe first is processed and causes 1 or more flush requests to be sent\nto lower devices. These get queued within generic_make_request too.\nThen the second flush to the md device gets handled and it blocks\nwaiting for the first flush to complete. But it won\u0027t complete until\nthe two lower-device requests complete, and they haven\u0027t even been\nsubmitted yet as they are on the generic_make_request queue.\n\nThe deadlock can be broken by using a separate thread to submit the\nrequests to lower devices. md has such a thread readily available:\nmd_wq.\n\nSo use it to submit these requests.\n\nReported-by: Giacomo Catenazzi \u003ccate@cateee.net\u003e\nTested-by: Giacomo Catenazzi \u003ccate@cateee.net\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a7a07e69653acf8540daa1da053cd84bf86e8e66", "tree": "edeb5ec7fc686c8699193338d3c3964aeb6279a8", "parents": [ "2b74e12e567feb4163e32815bce0be57489e73b9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:04:25 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 16:04:25 2010 +1100" }, "message": "md: move code in to submit_flushes.\n\nsubmit_flushes is called from exactly one place.\nMove the code that is before and after that call into\nsubmit_flushes.\n\nThis has not functional change, but will make the next patch\nsmaller and easier to follow.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b74e12e567feb4163e32815bce0be57489e73b9", "tree": "2eef8f977209a7e5ff88b4366bd72ccf59c8a987", "parents": [ "be20e6c67b6c6024c19094d3447f144d118733b0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 15:59:01 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Dec 09 15:59:01 2010 +1100" }, "message": "md: remove handling of flush_pending in md_submit_flush_data\n\nNone of the functions called between setting flush_pending to 1, and\natomic_dec_and_test can change flush_pending, or will anything\nrunning in any other thread (as -\u003eflush_bio is not NULL). So the\natomic_dec_and_test will always succeed.\nSo remove the atomic_sec and the atomic_dec_and_test.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f30195c50245d8ace628e1978b1daa8df86e7224", "tree": "edf4f91c4356c41d50fb07f15cebd1e6442538ad", "parents": [ "d07335e51df0c6dec202d315fc4f1f7e100eec4e", "d4d77629953eabd3c14f6fa5746f6b28babfc55f" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Nov 27 19:49:18 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Nov 27 19:49:18 2010 +0100" }, "message": "Merge branch \u0027cleanup-bd_claim\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc into for-2.6.38/core\n" }, { "commit": "be20e6c67b6c6024c19094d3447f144d118733b0", "tree": "170f7a810d5947cdba7edc965676ad7d5ea8948c", "parents": [ "8f9e0ee38f75d4740daa9e42c8af628d33d19a02" ], "author": { "name": "Darrick J. Wong", "email": "djwong@us.ibm.com", "time": "Wed Nov 24 16:40:33 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Nov 24 16:40:33 2010 +1100" }, "message": "md: Call blk_queue_flush() to establish flush/fua support\n\nBefore 2.6.37, the md layer had a mechanism for catching I/Os with the\nbarrier flag set, and translating the barrier into barriers for all\nthe underlying devices. With 2.6.37, I/O barriers have become plain\nold flushes, and the md code was updated to reflect this. However,\none piece was left out -- the md layer does not tell the block layer\nthat it supports flushes or FUA access at all, which results in md\nsilently dropping flush requests.\n\nSince the support already seems there, just add this one piece of\nbookkeeping.\n\nSigned-off-by: Darrick J. Wong \u003cdjwong@us.ibm.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c26a44ed1e552aaa1d4ceb71842002d235fe98d7", "tree": "47e0d62c2693ae60ae7a5bc44c2dfcd6a2f7e47e", "parents": [ "3561d43fd289f590fdae672e5eb831b8d5cf0bf6" ], "author": { "name": "Justin Maggard", "email": "jmaggard10@gmail.com", "time": "Wed Nov 24 16:36:17 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Nov 24 16:36:17 2010 +1100" }, "message": "md: fix return value of rdev_size_change()\n\nWhen trying to grow an array by enlarging component devices,\nrdev_size_store() expects the return value of rdev_size_change() to be\nin sectors, but the actual value is returned in KBs.\n\nThis functionality was broken by commit\n dd8ac336c13fd8afdb082ebacb1cddd5cf727889\nso this patch is suitable for any kernel since 2.6.30.\n\nCc: stable@kernel.org\nSigned-off-by: Justin Maggard \u003cjmaggard10@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d4d77629953eabd3c14f6fa5746f6b28babfc55f", "tree": "38cce0d4764ecb34a9f7f49332959780e28bb786", "parents": [ "75f1dc0d076d1c1168f2115f1941ea627d38bd5a" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:18 2010 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:18 2010 +0100" }, "message": "block: clean up blkdev_get() wrappers and their users\n\nAfter recent blkdev_get() modifications, open_by_devnum() and\nopen_bdev_exclusive() are simple wrappers around blkdev_get().\nReplace them with blkdev_get_by_dev() and blkdev_get_by_path().\n\nblkdev_get_by_dev() is identical to open_by_devnum().\nblkdev_get_by_path() is slightly different in that it doesn\u0027t\nautomatically add %FMODE_EXCL to @mode.\n\nAll users are converted. Most conversions are mechanical and don\u0027t\nintroduce any behavior difference. There are several exceptions.\n\n* btrfs now sets FMODE_EXCL in btrfs_device-\u003emode, so there\u0027s no\n reason to OR it explicitly on blkdev_put().\n\n* gfs2, nilfs2 and the generic mount_bdev() now set FMODE_EXCL in\n sb-\u003es_mode.\n\n* With the above changes, sb-\u003es_mode now always should contain\n FMODE_EXCL. WARN_ON_ONCE() added to kill_block_super() to detect\n errors.\n\nThe new blkdev_get_*() functions are with proper docbook comments.\nWhile at it, add function description to blkdev_get() too.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Philipp Reisner \u003cphilipp.reisner@linbit.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: Joern Engel \u003cjoern@lazybastard.org\u003e\nCc: Chris Mason \u003cchris.mason@oracle.com\u003e\nCc: Jan Kara \u003cjack@suse.cz\u003e\nCc: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: KONISHI Ryusuke \u003ckonishi.ryusuke@lab.ntt.co.jp\u003e\nCc: reiserfs-devel@vger.kernel.org\nCc: xfs-masters@oss.sgi.com\nCc: Alexander Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "e525fd89d380c4a94c0d63913a1dd1a593ed25e7", "tree": "d226ef40d3f99e42fcf272ad432585cbd641ebec", "parents": [ "e09b457bdb7e8d23fc54dcef0930ac697d8de895" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:17 2010 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:17 2010 +0100" }, "message": "block: make blkdev_get/put() handle exclusive access\n\nOver time, block layer has accumulated a set of APIs dealing with bdev\nopen, close, claim and release.\n\n* blkdev_get/put() are the primary open and close functions.\n\n* bd_claim/release() deal with exclusive open.\n\n* open/close_bdev_exclusive() are combination of open and claim and\n the other way around, respectively.\n\n* bd_link/unlink_disk_holder() to create and remove holder/slave\n symlinks.\n\n* open_by_devnum() wraps bdget() + blkdev_get().\n\nThe interface is a bit confusing and the decoupling of open and claim\nmakes it impossible to properly guarantee exclusive access as\nin-kernel open + claim sequence can disturb the existing exclusive\nopen even before the block layer knows the current open if for another\nexclusive access. Reorganize the interface such that,\n\n* blkdev_get() is extended to include exclusive access management.\n @holder argument is added and, if is @FMODE_EXCL specified, it will\n gain exclusive access atomically w.r.t. other exclusive accesses.\n\n* blkdev_put() is similarly extended. It now takes @mode argument and\n if @FMODE_EXCL is set, it releases an exclusive access. Also, when\n the last exclusive claim is released, the holder/slave symlinks are\n removed automatically.\n\n* bd_claim/release() and close_bdev_exclusive() are no longer\n necessary and either made static or removed.\n\n* bd_link_disk_holder() remains the same but bd_unlink_disk_holder()\n is no longer necessary and removed.\n\n* open_bdev_exclusive() becomes a simple wrapper around lookup_bdev()\n and blkdev_get(). It also has an unexpected extra bdev_read_only()\n test which probably should be moved into blkdev_get().\n\n* open_by_devnum() is modified to take @holder argument and pass it to\n blkdev_get().\n\nMost of bdev open/close operations are unified into blkdev_get/put()\nand most exclusive accesses are tested atomically at the open time (as\nit should). This cleans up code and removes some, both valid and\ninvalid, but unnecessary all the same, corner cases.\n\nopen_bdev_exclusive() and open_by_devnum() can use further cleanup -\nrename to blkdev_get_by_path() and blkdev_get_by_devt() and drop\nspecial features. Well, let\u0027s leave them for another day.\n\nMost conversions are straight-forward. drbd conversion is a bit more\ninvolved as there was some reordering, but the logic should stay the\nsame.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Neil Brown \u003cneilb@suse.de\u003e\nAcked-by: Ryusuke Konishi \u003ckonishi.ryusuke@lab.ntt.co.jp\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nAcked-by: Philipp Reisner \u003cphilipp.reisner@linbit.com\u003e\nCc: Peter Osterlund \u003cpetero2@telia.com\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nCc: Jan Kara \u003cjack@suse.cz\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nCc: Andreas Dilger \u003cadilger.kernel@dilger.ca\u003e\nCc: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: Mark Fasheh \u003cmfasheh@suse.com\u003e\nCc: Joel Becker \u003cjoel.becker@oracle.com\u003e\nCc: Alex Elder \u003caelder@sgi.com\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nCc: dm-devel@redhat.com\nCc: drbd-dev@lists.linbit.com\nCc: Leo Chen \u003cleochen@broadcom.com\u003e\nCc: Scott Branden \u003csbranden@broadcom.com\u003e\nCc: Chris Mason \u003cchris.mason@oracle.com\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nCc: Joern Engel \u003cjoern@logfs.org\u003e\nCc: reiserfs-devel@vger.kernel.org\nCc: Alexander Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "e09b457bdb7e8d23fc54dcef0930ac697d8de895", "tree": "41035bf61a8acb08292d01c0f66e5450502688af", "parents": [ "37004c42f7240035bc2726c340c4efa726b4818e" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:17 2010 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat Nov 13 11:55:17 2010 +0100" }, "message": "block: simplify holder symlink handling\n\nCode to manage symlinks in /sys/block/*/{holders|slaves} are overly\ncomplex with multiple holder considerations, redundant extra\nreferences to all involved kobjects, unused generic kobject holder\nsupport and unnecessary mixup with bd_claim/release functionalities.\n\nStrip it down to what\u0027s necessary (single gendisk holder) and make it\nuse a separate interface. This is a step for cleaning up\nbd_claim/release. This patch makes dm-table slightly more complex but\nit will be simplified again with further changes.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Neil Brown \u003cneilb@suse.de\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: dm-devel@redhat.com\n" }, { "commit": "77304d2abac6101f7249754ffdd4421258877ab0", "tree": "8ee5914529d70120528a630a8e904c22f8c4f98c", "parents": [ "90fdb0b98a62d78a0650b9fd3ddc58a48f71d740" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Nov 08 14:39:12 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Nov 10 14:40:53 2010 +0100" }, "message": "block: read i_size with i_size_read()\n\nConvert direct reads of an inode\u0027s i_size to using i_size_read().\n\ni_size_{read,write} use a seqcount to protect reads from accessing\nincomple writes. Concurrent i_size_write()s require mutual exclussion\nto protect the seqcount that is used by i_size_{read,write}. But\ni_size_read() callers do not need to use additional locking.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nAcked-by: NeilBrown \u003cneilb@suse.de\u003e\nAcked-by: Lars Ellenberg \u003clars.ellenberg@linbit.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a167f663243662aa9153c01086580a11cde9ffdc", "tree": "886e64787421bbf17a1eab7853d67258b598f050", "parents": [ "2b193363ef68667ad717a6723165e0dccf99470f" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 26 18:31:13 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:36:15 2010 +1100" }, "message": "md: use separate bio pool for each md device.\n\nbio_clone and bio_alloc allocate from a common bio pool.\nIf an md device is stacked with other devices that use this pool, or under\nsomething like swap which uses the pool, then the multiple calls on\nthe pool can cause deadlocks.\n\nSo allocate a local bio pool for each md array and use that rather\nthan the common pool.\n\nThis pool is used both for regular IO and metadata updates.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" } ], "next": "2b193363ef68667ad717a6723165e0dccf99470f" }