)]}' { "log": [ { "commit": "ad361c9884e809340f6daca80d56a9e9c871690a", "tree": "7ec02c9934964fecdc791a0df0fc722d3bda5c53", "parents": [ "e3288775ff63900fbb7db505f2b9a1bee98f07df" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Mon Jul 06 13:05:40 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jul 08 10:30:03 2009 -0700" }, "message": "Remove multiple KERN_ prefixes from printk formats\n\nCommit 5fd29d6ccbc98884569d6f3105aeca70858b3e0f (\"printk: clean up\nhandling of log-levels and newlines\") changed printk semantics. printk\nlines with multiple KERN_\u003clevel\u003e prefixes are no longer emitted as\nbefore the patch.\n\n\u003clevel\u003e is now included in the output on each additional use.\n\nRemove all uses of multiple KERN_\u003clevel\u003es in formats.\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e62e58a5ffdc98ac28d8dbd070c857620d541f99", "tree": "0ec3471f4e66e3a376ac8cb2da79d6123e7aa2cf", "parents": [ "a5c308d4d1659b1f4833b863394e3e24cdbdfc6e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 13:15:35 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 13:15:35 2009 +1000" }, "message": "md: use interruptible wait when duration is controlled by userspace.\n\nUser space can set various limits on an md array so that resync waits\nwhen it gets to a certain point, or so that I/O is blocked for a short\nwhile.\nWhen md is waiting against one of these limit, it should use an\ninterruptible wait so as not to add to the load average, and so are\nnot to trigger a warning if the wait goes on for too long.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0909dc448c98ed5021c87ffdfc09fb473aa464ab", "tree": "17c48d146c0f9eb7367a0d0feafea29838eae8eb", "parents": [ "1ec22eb2b4a2e1a763106bce36b11c02eaa84e61" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 12:27:21 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 12:27:21 2009 +1000" }, "message": "md: tidy up error paths in md_alloc\n\nAs the recent bug in md_alloc showed, having a single exit path for\nunlocking and putting is a good idea. So restructure md_alloc to have\na single mutex_unlock and mddev_put, and use gotos where necessary.\n\nFound-by: Jiri Slaby \u003cjirislaby@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1ec22eb2b4a2e1a763106bce36b11c02eaa84e61", "tree": "2aa6e6539342a28263417f6a2f2551d541e9ead9", "parents": [ "b8d966efd9a46a9a35beac50cbff6e30565125ef" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 12:27:21 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 12:27:21 2009 +1000" }, "message": "md: fix error path when duplicate name is found on md device creation.\n\nWhen an md device is created by name (rather than number) we need to\ncheck that the name is not already in use. If this check finds a\nduplicate, we return an error without dropping the lock or freeing\nthe newly create mddev.\nThis patch fixes that.\n\nCc: stable@kernel.org\nFound-by: Jiri Slaby \u003cjirislaby@gmail.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b8d966efd9a46a9a35beac50cbff6e30565125ef", "tree": "d1c59125e8ee9677e8594bdc4ab223e692a87e50", "parents": [ "8f6c2e4b325a8e9f8f47febb2fd0ed4fae7d45a9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 11:14:04 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 11:14:04 2009 +1000" }, "message": "md: avoid dereferencing NULL pointer when accessing suspend_* sysfs attributes.\n\nIf we try to modify one of the md/ sysfs files\n suspend_lo or suspend_hi\nwhen the array is not active, we dereference a NULL.\nProtect against that.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0894cc3066aaa3e75a99383c0d25feebf9b688ac", "tree": "651664e6b288e36dc9553443e334e650139851c8", "parents": [ "8190e754e0723de7cecb80bdd9eb93911dfa04a1" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Thu Jun 18 08:49:23 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:49:23 2009 +1000" }, "message": "md: Move check for bitmap presence to personality code.\n\nIf the superblock of a component device indicates the presence of a\nbitmap but the corresponding raid personality does not support bitmaps\n(raid0, linear, multipath, faulty), then something is seriously wrong\nand we\u0027d better refuse to run such an array.\n\nCurrently, this check is performed while the superblocks are examined,\ni.e. before entering personality code. Therefore the generic md layer\nmust know which raid levels support bitmaps and which do not.\n\nThis patch avoids this layer violation without adding identical code\nto various personalities. This is accomplished by introducing a new\npublic function to md.c, md_check_no_bitmap(), which replaces the\nhard-coded checks in the superblock loading functions.\n\nA call to md_check_no_bitmap() is added to the -\u003erun method of each\npersonality which does not support bitmaps and assembly is aborted\nif at least one component device contains a bitmap.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8190e754e0723de7cecb80bdd9eb93911dfa04a1", "tree": "6fbd0edd64cb767b9421103301e198804c3aa40b", "parents": [ "13f2682b7216ebebd72b3d5868fe7fccec91a92d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:48:58 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:48:58 2009 +1000" }, "message": "md: remove chunksize rounding from common code.\n\nIt is easiest to round sizes to multiples of chunk size in\nthe personality code for those personalities which care.\nThose personalities now do the rounding, so we can\nremove that function from common code.\n\nAlso remove the upper bound on the size of a chunk, and the lower\nbound on the size of a device (1 chunk), neither of which really buy\nus anything.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1b57f132231593923cb4ab99943ddd777e8745bc", "tree": "b76f936d08e5ef72430c6fbcf9352f53721481ec", "parents": [ "8c6ac868b107ed50a46204f6d14e2ad9443ff146" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:48:19 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:48:19 2009 +1000" }, "message": "md: move assignment of -\u003eutime so that it never gets skipped.\n\nCurrently the assignment to utime gets skipped for \u0027external\u0027\nmetadata. So move it to the top of the function so that it\nalways gets effected.\nThis is of largely cosmetic interest. Nothing actually depends\non -\u003eutime being right for external arrays.\n\"mdadm --monitor\" does use it for 0.90 and 1.x arrays, but with\nmdadm-3.0, this is not important for external metadata.\n\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": "50ac168a6e0a061bf5346d53aa9e7beb94c97527", "tree": "4f1eff67793bd8349a1995c921a63e9512be7818", "parents": [ "597a711b69cfff95c4b8f6069037e7ad3fc71f56" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:47:55 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:47:55 2009 +1000" }, "message": "md: merge reconfig and check_reshape methods.\n\nThe difference between these two methods is artificial.\nBoth check that a pending reshape is valid, and perform any\naspect of it that can be done immediately.\n\u0027reconfig\u0027 handles chunk size and layout.\n\u0027check_reshape\u0027 handles raid_disks.\n\nSo make them just one method.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "597a711b69cfff95c4b8f6069037e7ad3fc71f56", "tree": "8c2cfa4c1223827a560de2f14da426d4822151c4", "parents": [ "01ee22b496c41384eaa6dcae983c86d8bc32fbb8" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:47:42 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:47:42 2009 +1000" }, "message": "md: remove unnecessary arguments from -\u003ereconfig method.\n\nPassing the new layout and chunksize as args is not necessary as\nthe mddev has fields for new_check and new_layout.\n\nThis is preparation for combining the check_reshape and reconfig\nmethods\n\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": "2ac06c3332898103210b478c5a17c20e28929287", "tree": "6f34315bbdcc8a8300ab2ab5edcd4cb96150e583", "parents": [ "740da44918680a0c72411ae4ccdd1861069afcc4" ], "author": { "name": "raz ben yehuda", "email": "raziebe@gmail.com", "time": "Tue Jun 16 17:01:42 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 16 17:01:42 2009 +1000" }, "message": "md: prepare for non-power-of-two chunk sizes\n\nRemove chunk size check from md as this is now performed in the run\nfunction in each personality.\n\nReplace chunk size power 2 code calculations by a regular division.\n\nSigned-off-by: raziebe@gmail.com\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c9059598ea8981d02356eead3188bf7fa4d717b8", "tree": "03e73b20a30e988da7c6a3e0ad93b2dc5843274d", "parents": [ "0a33f80a8373eca7f4bea3961d1346c3815fa5ed", "b0fd271d5fba0b2d00888363f3869e3f9b26caa9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 11 10:52:27 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 11 11:10:35 2009 -0700" }, "message": "Merge branch \u0027for-2.6.31\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.31\u0027 of git://git.kernel.dk/linux-2.6-block: (153 commits)\n block: add request clone interface (v2)\n floppy: fix hibernation\n ramdisk: remove long-deprecated \"ramdisk\u003d\" boot-time parameter\n fs/bio.c: add missing __user annotation\n block: prevent possible io_context-\u003erefcount overflow\n Add serial number support for virtio_blk, V4a\n block: Add missing bounce_pfn stacking and fix comments\n Revert \"block: Fix bounce limit setting in DM\"\n cciss: decode unit attention in SCSI error handling code\n cciss: Remove no longer needed sendcmd reject processing code\n cciss: change SCSI error handling routines to work with interrupts enabled.\n cciss: separate error processing and command retrying code in sendcmd_withirq_core()\n cciss: factor out fix target status processing code from sendcmd functions\n cciss: simplify interface of sendcmd() and sendcmd_withirq()\n cciss: factor out core of sendcmd_withirq() for use by SCSI error handling code\n cciss: Use schedule_timeout_uninterruptible in SCSI error handling code\n block: needs to set the residual length of a bidi request\n Revert \"block: implement blkdev_readpages\"\n block: Fix bounce limit setting in DM\n Removed reference to non-existing file Documentation/PCI/PCI-DMA-mapping.txt\n ...\n\nManually fix conflicts with tracing updates in:\n\tblock/blk-sysfs.c\n\tdrivers/ide/ide-atapi.c\n\tdrivers/ide/ide-cd.c\n\tdrivers/ide/ide-floppy.c\n\tdrivers/ide/ide-tape.c\n\tinclude/trace/events/block.h\n\tkernel/trace/blktrace.c\n" }, { "commit": "b492b852cd8c99505708152c29a5e09a787af9de", "tree": "f26aa9676231a583e209a9397acfc4512061325e", "parents": [ "7a91ee1f628ef6bfe3f13067c0ddf9db520cb86b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:36 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:36 2009 +1000" }, "message": "md: don\u0027t use locked_ioctl.\n\nmd has no need for the BKL - it does its own locking.\nSo md_ioctl doesn\u0027t need to be a locked_ioctl.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7a91ee1f628ef6bfe3f13067c0ddf9db520cb86b", "tree": "b6b317a2dfb067784fccae113001cb3957e25337", "parents": [ "848b3182365fdf5a05bcd5ed949071cac2c894b3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:21 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:21 2009 +1000" }, "message": "md: don\u0027t update curr_resync_completed without also updating reshape_position.\n\nIn order for the metadata to always be consistent, we mustn\u0027t updated\ncurr_resync_completed without also updating reshape_position.\n\nThe reshape code updates both at the same time. However since\ncommit 97e4f42d62badb0f9fbc27c013e89bc1336a03bc\nthe common md_do_sync will sometimes update curr_resync_completed\nbut is not in a position to update reshape_position.\nSo if MD_RECOVERY_RESHAPE is set (indicating that a reshape is\nhappening, so reshape_position might change), don\u0027t update\ncurr_resync_completed in md_do_sync, leave it to the per-personality\nreshape code.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b6a9ce688f613e2ee5f15e6720e0bb8520efc36e", "tree": "7fc4c8ab5ea14df5e544e740ec3154b707766a08", "parents": [ "be512691036cc989c11d0f418187efbbf14468e6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "message": "md: export \u0027frozen\u0027 resync state through sysfs\n\nThe md resync engine has a \u0027frozen\u0027 state which ensures that\nno resync/recovery. This is used to avoid races.\n\nExport this state through the \u0027sync_action\u0027 sysfs attribute\nso that user-space can benefit and also avoid some races.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b69c83924396ad1eda36fdd267c9d2f360f5555", "tree": "14c01b6dd6b596859835b3f0d176bf02129d2c45", "parents": [ "62e1e389f87a8839ad83b08c44691d1df8320846" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "message": "md: improve errno return when setting array_size\n\nInstead of always returns EINVAL if anything goes wrong\nwhen setting the array size, add the option of\n E2BIG\nif the size requested is too large. This makes it easier\nfor user-space to be sure what went wrong.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "62e1e389f87a8839ad83b08c44691d1df8320846", "tree": "d8915df95d7164188334052b31ffe7ac40aa4e94", "parents": [ "59a3759d0fe8d969888c741bb33f4946e4d3750d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:40:59 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:40:59 2009 +1000" }, "message": "md: always update level / chunk_size / layout when writing v1.x metadata.\n\nWe previously didn\u0027t update these fields when writing the metadata\nbecause they could never change. They can now, so we better write\nthem.\nv0.90 metadata always updated these fields.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e1defc4ff0cf57aca6c5e3ff99fa503f5943c1f1", "tree": "d60d15a082171c58ac811d547d51a9c3119f23e3", "parents": [ "9bd7de51ee8537094656149eaf45338cadb7d7d4" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri May 22 17:17:49 2009 -0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri May 22 23:22:54 2009 +0200" }, "message": "block: Do away with the notion of hardsect_size\n\nUntil now we have had a 1:1 mapping between storage device physical\nblock size and the logical block sized used when addressing the device.\nWith SATA 4KB drives coming out that will no longer be the case. The\nsector size will be 4KB but the logical block size will remain\n512-bytes. Hence we need to distinguish between the physical block size\nand the logical ditto.\n\nThis patch renames hardsect_size to logical_block_size.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "c4647292fda0833bebe45be27f04453b736981fa", "tree": "85c89a4161b06b1c591a76c2a68ff51fc02ff6f4", "parents": [ "5bf295975416f8e97117bbbcfb0191c00bc3e2b4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:51:06 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:51:06 2009 +1000" }, "message": "md: remove rd%d links immediately after stopping an array.\n\nmd maintains link in sys/mdXX/md/ to identify which device has\nwhich role in the array. e.g.\n rd2 -\u003e dev-sda\n\nindicates that the device with role \u00272\u0027 in the array is sda.\n\nThese links are only present when the array is active. They are\ncreated immediately after -\u003erun is called, and so should be removed\nimmediately after -\u003estop is called.\nHowever they are currently removed a little bit later, and it is\npossible for -\u003erun to be called again, thus adding these links, before\nthey are removed.\n\nSo move the removal earlier so they are consistently only present when\nthe array is active.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5bf295975416f8e97117bbbcfb0191c00bc3e2b4", "tree": "9d7ddfe9d4632c8d6cacb6c4dd7b9cb49f837432", "parents": [ "110518bccf076726cc93bf604527d8019aae50ba" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:50:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:50:57 2009 +1000" }, "message": "md: remove ability to explicit set an inactive array to \u0027clean\u0027.\n\nBeing able to write \u0027clean\u0027 to an \u0027array_state\u0027 of an inactive array\nto activate it in \u0027clean\u0027 mode is both unnecessary and inconvenient.\n\nIt is unnecessary because the same can be achieved by writing\n\u0027active\u0027. This activates and array, but it still remains \u0027clean\u0027\nuntil the first write.\n\nIt is inconvenient because writing \u0027clean\u0027 is more often used to\ncause an \u0027active\u0027 array to revert to \u0027clean\u0027 mode (thus blocking\nany writes until a \u0027write-pending\u0027 is promoted to \u0027active\u0027).\n\nAllowing \u0027clean\u0027 to both activate an array and mark an active array as\nclean can lead to races: One program writes \u0027clean\u0027 to mark the\nactive array as clean at the same time as another program writes\n\u0027inactive\u0027 to deactivate (stop) and active array. Depending on which\nwrites first, the array could be deactivated and immediately\nreactivated which isn\u0027t what was desired.\n\nSo just disable the use of \u0027clean\u0027 to activate an array.\n\nThis avoids a race that can be triggered with mdadm-3.0 and external\nmetadata, so it suitable for -stable.\n\nReported-by: Rafal Marszewski \u003crafal.marszewski@intel.com\u003e\nAcked-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "110518bccf076726cc93bf604527d8019aae50ba", "tree": "f2df7daf983640c64f66bcd76aea475b35f9f1d3", "parents": [ "dd71cf6b2773310b01c6fe6c773064c80fd2476b" ], "author": { "name": "Jan Engelhardt", "email": "jengelh@medozas.de", "time": "Thu May 07 12:49:37 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:37 2009 +1000" }, "message": "md: constify VFTs\n\n\nSigned-off-by: Jan Engelhardt \u003cjengelh@medozas.de\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "dd71cf6b2773310b01c6fe6c773064c80fd2476b", "tree": "33a4cc4fe4db45ec4f1a8f477bb294a0b685efa4", "parents": [ "db305e507d554430a69ede901a6308e6ecb72349" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:35 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:35 2009 +1000" }, "message": "md: tidy up status_resync to handle large arrays.\n\nTwo problems in status_resync.\n1/ It still used Kilobytes as the basic block unit, while most code\n now uses sectors uniformly.\n2/ It doesn\u0027t allow for the possibility that max_sectors exceeds\n the range of \"unsigned long\".\n\nSo\n - change \"max_blocks\" to \"max_sectors\", and store sector numbers\n in there and in \u0027resync\u0027\n - Make \u0027rt\u0027 a \u0027sector_t\u0027 so it can temporarily hold the number of\n remaining sectors.\n - use sector_div rather than normal division.\n - change the magic \u0027100\u0027 used to preserve precision to \u002732\u0027.\n + making it a power of 2 makes division easier\n + it doesn\u0027t need to be as large as it was chosen when we averaged\n speed over the entire run. Now we average speed over the last 30\n seconds or so.\n\nReported-by: \"Mario \u0027BitKoenig\u0027 Holbe\" \u003cMario.Holbe@TU-Ilmenau.DE\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c03f6a19699fce4389888a45db8245969311d868", "tree": "2a3be3033d5fad42f25fe2678e98efcf6abb04bc", "parents": [ "acb180b0e335ad88acfed6c8a33e39c05b95dc49" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Apr 17 11:06:30 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Apr 17 11:06:30 2009 +1000" }, "message": "md: update sync_completed and reshape_position even more often.\n\nThere are circumstances when a user-space process might need to\n\"oversee\" a resync/reshape process. For example when doing an\nin-place reshape of a raid5, it is prudent to take a backup of each\nsection before reshaping it as this is the only way to provide\nsafety against an unplanned shutdown (i.e. crash/power failure).\n\nThe sync_max sysfs value can be used to stop the resync from\nadvancing beyond a particular point.\nSo user-space can:\n suspend IO to the first section and back it up\n set \u0027sync_max\u0027 to the end of the section\n wait for \u0027sync_completed\u0027 to reach that point\n resume IO on the first section and move on to the next section.\n\nHowever this process requires the kernel and user-space to run in\nlock-step which could introduce unnecessary delays.\n\nIt would be better if a \u0027double buffered\u0027 approach could be used with\nuserspace and kernel space working on different sections with the\n\u0027next\u0027 section always ready when the \u0027current\u0027 section is finished.\n\nOne problem with implementing this is that sync_completed is only\nguaranteed to be updated when the sync process reaches sync_max.\n(it is updated on a time basis at other times, but it is hard to rely\non that). This defeats some of the double buffering.\n\nWith this patch, sync_completed (and reshape_position) get updated as\nthe current position approaches sync_max, so there is room for\nuserspace to advance sync_max early without losing updates.\n\nTo be precise, sync_completed is updated when the current sync\nposition reaches half way between the current value of sync_completed\nand the value of sync_max. This will usually be a good time for user\nspace to update sync_max.\n\nIf sync_max does not get updated, the updates to sync_completed\n(together with associated metadata updates) will occur at an\nexponentially increasing frequency which will get unreasonably fast\n(one update every page) immediately before the process hits sync_max\nand stops. So the update rate will be unreasonably fast only for an\ninsignificant period of time.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "acb180b0e335ad88acfed6c8a33e39c05b95dc49", "tree": "fa980978bb66b178fd8c2f474194754139c0921d", "parents": [ "6d56e278444bc8323b1bcfcada126b8e4dbba0f4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 16:28:34 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 16:28:34 2009 +1000" }, "message": "md: improve usefulness and accuracy of sysfs file md/sync_completed.\n\nThe sync_completed file reports how much of a resync (or recovery or\nreshape) has been completed.\nHowever due to the possibility of out-of-order completion of writes,\nit is not certain to be accurate.\n\nWe have an internal value - mddev-\u003ecurr_resync_completed - which is an\naccurate value (though it might not always be quite so uptodate).\n\nSo:\n - make curr_resync_completed be uptodate a little more often,\n particularly when raid5 reshape updates status in the metadata\n - report curr_resync_completed in the sysfs file\n - allow poll/select to report all updates to md/sync_completed.\n\nThis makes sync_completed completed usable by any external metadata\nhandler that wants to record this status information in its metadata.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "6d56e278444bc8323b1bcfcada126b8e4dbba0f4", "tree": "4a1e838b3bb46ce0650f08daa4cdf976eb49317e", "parents": [ "63fe08177f92ce21929df8af6361491b98ad12cd" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 12:01:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 12:01:57 2009 +1000" }, "message": "md: allow setting newly added device to \u0027in_sync\u0027 via sysfs.\n\nWhen adding devices to an active array via sysfs, there is currently\nno way to mark a device as \u0027in-sync\u0027 which is useful when\nincrementally assembling an array.\n\nSo add that option.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "223cdea4c4b5af5181b2da00ac85711d1e0c737c", "tree": "dfe7226c70ddabbf2e2e63924ba636345278e79c", "parents": [ "31e6e2dac575c9d21a6ec56ca52ae89086baa705", "c8f517c444e4f9f55b5b5ca202b8404691a35805" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 09:08:19 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 09:08:19 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md: (53 commits)\n md/raid5 revise rules for when to update metadata during reshape\n md/raid5: minor code cleanups in make_request.\n md: remove CONFIG_MD_RAID_RESHAPE config option.\n md/raid5: be more careful about write ordering when reshaping.\n md: don\u0027t display meaningless values in sysfs files resync_start and sync_speed\n md/raid5: allow layout and chunksize to be changed on active array.\n md/raid5: reshape using largest of old and new chunk size\n md/raid5: prepare for allowing reshape to change layout\n md/raid5: prepare for allowing reshape to change chunksize.\n md/raid5: clearly differentiate \u0027before\u0027 and \u0027after\u0027 stripes during reshape.\n Documentation/md.txt update\n md: allow number of drives in raid5 to be reduced\n md/raid5: change reshape-progress measurement to cope with reshaping backwards.\n md: add explicit method to signal the end of a reshape.\n md/raid5: enhance raid5_size to work correctly with negative delta_disks\n md/raid5: drop qd_idx from r6_state\n md/raid6: move raid6 data processing to raid6_pq.ko\n md: raid5 run(): Fix max_degraded for raid level 4.\n md: \u0027array_size\u0027 sysfs attribute\n md: centralize -\u003earray_sectors modifications\n ...\n" }, { "commit": "d1a7c50369835f9ecbd7752016cd9302ecfae678", "tree": "2c6bcc8c5cb4d045d555e0d030321cb887a4e05a", "parents": [ "88ce4930e2b80378d45506ce2c3bb5820e156e85" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:32 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:32 2009 +1100" }, "message": "md: don\u0027t display meaningless values in sysfs files resync_start and sync_speed\n\nWhen no resync if happening, both of these files currently have\nmeaningless values (is slightly different ways).\nChange them to \"none\" in that case.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "cea9c22800773cecb1d41f4a6139f9eb6a95368b", "tree": "d070d22390b86768fefb8ba04c64d74993ed28e9", "parents": [ "7ec0547838976d088dfb9cb0adb073e6e8a15aa3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:15:05 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:15:05 2009 +1100" }, "message": "md: add explicit method to signal the end of a reshape.\n\nCurrently raid5 (the only module that supports restriping)\nnotices that the reshape has finished be sync_request being\ngiven a large value, and handles any cleanup them.\n\nThis patch changes it so md_check_recovery calls into an\nexplicit finish_reshape method as well.\n\nThe clean-up from sync_request can do things that need to be\ndone promptly, typically things local to the raid5_conf_t\nstructure.\n\nThe \"finish_reshape\" method is called under the mddev_lock\nso it can do things involving reconfiguring the device.\n\nThis allows us to get rid of md_set_array_sectors_locked, which\nwould have caused a deadlock if you tried to stop and array\nwhile a reshape was happening.\n\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": "b3546035277847028df650b147469fc943cf5c71", "tree": "87966abc5456a62845326eb8d5a5cf0f88879b2d", "parents": [ "d562b0c4313e3ddea402a400371afa47ddf679f9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:56:41 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:56:41 2009 +1100" }, "message": "md/raid5: allow layout/chunksize to be changed on an active 2-drive raid5.\n\n2-drive raid5\u0027s aren\u0027t very interesting. But if you are converting\na raid1 into a raid5, you will at least temporarily have one. And\nthat it a good time to set the layout/chunksize for the new RAID5\nif you aren\u0027t happy with the defaults.\n\nlayout and chunksize don\u0027t actually affect the placement of data\non a 2-drive raid5, so we just do some internal book-keeping.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "245f46c2c221ef09c7db892f0e3fc2149be42052", "tree": "be7e6f4b7b65b682c43e59ef17f7ed580bc8a82b", "parents": [ "409c57f3801701dfee27a28103dda4831306cb20" ], "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: add -\u003etakeover method to support changing the personality managing an array\n\nImplement this for RAID6 to be able to \u0027takeover\u0027 a RAID5 array. The\nnew RAID6 will use a layout which places Q on the last device, and\nthat device will be missing.\nIf there are any available spares, one will immediately have Q\nrecovered onto it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\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": "e0cf8f045b2023b0b3f919ee93eb94345f648434", "tree": "21d8d0741a5ae4676fea5b53939cc806ae2b41bd", "parents": [ "91adb56473febeeb3ef657bb5147ddd355465700" ], "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: md_unregister_thread should cope with being passed NULL\n\nMostly md_unregister_thread is only called when we know that the\nthread is NULL, but sometimes we need to check first. It is safer\nto put the check inside md_unregister_thread itself.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "34817e8c3948ea20316dfa8fd8947d6d0ee82ba9", "tree": "f4d2e75473b46db94928e21e9fe54b029432418b", "parents": [ "67cc2b8165857ba019920d1f00d64bcc4140075d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:39:38 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:39:38 2009 +1100" }, "message": "md: make sure new_level, new_chunksize, new_layout always have sensible values.\n\nWhen an md array is undergoing a change, we have new_* fields that\nshow the new values.\nWhen no change is happening, it is least confusing if these have\nthe same value as the normal fields.\nThis is true in most cases, but not when the values are set via sysfs.\n\nSo fix this up.\n\nA subsequent patch will BUG_ON if these things aren\u0027t consistent.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "dd8ac336c13fd8afdb082ebacb1cddd5cf727889", "tree": "0baa8111d7406774525a1053be461135257d73d2", "parents": [ "58c0fed400603a802968b23ddf78f029c5a84e41" ], "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: Represent raid device size in sectors.\n\nThis patch renames the \"size\" field of struct mdk_rdev_s to\n\"sectors\" and changes this field to store sectors instead of\nblocks.\n\nAll users of this field, linear.c, raid0.c and md.c, are fixed up\naccordingly which gets rid of many multiplications and divisions.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\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": "575a80fa4f623141e9791e41879d87800fb6d862", "tree": "f316efc3ee33b2914d8b9f3a3041394560ee9e90", "parents": [ "97e4f42d62badb0f9fbc27c013e89bc1336a03bc" ], "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: be more consistent about setting WriteMostly flag when adding a drive to an array\n\nWhen a drive is added to an array using ADD_NEW_DISK, there are two\nplaces we can get certain flags from: the metadata on the disk or the\nflags passed through the IOCTL.\n\nFor the WriteMostly flag (aka MD_DISK_WRITEMOSTLY) we take the value\nfrom either of those sources depending on if it is set (i.e. we\neffectively \u0027or\u0027 the two sources together).\n\nThis makes it awkward to clear, and is at best inconsistent.\n\nAs documented code (in mdadm) requires that setting\nMD_DISK_WRITEMOSTLY in the ioctl will be effective, we resolve the\ninconsistency by always using the value for this flag from the ioctl,\nand ignoring the value on disk.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "97e4f42d62badb0f9fbc27c013e89bc1336a03bc", "tree": "04cc809702a6b080c417c4ddf605642bbf7de521", "parents": [ "43b2e5d86d8bdd77386226db0bc961529492c043" ], "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: occasionally checkpoint drive recovery to reduce duplicate effort after a crash\n\nVersion 1.x metadata has the ability to record the status of a\npartially completed drive recovery.\nHowever we only update that record on a clean shutdown.\nIt would be nice to update it on unclean shutdowns too, particularly\nwhen using a bitmap that removes much to the \u0027sync\u0027 effort after an\nunclean shutdown.\n\nOne complication with checkpointing recovery is that we only know\nwhere we are up to in terms of IO requests started, not which ones\nhave completed. And we need to know what has completed to record\nhow much is recovered. So occasionally pause the recovery until all\nsubmitted requests are completed, then update the record of where\nwe are up to.\n\nWhen we have a bitmap, we already do that pause occasionally to keep\nthe bitmap up-to-date. So enhance that code to record the recovery\noffset and schedule a superblock update.\nAnd when there is no bitmap, just pause 16 times during the resync to\ndo a checkpoint.\n\u002716\u0027 is a fairly arbitrary number. But we don\u0027t really have any good\nway to judge how often is acceptable, and it seems like a reasonable\nnumber for now.\n\n\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": "8b2b5c217c20b5460218ab8731295f2e46c7dd29", "tree": "f5da8b06b3d2a53e80c31e0c0aa386e381b3a841", "parents": [ "ef740c372dfd80e706dbf955d4e4aedda6c0c148" ], "author": { "name": "NeilBrown", "email": "neilb@suse.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 LEVEL_* definition from md_k.h to md_u.h\n\n.. as they are part of the user-space interface.\nAlso move MdpMinorShift into there so we can remove duplication.\n\nLastly move mdp_major in. It is less obviously part of the user-space\ninterface, but do_mounts_md.c uses it, and it is acting a bit like\nuser-space.\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": "3dbd8c2e3ff0185585e068f190289d2a267a3e83", "tree": "f49dfbb19fc57e7888a53e763449c36e692367c4", "parents": [ "3f9d99c12a533809342b475c95452e82761bcc1c" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "message": "md: stop defining MAJOR_NR\n\nMAJOR_NR was only required for magic in linux/blk.h in 2.4 or earlier\nkernels, so no need to keep it around.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3f9d99c12a533809342b475c95452e82761bcc1c", "tree": "8e8bcc72dce1b69e958317f77c351bc0bd4e1c28", "parents": [ "355a43e641b948a7b755cb4c2466ec548d5b495f" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "message": "MD data integrity support\n\nmd: Add support for data integrity to MD\n\nIf all subdevices support the same protection format the MD device is\nflagged as integrity capable.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "eea1bf384e05b5ab747f8530c4fba9e9e6907fff", "tree": "ed4c05f8bd7e5805092e9c6a2c257ca71dc0fe2d", "parents": [ "99adcd9d67aaf04e28f5ae96df280f236bde4b66" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 14:27:02 2009 +1100" }, "message": "md: Fix is_mddev_idle test (again).\n\nThere are two problems with is_mddev_idle.\n\n1/ sync_io is \u0027atomic_t\u0027 and hence \u0027int\u0027. curr_events and all the\n rest are \u0027long\u0027.\n So if sync_io were to wrap on a 64bit host, the value of\n curr_events would go very negative suddenly, and take a very\n long time to return to positive.\n\n So do all calculations as \u0027int\u0027. That gives us plenty of precision\n for what we need.\n\n2/ To initialise rdev-\u003elast_events we simply call is_mddev_idle, on\n the assumption that it will make sure that last_events is in a\n suitable range. It used to do this, but now it does not.\n So now we need to be more explicit about initialisation.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5fd3a17ed456637a224cf4ca82b9ad9d005bc8d4", "tree": "24906c956159349cddecc43b3479d2a9add364df", "parents": [ "fec6c6fec3e20637bee5d276fb61dd8b49a3f9cc" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Mar 04 00:57:25 2009 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Mar 04 00:57:25 2009 -0700" }, "message": "md: fix deadlock when stopping arrays\n\nResolve a deadlock when stopping redundant arrays, i.e. ones that\nrequire a call to sysfs_remove_group when shutdown. The deadlock is\nsummarized below:\n\nThread1 Thread2\n------- -------\nread sysfs attribute stop array\n take mddev lock\n sysfs_remove_group\nsysfs_get_active\nwait for mddev lock\n wait for active\n\nSysrq-w:\n--------\nmdmon S 00000017 2212 4163 1\n f1982ea8 00000046 2dcf6b85 00000017 c0b23100 f2f83ed0 c0b23100 f2f8413c\n c0b23100 c0b23100 c0b1fb98 f2f8413c 00000000 f2f8413c c0b23100 f2291ecc\n 00000002 c0b23100 00000000 00000017 f2f83ed0 f1982eac 00000046 c044d9dd\nCall Trace:\n [\u003cc044d9dd\u003e] ? debug_mutex_add_waiter+0x1d/0x58\n [\u003cc06ef451\u003e] __mutex_lock_common+0x1d9/0x338\n [\u003cc06ef451\u003e] ? __mutex_lock_common+0x1d9/0x338\n [\u003cc06ef5e3\u003e] mutex_lock_interruptible_nested+0x33/0x3a\n [\u003cc0634553\u003e] ? mddev_lock+0x14/0x16\n [\u003cc0634553\u003e] mddev_lock+0x14/0x16\n [\u003cc0634eda\u003e] md_attr_show+0x2a/0x49\n [\u003cc04e9997\u003e] sysfs_read_file+0x93/0xf9\nmdadm D 00000017 2812 4177 1\n f0401d78 00000046 430456f8 00000017 f0401d58 f0401d20 c0b23100 f2da2c4c\n c0b23100 c0b23100 c0b1fb98 f2da2c4c 0a10fc36 00000000 c0b23100 f0401d70\n 00000003 c0b23100 00000000 00000017 f2da29e0 00000001 00000002 00000000\nCall Trace:\n [\u003cc06eed1b\u003e] schedule_timeout+0x1b/0x95\n [\u003cc06eed1b\u003e] ? schedule_timeout+0x1b/0x95\n [\u003cc06eeb97\u003e] ? wait_for_common+0x34/0xdc\n [\u003cc044fa8a\u003e] ? trace_hardirqs_on_caller+0x18/0x145\n [\u003cc044fbc2\u003e] ? trace_hardirqs_on+0xb/0xd\n [\u003cc06eec03\u003e] wait_for_common+0xa0/0xdc\n [\u003cc0428c7c\u003e] ? default_wake_function+0x0/0x12\n [\u003cc06eeccc\u003e] wait_for_completion+0x17/0x19\n [\u003cc04ea620\u003e] sysfs_addrm_finish+0x19f/0x1d1\n [\u003cc04e920e\u003e] sysfs_hash_and_remove+0x42/0x55\n [\u003cc04eb4db\u003e] sysfs_remove_group+0x57/0x86\n [\u003cc0638086\u003e] do_md_stop+0x13a/0x499\n\nThis has been there for a while, but is easier to trigger now that mdmon\nis closely watching sysfs.\n\nCc: \u003cstable@kernel.org\u003e\nReported-by: Jacek Danecki \u003cjacek.danecki@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "93dbb393503d53cd226e5e1f0088fe8f4dbaa2b8", "tree": "790365e207951cf6810e8995f3141ddc0b74519b", "parents": [ "c1c201200a359cf3b6e2e36a4236cdca77a3cd8e" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Feb 16 10:25:40 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Feb 18 10:32:00 2009 +0100" }, "message": "block: fix bad definition of BIO_RW_SYNC\n\nWe can\u0027t OR shift values, so get rid of BIO_RW_SYNC and use BIO_RW_SYNCIO\nand BIO_RW_UNPLUG explicitly. This brings back the behaviour from before\n213d9417fec62ef4c3675621b9364a667954d4dd.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "de01dfadf25bf83cfe3d85c163005c4320532658", "tree": "dc1cca5a6aa10ecc0fea2fb9f13534685ba8ff61", "parents": [ "852c8bf484a0e17ee27f413ef26e87f522af5607" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Feb 06 18:02:46 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Feb 06 18:02:46 2009 +1100" }, "message": "md: Ensure an md array never has too many devices.\n\nEach different metadata format supported by md supports a\ndifferent maximum number of devices.\nWe really should be enforcing this maximum in the kernel, but\nwe aren\u0027t quite doing that properly.\n\nWe currently only enforce it at the \u0027hot_add\u0027 point, which is an\nolder interface which is not used by current userspace.\n\nWe need to also enforce it at \u0027add_new_disk\u0027 time for active arrays\nand at \u0027do_md_run\u0027 time when starting a new array.\n\nSo move the test from \u0027hot_add\u0027 into \u0027bind_rdev_to_array\u0027 which is\ncalled from both \u0027hot_add\u0027 and \u0027add_new_disk, and add a new\ntest in \u0027analyse_sbs\u0027 which is called from \u0027do_md_run\u0027.\n\nThis bug (or missing feature) has been around \"forever\" and so\nthe patch is suitable for any -stable that is currently maintained.\n\nCc: stable@kernel.org\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": "efeb53c0e57213e843b7ef3cc6ebcdea7d6186ac", "tree": "ef580effd8809fb0bc5d6ffa6f3b8abaa9180539", "parents": [ "d3374825ce57ba2214d375023979f6197ccc1385" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:10 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:10 2009 +1100" }, "message": "md: Allow md devices to be created by name.\n\nUsing sequential numbers to identify md devices is somewhat artificial.\nUsing names can be a lot more user-friendly.\n\nAlso, creating md devices by opening the device special file is a bit\nawkward.\n\nSo this patch provides a new option for creating and naming devices.\n\nWriting a name such as \"md_home\" to\n /sys/modules/md_mod/parameters/new_array\nwill cause an array with that name to be created. It will appear in\n/sys/block/ /proc/partitions and /proc/mdstat as \u0027md_home\u0027.\nIt will have an arbitrary minor number allocated.\n\nmd devices that a created by an open are destroyed on the last\nclose when the device is inactive.\nFor named md devices, they will not be destroyed until the array\nis explicitly stopped, either with the STOP_ARRAY ioctl or by\nwriting \u0027clear\u0027 to /sys/block/md_XXXX/md/array_state.\n\nThe name of the array must start \u0027md_\u0027 to avoid conflict with\nother devices.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d3374825ce57ba2214d375023979f6197ccc1385", "tree": "441ea927a7c702e4eadeafbac8be97d664bfb83b", "parents": [ "a21d15042d8cd736caf82c2bac564f3f93f3d017" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:10 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:10 2009 +1100" }, "message": "md: make devices disappear when they are no longer needed.\n\nCurrently md devices, once created, never disappear until the module\nis unloaded. This is essentially because the gendisk holds a\nreference to the mddev, and the mddev holds a reference to the\ngendisk, this a circular reference.\n\nIf we drop the reference from mddev to gendisk, then we need to ensure\nthat the mddev is destroyed when the gendisk is destroyed. However it\nis not possible to hook into the gendisk destruction process to enable\nthis.\n\nSo we drop the reference from the gendisk to the mddev and destroy the\ngendisk when the mddev gets destroyed. However this has a\ncomplication.\nBetween the call\n __blkdev_get-\u003eget_gendisk-\u003ekobj_lookup-\u003emd_probe\nand the call\n __blkdev_get-\u003emd_open\n\nthere is no obvious way to hold a reference on the mddev any more, so\nunless something is done, it will disappear and gendisk will be\ndestroyed prematurely.\n\nAlso, once we decide to destroy the mddev, there will be an unlockable\nmoment before the gendisk is unlinked (blk_unregister_region) during\nwhich a new reference to the gendisk can be created. We need to\nensure that this reference can not be used. i.e. the -\u003eopen must\nfail.\n\nSo:\n 1/ in md_probe we set a flag in the mddev (hold_active) which\n indicates that the array should be treated as active, even\n though there are no references, and no appearance of activity.\n This is cleared by md_release when the device is closed if it\n is no longer needed.\n This ensures that the gendisk will survive between md_probe and\n md_open.\n\n 2/ In md_open we check if the mddev we expect to open matches\n the gendisk that we did open.\n If there is a mismatch we return -ERESTARTSYS and modify\n __blkdev_get to retry from the top in that case.\n In the -ERESTARTSYS sys case we make sure to wait until\n the old gendisk (that we succeeded in opening) is really gone so\n we loop at most once.\n\nSome udev configurations will always open an md device when it first\nappears. If we allow an md device that was just created by an open\nto disappear on an immediate close, then this can race with such udev\nconfigurations and result in an infinite loop the device being opened\nand closed, then re-open due to the \u0027ADD\u0027 even from the first open,\nand then close and so on.\nSo we make sure an md device, once created by an open, remains active\nat least until some md \u0027ioctl\u0027 has been made on it. This means that\nall normal usage of md devices will allow them to disappear promptly\nwhen not needed, but the worst that an incorrect usage will do it\ncause an inactive md device to be left in existence (it can easily be\nremoved).\n\nAs an array can be stopped by writing to a sysfs attribute\n echo clear \u003e /sys/block/mdXXX/md/array_state\nwe need to use scheduled work for deleting the gendisk and other\nkobjects. This allows us to wait for any pending gendisk deletion to\ncomplete by simply calling flush_scheduled_work().\n\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a21d15042d8cd736caf82c2bac564f3f93f3d017", "tree": "d430b6ae126e9bc8139d1c491b80a18cc86b27f1", "parents": [ "8b76539823d71576927e3eb08b395eb6620f628d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:09 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:09 2009 +1100" }, "message": "md: centralise all freeing of an \u0027mddev\u0027 in \u0027md_free\u0027\n\nmd_free is the .release handler for the md kobj_type.\nSo it makes sense to release all the objects referenced by\nthe mddev in there, rather than just prior to calling kobject_put\nfor what we think is the last time.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8b76539823d71576927e3eb08b395eb6620f628d", "tree": "37f9c4da6c861a69b4695ea7818eab2cf6ce95f4", "parents": [ "cd2ac9321c26dc7a76455cd2a4df89123fa2b73e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "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: move allocation of -\u003equeue from mddev_find to md_probe\n\nIt is more balanced to just do simple initialisation in mddev_find,\nwhich allocates and links a new md device, and leave all the\nmore sophisticated allocation to md_probe (which calls mddev_find).\nmd_probe already allocated the gendisk. It should allocate the\nqueue too.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "cd2ac9321c26dc7a76455cd2a4df89123fa2b73e", "tree": "8163a2cc791282cf32fafed19a60f357882d6169", "parents": [ "159ec1fc060ab22b157a62364045f5e98749c4d3" ], "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: need another print_sb for mdp_superblock_1\n\nmd_print_devices is called in two code path: MD_BUG(...), and md_ioctl\nwith PRINT_RAID_DEBUG. it will dump out all in use md devices\ninformation;\n\nHowever, it wrongly processed two types of superblock in one:\n\nThe header file \u003clinux/raid/md_p.h\u003e has defined two types of superblock,\nstruct mdp_superblock_s (typedefed with mdp_super_t) according to md with\nmetadata 0.90, and struct mdp_superblock_1 according to md with metadata\n1.0 and later,\n\nThese two types of superblock are very different,\n\nThe md_print_devices code processed them both in mdp_super_t, that would\nlead to wrong informaton dump like:\n\n\t[ 6742.345877]\n\t[ 6742.345887] md:\t**********************************\n\t[ 6742.345890] md:\t* \u003cCOMPLETE RAID STATE PRINTOUT\u003e *\n\t[ 6742.345892] md:\t**********************************\n\t[ 6742.345896] md1: \u003cram7\u003e\u003cram6\u003e\u003cram5\u003e\u003cram4\u003e\n\t[ 6742.345907] md: rdev ram7, SZ:00065472 F:0 S:1 DN:3\n\t[ 6742.345909] md: rdev superblock:\n\t[ 6742.345914] md: SB: (V:0.90.0) ID:\u003c42ef13c7.598c059a.5f9f1645.801e9ee6\u003e CT:4919856d\n\t[ 6742.345918] md: L5 S00065472 ND:4 RD:4 md1 LO:2 CS:65536\n\t[ 6742.345922] md: UT:4919856d ST:1 AD:4 WD:4 FD:0 SD:0 CSUM:b7992907 E:00000001\n\t[ 6742.345924] D 0: DISK\u003cN:0,(1,8),R:0,S:6\u003e\n\t[ 6742.345930] D 1: DISK\u003cN:1,(1,10),R:1,S:6\u003e\n\t[ 6742.345933] D 2: DISK\u003cN:2,(1,12),R:2,S:6\u003e\n\t[ 6742.345937] D 3: DISK\u003cN:3,(1,14),R:3,S:6\u003e\n\t[ 6742.345942] md: THIS: DISK\u003cN:3,(1,14),R:3,S:6\u003e\n\t...\n\t[ 6742.346058] md0: \u003cram3\u003e\u003cram2\u003e\u003cram1\u003e\u003cram0\u003e\n\t[ 6742.346067] md: rdev ram3, SZ:00065472 F:0 S:1 DN:3\n\t[ 6742.346070] md: rdev superblock:\n\t[ 6742.346073] md: SB: (V:1.0.0) ID:\u003c369aad81.00000000.00000000.00000000\u003e CT:9a322a9c\n\t[ 6742.346077] md: L-1507699579 S976570180 ND:48 RD:0 md0 LO:65536 CS:196610\n\t[ 6742.346081] md: UT:00000018 ST:0 AD:131048 WD:0 FD:8 SD:0 CSUM:00000000 E:00000000\n\t[ 6742.346084] D 0: DISK\u003cN:-1,(-1,-1),R:-1,S:-1\u003e\n\t[ 6742.346089] D 1: DISK\u003cN:-1,(-1,-1),R:-1,S:-1\u003e\n\t[ 6742.346092] D 2: DISK\u003cN:-1,(-1,-1),R:-1,S:-1\u003e\n\t[ 6742.346096] D 3: DISK\u003cN:-1,(-1,-1),R:-1,S:-1\u003e\n\t[ 6742.346102] md: THIS: DISK\u003cN:0,(0,0),R:0,S:0\u003e\n\t...\n\t[ 6742.346219] md:\t**********************************\n\t[ 6742.346221]\n\nHere md1 is metadata 0.90.0, and md0 is metadata 1.2\n\nAfter some more code to distinguish these two types of superblock, in this patch,\n\nit will generate dump information like:\n\n\t[ 7906.755790]\n\t[ 7906.755799] md:\t**********************************\n\t[ 7906.755802] md:\t* \u003cCOMPLETE RAID STATE PRINTOUT\u003e *\n\t[ 7906.755804] md:\t**********************************\n\t[ 7906.755808] md1: \u003cram7\u003e\u003cram6\u003e\u003cram5\u003e\u003cram4\u003e\n\t[ 7906.755819] md: rdev ram7, SZ:00065472 F:0 S:1 DN:3\n\t[ 7906.755821] md: rdev superblock (MJ:0):\n\t[ 7906.755826] md: SB: (V:0.90.0) ID:\u003c3fca7a0d.a612bfed.5f9f1645.801e9ee6\u003e CT:491989f3\n\t[ 7906.755830] md: L5 S00065472 ND:4 RD:4 md1 LO:2 CS:65536\n\t[ 7906.755834] md: UT:491989f3 ST:1 AD:4 WD:4 FD:0 SD:0 CSUM:00fb52ad E:00000001\n\t[ 7906.755836] D 0: DISK\u003cN:0,(1,8),R:0,S:6\u003e\n\t[ 7906.755842] D 1: DISK\u003cN:1,(1,10),R:1,S:6\u003e\n\t[ 7906.755845] D 2: DISK\u003cN:2,(1,12),R:2,S:6\u003e\n\t[ 7906.755849] D 3: DISK\u003cN:3,(1,14),R:3,S:6\u003e\n\t[ 7906.755855] md: THIS: DISK\u003cN:3,(1,14),R:3,S:6\u003e\n\t...\n\t[ 7906.755972] md0: \u003cram3\u003e\u003cram2\u003e\u003cram1\u003e\u003cram0\u003e\n\t[ 7906.755981] md: rdev ram3, SZ:00065472 F:0 S:1 DN:3\n\t[ 7906.755984] md: rdev superblock (MJ:1):\n\t[ 7906.755989] md: SB: (V:1) (F:0) Array-ID:\u003c5fbcf158:55aa:5fbe:9a79:1e939880dcbd\u003e\n\t[ 7906.755990] md: Name: \"DG5:0\" CT:1226410480\n\t[ 7906.755998] md: L5 SZ130944 RD:4 LO:2 CS:128 DO:24 DS:131048 SO:8 RO:0\n\t[ 7906.755999] md: Dev:00000003 UUID: 9194d744:87f7:a448:85f2:7497b84ce30a\n\t[ 7906.756001] md: (F:0) UT:1226410480 Events:0 ResyncOffset:-1 CSUM:0dbcd829\n\t[ 7906.756003] md: (MaxDev:384)\n\t...\n\t[ 7906.756113] md:\t**********************************\n\t[ 7906.756116]\n\nthis md0 (metadata 1.2) information dumping is exactly according to struct\nmdp_superblock_1.\n\nSigned-off-by: Cheng Renquan \u003ccrquan@gmail.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: Dan Williams \u003cdan.j.williams@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\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": "0c3573f19d135d718264e38c46597295bd6154b7", "tree": "0943035f89d163d9d7b354445303fb03e5ea7fb1", "parents": [ "538452700d95480c16e7aa6b10ff77cd937d33f4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:05 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jan 09 08:31:05 2009 +1100" }, "message": "md: use sysfs_notify_dirent to notify changes to md/sync_action.\n\nThere is no compelling need for this, but sysfs_notify_dirent is a\nnicer interface and the change is good for consistency.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "cb3ac42b8af357fdd9ad838234245b39e5bdb7fe", "tree": "f6c354c2ec374fa35fe64902204d212f0ef30bd4", "parents": [ "45beca08dd8b6d6a65c5ffd730af2eac7a2c7a03" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Nov 06 17:28:01 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Nov 06 17:28:01 2008 +1100" }, "message": "md: revert the recent addition of a call to the BLKRRPART ioctl.\n\nIt turns out that it is only safe to call blkdev_ioctl when the device\nis actually open (as -\u003ebd_disk is set to NULL on last close). And it\nis quite possible for do_md_stop to be called when the device is not\nopen. So discard the call to blkdev_ioctl(BLKRRPART) which was\nadded in\n commit 934d9c23b4c7e31840a895ba4b7e88d6413c81f3\n\nIt is just as easy to call this ioctl from userspace when needed (on\nmdadm -S) so leave it out of the kernel\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "934d9c23b4c7e31840a895ba4b7e88d6413c81f3", "tree": "989d1ac0be656e51d44eaf8513076917bfed3adf", "parents": [ "f8d56f1771e4867acc461146764b4feeb5245669" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 28 17:01:23 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 28 17:01:23 2008 +1100" }, "message": "md: destroy partitions and notify udev when md array is stopped.\n\nmd arrays are not currently destroyed when they are stopped - they\nremain in /sys/block. Last time I tried this I tripped over locking\ntoo much.\n\nA consequence of this is that udev doesn\u0027t remove anything from /dev.\nThis is rather ugly.\n\nAs an interim measure until proper device removal can be achieved,\nmake sure all partitions are removed using the BLKRRPART ioctl, and\nsend a KOBJ_CHANGE when an md array is stopped.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f8d56f1771e4867acc461146764b4feeb5245669", "tree": "9d4857b72287f3170818b4b883c232e3ffb677af", "parents": [ "3d6eadcb5008beca1b289983ffd7771d1e947bac", "92850bbd71228730c80efd491e7427650188d359" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Oct 26 16:42:18 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Oct 26 16:42:18 2008 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md:\n md: allow extended partitions on md devices.\n md: use sysfs_notify_dirent to notify changes to md/dev-xxx/state\n md: use sysfs_notify_dirent to notify changes to md/array_state\n" }, { "commit": "9a1c3542768b5a58e45a9216921cd10a3bae1205", "tree": "c20ffda950db868ec7e1e35aed532962de2ecfd9", "parents": [ "511de73ff09034fb89c8d54bed201a10d057328c" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Feb 22 20:40:24 2008 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Oct 21 07:48:58 2008 -0400" }, "message": "[PATCH] pass fmode_t to blkdev_put()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "a39907fa2fdb73d3c0fb06eff06d855c3298d707", "tree": "d8be7f2fb0fe6ddf037e40b2cc958a6c07a32d95", "parents": [ "fe5f9f2cd57c2ce56f36c66e87a10d4b7a158505" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Mar 02 10:31:15 2008 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Oct 21 07:48:31 2008 -0400" }, "message": "[PATCH] switch md\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "d4430d62fa77208824a37fe6f85ab2831d274769", "tree": "5d4d0bca31e63eb208fbebe4f39c912b964c1e4d", "parents": [ "badf8082c33d18b118d3a6f1b32d5ea6b97d3839" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Mar 02 09:09:22 2008 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Oct 21 07:47:32 2008 -0400" }, "message": "[PATCH] beginning of methods conversion\n\nTo keep the size of changesets sane we split the switch by drivers;\nto keep the damn thing bisectable we do the following:\n\t1) rename the affected methods, add ones with correct\nprototypes, make (few) callers handle both. That\u0027s this changeset.\n\t2) for each driver convert to new methods. *ALL* drivers\nare converted in this series.\n\t3) kill the old (renamed) methods.\n\nNote that it _is_ a flagday; all in-tree drivers are converted and by the\nend of this series no trace of old methods remain. The only reason why\nwe do that this way is to keep the damn thing bisectable and allow per-driver\ndebugging if anything goes wrong.\n\nNew methods:\n\topen(bdev, mode)\n\trelease(disk, mode)\n\tioctl(bdev, mode, cmd, arg)\t\t/* Called without BKL */\n\tcompat_ioctl(bdev, mode, cmd, arg)\n\tlocked_ioctl(bdev, mode, cmd, arg)\t/* Called with BKL, legacy */\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "92850bbd71228730c80efd491e7427650188d359", "tree": "1d9d87d8eead78e2f6d37db62b580a8b7961844e", "parents": [ "3c0ee63a64a20351ed6c16ec797e1f8c850741ea" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:32 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:32 2008 +1100" }, "message": "md: allow extended partitions on md devices.\n\nThe new extended partition support provides a much nicer was\nto have partitions on md devices that the \u0027mdp\u0027 alternate major.\nWe cannot really get rid of \u0027mdp\u0027 at this time, but we can\nenable extended partitions as that will probably make life\neasier for sysadmins.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3c0ee63a64a20351ed6c16ec797e1f8c850741ea", "tree": "482b825a00356a2b7b605065ce1d6ebe87591eb5", "parents": [ "b62b75905d571c29262a6c38cf9e5f089c203871" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:28 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:28 2008 +1100" }, "message": "md: use sysfs_notify_dirent to notify changes to md/dev-xxx/state\n\nThe \u0027state\u0027 file for a device reports, for example, when the device\nhas failed. Changes should be reported to userspace ASAP without\nthe possibility of blocking on low-memory. sysfs_notify does\nhave that possibility (as it takes a mutex which can be held\nacross a kmalloc) so use sysfs_notify_dirent instead.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b62b75905d571c29262a6c38cf9e5f089c203871", "tree": "0ca9a74c1d53aad0532aaebebb031b9d8edc5d90", "parents": [ "0cfd81031a26717fe14380d18275f8e217571615" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:21 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Oct 21 13:25:21 2008 +1100" }, "message": "md: use sysfs_notify_dirent to notify changes to md/array_state\n\nNow that we have sysfs_notify_dirent, use it to notify changes\nto md/array_state.\nAs sysfs_notify_dirent can be called in atomic context, we can\nremove the delayed notify and the MD_NOTIFY_ARRAY_STATE flag.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "97ce0a7f9caf9d715cee815a016ee21575f71c95", "tree": "b35d4f84795a9b3f20a22faa43148a9c2c68d051", "parents": [ "08ff39f1c8f2134f7d0f38123ca5952371665cc5" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@gmail.com", "time": "Wed Sep 24 22:48:19 2008 -0700" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Oct 16 17:03:08 2008 +1100" }, "message": "md: fix input truncation in safe_delay_store()\n\nsafe_delay_store() currently truncates the last character of input since\nit tells strlcpy that the buffer can only hold \u0027len\u0027 characters, off by\none. sysfs already null terminates the buffer, so just increase the\nlast argument to strlcpy.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.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": "4bbf3771ca40d0aaec8316d0e7476b16010288e5", "tree": "f7cba1f72612c5d2132f549881c46ff572e01933", "parents": [ "d710e13812600037a723a673dc5c96a071de98d3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "message": "md: Relax minimum size restrictions on chunk_size.\n\nCurrently, the \u0027chunk_size\u0027 of an array must be at-least PAGE_SIZE.\n\nThis makes moving an array to a machine with a larger PAGE_SIZE, or\nchanging the kernel to use a larger PAGE_SIZE, can stop an array from\nworking.\n\nFor RAID10 and RAID4/5/6, this is non-trivial to fix as the resync\nprocess works on whole pages at a time, and assumes them to be wholly\nwithin a stripe. For other raid personalities, this restriction is\nnot needed at all and can be dropped.\n\nSo remove the test on chunk_size from common can, and add it in just\nthe places where it is needed: raid10 and raid4/5/6.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d710e13812600037a723a673dc5c96a071de98d3", "tree": "df8e34fb02cb7909090aa697da7c1db4814ee122", "parents": [ "fb4d8c76e56a887b9eee99fbc55fe82b18625d30" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "message": "md: remove space after function name in declaration and call.\n\nHaving\n function (args)\ninstead of\n function(args)\n\nmake is harder to search for calls of particular functions.\nSo remove all those spaces.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "fb4d8c76e56a887b9eee99fbc55fe82b18625d30", "tree": "0ca96887e40db671957b79d493830b9527b04949", "parents": [ "ab5bd5cbc8d4b868378d062eed3d4240930fbb86" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "message": "md: Remove unnecessary #includes, #defines, and function declarations.\n\nA lot of cruft has gathered over the years. Time to remove it.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "80268ee9270ebe4847365a7426de91d179e870d0", "tree": "0f5d10d1cc86b64030d036e6e322b23caf6aec5c", "parents": [ "ea43ddd8491feccf36267349748ea91b1194481e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:12 2008 +1100" }, "message": "md: Don\u0027t try to set an array to \u0027read-auto\u0027 if it is already in that state.\n\n\u0027read-auto\u0027 is a variant of \u0027readonly\u0027 which will switch to writable\non the first write attempt.\n\nCalling do_md_stop to set the array readonly when it is already readonly\nreturns an error. So make sure not to do that.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ea43ddd8491feccf36267349748ea91b1194481e", "tree": "89ca77c1f9794994eefa16b76f985b502326cfcd", "parents": [ "7d3c6f8717ee6c2bf6cba5fa0bda3b28fbda6015" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:11 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:11 2008 +1100" }, "message": "md: Allow metadata_version to be updated for externally managed metadata.\n\nFor externally managed metadata, the \u0027metadata_version\u0027 sysfs\nattribute is really just a channel for user-space programs to\ncommunicate about how the array is being managed.\nIt can be useful for this to be changed while the array is active.\n\nNormally changes to metadata_version are not permitted while the array\nis active. Change that so that if the metadata is externally managed,\nthe metadata_version can be changed to a different flavour of external\nmanagement.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7d3c6f8717ee6c2bf6cba5fa0bda3b28fbda6015", "tree": "118b13de811d5eca3e6e3e6410c667ca9cbba607", "parents": [ "fd048088306656824958e7783ffcee27e241b361" ], "author": { "name": "Chris Webb", "email": "chris@arachsys.com", "time": "Mon Oct 13 11:55:11 2008 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Oct 13 11:55:11 2008 +1100" }, "message": "md: Fix rdev_size_store with size \u003d\u003d 0\n\n\nFix rdev_size_store with size \u003d\u003d 0.\nsize \u003d\u003d 0 means to use the largest size allowed by the\nunderlying device and is used when modifying an active array.\n\nThis fixes a regression introduced by\n commit d7027458d68b2f1752a28016dcf2ffd0a7e8f567\n\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Chris Webb \u003cchris@arachsys.com\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": "0762b8bde9729f10f8e6249809660ff2ec3ad735", "tree": "12aa94ef16b840f713f833c712d335646c816dc1", "parents": [ "4c46501d1659475dc6c89554af6ce7fe6ecf615c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:56:12 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:08 2008 +0200" }, "message": "block: always set bdev-\u003ebd_part\n\nTill now, bdev-\u003ebd_part is set only if the bdev was for parts other\nthan part0. This patch makes bdev-\u003ebd_part always set so that code\npaths don\u0027t have to differenciate common handling.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "ed9e1982347b36573cd622ee5f4e2a7ccd79b3fd", "tree": "79d834094d655ec97cfc0a382a9207ebc8e711a5", "parents": [ "870d6656126add8e383645732b03df2b7ccd4f94" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:56:05 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:07 2008 +0200" }, "message": "block: implement and use {disk|part}_to_dev()\n\nImplement {disk|part}_to_dev() and use them to access generic device\ninstead of directly dereferencing {disk|part}-\u003edev. To make sure no\nuser is left behind, rename generic devices fields to __dev.\n\nThis is in preparation of unifying partition 0 handling with other\npartitions.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "9744197c3d7b329590c2be33ad7b17409bd798fe", "tree": "b82478694d768a54bad23661e31830cc48e95e0f", "parents": [ "45e9c0de2e86485f8b6633fd64ab19cfbff167f6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Sep 19 11:49:54 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Sep 19 11:49:54 2008 +1000" }, "message": "md: Don\u0027t wait UNINTERRUPTIBLE for other resync to finish\n\nWhen two md arrays share some block device (e.g each uses different\npartitions on the one device), a resync of one array will wait for\nthe resync on the other to finish.\n\nThis can be a long time and as it currently waits TASK_UNINTERRUPTIBLE,\nthe softlockup code notices and complains.\n\nSo use TASK_INTERRUPTIBLE instead and make sure to flush signals\nbefore calling schedule.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "271f5a9b8f8ae0db95de72779d115c9d0b9d3cc5", "tree": "814bfe03f1c5ae97da163c8f6b24a0cf67cfd783", "parents": [ "bef69ea0dcce574a425feb0a5aa4c63dd108b9a6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Sep 01 12:32:52 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Sep 01 12:32:52 2008 +1000" }, "message": "Remove invalidate_partition call from do_md_stop.\n\nWhen stopping an md array, or just switching to read-only, we\ncurrently call invalidate_partition while holding the mddev lock.\nThe main reason for this is probably to ensure all dirty buffers\nare flushed (invalidate_partition calls fsync_bdev).\n\nHowever if any dirty buffers are found, it will almost certainly cause\na deadlock as starting writeout will require an update to the\nsuperblock, and performing that updates requires taking the mddev\nlock - which is already held.\n\nThis deadlock can be demonstrated by running \"reboot -f -n\" with\na root filesystem on md/raid, and some dirty buffers in memory.\n\nAll other calls to stop an array should already happen after a flush.\nThe normal sequence is to stop using the array (e.g. umount) which\nwill cause __blkdev_put to call sync_blockdev. Then open the\narray and issue the STOP_ARRAY ioctl while the buffers are all still\nclean.\n\nSo this invalidate_partition is normally a no-op, except for one case\nwhere it will cause a deadlock.\n\nSo remove it.\n\nThis patch possibly addresses the regression recored in\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d11460\nand\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d11452\n\nthough it isn\u0027t yet clear how it ever worked.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "56ac36d722d0d27c03599d1245ac0ab59e474e5c", "tree": "e65f4d860549532a570b92995906f3c3dec9c529", "parents": [ "0310fa216decc3ecfab41f327638fa48a81f3735" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Aug 07 10:02:47 2008 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Aug 07 10:02:47 2008 -0700" }, "message": "md: cancel check/repair requests when recovery is needed\n\nIf a \u0027repair\u0027 is requested when an array is in a position to \u0027recover\u0027 raid1\nwill perform the repair while md believes a recovery is happening. Address\nthis at both ends, i.e. cancel check/repair requests upon detecting a\nrecover condition and do not call -\u003espare_active after completing a\ncheck/repair.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "c89a8eee61540df04fc83f32f51ef0f46ec018b1", "tree": "7bffba40357f654de8ed31bc6a0e7468daa61234", "parents": [ "ac4090d24c6a26211bc4523d920376e054d4f3f8" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:54:13 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:56:32 2008 +1000" }, "message": "Allow faulty devices to be removed from a readonly array.\n\nRemoving faulty devices from an array is a two stage process.\nFirst the device is moved from being a part of the active array\nto being similar to a spare device. Then it can be removed\nby a request from user space.\n\nThe first step is currently not performed for read-only arrays,\nso the second step can never succeed.\n\nSo allow readonly arrays to remove failed devices (which aren\u0027t\nblocked).\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "dba034eef2456d2a9f9a76806846c97acf6c3ad1", "tree": "3cf51b72fc5be671e986a39c389aa5332994e2a2", "parents": [ "2b25000bf5157c28d8591f03f0575248a8cbd900" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:54:13 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:56:32 2008 +1000" }, "message": "Fail safely when trying to grow an array with a write-intent bitmap.\n\nWe cannot currently change the size of a write-intent bitmap.\nSo if we change the size of an array which has such a bitmap, it\ntries to set bits beyond the end of the bitmap.\n\nFor now, simply reject any request to change the size of an array\nwhich has a bitmap. mdadm can remove the bitmap and add a new one\nafter the array has changed size.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b25000bf5157c28d8591f03f0575248a8cbd900", "tree": "f01260723348738076becc9f62872d87819f3159", "parents": [ "19052c0e85a3e9d3b7d190b29fcdbf0e6c105381" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:54:13 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:56:31 2008 +1000" }, "message": "Restore force switch of md array to readonly at reboot time.\n\nA recent patch allowed do_md_stop to know whether it was being called\nvia an ioctl or not, and thus where to allow for an extra open file\ndescriptor when checking if it is in use.\nThis broke then switch to readonly performed by the shutdown notifier,\nwhich needs to work even when the array is still (apparently) active\n(as md doesn\u0027t get told when the filesystem becomes readonly).\n\nSo restore this feature by pretending that there can be lots of\nfile descriptors open, but we still want do_md_stop to switch to\nreadonly.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "19052c0e85a3e9d3b7d190b29fcdbf0e6c105381", "tree": "6f61eacfe7ade13f0bf06adba5efac052e1038c0", "parents": [ "2b12a4c524812fb3f6ee590a02e65b95c8c32229" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:54:13 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 05 15:56:31 2008 +1000" }, "message": "Make writes to md/safe_mode_delay immediately effective.\n\nIf we reduce the \u0027safe_mode_delay\u0027, it could still wait for the old\ndelay to completely expire before doing anything about safe_mode.\nThus the effect if the change is delayed.\n\nTo make the effect more immediate, run the timeout function\nimmediately if the delay was reduced. This may cause it to run\nslightly earlier that required, but that is the safer option.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e542713529e323ff09d7aeb5806cf29f6f160f53", "tree": "751dffc2043342eb6f76b0c37a07a698462b5516", "parents": [ "df10cfbc4d7ab93260d997df754219d390d62a9d" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Jul 28 23:28:06 2008 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Jul 28 17:52:44 2008 -0700" }, "message": "md: do not count blocked devices as spares\n\nremove_and_add_spares() assumes that failed devices have been hot-removed\nfrom the array. Removal is skipped in the \u0027blocked\u0027 case so do not count a\ndevice in this state as \u0027spare\u0027.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "d8e64406a037a64444175730294e449c9e21f5ec", "tree": "5ac7525899ebc1d2fd9a83a6e19cd0b1b1b1711d", "parents": [ "2339788376e2d69a9154130e4dacd5b21ce63094" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Jul 23 13:09:48 2008 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Jul 23 13:09:48 2008 -0700" }, "message": "md: delay notification of \u0027active_idle\u0027 to the recovery thread\n\nsysfs_notify might sleep, so do not call it from md_safemode_timeout.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "4b80991c6cb9efa607bc4fd6f3ecdf5511c31bb0", "tree": "5e2ba7d509af245c29bdf04b00960cc367972c44", "parents": [ "f2ea68cf42aafdd93393b6b8b20fc3c2b5f4390c" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 17:05:25 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 17:05:25 2008 +1000" }, "message": "md: Protect access to mddev-\u003edisks list using RCU\n\nAll modifications and most access to the mddev-\u003edisks list are made\nunder the reconfig_mutex lock. However there are three places where\nthe list is walked without any locking. If a reconfig happens at this\ntime, havoc (and oops) can ensue.\n\nSo use RCU to protect these accesses:\n - wrap them in rcu_read_{,un}lock()\n - use list_for_each_entry_rcu\n - add to the list with list_add_rcu\n - delete from the list with list_del_rcu\n - delay the \u0027free\u0027 with call_rcu rather than schedule_work\n\nNote that export_rdev did a list_del_init on this list. In almost all\ncases the entry was not in the list anymore so it was a no-op and so\nsafe. It is no longer safe as after list_del_rcu we may not touch\nthe list_head.\nAn audit shows that export_rdev is called:\n - after unbind_rdev_from_array, in which case the delete has\n already been done,\n - after bind_rdev_to_array fails, in which case the delete isn\u0027t needed.\n - before the device has been put on a list at all (e.g. in\n add_new_disk where reading the superblock fails).\n - and in autorun devices after a failure when the device is on a\n different list.\n\nSo remove the list_del_init call from export_rdev, and add it back\nimmediately before the called to export_rdev for that last case.\n\nNote also that -\u003esame_set is sometimes used for lists other than\nmddev-\u003elist (e.g. candidates). In these cases rcu is not needed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f2ea68cf42aafdd93393b6b8b20fc3c2b5f4390c", "tree": "00b025b91898ff32dba742b5075dd290fec4dc91", "parents": [ "d6e2215052810678bc9782fd980b52706fc71f50" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 17:05:25 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 17:05:25 2008 +1000" }, "message": "md: only count actual openers as access which prevent a \u0027stop\u0027\n\nOpen isn\u0027t the only thing that increments -\u003eactive. e.g. reading\n/proc/mdstat will increment it briefly. So to avoid false positives\nin testing for concurrent access, introduce a new counter that counts\njust the number of times the md device it open.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\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": "15f4a5fdf3aa07b53f6a7969664741db5882e485", "tree": "1bd897fbbcf512d3c0c3b2b55633ae590558f314", "parents": [ "d07bd3bcc456228b56a790897162a634691fed9b" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Mon Jul 21 14:42:12 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 14:42:12 2008 +1000" }, "message": "md: Make super_type-\u003erdev_size_change() take sector-based sizes.\n\nAlso, change the type of the size parameter from unsigned long long to\nsector_t and rename it to num_sectors.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d07bd3bcc456228b56a790897162a634691fed9b", "tree": "cabb0feee45a7b3383fff4202adc35793c8bdbb1", "parents": [ "d7027458d68b2f1752a28016dcf2ffd0a7e8f567" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Mon Jul 21 14:42:07 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 14:42:07 2008 +1000" }, "message": "md: Fix check for overlapping devices.\n\nThe checks in overlaps() expect all parameters either in block-based\nor sector-based quantities. However, its single caller passes two\nrdev-\u003edata_offset arguments as well as two rdev-\u003esize arguments, the\nformer being sector counts while the latter are measured in 1K blocks.\n\nThis could cause rdev_size_store() to accept an invalid size from user\nspace. Fix it by passing only sector-based quantities to overlaps().\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d7027458d68b2f1752a28016dcf2ffd0a7e8f567", "tree": "03bb7d75f4951eba4eff21926d1ce71c6d728260", "parents": [ "7e93a89251d4ed7bd4475db62616ccd03ddfd01a" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Sat Jul 12 10:37:50 2008 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Jul 21 14:22:18 2008 +1000" }, "message": "md: Tidy up rdev_size_store a bit:\n\n - used strict_strtoull in place of simple_strtoull\n - use my_mddev in place of rdev-\u003emddev (they have the same value)\nand more significantly,\n - don\u0027t adjust mddev-\u003esize to fit, rather reject changes which make\n rdev-\u003esize smaller than mddev-\u003esize\n\nAdjusting mddev-\u003esize is a hangover from bind_rdev_to_array which\ndoes a similar thing. But it really is a better design to insist that\nmddev-\u003esize is set as required, then the rdev-\u003esizes are set to allow\nfor that. The previous way invites confusion.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "0f420358e3a2abc028320ace7783e2e38cae77bf", "tree": "99b068212ad1dd8c5e97b151d796d66f53d1f3bb", "parents": [ "b73df2d3d629aefa187a0a3574fd81455e026bc8" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "message": "md: Turn rdev-\u003esb_offset into a sector-based quantity.\n\nRename it to sb_start to make sure all users have been converted.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "b73df2d3d629aefa187a0a3574fd81455e026bc8", "tree": "87e2a32c602c99cf69c298d3ed6e145a6202b481", "parents": [ "e7debaa4951b37d6c9ace4c6b984cd4805c5bfbb" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "message": "md: Make calc_dev_sboffset() return a sector count.\n\nAs BLOCK_SIZE_BITS is 10 and\n\n\tMD_NEW_SIZE_SECTORS(2 * x) \u003d 2 * NEW_SIZE_BLOCKS(x),\n\nthe return value of calc_dev_sboffset() doubles. Fix up all three\ncallers accordingly.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "e7debaa4951b37d6c9ace4c6b984cd4805c5bfbb", "tree": "9d61e2341af2161c0298c1396e6a85a1d4437eef", "parents": [ "d71f9f88d74166dcdef743a057f9222d64d2d509" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:23 2008 +1000" }, "message": "md: Replace calc_dev_size() by calc_num_sectors().\n\nNumber of sectors is the preferred unit for sizes of raid devices,\nso change calc_dev_size() so that it returns this unit instead of\nthe number of 1K blocks.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "d71f9f88d74166dcdef743a057f9222d64d2d509", "tree": "af2c0a93855347bce2fc24ecffa27be035bc8ef0", "parents": [ "df5b20cf68f9c90204c5fd36b7b090635cee3cdf" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Fri Jul 11 22:02:22 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:22 2008 +1000" }, "message": "md: Make update_size() take the number of sectors.\n\nChanging the internal representations of sizes of raid devices\nfrom 1K blocks to sector counts (512B units) is desirable because\nit allows to get rid of many divisions/multiplications and unnecessary\ncasts that are present in the current code.\n\nThis patch is a first step in this direction. It replaces the old\n1K-based \"size\" argument of update_size() by \"num_sectors\" and\nfixes up its two callers.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" }, { "commit": "df5b20cf68f9c90204c5fd36b7b090635cee3cdf", "tree": "d3d5690609085352485e8ddcc1650833df4be224", "parents": [ "26ef379f53993b1da3c19b63257cd47e1d9cd672" ], "author": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:22 2008 +1000" }, "committer": { "name": "Neil Brown", "email": "neilb@suse.de", "time": "Fri Jul 11 22:02:22 2008 +1000" }, "message": "md: Better control of when do_md_stop is allowed to stop the array.\n\ndo_md_stop check the number of active users before allowing the array\nto be stopped.\nTwo problems:\n 1/ it assumes the request is coming through an open file descriptor\n (via ioctl) so it allows for that. This is not always the case.\n 2/ it doesn\u0027t do the check it the array hasn\u0027t been activated.\n This is not good for cases when we use an inactive array to hold\n some devices in a container.\n\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\n" } ], "next": "26ef379f53993b1da3c19b63257cd47e1d9cd672" }