)]}' { "log": [ { "commit": "2c7d46ec192e4f2b350f67a0e185b9bce646cd6b", "tree": "2a49c1e99de3442184726be20d03e3a15d80cee0", "parents": [ "6b9656205469269c050963c71fca1998b247a560" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 18 16:16:05 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 18 16:16:05 2010 +1000" }, "message": "md raid-1/10 Fix bio_rw bit manipulations again\n\ncommit 7b6d91daee5cac6402186ff224c3af39d79f4a0e changed the behaviour\nof a few variables in raid1 and raid10 from flags to bit-sets, but\nleft them as type \u0027bool\u0027 so they did not work.\n\nChange them (back) to unsigned long.\n(historical note: see 1ef04fefe2241087d9db7e9615c3f11b516e36cf)\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReported-by: Jiri Slaby \u003cjslaby@suse.cz\u003e and many others\n" }, { "commit": "6b9656205469269c050963c71fca1998b247a560", "tree": "9d090d2e363d269cdc55549213025d86af8ef70a", "parents": [ "e6ffbcb6cd0ac471223df24ae77eb486c1ee68cc" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 18 11:56:59 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 18 12:04:32 2010 +1000" }, "message": "md: provide appropriate return value for spare_active functions.\n\nmd_check_recovery expects -\u003espare_active to return \u0027true\u0027 if any\nspares were activated, but none of them do, so the consequent change\nin \u0027degraded\u0027 is not notified through sysfs.\n\nSo count the number of spares activated, subtract it from \u0027degraded\u0027\njust once, and return it.\n\nReported-by: Adrian Drzewiecki \u003cadriand@vmware.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e6ffbcb6cd0ac471223df24ae77eb486c1ee68cc", "tree": "8739230c95a051c8ab95fdbfd90ec2e6ce0bf3c9", "parents": [ "3a3a5ddb7a0f43c3dd0f98673f3d930a456725f8" ], "author": { "name": "Adrian Drzewiecki", "email": "adriand@vmware.com", "time": "Wed Aug 18 11:49:02 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 18 11:49:02 2010 +1000" }, "message": "md: Notify sysfs when RAID1/5/10 disk is In_sync.\n\nWhen RAID1 is done syncing disks, it\u0027ll update the state\nof synced rdevs to In_sync. But it neglected to notify\nsysfs that the attribute changed. So any programs that\nare waiting for an rdev\u0027s state to change will not be\nwoken.\n\n(raid5/raid10 added by neilb)\n\nSigned-off-by: Adrian Drzewiecki \u003cadriand@vmware.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n\n" }, { "commit": "7b6d91daee5cac6402186ff224c3af39d79f4a0e", "tree": "b1518cf0b6c301178e0a320f80610cd5b3aa7625", "parents": [ "33659ebbae262228eef4e0fe990f393d1f0ed941" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Sat Aug 07 18:20:39 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:20:39 2010 +0200" }, "message": "block: unify flags for struct bio and struct request\n\nRemove the current bio flags and reuse the request flags for the bio, too.\nThis allows to more easily trace the type of I/O from the filesystem\ndown to the block driver. There were two flags in the bio that were\nmissing in the requests: BIO_RW_UNPLUG and BIO_RW_AHEAD. Also I\u0027ve\nrenamed two request flags that had a superflous RW in them.\n\nNote that the flags are in bio.h despite having the REQ_ name - as\nblkdev.h includes bio.h that is the only way to go for now.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "19fdb9eefb21b72edbc365b838502780c392bad6", "tree": "deae04c48532d6eab64ed4b0396737bb854b5506", "parents": [ "be6800a73aa2f3dc14744c3b80e676d189789f04", "3ff195b011d7decf501a4d55aeed312731094796" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sat May 22 08:31:36 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sat May 22 08:31:36 2010 +1000" }, "message": "Merge commit \u00273ff195b011d7decf501a4d55aeed312731094796\u0027 into for-linus\n\nConflicts:\n\tdrivers/md/md.c\n\n- Resolved conflict in md_update_sb\n- Added extra \u0027NULL\u0027 arg to new instance of sysfs_get_dirent.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "af3a2cd6b8a479345786e7fe5e199ad2f6240e56", "tree": "10690853d80455e69bcc0d027ad6c8b9c7951332", "parents": [ "2dc40f80945ac3e5ec05c3a6c75baf09b13cee51" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sat May 08 08:20:17 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:28:00 2010 +1000" }, "message": "md: Fix read balancing in RAID1 and RAID10 on drives \u003e 2TB\n\nread_balance uses a \"unsigned long\" for a sector number which\nwill get truncated beyond 2TB.\nThis will cause read-balancing to be non-optimal, and can cause\ndata to be read from the \u0027wrong\u0027 branch during a resync. This has a\nvery small chance of returning wrong data.\n\nReported-by: Jordan Russell \u003cjr-list-2010@quo.to\u003e\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "9dd1e2faf72f79a2af9dcbd059473c06648726c2", "tree": "a3b400d49671930f6d6ff0b9d5890e598af7a94f", "parents": [ "0c55e02259115c151e4835dd417cf41467bb02e2" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon May 03 14:30:35 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:59 2010 +1000" }, "message": "md/raid1: improve printk messages\n\nMake sure the array name is included in a uniform way in all printk\nmessages.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e555190d82c0f58e825e3cbd9e6ebe2e7ac713bd", "tree": "ea9c8e548c4d5eb5fc903bab05af3bda4192ed00", "parents": [ "d754c5ae1ff76b20d3ecde8ad666d7865eada8ae" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Mar 31 11:21:44 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:57 2010 +1000" }, "message": "md/raid1: delay reads that could overtake behind-writes.\n\nWhen a raid1 array is configured to support write-behind\non some devices, it normally only reads from other devices.\nIf all devices are write-behind (because the rest have failed)\nit is possible for a read request to be serviced before a\nbehind-write request, which would appear as data corruption.\n\nSo when forced to read from a WriteMostly device, wait for any\nwrite-behind to complete, and don\u0027t start any more behind-writes.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d754c5ae1ff76b20d3ecde8ad666d7865eada8ae", "tree": "60523d76c5e437ea39ce158295d75ea4247f70e1", "parents": [ "9e35b99c7efacfddc748c89a0c53b1122b0ee72c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Apr 07 12:14:43 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:56 2010 +1000" }, "message": "md/raid1: fix confusing \u0027redirect sector\u0027 message.\n\nThis message seems to suggest the named device is the one on which a\nread failed, however it is actually the device that the read will be\nredirected to.\nSo make the message a little clearer.\n\nReported-by: Tim Burgess \u003cozburgess@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n\n" }, { "commit": "21a52c6d05c15f862797736393915bfa8cd40ee9", "tree": "44ffb633d209457979177a52132761a368fcb976", "parents": [ "cca9cf90c504d98644ace52c474770970729f0eb" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Apr 01 15:02:13 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:55 2010 +1000" }, "message": "md: pass mddev to make_request functions rather than request_queue\n\nWe used to pass the personality make_request function direct\nto the block layer so the first argument had to be a queue.\nBut now we have the intermediary md_make_request so it makes\nat lot more sense to pass a struct mddev_s.\nIt makes it possible to have an mddev without its own queue too.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b821eaa572fd737faaf6928ba046e571526c36c6", "tree": "0db462a3c7da3dbf42d28e2483a27d78839b4589", "parents": [ "49ce6cea85fb8d25ee59486c919406e9cecf1762" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Mar 29 11:18:15 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:53 2010 +1000" }, "message": "md: remove -\u003echanged and related code.\n\nWe set -\u003echanged to 1 and call check_disk_change at the end\nof md_open so that bd_invalidated would be set and thus\npartition rescan would happen appropriately.\n\nNow that we call revalidate_disk directly, which sets bd_invalidates,\nthat indirection is no longer needed and can be removed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "490773268cf64f68da2470e07b52c7944da6312d", "tree": "d394aafa7203c316db6b63f128b8894e18993fca", "parents": [ "2b7f22284d71975e37a82db154386348eec0e52c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Mar 25 16:20:56 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:52 2010 +1000" }, "message": "md: move io accounting out of personalities into md_make_request\n\nWhile I generally prefer letting personalities do as much as possible,\ngiven that we have a central md_make_request anyway we may as well use\nit to simplify code.\nAlso this centralises knowledge of -\u003egendisk which will help later.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7b92813c3c0b6990f14838e3985fb385d2655d0c", "tree": "c072a6684185f2c18734e704c488953250e8353a", "parents": [ "696fcd535b5a8cfc0617e9cf1d9d69a13895cc1e" ], "author": { "name": "H Hartley Sweeten", "email": "hartleys@visionengravers.com", "time": "Mon Mar 08 16:02:40 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:46 2010 +1000" }, "message": "drivers/md: Remove unnecessary casts of void *\n\nvoid pointers do not need to be cast to other pointer types.\n\nSigned-off-by: H Hartley Sweeten \u003chsweeten@visionengravers.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "964147d5c86d63be79b442c30f3783d49860c078", "tree": "5c1eab8814cab3511c85037dec5220c5c24f91af", "parents": [ "a64c876fd357906a1f7193723866562ad290654c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:13 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 18 15:27:13 2010 +1000" }, "message": "md/raid1: fix counting of write targets.\n\nThere is a very small race window when writing to a\nRAID1 such that if a device is marked faulty at exactly the wrong\ntime, the write-in-progress will not be sent to the device,\nbut the bitmap (if present) will be updated to say that\nthe write was sent.\n\nThen if the device turned out to still be usable as was re-added\nto the array, the bitmap-based-resync would skip resyncing that\nblock, possibly leading to corruption. This would only be a problem\nif no further writes were issued to that area of the device (i.e.\nthat bitmap chunk).\n\nSuitable for any pending -stable kernel.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "627a2d3c29427637f4c5d31ccc7fcbd8d312cd71", "tree": "f0de68842ca3b47d0152254e60d31cc561009119", "parents": [ "25cf84cf377c0aae5dbcf937ea89bc7893db5176" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Mar 08 16:44:38 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 16 17:04:24 2010 +1100" }, "message": "md: deal with merge_bvec_fn in component devices better.\n\nIf a component device has a merge_bvec_fn then as we never call it\nwe must ensure we never need to. Currently this is done by setting\nmax_sector to 1 PAGE, however this does not stop a bio being created\nwith several sub-page iovecs that would violate the merge_bvec_fn.\n\nSo instead set max_segments to 1 and set the segment boundary to the\nsame as a page boundary to ensure there is only ever one single-page\nsegment of IO requested at a time.\n\nThis can particularly be an issue when \u0027xen\u0027 is used as it is\nknown to submit multiple small buffers in a single bio.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nCc: stable@kernel.org\n" }, { "commit": "086fa5ff0854c676ec333760f4c0154b3b242616", "tree": "ee63fb3c7c7d964bd799355b7cde18ba95f91f07", "parents": [ "eb28d31bc97e6374d81f404da309401ffaed467b" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Feb 26 00:20:38 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 26 13:58:08 2010 +0100" }, "message": "block: Rename blk_queue_max_sectors to blk_queue_max_hw_sectors\n\nThe block layer calling convention is blk_queue_\u003climit name\u003e.\nblk_queue_max_sectors predates this practice, leading to some confusion.\nRename the function to appropriately reflect that its intended use is to\nset max_hw_sectors.\n\nAlso introduce a temporary wrapper for backwards compability. This can\nbe removed after the merge window is closed.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "0efb9e6191e1d3d34c1db90b829b742bc36d532e", "tree": "5960d97df87a7eeea56d7dbe801259fc43398d5e", "parents": [ "1e50915fe0bbf7a46db0fa7e1e604d3fc95f057d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:49:58 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:51:41 2009 +1100" }, "message": "md: add MODULE_DESCRIPTION for all md related modules.\n\nSuggested by Oren Held \u003corenhe@il.ibm.com\u003e\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "42a04b5078ce73a32f85762551d5703c5bd646a1", "tree": "3ef384933cd33d000516c292712da9a99e273360", "parents": [ "c3d9714e88c8685cf9bc837c3241fc005f95fb82" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:49:53 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:51:41 2009 +1100" }, "message": "md: move offset, daemon_sleep and chunksize out of bitmap structure\n\n... and into bitmap_info. These are all configuration parameters\nthat need to be set before the bitmap is created.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "709ae4879ae33628ded276ce7da8cd5acfec476b", "tree": "db9714769f0b7d9b764eaf26ca51185519ae07bc", "parents": [ "6eef4b21ffc9207b78fdd718154f5b004644b97d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:49:51 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:51:41 2009 +1100" }, "message": "md/raid1: add takeover support for raid5-\u003eraid1\n\nA 2-device raid5 array can now be converted to raid1.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "6eef4b21ffc9207b78fdd718154f5b004644b97d", "tree": "554d8dafa2cd8ff7a9e719b12bd2f07c9745e7be", "parents": [ "729a18663a30a9c8076e3adc2b3e4c866974f935" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:49:51 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Dec 14 12:51:40 2009 +1100" }, "message": "md: add honouring of suspend_{lo,hi} to raid1.\n\nThis will allow us to stop writeout to portions of the array\nwhile they are resynced by someone else - e.g. another node in\na cluster.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d0e260782c3702a009645c3caa02e381dab8798b", "tree": "588f7776216df124d9372c7ced833b0bfb6d0a7f", "parents": [ "a9366e61b03f55a6e009e687ad10e706714c9907" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Dec 01 17:30:59 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Dec 01 17:30:59 2009 +1100" }, "message": "md: revert incorrect fix for read error handling in raid1.\n\ncommit 4706b349f was a forward port of a fix that was needed\nfor SLES10. But in fact it is not needed in mainline because\nthe earlier commit dd00a99e7a fixes the same problem in a\nbetter way.\nFurther, this commit introduces a bug in the way it interacts with\nthe automatic read-error-correction. If, after a read error is\nsuccessfully corrected, the same disk is chosen to re-read - the\nre-read won\u0027t be attempted but an error will be returned instead.\n\nAfter reverting that commit, there is the possibility that a\nread error on a read-only array (where read errors cannot\nbe corrected as that requires a write) will repeatedly read the same\ndevice and continue to get an error.\nSo in the \"Array is readonly\" case, fail the drive immediately on\na read error.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nCc: stable@kernel.org\n" }, { "commit": "ed9bfdf1a40952fd0f8094ec77f876b84ead69af", "tree": "2b92f23e861fa2695a55a0cd797233d032634c9d", "parents": [ "f5efd45ae597c96ed017afad5662b67d55b402a0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 16 15:55:44 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 16 15:55:44 2009 +1100" }, "message": "md: raid1/raid10: handle allocation errors during array setup.\n\nBoth raid1 and raid10 create a mempool during startup.\nIf the \u0027alloc\u0027 function for this mempool fails, unplug_slaves\nis called.\nIf that happens when the pool is being initialised, unplug_slaves\nwill try to use the \u0027conf\u0027 structure that isn\u0027t filled in yet, and\nbadness will happen.\n\nSo ensure that unplug_slaves doesn\u0027t get called unless we know\nthat the conf structure if fully initialised.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1d9d52416c0445019ccc1f0fddb9a227456eb61b", "tree": "1dc189d68babe4827e2a16d98fae6aeaa40792d7", "parents": [ "1442577bf6da1a31ae6555212202be9a2cec5642" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 16 15:55:32 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 16 15:55:32 2009 +1100" }, "message": "md/raid1/raid10: add a cond_resched\n\nDuring \u0027check\u0027 of a raid1 or raid10 it is possible for the management\nthread to spend a lot of time running \u0027memcmp\u0027 on blocks from\ndifferent devices, so make sure the thread has a chance to schedule.\nraid5d already has a cond_resched (in process_stripe).\n\nReported-By: Lee Howard \u003cfaxguy@howardsilvan.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1ef04fefe2241087d9db7e9615c3f11b516e36cf", "tree": "c050718dcdd9f61c698fe6c0247fa0f99b7c4f72", "parents": [ "f28f4e272876317626cd399288b4030b627c4b91" ], "author": { "name": "Dmitry Monakhov", "email": "rjevskiy@gmail.com", "time": "Sun Sep 20 05:52:25 2009 +0400" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 23 18:20:15 2009 +1000" }, "message": "md: raid-1/10: fix RW bits manipulation\n\nRecently Jens has changed bio_rw_flagged() logic by following\ncommit 1f98a13f623e0ef666690a18c1250335fc6d7ef1. Now it returns\nbool instead of int. This broke raid1/raid10 RW bits manipulation logic.\nOne of visible result is BUG_ON triggering due to empty barrier\nhere scsi_lib.c:1108 scsi_setup_fs_cmnd()\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3fa841d7e7266f6fcc1b3885b905f5153ba897d8", "tree": "7afa971a10eddcf52787340ea43fa1e910b5805b", "parents": [ "0da3c6194ec2f32617b272df4505a1cf022faea5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 23 18:10:29 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 23 18:10:29 2009 +1000" }, "message": "md: report device as congested when suspended\n\nThis should writeback from coming when the device is temporarily\nsuspended.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0da3c6194ec2f32617b272df4505a1cf022faea5", "tree": "5864b61b8fac2e7477668bdb9a493d2a64d0b3dd", "parents": [ "ee305acef5c7841dc25cc32e84bb94f744e1e9b9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 23 18:09:45 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Sep 23 18:09:45 2009 +1000" }, "message": "md: Improve name of threads created by md_register_thread\n\nThe management thread for raid4,5,6 arrays are all called\nmdX_raid5, independent of the actual raid level, which is wrong and\ncan be confusion.\n\nSo change md_register_thread to use the name from the personality\nunless no alternate name (like \u0027resync\u0027 or \u0027reshape\u0027) is given.\n\nThis is simpler and more correct.\n\nCc: Jinzc \u003czhenchengjin@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1f98a13f623e0ef666690a18c1250335fc6d7ef1", "tree": "15ca2dddffaa18a0d1844957f4f8cc707cbb8117", "parents": [ "e7e503aedb1f4d165081cb8d47a58c38f80f0cb4" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Sep 11 14:32:04 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Sep 11 14:33:31 2009 +0200" }, "message": "bio: first step in sanitizing the bio-\u003ebi_rw flag testing\n\nGet rid of any functions that test for these bits and make callers\nuse bio_rw_flagged() directly. Then it is at least directly apparent\nwhat variable and flag they check.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "449aad3e25358812c43afc60918c5ad3819488e7", "tree": "45fab3b82fc1ed06959537a0801319045cf4c102", "parents": [ "64bd660b51b2da92e99a5e97349f6558349f11c5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "message": "md: Use revalidate_disk to effect changes in size of device.\n\nAs revalidate_disk calls check_disk_size_change, it will cause\nany capacity change of a gendisk to be propagated to the blockdev\ninode. So use that instead of mucking about with locks and\ni_size_write.\n\nAlso add a call to revalidate_disk in do_md_run and a few other places\nwhere the gendisk capacity is changed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ac5e7113e74872928844d00085bd47c988f12728", "tree": "dabce6a99c2af9f00b33660f6b2d906cce836764", "parents": [ "95fc17aac45300f45968aacd97a536ddd8db8101" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Mon Aug 03 10:59:47 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:47 2009 +1000" }, "message": "md: Push down data integrity code to personalities.\n\nThis patch replaces md_integrity_check() by two new public functions:\nmd_integrity_register() and md_integrity_add_rdev() which are both\npersonality-independent.\n\nmd_integrity_register() is called from the -\u003erun and -\u003ehot_remove\nmethods of all personalities that support data integrity. The\nfunction iterates over the component devices of the array and\ndetermines if all active devices are integrity capable and if their\nprofiles match. If this is the case, the common profile is registered\nfor the mddev via blk_integrity_register().\n\nThe second new function, md_integrity_add_rdev() is called from the\n-\u003ehot_add_disk methods, i.e. whenever a new device is being added\nto a raid array. If the new device does not support data integrity,\nor has a profile different from the one already registered, data\nintegrity for the mddev is disabled.\n\nFor raid0 and linear, only the call to md_integrity_register() from\nthe -\u003erun method is necessary.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8f6c2e4b325a8e9f8f47febb2fd0ed4fae7d45a9", "tree": "6e383e2ec48b5c90fe07325a7f6ab38ea1a97dfa", "parents": [ "5a4f13fad1ab5bd08dea78fc55321e429d83cddf" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Jul 01 11:13:45 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 11:13:45 2009 +1000" }, "message": "md: Use new topology calls to indicate alignment and I/O sizes\n\nSwitch MD over to the new disk_stack_limits() function which checks for\naligment and adjusts preferred I/O sizes when stacking.\n\nAlso indicate preferred I/O sizes where applicable.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8c6ac868b107ed50a46204f6d14e2ad9443ff146", "tree": "a0690ce92ed1993cc3211448948f3964c389c082", "parents": [ "50ac168a6e0a061bf5346d53aa9e7beb94c97527" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Thu Jun 18 08:48:06 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:48:06 2009 +1000" }, "message": "md: Push down reconstruction log message to personality code.\n\nCurrently, the md layer checks in analyze_sbs() if the raid level\nsupports reconstruction (mddev-\u003elevel \u003e\u003d 1) and if reconstruction is\nin progress (mddev-\u003erecovery_cp !\u003d MaxSector).\n\nMove that printk into the personality code of those raid levels that\ncare (levels 1, 4, 5, 6, 10).\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "664e7c413f1e90eceb0b2596dd73a0832faec058", "tree": "72ea827a0f2a493766d3ea1fd14909c756aa4496", "parents": [ "9d8f0363623b3da12c43007cf77f5e1a4e8a5964" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Thu Jun 18 08:45:27 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:45:27 2009 +1000" }, "message": "md: Convert mddev-\u003enew_chunk to sectors.\n\nA straight-forward conversion which gets rid of some\nmultiplications/divisions/shifts. The patch also introduces a couple\nof new ones, most of which are due to conf-\u003echunk_size still being\nrepresented in bytes. This will be cleaned up in subsequent patches.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "9d8f0363623b3da12c43007cf77f5e1a4e8a5964", "tree": "0fee53971a397ade209dd36c4f1ed50db6450faf", "parents": [ "fbb704efb784e2c8418e34dc3013af76bdd58101" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Thu Jun 18 08:45:01 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:45:01 2009 +1000" }, "message": "md: Make mddev-\u003echunk_size sector-based.\n\nThis patch renames the chunk_size field to chunk_sectors with the\nimplied change of semantics. Since\n\n\tis_power_of_2(chunk_size) \u003d is_power_of_2(chunk_sectors \u003c\u003c 9)\n\t\t\t\t \u003d is_power_of_2(chunk_sectors)\n\nthese bits don\u0027t need an adjustment for the shift.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "070ec55d07157a3041f92654135c3c6e2eaaf901", "tree": "10f24d859e669ba4a671204ce4176a2b43fdaae5", "parents": [ "a6b3deafe0c50e3e873e8ed5cc8abfcb25c05eff" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 16 16:54:21 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 16 16:54:21 2009 +1000" }, "message": "md: remove mddev_to_conf \"helper\" macro\n\nHaving a macro just to cast a void* isn\u0027t really helpful.\nI would must rather see that we are simply de-referencing -\u003eprivate,\nthan have to know what the macro does.\n\nSo open code the macro everywhere and remove the pointless cast.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ae03bf639a5027d27270123f5f6e3ee6a412781d", "tree": "d705f41a188ad656b1f47f7952626a9f992e3b8f", "parents": [ "e1defc4ff0cf57aca6c5e3ff99fa503f5943c1f1" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri May 22 17:17:50 2009 -0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri May 22 23:22:54 2009 +0200" }, "message": "block: Use accessor functions for queue limits\n\nConvert all external users of queue limits to using wrapper functions\ninstead of poking the request queue variables directly.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "8f3d8ba20e67991b531e9c0227dcd1f99271a32c", "tree": "337efd46f067ce3b7eeab0c3627b02a7edef4afd", "parents": [ "0882e8dd3aad33eca41696d463bb896e6c8817eb" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Apr 07 19:55:13 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Apr 15 08:28:09 2009 +0200" }, "message": "block: move bio list helpers into bio.h\n\nIt\u0027s used by DM and MD and generally useful, so move the bio list\nhelpers into bio.h.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "91a9e99d76d71b1b16f422fae4bd844245f14638", "tree": "4b38dfbb004bb39a670bebdcb1d99ed452f2de49", "parents": [ "bf538fe4689694b0b90d358dda91cde7b438097d" ], "author": { "name": "Alexander Beregalov", "email": "a.beregalov@gmail.com", "time": "Tue Apr 07 01:35:56 2009 +0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 06 14:40:07 2009 -0700" }, "message": "md/raid1: fix build breakage\n\nFix this build error:\n\n drivers/md/raid1.c: In function \u0027raid1_congested\u0027:\n drivers/md/raid1.c:589: error: \u0027BDI_write_congested\u0027 undeclared\n\nBDI_write_congested was changed in commit 1faa16d228 (\"block: change the\nrequest allocation/congestion logic to be sync/async based\")\n\nSigned-off-by: Alexander Beregalov \u003ca.beregalov@gmail.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "303a0e11d0ee136ad8f53f747f3c377daece763b", "tree": "197f4a00ec3eba9e00cdc264c25749914667c389", "parents": [ "6bb597507f9839b13498781e481f5458aea33620" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 06 14:40:38 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 06 14:40:38 2009 +1000" }, "message": "md/raid1 - don\u0027t assume newly allocated bvecs are initialised.\n\nSince commit d3f761104b097738932afcc310fbbbbfb007ef92\nnewly allocated bvecs aren\u0027t initialised to NULL, so we have\nto be more careful about freeing a bio which only managed\nto get a few pages allocated to it. Otherwise the resync\nprocess crashes.\n\nThis patch is appropriate for 2.6.29-stable.\n\nCc: stable@kernel.org\nCc: \"Jens Axboe\" \u003cjens.axboe@oracle.com\u003e\nReported-by: Gabriele Tozzi \u003cgabriele@tozzi.eu\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b522adcde9c4d3fb7b579cfa9160d8bde7744be8", "tree": "4208e3dcec3ebdfa6ad9bc153f76129400532717", "parents": [ "1f403624bde3c678a166984b1e6a727a0ce06f2b" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 31 15:00:31 2009 +1100" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 31 15:00:31 2009 +1100" }, "message": "md: \u0027array_size\u0027 sysfs attribute\n\nAllow userspace to set the size of the array according to the following\nsemantics:\n\n1/ size must be \u003c\u003d to the size returned by mddev-\u003epers-\u003esize(mddev, 0, 0)\n a) If size is set before the array is running, do_md_run will fail\n if size is greater than the default size\n b) A reshape attempt that reduces the default size to less than the set\n array size should be blocked\n2/ once userspace sets the size the kernel will not change it\n3/ writing \u0027default\u0027 to this attribute returns control of the size to the\n kernel and reverts to the size reported by the personality\n\nAlso, convert locations that need to know the default size from directly\nreading -\u003earray_sectors to \u003cpers\u003e_size. Resync/reshape operations\nalways follow the default size.\n\nFinally, fixup other locations that read a number of 1k-blocks from\nuserspace to use strict_blocks_to_sectors() which checks for unsigned\nlong long to sector_t overflow and blocks to sectors overflow.\n\nReviewed-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "1f403624bde3c678a166984b1e6a727a0ce06f2b", "tree": "b66ed37091e825bcde1cd0d333d4ebba86772805", "parents": [ "80c3a6ce4ba4470379b9e6a4d9bcd9d2ee26ae03" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 31 14:59:03 2009 +1100" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 31 14:59:03 2009 +1100" }, "message": "md: centralize -\u003earray_sectors modifications\n\nGet personalities out of the business of directly modifying\n-\u003earray_sectors. Lays groundwork to introduce policy on when\n-\u003earray_sectors can be modified.\n\nReviewed-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "80c3a6ce4ba4470379b9e6a4d9bcd9d2ee26ae03", "tree": "de88622f05a7e681ac7bff7714e8f2d8953b66ac", "parents": [ "93ed05e2a56b5ee06c0100a169433b11c4aa5d0b" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 17 18:10:40 2009 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Mar 31 14:57:49 2009 +1100" }, "message": "md: add \u0027size\u0027 as a personality method\n\nIn preparation for giving userspace control over -\u003earray_sectors we need\nto be able to retrieve the \u0027default\u0027 size, and the \u0027anticipated\u0027 size\nwhen a reshape is requested. For personalities that do not reshape emit\na warning if anything but the default size is requested.\n\nIn the raid5 case we need to update -\u003eprevious_raid_disks to make the\nnew \u0027default\u0027 size available.\n\nReviewed-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "409c57f3801701dfee27a28103dda4831306cb20", "tree": "430c8e4ebe879b27250e061dc1a1171b12aaadf0", "parents": [ "e0cf8f045b2023b0b3f919ee93eb94345f648434" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:39:39 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:39:39 2009 +1100" }, "message": "md: enable suspend/resume of md devices.\n\nTo be able to change the \u0027level\u0027 of an md/raid array, we need to\nsuspend the device so that no requests are active - then move some\npointers around etc.\n\nThe code already keeps counts of active requests and the -\u003equiesce\nfunction can be used to wait until those counts hit zero.\nHowever the quiesce function blocks new requests once they are all\nready \u0027inside\u0027 the personality module, and that is too late if we want\nto replace the personality modules.\n\nSo make all md requests come in through a common md_make_request\nfunction that keeps track of how many requests have entered the\nmodules but may not yet be on the internal reference counts.\nAllow md_make_request to be blocked when we want to suspend the\ndevice, and make it possible to wait for all those in-transit requests\nto be added to internal lists so that -\u003equiesce can wait for them.\n\nThere is still a problem that when a request completes, we drop the\nref count inside the personality code so there is a short time between\nwhen the refcount hits zero, and when the personality code is no\nlonger being used.\nThe personality code never blocks (schedule or spinlock) between\ndropping the refcount and exiting the routine, so this should be safe\n(as put_module calls synchronize_sched() before unmapping the module\ncode).\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "58c0fed400603a802968b23ddf78f029c5a84e41", "tree": "474fcb9775bb07f39ebb7802fb9b51d69222dcbb", "parents": [ "575a80fa4f623141e9791e41879d87800fb6d862" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "message": "md: Make mddev-\u003esize sector-based.\n\nThis patch renames the \"size\" field of struct mddev_s to \"dev_sectors\"\nand stores the number of 512-byte sectors instead of the number of\n1K-blocks in it.\n\nAll users of that field, including raid levels 1,4-6,10, are adjusted\naccordingly. This simplifies the code a bit because it allows to get\nrid of a couple of divisions/multiplications by two.\n\nIn order to make checkpatch happy, some minor coding style issues\nhave also been addressed. In particular, size_store() now uses\nstrict_strtoull() instead of simple_strtoull().\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "43b2e5d86d8bdd77386226db0bc961529492c043", "tree": "601a885b1410324678cf319eee77cc54a02f2cf3", "parents": [ "bff61975b3d6c18ee31457cc5b4d73042f44915f" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "message": "md: move md_k.h from include/linux/raid/ to drivers/md/\n\nIt really is nicer to keep related code together..\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "bff61975b3d6c18ee31457cc5b4d73042f44915f", "tree": "3aff48088b35172e74f56ae54f0b53e76a0c2150", "parents": [ "92022950c6b1bb3da90b2976b20271cdfd98b8a3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:33:13 2009 +1100" }, "message": "md: move lots of #include lines out of .h files and into .c\n\nThis makes the includes more explicit, and is preparation for moving\nmd_k.h to drivers/md/md.h\n\nRemove include/raid/md.h as its only remaining use was to #include\nother files.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ef740c372dfd80e706dbf955d4e4aedda6c0c148", "tree": "8d9ef9db346ee1ba319a125c9de83cdde049510d", "parents": [ "2a40a8aed083d988df6822bb9b1b08fb7ce21e1d" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Mar 31 14:27:03 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:27:03 2009 +1100" }, "message": "md: move headers out of include/linux/raid/\n\nMove the headers with the local structures for the disciplines and\nbitmap.h into drivers/md/ so that they are more easily grepable for\nhacking and not far away. md.h is left where it is for now as there\nare some uses from the outside.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "73d5c38a9536142e062c35997b044e89166e063b", "tree": "68fd385cbdee1fa75269974ef210d53a0ae5e311", "parents": [ "78200d45cde2a79c0d0ae0407883bb264caa3c18" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 25 13:18:47 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 25 13:18:47 2009 +1100" }, "message": "md: avoid races when stopping resync.\n\nThere has been a race in raid10 and raid1 for a long time\nwhich has only recently started showing up due to a scheduler changed.\n\nWhen a sync_read request finishes, as soon as reschedule_retry\nis called, another thread can mark the resync request as having\ncompleted, so md_do_sync can finish, -\u003estop can be called, and\n-\u003econf can be freed. So using conf after reschedule_retry is not\nsafe.\n\nSimilarly, when finishing a sync_write, calling md_done_sync must be\nthe last thing we do, as it allows a chain of events which will free\nconf and other data structures.\n\nThe first of these requires action in raid10.c\nThe second requires action in raid1.c and raid10.c\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "4706b349f4a8312d31b3d0cf61fe721699356920", "tree": "86c0ee896722e45e324a97b704c0419fbb968d4a", "parents": [ "eda58a85ec3fc05855a26654d97a2b53f0e715b9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Feb 06 15:06:47 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Feb 06 15:06:47 2009 +1100" }, "message": "md: Allow read error in a single drive raid1 to be passed up.\n\nIf a raid1 only has a single working device and gets a read error, \nwe choose to simply return that error up to the filesystem (or whatever)\nrather than failing the whole array.\n\nHowever the codes doesn\u0027t quite do that. We attempt a readbalance\nwhich allocates the same drive, so we retry the read - indefinitely. \n\nInstead: If read_balance in the error case chooses the same drive that just\nfailed, treat it as a failure and don\u0027t retry.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "4044ba58dd15cb01797c4fd034f39ef4a75f7cc3", "tree": "e1bea8143538fc3eaeeb8578c2f9231e32809a25", "parents": [ "efeb53c0e57213e843b7ef3cc6ebcdea7d6186ac" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:11 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:11 2009 +1100" }, "message": "md: don\u0027t retry recovery of raid1 that fails due to error on source drive.\n\nIf a raid1 has only one working drive and it has a sector which\ngives an error on read, then an attempt to recover onto a spare will\nfail, but as the single remaining drive is not removed from the\narray, the recovery will be immediately re-attempted, resulting\nin an infinite recovery loop.\n\nSo detect this situation and don\u0027t retry recovery once an error\non the lone remaining drive is detected.\n\nAllow recovery to be retried once every time a spare is added\nin case the problem wasn\u0027t actually a media error.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "159ec1fc060ab22b157a62364045f5e98749c4d3", "tree": "1de0edfd782245b271d2898e36ae76c00e1e1b6d", "parents": [ "ccacc7d2cf03114a24ab903f710118e9e5d43273" ], "author": { "name": "Cheng Renquan", "email": "crquan@gmail.com", "time": "Fri Jan 09 08:31:08 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:08 2009 +1100" }, "message": "md: use list_for_each_entry macro directly\n\nThe rdev_for_each macro defined in \u003clinux/raid/md_k.h\u003e is identical to\nlist_for_each_entry_safe, from \u003clinux/list.h\u003e, it should be defined to\nuse list_for_each_entry_safe, instead of reinventing the wheel.\n\nBut some calls to each_entry_safe don\u0027t really need a safe version,\njust a direct list_for_each_entry is enough, this could save a temp\nvariable (tmp) in every function that used rdev_for_each.\n\nIn this patch, most rdev_for_each loops are replaced by list_for_each_entry,\ntotally save many tmp vars; and only in the other situations that will call\nlist_del to delete an entry, the safe version is used.\n\nSigned-off-by: Cheng Renquan \u003ccrquan@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "255707274ea25d486b7de060a30ba4ac50593408", "tree": "2057cd3ee600f4b5a88c743674fcf9c138d44b48", "parents": [ "4bbf3771ca40d0aaec8316d0e7476b16010288e5" ], "author": { "name": "Stephen Rothwell", "email": "sfr@canb.auug.org.au", "time": "Wed Oct 15 09:09:21 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Oct 15 21:57:05 2008 +1100" }, "message": "md: build failure due to missing delay.h\n\nToday\u0027s linux-next build (powerpc ppc64_defconfig) failed like this:\n\ndrivers/md/raid1.c: In function \u0027sync_request\u0027:\ndrivers/md/raid1.c:1759: error: implicit declaration of function \u0027msleep_interruptible\u0027\nmake[3]: *** [drivers/md/raid1.o] Error 1\nmake[3]: *** Waiting for unfinished jobs....\ndrivers/md/raid10.c: In function \u0027sync_request\u0027:\ndrivers/md/raid10.c:1749: error: implicit declaration of function \u0027msleep_interruptible\u0027\nmake[3]: *** [drivers/md/raid10.o] Error 1\ndrivers/md/md.c: In function \u0027md_do_sync\u0027:\ndrivers/md/md.c:5915: error: implicit declaration of function \u0027msleep\u0027\n\nCaused by commit 6caa3b0bbdb474647f6bdd8a958ffc46f78d8d58 (\"md: Remove\nunnecessary #includes, #defines, and function declarations\"). I added\nthe following patch.\n\nSigned-off-by: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "074a7aca7afa6f230104e8e65eba3420263714a5", "tree": "f418313e45bd55be8156c8a3e8f9a216cf63058d", "parents": [ "eddb2e26b5ee3c5da68ba4bf1921ba20e2097bff" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:56:14 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:08 2008 +0200" }, "message": "block: move stats from disk to part0\n\nMove stats related fields - stamp, in_flight, dkstats - from disk to\npart0 and unify stat handling such that...\n\n* part_stat_*() now updates part0 together if the specified partition\n is not part0. ie. part_stat_*() are now essentially all_stat_*().\n\n* {disk|all}_stat_*() are gone.\n\n* part_round_stats() is updated similary. It handles part0 stats\n automatically and disk_round_stats() is killed.\n\n* part_{inc|dec}_in_fligh() is implemented which automatically updates\n part0 stats for parts other than part0.\n\n* disk_map_sector_rcu() is updated to return part0 if no part matches.\n Combined with the above changes, this makes NULL special case\n handling in callers unnecessary.\n\n* Separate stats show code paths for disk are collapsed into part\n stats show code paths.\n\n* Rename disk_stat_lock/unlock() to part_stat_lock/unlock()\n\nWhile at it, reposition stat handling macros a bit and add missing\nparentheses around macro parameters.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "c9959059161ddd7bf4670cf47367033d6b2f79c4", "tree": "6454db55f8e34361fe472358e10e0c5cfac1e366", "parents": [ "e71bf0d0ee89e51b92776391c5634938236977d5" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:47:21 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:06 2008 +0200" }, "message": "block: fix diskstats access\n\nThere are two variants of stat functions - ones prefixed with double\nunderbars which don\u0027t care about preemption and ones without which\ndisable preemption before manipulating per-cpu counters. It\u0027s unclear\nwhether the underbarred ones assume that preemtion is disabled on\nentry as some callers don\u0027t do that.\n\nThis patch unifies diskstats access by implementing disk_stat_lock()\nand disk_stat_unlock() which take care of both RCU (for partition\naccess) and preemption (for per-cpu counter access). diskstats access\nshould always be enclosed between the two functions. As such, there\u0027s\nno need for the versions which disables preemption. They\u0027re removed\nand double underbars ones are renamed to drop the underbars. As an\nextra argument is added, there\u0027s no danger of using the old version\nunconverted.\n\ndisk_stat_lock() uses get_cpu() and returns the cpu index and all\ndiskstat functions which access per-cpu counters now has @cpu\nargument to help RT.\n\nThis change adds RCU or preemption operations at some places but also\ncollapses several preemption ops into one at others. Overall, the\nperformance difference should be negligible as all involved ops are\nvery lightweight per-cpu ones.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "960e739d9e9f1c2346d8bdc65299ee2e1ed42218", "tree": "d3b38cbfbf503338cdf08e9ad6c4d41f7684ce34", "parents": [ "5df97b91b5d7ed426034fcc84cb6e7cf682b8838" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Aug 15 10:41:18 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:03 2008 +0200" }, "message": "block: raid fixups for removal of bi_hw_segments\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "5df97b91b5d7ed426034fcc84cb6e7cf682b8838", "tree": "727b9fb778f72d2b1ff44c007fb5209bacf67f4a", "parents": [ "b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Aug 15 10:20:02 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:03 2008 +0200" }, "message": "drop vmerge accounting\n\nRemove hw_segments field from struct bio and struct request. Without virtual\nmerge accounting they have no purpose.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "f233ea5c9e0d8b95e4283bf6a3436b88f6fd3586", "tree": "8a36b621de46da23f3957c6e3ea4d653c2937544", "parents": [ "15f4a5fdf3aa07b53f6a7969664741db5882e485" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Mon Jul 21 17:05:22 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 17:05:22 2008 +1000" }, "message": "md: Make mddev-\u003earray_size sector-based.\n\nThis patch renames the array_size field of struct mddev_s to array_sectors\nand converts all instances to use units of 512 byte sectors instead of 1k\nblocks.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b5470dc5fc18a8ff6517c3bb538d1479e58ecb02", "tree": "37b0eb3a4691bdbe58dc5c6c73b2dc8d3925b332", "parents": [ "1fe797e67fb07d605b82300934d0de67068a0aca" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Jun 27 21:44:04 2008 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Jun 30 17:18:19 2008 -0700" }, "message": "md: resolve external metadata handling deadlock in md_allow_write\n\nmd_allow_write() marks the metadata dirty while holding mddev-\u003elock and then\nwaits for the write to complete. For externally managed metadata this causes a\ndeadlock as userspace needs to take the lock to communicate that the metadata\nupdate has completed.\n\nChange md_allow_write() in the \u0027external\u0027 case to start the \u0027mark active\u0027\noperation and then return -EAGAIN. The expected side effects while waiting for\nuserspace to write \u0027active\u0027 to \u0027array_state\u0027 are holding off reshape (code\ncurrently handles -ENOMEM), cause some \u0027stripe_cache_size\u0027 change requests to\nfail, cause some GET_BITMAP_FILE ioctl requests to fall back to GFP_NOIO, and\ncause updates to \u0027raid_disks\u0027 to fail. Except for \u0027stripe_cache_size\u0027 changes\nthese failures can be mitigated by coordinating with mdmon.\n\nmd_write_start() still prevents writes from occurring until the metadata\nhandler has had a chance to take action as it unconditionally waits for\nMD_CHANGE_CLEAN to be cleared.\n\n[neilb@suse.de: return -EAGAIN, try GFP_NOIO]\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "199050ea1ff2270174ee525b73bc4c3323098897", "tree": "b2a7851511b2ddb943b9ff7ce7c8a96c3ac2a8bb", "parents": [ "6c2fce2ef6b4821c21b5c42c7207cb9cf8c87eda" ], "author": { "name": "Neil Brown", "email": "neilb@notabene.brown", "time": "Sat Jun 28 08:31:33 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@notabene.brown", "time": "Sat Jun 28 08:31:33 2008 +1000" }, "message": "rationalise return value for -\u003ehot_add_disk method.\n\nFor all array types but linear, -\u003ehot_add_disk returns 1 on\nsuccess, 0 on failure.\nFor linear, it returns 0 on success and -errno on failure.\n\nThis doesn\u0027t cause a functional problem because the -\u003ehot_add_disk\nfunction of linear is used quite differently to the others.\nHowever it is confusing.\n\nSo convert all to return 0 for success or -errno on failure\nand fix call sites to match.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "6c2fce2ef6b4821c21b5c42c7207cb9cf8c87eda", "tree": "726b16f46c039df387f7cdfe0d195821d8955532", "parents": [ "8ed0a5216a0238f53b482ec88ce4aeed4b9f0da1" ], "author": { "name": "Neil Brown", "email": "neilb@notabene.brown", "time": "Sat Jun 28 08:31:31 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@notabene.brown", "time": "Sat Jun 28 08:31:31 2008 +1000" }, "message": "Support adding a spare to a live md array with external metadata.\n\ni.e. extend the \u0027md/dev-XXX/slot\u0027 attribute so that you can\ntell a device to fill an vacant slot in an and md array.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "dfc7064500061677720fa26352963c772d3ebe6b", "tree": "a8ca495bccf98837c6762ffba54a8009c9772259", "parents": [ "90b08710e41a07d4ff0fb8940dcce3a552991a56" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri May 23 13:04:39 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat May 24 09:56:10 2008 -0700" }, "message": "md: restart recovery cleanly after device failure.\n\nWhen we get any IO error during a recovery (rebuilding a spare), we abort\nthe recovery and restart it.\n\nFor RAID6 (and multi-drive RAID1) it may not be best to restart at the\nbeginning: when multiple failures can be tolerated, the recovery may be\nable to continue and re-doing all that has already been done doesn\u0027t make\nsense.\n\nWe already have the infrastructure to record where a recovery is up to\nand restart from there, but it is not being used properly.\nThis is because:\n - We sometimes abort with MD_RECOVERY_ERR rather than just MD_RECOVERY_INTR,\n which causes the recovery not be be checkpointed.\n - We remove spares and then re-added them which loses important state\n information.\n\nThe distinction between MD_RECOVERY_ERR and MD_RECOVERY_INTR really isn\u0027t\nneeded. If there is an error, the relevant drive will be marked as\nFaulty, and that is enough to ensure correct handling of the error. So we\nfirst remove MD_RECOVERY_ERR, changing some of the uses of it to\nMD_RECOVERY_INTR.\n\nThen we cause the attempt to remove a non-faulty device from an array to\nfail (unless recovery is impossible as the array is too degraded). Then\nwhen remove_and_add_spares attempts to remove the devices on which\nrecovery can continue, it will fail, they will remain in place, and\nrecovery will continue on them as desired.\n\nIssue: If we are halfway through rebuilding a spare and another drive\nfails, and a new spare is immediately available, do we want to:\n 1/ complete the current rebuild, then go back and rebuild the new spare or\n 2/ restart the rebuild from the start and rebuild both devices in\n parallel.\n\nBoth options can be argued for. The code currently takes option 2 as\n a/ this requires least code change\n b/ this results in a minimally-degraded array in minimal time.\n\nCc: \"Eivind Sarto\" \u003civan@kasenna.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "698b18c1e8bddf39cbf1ba50792b0fe302dbe6d6", "tree": "9a65cf296b2b37654e30a1f2f9f6f13468259345", "parents": [ "6be9d4940134b36f9ed020aead36f831f19b49f1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri May 23 13:04:35 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat May 24 09:56:10 2008 -0700" }, "message": "md: raid1: Fix restoration of bio between failed read and write.\n\nWhen performing a \"recovery\" or \"check\" pass on a RAID1 array, we read\nfrom each device and possible, if there is a difference or a read error,\nwrite back to some devices.\n\nWe use the same \u0027bio\u0027 for both read and write, resetting various fields\nbetween the two operations.\n\nWe forgot to reset bv_offset and bv_len however. These are often left\nunchanged, but in the case where there is an IO error one or two sectors\ninto a page, they are changed.\n\nThis results in correctable errors not being corrected properly. It does\nnot result in any data corruption.\n\nCc: \"Fairbanks, David\" \u003cDavid.Fairbanks@stratus.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "84255d1018c50e72c71a49f359989597d53a3f53", "tree": "242c639d1e110f241e802e6b0a27a670deb8b174", "parents": [ "80119ef5c8153e0a6cc5edf00c083dc98a9bd348" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri May 23 13:04:32 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat May 24 09:56:09 2008 -0700" }, "message": "md: fix possible oops when removing a bitmap from an active array\n\nIt is possible to add a write-intent bitmap to an active array, or remove\nthe bitmap that is there.\n\nWhen we do with the \u0027quiesce\u0027 the array, which causes make_request to\nblock in \"wait_barrier()\".\n\nHowever we are sampling the value of \"mddev-\u003ebitmap\" before the\nwait_barrier call, and using it afterwards. This can result in using a\nbitmap structure that has been freed.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e7e72bf641b1fc7b9df6f40bd2c36dfccd8d647c", "tree": "81b1db5434c9635bf23fb40415056e10390cd692", "parents": [ "4920916f728fe3c51f54c25ab7b3d271254aab5a" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Wed May 14 16:05:54 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed May 14 19:11:15 2008 -0700" }, "message": "Remove blkdev warning triggered by using md\n\nAs setting and clearing queue flags now requires that we hold a spinlock\non the queue, and as blk_queue_stack_limits is called without that lock,\nget the lock inside blk_queue_stack_limits.\n\nFor blk_queue_stack_limits to be able to find the right lock, each md\npersonality needs to set q-\u003equeue_lock to point to the appropriate lock.\nThose personalities which didn\u0027t previously use a spin_lock, us\nq-\u003e__queue_lock. So always initialise that lock when allocated.\n\nWith this in place, setting/clearing of the QUEUE_FLAG_PLUGGED bit will no\nlonger cause warnings as it will be clear that the proper lock is held.\n\nThanks to Dan Williams for review and fixing the silly bugs.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nCc: Dan Williams \u003cdan.j.williams@intel.com\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nCc: Alistair John Strachan \u003calistair@devzero.co.uk\u003e\nCc: Nick Piggin \u003cnpiggin@suse.de\u003e\nCc: \"Rafael J. Wysocki\" \u003crjw@sisk.pl\u003e\nCc: Jacek Luczak \u003cdifrost.kernel@gmail.com\u003e\nCc: Prakash Punnoor \u003cprakash@punnoor.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6bfe0b499082fd3950429017cd8ebf2a6c458aa5", "tree": "81476cf7f7ddbea135bdb93729e0bffae0e7c163", "parents": [ "11e2ede0228ee0f81ccacd15894908c3bf241f73" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Apr 30 00:52:32 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Apr 30 08:29:33 2008 -0700" }, "message": "md: support blocking writes to an array on device failure\n\nAllows a userspace metadata handler to take action upon detecting a device\nfailure.\n\nBased on an original patch by Neil Brown.\n\nChanges:\n-added blocked_wait waitqueue to rdev\n-don\u0027t qualify Blocked with Faulty always let userspace block writes\n-added md_wait_for_blocked_rdev to wait for the block device to be clear, if\n userspace misses the notification another one is sent every 5 seconds\n-set MD_RECOVERY_NEEDED after clearing \"blocked\"\n-kill DoBlock flag, just test mddev-\u003eexternal\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d7a420c9472a95c46600a0345434b7b166e0b9c7", "tree": "b95005ce8ab92efdb5cd04b3d85c3b01a0455ebc", "parents": [ "4ef197d87ad7d4bb326de3e9b8ecbb26f9e86253" ], "author": { "name": "Nick Andrew", "email": "nick@nick-andrew.net", "time": "Mon Apr 28 02:15:55 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 28 08:58:42 2008 -0700" }, "message": "raid: remove leading TAB on printk messages\n\nMD drivers use one printk() call to print 2 log messages and the second line\nmay be prefixed by a TAB character. It may also output a trailing space\nbefore newline. klogd (I think) turns the TAB character into the 2 characters\n\u0027^I\u0027 when logging to a file. This looks ugly.\n\nInstead of a leading TAB to indicate continuation, prefix both output lines\nwith \u0027raid:\u0027 or similar. Also remove any trailing space in the vicinity of\nthe affected code and consistently end the sentences with a period.\n\nSigned-off-by: Nick Andrew \u003cnick@nick-andrew.net\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "1c830532f6b44d10a1743ccd00e990c6b83396f5", "tree": "506c8fef4d84f54e76b533756cff86c45bb61746", "parents": [ "8ed3a19563b6c05b7625649b1769ddb063d53253" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 04 14:29:35 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue Mar 04 16:35:18 2008 -0800" }, "message": "md: fix possible raid1/raid10 deadlock on read error during resync\n\nThanks to K.Tanaka and the scsi fault injection framework, here is a fix for\nanother possible deadlock in raid1/raid10 error handing.\n\nIf a read request returns an error while a resync is happening and a resync\nrequest is pending, the attempt to fix the error will block until the resync\nprogresses, and the resync will block until the read request completes. Thus\na deadlock.\n\nThis patch fixes the problem.\n\nCc: \"K.Tanaka\" \u003ck-tanaka@ce.jp.nec.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a35e63efa1fb18c6f20f38e3ddf3f8ffbcf0f6e7", "tree": "8dddd54c45ebaad84a6178765d29d9536df944d1", "parents": [ "466634488e80968f12e73dd1fe6af5c37a1fbfe2" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 04 14:29:29 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue Mar 04 16:35:17 2008 -0800" }, "message": "md: fix deadlock in md/raid1 and md/raid10 when handling a read error\n\nWhen handling a read error, we freeze the array to stop any other IO while\nattempting to over-write with correct data.\n\nThis is done in the raid1d(raid10d) thread and must wait for all submitted IO\nto complete (except for requests that failed and are sitting in the retry\nqueue - these are counted in -\u003enr_queue and will stay there during a freeze).\n\nHowever write requests need attention from raid1d as bitmap updates might be\nrequired. This can cause a deadlock as raid1 is waiting for requests to\nfinish that themselves need attention from raid1d.\n\nSo we create a new function \u0027flush_pending_writes\u0027 to give that attention, and\ncall it in freeze_array to be sure that we aren\u0027t waiting on raid1d.\n\nThanks to \"K.Tanaka\" \u003ck-tanaka@ce.jp.nec.com\u003e for finding and reporting this\nproblem.\n\nCc: \"K.Tanaka\" \u003ck-tanaka@ce.jp.nec.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d089c6af10c2be5988f03667d6d22fe6085fbe5e", "tree": "815eb0596bd7c908f30c268da6b5c5eff0e0bc21", "parents": [ "29ac4aa3fc68a86279aca50f20df4d614de2e204" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 06 01:39:59 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Feb 06 10:41:19 2008 -0800" }, "message": "md: change ITERATE_RDEV to rdev_for_each\n\nAs this is more in line with common practice in the kernel. Also swap the\nargs around to be more like list_for_each.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c620727779f7cc8ea96efb71f0651a26349e59c1", "tree": "777abdad9c9ef10cb4df5c0efc736e6c64851ed8", "parents": [ "c303da6d713b87b7b3f999f5acce8ecc76ff1adb" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 06 01:39:52 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Feb 06 10:41:18 2008 -0800" }, "message": "md: allow a maximum extent to be set for resyncing\n\nThis allows userspace to control resync/reshape progress and synchronise it\nwith other activities, such as shared access in a SAN, or backing up critical\nsections during a tricky reshape.\n\nWriting a number of sectors (which must be a multiple of the chunk size if\nsuch is meaningful) causes a resync to pause when it gets to that point.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b47490c9bc73d0b34e4c194db40de183e592e446", "tree": "8334581aff7aa0b199df04c6f9d707c1902ad14f", "parents": [ "66c811e99322767bad5db4368de93aac604f02a2" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Feb 06 01:39:50 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Feb 06 10:41:18 2008 -0800" }, "message": "md: Update md bitmap during resync.\n\nCurrently an md array with a write-intent bitmap does not updated that bitmap\nto reflect successful partial resync. Rather the entire bitmap is updated\nwhen the resync completes.\n\nThis is because there is no guarentee that resync requests will complete in\norder, and tracking each request individually is unnecessarily burdensome.\n\nHowever there is value in regularly updating the bitmap, so add code to\nperiodically pause while all pending sync requests complete, then update the\nbitmap. Doing this only every few seconds (the same as the bitmap update\ntime) does not notciably affect resync performance.\n\n[snitzer@gmail.com: export bitmap_cond_end_sync]\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nCc: \"Mike Snitzer\" \u003csnitzer@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "2ad8b1ef11c98c5603580878aebf9f1bc74129e4", "tree": "f7bdc2484513f6ffd174b1385bb216dcf97d2c78", "parents": [ "d85532ed284e63b5c56eaf2418f262822af60be4" ], "author": { "name": "Alan D. Brunelle", "email": "Alan.Brunelle@hp.com", "time": "Wed Nov 07 14:26:56 2007 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Nov 09 13:41:32 2007 +0100" }, "message": "Add UNPLUG traces to all appropriate places\n\nAdded blk_unplug interface, allowing all invocations of unplugs to result\nin a generated blktrace UNPLUG.\n\nSigned-off-by: Alan D. Brunelle \u003cAlan.Brunelle@hp.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "96de0e252cedffad61b3cb5e05662c591898e69a", "tree": "e3eb7d3e65ec27d39e1da13a17f6f0f91b28f5e9", "parents": [ "3f5b98a2a0cba3351f96fcaa6d79aa1a0d93ee78" ], "author": { "name": "Jan Engelhardt", "email": "jengelh@gmx.de", "time": "Fri Oct 19 23:21:04 2007 +0200" }, "committer": { "name": "Adrian Bunk", "email": "bunk@kernel.org", "time": "Fri Oct 19 23:21:04 2007 +0200" }, "message": "Convert files to UTF-8 and some cleanups\n\n* Convert files to UTF-8.\n\n * Also correct some people\u0027s names\n (one example is Eißfeldt, which was found in a source file.\n Given that the author used an ß at all in a source file\n indicates that the real name has in fact a \u0027ß\u0027 and not an \u0027ss\u0027,\n which is commonly used as a substitute for \u0027ß\u0027 when limited to\n 7bit.)\n\n * Correct town names (Goettingen -\u003e Göttingen)\n\n * Update Eberhard Mönkeberg\u0027s address (http://lkml.org/lkml/2007/1/8/313)\n\nSigned-off-by: Jan Engelhardt \u003cjengelh@gmx.de\u003e\nSigned-off-by: Adrian Bunk \u003cbunk@kernel.org\u003e\n" }, { "commit": "cf7a44168d743ca97863b7f34ed42a45fd7b5ab0", "tree": "884f5094a46be4da9f30292c9a5a417f100d5d5f", "parents": [ "d7f3d291a0e1330f341fdf1128b2d12fff7932ee" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 16 23:30:55 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Oct 17 08:43:03 2007 -0700" }, "message": "md: make sure read errors are auto-corrected during a \u0027check\u0027 resync in raid1\n\nWhenever a read error is found, we should attempt to overwrite with correct\ndata to \u0027fix\u0027 it.\n\nHowever when do a \u0027check\u0027 pass (which compares data blocks that are\nsuccessfully read, but doesn\u0027t normally overwrite) we don\u0027t do that. We\nshould.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "fd5d806266935179deda1502101624832eacd01f", "tree": "b0d01c2ed7d754f2a6daef6c77d108a81d8a470e", "parents": [ "bf2de6f5a4faf0197268f18d08969b003b87b6e8" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Oct 16 11:05:02 2007 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Oct 16 11:05:02 2007 +0200" }, "message": "block: convert blkdev_issue_flush() to use empty barriers\n\nThen we can get rid of -\u003eissue_flush_fn() and all the driver private\nimplementations of that.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "6712ecf8f648118c3363c142196418f89a510b90", "tree": "347d39a7d5a7ed96d3b1afecd28de2a0f98b98c9", "parents": [ "5bb23a688b2de23d7765a1dd439d89c038378978" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Sep 27 12:47:43 2007 +0200" }, "committer": { "name": "Jens Axboe", "email": "axboe@carl.home.kernel.dk", "time": "Wed Oct 10 09:25:57 2007 +0200" }, "message": "Drop \u0027size\u0027 argument from bio_endio and bi_end_io\n\nAs bi_end_io is only called once when the reqeust is complete,\nthe \u0027size\u0027 argument is now redundant. Remove it.\n\nNow there is no need for bio_endio to subtract the size completed\nfrom bi_size. So don\u0027t do that either.\n\nWhile we are at it, change bi_end_io to return void.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "a88aa7865bc1d1d451af2d585ac0119164ce8e00", "tree": "2ecde2e24540c611f8930bbee94e1e428038f3a1", "parents": [ "918f02383fb9ff5dba29709f3199189eeac55021" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 22 14:01:53 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Aug 22 19:52:46 2007 -0700" }, "message": "md: correctly update sysfs when a raid1 is reshaped\n\nWhen a raid1 array is reshaped (number of drives changed), the list of devices\nis compacted, so that slots for missing devices are filled with working\ndevices from later slots. This requires the \"rd%d\" symlinks in sysfs to be\nupdated.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "918f02383fb9ff5dba29709f3199189eeac55021", "tree": "26a3b5edb812ba02ecf8020d6734d8d6a2f8080c", "parents": [ "59d9445e851976d973a5a4009f80a3d55959d231" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Aug 22 14:01:52 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Aug 22 19:52:46 2007 -0700" }, "message": "md: make sure a re-add after a restart honours bitmap when resyncing\n\nCommit 1757128438d41670ded8bc3bc735325cc07dc8f9 was slightly bad. If an array\nhas a write-intent bitmap, and you remove a drive, then readd it, only the\nchanged parts should be resynced. However after the above commit, this only\nworks if the array has not been shut down and restarted.\n\nThis is because it sets \u0027fullsync\u0027 at little more often than it should. This\npatch is more careful.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "165125e1e480f9510a5ffcfbfee4e3ee38c05f23", "tree": "8009c8a5ff09e26dc2418d42f66ecafb055c52a2", "parents": [ "f695baf2df9e0413d3521661070103711545207a" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jul 24 09:28:11 2007 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jul 24 09:28:11 2007 +0200" }, "message": "[BLOCK] Get rid of request_queue_t typedef\n\nSome of the code has been gradually transitioned to using the proper\nstruct request_queue, but there\u0027s lots left. So do a full sweet of\nthe kernel and get rid of this typedef and replace its uses with\nthe proper type.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "4ad1366376bfef32ec0ffa12d1faa483d6f330bd", "tree": "87a9c864ec5ebd446f6c99f3298bc7638efa8f1e", "parents": [ "f0d76d70bc77b9b11256a3a23e98e80878be1578" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jul 17 04:06:13 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue Jul 17 10:23:15 2007 -0700" }, "message": "md: change bitmap_unplug and others to void functions\n\nbitmap_unplug only ever returns 0, so it may as well be void. Two callers try\nto print a message if it returns non-zero, but that message is already printed\nby bitmap_file_kick.\n\nwrite_page returns an error which is not consistently checked. It always\ncauses BITMAP_WRITE_ERROR to be set on an error, and that can more\nconveniently be checked.\n\nWhen the return of write_page is checked, an error causes bitmap_file_kick to\nbe called - so move that call into write_page - and protect against recursive\ncalls into bitmap_file_kick.\n\nbitmap_update_sb returns an error that is never checked.\n\nSo make these \u0027void\u0027 and be consistent about checking the bit.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "ed45666271f4fafa95b9d8ad44050e9a9bd2376e", "tree": "6f626a05e7d78029252768c2ac5f9b1d5309319b", "parents": [ "af03b8e4e81c3789e597632268940edd11ffe870" ], "author": { "name": "Mike Accetta", "email": "maccetta@laurelnetworks.com", "time": "Sat Jun 16 10:16:07 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Sat Jun 16 13:16:15 2007 -0700" }, "message": "md: fix bug in error handling during raid1 repair\n\nIf raid1/repair (which reads all block and fixes any differences it finds)\nhits a read error, it doesn\u0027t reset the bio for writing before writing\ncorrect data back, so the read error isn\u0027t fixed, and the device probably\ngets a zero-length write which it might complain about.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "dd00a99e7a4b739bd41ef4093760efc7e447f963", "tree": "2bb5bfafc0de89bd00ef530540e91e8297fc5d57", "parents": [ "c5ddb547e899993be56dc7d0bf72bfd7a8d4ae1e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 10 03:15:50 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu May 10 09:26:53 2007 -0700" }, "message": "md: avoid a possibility that a read error can wrongly propagate through md/raid1 to a filesystem.\n\nWhen a raid1 has only one working drive, we want read error to propagate up\nto the filesystem as there is no point failing the last drive in an array.\n\nCurrently the code perform this check is racy. If a write and a read a\nboth submitted to a device on a 2-drive raid1, and the write fails followed\nby the read failing, the read will see that there is only one working drive\nand will pass the failure up, even though the one working drive is actually\nthe *other* one.\n\nSo, tighten up the locking.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "44ce6294d07555c3d313757105fd44b78208407f", "tree": "29659bee16a4d7111124c6f41289bc9c11669e70", "parents": [ "497f050c42e46a4b1f6a9bcd8827fa5d97fe1feb" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed May 09 18:51:36 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed May 09 18:51:36 2007 -0700" }, "message": "Revert \"md: improve partition detection in md array\"\n\nThis reverts commit 5b479c91da90eef605f851508744bfe8269591a0.\n\nQuoth Neil Brown:\n\n \"It causes an oops when auto-detecting raid arrays, and it doesn\u0027t\n seem easy to fix.\n\n The array may not be \u0027open\u0027 when do_md_run is called, so\n bdev-\u003ebd_disk might be NULL, so bd_set_size can oops.\n\n This whole approach of opening an md device before it has been\n assembled just seems to get more and more painful. I think I\u0027m going\n to have to come up with something clever to provide both backward\n comparability with usage expectation, and sane integration into the\n rest of the kernel.\"\n\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5b479c91da90eef605f851508744bfe8269591a0", "tree": "d7c2844926d96d794d66b974b300eba2d047465e", "parents": [ "08a02ecd28bad35a47357e611044dcbeab06e3d7" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 09 02:35:39 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed May 09 12:30:57 2007 -0700" }, "message": "md: improve partition detection in md array\n\nmd currently uses -\u003emedia_changed to make sure rescan_partitions\nis call on md array after they are assembled.\n\nHowever that doesn\u0027t happen until the array is opened, which is later\nthan some people would like.\n\nSo use blkdev_ioctl to do the rescan immediately that the\narray has been assembled.\n\nThis means we can remove all the -\u003echange infrastructure as it was only used\nto trigger a partition rescan.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "2a2275d630b982e5f90206f9bc497f6695a3ec5d", "tree": "88cddee709b2107b74e5424810d4ffb6e3772382", "parents": [ "a0ad13ef643a5829d63c456ab6143bbda60b44a9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 26 00:57:11 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Jan 26 13:51:00 2007 -0800" }, "message": "[PATCH] md: fix potential memalloc deadlock in md\n\nIf a GFP_KERNEL allocation is attempted in md while the mddev_lock is held,\nit is possible for a deadlock to eventuate.\n\nThis happens if the array was marked \u0027clean\u0027, and the memalloc triggers a\nwrite-out to the md device.\n\nFor the writeout to succeed, the array must be marked \u0027dirty\u0027, and that\nrequires getting the mddev_lock.\n\nSo, before attempting a GFP_KERNEL allocation while holding the lock, make\nsure the array is marked \u0027dirty\u0027 (unless it is currently read-only).\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3eda22d19b76b15ef3420b251bd47a0ba0127589", "tree": "56e33efead2809440328da65174b44eb0a8b25c6", "parents": [ "524418bb8ecd3dfd2975bc0aa3c2cc7e8e081f24" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 26 00:57:01 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Jan 26 13:50:59 2007 -0800" }, "message": "[PATCH] md: make \u0027repair\u0027 actually work for raid1\n\nWhen \u0027repair\u0027 finds a block that is different one the various parts of the\nmirror. it is meant to write a chosen good version to the others. However it\ncurrently writes out the original data to each. The memcpy to make all the\ndata the same is missing.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e3881a6816b45668df60a426e5c3431ece1539a7", "tree": "617032a76e25e7167b95ebe4c4a6c90ee301c4c6", "parents": [ "664c0d3d575946bce24ecf5b7f93ee9541e4caf3" ], "author": { "name": "Lars Ellenberg", "email": "Lars.Ellenberg@linbit.com", "time": "Wed Jan 10 23:15:37 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Thu Jan 11 18:18:21 2007 -0800" }, "message": "[PATCH] md: pass down BIO_RW_SYNC in raid{1,10}\n\nmd raidX make_request functions strip off the BIO_RW_SYNC flag, thus\nintroducing additional latency.\n\nFixing this in raid1 and raid10 seems to be straightforward enough.\n\nFor our particular usage case in DRBD, passing this flag improved some\ninitialization time from ~5 minutes to ~5 seconds.\n\nAcked-by: NeilBrown \u003cneilb@suse.de\u003e\nSigned-off-by: Lars Ellenberg \u003clars@linbit.com\u003e\nAcked-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "802ba064c49f655d20fed563f2a4924c8256ea10", "tree": "f705b597b2da6583239e9f028fa0d6f6ec319682", "parents": [ "011e3a9ad4891ba1b27e281e085228189c56b137" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Dec 13 00:34:13 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Wed Dec 13 09:05:48 2006 -0800" }, "message": "[PATCH] md: Don\u0027t assume that READ\u003d\u003d0 and WRITE\u003d\u003d1 - use the names explicitly\n\nThanks Jens for alerting me to this.\n\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nCc: \u003craziebe@gmail.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "1757128438d41670ded8bc3bc735325cc07dc8f9", "tree": "e85679cbe949e337616ac53ab3b3fd1a3fa14a63", "parents": [ "c2b00852fbae4f8c45c2651530ded3bd01bde814" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sun Dec 10 02:20:52 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Sun Dec 10 09:57:21 2006 -0800" }, "message": "[PATCH] md: assorted md and raid1 one-liners\n\nFix few bugs that meant that:\n - superblocks weren\u0027t alway written at exactly the right time (this\n could show up if the array was not written to - writting to the array\n causes lots of superblock updates and so hides these errors).\n\n - restarting device recovery after a clean shutdown (version-1 metadata\n only) didn\u0027t work as intended (or at all).\n\n1/ Ensure superblock is updated when a new device is added.\n2/ Remove an inappropriate test on MD_RECOVERY_SYNC in md_do_sync.\n The body of this if takes one of two branches depending on whether\n MD_RECOVERY_SYNC is set, so testing it in the clause of the if\n is wrong.\n3/ Flag superblock for updating after a resync/recovery finishes.\n4/ If we find the neeed to restart a recovery in the middle (version-1\n metadata only) make sure a full recovery (not just as guided by\n bitmaps) does get done.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "969b755aadf7bcf3df5991a127a103acd0145a52", "tree": "d2c2db7b98e1e760c47e508f879619cd6b71f373", "parents": [ "750a8f3e8f64654a584e54038c2c8db380813c79" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Sat Oct 28 10:38:32 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Oct 28 11:30:52 2006 -0700" }, "message": "[PATCH] md: fix printk format warnings, seen on powerpc64:\n\ndrivers/md/raid1.c:1479: warning: long long unsigned int format, long unsigned int arg (arg 4)\ndrivers/md/raid10.c:1475: warning: long long unsigned int format, long unsigned int arg (arg 4)\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "0d12922823408b26f83b15cae4a4feff4bd22f28", "tree": "a3509f442e6fc75ca9304757ffa3dce4918ef55e", "parents": [ "26be34dc3a46be983352dd89683db374b0cb73fa" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 03 01:15:54 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Oct 03 08:04:18 2006 -0700" }, "message": "[PATCH] md: define -\u003econgested_fn for raid1, raid10, and multipath\n\nraid1, raid10 and multipath don\u0027t report their \u0027congested\u0027 status through\nbdi_*_congested, but should.\n\nThis patch adds the appropriate functions which just check the \u0027congested\u0027\nstatus of all active members (with appropriate locking).\n\nraid1 read_balance should be modified to prefer devices where\nbdi_read_congested returns false. Then we could use the \u0027\u0026\u0027 branch rather\nthan the \u0027|\u0027 branch. However that should would need some benchmarking first\nto make sure it is actually a good idea.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "c04be0aa82ff535e3676ab3e573957bdeef41879", "tree": "8481ab270218bed246eb02b3ba04a4b84d4e90d0", "parents": [ "11ce99e625fe2718ad2682bfdd99070b337e6252" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 03 01:15:53 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Oct 03 08:04:18 2006 -0700" }, "message": "[PATCH] md: Improve locking around error handling\n\nThe error handling routines don\u0027t use proper locking, and so two concurrent\nerrors could trigger a problem.\n\nSo:\n - use test-and-set and test-and-clear to synchonise\n the In_sync bits with the -\u003edegraded count\n - use the spinlock to protect updates to the\n degraded count (could use an atomic_t but that\n would be a bigger change in code, and isn\u0027t\n really justified)\n - remove un-necessary locking in raid5\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "11ce99e625fe2718ad2682bfdd99070b337e6252", "tree": "48e40104a058f27e9cd717c42edcf6738fef2372", "parents": [ "867868fb557eb2a2120b80779468fa372584ddb0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 03 01:15:52 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Oct 03 08:04:17 2006 -0700" }, "message": "[PATCH] md: Remove working_disks from raid1 state data\n\nIt is equivalent to conf-\u003eraid_disks - conf-\u003emddev-\u003edegraded.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "867868fb557eb2a2120b80779468fa372584ddb0", "tree": "feb35d5c59cf4d22a14ee04d285c132429e88b06", "parents": [ "2d2063ceae73660d5142f4754d50a75b655fd1f9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 03 01:15:51 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Oct 03 08:04:17 2006 -0700" }, "message": "[PATCH] md: Factor out part of raid1d into a separate function\n\nraid1d has toooo many nested block, so take the fix_read_error functionality\nout into a separate function.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "850b2b420cd5b363ed4cf48a8816d656c8b5251b", "tree": "b63aa37b3fb7dfb7a0b674d5a2ebe9da13cc7405", "parents": [ "6814d5368d68341ec6b5e4ecd10ea5947130775a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 03 01:15:46 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Tue Oct 03 08:04:17 2006 -0700" }, "message": "[PATCH] md: replace magic numbers in sb_dirty with well defined bit flags\n\nInstead of magic numbers (0,1,2,3) in sb_dirty, we have\nsome flags instead:\nMD_CHANGE_DEVS\n Some device state has changed requiring superblock update\n on all devices.\nMD_CHANGE_CLEAN\n The array has transitions from \u0027clean\u0027 to \u0027dirty\u0027 or back,\n requiring a superblock update on active devices, but possibly\n not on spares\nMD_CHANGE_PENDING\n A superblock update is underway.\n\nWe wait for an update to complete by waiting for all flags to be clear. A\nflag can be set at any time, even during an update, without risk that the\nchange will be lost.\n\nStop exporting md_update_sb - isn\u0027t needed.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "ddac7c7e3a0fe9cfdcef0de24476b8d69f8cf3e7", "tree": "b75845fa2424284342869566b3ed5233abecde92", "parents": [ "df9ecaba3f152d1ea79f2a5e0b87505e03f47590" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 31 21:27:36 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Sep 01 11:39:08 2006 -0700" }, "message": "[PATCH] md: Fix issues with referencing rdev in md/raid1\n\nWe need to be careful when referencing mirrors[i].rdev. It can disappear\nunder us at various times.\n\nSo:\n fix a couple of problem places.\n comment a couple of non-problem places\n move an \u0027atomic_add\u0027 which deferences rdev down a little\n way to some where where it is sure to not be NULL.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "6394cca54894f6a9bcf927ab78d28985944298ff", "tree": "8b8376691f329c3bcc491fd2d0425c4e6278c5e3", "parents": [ "84692195969b83f0ba57dc33ecf73e6c124dd186" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sun Aug 27 01:23:50 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Aug 27 11:01:31 2006 -0700" }, "message": "[PATCH] md: fix recent breakage of md/raid1 array checking\n\nA recent patch broke the ability to do a user-request check of a raid1.\nThis patch fixes the breakage and also moves a comment that was dislocated\nby the same patch.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "d69504325978c461b51b03cca49626026970307b", "tree": "73ea1d68d5d3cca777cb6ea9828c5757b4cd6df8", "parents": [ "67463acb646904d76a8e237cc31eaa87872f30cc" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 10 04:44:20 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Mon Jul 10 13:24:17 2006 -0700" }, "message": "[PATCH] md: include sector number in messages about corrected read errors\n\nThis is generally useful, but particularly helps see if it is the same sector\nthat always needs correcting, or different ones.\n\n[akpm@osdl.org: fix printk warnings]\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "5e3db645f890660ce8774a18bcd418570298937e", "tree": "a6664a8c864e40c69c7be4da6e1ff984f11873c5", "parents": [ "ae3c20ccf84c88d45616f12122f781a900118f09" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 10 04:44:18 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Mon Jul 10 13:24:17 2006 -0700" }, "message": "[PATCH] md: fix usage of wrong variable in raid1\n\nThough it rarely matters, we should be using \u0027s\u0027 rather than r1_bio-\u003esector\nhere.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "07d84d109d8beedd68df9da2e4e9f25c8217e7fb", "tree": "09b4749cb2ff1759c076c362891a28ceda2b8520", "parents": [ "3285edf152cefff482f95ceb90b1bd46ac169df1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jun 26 00:27:56 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Mon Jun 26 09:58:39 2006 -0700" }, "message": "[PATCH] md: Allow re-add to work on array without bitmaps\n\nWhen an array has a bitmap, a device can be removed and re-added and only\nblocks changes since the removal (as recorded in the bitmap) will be resynced.\n\nIt should be possible to do a similar thing to arrays without bitmaps. i.e.\nif a device is removed and re-added and *no* changes have been made in the\ninterim, then the add should not require a resync.\n\nThis patch allows that option. This means that when assembling an array one\ndevice at a time (e.g. during device discovery) the array can be enabled\nread-only as soon as enough devices are available, but extra devices can still\nbe added without causing a resync.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" } ], "next": "5fd6c1dce06ec24ef3de20fe0c7ecf2ba9fe5ef9" }