)]}' { "log": [ { "commit": "7c88a168da8003fd4d8fb6ae103c4ecf29cb1130", "tree": "121d1b65615faf98e3c77f066b50527275d4b4c3", "parents": [ "3aa72873ffdcc2f7919743efbbefc351ec73f5cb" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Apr 21 19:43:58 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Apr 21 19:43:58 2011 +0200" }, "message": "block: don\u0027t propagate unlisted DISK_EVENTs to userland\n\nDISK_EVENT_MEDIA_CHANGE is used for both userland visible event and\ninternal event for revalidation of removeable devices. Some legacy\ndrivers don\u0027t implement proper event detection and continuously\ngenerate events under certain circumstances. For example, ide-cd\ngenerates media changed continuously if there\u0027s no media in the drive,\nwhich can lead to infinite loop of events jumping back and forth\nbetween the driver and userland event handler.\n\nThis patch updates disk event infrastructure such that it never\npropagates events not listed in disk-\u003eevents to userland. Those\nevents are processed the same for internal purposes but uevent\ngeneration is suppressed.\n\nThis also ensures that userland only gets events which are advertised\nin the @events sysfs node lowering risk of confusion.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "3aa72873ffdcc2f7919743efbbefc351ec73f5cb", "tree": "f159c777a489c6ac3721ba1f1cadbe2d86b19fa5", "parents": [ "584f79046780e10cb24367a691f8c28398a00e84" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Apr 21 19:28:35 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Apr 21 19:28:35 2011 +0200" }, "message": "elevator: check for ELEVATOR_INSERT_SORT_MERGE in !elvpriv case too\n\nThe sort insert is the one that goes to the IO scheduler. With\nthe SORT_MERGE addition, we could bypass IO scheduler setup\nbut still ask the IO scheduler to insert the request. This would\ncause an oops on switching IO schedulers through the sysfs\ninterface, unless the disk just happened to be idle while it\noccured.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "60735b6362f29b52b5635a2dfa9ab5ad39948345", "tree": "e590ce1697965a4ccb48af5cb729c64452cdcc6f", "parents": [ "ed5302d3c25006a9edc7a7fbea97a30483f89ef7" ], "author": { "name": "Tao Ma", "email": "boyu.mt@taobao.com", "time": "Tue Apr 19 13:50:40 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:51:53 2011 +0200" }, "message": "block: Remove the extra check in queue_requests_store\n\nIn queue_requests_store, the code looks like\n\tif (rl-\u003ecount[BLK_RW_SYNC] \u003e\u003d q-\u003enr_requests) {\n\t\tblk_set_queue_full(q, BLK_RW_SYNC);\n\t} else if (rl-\u003ecount[BLK_RW_SYNC]+1 \u003c\u003d q-\u003enr_requests) {\n\t\tblk_clear_queue_full(q, BLK_RW_SYNC);\n\t\twake_up(\u0026rl-\u003ewait[BLK_RW_SYNC]);\n\t}\nIf we don\u0027t satify the situation of \"if\", we can get that\nrl-\u003ecount[BLK_RW_SYNC} \u003c q-\u003enr_quests. It is the same as\nrl-\u003ecount[BLK_RW_SYNC]+1 \u003c\u003d q-\u003enr_requests.\nAll the \"else\" should satisfy the \"else if\" check so it isn\u0027t\nneeded actually.\n\nSigned-off-by: Tao Ma \u003cboyu.mt@taobao.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "ed5302d3c25006a9edc7a7fbea97a30483f89ef7", "tree": "7322cdd2607d7d8ffaf8a6539ca001771928363f", "parents": [ "d350e6b6e819df0a383ff34465720bfaa0f91c79" ], "author": { "name": "Liu Yuan", "email": "tailai.ly@taobao.com", "time": "Tue Apr 19 13:47:58 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:51:53 2011 +0200" }, "message": "block, blk-sysfs: Fix an err return path in blk_register_queue()\n\nWe do not call blk_trace_remove_sysfs() in err return path\nif kobject_add() fails. This path fixes it.\n\nCc: stable@kernel.org\nSigned-off-by: Liu Yuan \u003ctailai.ly@taobao.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "d350e6b6e819df0a383ff34465720bfaa0f91c79", "tree": "338c44b5ffaab070c3fdf2f2772cfd50f6163559", "parents": [ "c21e6beba8835d09bb80e34961430b13e60381c5" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:34:14 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:34:14 2011 +0200" }, "message": "block: remove stale kerneldoc member from __blk_run_queue()\n\nWe don\u0027t pass in a \u0027force_kblockd\u0027 anymore, get rid of the\nstsale comment.\n\nReported-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "c21e6beba8835d09bb80e34961430b13e60381c5", "tree": "cdf6f6d40130b95e641ab5db1de0f6ecc179054a", "parents": [ "5f45c69589b7d2953584e6cd0b31e35dbe960ad0" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:32:46 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 13:32:46 2011 +0200" }, "message": "block: get rid of QUEUE_FLAG_REENTER\n\nWe are currently using this flag to check whether it\u0027s safe\nto call into -\u003erequest_fn(). If it is set, we punt to kblockd.\nBut we get a lot of false positives and excessive punts to\nkblockd, which hurts performance.\n\nThe only real abuser of this infrastructure is SCSI. So export\nthe async queue run and convert SCSI over to use that. There\u0027s\nroom for improvement in that SCSI need not always use the async\ncall, but this fixes our performance issue and they can fix that\nup in due time.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "5f45c69589b7d2953584e6cd0b31e35dbe960ad0", "tree": "7f920cebcbbe6a1435975e355cee15289ddbc428", "parents": [ "bd900d4580107c899d43b262fbbd995f11097a43" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 09:10:35 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 19 09:10:35 2011 +0200" }, "message": "cfq-iosched: read_lock() does not always imply rcu_read_lock()\n\nFor some configurations of CONFIG_PREEMPT that is not true. So\nget rid of __call_for_each_cic() and always uses the explicitly\nrcu_read_lock() protected call_for_each_cic() instead.\n\nThis fixes a potential bug related to IO scheduler removal or\nonline switching.\n\nThanks to Paul McKenney for clarifying this.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "bd900d4580107c899d43b262fbbd995f11097a43", "tree": "c58b69acd1e5dc54e0e675901ab3d609c0cae945", "parents": [ "24ecfbe27f65563909b14492afda2f1c21f7c044" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 22:06:57 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 22:06:57 2011 +0200" }, "message": "block: kill blk_flush_plug_list() export\n\nWith all drivers and file systems converted, we only have\nin-core use of this function. So remove the export.\n\nReporteed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "24ecfbe27f65563909b14492afda2f1c21f7c044", "tree": "a7e51d903c400d0925f87be5f3069a5a44e0af24", "parents": [ "4521cc4ed5173f92714f6999a69910c3385fed68" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Mon Apr 18 11:41:33 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 11:41:33 2011 +0200" }, "message": "block: add blk_run_queue_async\n\nInstead of overloading __blk_run_queue to force an offload to kblockd\nadd a new blk_run_queue_async helper to do it explicitly. I\u0027ve kept\nthe blk_queue_stopped check for now, but I suspect it\u0027s not needed\nas the check we do when the workqueue items runs should be enough.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4521cc4ed5173f92714f6999a69910c3385fed68", "tree": "26ff58e87dfd303024df098935cc55e9a7bbe48d", "parents": [ "c3b328ac846bcf6b9a62c5563380a81ab723006d" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 11:36:39 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 11:36:39 2011 +0200" }, "message": "block: blk_delay_queue() should use kblockd workqueue\n\nReported-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "99e22598e9a8e0a996d69c8c0f6b7027cb57720a", "tree": "9cf18bc4681889bdfcbc0c845e384f809fb29fce", "parents": [ "b4cb290e0a7d19235bd075c2ad4d60dbab0bac15" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 09:59:55 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 09:59:55 2011 +0200" }, "message": "block: drop queue lock before calling __blk_run_queue() for kblockd punt\n\nIf we know we are going to punt to kblockd, we can drop the queue\nlock before calling into __blk_run_queue() since it only does a\nsafe bit test and a workqueue call. Since kblockd needs to grab\nthis very lock as one of the first things it does, it\u0027s a good\noptimization to drop the lock before waking kblockd.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "b4cb290e0a7d19235bd075c2ad4d60dbab0bac15", "tree": "2674fc5e3d0ee76e8b8205ab771fe7bbed173a88", "parents": [ "048c9374a749a27f16493cea033fa4a8ff492356" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 09:54:05 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 09:54:05 2011 +0200" }, "message": "Revert \"block: add callback function for unplug notification\"\n\nMD can\u0027t use this since it really requires us to be able to\nkeep more than a single piece of state for the unplug. Commit\n048c9374 added the required support for MD, so get rid of this\nnow unused code.\n\nThis reverts commit f75664570d8b75469cc468f23c2b27220984983b.\n\nConflicts:\n\n\tblock/blk-core.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "048c9374a749a27f16493cea033fa4a8ff492356", "tree": "64f520a1a2d2e1a9c30b45e306ce6f901f8f8d49", "parents": [ "a1b49cb7e2a7961ec3aa8b64860bf480d4ec9077" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 18 09:52:22 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 18 09:52:22 2011 +0200" }, "message": "block: Enhance new plugging support to support general callbacks\n\nmd/raid requires an unplug callback, but as it does not uses\nrequests the current code cannot provide one.\n\nSo allow arbitrary callbacks to be attached to the blk_plug.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "49cac01e1fa74174d72adb0e872504a7fefd7c01", "tree": "a1ab1974eceea3179a604413955ad8369ba715d7", "parents": [ "a237c1c5bc5dc5c76a21be922dca4826f3eca8ca" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Apr 16 13:51:05 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Apr 16 13:51:05 2011 +0200" }, "message": "block: make unplug timer trace event correspond to the schedule() unplug\n\nIt\u0027s a pretty close match to what we had before - the timer triggering\nwould mean that nobody unplugged the plug in due time, in the new\nscheme this matches very closely what the schedule() unplug now is.\nIt\u0027s essentially the difference between an explicit unplug (IO unplug)\nor an implicit unplug (timer unplug, we scheduled with pending IO\nqueued).\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f6603783f9f099bf7a83b3f6c689bbbf74f0e96e", "tree": "450065f77e95b6cd0eee13c9d8f721016be79839", "parents": [ "88b996cd0652280cc9b9fc70008fda15f14175e1" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Apr 15 15:49:07 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Apr 15 15:49:07 2011 +0200" }, "message": "block: only force kblockd unplugging from the schedule() path\n\nFor the explicit unplugging, we\u0027d prefer to kick things off\nimmediately and not pay the penalty of the latency to switch\nto kblockd. So let blk_finish_plug() do the run inline, while\nthe implicit-on-schedule-out unplug will punt to kblockd.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "88b996cd0652280cc9b9fc70008fda15f14175e1", "tree": "c842575e47ab094bced8a01b241568616e89366a", "parents": [ "80656b67b3988f83edd86a280d9937124fe62050" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Apr 15 15:20:10 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Apr 15 15:20:10 2011 +0200" }, "message": "block: cleanup the block plug helper functions\n\nIt\u0027s a bit of a mess currently. task-\u003eplug is being cleared\nand reset in __blk_finish_plug(), and blk_finish_plug() is\ntesting for a NULL plug which cannot happen even from schedule()\nanymore since it uses blk_needs_flush_plug() to determine\nwhether to call into this function at all.\n\nSo get rid of some of the cruft.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "80656b67b3988f83edd86a280d9937124fe62050", "tree": "58d23b396fa17262b697134cf2a58651f26d3460", "parents": [ "f4af3c3d077a004762aaad052049c809fd8c6f0c" ], "author": { "name": "Liu Yuan", "email": "tailai.ly@taobao.com", "time": "Wed Apr 13 22:14:54 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Apr 13 22:14:54 2011 +0200" }, "message": "block, blk-sysfs: Use the variable directly instead of a function call\n\nIn the function blk_register_queue(), var _dev_ is already assigned by\ndisk_to_dev().So use it directly instead of calling disk_to_dev() again.\n\nSigned-off-by: Liu Yuan \u003ctailai.ly@taobao.com\u003e\n\nModified by me to delete an empty line in the same function while\nin there anyway.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f4af3c3d077a004762aaad052049c809fd8c6f0c", "tree": "c4cbbc37e357775fc8200e16c6eb9b3f14d30069", "parents": [ "cf82c798394cd443eed7d91f998b79a63f341e91" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 14:58:51 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 14:58:51 2011 +0200" }, "message": "block: move queue run on unplug to kblockd\n\nThere are worries that we are now consuming a lot more stack in\nsome cases, since we potentially call into IO dispatch from\nschedule() or io_schedule(). We can reduce this problem by moving\nthe running of the queue to kblockd, like the old plugging scheme\ndid as well.\n\nThis may or may not be a good idea from a performance perspective,\ndepending on how many tasks have queue plugs running at the same\ntime. For even the slightly contended case, doing just a single\nqueue run from kblockd instead of multiple runs directly from the\nunpluggers will be faster.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "cf82c798394cd443eed7d91f998b79a63f341e91", "tree": "7ec33967528f4106dbbe32dad5e823c2c299b0a5", "parents": [ "dc6d36c9710d1fed42d1bbe7d8e4f742abd844c6" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:30:53 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:30:53 2011 +0200" }, "message": "block: kill queue_sync_plugs()\n\nThe original use for this dates back to when we had to track write\nrequests for serializing around barriers. That\u0027s not needed anymore,\nso kill it.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "dc6d36c9710d1fed42d1bbe7d8e4f742abd844c6", "tree": "88062deeeb7f8d6973056422f08d47f8c5a27c51", "parents": [ "f75664570d8b75469cc468f23c2b27220984983b" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:28:28 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:28:28 2011 +0200" }, "message": "block: readd plug trace event\n\nThis was removed with the queue plug state. But we can easily readd\nby checking if this is the first request going to this queue. It\u0027s\ngood information to have when tracing to see how effective the\nplugging is.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f75664570d8b75469cc468f23c2b27220984983b", "tree": "48f1e9b8576892cab65361e39b874e0e07d3b32a", "parents": [ "188112722cce083c8f1a7d0d84f55c2cd885920c" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:17:31 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:17:31 2011 +0200" }, "message": "block: add callback function for unplug notification\n\nMD would like to know when a queue is unplugged, so it can flush\nit\u0027s bitmap writes. Add such a callback.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "188112722cce083c8f1a7d0d84f55c2cd885920c", "tree": "eb285b823d2dd5a7fa88e9d6a1ee77299c14ff88", "parents": [ "94b5eb28b41cc79d9713696e0005ae167b5afd1b" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:11:24 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:12:29 2011 +0200" }, "message": "block: add comment on why we save and disable interrupts in flush_plug_list()\n\nIt\u0027s done at the top to avoid doing it for every queue we unplug.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "94b5eb28b41cc79d9713696e0005ae167b5afd1b", "tree": "404680c86a360bfe623b5a287ee87dd5263b5a8e", "parents": [ "d9c97833179036408e53ef5f3f5c7eaf781769bc" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:12:19 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 12 10:12:19 2011 +0200" }, "message": "block: fixup block IO unplug trace call\n\nIt was removed with the on-stack plugging, readd it and track the\ndepth of requests added when flushing the plug.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "109b81296c63228578d4760794d8dd46e02eddfb", "tree": "2f723e1b69ab08e12528f677f04edbfc7473b7c5", "parents": [ "4263a2f1dad8c8e7ce2352a0cbc882c2b0c044a9" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 11 14:13:10 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Apr 11 14:13:10 2011 +0200" }, "message": "block: splice plug list to local context\n\nIf the request_fn ends up blocking, we could be re-entering\nthe plug flush. Since the list is protected by explicitly\nnot allowing schedule events, this isn\u0027t a terribly good idea.\n\nAdditionally, it can cause us to recurse. As request_fn called by\n__blk_run_queue is allowed to \u0027schedule()\u0027 (after dropping the queue\nlock of course), it is possible to get a recursive call:\n\n schedule -\u003e blk_flush_plug -\u003e __blk_finish_plug -\u003e flush_plug_list\n -\u003e __blk_run_queue -\u003e request_fn -\u003e schedule\n\nWe must make sure that the second schedule does not call into\nblk_flush_plug again. So instead of leaving the list of requests on\nblk_plug-\u003elist, move them to a separate list leaving blk_plug-\u003elist\nempty.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "42933bac11e811f02200c944d8562a15f8ec4ff0", "tree": "fcdd9afe56eb0e746565ddd1f92f22d36678b843", "parents": [ "2b9accbee563f535046ff2cd382d0acaa92e130c", "25985edcedea6396277003854657b5f3cb31a628" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Apr 07 11:14:49 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Apr 07 11:14:49 2011 -0700" }, "message": "Merge branch \u0027for-linus2\u0027 of git://git.profusion.mobi/users/lucas/linux-2.6\n\n* \u0027for-linus2\u0027 of git://git.profusion.mobi/users/lucas/linux-2.6:\n Fix common misspellings\n" }, { "commit": "f83e826181f7f8fb152e4190d03854fc3a5dd040", "tree": "989f440665d529587fcc6365b1968e2b29fb7c94", "parents": [ "a63a5cf84dac7a23a57c800eea5734701e7d3c04" ], "author": { "name": "Konstantin Khlebnikov", "email": "khlebnikov@openvz.org", "time": "Mon Apr 04 00:15:02 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:52:49 2011 +0200" }, "message": "block: fix request sorting at unplug\n\nComparison function for list_sort() must be anticommutative,\notherwise it is not sorting in ordinary meaning.\n\nBut fortunately list_sort() always check ((*cmp)(priv, a, b) \u003c\u003d 0)\nit not distinguish negative and zero, so comparison function can\nimplement only less-or-equal instead of full three-way comparison.\n\nSigned-off-by: Konstantin Khlebnikov \u003ckhlebnikov@openvz.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a63a5cf84dac7a23a57c800eea5734701e7d3c04", "tree": "965552b96d7eacc21206a4f28672efd24a04d913", "parents": [ "7dcda1c96d7c643101d4a05579ef4512a4baa7ef" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Apr 01 21:02:31 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:52:43 2011 +0200" }, "message": "dm: improve block integrity support\n\nThe current block integrity (DIF/DIX) support in DM is verifying that\nall devices\u0027 integrity profiles match during DM device resume (which\nis past the point of no return). To some degree that is unavoidable\n(stacked DM devices force this late checking). But for most DM\ndevices (which aren\u0027t stacking on other DM devices) the ideal time to\nverify all integrity profiles match is during table load.\n\nIntroduce the notion of an \"initialized\" integrity profile: a profile\nthat was blk_integrity_register()\u0027d with a non-NULL \u0027blk_integrity\u0027\ntemplate. Add blk_integrity_is_initialized() to allow checking if a\nprofile was initialized.\n\nUpdate DM integrity support to:\n- check all devices with _initialized_ integrity profiles match\n during table load; uninitialized profiles (e.g. for underlying DM\n device(s) of a stacked DM device) are ignored.\n- disallow a table load that would result in an integrity profile that\n conflicts with a DM device\u0027s existing (in-use) integrity profile\n- avoid clearing an existing integrity profile\n- validate all integrity profiles match during resume; but if they\n don\u0027t all we can do is report the mismatch (during resume we\u0027re past\n the point of no return)\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6f0379377047b18103b88ce33c03e5b19747ae57", "tree": "83a39f34bd29c38ab46faa21d65a45831a659287", "parents": [ "ee3dea3549444e6e76d27af48b4929983e6f023c" ], "author": { "name": "Andreas Schwab", "email": "schwab@linux-m68k.org", "time": "Wed Mar 30 12:21:56 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:51:37 2011 +0200" }, "message": "blk-throttle: don\u0027t call xchg on bool\n\nxchg does not work portably with smaller than 32bit types.\n\nSigned-off-by: Andreas Schwab \u003cschwab@linux-m68k.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "53d63e6b0dfb95882ec0219ba6bbd50cde423794", "tree": "46fe9518a681943b0daccf39267d588ff03717f5", "parents": [ "b710a480554f2be682bac3cb59b0e085ba3d644b" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 30 13:27:09 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:51:37 2011 +0200" }, "message": "block: make the flush insertion use the tail of the dispatch list\n\nIt\u0027s not a preempt type request, in fact we have to insert it\nbehind requests that do specify INSERT_FRONT.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "b710a480554f2be682bac3cb59b0e085ba3d644b", "tree": "bd1b9d5080bca2b35c861ddbc98d4116445bebfb", "parents": [ "8182924bc5850281985d73c312876746acd390b5" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 30 09:52:30 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:51:37 2011 +0200" }, "message": "block: get rid of elv_insert() interface\n\nMerge it with __elv_add_request(), it\u0027s pretty pointless to\nhave a function with only two callers. The main interface\nis elv_add_request()/__elv_add_request().\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "8182924bc5850281985d73c312876746acd390b5", "tree": "b095bab06ba6fa55b26047534ebe31004f531e18", "parents": [ "b2a8b4b81966094703088a7bc76a313af841924d" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 30 09:51:33 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Apr 05 23:51:37 2011 +0200" }, "message": "block: dump request state on seeing a corrupted request completion\n\nCurrently we just dump a non-informative \u0027request botched\u0027 message.\nLets actually try and print something sane to help debug issues\naround this.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "25985edcedea6396277003854657b5f3cb31a628", "tree": "f026e810210a2ee7290caeb737c23cb6472b7c38", "parents": [ "6aba74f2791287ec407e0f92487a725a25908067" ], "author": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Wed Mar 30 22:57:33 2011 -0300" }, "committer": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Thu Mar 31 11:26:23 2011 -0300" }, "message": "Fix common misspellings\n\nFixes generated by \u0027codespell\u0027 and manually reviewed.\n\nSigned-off-by: Lucas De Marchi \u003clucas.demarchi@profusion.mobi\u003e\n" }, { "commit": "ad3d9d7ede04a9c71be7a9fe1a23961817f371f7", "tree": "33b0eded3b689020529cf8c0d4febf956ff10140", "parents": [ "401a18e92ce32cd0ddfa5738899ca2b8114f2bbf" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 25 16:58:59 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 25 17:04:08 2011 +0100" }, "message": "block: fix issue with calling blk_stop_queue() from the request_fn handler\n\nWhen the queue work handler was converted to delayed work, the\nstopping was inadvertently made sync as well. Change this back\nto being async stop, using __cancel_delayed_work() instead of\ncancel_delayed_work().\n\nReported-by: Jeremy Fitzhardinge \u003cjeremy@goop.org\u003e\nReported-by: Chris Mason \u003cchris.mason@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "401a18e92ce32cd0ddfa5738899ca2b8114f2bbf", "tree": "f8c7dbb61a7a90ae43065075b640512899fafd28", "parents": [ "6c5103890057b1bb781b26b7aae38d33e4c517d8" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 25 16:57:52 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 25 17:04:08 2011 +0100" }, "message": "block: fix bug with inserting flush requests as sort/merge\n\nWith the introduction of the on-stack plugging, we would assume\nthat any request being inserted was a normal file system request.\nAs flush/fua requires a special insert mode, this caused problems.\n\nFix this up by checking for this in flush_plug_list() and use\nthe appropriate insert mechanism.\n\nBig thanks goes to Markus Tripplesdorf for tirelessly testing\npatches, and to Sergey Senozhatsky for helping find the real\nissue.\n\nReported-by: Markus Tripplesdorf \u003cmarkus@trippelsdorf.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6c5103890057b1bb781b26b7aae38d33e4c517d8", "tree": "e6e57961dcddcb5841acb34956e70b9dc696a880", "parents": [ "3dab04e6978e358ad2307bca563fabd6c5d2c58b", "9d2e157d970a73b3f270b631828e03eb452d525e" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 24 10:16:26 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 24 10:16:26 2011 -0700" }, "message": "Merge branch \u0027for-2.6.39/core\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.39/core\u0027 of git://git.kernel.dk/linux-2.6-block: (65 commits)\n Documentation/iostats.txt: bit-size reference etc.\n cfq-iosched: removing unnecessary think time checking\n cfq-iosched: Don\u0027t clear queue stats when preempt.\n blk-throttle: Reset group slice when limits are changed\n blk-cgroup: Only give unaccounted_time under debug\n cfq-iosched: Don\u0027t set active queue in preempt\n block: fix non-atomic access to genhd inflight structures\n block: attempt to merge with existing requests on plug flush\n block: NULL dereference on error path in __blkdev_get()\n cfq-iosched: Don\u0027t update group weights when on service tree\n fs: assign sb-\u003es_bdi to default_backing_dev_info if the bdi is going away\n block: Require subsystems to explicitly allocate bio_set integrity mempool\n jbd2: finish conversion from WRITE_SYNC_PLUG to WRITE_SYNC and explicit plugging\n jbd: finish conversion from WRITE_SYNC_PLUG to WRITE_SYNC and explicit plugging\n fs: make fsync_buffers_list() plug\n mm: make generic_writepages() use plugging\n blk-cgroup: Add unaccounted time to timeslice_used.\n block: fixup plugging stubs for !CONFIG_BLOCK\n block: remove obsolete comments for blkdev_issue_zeroout.\n blktrace: Use rq-\u003ecmd_flags directly in blk_add_trace_rq.\n ...\n\nFix up conflicts in fs/{aio.c,super.c}\n" }, { "commit": "c4ade94fc00f8b34589719d8a347f658b6d3951e", "tree": "24868fc90f85ce3cba1b8218e771ce00506c3f51", "parents": [ "62a37f6badd1ac97ba07d543b5d4be2f9cb17217" ], "author": { "name": "Li, Shaohua", "email": "shaohua.li@intel.com", "time": "Wed Mar 23 08:30:34 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 23 08:30:34 2011 +0100" }, "message": "cfq-iosched: removing unnecessary think time checking\n\nRemoving think time checking. A high thinktime queue might means the queue\ndispatches several requests and then do away. Limitting such queue seems\nmeaningless. And also this can simplify code. This is suggested by Vivek.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "62a37f6badd1ac97ba07d543b5d4be2f9cb17217", "tree": "b659ac7b4c9fce787694bb8a33affa229d1cd552", "parents": [ "04521db04e9a11e74b0252d222051cb194487f4d" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Wed Mar 23 08:25:44 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 23 08:25:44 2011 +0100" }, "message": "cfq-iosched: Don\u0027t clear queue stats when preempt.\n\nFor v2, I added back lines to cfq_preempt_queue() that were removed\nduring updates for accounting unaccounted_time. Thanks for pointing out\nthat I\u0027d missed these, Vivek.\n\nPrevious commit \"cfq-iosched: Don\u0027t set active queue in preempt\" wrongly\ncleared stats for preempting queues when it shouldn\u0027t have, because when\nwe choose a queue to preempt, it still isn\u0027t necessarily scheduled next.\n\nThanks to Vivek Goyal for figuring this out and understanding how the\npreemption code works.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "04521db04e9a11e74b0252d222051cb194487f4d", "tree": "4a07811529a16f9e25c4de44ef0bf2621d360249", "parents": [ "9026e521c0da0731eb31f9f9022dd00cc3cd8885" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Tue Mar 22 21:54:29 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 22 21:55:00 2011 +0100" }, "message": "blk-throttle: Reset group slice when limits are changed\n\nLina reported that if throttle limits are initially very high and then\ndropped, then no new bio might be dispatched for a long time. And the\nreason being that after dropping the limits we don\u0027t reset the existing\nslice and do the rate calculation with new low rate and account the bios\ndispatched at high rate. To fix it, reset the slice upon rate change.\n\nhttps://lkml.org/lkml/2011/3/10/298\n\nAnother problem with very high limit is that we never queued the\nbio on throtl service tree. That means we kept on extending the\ngroup slice but never trimmed it. Fix that also by regulary\ntrimming the slice even if bio is not being queued up.\n\nReported-by: Lina Lu \u003clulina_nuaa@foxmail.com\u003e\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "9026e521c0da0731eb31f9f9022dd00cc3cd8885", "tree": "b2e9f44d22d009efcac6bf4a25a620017ac903a0", "parents": [ "eda5e0c91fed2d2a38a341b0957263406d330274" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Tue Mar 22 21:26:54 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 22 21:26:54 2011 +0100" }, "message": "blk-cgroup: Only give unaccounted_time under debug\n\nThis change moves unaccounted_time to only be reported when\nCONFIG_DEBUG_BLK_CGROUP is true.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "eda5e0c91fed2d2a38a341b0957263406d330274", "tree": "d09564b4ef18dc8f9703a3d67def655b24b992c8", "parents": [ "1e9bb8808ac11094d711d20d580e7b45a4992d0c" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Tue Mar 22 21:26:49 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 22 21:26:49 2011 +0100" }, "message": "cfq-iosched: Don\u0027t set active queue in preempt\n\nCommit \"Add unaccounted time to timeslice_used\" changed the behavior of\ncfq_preempt_queue to set cfqq active. Vivek pointed out that other\npreemption rules might get involved, so we shouldn\u0027t manually set which\nqueue is active.\n\nThis cleans up the code to just clear the queue stats at preemption\ntime.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "5e84ea3a9c662dc2d7a48703a4468fad954a3b7f", "tree": "3fa0fb26a7c8a970213584104cc2498ef46d60a3", "parents": [ "4345caba340f051e10847924fc078ae18ed6695c" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 21 10:14:27 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 21 10:14:27 2011 +0100" }, "message": "block: attempt to merge with existing requests on plug flush\n\nOne of the disadvantages of on-stack plugging is that we potentially\nlose out on merging since all pending IO isn\u0027t always visible to\neverybody. When we flush the on-stack plugs, right now we don\u0027t do\nany checks to see if potential merge candidates could be utilized.\n\nCorrect this by adding a new insert variant, ELEVATOR_INSERT_SORT_MERGE.\nIt works just ELEVATOR_INSERT_SORT, but first checks whether we can\nmerge with an existing request before doing the insertion (if we fail\nmerging).\n\nThis fixes a regression with multiple processes issuing IO that\ncan be merged.\n\nThanks to Shaohua Li \u003cshaohua.li@intel.com\u003e for testing and fixing\nan accounting bug.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "c55d267de274d308927b60c3e740c1a826832317", "tree": "21b53a8c725d9f9650f60d94b349459d5b8dae10", "parents": [ "61ef46fd45c3c62dc7c880a45dd2aa841b9af8fb", "bc898c97f7ba24def788d9f80786cf028a197122" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 17 17:54:40 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 17 17:54:40 2011 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6: (170 commits)\n [SCSI] scsi_dh_rdac: Add MD36xxf into device list\n [SCSI] scsi_debug: add consecutive medium errors\n [SCSI] libsas: fix ata list corruption issue\n [SCSI] hpsa: export resettable host attribute\n [SCSI] hpsa: move device attributes to avoid forward declarations\n [SCSI] scsi_debug: Logical Block Provisioning (SBC3r26)\n [SCSI] sd: Logical Block Provisioning update\n [SCSI] Include protection operation in SCSI command trace\n [SCSI] hpsa: fix incorrect PCI IDs and add two new ones (2nd try)\n [SCSI] target: Fix volume size misreporting for volumes \u003e 2TB\n [SCSI] bnx2fc: Broadcom FCoE offload driver\n [SCSI] fcoe: fix broken fcoe interface reset\n [SCSI] fcoe: precedence bug in fcoe_filter_frames()\n [SCSI] libfcoe: Remove stale fcoe-netdev entries\n [SCSI] libfcoe: Move FCOE_MTU definition from fcoe.h to libfcoe.h\n [SCSI] libfc: introduce __fc_fill_fc_hdr that accepts fc_hdr as an argument\n [SCSI] fcoe, libfc: initialize EM anchors list and then update npiv EMs\n [SCSI] Revert \"[SCSI] libfc: fix exchange being deleted when the abort itself is timed out\"\n [SCSI] libfc: Fixing a memory leak when destroying an interface\n [SCSI] megaraid_sas: Version and Changelog update\n ...\n\nFix up trivial conflicts due to whitespace differences in\ndrivers/scsi/libsas/{sas_ata.c,sas_scsi_host.c}\n" }, { "commit": "8184f93eced1e304721c2a55c00d87d5a14f8907", "tree": "b5ed43533eee883bc25920c191cb095584d29822", "parents": [ "95f28604a65b1c40b6c6cd95e58439cd7ded3add" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Thu Mar 17 16:12:36 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 17 16:12:36 2011 +0100" }, "message": "cfq-iosched: Don\u0027t update group weights when on service tree\n\nVersion 3 is updated to apply to for-2.6.39/core.\n\nFor version 2, I took Vivek\u0027s advice and made sure we update the group\nweight from cfq_group_service_tree_add().\n\nIf a weight was updated while a group is on the service tree, the\ncalculation for the total weight of the service tree can be adjusted\nimproperly, which either leads to bad service tree weights, or\npotentially crashes (if total_weight becomes 0).\n\nThis patch defers updates to the weight until a group is off the service\ntree.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "167400d34070ebbc408dc0f447c4ddb4bf837360", "tree": "19100b0511a7e3e18f6f61d90a5fe5f1c40f59c4", "parents": [ "1f940bdfc0d03265d178d9dfd840d854819f797d" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Sat Mar 12 16:54:00 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Mar 12 16:54:00 2011 +0100" }, "message": "blk-cgroup: Add unaccounted time to timeslice_used.\n\nThere are two kind of times that tasks are not charged for: the first\nseek and the extra time slice used over the allocated timeslice. Both\nof these exported as a new unaccounted_time stat.\n\nI think it would be good to have this reported in \u0027time\u0027 as well, but\nthat is probably a separate discussion.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "eba2ed9c9636d9e7ed203a2498ed230b48341709", "tree": "c989fdc098b09947888f67badbfeecb1e04cd1fc", "parents": [ "805f6b5e1cbfedfb9b3d354013e7f4b13a79270f" ], "author": { "name": "Tao Ma", "email": "boyu.mt@taobao.com", "time": "Fri Mar 11 20:13:54 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 11 20:13:54 2011 +0100" }, "message": "block: remove obsolete comments for blkdev_issue_zeroout.\n\nbarrier is already removed, so remove the obsolete comments\nin blkdev_issue_zeroout.\n\nCc: Jens Axboe \u003cjaxboe@fusionio.com\u003e\nSigned-off-by: Tao Ma \u003cboyu.mt@taobao.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "0aeea18964173715a1037034ef6838198f319319", "tree": "62ff402a41a675f0269bf5172d14b7db8c4e47ee", "parents": [ "9179746652faf0aba07b8b7f770dcf29892a24c6" ], "author": { "name": "Lukas Czerner", "email": "lczerner@redhat.com", "time": "Fri Mar 11 10:23:53 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 11 15:36:08 2011 +0100" }, "message": "block: fix mis-synchronisation in blkdev_issue_zeroout()\n\nBZ29402\nhttps://bugzilla.kernel.org/show_bug.cgi?id\u003d29402\n\nWe can hit serious mis-synchronization in bio completion path of\nblkdev_issue_zeroout() leading to a panic.\n\nThe problem is that when we are going to wait_for_completion() in\nblkdev_issue_zeroout() we check if the bb.done equals issued (number of\nsubmitted bios). If it does, we can skip the wait_for_completition()\nand just out of the function since there is nothing to wait for.\nHowever, there is a ordering problem because bio_batch_end_io() is\ncalling atomic_inc(\u0026bb-\u003edone) before complete(), hence it might seem to\nblkdev_issue_zeroout() that all bios has been completed and exit. At\nthis point when bio_batch_end_io() is going to call complete(bb-\u003ewait),\nbb and wait does not longer exist since it was allocated on stack in\nblkdev_issue_zeroout() \u003d\u003d\u003e panic!\n\n(thread 1) (thread 2)\nbio_batch_end_io() blkdev_issue_zeroout()\n if(bb) { ...\n if (bb-\u003eend_io) ...\n bb-\u003eend_io(bio, err); ...\n atomic_inc(\u0026bb-\u003edone); ...\n ... while (issued !\u003d atomic_read(\u0026bb.done))\n ... (let issued \u003d\u003d bb.done)\n ... (do the rest of the function)\n ... return ret;\n complete(bb-\u003ewait);\n ^^^^^^^^\n panic\n\nWe can fix this easily by simplifying bio_batch and completion counting.\n\nAlso remove bio_end_io_t *end_io since it is not used.\n\nSigned-off-by: Lukas Czerner \u003clczerner@redhat.com\u003e\nReported-by: Eric Whitney \u003ceric.whitney@hp.com\u003e\nTested-by: Eric Whitney \u003ceric.whitney@hp.com\u003e\nReviewed-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nCC: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4c63f5646e405b5010cc9499419060bf2e838f5b", "tree": "df91ba315032c8ec4aafeb3ab96fdfa7c6c656e1", "parents": [ "cafb0bfca1a73efd6d8a4a6a6a716e6134b96c24", "69d60eb96ae8a73cf9b79cf28051caf973006011" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "message": "Merge branch \u0027for-2.6.39/stack-plug\u0027 into for-2.6.39/core\n\nConflicts:\n\tblock/blk-core.c\n\tblock/blk-flush.c\n\tdrivers/md/raid1.c\n\tdrivers/md/raid10.c\n\tdrivers/md/raid5.c\n\tfs/nilfs2/btnode.c\n\tfs/nilfs2/mdt.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "69d60eb96ae8a73cf9b79cf28051caf973006011", "tree": "254be7302abbd6a21d4f54cfdad6a0af6db282d0", "parents": [ "721a9602e6607417c6bc15b18e97a2f35266c690" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Mar 09 08:27:37 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:27 2011 +0100" }, "message": "blk-throttle: Use blk_plug in throttle dispatch\n\nUse plug in throttle dispatch also as we are dispatching a bunch of\nbios in throttle context and some of them might merge.\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "721a9602e6607417c6bc15b18e97a2f35266c690", "tree": "4987991e43f35b8b3b685fea0040c5265b578996", "parents": [ "cf15900e1209d5b46ec2d24643adbf561830935f" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 09 11:56:30 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:27 2011 +0100" }, "message": "block: kill off REQ_UNPLUG\n\nWith the plugging now being explicitly controlled by the\nsubmitter, callers need not pass down unplugging hints\nto the block layer. If they want to unplug, it\u0027s because they\nmanually plugged on their own - in which case, they should just\nunplug at will.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7eaceaccab5f40bbfda044629a6298616aeaed50", "tree": "33954d12f63e25a47eb6d86ef3d3d0a5e62bf752", "parents": [ "73c101011926c5832e6e141682180c4debe2cf45" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "message": "block: remove per-queue plugging\n\nCode has been converted over to the new explicit on-stack plugging,\nand delay users have been converted to use the new API for that.\nSo lets kill off the old plugging along with aops-\u003esync_page().\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "73c101011926c5832e6e141682180c4debe2cf45", "tree": "b8eeb521a7833cb198d6f39d5a931d820e2a663f", "parents": [ "a488e74976bf0a9bccecdd094378394942dacef1" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 08 13:19:51 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:45:54 2011 +0100" }, "message": "block: initial patch for on-stack per-task plugging\n\nThis patch adds support for creating a queuing context outside\nof the queue itself. This enables us to batch up pieces of IO\nbefore grabbing the block device queue lock and submitting them to\nthe IO scheduler.\n\nThe context is created on the stack of the process and assigned in\nthe task structure, so that we can auto-unplug it if we hit a schedule\nevent.\n\nThe current queue plugging happens implicitly if IO is submitted to\nan empty device, yet callers have to remember to unplug that IO when\nthey are going to wait for it. This is an ugly API and has caused bugs\nin the past. Additionally, it requires hacks in the vm (-\u003esync_page()\ncallback) to handle that logic. By switching to an explicit plugging\nscheme we make the API a lot nicer and can get rid of the -\u003esync_page()\nhack in the vm.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "3cca6dc1c81e2407928dc4c6105252146fd3924f", "tree": "b78b0d93e7c02abdc37e1d5a6204ab6b94d56fd4", "parents": [ "53f22956effe1c9e7961b8c6e4362ecca5e460b7" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 11:08:00 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:45:54 2011 +0100" }, "message": "block: add API for delaying work/request_fn a little bit\n\nCurrently we use plugging for that, but as plugging is going away,\nwe need an alternative mechanism.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "facc31ddc3570a3a0d8951c94f16b898e01b464d", "tree": "23b8294910c26cabb4335a418b7ca27e52eaaaa8", "parents": [ "df457f845e5449be2e7d96668791f789b3770ac7" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 09 19:54:27 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 09 19:54:27 2011 +0100" }, "message": "block: Don\u0027t implicitly trigger event check on disk_unblock_events()\n\nCurrently, disk_unblock_events() implicitly kick event check if the\nblock count reaches zero. This behavior is not described in the\ncomment and hinders with future changes. Make the unblocker\nexplicitly check events by calling disk_check_events() as necessary.\n\nThis patch doesn\u0027t cause any behavior difference.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Jens Axboe \u003caxboe@kernel.dk\u003e\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\n" }, { "commit": "df457f845e5449be2e7d96668791f789b3770ac7", "tree": "5279d8ad1379ab890a90453d5018bb473c90d7d5", "parents": [ "df677140281beb608f6748c341af7612f7bfe7a0" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Tue Mar 08 19:45:00 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 08 19:45:00 2011 +0100" }, "message": "blk-cgroup: Lower minimum weight from 100 to 10.\n\nWe\u0027ve found that we still get good, useful isolation at weights this\nlow. I\u0027d like to adjust the minimum so that any other changes can take\nthese values into account.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "de701c74a34005e637e1ca2634fbf28fd1debba2", "tree": "45a4af16e2a76cbf5866cfb3f284956988918a13", "parents": [ "231d704b4ab7491473c0b1a9cd0c6e0d1cba85b9" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Mon Mar 07 21:09:32 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 21:09:32 2011 +0100" }, "message": "blk-throttle: Some cleanups and race fixes in limit update code\n\nWhen throttle group limits are updated through cgroups, a thread is\nwoken up to process these updates. While reviewing that code, oleg noted\ncouple of race conditions existed in the code and he also suggested that\ncode can be simplified.\n\nThis patch fixes the races simplifies the code based on Oleg\u0027s suggestions:\n\n\t- Use xchg().\n\t- Introduced a common function throtl_update_blkio_group_common()\n which is shared now by all iops/bps update functions.\n\nReviewed-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Paul E. McKenney \u003cpaulmck@linux.vnet.ibm.com\u003e\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\n\nFixed a merge issue, throtl_schedule_delayed_work() takes throtl_data\nas the argument now, not the queue.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "231d704b4ab7491473c0b1a9cd0c6e0d1cba85b9", "tree": "a4666d35b6e11f92eb0fc4506bdd7f65185a7aa6", "parents": [ "b873c5d692d4d5453cceed18bb06c62bb1a73ac0" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Mon Mar 07 21:05:14 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 21:05:14 2011 +0100" }, "message": "blk-throttle: process limit change only through one function\n\nWith the help of cgroup interface one can go and upate the bps/iops\nlimits of existing group. Once the limits are udpated, a thread is\nwoken up to see if some blocked group needs recalculation based on new\nlimits and needs to be requeued.\n\nThere was also a piece of code where I was checking for group limit\nupdate when a fresh bio comes in. This patch gets rid of that piece of\ncode and keeps processing the limit change at one place\nthrotl_process_limit_change(). It just keeps the code simple and easy\nto understand.\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "b873c5d692d4d5453cceed18bb06c62bb1a73ac0", "tree": "f9d5816bc26ced37f187a141ee6c74c7b8e00fcf", "parents": [ "a60327107b56573c305ecc78e471dbdbb4d2f426", "e83a46bbb1d4c03defd733a64b727632a40059ad" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 09:40:21 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 09:40:21 2011 +0100" }, "message": "Merge branch \u0027block-for-2.6.39-core\u0027 of ssh://master.kernel.org/pub/scm/linux/kernel/git/tj/misc into for-2.6.39/core\n" }, { "commit": "a60327107b56573c305ecc78e471dbdbb4d2f426", "tree": "239521730642bc2f263d1daa9c49737467d6c1b9", "parents": [ "ef8a41df8c140f10108de75b01b6369d6e49113c" ], "author": { "name": "Gui Jianfeng", "email": "guijianfeng@cn.fujitsu.com", "time": "Mon Mar 07 09:28:09 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 09:28:09 2011 +0100" }, "message": "cfq-iosched: Fix update_vdisktime logic\n\nThe update_vdisktime logic is broken since commit\nb54ce60eb7f61f8e314b8b241b0469eda3bb1d42, st-\u003emin_vdisktime never makes\na progress. Fix it.\n\nThanks Vivek for pointing it out.\n\nSigned-off-by: Gui Jianfeng \u003cguijianfen@cn.fujitsu.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "ef8a41df8c140f10108de75b01b6369d6e49113c", "tree": "2407ce27d3b51205a56193f26db1c1b40a5c556f", "parents": [ "93803e0140c6216b68fe926ccc611297120da273" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Mon Mar 07 09:26:29 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 09:26:29 2011 +0100" }, "message": "cfq-iosched: give busy sync queue no dispatch limit\n\nIf there are a sync and an async queue and the sync queue\u0027s think time\nis small, we can ignore the sync queue\u0027s dispatch quantum. Because the\nsync queue will always preempt the async queue, we don\u0027t need to care\nabout async\u0027s latency. This can fix a performance regression of\naiostress test, which is introduced by commit f8ae6e3eb825. The issue\nshould exist even without the commit, but the commit amplifies the\nimpact.\n\nThe initial post does the same optimization for RT queue too, but since\nI have no real workload for it, Vivek suggests to drop it.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nReviewed-by: Gui Jianfeng \u003cguijianfeng@cn.fujitsu.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "93803e0140c6216b68fe926ccc611297120da273", "tree": "c70e354f3029a00f83293b0d6bedf6738437ebab", "parents": [ "da527770007fce8e4541947d47918248286da875" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 08:59:06 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 07 08:59:06 2011 +0100" }, "message": "cfq-iosched: fix race in cfq_set_request()\n\nWe need to hold the queue lock over the reference increment,\nit\u0027s not atomic anymore.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "e83a46bbb1d4c03defd733a64b727632a40059ad", "tree": "c4bc4822b2d3af1bf38095f531adc0a2aac054a5", "parents": [ "da527770007fce8e4541947d47918248286da875", "fd51469fb68b987032e46297e0a4fe9020063c20" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Mar 04 19:09:02 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Mar 04 19:09:02 2011 +0100" }, "message": "Merge branch \u0027for-linus\u0027 of ../linux-2.6-block into block-for-2.6.39/core\n\nThis merge creates two set of conflicts. One is simple context\nconflicts caused by removal of throtl_scheduled_delayed_work() in\nfor-linus and removal of throtl_shutdown_timer_wq() in\nfor-2.6.39/core.\n\nThe other is caused by commit 255bb490c8 (block: blk-flush shouldn\u0027t\ncall directly into q-\u003erequest_fn() __blk_run_queue()) in for-linus\ncrashing with FLUSH reimplementation in for-2.6.39/core. The conflict\nisn\u0027t trivial but the resolution is straight-forward.\n\n* __blk_run_queue() calls in flush_end_io() and flush_data_end_io()\n should be called with @force_kblockd set to %true.\n\n* elv_insert() in blk_kick_flush() should use\n %ELEVATOR_INSERT_REQUEUE.\n\nBoth changes are to avoid invoking -\u003erequest_fn() directly from\nrequest completion path and closely match the changes in the commit\n255bb490c8.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "da527770007fce8e4541947d47918248286da875", "tree": "fc518172887409133e8dea827067209adfdb1c3c", "parents": [ "cd25f54961273c2e4cbd47441e04832468382a5e" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Mar 02 19:05:33 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 19:06:49 2011 -0500" }, "message": "block: Move blk_throtl_exit() call to blk_cleanup_queue()\n\nMove blk_throtl_exit() in blk_cleanup_queue() as blk_throtl_exit() is\nwritten in such a way that it needs queue lock. In blk_release_queue()\nthere is no gurantee that -\u003equeue_lock is still around.\n\nInitially blk_throtl_exit() was in blk_cleanup_queue() but Ingo reported\none problem.\n\n https://lkml.org/lkml/2010/10/23/86\n\n And a quick fix moved blk_throtl_exit() to blk_release_queue().\n\n commit 7ad58c028652753814054f4e3ac58f925e7343f4\n Author: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n Date: Sat Oct 23 20:40:26 2010 +0200\n\n block: fix use-after-free bug in blk throttle code\n\nThis patch reverts above change and does not try to shutdown the\nthrotl work in blk_sync_queue(). By avoiding call to\nthrotl_shutdown_timer_wq() from blk_sync_queue(), we should also avoid\nthe problem reported by Ingo.\n\nblk_sync_queue() seems to be used only by md driver and it seems to be\nusing it to make sure q-\u003eunplug_fn is not called as md registers its\nown unplug functions and it is about to free up the data structures\nused by unplug_fn(). Block throttle does not call back into unplug_fn()\nor into md. So there is no need to cancel blk throttle work.\n\nIn fact I think cancelling block throttle work is bad because it might\nhappen that some bios are throttled and scheduled to be dispatched later\nwith the help of pending work and if work is cancelled, these bios might\nnever be dispatched.\n\nBlock layer also uses blk_sync_queue() during blk_cleanup_queue() and\nblk_release_queue() time. That should be safe as we are also calling\nblk_throtl_exit() which should make sure all the throttling related\ndata structures are cleaned up.\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "c94a96ac93b4f5b8d1ff8430b1afa1a25610cf53", "tree": "3cc138b16df4c0802ec42868a6d1f2eed42ba695", "parents": [ "53f22956effe1c9e7961b8c6e4362ecca5e460b7" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Mar 02 19:04:42 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 19:06:49 2011 -0500" }, "message": "block: Initialize -\u003equeue_lock to internal lock at queue allocation time\n\nThere does not seem to be a clear convention whether q-\u003equeue_lock is\ninitialized or not when blk_cleanup_queue() is called. In the past it\nwas not necessary but now blk_throtl_exit() takes up queue lock by\ndefault and needs queue lock to be available.\n\nIn fact elevator_exit() code also has similar requirement just that it\nis less stringent in the sense that elevator_exit() is called only if\nelevator is initialized.\n\nTwo problems have been noticed because of ambiguity about spin lock\nstatus.\n\n - If a driver calls blk_alloc_queue() and then soon calls\n blk_cleanup_queue() almost immediately, (because some other\n\tdriver structure allocation failed or some other error happened)\n\tthen blk_throtl_exit() will run into issues as queue lock is not\n\tinitialized. Loop driver ran into this issue recently and I\n\tnoticed error paths in md driver too. Similar error paths should\n\texist in other drivers too.\n\n - If some driver provided external spin lock and zapped the lock\n before blk_cleanup_queue(), then it can lead to issues.\n\nSo this patch initializes the default queue lock at queue allocation time.\n\nblock throttling code is one of the users of queue lock and it is\ninitialized at the queue allocation time, so it makes sense to\ninitialize -\u003equeue_lock also to internal lock. A driver can overide that\nlock later. This will take care of the issue where a driver does not have\nto worry about initializing the queue lock to default before calling\nblk_cleanup_queue()\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "53f22956effe1c9e7961b8c6e4362ecca5e460b7", "tree": "a2f897a90971f68f0e10d065fe3289a243f2aa19", "parents": [ "0bbfeb8320421989d3e12bd95fae86b9ac0712aa" ], "author": { "name": "Liu Yuan", "email": "tailai.ly@taobao.com", "time": "Wed Mar 02 11:00:15 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 11:00:15 2011 -0500" }, "message": "block/genhd: Change some numerals into macros\n\nRename the numerals in the diskstats_show() into the macros.\n\nCc: Jens Axboe \u003caxboe@kernel.dk\u003e\nSigned-off-by: Liu Yuan \u003ctailai.ly@taobao.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "255bb490c8c27eed484d538efe6ef6a7473bd3f6", "tree": "630289b9de253a1f0575f29d2de019494016ff79", "parents": [ "1654e7411a1ad4999fe7890ef51d2a2bbb1fcf76" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 02 08:48:06 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 08:48:06 2011 -0500" }, "message": "block: blk-flush shouldn\u0027t call directly into q-\u003erequest_fn() __blk_run_queue()\n\nblk-flush decomposes a flush into sequence of multiple requests. On\ncompletion of a request, the next one is queued; however, block layer\nmust not implicitly call into q-\u003erequest_fn() directly from completion\npath. This makes the queue behave unexpectedly when seen from the\ndrivers and violates the assumption that q-\u003erequest_fn() is called\nwith process context + queue_lock.\n\nThis patch makes blk-flush the following two changes to make sure\nq-\u003erequest_fn() is not called directly from request completion path.\n\n- blk_flush_complete_seq_end_io() now asks __blk_run_queue() to always\n use kblockd instead of calling directly into q-\u003erequest_fn().\n\n- queue_next_fseq() uses ELEVATOR_INSERT_REQUEUE instead of\n ELEVATOR_INSERT_FRONT so that elv_insert() doesn\u0027t try to unplug the\n request queue directly.\n\nReported by Jan in the following threads.\n\n http://thread.gmane.org/gmane.linux.ide/48778\n http://thread.gmane.org/gmane.linux.ide/48786\n\nstable: applicable to v2.6.37.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Jan Beulich \u003cJBeulich@novell.com\u003e\nCc: \"David S. Miller\" \u003cdavem@davemloft.net\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "1654e7411a1ad4999fe7890ef51d2a2bbb1fcf76", "tree": "c8071cf8cc1aef9e776697b72aaca5a22a47c3d7", "parents": [ "291d24f6d9e7bbef81454fade8a44720665c7302" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 02 08:48:05 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Mar 02 08:48:05 2011 -0500" }, "message": "block: add @force_kblockd to __blk_run_queue()\n\n__blk_run_queue() automatically either calls q-\u003erequest_fn() directly\nor schedules kblockd depending on whether the function is recursed.\nblk-flush implementation needs to be able to explicitly choose\nkblockd. Add @force_kblockd.\n\nAll the current users are converted to specify %false for the\nparameter and this patch doesn\u0027t introduce any behavior change.\n\nstable: This is prerequisite for fixing ide oops caused by the new\n blk-flush implementation.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Jan Beulich \u003cJBeulich@novell.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "0bbfeb8320421989d3e12bd95fae86b9ac0712aa", "tree": "a23bb6686bd2586fe7c27d0773c63f9aeb254cd1", "parents": [ "6fae9c25134baffbeeb20031479e7ff6f6d8eec0" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Tue Mar 01 15:05:08 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 01 15:05:08 2011 -0500" }, "message": "cfq-iosched: Always provide group isolation.\n\nEffectively, make group_isolation\u003d1 the default and remove the tunable.\nThe setting group_isolation\u003d0 was because by default we idle on\nsync-noidle tree and on fast devices, this can be very harmful for\nthroughput.\n\nHowever, this problem can also be addressed by tuning slice_idle and\npossibly group_idle on faster storage devices.\n\nThis change simplifies the CFQ code by removing the feature entirely.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6fae9c25134baffbeeb20031479e7ff6f6d8eec0", "tree": "c9ab89992ce5293a43cd455a81dc8a5926a28a5e", "parents": [ "c186794dbb466b45cf40f942f2d09d6d5b4b0e42", "f5412be599602124d2bdd49947b231dd77c0bf99" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 01 15:04:39 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 01 15:04:39 2011 -0500" }, "message": "Merge commit \u0027v2.6.38-rc6\u0027 into for-2.6.39/core\n\nConflicts:\n\tblock/cfq-iosched.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "291d24f6d9e7bbef81454fade8a44720665c7302", "tree": "8e23a24b51e9a0106cbd7e5375e53a972794b8cd", "parents": [ "450adcbe518ab3a3953d8475309525d22de77cba" ], "author": { "name": "Ben Hutchings", "email": "ben@decadent.org.uk", "time": "Tue Mar 01 13:45:24 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 01 13:45:24 2011 -0500" }, "message": "block: fix kernel-doc format for blkdev_issue_zeroout\n\nSigned-off-by: Ben Hutchings \u003cben@decadent.org.uk\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "450adcbe518ab3a3953d8475309525d22de77cba", "tree": "352fd47d75b86804e590fd88c09f953a798ba8b0", "parents": [ "3e1f2356ce231488dc1fa844e5ce91bcb59fc2a1" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Tue Mar 01 13:40:54 2011 -0500" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 01 13:41:53 2011 -0500" }, "message": "blk-throttle: Do not use kblockd workqueue for throtl work\n\no Dominik Klein reported a system hang issue while doing some blkio\n throttling testing.\n\n https://lkml.org/lkml/2011/2/24/173\n\no Some tracing revealed that CFQ was not dispatching any more jobs as\n queue unplug was not happening. And queue unplug was not happening\n because unplug work was not being called as there was one throttling\n work on same cpu which as not finished yet. And throttling work had not\n finished as it was tyring to dispatch a bio to CFQ but all the request\n descriptors were consume to it was put to sleep.\n\no So basically it is a cyclic dependecny between CFQ unplug work and\n throtl dispatch work. Tejun suggested that use separate workqueue for\n such cases.\n\no This patch uses a separate workqueue for throttle related work and\n does not rely on kblockd workqueue anymore.\n\nCc: stable@kernel.org\nReported-by: Dominik Klein \u003cdk@in-telegence.net\u003e\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "638691a7a46a4ae9a7b56c603299e42d7f6e722e", "tree": "f744b3fbdd4704ef9ae122b67a14785b4a7a6b65", "parents": [ "0a93ea2e897bd793cc0aaaddc397eff32ac8d6fe", "f0b4f7e2f29af678bd9af43422c537dcb6008603" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Feb 25 11:13:26 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Feb 25 11:13:26 2011 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md:\n md: Fix - again - partition detection when array becomes active\n Fix over-zealous flush_disk when changing device size.\n md: avoid spinlock problem in blk_throtl_exit\n md: correctly handle probe of an \u0027mdp\u0027 device.\n md: don\u0027t set_capacity before array is active.\n md: Fix raid1-\u003eraid0 takeover\n" }, { "commit": "3c522cedb572bb8d2e4867f358bdaa7d0c53d88c", "tree": "b348cabd2574638702f9abcccd0b8b711b136162", "parents": [ "dcace5ac85c628af21878a1fa151e5e6403fb8eb" ], "author": { "name": "Miklos Szeredi", "email": "mszeredi@suse.cz", "time": "Thu Feb 24 15:45:41 2011 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Feb 24 08:54:21 2011 -0800" }, "message": "block: fix refcounting in BLKBSZSET\n\nAdam Kovari and others reported that disconnecting an USB drive with\nan ntfs-3g filesystem would cause \"kernel BUG at fs/inode.c:1421!\" to\nbe triggered.\n\nThe BUG could be traced back to ioctl(BLKBSZSET), which would\nerroneously decrement the refcount on the bdev. This is because\nblkdev_get() expects the refcount to be already incremented and either\nreturns success or decrements the refcount and returns an error.\n\nThe bug was introduced by e525fd89 (block: make blkdev_get/put()\nhandle exclusive access), which didn\u0027t take into account this behavior\nof blkdev_get().\n\nThis fixes\n https://bugzilla.kernel.org/show_bug.cgi?id\u003d29202\n(and likely 29792 too)\n\nReported-by: Adam Kovari \u003ckovariadam@gmail.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Miklos Szeredi \u003cmszeredi@suse.cz\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "93b270f76e7ef3b81001576860c2701931cdc78b", "tree": "abaca0e4d3e86721815498fafd06295dd9cfd002", "parents": [ "da9cf5050a2e3dbc3cf26a8d908482eb4485ed49" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Feb 24 17:25:47 2011 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu Feb 24 17:25:47 2011 +1100" }, "message": "Fix over-zealous flush_disk when changing device size.\n\nThere are two cases when we call flush_disk.\nIn one, the device has disappeared (check_disk_change) so any\ndata will hold becomes irrelevant.\nIn the oter, the device has changed size (check_disk_size_change)\nso data we hold may be irrelevant.\n\nIn both cases it makes sense to discard any \u0027clean\u0027 buffers,\nso they will be read back from the device if needed.\n\nIn the former case it makes sense to discard \u0027dirty\u0027 buffers\nas there will never be anywhere safe to write the data. In the\nsecond case it *does*not* make sense to discard dirty buffers\nas that will lead to file system corruption when you simply enlarge\nthe containing devices.\n\nflush_disk calls __invalidate_devices.\n__invalidate_device calls both invalidate_inodes and invalidate_bdev.\n\ninvalidate_inodes *does* discard I_DIRTY inodes and this does lead\nto fs corruption.\n\ninvalidate_bev *does*not* discard dirty pages, but I don\u0027t really care\nabout that at present.\n\nSo this patch adds a flag to __invalidate_device (calling it\n__invalidate_device2) to indicate whether dirty buffers should be\nkilled, and this is passed to invalidate_inodes which can choose to\nskip dirty inodes.\n\nflusk_disk then passes true from check_disk_change and false from\ncheck_disk_size_change.\n\ndm avoids tripping over this problem by calling i_size_write directly\nrathher than using check_disk_size_change.\n\nmd does use check_disk_size_change and so is affected.\n\nThis regression was introduced by commit 608aeef17a which causes\ncheck_disk_size_change to call flush_disk, so it is suitable for any\nkernel since 2.6.27.\n\nCc: stable@kernel.org\nAcked-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nCc: Andrew Patterson \u003candrew.patterson@hp.com\u003e\nCc: Jens Axboe \u003caxboe@kernel.dk\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "79775567e0439ca47eb9f501e52c4b713d44cf89", "tree": "1cb890e0a061b007ae10bd281a60ada60a6d2bb1", "parents": [ "751b2a7d623ead9e55f751a6087efeab454b5659" ], "author": { "name": "Hannes Reinecke", "email": "hare@suse.de", "time": "Tue Jan 18 10:13:13 2011 +0100" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Sat Feb 12 10:33:37 2011 -0600" }, "message": "[SCSI] block: improve detail in I/O error messages\n\nClassify severity of I/O errors for target, nexus, and\ntransport errors.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Hannes Reinecke \u003chare@suse.de\u003e\nAcked-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "c186794dbb466b45cf40f942f2d09d6d5b4b0e42", "tree": "07ccd819bb6cced2c19053d2ed97ebaac88f824e", "parents": [ "9d5a4e946ce5352f19400b6370f4cd8e72806278" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Feb 11 11:08:00 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Feb 11 11:08:00 2011 +0100" }, "message": "block: share request flush fields with elevator_private\n\nFlush requests are never put on the IO scheduler. Convert request\nstructure\u0027s elevator_private* into an array and have the flush fields\nshare a union with it.\n\nReclaim the space lost in \u0027struct request\u0027 by moving \u0027completion_data\u0027\nback in the union with \u0027rb_node\u0027.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "9d5a4e946ce5352f19400b6370f4cd8e72806278", "tree": "ba62a2ab56da1f943b0dbcae527c70b817a104de", "parents": [ "ae1b1539622fb46e51b4d13b3f9e5f4c713f86ae" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Feb 11 11:05:46 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Feb 11 11:05:46 2011 +0100" }, "message": "block: skip elevator data initialization for flush requests\n\nSkip elevator initialization for flush requests by passing priv\u003d0 to\nblk_alloc_request() in get_request(). As such elv_set_request() is\nnever called for flush requests.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "aceb91cd351bc3a19a783c901fe8a8070d5f6fa9", "tree": "a0863e940a77ae10d3c8b391488e29a8245b31be", "parents": [ "ae8eed2d0906bf0d8eb0c2a4651676a41d361297", "b8cf0e0e552ca48e9a00f518aeb4f5e03984022b" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Feb 09 11:45:21 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Feb 09 11:45:21 2011 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block:\n cdrom: support devices that have check_events but not media_changed\n cfq-iosched: Don\u0027t wait if queue already has requests.\n blkio-throttle: Avoid calling blkiocg_lookup_group() for root group\n cfq: rename a function to give it more appropriate name\n cciss: make cciss_revalidate not loop through CISS_MAX_LUNS volumes unnecessarily.\n drivers/block/aoe/Makefile: replace the use of \u003cmodule\u003e-objs with \u003cmodule\u003e-y\n loop: queue_lock NULL pointer derefence in blk_throtl_exit\n drivers/block/Makefile: replace the use of \u003cmodule\u003e-objs with \u003cmodule\u003e-y\n blktrace: Don\u0027t output messages if NOTIFY isn\u0027t set.\n" }, { "commit": "02a8f01b5a9f396d0327977af4c232d0f94c45fd", "tree": "ace818b81f26c30c473af698882dc9786d13afe5", "parents": [ "be2c6b1990904dbd43f3d9b90fa2c530504375cd" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Wed Feb 09 14:20:03 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Feb 09 14:22:36 2011 +0100" }, "message": "cfq-iosched: Don\u0027t wait if queue already has requests.\n\nCommit 7667aa0630407bc07dc38dcc79d29cc0a65553c1 added logic to wait for\nthe last queue of the group to become busy (have at least one request),\nso that the group does not lose out for not being continuously\nbacklogged. The commit did not check for the condition that the last\nqueue already has some requests. As a result, if the queue already has\nrequests, wait_busy is set. Later on, cfq_select_queue() checks the\nflag, and decides that since the queue has a request now and wait_busy\nis set, the queue is expired. This results in early expiration of the\nqueue.\n\nThis patch fixes the problem by adding a check to see if queue already\nhas requests. If it does, wait_busy is not set. As a result, time slices\ndo not expire early.\n\nThe queues with more than one request are usually buffered writers.\nTesting shows improvement in isolation between buffered writers.\n\nCc: stable@kernel.org\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nReviewed-by: Gui Jianfeng \u003cguijianfeng@cn.fujitsu.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "ae1b1539622fb46e51b4d13b3f9e5f4c713f86ae", "tree": "c5cb540141003a3ec7ebf0b8c6e01653ab6aaef5", "parents": [ "143a87f4c9c629067afea5b6703d66ea88c82f8e" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 25 12:43:54 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jan 25 12:43:54 2011 +0100" }, "message": "block: reimplement FLUSH/FUA to support merge\n\nThe current FLUSH/FUA support has evolved from the implementation\nwhich had to perform queue draining. As such, sequencing is done\nqueue-wide one flush request after another. However, with the\ndraining requirement gone, there\u0027s no reason to keep the queue-wide\nsequential approach.\n\nThis patch reimplements FLUSH/FUA support such that each FLUSH/FUA\nrequest is sequenced individually. The actual FLUSH execution is\ndouble buffered and whenever a request wants to execute one for either\nPRE or POSTFLUSH, it queues on the pending queue. Once certain\nconditions are met, a flush request is issued and on its completion\nall pending requests proceed to the next sequence.\n\nThis allows arbitrary merging of different type of flushes. How they\nare merged can be primarily controlled and tuned by adjusting the\nabove said \u0027conditions\u0027 used to determine when to issue the next\nflush.\n\nThis is inspired by Darrick\u0027s patches to merge multiple zero-data\nflushes which helps workloads with highly concurrent fsync requests.\n\n* As flush requests are never put on the IO scheduler, request fields\n used for flush share space with rq-\u003erb_node. rq-\u003ecompletion_data is\n moved out of the union. This increases the request size by one\n pointer.\n\n As rq-\u003eelevator_private* are used only by the iosched too, it is\n possible to reduce the request size further. However, to do that,\n we need to modify request allocation path such that iosched data is\n not allocated for flush requests.\n\n* FLUSH/FUA processing happens on insertion now instead of dispatch.\n\n- Comments updated as per Vivek and Mike.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: \"Darrick J. Wong\" \u003cdjwong@us.ibm.com\u003e\nCc: Shaohua Li \u003cshli@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nCc: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "143a87f4c9c629067afea5b6703d66ea88c82f8e", "tree": "6abeae59e1d309d3c1a11eab97e22639c7fc5bd5", "parents": [ "414b4ff5eecff0097d09c4a7da12e435fd503692" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 25 12:43:52 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jan 25 12:43:52 2011 +0100" }, "message": "block: improve flush bio completion\n\nbio\u0027s for flush are completed twice - once during the data phase and\none more time after the whole sequence is complete. The first\ncompletion shouldn\u0027t notify completion to the issuer.\n\nThis was achieved by skipping all bio completion steps in\nreq_bio_endio() for the first completion; however, this has two\ndrawbacks.\n\n* Error is not recorded in bio and must be tracked somewhere else.\n\n* Partial completion is not supported.\n\nBoth don\u0027t cause problems for the current users; however, they make\nfurther improvements difficult. Change req_bio_endio() such that it\nonly skips the actual notification part for the first completion. bio\ncompletion is implemented with partial completions on mind anyway so\nthis is as simple as moving the REQ_FLUSH_SEQ conditional such that\nonly calling of bio_endio() is skipped.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "414b4ff5eecff0097d09c4a7da12e435fd503692", "tree": "cc7cd46c1455712af672b3ff872800b3da895837", "parents": [ "3c0eee3fe6a3a1c745379547c7e7c904aa64f6d5" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 25 12:43:49 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jan 25 12:43:49 2011 +0100" }, "message": "block: add REQ_FLUSH_SEQ\n\nrq \u003d\u003d \u0026q-\u003eflush_rq was used to determine whether a rq is part of a\nflush sequence, which worked because all requests in a flush sequence\nwere sequenced using the single dedicated request. This is about to\nchange, so introduce REQ_FLUSH_SEQ flag to distinguish flush sequence\nrequests.\n\nThis patch doesn\u0027t cause any behavior change.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6a108a14fa356ef607be308b68337939e56ea94e", "tree": "1bf260572bd8f95ed867307a2bcf5d881c8ae4a6", "parents": [ "12fcdba1b7ae8b25696433f420b775aeb556d89b" ], "author": { "name": "David Rientjes", "email": "rientjes@google.com", "time": "Thu Jan 20 14:44:16 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 20 17:02:05 2011 -0800" }, "message": "kconfig: rename CONFIG_EMBEDDED to CONFIG_EXPERT\n\nThe meaning of CONFIG_EMBEDDED has long since been obsoleted; the option\nis used to configure any non-standard kernel with a much larger scope than\nonly small devices.\n\nThis patch renames the option to CONFIG_EXPERT in init/Kconfig and fixes\nreferences to the option throughout the kernel. A new CONFIG_EMBEDDED\noption is added that automatically selects CONFIG_EXPERT when enabled and\ncan be used in the future to isolate options that should only be\nconsidered for embedded systems (RISC architectures, SLOB, etc).\n\nCalling the option \"EXPERT\" more accurately represents its intention: only\nexpert users who understand the impact of the configuration changes they\nare making should enable it.\n\nReviewed-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nAcked-by: David Woodhouse \u003cdavid.woodhouse@intel.com\u003e\nSigned-off-by: David Rientjes \u003crientjes@google.com\u003e\nCc: Greg KH \u003cgregkh@suse.de\u003e\nCc: \"David S. Miller\" \u003cdavem@davemloft.net\u003e\nCc: Jens Axboe \u003caxboe@kernel.dk\u003e\nCc: Arnd Bergmann \u003carnd@arndb.de\u003e\nCc: Robin Holt \u003cholt@sgi.com\u003e\nCc: \u003clinux-arch@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "be2c6b1990904dbd43f3d9b90fa2c530504375cd", "tree": "f2ec4322adc9f0607b5a78367bc66d8fa333cf54", "parents": [ "ba5bd520f679c450fb6efa439618703bd0956daa" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Jan 19 08:25:02 2011 -0700" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jan 19 08:25:02 2011 -0700" }, "message": "blkio-throttle: Avoid calling blkiocg_lookup_group() for root group\n\no Jeff Moyer was doing some testing on a RAM backed disk and\n blkiocg_lookup_group() showed up high overhead after memcpy(). Similarly\n somebody else reported that blkiocg_lookup_group() is eating 6% extra\n cpu. Though looking at the code I can\u0027t think why the overhead of\n this function is so high. One thing is that it is called with very high\n frequency (once for every IO).\n\no For lot of folks blkio controller will be compiled in but they might\n not have actually created cgroups. Hence optimize the case of root\n cgroup where we can avoid calling blkiocg_lookup_group() if IO is happening\n in root group (common case).\n\nReported-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nAcked-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "ba5bd520f679c450fb6efa439618703bd0956daa", "tree": "f3957e89e43c4955ae261ea96da570f36450f51f", "parents": [ "68264e9d6781f7163e92c517769bb470fa43f6cd" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Jan 19 08:25:02 2011 -0700" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jan 19 08:25:02 2011 -0700" }, "message": "cfq: rename a function to give it more appropriate name\n\no Rename a function to give it more approprate name. We are calculating\n cfq queue slice and function name gives the impression as if cfq group\n slice length is being calculated.\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "c553f8e335c00a7cff3ab3f13e793b13d3f2207f", "tree": "f9371c9e1191711dd2e774e04d99f3fa90d697fa", "parents": [ "f8ae6e3eb8251be32c6e913393d9f8d9e0609489" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jan 14 08:41:03 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 14 08:41:03 2011 +0100" }, "message": "block cfq: compensate preempted queue even if it has no slice assigned\n\nIf a queue is preempted before it gets slice assigned, the queue doesn\u0027t get\ncompensation, which looks unfair. For such queue, we compensate it for a whole\nslice.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f8ae6e3eb8251be32c6e913393d9f8d9e0609489", "tree": "50b096572a4f8e7992e7e7e2e599e77d334c2b5b", "parents": [ "52cfd503ad7176d23a5dd7af3981744feb60622f" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jan 14 08:41:02 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 14 08:41:02 2011 +0100" }, "message": "block cfq: make queue preempt work for queues from different workload\n\nI got this:\n fio-874 [007] 2157.724514: 8,32 m N cfq874 preempt\n fio-874 [007] 2157.724519: 8,32 m N cfq830 slice expired t\u003d1\n fio-874 [007] 2157.724520: 8,32 m N cfq830 sl_used\u003d1 disp\u003d0 charge\u003d1 iops\u003d0 sect\u003d0\n fio-874 [007] 2157.724521: 8,32 m N cfq830 set_active wl_prio:0 wl_type:0\n fio-874 [007] 2157.724522: 8,32 m N cfq830 Not idling. st-\u003ecount:1\n\ncfq830 is an async queue, and preempted by a sync queue cfq874. But since we\nhave cfqg-\u003esaved_workload_slice mechanism, the preempt is a nop.\nLooks currently our preempt is totally broken if the two queues are not from\nthe same workload type.\nBelow patch fixes it. This will might make async queue starvation, but it\u0027s\nwhat our old code does before cgroup is added.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "275220f0fcff1adf28a717076e00f575edf05fda", "tree": "d249bccc80c64443dab211639050c4fb14332648", "parents": [ "fe3c560b8a22cb28e54fe8950abef38e88d75831", "81c5e2ae33c4b19e53966b427e33646bf6811830" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 10:45:01 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 13 10:45:01 2011 -0800" }, "message": "Merge branch \u0027for-2.6.38/core\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.38/core\u0027 of git://git.kernel.dk/linux-2.6-block: (43 commits)\n block: ensure that completion error gets properly traced\n blktrace: add missing probe argument to block_bio_complete\n block cfq: don\u0027t use atomic_t for cfq_group\n block cfq: don\u0027t use atomic_t for cfq_queue\n block: trace event block fix unassigned field\n block: add internal hd part table references\n block: fix accounting bug on cross partition merges\n kref: add kref_test_and_get\n bio-integrity: mark kintegrityd_wq highpri and CPU intensive\n block: make kblockd_workqueue smarter\n Revert \"sd: implement sd_check_events()\"\n block: Clean up exit_io_context() source code.\n Fix compile warnings due to missing removal of a \u0027ret\u0027 variable\n fs/block: type signature of major_to_index(int) to major_to_index(unsigned)\n block: convert !IS_ERR(p) \u0026\u0026 p to !IS_ERR_NOR_NULL(p)\n cfq-iosched: don\u0027t check cfqg in choose_service_tree()\n fs/splice: Pull buf-\u003eops-\u003econfirm() from splice_from_pipe actors\n cdrom: export cdrom_check_events()\n sd: implement sd_check_events()\n sr: implement sr_check_events()\n ...\n" }, { "commit": "81c5e2ae33c4b19e53966b427e33646bf6811830", "tree": "a602a6dd100165c8948bfa713e6f0b422dcba5d8", "parents": [ "797a455d2c682476c3797dbfecf5bf84c1e3b9d3", "fcc57045d53edc35bcce456e60ac4aa802712934" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Jan 13 14:47:54 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Jan 13 14:47:54 2011 +0100" }, "message": "Merge branch \u0027for-2.6.38/event-handling\u0027 into for-2.6.38/core\n" }, { "commit": "329a67815b596d23daf0caa588ae0800e925320f", "tree": "5fcf8a7c4a4d43b2cc7dc220d6bb25bdf27d153f", "parents": [ "30d7b9448f03f2c82d0fd44738674cc156a8ce0a" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jan 07 08:48:28 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:48:28 2011 +0100" }, "message": "block cfq: don\u0027t use atomic_t for cfq_group\n\ncfq_group-\u003eref is used with queue_lock hold, the only exception is\ncfq_set_request, which looks like a bug to me, so ref doesn\u0027t need\nto be an atomic and atomic operation is slower.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nReviewed-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "30d7b9448f03f2c82d0fd44738674cc156a8ce0a", "tree": "f925a59dfe6a7b2da5240e9d43d873de7b1de273", "parents": [ "b7908c1035af7652cd613991b54dbff9c8b6bd3a" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jan 07 08:46:59 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:46:59 2011 +0100" }, "message": "block cfq: don\u0027t use atomic_t for cfq_queue\n\ncfq_queue-\u003eref is used with queue_lock hold, so ref doesn\u0027t need to be an atomic\nand atomic operation is slower.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nReviewed-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6c23a9681c0fe7fb7dd331b39dda11926f43746e", "tree": "bf113a475a17faa866e7e59806378107dcb3aa40", "parents": [ "09e099d4bafea3b15be003d548bdf94b4b6e0e17" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:43:37 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:43:37 2011 +0100" }, "message": "block: add internal hd part table references\n\nWe can\u0027t use krefs since it\u0027s apparently restricted to very basic\nreference counting.\n\nThis reverts commit e4a683c8.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "09e099d4bafea3b15be003d548bdf94b4b6e0e17", "tree": "a4199338ad73e88c0863bbfc6604c4972055f16d", "parents": [ "e4a683c899cd5a49f8d684a054c95bd115a0c005" ], "author": { "name": "Jerome Marchand", "email": "jmarchan@redhat.com", "time": "Wed Jan 05 16:57:38 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jan 05 16:57:38 2011 +0100" }, "message": "block: fix accounting bug on cross partition merges\n\n/proc/diskstats would display a strange output as follows.\n\n$ cat /proc/diskstats |grep sda\n 8 0 sda 90524 7579 102154 20464 0 0 0 0 0 14096 20089\n 8 1 sda1 19085 1352 21841 4209 0 0 0 0 4294967064 15689 4293424691\n ~~~~~~~~~~\n 8 2 sda2 71252 3624 74891 15950 0 0 0 0 232 23995 1562390\n 8 3 sda3 54 487 2188 92 0 0 0 0 0 88 92\n 8 4 sda4 4 0 8 0 0 0 0 0 0 0 0\n 8 5 sda5 81 2027 2130 138 0 0 0 0 0 87 137\n\nIts reason is the wrong way of accounting hd_struct-\u003ein_flight. When a bio is\nmerged into a request belongs to different partition by ELEVATOR_FRONT_MERGE.\n\nThe detailed root cause is as follows.\n\nAssuming that there are two partition, sda1 and sda2.\n\n1. A request for sda2 is in request_queue. Hence sda1\u0027s hd_struct-\u003ein_flight\n is 0 and sda2\u0027s one is 1.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | 0\n sda2 | 1\n ---------------------------\n\n2. A bio belongs to sda1 is issued and is merged into the request mentioned on\n step1 by ELEVATOR_BACK_MERGE. The first sector of the request is changed\n from sda2 region to sda1 region. However the two partition\u0027s\n hd_struct-\u003ein_flight are not changed.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | 0\n sda2 | 1\n ---------------------------\n\n3. The request is finished and blk_account_io_done() is called. In this case,\n sda2\u0027s hd_struct-\u003ein_flight, not a sda1\u0027s one, is decremented.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | -1\n sda2 | 1\n ---------------------------\n\nThe patch fixes the problem by caching the partition lookup\ninside the request structure, hence making sure that the increment\nand decrement will always happen on the same partition struct. This\nalso speeds up IO with accounting enabled, since it cuts down on\nthe number of lookups we have to do.\n\nAlso add a refcount to struct hd_struct to keep the partition in\nmemory as long as users exist. We use kref_test_and_get() to ensure\nwe don\u0027t add a reference to a partition which is going away.\n\nSigned-off-by: Jerome Marchand \u003cjmarchan@redhat.com\u003e\nSigned-off-by: Yasuaki Ishimatsu \u003cisimatu.yasuaki@jp.fujitsu.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "89b90be2d877a904b1704e4029db65655bfc6282", "tree": "0dbeda7f0efa17373bdd3c59793a36c417fa9d1c", "parents": [ "27667c996f6a0bed4ad1e10ac0a0dbb6037968db" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Jan 03 15:01:47 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Jan 03 15:01:47 2011 +0100" }, "message": "block: make kblockd_workqueue smarter\n\nkblockd is used for unplugging and may affect IO latency and\nthroughput and the max number of concurrent work items are bound by\nthe number of block devices. Make it HIGHPRI workqueue w/ default max\nconcurrency.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4b7bd364700d9ac8372eff48832062b936d0793b", "tree": "0dbf78c95456a0b02d07fcd473281f04a87e266d", "parents": [ "c0d8768af260e2cbb4bf659ae6094a262c86b085", "90a8a73c06cc32b609a880d48449d7083327e11a" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Wed Dec 22 18:57:02 2010 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Wed Dec 22 18:57:02 2010 +0100" }, "message": "Merge branch \u0027master\u0027 into for-next\n\nConflicts:\n\tMAINTAINERS\n\tarch/arm/mach-omap2/pm24xx.c\n\tdrivers/scsi/bfa/bfa_fcpim.c\n\nNeeded to update to apply fixes for which the old branch was too\noutdated.\n" }, { "commit": "27667c996f6a0bed4ad1e10ac0a0dbb6037968db", "tree": "dcbcd7354f6ccaa3fe88fdf0a0a5f1e168ff8c40", "parents": [ "3603b8eaccc8e41d3f355b3cadd662a3dd6699fd" ], "author": { "name": "Bart Van Assche", "email": "bvanassche@acm.org", "time": "Tue Dec 21 15:07:45 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Dec 21 15:07:45 2010 +0100" }, "message": "block: Clean up exit_io_context() source code.\n\nThis patch fixes a spelling error in a source code comment and removes\nsuperfluous braces in the function exit_io_context().\n\nSigned-off-by: Bart Van Assche \u003cbvanassche@acm.org\u003e\nCc: Jens Axboe \u003cjaxboe@fusionio.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7f8635cc9e66a26d7280ba680b044fa2f65104af", "tree": "0d8506e86d07e15c473aca1a09af7ad6ff7d8b49", "parents": [ "3cb50ddf97a0a1ca4c68bc12fa1e727a6b45fbf2", "0fc13c8995cd96f4123de400c71c223d80400ed9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 20 09:19:46 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 20 09:19:46 2010 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-linus\u0027 of git://git.kernel.dk/linux-2.6-block:\n cciss: fix cciss_revalidate panic\n block: max hardware sectors limit wrapper\n block: Deprecate QUEUE_FLAG_CLUSTER and use queue_limits instead\n blk-throttle: Correct the placement of smp_rmb()\n blk-throttle: Trim/adjust slice_end once a bio has been dispatched\n block: check for proper length of iov entries earlier in blk_rq_map_user_iov()\n drbd: fix for spin_lock_irqsave in endio callback\n drbd: don\u0027t recvmsg with zero length\n" }, { "commit": "e61eb2e93fe86931d46831752a82dab25a5335ca", "tree": "631aeb9bf8614903972a6a9945040a4d528f1029", "parents": [ "b9f985b6e05ebd7af2aaef0eb3ae369390ef191f" ], "author": { "name": "Yang Zhang", "email": "kthreadd@gmail.com", "time": "Fri Dec 17 09:00:18 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 09:00:18 2010 +0100" }, "message": "fs/block: type signature of major_to_index(int) to major_to_index(unsigned)\n\nThe major/minor device numbers are always defined and used as `unsigned\u0027.\n\nSigned-off-by: Yang Zhang \u003ckthreadd@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "b9f985b6e05ebd7af2aaef0eb3ae369390ef191f", "tree": "69aa4bd9501f988302a155ebfdc630bd2a37bfde", "parents": [ "7278c9c19bd85cf33213a2e0b538a18d3ac8ad00" ], "author": { "name": "Yang Zhang", "email": "kthreadd@gmail.com", "time": "Fri Dec 17 08:58:36 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 08:58:36 2010 +0100" }, "message": "block: convert !IS_ERR(p) \u0026\u0026 p to !IS_ERR_NOR_NULL(p)\n\nSigned-off-by: Yang Zhang \u003ckthreadd@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7278c9c19bd85cf33213a2e0b538a18d3ac8ad00", "tree": "e080ada1f8cc00fef4600976f8cf489cc94d1c03", "parents": [ "a8adbe378b56acd5945df70753c7e8f6fe223304" ], "author": { "name": "Gui Jianfeng", "email": "guijianfeng@cn.fujitsu.com", "time": "Fri Dec 17 08:57:14 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 08:57:14 2010 +0100" }, "message": "cfq-iosched: don\u0027t check cfqg in choose_service_tree()\n\nWhen cfq_choose_cfqg() is called in select_queue(), there must be at least one\nbacklogged CFQ queue waiting for dispatching, hence there must be at least one\nbacklogged CFQ group on service tree. So we never call choose_service_tree()\nwith cfqg \u003d\u003d NULL.\n\nSigned-off-by: Gui Jianfeng \u003cguijianfeng@cn.fujitsu.com\u003e\nReviewed-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nAcked-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "72d4cd9f38b5ed96b75df4c622be25e1c2648dd3", "tree": "a229645cbbc63aad3fa5b57ea9f092e618588890", "parents": [ "e692cb668fdd5a712c6ed2a2d6f2a36ee83997b4" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Fri Dec 17 08:34:20 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 08:36:01 2010 +0100" }, "message": "block: max hardware sectors limit wrapper\n\nImplement blk_limits_max_hw_sectors() and make\nblk_queue_max_hw_sectors() a wrapper around it.\n\nDM needs this to avoid setting queue_limits\u0027 max_hw_sectors and\nmax_sectors directly. dm_set_device_limits() now leverages\nblk_limits_max_hw_sectors() logic to establish the appropriate\nmax_hw_sectors minimum (PAGE_SIZE). Fixes issue where DM was\nincorrectly setting max_sectors rather than max_hw_sectors (which\ncaused dm_merge_bvec()\u0027s max_hw_sectors check to be ineffective).\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: stable@kernel.org\nAcked-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" } ], "next": "e692cb668fdd5a712c6ed2a2d6f2a36ee83997b4" }