)]}' { "log": [ { "commit": "ae0b7448e91353ea5f821601a055aca6b58042cd", "tree": "a5c5d0532c808fb1eb2c01414edfcd2d0c039a7e", "parents": [ "2defcc3fb4661e7351cb2ac48d843efc4c64db13" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Sep 04 20:40:43 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:43 2009 +0100" }, "message": "dm snapshot: fix on disk chunk size validation\n\nFix some problems seen in the chunk size processing when activating a\npre-existing snapshot.\n\nFor a new snapshot, the chunk size can either be supplied by the creator\nor a default value can be used. For an existing snapshot, the\nchunk size in the snapshot header on disk should always be used.\n\nIf someone attempts to load an existing snapshot and has the \u0027default\nchunk size\u0027 option set, the kernel uses its default value even when it\nis incorrect for the snapshot being loaded. This patch ensures the\ncorrect on-disk value is always used.\n\nSecondly, when the code does use the chunk size stored on the disk it is\nprudent to revalidate it, so the code can exit cleanly if it got\ncorrupted as happened in\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d461506 .\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "2defcc3fb4661e7351cb2ac48d843efc4c64db13", "tree": "18be3f5f6c38f8b8b535f05e6004090cd9143af6", "parents": [ "61578dcd3fafe6babd72e8db32110cc0b630a432" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Sep 04 20:40:41 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:41 2009 +0100" }, "message": "dm exception store: split set_chunk_size\n\nBreak the function set_chunk_size to two functions in preparation for\nthe fix in the following patch.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "61578dcd3fafe6babd72e8db32110cc0b630a432", "tree": "8e5bb9e66ec43666dd5079cc9203fc56c233b469", "parents": [ "02d2fd31defce6ff77146ad0fef4f19006055d86" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Sep 04 20:40:39 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:39 2009 +0100" }, "message": "dm snapshot: fix header corruption race on invalidation\n\nIf a persistent snapshot fills up, a race can corrupt the on-disk header\nwhich causes a crash on any future attempt to activate the snapshot\n(typically while booting). This patch fixes the race.\n\nWhen the snapshot overflows, __invalidate_snapshot is called, which calls\nsnapshot store method drop_snapshot. It goes to persistent_drop_snapshot that\ncalls write_header. write_header constructs the new header in the \"area\"\nlocation.\n\nConcurrently, an existing kcopyd job may finish, call copy_callback\nand commit_exception method, that goes to persistent_commit_exception.\npersistent_commit_exception doesn\u0027t do locking, relying on the fact that\ncallbacks are single-threaded, but it can race with snapshot invalidation and\noverwrite the header that is just being written while the snapshot is being\ninvalidated.\n\nThe result of this race is a corrupted header being written that can\nlead to a crash on further reactivation (if chunk_size is zero in the\ncorrupted header).\n\nThe fix is to use separate memory areas for each.\n\nSee the bug: https://bugzilla.redhat.com/show_bug.cgi?id\u003d461506\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "02d2fd31defce6ff77146ad0fef4f19006055d86", "tree": "de8103e62595fbc1efcd39ff83bf146018e5180b", "parents": [ "7ec23d50949d5062b5b749638dd9380ed75e58e5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Sep 04 20:40:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:37 2009 +0100" }, "message": "dm snapshot: refactor zero_disk_area to use chunk_io\n\nRefactor chunk_io to prepare for the fix in the following patch.\n\nPass an area pointer to chunk_io and simplify zero_disk_area to use\nchunk_io. No functional change.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "7ec23d50949d5062b5b749638dd9380ed75e58e5", "tree": "28183b6468683cf28abd63b9c322b9f31bcddc5e", "parents": [ "d2b698644c97cb033261536a4f2010924a00eac9" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Sep 04 20:40:34 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:34 2009 +0100" }, "message": "dm log: userspace add luid to distinguish between concurrent log instances\n\nDevice-mapper userspace logs (like the clustered log) are\nidentified by a universally unique identifier (UUID). This\nidentifier is used to associate requests from the kernel to\na specific log in userspace. The UUID must be unique everywhere,\nsince multiple machines may use this identifier when communicating\nabout a particular log, as is the case for cluster logs.\n\nSometimes, device-mapper/LVM may re-use a UUID. This is the\ncase during pvmoves, when moving from one segment of an LV\nto another, or when resizing a mirror, etc. In these cases,\na new log is created with the same UUID and loaded in the\n\"inactive\" slot. When a device-mapper \"resume\" is issued,\nthe \"live\" table is deactivated and the new \"inactive\" table\nbecomes \"live\". (The \"inactive\" table can also be removed\nvia a device-mapper \u0027clear\u0027 command.)\n\nThe above two issues were colliding. More than one log was being\ncreated with the same UUID, and there was no way to distinguish\nbetween them. So, sometimes the wrong log would be swapped\nout during the exchange.\n\nThe solution is to create a locally unique identifier,\n\u0027luid\u0027, to go along with the UUID. This new identifier is used\nto determine exactly which log is being referenced by the kernel\nwhen the log exchange is made. The identifier is not\nuniversally safe, but it does not need to be, since\ncreate/destroy/suspend/resume operations are bound to a specific\nmachine; and these are the operations that make up the exchange.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "d2b698644c97cb033261536a4f2010924a00eac9", "tree": "32192650a01c913a85953c0a756fca06320fef46", "parents": [ "b8313b6da7e2e7c7f47d93d8561969a3ff9ba0ea" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Sep 04 20:40:32 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:32 2009 +0100" }, "message": "dm raid1: do not allow log_failure variable to unset after being set\n\nThis patch fixes a bug which was triggering a case where the primary leg\ncould not be changed on failure even when the mirror was in-sync.\n\nThe case involves the failure of the primary device along with\nthe transient failure of the log device. The problem is that\nbios can be put on the \u0027failures\u0027 list (due to log failure)\nbefore \u0027fail_mirror\u0027 is called due to the primary device failure.\nNormally, this is fine, but if the log device failure is transient,\na subsequent iteration of the work thread, \u0027do_mirror\u0027, will\nreset \u0027log_failure\u0027. The \u0027do_failures\u0027 function then resets\nthe \u0027in_sync\u0027 variable when processing bios on the failures list.\nThe \u0027in_sync\u0027 variable is what is used to determine if the\nprimary device can be switched in the event of a failure. Since\nthis has been reset, the primary device is incorrectly assumed\nto be not switchable.\n\nThe case has been seen in the cluster mirror context, where one\nmachine realizes the log device is dead before the other machines.\nAs the responsibilities of the server migrate from one node to\nanother (because the mirror is being reconfigured due to the failure),\nthe new server may think for a moment that the log device is fine -\nthus resetting the \u0027log_failure\u0027 variable.\n\nIn any case, it is inappropiate for us to reset the \u0027log_failure\u0027\nvariable. The above bug simply illustrates that it can actually\nhurt us.\n\nCc: stable@kernel.org\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "b8313b6da7e2e7c7f47d93d8561969a3ff9ba0ea", "tree": "c605b02cb6ad5c4b68d4066eca4aac6c4747e839", "parents": [ "4142a969175302bc843d1505133488bfdbfa4732" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Sep 04 20:40:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:30 2009 +0100" }, "message": "dm log: remove incorrect field from userspace table output\n\nThe output of \u0027dmsetup table\u0027 includes an internal field that should not\nbe there. This patch removes it. To make the fix simpler, we first\nreorder a constructor argument\n\nThe \u0027device size\u0027 argument is generated internally. Currently it is\nplaced as the last space-separated word of the constructor string.\nHowever, we need to use a version of the string without this word, so we\nmove it to the beginning instead so it is trivial to skip past it.\n\nWe keep a copy of the arguments passed to userspace for creating a log,\njust in case we need to resend them. These are the same arguments that\nare desired in the STATUSTYPE_TABLE request, except for one. When\ncreating the userspace log, the userspace daemon must know the size of\nthe mirror, so that is added to the arguments given in the constructor\ntable. We were printing this extra argument out as well, which is a\nmistake.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "4142a969175302bc843d1505133488bfdbfa4732", "tree": "737cce5d0dc543dacefd7bc2fa27fc43b118318a", "parents": [ "40bea431274c247425e7f5970d796ff7b37a2b22" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Fri Sep 04 20:40:28 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:28 2009 +0100" }, "message": "dm log: fix userspace status output\n\nFix \u0027dmsetup table\u0027 output.\n\nThere is a missing \u0027 \u0027 at the end of the string causing two\nwords to run together.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "40bea431274c247425e7f5970d796ff7b37a2b22", "tree": "cc9791861c01605d2dca28d3eac9182c301a832c", "parents": [ "a963a956225eb0f8c4d3537f428153c30adf54b8" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Sep 04 20:40:25 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:25 2009 +0100" }, "message": "dm stripe: expose correct io hints\n\nSet sensible I/O hints for striped DM devices in the topology\ninfrastructure added for 2.6.31 for userspace tools to\nobtain via sysfs.\n\nAdd .io_hints to \u0027struct target_type\u0027 to allow the I/O hints portion\n(io_min and io_opt) of the \u0027struct queue_limits\u0027 to be set by each\ntarget and implement this for dm-stripe.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a963a956225eb0f8c4d3537f428153c30adf54b8", "tree": "d4204f64aaf23b7a1615f4f293358eca5125c748", "parents": [ "f6a1ed10864b7540fa758bbccf3433fe17070329" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Sep 04 20:40:24 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:24 2009 +0100" }, "message": "dm table: add more context to terse warning messages\n\nA couple of recent warning messages make it difficult for the reader to\ndetermine exactly what is wrong. This patch adds more information to\nthose messages.\n\nThe messages were added by these commits:\n 5dea271b6d87bd1d79a59c1d5baac2596a841c37 (\"dm table: pass correct dev area size\nto device_area_is_valid\")\n ea9df47cc92573b159ef3b4fda516c32cba9c4fd (\"dm table: fix blk_stack_limits arg\nto use bytes not sectors\")\n\nThe patch also corrects references to logical_block_size in printk format\nstrings from %hu to %u.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f6a1ed10864b7540fa758bbccf3433fe17070329", "tree": "e0b8f1c02f0527e0870dee2977f311695856f8a5", "parents": [ "8811f46c1f9386fc7017150de9d52359e5b1826e" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Sep 04 20:40:22 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:22 2009 +0100" }, "message": "dm table: fix queue_limit checking device iterator\n\nThe logic to check for valid device areas is inverted relative to proper\nuse with iterate_devices.\n\nThe iterate_devices method calls its callback for every underlying\ndevice in the target. If any callback returns non-zero, iterate_devices\nexits immediately. But the callback device_area_is_valid() returns 0 on\nerror and 1 on success. The overall effect without is that an error is\nissued only if every device is invalid.\n\nThis patch renames device_area_is_valid to device_area_is_invalid and\ninverts the logic so that one invalid device is sufficient to raise\nan error.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "8811f46c1f9386fc7017150de9d52359e5b1826e", "tree": "d7ff12ec29d2911801e81b50440f160d58b0f9a8", "parents": [ "a77e28c7e1dc1a6a035c7627d4a88ecf3ea09aea" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Sep 04 20:40:19 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:19 2009 +0100" }, "message": "dm snapshot: implement iterate devices\n\nImplement the .iterate_devices for the origin and snapshot targets.\ndm-snapshot\u0027s lack of .iterate_devices resulted in the inability to\nproperly establish queue_limits for both targets.\n\nWith 4K sector drives: an unfortunate side-effect of not establishing\nproper limits in either targets\u0027 DM device was that IO to the devices\nwould fail even though both had been created without error.\n\nCommit af4874e03ed82f050d5872d8c39ce64bf16b5c38 (\"dm target:s introduce\niterate devices fn\") in 2.6.31-rc1 should have implemented .iterate_devices\nfor dm-snap.c\u0027s origin and snapshot targets.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a77e28c7e1dc1a6a035c7627d4a88ecf3ea09aea", "tree": "0d481ef49120eda629409cbac9dbe63ba472f873", "parents": [ "37d0892c5a94e208cf863e3b7bac014edee4346d" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Fri Sep 04 20:40:16 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Fri Sep 04 20:40:16 2009 +0100" }, "message": "dm multipath: fix oops when request based io fails when no paths\n\nThe patch posted at http://marc.info/?l\u003ddm-devel\u0026m\u003d124539787228784\u0026w\u003d2\nwhich was merged into cec47e3d4a861e1d942b3a580d0bbef2700d2bb2 (\"dm:\nprepare for request based option\") introduced a regression in\nrequest-based dm.\n\nIf map_request() calls dm_kill_unmapped_request() to complete a cloned\nbio without dispatching it, clone-\u003ebio is still set when\ndm_end_request() is called and the BUG_ON(clone-\u003ebio) is incorrect.\n\nThe patch fixes this bug by freeing bio in dm_end_request() if the clone\nhas bio. I\u0027ve redone my tests to cover all I/O paths and confirmed\nthere\u0027s no other regression.\n\nHere is the oops I hit in request-based dm when I do I/O to a multipath\ndevice which doesn\u0027t have any active path nor queue_if_no_path setting:\n\n------------[ cut here ]------------\nkernel BUG at /root/2.6.31-rc4.rqdm/drivers/md/dm.c:828!\ninvalid opcode: 0000 [#1] SMP\nlast sysfs file: /sys/devices/system/cpu/cpu3/cache/index2/shared_cpu_map\nCPU 1\nModules linked in: autofs4 sunrpc cpufreq_ondemand acpi_cpufreq dm_mirror dm_region_hash dm_log dm_service_time dm_multipath scsi_dh dm_mod video output sbs sbshc battery ac sg sr_mod e1000e button cdrom serio_raw rtc_cmos rtc_core rtc_lib piix lpfc scsi_transport_fc ata_piix libata megaraid_sas sd_mod scsi_mod crc_t10dif ext3 jbd uhci_hcd ohci_hcd ehci_hcd [last unloaded: microcode]\nPid: 7, comm: ksoftirqd/1 Not tainted 2.6.31-rc4.rqdm #1 Express5800/120Lj [N8100-1417]\nRIP: 0010:[\u003cffffffffa023629d\u003e] [\u003cffffffffa023629d\u003e] dm_softirq_done+0xbd/0x100 [dm_mod]\nRSP: 0018:ffff8800280a1f08 EFLAGS: 00010282\nRAX: ffffffffa02544e0 RBX: ffff8802aa1111d0 RCX: ffff8802aa1111e0\nRDX: ffff8802ab913e70 RSI: 0000000000000000 RDI: ffff8802ab913e70\nRBP: ffff8800280a1f28 R08: ffffc90005457040 R09: 0000000000000000\nR10: 0000000000000001 R11: 0000000000000000 R12: 00000000fffffffb\nR13: ffff8802ab913e88 R14: ffff8802ab9c1438 R15: 0000000000000100\nFS: 0000000000000000(0000) GS:ffff88002809e000(0000) knlGS:0000000000000000\nCS: 0010 DS: 0018 ES: 0018 CR0: 000000008005003b\nCR2: 0000003d54a98640 CR3: 000000029f0a1000 CR4: 00000000000006e0\nDR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000\nDR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400\nProcess ksoftirqd/1 (pid: 7, threadinfo ffff8802ae50e000, task ffff8802ae4f8040)\nStack:\n ffff8800280a1f38 0000000000000020 ffffffff814f30a0 0000000000000004\n\u003c0\u003e ffff8800280a1f58 ffffffff8116b245 ffff8800280a1f38 ffff8800280a1f38\n\u003c0\u003e ffff8800280a1f58 0000000000000001 ffff8800280a1fa8 ffffffff810477bc\nCall Trace:\n \u003cIRQ\u003e\n [\u003cffffffff8116b245\u003e] blk_done_softirq+0x75/0x90\n [\u003cffffffff810477bc\u003e] __do_softirq+0xcc/0x210\n [\u003cffffffff81047170\u003e] ? ksoftirqd+0x0/0x110\n [\u003cffffffff8100ce7c\u003e] call_softirq+0x1c/0x50\n \u003cEOI\u003e\n [\u003cffffffff8100e785\u003e] do_softirq+0x65/0xa0\n [\u003cffffffff81047170\u003e] ? ksoftirqd+0x0/0x110\n [\u003cffffffff810471e0\u003e] ksoftirqd+0x70/0x110\n [\u003cffffffff81059559\u003e] kthread+0x99/0xb0\n [\u003cffffffff8100cd7a\u003e] child_rip+0xa/0x20\n [\u003cffffffff8100c73c\u003e] ? restore_args+0x0/0x30\n [\u003cffffffff810594c0\u003e] ? kthread+0x0/0xb0\n [\u003cffffffff8100cd70\u003e] ? child_rip+0x0/0x20\nCode: 44 89 e6 48 89 df e8 23 fb f2 e0 be 01 00 00 00 4c 89 f7 e8 f6 fd ff ff 5b 41 5c 41 5d 41 5e c9 c3 4c 89 ef e8 85 fe ff ff eb ed \u003c0f\u003e 0b eb fe 41 8b 85 dc 00 00 00 48 83 bb 10 01 00 00 00 89 83\nRIP [\u003cffffffffa023629d\u003e] dm_softirq_done+0xbd/0x100 [dm_mod]\n RSP \u003cffff8800280a1f08\u003e\n---[ end trace 16af0a1d8542da55 ]---\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "435a71d9ef68b03343949c814986e01dae849763", "tree": "04fb8dee9d5bd751da1cc8cbeb0e5a5f91c72ffc", "parents": [ "df4ecf1524c7793de3121b2d4e5fc6bcc0da3bfb", "80ffb3cceaefa405f2ecd46d66500ed8d53efe74" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 18 13:54:08 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 18 13:54:08 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:\n Fix new incorrect error return from do_md_stop.\n" }, { "commit": "80ffb3cceaefa405f2ecd46d66500ed8d53efe74", "tree": "867d6ec951663255076ea42fab29a31eac4e51c0", "parents": [ "4d484a4a7a5126410eed5f8dd329a33f6eeed068" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 18 10:35:26 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Aug 18 10:35:26 2009 +1000" }, "message": "Fix new incorrect error return from do_md_stop.\n\nRecent commit c8c00a6915a2e3d10416e8bdd3138429beb96210\nchanged the exit paths in do_md_stop and was not quite\ncareful enough. There is one path were \u0027err\u0027 now needs\nto be cleared but it isn\u0027t.\nSo setting an array to readonly (with mdadm --readonly) will\nwork, but will incorrectly report and error: ENXIO.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "894ef820b10d77e2d6d717342fc408bdd9825139", "tree": "4d79204fdee6bc23a555503fc224f7e13a10e304", "parents": [ "b2add73dbf93fd50f00564d7abc3e2b9aa9dd20c" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Sun Aug 16 07:33:30 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Aug 16 08:35:58 2009 -0700" }, "message": "dm-log-userspace: fix printk format warning\n\ndrivers/md/dm-log-userspace-transfer.c:110: warning: format \u0027%lu\u0027 expects type \u0027long unsigned int\u0027, but argument 4 has type \u0027size_t\u0027\n\nPreviously posted and acked, but apparently lost.\nhttp://lkml.indiana.edu/hypermail/linux/kernel/0906.2/02074.html\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nCc: dm-devel@redhat.com\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4d484a4a7a5126410eed5f8dd329a33f6eeed068", "tree": "9fe49a23117adc2d475711f39a16c1718bab4b7f", "parents": [ "1a67dde0abba36421a1257d01ba9de2f6d1c160a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:41:50 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:41:50 2009 +1000" }, "message": "md: allow upper limit for resync/reshape to be set when array is read-only\n\nNormally we only allow the upper limit for a reshape to be decreased\nwhen the array not performing a sync/recovery/reshape, otherwise there\ncould be races. But if an array is part-way through a reshape when it\nis assembled the reshape is started immediately leaving no window\nto set an upper bound.\n\nIf the array is started read-only, the reshape will be suspended until\nthe array becomes writable, so that provides a window during which it\nis perfectly safe to reduce the upper limit of a reshape.\n\nSo: allow the upper limit (sync_max) to be reduced even if the reshape\nthread is running, as long as the array is still read-only.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "1a67dde0abba36421a1257d01ba9de2f6d1c160a", "tree": "c182c31207c46824097be7ce6b8813f05db4eb65", "parents": [ "a639755cf885e437b2fe4168d35157fa90d530ab" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:41:49 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:41:49 2009 +1000" }, "message": "md/raid5: Properly remove excess drives after shrinking a raid5/6\n\nWe were removing the drives, from the array, but not\nremoving symlinks from /sys/.... and not marking the device\nas having been removed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a639755cf885e437b2fe4168d35157fa90d530ab", "tree": "01d9a04a1c1c0b16863a51ec389470e436e9fb13", "parents": [ "67ac6011db5d2b0c853d573ff474b25c85dfb644" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:13:00 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:13:00 2009 +1000" }, "message": "md/raid5: make sure a reshape restarts at the correct address.\n\nThis \"if\" don\u0027t allow for the possibility that the number of devices\ndoesn\u0027t change, and so sector_nr isn\u0027t set correctly in that case.\nSo change \u0027\u003e\u0027 to \u0027\u003e\u003d\u0027.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "67ac6011db5d2b0c853d573ff474b25c85dfb644", "tree": "29b30e5e4eac0564c34e0b0b3d655383ba9c783f", "parents": [ "51d5668cb2e3fd1827a55184e48606fff054c5be" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:06:24 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 10:06:24 2009 +1000" }, "message": "md/raid5: allow new reshape modes to be restarted in the middle.\n\nmd/raid5 doesn\u0027t allow a reshape to restart if it involves writing\nover the same part of disk that it would be reading from.\nThis happens at the beginning of a reshape that increases the number\nof devices, at the end of a reshape that decreases the number of\ndevices, and continuously for a reshape that does not change the\nnumber of devices.\n\nThe current code is correct for the \"increase number of devices\"\ncase as the critical section at the start is handled by userspace\nperforming a backup.\n\nIt does not work for reducing the number of devices, or the\nno-change case.\nFor \u0027reducing\u0027, we need to invert the test. For no-change we cannot\nreally be sure things will be safe, so simply require the array\nto be read-only, which is how the user-space code which carefully\nstarts such arrays works.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "51d5668cb2e3fd1827a55184e48606fff054c5be", "tree": "117824cc3a25c04c939856578105c6c93af6ac8b", "parents": [ "c8c00a6915a2e3d10416e8bdd3138429beb96210" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 09:54:02 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Aug 13 09:54:02 2009 +1000" }, "message": "md: never advance \u0027events\u0027 counter by more than 1.\n\nWhen assembling arrays, md allows two devices to have different event\ncounts as long as the difference is only \u00271\u0027. This is to cope with\na system failure between updating the metadata on two difference\ndevices.\n\nHowever there are currently times when we update the event count by\n2. This was done to keep the event count even when the array is clean\nand odd when it is dirty, which allows us to avoid writing common\nupdate to spare devices and so allow those spares to go to sleep.\n\nThis is bad for the above reason. So change it to never increase by\ntwo. This means that the alignment between \u0027odd/even\u0027 and\n\u0027clean/dirty\u0027 might take a little longer to attain, but that is only a\nsmall cost. The spares will get a few more updates but that will\nstill be spared (;-) most updates and can still go to sleep.\n\nPrior to this patch there was a small chance that after a crash an\narray would fail to assemble due to the overly large event count\nmismatch.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c8c00a6915a2e3d10416e8bdd3138429beb96210", "tree": "ecf06a9e2b08edefe707da450b52a69f818ec7d6", "parents": [ "7b2aa037e878c939676675969983284a02958ae3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 10 12:50:52 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 10 12:50:52 2009 +1000" }, "message": "Remove deadlock potential in md_open\n\nA recent commit:\n commit 449aad3e25358812c43afc60918c5ad3819488e7\n\nintroduced the possibility of an A-B/B-A deadlock between\nbd_mutex and reconfig_mutex.\n\n__blkdev_get holds bd_mutex while calling md_open which takes\n reconfig_mutex,\ndo_md_run is always called with reconfig_mutex held, and it now\n takes bd_mutex in the call the revalidate_disk.\n\nThis potential deadlock was not caught by lockdep due to the\nuse of mutex_lock_interruptible_nexted which was introduced\nby\n commit d63a5a74dee87883fda6b7d170244acaac5b05e8\ndo avoid a warning of an impossible deadlock.\n\nIt is quite possible to split reconfig_mutex in to two locks.\nOne protects the array data structures while it is being\nreconfigured, the other ensures that an array is never even partially\nopen while it is being deactivated.\nIn particular, the second lock prevents an open from completing\nbetween the time when do_md_stop checks if there are any active opens,\nand the time when the array is either set read-only, or when -\u003epers is\nset to NULL. So we can be certain that no IO is in flight as the\narray is being destroyed.\n\nSo create a new lock, open_mutex, just to ensure exclusion between\n\u0027open\u0027 and \u0027stop\u0027.\n\nThis avoids the deadlock and also avoids the lockdep warning mentioned\nin commit d63a5a74d\n\nReported-by: \"Mike Snitzer\" \u003csnitzer@gmail.com\u003e\nReported-by: \"H. Peter Anvin\" \u003chpa@zytor.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "449aad3e25358812c43afc60918c5ad3819488e7", "tree": "45fab3b82fc1ed06959537a0801319045cf4c102", "parents": [ "64bd660b51b2da92e99a5e97349f6558349f11c5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "message": "md: Use revalidate_disk to effect changes in size of device.\n\nAs revalidate_disk calls check_disk_size_change, it will cause\nany capacity change of a gendisk to be propagated to the blockdev\ninode. So use that instead of mucking about with locks and\ni_size_write.\n\nAlso add a call to revalidate_disk in do_md_run and a few other places\nwhere the gendisk capacity is changed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "64bd660b51b2da92e99a5e97349f6558349f11c5", "tree": "d35be79528dc8730b908e9d78357454ff22aee72", "parents": [ "e516402c0d4fc02be4af9fa8c18954d4f9deb44e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:58 2009 +1000" }, "message": "md: allow raid5_quiesce to work properly when reshape is happening.\n\nThe -\u003equiesce method is not supposed to stop resync/recovery/reshape,\njust normal IO.\nBut in raid5 we don\u0027t have a way to know which stripes are being\nused for normal IO and which for resync etc, so we need to wait for\nall stripes to be idle to be sure that all writes have completed.\n\nHowever reshape keeps at least some stripe busy for an extended period\nof time, so a call to raid5_quiesce can block for several seconds\nneedlessly.\nSo arrange for reshape etc to pause briefly while raid5_quiesce is\ntrying to quiesce the array so that the active_stripes count can\ndrop to zero.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e516402c0d4fc02be4af9fa8c18954d4f9deb44e", "tree": "8fa40d8f835d643ed903e1a89567a53256acc395", "parents": [ "70471dafe3390243c598a3165dfb86b8b8b3f4fe" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:57 2009 +1000" }, "message": "md/raid5: set reshape_position correctly when reshape starts.\n\nAs the internal reshape_progress counter is the main driver\nfor reshape, the fact that reshape_position sometimes starts with the\nwrong value has minimal effect. It is visible in sysfs and that\nis all.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "70471dafe3390243c598a3165dfb86b8b8b3f4fe", "tree": "febb8dfdbbb981370e1ea046714628bf6b790e5a", "parents": [ "3673f305faf1bc66ead751344f8262ace851ff44" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:57 2009 +1000" }, "message": "md: Handle growth of v1.x metadata correctly.\n\nThe v1.x metadata does not have a fixed size and can grow\nwhen devices are added.\nIf it grows enough to require an extra sector of storage,\nwe need to update the \u0027sb_size\u0027 to match.\n\nWithout this, md can write out an incomplete superblock with a\nbad checksum, which will be rejected when trying to re-assemble\nthe array.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3673f305faf1bc66ead751344f8262ace851ff44", "tree": "4cbdd23d9af20632678e95b3e8f02ede241a3917", "parents": [ "3a981b03f38dc3b8a69b77cbc679e66c1318a44a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:56 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:56 2009 +1000" }, "message": "md: avoid array overflow with bad v1.x metadata\n\nWe trust the \u0027desc_nr\u0027 field in v1.x metadata enough to use it\nas an index in an array. This isn\u0027t really safe.\nSo range-check the value first.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "3a981b03f38dc3b8a69b77cbc679e66c1318a44a", "tree": "7f8b0747385a02c793edd7df30d34c4c9c95b1a5", "parents": [ "ac5e7113e74872928844d00085bd47c988f12728" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:55 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:55 2009 +1000" }, "message": "md: when a level change reduces the number of devices, remove the excess.\n\nWhen an array is changed from RAID6 to RAID5, fewer drives are\nneeded. So any device that is made superfluous by the level\nconversion must be marked as not-active.\nFor the RAID6-\u003eRAID5 conversion, this will be a drive which only\nhas \u0027Q\u0027 blocks on it.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ac5e7113e74872928844d00085bd47c988f12728", "tree": "dabce6a99c2af9f00b33660f6b2d906cce836764", "parents": [ "95fc17aac45300f45968aacd97a536ddd8db8101" ], "author": { "name": "Andre Noll", "email": "maan@systemlinux.org", "time": "Mon Aug 03 10:59:47 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Aug 03 10:59:47 2009 +1000" }, "message": "md: Push down data integrity code to personalities.\n\nThis patch replaces md_integrity_check() by two new public functions:\nmd_integrity_register() and md_integrity_add_rdev() which are both\npersonality-independent.\n\nmd_integrity_register() is called from the -\u003erun and -\u003ehot_remove\nmethods of all personalities that support data integrity. The\nfunction iterates over the component devices of the array and\ndetermines if all active devices are integrity capable and if their\nprofiles match. If this is the case, the common profile is registered\nfor the mddev via blk_integrity_register().\n\nThe second new function, md_integrity_add_rdev() is called from the\n-\u003ehot_add_disk methods, i.e. whenever a new device is being added\nto a raid array. If the new device does not support data integrity,\nor has a profile different from the one already registered, data\nintegrity for the mddev is disabled.\n\nFor raid0 and linear, only the call to md_integrity_register() from\nthe -\u003erun method is necessary.\n\nSigned-off-by: Andre Noll \u003cmaan@systemlinux.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "95fc17aac45300f45968aacd97a536ddd8db8101", "tree": "9ca3004304165a17c2923f25ee1df196928ce858", "parents": [ "7d3e91b8a1f5179d56a7412d4b499f2d5fc6b25d" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Jul 31 12:39:15 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Jul 31 12:39:15 2009 +1000" }, "message": "md/raid6: release spare page at -\u003estop()\n\nAdd missing call to safe_put_page from stop() by unifying open coded\nraid5_conf_t de-allocation under free_conf().\n\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5dea271b6d87bd1d79a59c1d5baac2596a841c37", "tree": "c85bcee4f90729927285fc282c6df94bf18d023d", "parents": [ "a732c207d19e899845ae47139708af898daaf9fd" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Thu Jul 23 20:30:42 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Jul 23 20:30:42 2009 +0100" }, "message": "dm table: pass correct dev area size to device_area_is_valid\n\nIncorrect device area lengths are being passed to device_area_is_valid().\n\nThe regression appeared in 2.6.31-rc1 through commit\n754c5fc7ebb417b23601a6222a6005cc2e7f2913.\n\nWith the dm-stripe target, the size of the target (ti-\u003elen) was used\ninstead of the stripe_width (ti-\u003elen/#stripes). An example of a\nconsequent incorrect error message is:\n\n device-mapper: table: 254:0: sdb too small for target\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a732c207d19e899845ae47139708af898daaf9fd", "tree": "aed98221e373868b4fe0fbba9c4fcf5ac8ede128", "parents": [ "69885683d22d8c05910fd808c01fdce1322739b4" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Thu Jul 23 20:30:40 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Jul 23 20:30:40 2009 +0100" }, "message": "dm: remove queue next_ordered workaround for barriers\n\nThis patch removes DM\u0027s bio-based vs request-based conditional setting\nof next_ordered. For bio-based DM the next_ordered check is no longer a\nconcern (as that check is now in the __make_request path). For\nrequest-based DM the default of QUEUE_ORDERED_NONE is now appropriate.\n\nbio-based DM was changed to work-around the previously misplaced\nnext_ordered check with this commit:\n99360b4c18f7675b50d283301d46d755affe75fd\n\nrequest-based DM does not yet support barriers but reacted to the above\nbio-based DM change with this commit:\n5d67aa2366ccb8257d103d0b43df855605c3c086\n\nThe above changes are no longer needed given Neil Brown\u0027s recent fix to\nput the next_ordered check in the __make_request path:\ndb64f680ba4b5c56c4be59f0698000df89ff0281\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nCc: NeilBrown \u003cneilb@suse.de\u003e\nAcked-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nAcked-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "69885683d22d8c05910fd808c01fdce1322739b4", "tree": "ad890ab1f667b7528e347e1886a9b7daf6f6c168", "parents": [ "4be3bd7849165e7efa6b0b35a23d6a3598d97465" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Jul 23 20:30:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Jul 23 20:30:37 2009 +0100" }, "message": "dm raid1: wake kmirrord when requeueing delayed bios after remote recovery\n\nThe recent commit 7513c2a761d69d2a93f17146b3563527d3618ba0 (dm raid1:\nadd is_remote_recovering hook for clusters) changed do_writes() to\nupdate the ms-\u003ewrites list but forgot to wake up kmirrord to process it.\n\nThe rule is that when anything is being added on ms-\u003ereads, ms-\u003ewrites\nor ms-\u003efailures and the list was empty before we must call\nwakeup_mirrord (for immediate processing) or delayed_wake (for delayed\nprocessing). Otherwise the bios could sit on the list indefinitely.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nCC: stable@kernel.org\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "8aa7e847d834ed937a9ad37a0f2ad5b8584c1ab0", "tree": "76c8b4f1362a928d426f2201790ab5d128f57724", "parents": [ "c2cc49a2f8a479dde96a599646d30b6cc9dbed78" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Jul 09 14:52:32 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "axboe@carl.(none)", "time": "Fri Jul 10 20:31:53 2009 +0200" }, "message": "Fix congestion_wait() sync/async vs read/write confusion\n\nCommit 1faa16d22877f4839bd433547d770c676d1d964c accidentally broke\nthe bdi congestion wait queue logic, causing us to wait on congestion\nfor WRITE (\u003d\u003d 1) when we really wanted BLK_RW_ASYNC (\u003d\u003d 0) instead.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "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": "2027bd9f9290ede1664d5fa18b3fbd6c5f7401d3", "tree": "ba5a8fe8e2373744edc22974571a953840d6c9e8", "parents": [ "544ae5f96e14998cabc637fa20cf409eb92a0dd0", "b706f64281b24d8b1fdc8ae883700131d365c412" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jul 01 10:41:09 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jul 01 10:41:09 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block:\n cfq-iosched: remove redundant check for NULL cfqq in cfq_set_request()\n blocK: Restore barrier support for md and probably other virtual devices.\n block: get rid of queue-private command filter\n block: Create bip slabs with embedded integrity vectors\n cfq-iosched: get rid of the need for __GFP_NOFAIL in cfq_find_alloc_queue()\n cfq-iosched: move cfqq initialization out of cfq_find_alloc_queue()\n Trivial typo fixes in Documentation/block/data-integrity.txt.\n" }, { "commit": "544ae5f96e14998cabc637fa20cf409eb92a0dd0", "tree": "163523ded713a8b90ac68543979e127795ecbc58", "parents": [ "7b85425fac72588674d5c71604af618f690c91d7", "e62e58a5ffdc98ac28d8dbd070c857620d541f99" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jul 01 10:31:26 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jul 01 10:31:26 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:\n md: use interruptible wait when duration is controlled by userspace.\n md/raid5: suspend shouldn\u0027t affect read requests.\n md: tidy up error paths in md_alloc\n md: fix error path when duplicate name is found on md device creation.\n md: avoid dereferencing NULL pointer when accessing suspend_* sysfs attributes.\n md: Use new topology calls to indicate alignment and I/O sizes\n" }, { "commit": "7878cba9f0037f5599004b03a1260b32d9050360", "tree": "bff5e1a47b6e64e45df0428925cc6db8542cad62", "parents": [ "6118b70b3a0b4c583439bb77600194c82f220ce3" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Jun 26 15:37:49 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Jul 01 10:56:25 2009 +0200" }, "message": "block: Create bip slabs with embedded integrity vectors\n\nThis patch restores stacking ability to the block layer integrity\ninfrastructure by creating a set of dedicated bip slabs. Each bip slab\nhas an embedded bio_vec array at the end. This cuts down on memory\nallocations and also simplifies the code compared to the original bvec\nversion. Only the largest bip slab is backed by a mempool. The pool is\ncontained in the bio_set so stacking drivers can ensure forward\nprogress.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003caxboe@carl.(none)\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": "a5c308d4d1659b1f4833b863394e3e24cdbdfc6e", "tree": "6affde6f3f03c01f651210cc042a07ba70533d03", "parents": [ "0909dc448c98ed5021c87ffdfc09fb473aa464ab" ], "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/raid5: suspend shouldn\u0027t affect read requests.\n\nmd allows write to regions on an array to be suspended temporarily.\nThis allows user-space to participate is aspects of reshape.\nIn particular, data can be copied with not risk of a race.\nWe should not be blocking read requests though, so don\u0027t.\n\nCc: stable@kernel.org\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": "8f6c2e4b325a8e9f8f47febb2fd0ed4fae7d45a9", "tree": "6e383e2ec48b5c90fe07325a7f6ab38ea1a97dfa", "parents": [ "5a4f13fad1ab5bd08dea78fc55321e429d83cddf" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Jul 01 11:13:45 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed Jul 01 11:13:45 2009 +1000" }, "message": "md: Use new topology calls to indicate alignment and I/O sizes\n\nSwitch MD over to the new disk_stack_limits() function which checks for\naligment and adjusts preferred I/O sizes when stacking.\n\nAlso indicate preferred I/O sizes where applicable.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ea9df47cc92573b159ef3b4fda516c32cba9c4fd", "tree": "5d951e91c9be014420d5eb5772d7546e8ce5a435", "parents": [ "874d2f61d31e596c36af7732dc1b3aa2dc233824" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Tue Jun 30 15:18:17 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Tue Jun 30 15:18:17 2009 +0100" }, "message": "dm table: fix blk_stack_limits arg to use bytes not sectors\n\nThe offset passed to blk_stack_limits() must be in bytes not sectors.\nFixes false warnings like the following:\ndevice-mapper: table: 254:1: target device sda6 is misaligned\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nReported-by: Frans Pop \u003celendil@planet.nl\u003e\nTested-by: Frans Pop \u003celendil@planet.nl\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "874d2f61d31e596c36af7732dc1b3aa2dc233824", "tree": "5d135e78c6d8cabcaab563239933bf21ac3a7203", "parents": [ "5a4f13fad1ab5bd08dea78fc55321e429d83cddf" ], "author": { "name": "Milan Broz", "email": "mbroz@redhat.com", "time": "Tue Jun 30 15:18:14 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Tue Jun 30 15:18:14 2009 +0100" }, "message": "dm exception store: really fix type lookup\n\nFix exception store name handling.\n\nWe need to reference exception store by zero terminated string.\n\nFixes regression introduced in commit f6bd4eb73cdf2a5bf954e497972842f39cabb7e3\n\nCc: Yi Yang \u003cyi.y.yang@intel.com\u003e\nCc: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nCc: stable@kernel.org\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Milan Broz \u003cmbroz@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f40c67f0f7e2767f80f7cbcbc1ab86c4113c202e", "tree": "e8a20f3ceac54ac6edd0f25ffa95c0216a949c52", "parents": [ "523d9297d43cce3fa6de6474b7674329e98743b1" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:37 2009 +0100" }, "message": "dm mpath: change to be request based\n\nThis patch converts dm-multipath target to request-based from bio-based.\n\nBasically, the patch just converts the I/O unit from struct bio\nto struct request.\nIn the course of the conversion, it also changes the I/O queueing\nmechanism. The change in the I/O queueing is described in details\nas follows.\n\nI/O queueing mechanism change\n-----------------------------\nIn I/O submission, map_io(), there is no mechanism change from\nbio-based, since the clone request is ready for retry as it is.\nHowever, in I/O complition, do_end_io(), there is a mechanism change\nfrom bio-based, since the clone request is not ready for retry.\n\nIn do_end_io() of bio-based, the clone bio has all needed memory\nfor resubmission. So the target driver can queue it and resubmit\nit later without memory allocations.\nThe mechanism has almost no overhead.\n\nOn the other hand, in do_end_io() of request-based, the clone request\ndoesn\u0027t have clone bios, so the target driver can\u0027t resubmit it\nas it is. To resubmit the clone request, memory allocation for\nclone bios is needed, and it takes some overheads.\nTo avoid the overheads just for queueing, the target driver doesn\u0027t\nqueue the clone request inside itself.\nInstead, the target driver asks dm core for queueing and remapping\nthe original request of the clone request, since the overhead for\nqueueing is just a freeing memory for the clone request.\n\nAs a result, the target driver doesn\u0027t need to record/restore\nthe information of the original request for resubmitting\nthe clone request. So dm_bio_details in dm_mpath_io is removed.\n\nmultipath_busy()\n---------------------\nThe target driver returns \"busy\", only when the following case:\n o The target driver will map I/Os, if map() function is called\n and\n o The mapped I/Os will wait on underlying device\u0027s queue due to\n their congestions, if map() function is called now.\n\nIn other cases, the target driver doesn\u0027t return \"busy\".\nOtherwise, dm core will keep the I/Os and the target driver can\u0027t\ndo what it wants.\n(e.g. the target driver can\u0027t map I/Os now, so wants to kill I/Os.)\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nAcked-by: Hannes Reinecke \u003chare@suse.de\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "523d9297d43cce3fa6de6474b7674329e98743b1", "tree": "b889df9e26458c134a909c9ada51fa961fbf1291", "parents": [ "5d67aa2366ccb8257d103d0b43df855605c3c086" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:37 2009 +0100" }, "message": "dm: disable interrupt when taking map_lock\n\nThis patch disables interrupt when taking map_lock to avoid\nlockdep warnings in request-based dm.\n\nrequest-based dm takes map_lock after taking queue_lock with\ndisabling interrupt:\n spin_lock_irqsave(queue_lock)\n q-\u003erequest_fn() \u003d\u003d dm_request_fn()\n \u003d\u003e dm_get_table()\n \u003d\u003e read_lock(map_lock)\nwhile queue_lock could be (but isn\u0027t) taken in interrupt context.\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nAcked-by: Christof Schmitt \u003cchristof.schmitt@de.ibm.com\u003e\nAcked-by: Hannes Reinecke \u003chare@suse.de\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5d67aa2366ccb8257d103d0b43df855605c3c086", "tree": "1d03f2e89ef47773a757f05b17741b8cbe47e882", "parents": [ "e6ee8c0b767540f59e20da3ced282601db8aa502" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:36 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:36 2009 +0100" }, "message": "dm: do not set QUEUE_ORDERED_DRAIN if request based\n\nRequest-based dm doesn\u0027t have barrier support yet.\nSo we need to set QUEUE_ORDERED_DRAIN only for bio-based dm.\nSince the device type is decided at the first table loading time,\nthe flag set is deferred until then.\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nAcked-by: Hannes Reinecke \u003chare@suse.de\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "e6ee8c0b767540f59e20da3ced282601db8aa502", "tree": "101cb830994734eb45a4a47cd5988f24da67fa4f", "parents": [ "cec47e3d4a861e1d942b3a580d0bbef2700d2bb2" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:36 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:36 2009 +0100" }, "message": "dm: enable request based option\n\nThis patch enables request-based dm.\n\no Request-based dm and bio-based dm coexist, since there are\n some target drivers which are more fitting to bio-based dm.\n Also, there are other bio-based devices in the kernel\n (e.g. md, loop).\n Since bio-based device can\u0027t receive struct request,\n there are some limitations on device stacking between\n bio-based and request-based.\n\n type of underlying device\n bio-based request-based\n ----------------------------------------------\n bio-based OK OK\n request-based -- OK\n\n The device type is recognized by the queue flag in the kernel,\n so dm follows that.\n\no The type of a dm device is decided at the first table binding time.\n Once the type of a dm device is decided, the type can\u0027t be changed.\n\no Mempool allocations are deferred to at the table loading time, since\n mempools for request-based dm are different from those for bio-based\n dm and needed mempool type is fixed by the type of table.\n\no Currently, request-based dm supports only tables that have a single\n target. To support multiple targets, we need to support request\n splitting or prevent bio/request from spanning multiple targets.\n The former needs lots of changes in the block layer, and the latter\n needs that all target drivers support merge() function.\n Both will take a time.\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "cec47e3d4a861e1d942b3a580d0bbef2700d2bb2", "tree": "2f92b957d515a5d887fe0147984cda3203c8b8ea", "parents": [ "f5db4af466e2dca0fe822019812d586ca910b00c" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:35 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:35 2009 +0100" }, "message": "dm: prepare for request based option\n\nThis patch adds core functions for request-based dm.\n\nWhen struct mapped device (md) is initialized, md-\u003equeue has\nan I/O scheduler and the following functions are used for\nrequest-based dm as the queue functions:\n make_request_fn: dm_make_request()\n pref_fn: dm_prep_fn()\n request_fn: dm_request_fn()\n softirq_done_fn: dm_softirq_done()\n lld_busy_fn: dm_lld_busy()\nActual initializations are done in another patch (PATCH 2).\n\nBelow is a brief summary of how request-based dm behaves, including:\n - making request from bio\n - cloning, mapping and dispatching request\n - completing request and bio\n - suspending md\n - resuming md\n\n bio to request\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n md-\u003equeue-\u003emake_request_fn() (dm_make_request()) calls __make_request()\n for a bio submitted to the md.\n Then, the bio is kept in the queue as a new request or merged into\n another request in the queue if possible.\n\n Cloning and Mapping\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Cloning and mapping are done in md-\u003equeue-\u003erequest_fn() (dm_request_fn()),\n when requests are dispatched after they are sorted by the I/O scheduler.\n\n dm_request_fn() checks busy state of underlying devices using\n target\u0027s busy() function and stops dispatching requests to keep them\n on the dm device\u0027s queue if busy.\n It helps better I/O merging, since no merge is done for a request\n once it is dispatched to underlying devices.\n\n Actual cloning and mapping are done in dm_prep_fn() and map_request()\n called from dm_request_fn().\n dm_prep_fn() clones not only request but also bios of the request\n so that dm can hold bio completion in error cases and prevent\n the bio submitter from noticing the error.\n (See the \"Completion\" section below for details.)\n\n After the cloning, the clone is mapped by target\u0027s map_rq() function\n and inserted to underlying device\u0027s queue using\n blk_insert_cloned_request().\n\n Completion\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Request completion can be hooked by rq-\u003eend_io(), but then, all bios\n in the request will have been completed even error cases, and the bio\n submitter will have noticed the error.\n To prevent the bio completion in error cases, request-based dm clones\n both bio and request and hooks both bio-\u003ebi_end_io() and rq-\u003eend_io():\n bio-\u003ebi_end_io(): end_clone_bio()\n rq-\u003eend_io(): end_clone_request()\n\n Summary of the request completion flow is below:\n blk_end_request() for a clone request\n \u003d\u003e blk_update_request()\n \u003d\u003e bio-\u003ebi_end_io() \u003d\u003d end_clone_bio() for each clone bio\n \u003d\u003e Free the clone bio\n \u003d\u003e Success: Complete the original bio (blk_update_request())\n Error: Don\u0027t complete the original bio\n \u003d\u003e blk_finish_request()\n \u003d\u003e rq-\u003eend_io() \u003d\u003d end_clone_request()\n \u003d\u003e blk_complete_request()\n \u003d\u003e dm_softirq_done()\n \u003d\u003e Free the clone request\n \u003d\u003e Success: Complete the original request (blk_end_request())\n Error: Requeue the original request\n\n end_clone_bio() completes the original request on the size of\n the original bio in successful cases.\n Even if all bios in the original request are completed by that\n completion, the original request must not be completed yet to keep\n the ordering of request completion for the stacking.\n So end_clone_bio() uses blk_update_request() instead of\n blk_end_request().\n In error cases, end_clone_bio() doesn\u0027t complete the original bio.\n It just frees the cloned bio and gives over the error handling to\n end_clone_request().\n\n end_clone_request(), which is called with queue lock held, completes\n the clone request and the original request in a softirq context\n (dm_softirq_done()), which has no queue lock, to avoid a deadlock\n issue on submission of another request during the completion:\n - The submitted request may be mapped to the same device\n - Request submission requires queue lock, but the queue lock\n has been held by itself and it doesn\u0027t know that\n\n The clone request has no clone bio when dm_softirq_done() is called.\n So target drivers can\u0027t resubmit it again even error cases.\n Instead, they can ask dm core for requeueing and remapping\n the original request in that cases.\n\n suspend\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Request-based dm uses stopping md-\u003equeue as suspend of the md.\n For noflush suspend, just stops md-\u003equeue.\n\n For flush suspend, inserts a marker request to the tail of md-\u003equeue.\n And dispatches all requests in md-\u003equeue until the marker comes to\n the front of md-\u003equeue. Then, stops dispatching request and waits\n for the all dispatched requests to complete.\n After that, completes the marker request, stops md-\u003equeue and\n wake up the waiter on the suspend queue, md-\u003ewait.\n\n resume\n \u003d\u003d\u003d\u003d\u003d\u003d\n Starts md-\u003equeue.\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f5db4af466e2dca0fe822019812d586ca910b00c", "tree": "1bbaaa36509df9f7eecc19ccffa434048cf4b555", "parents": [ "754c5fc7ebb417b23601a6222a6005cc2e7f2913" ], "author": { "name": "Jonthan Brassow", "email": "jbrassow@redhat.com", "time": "Mon Jun 22 10:12:35 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:35 2009 +0100" }, "message": "dm raid1: add userspace log\n\nThis patch contains a device-mapper mirror log module that forwards\nrequests to userspace for processing.\n\nThe structures used for communication between kernel and userspace are\nlocated in include/linux/dm-log-userspace.h. Due to the frequency,\ndiversity, and 2-way communication nature of the exchanges between\nkernel and userspace, \u0027connector\u0027 was chosen as the interface for\ncommunication.\n\nThe first log implementations written in userspace - \"clustered-disk\"\nand \"clustered-core\" - support clustered shared storage. A userspace\ndaemon (in the LVM2 source code repository) uses openAIS/corosync to\nprocess requests in an ordered fashion with the rest of the nodes in the\ncluster so as to prevent log state corruption. Other implementations\nwith no association to LVM or openAIS/corosync, are certainly possible.\n\n(Imagine if two machines are writing to the same region of a mirror.\nThey would both mark the region dirty, but you need a cluster-aware\nentity that can handle properly marking the region clean when they are\ndone. Otherwise, you might clear the region when the first machine is\ndone, not the second.)\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nCc: Evgeniy Polyakov \u003cjohnpol@2ka.mipt.ru\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "754c5fc7ebb417b23601a6222a6005cc2e7f2913", "tree": "6c31b055fc26ec541d67fc1123ebaa4b7a8eae7a", "parents": [ "18d8594dd93a1ae2fafd591ec026e87d743292bf" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:34 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:34 2009 +0100" }, "message": "dm: calculate queue limits during resume not load\n\nCurrently, device-mapper maintains a separate instance of \u0027struct\nqueue_limits\u0027 for each table of each device. When the configuration of\na device is to be changed, first its table is loaded and this structure\nis populated, then the device is \u0027resumed\u0027 and the calculated\nqueue_limits are applied.\n\nThis places restrictions on how userspace may process related devices,\nwhere it is often advantageous to \u0027load\u0027 tables for several devices\nat once before \u0027resuming\u0027 them together. As the new queue_limits\nonly take effect after the \u0027resume\u0027, if they are changing and one\ndevice uses another, the latter must be \u0027resumed\u0027 before the former\nmay be \u0027loaded\u0027.\n\nThis patch moves the calculation of these queue_limits out of\nthe \u0027load\u0027 operation into \u0027resume\u0027. Since we are no longer\npre-calculating this struct, we no longer need to maintain copies\nwithin our dm structs.\n\ndm_set_device_limits() now passes the \u0027start\u0027 of the device\u0027s\ndata area (aka pe_start) as the \u0027offset\u0027 to blk_stack_limits().\n\ninit_valid_queue_limits() is replaced by blk_set_default_limits().\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: martin.petersen@oracle.com\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "18d8594dd93a1ae2fafd591ec026e87d743292bf", "tree": "9a4b0ac8512864666fef0c6e3bc26c5db172f649", "parents": [ "af4874e03ed82f050d5872d8c39ce64bf16b5c38" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:33 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:33 2009 +0100" }, "message": "dm log: fix create_log_context to use logical_block_size of log device\n\ncreate_log_context() must use the logical_block_size from the log disk,\nwhere the I/O happens, not the target\u0027s logical_block_size.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "af4874e03ed82f050d5872d8c39ce64bf16b5c38", "tree": "38aa5dee43b4bb7a369995d4f38dee992cb051e0", "parents": [ "1197764e403d97231eb6da2b1e16f511a7fd3101" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:33 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:33 2009 +0100" }, "message": "dm target:s introduce iterate devices fn\n\nAdd .iterate_devices to \u0027struct target_type\u0027 to allow a function to be\ncalled for all devices in a DM target. Implemented it for all targets\nexcept those in dm-snap.c (origin and snapshot).\n\n(The raid1 version number jumps to 1.12 because we originally reserved\n1.1 to 1.11 for \u0027block_on_error\u0027 but ended up using \u0027handle_errors\u0027\ninstead.)\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nCc: martin.petersen@oracle.com\n" }, { "commit": "1197764e403d97231eb6da2b1e16f511a7fd3101", "tree": "9c38793927501989c4c74cae0887309bd17484d8", "parents": [ "5ab97588fb266187b88d1ad893251c94388f18ba" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:32 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:32 2009 +0100" }, "message": "dm table: establish queue limits by copying table limits\n\nCopy the table\u0027s queue_limits to the DM device\u0027s request_queue. This\nproperly initializes the queue\u0027s topology limits and also avoids having\nto track the evolution of \u0027struct queue_limits\u0027 in\ndm_table_set_restrictions()\n\nAlso fixes a bug that was introduced in dm_table_set_restrictions() via\ncommit ae03bf639a5027d27270123f5f6e3ee6a412781d. In addition to\nestablishing \u0027bounce_pfn\u0027 in the queue\u0027s limits blk_queue_bounce_limit()\nalso performs an allocation to setup the ISA DMA pool. This allocation\nresulted in \"sleeping function called from invalid context\" when called\nfrom dm_table_set_restrictions().\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5ab97588fb266187b88d1ad893251c94388f18ba", "tree": "0dfd693fb3a31e22353e5640db8c17c989b5c65d", "parents": [ "be6d4305db093ad1cc623f7dd3d2470b7bd73fa4" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:32 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:32 2009 +0100" }, "message": "dm table: replace struct io_restrictions with struct queue_limits\n\nUse blk_stack_limits() to stack block limits (including topology) rather\nthan duplicate the equivalent within Device Mapper.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "be6d4305db093ad1cc623f7dd3d2470b7bd73fa4", "tree": "27935ca83626cba6ba2fac2cc83deeca0ff15492", "parents": [ "02acc3a4fa0a6c2a5ccc4fb722b55fb710265882" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:31 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:31 2009 +0100" }, "message": "dm table: validate device logical_block_size\n\nImpose necessary and sufficient conditions on a devices\u0027s table such\nthat any incoming bio which respects its logical_block_size can be\nprocessed successfully.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "02acc3a4fa0a6c2a5ccc4fb722b55fb710265882", "tree": "5abd5bf61c8fe1a9bed391c828ffff34384f6b20", "parents": [ "60935eb21d3c5bac79618000f38f92c249d153c4" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Mon Jun 22 10:12:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:30 2009 +0100" }, "message": "dm table: ensure targets are aligned to logical_block_size\n\nEnsure I/O is aligned to the logical block size of target devices.\n\nRename check_device_area() to device_area_is_valid() for clarity and\nestablish the device limits including the logical block size prior to\ncalling it.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "60935eb21d3c5bac79618000f38f92c249d153c4", "tree": "0fcd78d7d69e23cd062e5214ddf2959d5c74e605", "parents": [ "486d220fe4909b5745c4faa67faddd30a707abe2" ], "author": { "name": "Milan Broz", "email": "mbroz@redhat.com", "time": "Mon Jun 22 10:12:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:30 2009 +0100" }, "message": "dm ioctl: support cookies for udev\n\nAdd support for passing a 32 bit \"cookie\" into the kernel with the\nDM_SUSPEND, DM_DEV_RENAME and DM_DEV_REMOVE ioctls. The (unsigned)\nvalue of this cookie is returned to userspace alongside the uevents\nissued by these ioctls in the variable DM_COOKIE.\n\nThis means the userspace process issuing these ioctls can be notified\nby udev after udev has completed any actions triggered.\n\nTo minimise the interface extension, we pass the cookie into the\nkernel in the event_nr field which is otherwise unused when calling\nthese ioctls. Incrementing the version number allows userspace to\ndetermine in advance whether or not the kernel supports the cookie.\nIf the kernel does support this but userspace does not, there should\nbe no impact as the new variable will just get ignored.\n\nSigned-off-by: Milan Broz \u003cmbroz@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "486d220fe4909b5745c4faa67faddd30a707abe2", "tree": "eb7184f8299a5446532a6ff96d48fcdc3a461268", "parents": [ "1b6da754594e6e26c24e6fbc1a34f9c03e4617a3" ], "author": { "name": "Peter Rajnoha", "email": "prajnoha@redhat.com", "time": "Mon Jun 22 10:12:29 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:29 2009 +0100" }, "message": "dm: sysfs add suspended attribute\n\nAdd a file named \u0027suspended\u0027 to each device-mapper device directory in\nsysfs. It holds the value 1 while the device is suspended. Otherwise\nit holds 0.\n\nSigned-off-by: Peter Rajnoha \u003cprajnoha@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "1b6da754594e6e26c24e6fbc1a34f9c03e4617a3", "tree": "713b521ab6f0769752c57bebb7c10621abc09cea", "parents": [ "f392ba889b019602976082bfe7bf486c2594f85c" ], "author": { "name": "Jonthan Brassow", "email": "jbrassow@redhat.com", "time": "Mon Jun 22 10:12:29 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:29 2009 +0100" }, "message": "dm table: improve warning message when devices not freed before destruction\n\nReport any devices forgotten to be freed before a table is destroyed.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f392ba889b019602976082bfe7bf486c2594f85c", "tree": "962e8f354dfe3df2021476412be8d1bcec8a03d0", "parents": [ "fd5e033908b7b743b5650790f196761dd930f988" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:28 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:28 2009 +0100" }, "message": "dm mpath: add service time load balancer\n\nThis patch adds a service time oriented dynamic load balancer,\ndm-service-time, which selects the path with the shortest estimated\nservice time for the incoming I/O.\nThe service time is estimated by dividing the in-flight I/O size\nby a performance value of each path.\n\nThe performance value can be given as a table argument at the table\nloading time. If no performance value is given, all paths are\nconsidered equal.\n\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "fd5e033908b7b743b5650790f196761dd930f988", "tree": "cd264d11371b9410e98388e00f4d87459a5e6325", "parents": [ "02ab823fd1a27d193bda06b74fdad685a20a3e5e" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:27 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:27 2009 +0100" }, "message": "dm mpath: add queue length load balancer\n\nThis patch adds a dynamic load balancer, dm-queue-length, which\nbalances the number of in-flight I/Os across the paths.\n\nThe code is based on the patch posted by Stefan Bader:\nhttps://www.redhat.com/archives/dm-devel/2005-October/msg00050.html\n\nSigned-off-by: Stefan Bader \u003cstefan.bader@canonical.com\u003e\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "02ab823fd1a27d193bda06b74fdad685a20a3e5e", "tree": "04b3db6e494f483fef34a55e487e88f6d7c585a5", "parents": [ "2bd023452592e5f5cf90dd426cc39b7632b15b76" ], "author": { "name": "Kiyoshi Ueda", "email": "k-ueda@ct.jp.nec.com", "time": "Mon Jun 22 10:12:27 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:27 2009 +0100" }, "message": "dm mpath: add start_io and nr_bytes to path selectors\n\nThis patch makes two additions to the dm path selector interface for\ndynamic load balancers:\n o a new hook, start_io()\n o a new parameter \u0027nr_bytes\u0027 to select_path()/start_io()/end_io()\n to pass the size of the I/O\n\nstart_io() is called when a target driver actually submits I/O\nto the selected path.\nPath selectors can use it to start accounting of the I/O.\n(e.g. counting the number of in-flight I/Os.)\nThe start_io hook is based on the patch posted by Stefan Bader:\nhttps://www.redhat.com/archives/dm-devel/2005-October/msg00050.html\n\nnr_bytes, the size of the I/O, is so path selectors can take the\nsize of the I/O into account when deciding which path to use.\ndm-service-time uses it to estimate service time, for example.\n(Added the nr_bytes member to dm_mpath_io instead of using existing\n details.bi_size, since request-based dm patch deletes it.)\n\nSigned-off-by: Stefan Bader \u003cstefan.bader@canonical.com\u003e\nSigned-off-by: Kiyoshi Ueda \u003ck-ueda@ct.jp.nec.com\u003e\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "2bd023452592e5f5cf90dd426cc39b7632b15b76", "tree": "62fe4d816c597e1b6c58bf150603dd72c9ce6135", "parents": [ "51aa322849581f1a73594e48ea0df63f914ee6a2" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:26 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:26 2009 +0100" }, "message": "dm snapshot: use barrier when writing exception store\n\nSend barrier requests when updating the exception area.\n\nException area updates need to be ordered w.r.t. data writes, so that\nthe writes are not reordered in hardware disk cache.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "51aa322849581f1a73594e48ea0df63f914ee6a2", "tree": "19b3545084700fc36b5a916d71473936628bb062", "parents": [ "5af443a7e1c0864100cc44525a9821aa2a2f4719" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:26 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:26 2009 +0100" }, "message": "dm io: retry after barrier error\n\nIf -EOPNOTSUPP was returned and the request was a barrier request, retry it\nwithout barrier.\n\nRetry all regions for now. Barriers are submitted only for one-region requests,\nso it doesn\u0027t matter. (In the future, retries can be limited to the actual\nregions that failed.)\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5af443a7e1c0864100cc44525a9821aa2a2f4719", "tree": "87ab19984e94c8aaba743e47ab62cf084afbdf2c", "parents": [ "494b3ee7d4f69210def80aecce28d08c3f0755d5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:25 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:25 2009 +0100" }, "message": "dm io: record eopnotsupp\n\nAdd another field, eopnotsupp_bits. It is subset of error_bits, representing\nregions that returned -EOPNOTSUPP. (The bit is set in both error_bits and\neopnotsupp_bits).\n\nThis value will be used in further patches.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "494b3ee7d4f69210def80aecce28d08c3f0755d5", "tree": "8cd8fc4de72cf9a1a56e45518019a3cae106515a", "parents": [ "8627921fa2ef6d40fd9b787e163ba3a9ff8f471d" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:25 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:25 2009 +0100" }, "message": "dm snapshot: support barriers\n\nFlush support for dm-snapshot target.\n\nThis patch just forwards the flush request to either the origin or the snapshot\ndevice. (It doesn\u0027t flush exception store metadata.)\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "8627921fa2ef6d40fd9b787e163ba3a9ff8f471d", "tree": "a34175f655e47785104b204e1b461baa8d10ecce", "parents": [ "c927259e34e518d913d86f51c71b786a513f94d6" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:24 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:24 2009 +0100" }, "message": "dm mpath: support barriers\n\nFlush support for dm-multipath target.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "c927259e34e518d913d86f51c71b786a513f94d6", "tree": "0797e9e4689749ad70b6739f845374dc9c57c65d", "parents": [ "647c7db14ef9cacc4ccb3683e206b61f0de6dc2b" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:23 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:23 2009 +0100" }, "message": "dm delay: support barriers\n\nFlush support for dm-delay target.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "647c7db14ef9cacc4ccb3683e206b61f0de6dc2b", "tree": "4ef2475e295813dcce1123f2c74ba04061db1b7e", "parents": [ "374bf7e7f6cc38b0483351a2029a97910eadde1b" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:23 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:23 2009 +0100" }, "message": "dm crypt: support flush\n\nFlush support for dm-crypt target.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "374bf7e7f6cc38b0483351a2029a97910eadde1b", "tree": "10db8f79cd22e0c4ecc683fdd304aca60815bf79", "parents": [ "433bcac5645508b71eab2710b6817c3ef937eba8" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:22 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:22 2009 +0100" }, "message": "dm: stripe support flush\n\nFlush support for the stripe target.\n\nThis sets ti-\u003enum_flush_requests to the number of stripes and\nremaps individual flush requests to the appropriate stripe devices.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "433bcac5645508b71eab2710b6817c3ef937eba8", "tree": "d2c1d8e9403d0f3e83c63085a9046f466de431c3", "parents": [ "52b1fd5a27c625c78373e024bf570af3c9d44a79" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:22 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:22 2009 +0100" }, "message": "dm: linear support flush\n\nFlush support for the linear target.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "52b1fd5a27c625c78373e024bf570af3c9d44a79", "tree": "a802b917b45685255220efaeb7b2b8ee2d04a2d4", "parents": [ "9015df24a8008d7bea2bd3df881783ebe0dcb9af" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:21 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:21 2009 +0100" }, "message": "dm: send empty barriers to targets in dm_flush\n\nPass empty barrier flushes to the targets in dm_flush().\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "9015df24a8008d7bea2bd3df881783ebe0dcb9af", "tree": "aff9a824a202525036d5f2dc52ad728dc9a38cbd", "parents": [ "f9ab94cee313746573b2d693bc2afb807ebb0998" ], "author": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:21 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:21 2009 +0100" }, "message": "dm: initialise tio in alloc_tio\n\nMove repeated dm_target_io initialisation inside alloc_tio().\n\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f9ab94cee313746573b2d693bc2afb807ebb0998", "tree": "150155f364db0f26c7217e1f92d31344aa67a3f0", "parents": [ "27eaa14975d8b53f0bad422e53cdf8e5f6dd44ec" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:20 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:20 2009 +0100" }, "message": "dm: introduce num_flush_requests\n\nIntroduce num_flush_requests for a target to set to say how many flush\ninstructions (empty barriers) it wants to receive. These are sent by\n__clone_and_map_empty_barrier with map_info-\u003eflush_request going from 0\nto (num_flush_requests - 1).\n\nOld targets without flush support won\u0027t receive any flush requests.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "27eaa14975d8b53f0bad422e53cdf8e5f6dd44ec", "tree": "647587ef410cfe862af501922704871f99ead273", "parents": [ "fdb9572b73abd008b80931c3b1f157dac3888bb9" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:20 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:20 2009 +0100" }, "message": "dm: remove check that prevents mapping empty bios\n\nRemove the check that the size of the cloned bio is not zero because a\nsubsequent patch needs to send zero-sized barriers down this path.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "fdb9572b73abd008b80931c3b1f157dac3888bb9", "tree": "ae84d6ed1c97e77d536d0bcf2e25fa35d5cbfcba", "parents": [ "5aa2781d964e9835c441932110484bc454b5c207" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:19 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:19 2009 +0100" }, "message": "dm: remove EOPNOTSUPP for barriers\n\nIf the underlying device doesn\u0027t support barriers and dm receives a\nbarrier, it waits until all requests on that device drain so it no\nlonger needs to report -EOPNOTSUPP to the caller.\n\nThis patch deals with the confusing situation when moving a volume from\none physical device to another triggers an EOPNOTSUPP on a volume that\ndidn\u0027t report it before.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5aa2781d964e9835c441932110484bc454b5c207", "tree": "4f771f3e933eaedf67e5f9e8d5fcc10340af80e2", "parents": [ "2761e95fe40ca0d01864310fa4d488d7c5e34e18" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:18 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:18 2009 +0100" }, "message": "dm: store only first barrier error\n\nWith the following patches, more than one error can occur during\nprocessing. Change md-\u003ebarrier_error so that only the first one is\nrecorded and returned to the caller.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "2761e95fe40ca0d01864310fa4d488d7c5e34e18", "tree": "a49245e8ebb022605c5d1ce04948cf4a7f9a5c3b", "parents": [ "531fe96364f30879753d46c1f52ab839e12d2e5d" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:18 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:18 2009 +0100" }, "message": "dm: process requeue in dm_wq_work\n\nIf barrier request was returned with DM_ENDIO_REQUEUE,\nrequeue it in dm_wq_work instead of dec_pending.\n\nThis allows us to correctly handle a situation when some targets\nare asking for a requeue and other targets signal an error.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "531fe96364f30879753d46c1f52ab839e12d2e5d", "tree": "61a625f68c3105c1525e2eebba0f046a3678eb37", "parents": [ "32a926da5a16c01a8213331e5764472ce2f14a8d" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:17 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:17 2009 +0100" }, "message": "dm: make dm_flush return void\n\nMake dm_flush return void.\n\nThe first error during flush is stored in md-\u003ebarrier_error instead.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "32a926da5a16c01a8213331e5764472ce2f14a8d", "tree": "edac1cef90e4754353a50fb948c4e34b4df8b5bc", "parents": [ "db8fef4fabe4a546ce74f80bff64fd43776e5912" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:17 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:17 2009 +0100" }, "message": "dm: always hold bdev reference\n\nFix a potential deadlock when creating multiple snapshots by holding a\nreference to struct block_device for the whole lifecycle of every dm\ndevice instead of obtaining it independently at each point it is needed.\n\nbdget_disk() was called while the device was being suspended, in\ndm_suspend(). However there could be other devices already suspended,\nfor example when creating additional snapshots of a device. bdget_disk()\ncan wait for IO and allocate memory resulting in waiting for the\nalready-suspended device - deadlock.\n\nThis patch changes the code so that it gets the reference to struct\nblock_device when struct mapped_device is allocated and initialized in\nalloc_dev() where it is always OK to allocate memory or wait for I/O.\nIt drops the reference when it is destroyed in free_dev(). Thus there\nis no call to bdget_disk() while any device is suspended.\n\nPreviously unlock_fs() was called only if bdev was held. Now it is\ncalled unconditionally, but the superfluous calls are harmless because\nit returns immediately if the filesystem was not previously frozen.\n\nThis patch also now allows the device size to be changed in a\nnoflush suspend because the bdev is held. This has no adverse effect.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "db8fef4fabe4a546ce74f80bff64fd43776e5912", "tree": "0e3cc2d7aa40783a1036cce6deb227a3a6d6c9e2", "parents": [ "f6bd4eb73cdf2a5bf954e497972842f39cabb7e3" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:15 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:15 2009 +0100" }, "message": "dm: rename suspended_bdev to bdev\n\nRename suspended_bdev to bdev.\n\nThis patch doesn\u0027t change any functionality, just renames the variable.\nIn the next patch, the variable will be used even for non-suspended device.\n\n(Pre-requisite for the per-target barrier support patches.)\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f6bd4eb73cdf2a5bf954e497972842f39cabb7e3", "tree": "f1f0ac6c9fe3ff31cccf6082397bd0802d999344", "parents": [ "5657e8fa45cf230df278040c420fb80e06309d8f" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Mon Jun 22 10:12:15 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:15 2009 +0100" }, "message": "dm exception store: fix exstore lookup to be case insensitive\n\nWhen snapshots are created using \u0027p\u0027 instead of \u0027P\u0027 as the\nexception store type, the device-mapper table loading fails.\n\nThis patch makes the code case insensitive as intended and fixes some\nregressions reported with device-mapper snapshots.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5657e8fa45cf230df278040c420fb80e06309d8f", "tree": "7817e1bd6c9b721be85fe54a1aec6ef971d7d96b", "parents": [ "8cbeb67ad50f7d68e5e83be2cb2284de8f9c03b5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:14 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:14 2009 +0100" }, "message": "dm: use i_size_read\n\nUse i_size_read() instead of reading i_size.\n\nIf someone changes the size of the device simultaneously, i_size_read\nis guaranteed to return a valid value (either the old one or the new one).\n\ni_size can return some intermediate invalid value (on 32-bit computers\nwith 64-bit i_size, the reads to both halves of i_size can be interleaved\nwith updates to i_size, resulting in garbage being returned).\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "8cbeb67ad50f7d68e5e83be2cb2284de8f9c03b5", "tree": "4fd4a467c06441ad7c07211b386c786062aa2aeb", "parents": [ "53b351f972a882ea8b6cdb19602535f1057c884a" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:14 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:14 2009 +0100" }, "message": "dm: avoid unsupported spanning of md stripe boundaries\n\nA bio that has two or more vector entries, size less than or equal to\npage size, that crosses a stripe boundary of an underlying md device is\naccepted by device mapper (it conforms to all its limits) but not by the\nunderlying device.\n\nThe fix is: If device mapper selects the one-page maximum request size,\nit also needs to set its own q-\u003emerge_bvec_fn to reject any bios with\nmultiple vector entries that span more pages.\n\nThe problem was discovered in the following scenario:\n * MD - RAID-0\n * LV on the top of it (raid1, snapshot or striped with chunk\nsize/stripe larger than RAID-0 stripe)\n * one of the logical volumes is exported to xen domU\n * inside xen domU it is partitioned, the key point is that the partition\nmust be unaligned on page boundary (fdisk normally aligns the partition to\n63 sectors which will trigger it)\n * install the system on the partitioned disk in domU\nThis causes I/O failures in dom0.\nReference: https://bugzilla.redhat.com/show_bug.cgi?id\u003d223947\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "53b351f972a882ea8b6cdb19602535f1057c884a", "tree": "0a3736a570dc7ff3c6ebf0993950a36a9784b142", "parents": [ "a72986c562eeec3f7b992198c168f0f41606fe53" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:13 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:13 2009 +0100" }, "message": "dm mpath: flush keventd queue in destructor\n\nThe commit fe9cf30eb8186ef267d1868dc9f12f2d0f40835a moves dm table event\nsubmission from kmultipath queue to kernel kevent queue to avoid a\ndeadlock.\n\nThere is a possibility of race condition because kevent queue is not flushed\nin the multipath destructor. The scenario is:\n- some event happens and is queued to keventd\n- keventd thread is delayed due to scheuling latency or some other work\n- multipath device is destroyed\n- keventd now attempts to process work_struct that is residing in already\n released memory.\n\nThe patch flushes the keventd queue in multipath constructor.\nI\u0027ve already fixed similar bug in dm-raid1.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "a72986c562eeec3f7b992198c168f0f41606fe53", "tree": "2ee7051fa43c04ab0d72487b249aa733ec129703", "parents": [ "e54f77ddda72781ec1c1696b21aabd6a30cbb7c6" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:13 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:13 2009 +0100" }, "message": "dm raid1: keep retrying alloc if mempool_alloc failed\n\nIf the code can\u0027t handle allocation failures, use __GFP_NOFAIL so that\nin case of memory pressure the allocator will retry indefinitely and\nwon\u0027t return NULL which would cause a crash in the function.\n\nThis is still not a correct fix, it may cause a classic deadlock when\nmemory manager waits for I/O being done and I/O waits for some free memory.\nI/O code shouldn\u0027t allocate any memory. But in this case it probably\ndoesn\u0027t matter much in practice, people usually do not swap on RAID.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "e54f77ddda72781ec1c1696b21aabd6a30cbb7c6", "tree": "9ba727373125fce5e75a02ac1b67b2b1f894c81e", "parents": [ "a0cf7ea9549ec60988369f90e5c0f855f08abac9" ], "author": { "name": "Chandra Seetharaman", "email": "sekharan@us.ibm.com", "time": "Mon Jun 22 10:12:12 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:12 2009 +0100" }, "message": "dm mpath: call activate fn for each path in pg_init\n\nFixed a problem affecting reinstatement of passive paths.\n\nBefore we moved the hardware handler from dm to SCSI, it performed a pg_init\nfor a path group and didn\u0027t maintain any state about each path in hardware\nhandler code.\n\nBut in SCSI dh, such state is now maintained, as we want to fail I/O early on a\npath if it is not the active path.\n\nAll the hardware handlers have a state now and set to active or some form of\ninactive. They have prep_fn() which uses this state to fail the I/O without\nit ever being sent to the device.\n\nSo in effect when dm-multipath calls scsi_dh_activate(), activate is\nsent to only one path and the \"state\" of that path is changed appropriately\nto \"active\" while other paths in the same path group are never changed\nas they never got an \"activate\".\n\nIn order make sure all the paths in a path group gets their state set\nproperly when a pg_init happens, we need to call scsi_dh_activate() on\nall paths in a path group.\n\nDoing this at the hardware handler layer is not a good option as we\nwant the multipath layer to define the relationship between path and path\ngroups and not the hardware handler.\n\nAttached patch sends an \"activate\" on each path in a path group when a\npath group is switched. It also sends an activate when a path is reinstated.\n\nSigned-off-by: Chandra Seetharaman \u003csekharan@us.ibm.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a0cf7ea9549ec60988369f90e5c0f855f08abac9", "tree": "9a073e2360afebe6485ac3a78c796627fc9e80b2", "parents": [ "4d89b7b4e4726893453d0fb4ddbb5b3e16353994" ], "author": { "name": "Hannes Reinecke", "email": "hare@suse.de", "time": "Mon Jun 22 10:12:11 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:11 2009 +0100" }, "message": "dm mpath: change attached scsi_dh\n\nWhen specifying a different hardware handler via multipath\nfeatures we should be able to override the built-in defaults.\n\nThe problem here is the hardware table from scsi_dh is compiled\nin and cannot be changed from userland. The multipath.conf OTOH\nis purely user-defined and, what\u0027s more, the user might have a valid\nreason for modifying it.\n(EG EMC Clariion can well be run in PNR mode even though ALUA is\nactive, or the user might want to try ALUA on any as-of-yet unknown\ndevices)\n\nSo _not_ allowing multipath to override the device handler setting\nwill just add to the confusion and makes error tracking even more\ndifficult.\n\nSigned-off-by: Hannes Reinecke \u003chare@suse.de\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "4d89b7b4e4726893453d0fb4ddbb5b3e16353994", "tree": "f0775811ee4942eadd555d5ddcdbb5ff313290bb", "parents": [ "e094f4f15f5169526c7200b9bde44b900548a81e" ], "author": { "name": "Milan Broz", "email": "mbroz@redhat.com", "time": "Mon Jun 22 10:12:11 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:11 2009 +0100" }, "message": "dm: sysfs skip output when device is being destroyed\n\nDo not process sysfs attributes when device is being destroyed.\n\nOtherwise code can cause\n BUG_ON(test_bit(DMF_FREEING, \u0026md-\u003eflags));\nin dm_put() call.\n\nCc: stable@kernel.org\nSigned-off-by: Milan Broz \u003cmbroz@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "e094f4f15f5169526c7200b9bde44b900548a81e", "tree": "5ca573d45371779a183276066578bed1f38f11c6", "parents": [ "0e0497c0c017664994819f4602dc07fd95896c52" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:12:10 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:12:10 2009 +0100" }, "message": "dm mpath: validate hw_handler argument count\n\nFix arg count parsing error in hw handlers.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "0e0497c0c017664994819f4602dc07fd95896c52", "tree": "d6a70d29e5eae6ca9cf3ca197db9f0ab8a3b5e51", "parents": [ "f234012f52a37e48f2330e1ca2df69800e797c3b" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Mon Jun 22 10:08:02 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Mon Jun 22 10:08:02 2009 +0100" }, "message": "dm mpath: validate table argument count\n\nThe parser reads the argument count as a number but doesn\u0027t check that\nsufficient arguments are supplied. This command triggers the bug:\n\ndmsetup create mpath --table \"0 `blockdev --getsize /dev/mapper/cr0`\n multipath 0 0 2 1 round-robin 1000 0 1 1 /dev/mapper/cr0\n round-robin 0 1 1 /dev/mapper/cr1 1000\"\nkernel BUG at drivers/md/dm-mpath.c:530!\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "31583d6acf940d2951bc8716557b06d9de5a0c4b", "tree": "607aef385b0998f0cb7ff63b05407afed5182840", "parents": [ "ee37ba7a4212c7420f12e6a2ad8a3966649ede7a", "f740f5ca056f0a4eff3abdf272a8a4ba3965d57d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jun 19 17:43:04 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jun 19 17:43:04 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block:\n Fix kernel-doc parameter name typo in blk-settings.c:\n block: rename CONFIG_LBD to CONFIG_LBDAF\n block: Fix bounce_pfn setting\n hd: stop defining MAJOR_NR\n" }, { "commit": "90c699a9ee4be165966d40f1837909ccb8890a68", "tree": "ff136ce7b73c7d7bbf5eb4241d01cb3b930d3566", "parents": [ "3a02c8e8142f7f133d4c6e72bc3e1d830e6b8b9e" ], "author": { "name": "Bartlomiej Zolnierkiewicz", "email": "bzolnier@gmail.com", "time": "Fri Jun 19 08:08:50 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Jun 19 08:08:50 2009 +0200" }, "message": "block: rename CONFIG_LBD to CONFIG_LBDAF\n\nFollow-up to \"block: enable by default support for large devices\nand files on 32-bit archs\".\n\nRename CONFIG_LBD to CONFIG_LBDAF to:\n- allow update of existing [def]configs for \"default y\" change\n- reflect that it is used also for large files support nowadays\n\nSigned-off-by: Bartlomiej Zolnierkiewicz \u003cbzolnier@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "9729a6eb5878a3daa18395f2b5fb38bf9359a761", "tree": "7ea32e9793c659d4059c49e7da2f38da795e7679", "parents": [ "5ae8606d5746bc84e19018fc3753cc1faf18843f", "48606a9f2fc034f0b308d088c1f7ab6d407c462c" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 18 13:11:50 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 18 13:11:50 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: (39 commits)\n md/raid5: correctly update sync_completed when we reach max_resync\n md/raid5: add missing call to schedule() after prepare_to_wait()\n md/linear: use call_rcu to free obsolete \u0027conf\u0027 structures.\n md linear: Protecting mddev with rcu locks to avoid races\n md: Move check for bitmap presence to personality code.\n md: remove chunksize rounding from common code.\n md: raid0/linear: ensure device sizes are rounded to chunk size.\n md: move assignment of -\u003eutime so that it never gets skipped.\n md: Push down reconstruction log message to personality code.\n md: merge reconfig and check_reshape methods.\n md: remove unnecessary arguments from -\u003ereconfig method.\n md: raid5: check stripe cache is large enough in start_reshape\n md: raid0: chunk_sectors cleanups.\n md: fix some comments.\n md/raid5: Use is_power_of_2() in raid5_reconfig()/raid6_reconfig().\n md: convert conf-\u003echunk_size and conf-\u003eprev_chunk to sectors.\n md: Convert mddev-\u003enew_chunk to sectors.\n md: Make mddev-\u003echunk_size sector-based.\n md: raid0 :Enables chunk size other than powers of 2.\n md: prepare for non-power-of-two chunk sizes\n ...\n" }, { "commit": "48606a9f2fc034f0b308d088c1f7ab6d407c462c", "tree": "4d1e9f14a3400905ff6c1a6d16c423da013bee8d", "parents": [ "7a3ab908948b6296ee7e81d42f7c176361c51975" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 09:14:12 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 09:14:12 2009 +1000" }, "message": "md/raid5: correctly update sync_completed when we reach max_resync\n\nAt the end of reshape_request we update cyrr_resync_completed\nif we are about to pause due to reaching resync_max.\nHowever we update it to the wrong value. We need to add the\n\"reshape_sectors\" that have just been reshaped.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7a3ab908948b6296ee7e81d42f7c176361c51975", "tree": "61a066da1bda9af8268bb865e609518be8efd024", "parents": [ "495d357301e1de01fabe30ce9a555301fb4675c3" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Jun 16 16:00:33 2009 -0700" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:50:18 2009 +1000" }, "message": "md/raid5: add missing call to schedule() after prepare_to_wait()\n\nIn the unlikely event that reshape progresses past the current request\nwhile it is waiting for a stripe we need to schedule() before retrying\nfor 2 reasons:\n1/ Prevent list corruption from duplicated list_add() calls without\n intervening list_del().\n2/ Give the reshape code a chance to make some progress to resolve the\n conflict.\n\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "495d357301e1de01fabe30ce9a555301fb4675c3", "tree": "cff9506eec5e4008004a913acf8f58d85eae6de6", "parents": [ "af11c397fd8835c70ec0bb777104e4ab98b2d660" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:49:42 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Jun 18 08:49:42 2009 +1000" }, "message": "md/linear: use call_rcu to free obsolete \u0027conf\u0027 structures.\n\nCurrent, when we update the \u0027conf\u0027 structure, when adding a\ndrive to a linear array, we keep the old version around until\nthe array is finally stopped, as it is not safe to free it\nimmediately.\n\nNow that we have rcu protection on all accesses to \u0027conf\u0027,\nwe can use call_rcu to free it more promptly.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" } ], "next": "af11c397fd8835c70ec0bb777104e4ab98b2d660" }