)]}' { "log": [ { "commit": "1ed7242e591af7e233234d483f12d33818b189d9", "tree": "87d4a65cdb3510a97da7f2aefb9534839c49cfa4", "parents": [ "0fd018af37dadbb7826850883ad8abfecdb1a00b" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Tue Jun 07 17:50:35 2011 -0500" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jun 08 15:11:31 2011 +1000" }, "message": "MD: raid1 changes to allow use by device mapper\n\nMD RAID1: Changes to allow RAID1 to be used by device-mapper (dm-raid.c)\n\nAdded the necessary congestion function and conditionalize calls requiring an\narray \u0027queue\u0027 or \u0027gendisk\u0027.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b098636cf04c89db4036fedc778da0acc666ad1a", "tree": "11a970bdc482b4e5cb2211eefb9a71ae34355bd3", "parents": [ "ab9d47e990c12c11cc95ed1247a3782234a7e33a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 15:52:21 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 15:52:21 2011 +1000" }, "message": "md: allow resync_start to be set while an array is active.\n\nThe sysfs attribute \u0027resync_start\u0027 (known internally as recovery_cp),\nrecords where a resync is up to. A value of 0 means the array is\nnot known to be in-sync at all. A value of MaxSector means the array\nis believed to be fully in-sync.\n\nWhen the size of member devices of an array (RAID1,RAID4/5/6) is\nincreased, the array can be increased to match. This process sets\nresync_start to the old end-of-device offset so that the new part of\nthe array gets resynced.\n\nHowever with RAID1 (and RAID6) a resync is not technically necessary\nand may be undesirable. So it would be good if the implied resync\nafter the array is resized could be avoided.\n\nSo: change \u0027resync_start\u0027 so the value can be changed while the array\nis active, and as a precaution only allow it to be changed while\nresync/recovery is \u0027frozen\u0027. Changing it once resync has started is\nnot going to be useful anyway.\n\nThis allows the array to be resized without a resync by:\n write \u0027frozen\u0027 to \u0027sync_action\u0027\n write new size to \u0027component_size\u0027 (this will set resync_start)\n write \u0027none\u0027 to \u0027resync_start\u0027\n write \u0027idle\u0027 to \u0027sync_action\u0027.\n\nAlso slightly improve some tests on recovery_cp when resizing\nraid1/raid5. Now that an arbitrary value could be set we should be\nmore careful in our tests.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "af6d7b760c7547c1a410a333bdb24daed24e1043", "tree": "5789fa005f94911bc6a92a6cb8700b8025a3adc4", "parents": [ "7ca78d57d11a91bc93b35342fa58647b85bedeb1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:51:19 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:51:19 2011 +1000" }, "message": "md/raid1: improve handling of pages allocated for write-behind.\n\nThe current handling and freeing of these pages is a bit fragile.\nWe only keep the list of allocated pages in each bio, so we need to\nstill have a valid bio when freeing the pages, which is a bit clumsy.\n\nSo simply store the allocated page list in the r1_bio so it can easily\nbe found and freed when we are finished with the r1_bio.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7ca78d57d11a91bc93b35342fa58647b85bedeb1", "tree": "398f2f91e5252b45872506763ae3677e1f35c026", "parents": [ "78d7f5f726deb562a51126603f2dc5d00990b223" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:50:37 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:50:37 2011 +1000" }, "message": "md/raid1: try fix_sync_read_error before process_checks.\n\nIf we get a read error during resync/recovery we current repeat with\nsingle-page reads to find out just where the error is, and possibly\nread each page from a different device.\n\nWith check/repair we don\u0027t currently do that, we just fail.\nHowever it is possible that while all devices fail on the large 64K\nread, we might be able to satisfy each 4K from one device or another.\n\nSo call fix_sync_read_error before process_checks to maximise the\nchance of finding good data and writing it out to the devices with\nread errors.\n\nFor this to work, we need to set the \u0027uptodate\u0027 flags properly after\nfix_sync_read_error has succeeded.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "78d7f5f726deb562a51126603f2dc5d00990b223", "tree": "eb4255cb674028b8709c4542e2a490db3264754f", "parents": [ "a68e58703575b9c03d610a818e0241564fc17f68" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:48:56 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:48:56 2011 +1000" }, "message": "md/raid1: tidy up new functions: process_checks and fix_sync_read_error.\n\nThese changes are mostly cosmetic:\n\n1/ change mddev-\u003eraid_disks to conf-\u003eraid_disks because the later is\n technically safer, though in current practice it doesn\u0027t matter in\n this particular context.\n2/ Rearrange two for / if loops to have an early \u0027continue\u0027 so the\n body of the \u0027if\u0027 doesn\u0027t need to be indented so much.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a68e58703575b9c03d610a818e0241564fc17f68", "tree": "f60a433b20a4e297d010a23775e78493687c160b", "parents": [ "6f8d0c77cef5849433dd7beb0bd97e573cc4a6a3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:40:44 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:40:44 2011 +1000" }, "message": "md/raid1: split out two sub-functions from sync_request_write\n\nsync_request_write is too big and too deep.\nSo split out two self-contains bits of functionality into separate\nfunction.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "76073054c95b12af6bd0cc9b9462a265b45ba38f", "tree": "78f830289dd8bb5337a7d3efa442ae44abd4dbab", "parents": [ "56d9912106b0974ffb6dd264c80c7e816677e998" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:34:56 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 11 14:34:56 2011 +1000" }, "message": "md/raid1: clean up read_balance.\n\nread_balance has two loops which both look for a \u0027best\u0027\ndevice based on slightly different criteria.\nThis is clumsy and makes is hard to add extra criteria.\n\nSo replace it all with a single loop that combines everything.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c3b328ac846bcf6b9a62c5563380a81ab723006d", "tree": "f9b78651beec3d66cf2ab7cf7a8080fa1cfe5b99", "parents": [ "7c13edc87510f665da3094174e1fd633e06649f4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:43 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:43 2011 +1000" }, "message": "md: fix up raid1/raid10 unplugging.\n\nWe just need to make sure that an unplug event wakes up the md\nthread, which is exactly what mddev_check_plugged does.\n\nAlso remove some plug-related code that is no longer needed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e1dfa0a29737142c32f00a3bac0f609dc85b4a82", "tree": "1af906cc5e959803e7d5ea0e56fb69b56754501a", "parents": [ "99e22598e9a8e0a996d69c8c0f6b7027cb57720a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:41 2011 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 18:25:41 2011 +1000" }, "message": "md: use new plugging interface for RAID IO.\n\nmd/raid submits a lot of IO from the various raid threads.\nSo adding start/finish plug calls to those so that some\nplugging happens.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a91a2785b200864aef2270ed6a3babac7a253a20", "tree": "3d21d7ef82ddabbdbfb74499d68a73f15adc819f", "parents": [ "82f04ab47e1d94d78503591a7460b2cad9601ede" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Thu Mar 17 11:11:05 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 17 11:11:05 2011 +0100" }, "message": "block: Require subsystems to explicitly allocate bio_set integrity mempool\n\nMD and DM create a new bio_set for every metadevice. Each bio_set has an\nintegrity mempool attached regardless of whether the metadevice is\ncapable of passing integrity metadata. This is a waste of memory.\n\nInstead we defer the allocation decision to MD and DM since we know at\nmetadevice creation time whether integrity passthrough is needed or not.\n\nAutomatic integrity mempool allocation can then be removed from\nbioset_create() and we make an explicit integrity allocation for the\nfs_bio_set.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nReported-by: Zdenek Kabelac \u003czkabelac@redhat.com\u003e\nAcked-by: Mike Snitzer \u003csnizer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4c63f5646e405b5010cc9499419060bf2e838f5b", "tree": "df91ba315032c8ec4aafeb3ab96fdfa7c6c656e1", "parents": [ "cafb0bfca1a73efd6d8a4a6a6a716e6134b96c24", "69d60eb96ae8a73cf9b79cf28051caf973006011" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "message": "Merge branch \u0027for-2.6.39/stack-plug\u0027 into for-2.6.39/core\n\nConflicts:\n\tblock/blk-core.c\n\tblock/blk-flush.c\n\tdrivers/md/raid1.c\n\tdrivers/md/raid10.c\n\tdrivers/md/raid5.c\n\tfs/nilfs2/btnode.c\n\tfs/nilfs2/mdt.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7eaceaccab5f40bbfda044629a6298616aeaed50", "tree": "33954d12f63e25a47eb6d86ef3d3d0a5e62bf752", "parents": [ "73c101011926c5832e6e141682180c4debe2cf45" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "message": "block: remove per-queue plugging\n\nCode has been converted over to the new explicit on-stack plugging,\nand delay users have been converted to use the new API for that.\nSo lets kill off the old plugging along with aops-\u003esync_page().\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "da9cf5050a2e3dbc3cf26a8d908482eb4485ed49", "tree": "d5e7ea4ef419d07d294e88b47a4aaf2676a605eb", "parents": [ "8f5f02c460b7ca74ce55ce126ce0c1e58a3f923d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Feb 21 18:25:57 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Feb 21 18:25:57 2011 +1100" }, "message": "md: avoid spinlock problem in blk_throtl_exit\n\nblk_throtl_exit assumes that -\u003equeue_lock still exists,\nso make sure that it does.\nTo do this, we stop redirecting -\u003equeue_lock to conf-\u003edevice_lock\nand leave it pointing where it is initialised - __queue_lock.\n\nAs the blk_plug functions check the -\u003equeue_lock is held, we now\ntake that spin_lock explicitly around the plug functions. We don\u0027t\nneed the locking, just the warning removal.\n\nThis is needed for any kernel with the blk_throtl code, which is\nwhich is 2.6.37 and later.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ccebd4c4159462c96397ae9af9c667bb394d7b70", "tree": "04554f0e7607d906c6b09f31a995fca0dd40c5b5", "parents": [ "57b2caa394393f8870ed41bdcc38a7542593018f" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md-new-param-to_sync_page_io\n\nAdd new parameter to \u0027sync_page_io\u0027.\n\nThe new parameter allows us to distinguish between metadata and data\noperations. This becomes important later when we add the ability to\nuse separate devices for data and metadata.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\n" }, { "commit": "067032bc628598606056412594042564fcf09e22", "tree": "5e64b6446c2156631cc66f34330ba7b134a451bf", "parents": [ "6c9879101442b08581e8a0e3ae6b7f643a78fd63" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 14 09:14:33 2011 +1100" }, "message": "md: Fix single printks with multiple KERN_\u003clevel\u003es\n\nNoticed-by: Russell King \u003clinux@arm.linux.org.uk\u003e\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8f9e0ee38f75d4740daa9e42c8af628d33d19a02", "tree": "4e1b3fe1460cd0c71258cfb2f68bb28cf71eeca4", "parents": [ "c26a44ed1e552aaa1d4ceb71842002d235fe98d7" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Nov 24 16:39:46 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Nov 24 16:39:46 2010 +1100" }, "message": "md/raid1: really fix recovery looping when single good device fails.\n\nCommit 4044ba58dd15cb01797c4fd034f39ef4a75f7cc3 supposedly fixed a\nproblem where if a raid1 with just one good device gets a read-error\nduring recovery, the recovery would abort and immediately restart in\nan infinite loop.\n\nHowever it depended on raid1_remove_disk removing the spare device\nfrom the array. But that does not happen in this case. So add a test\nso that in the \u0027recovery_disabled\u0027 case, the device will be removed.\n\nThis suitable for any kernel since 2.6.29 which is when\nrecovery_disabled was introduced.\n\nCc: stable@kernel.org\nReported-by: Sebastian Färber \u003cfaerber@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f3ac8bf7ce1c5abd763ea762e95d1cdcf7799372", "tree": "bb903d4265520d5c96a9ee08557dd7f92c1f08df", "parents": [ "046abeede717909feec38587d667cde1fc6c459c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Sep 06 14:10:08 2010 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 29 16:40:33 2010 +1100" }, "message": "md: tidy up device searches in read_balance.\n\nThe code for searching through the device list to read-balance in\nraid1 is rather clumsy and hard to follow. Try to simplify it a bit.\n\nNo important functionality change here.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "046abeede717909feec38587d667cde1fc6c459c", "tree": "e273b3c9aa887d2bc571b5ddff0bdff954f303e0", "parents": [ "9b19553e0bd342957846037c996861d280ed694d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 26 15:46:20 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 29 16:40:33 2010 +1100" }, "message": "md/raid1: fix some typos in comments.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n\n" }, { "commit": "9b19553e0bd342957846037c996861d280ed694d", "tree": "8275aadb773bf89a7f4a6c84ccd18412744ab8a7", "parents": [ "be2a2656eef95c32ca73e7a6a8c85671aa92e3f1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Oct 27 15:39:14 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Oct 29 16:40:33 2010 +1100" }, "message": "md/raid1: discard unused variable.\n\nThis structure field (flushing_bio_list) is never used, so remove it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a167f663243662aa9153c01086580a11cde9ffdc", "tree": "886e64787421bbf17a1eab7853d67258b598f050", "parents": [ "2b193363ef68667ad717a6723165e0dccf99470f" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 26 18:31:13 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:36:15 2010 +1100" }, "message": "md: use separate bio pool for each md device.\n\nbio_clone and bio_alloc allocate from a common bio pool.\nIf an md device is stacked with other devices that use this pool, or under\nsomething like swap which uses the pool, then the multiple calls on\nthe pool can cause deadlocks.\n\nSo allocate a local bio pool for each md array and use that rather\nthan the common pool.\n\nThis pool is used both for regular IO and metadata updates.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b193363ef68667ad717a6723165e0dccf99470f", "tree": "bd45d5186f0edf2b546c4503d794b479d126ad1f", "parents": [ "1c4588e9c19cae6209a28c9da2f16a18a610b935" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Oct 27 15:16:40 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:36:11 2010 +1100" }, "message": "md: change type of first arg to sync_page_io.\n\nCurrently sync_page_io takes a \u0027bdev\u0027.\nEvery caller passes \u0027rdev-\u003ebdev\u0027.\nWe will soon want another field out of the rdev in sync_page_io,\nSo just pass the rdev instead of the bdev out of it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1c4588e9c19cae6209a28c9da2f16a18a610b935", "tree": "74ffb14a5586c3a297b0fd372be5b72cf7e53bcd", "parents": [ "6746557f0325a66f57d179126426e38a8ea66945" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 26 17:41:22 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:36:09 2010 +1100" }, "message": "md/raid1: perform mem allocation before disabling writes during resync.\n\nThough this mem alloc is GFP_NOIO an so will not deadlock, it seems\nbetter to do the allocation before \u0027raise_barrier\u0027 which stops any IO\nrequests while the resync proceeds.\n\nraid10 always uses this order, so it is at least consistent to do the\nsame in raid1.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "6746557f0325a66f57d179126426e38a8ea66945", "tree": "3b1db04c854b6ccc4752560f0d30c85e6cbbda6a", "parents": [ "4e78064f42ad474ce9c31760861f7fb0cfc22532" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 26 17:33:54 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:36:06 2010 +1100" }, "message": "md: use bio_kmalloc rather than bio_alloc when failure is acceptable.\n\nbio_alloc can never fail (as it uses a mempool) but an block\nindefinitely, especially if the caller is holding a reference to a\npreviously allocated bio.\n\nSo these to places which both handle failure and hold multiple bios\nshould not use bio_alloc, they should use bio_kmalloc.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "4e78064f42ad474ce9c31760861f7fb0cfc22532", "tree": "3a1abaa98ebcbd62eacfbe95d72e44195fb3bc1f", "parents": [ "e804ac780e2f01cb3b914daca2fd4780d1743db1" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 19 12:54:01 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:34:07 2010 +1100" }, "message": "md: Fix possible deadlock with multiple mempool allocations.\n\nIt is not safe to allocate from a mempool while holding an item\npreviously allocated from that mempool as that can deadlock when the\nmempool is close to exhaustion.\n\nSo don\u0027t use a bio list to collect the bios to write to multiple\ndevices in raid1 and raid10.\nInstead queue each bio as it becomes available so an unplug will\nactivate all previously allocated bios and so a new bio has a chance\nof being allocated.\n\nThis means we must set the \u0027remaining\u0027 count to \u00271\u0027 before submitting\nany requests, then when all are submitted, decrement \u0027remaining\u0027 and\npossible handle the write completion at that point.\n\nReported-by: Torsten Kaiser \u003cjust.for.lkml@googlemail.com\u003e\nTested-by: Torsten Kaiser \u003cjust.for.lkml@googlemail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "57dab0bdf689d42972975ec646d862b0900a4bf3", "tree": "d136f69b8a68850e40844397ebaffde31787c988", "parents": [ "4b532c9b8c87eb8e51605c4d08dfb5139c758dc5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 19 10:03:39 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 28 17:32:26 2010 +1100" }, "message": "md: use sector_t in bitmap_get_counter\n\nbitmap_get_counter returns the number of sectors covered\nby the counter in a pass-by-reference variable.\nIn some cases this can be very large, so make it a sector_t\nfor safety.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n\n" }, { "commit": "fa251f89903d73989e2f63e13d0eaed1e07ce0da", "tree": "3f7fe779941e3b6d67754dd7c44a32f48ea47c74", "parents": [ "dd3932eddf428571762596e17b65f5dc92ca361b", "cd07202cc8262e1669edff0d97715f3dd9260917" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Oct 19 09:13:04 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Oct 19 09:13:04 2010 +0200" }, "message": "Merge branch \u0027v2.6.36-rc8\u0027 into for-2.6.37/barrier\n\nConflicts:\n\tblock/blk-core.c\n\tdrivers/block/loop.c\n\tmm/swapfile.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "db8d9d3591f77cc6b66248b2cdfa1c43deee5ec9", "tree": "2b8e727bc5f74883620934d3334323ce501c853e", "parents": [ "7571ae887d3b96d8e7ce63d43828866c6a91cdc6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 07 12:00:50 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 07 12:00:50 2010 +1100" }, "message": "md/raid1: minor bio initialisation improvements.\n\n\nWhen performing a resync we pre-allocate some bios and repeatedly use\nthem. This requires us to re-initialise them each time.\nOne field (bi_comp_cpu) and some flags weren\u0027t being initiaised\nreliably.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n\n" }, { "commit": "7571ae887d3b96d8e7ce63d43828866c6a91cdc6", "tree": "ecdc5d71d3d957604a172c23ef9943438ccee949", "parents": [ "7c6d45e665d5322401e4439060bbf758b08422d4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 07 11:54:46 2010 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 07 11:54:46 2010 +1100" }, "message": "md/raid1: avoid overflow in raid1 resync when bitmap is in use.\n\nbitmap_start_sync returns - via a pass-by-reference variable - the\nnumber of sectors before we need to check with the bitmap again.\nSince commit ef4256733506f245 this number can be substantially larger,\n2^27 is a common value.\n\nUnfortunately it is an \u0027int\u0027 and so when raid1.c:sync_request shifts\nit 9 places to the left it becomes 0. This results in a zero-length\nread which the scsi layer justifiably complains about.\n\nThis patch just removes the shift so the common case becomes safe with\na trivially-correct patch.\n\nIn the next merge window we will convert this \u0027int\u0027 to a \u0027sector_t\u0027\n\nReported-by: \"George Spelvin\" \u003clinux@horizon.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e9c7469bb4f502dafc092166201bea1ad5fc0fbf", "tree": "04202b0bb88623d3005c909eaafcb280778902da", "parents": [ "7bc9fddab074d6bb630344e1969e28d20b140621" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:18 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:38 2010 +0200" }, "message": "md: implment REQ_FLUSH/FUA support\n\nThis patch converts md to support REQ_FLUSH/FUA instead of now\ndeprecated REQ_HARDBARRIER. In the core part (md.c), the following\nchanges are notable.\n\n* Unlike REQ_HARDBARRIER, REQ_FLUSH/FUA don\u0027t interfere with\n processing of other requests and thus there is no reason to mark the\n queue congested while FLUSH/FUA is in progress.\n\n* REQ_FLUSH/FUA failures are final and its users don\u0027t need retry\n logic. Retry logic is removed.\n\n* Preflush needs to be issued to all member devices but FUA writes can\n be handled the same way as other writes - their processing can be\n deferred to request_queue of member devices. md_barrier_request()\n is renamed to md_flush_request() and simplified accordingly.\n\nFor linear, raid0 and multipath, the core changes are enough. raid1,\n5 and 10 need the following conversions.\n\n* raid1: Handling of FLUSH/FUA bio\u0027s can simply be deferred to\n request_queues of member devices. Barrier related logic removed.\n\n* raid5: Queue draining logic dropped. FUA bit is propagated through\n biodrain and stripe resconstruction such that all the updated parts\n of the stripe are written out with FUA writes if any of the dirtying\n writes was FUA. preread_active_stripes handling in make_request()\n is updated as suggested by Neil Brown.\n\n* raid10: FUA bit needs to be propagated to write clones.\n\nlinear, raid0, 1, 5 and 10 tested.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReviewed-by: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "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" } ], "next": "2ad8b1ef11c98c5603580878aebf9f1bc74129e4" }