)]}' { "log": [ { "commit": "4853abaae7e4a2af938115ce9071ef8684fb7af4", "tree": "167eb7cb1b48541fa6d0ca5042f7452e2dd9e4de", "parents": [ "bcf30e75b773b60379338768677a1301ef602ff9" ], "author": { "name": "Jeff Moyer", "email": "jmoyer@redhat.com", "time": "Mon Aug 15 21:37:25 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Aug 15 21:37:25 2011 +0200" }, "message": "block: fix flush machinery for stacking drivers with differring flush flags\n\nCommit ae1b1539622fb46e51b4d13b3f9e5f4c713f86ae, block: reimplement\nFLUSH/FUA to support merge, introduced a performance regression when\nrunning any sort of fsyncing workload using dm-multipath and certain\nstorage (in our case, an HP EVA). The test I ran was fs_mark, and it\ndropped from ~800 files/sec on ext4 to ~100 files/sec. It turns out\nthat dm-multipath always advertised flush+fua support, and passed\ncommands on down the stack, where those flags used to get stripped off.\nThe above commit changed that behavior:\n\nstatic inline struct request *__elv_next_request(struct request_queue *q)\n{\n struct request *rq;\n\n while (1) {\n- while (!list_empty(\u0026q-\u003equeue_head)) {\n+ if (!list_empty(\u0026q-\u003equeue_head)) {\n rq \u003d list_entry_rq(q-\u003equeue_head.next);\n- if (!(rq-\u003ecmd_flags \u0026 (REQ_FLUSH | REQ_FUA)) ||\n- (rq-\u003ecmd_flags \u0026 REQ_FLUSH_SEQ))\n- return rq;\n- rq \u003d blk_do_flush(q, rq);\n- if (rq)\n- return rq;\n+ return rq;\n }\n\nNote that previously, a command would come in here, have\nREQ_FLUSH|REQ_FUA set, and then get handed off to blk_do_flush:\n\nstruct request *blk_do_flush(struct request_queue *q, struct request *rq)\n{\n unsigned int fflags \u003d q-\u003eflush_flags; /* may change, cache it */\n bool has_flush \u003d fflags \u0026 REQ_FLUSH, has_fua \u003d fflags \u0026 REQ_FUA;\n bool do_preflush \u003d has_flush \u0026\u0026 (rq-\u003ecmd_flags \u0026 REQ_FLUSH);\n bool do_postflush \u003d has_flush \u0026\u0026 !has_fua \u0026\u0026 (rq-\u003ecmd_flags \u0026\n REQ_FUA);\n unsigned skip \u003d 0;\n...\n if (blk_rq_sectors(rq) \u0026\u0026 !do_preflush \u0026\u0026 !do_postflush) {\n rq-\u003ecmd_flags \u0026\u003d ~REQ_FLUSH;\n\t\tif (!has_fua)\n\t\t\trq-\u003ecmd_flags \u0026\u003d ~REQ_FUA;\n\t return rq;\n\t}\n\nSo, the flush machinery was bypassed in such cases (q-\u003eflush_flags \u003d\u003d 0\n\u0026\u0026 rq-\u003ecmd_flags \u0026 (REQ_FLUSH|REQ_FUA)).\n\nNow, however, we don\u0027t get into the flush machinery at all. Instead,\n__elv_next_request just hands a request with flush and fua bits set to\nthe scsi_request_fn, even if the underlying request_queue does not\nsupport flush or fua.\n\nThe agreed upon approach is to fix the flush machinery to allow\nstacking. While this isn\u0027t used in practice (since there is only one\nrequest-based dm target, and that target will now reflect the flush\nflags of the underlying device), it does future-proof the solution, and\nmake it function as designed.\n\nIn order to make this work, I had to add a field to the struct request,\ninside the flush structure (to store the original req-\u003eend_io). Shaohua\nhad suggested overloading the union with rb_node and completion_data,\nbut the completion data is used by device mapper and can also be used by\nother drivers. So, I didn\u0027t see a way around the additional field.\n\nI tested this patch on an HP EVA with both ext4 and xfs, and it recovers\nthe lost performance. Comments and other testers, as always, are\nappreciated.\n\nCheers,\nJeff\n\nSigned-off-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "aa387cc895672b00f807ad7c734a2defaf677712", "tree": "31c2c3f3e0d58ff2524cdf8f94abecc394f5eb49", "parents": [ "24c3047095fa3954f114bfff2e37b8fcbb216396" ], "author": { "name": "Mike Christie", "email": "michaelc@cs.wisc.edu", "time": "Sun Jul 31 22:05:09 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sun Jul 31 22:05:09 2011 +0200" }, "message": "block: add bsg helper library\n\nThis moves the FC classes bsg code to the block layer and\nmakes it a lib so that other classes like iscsi and SAS can use it.\n\nIt is helpful because working with the request queue, bios,\ncreating scatterlists, etc are a pain that the LLD does not\nhave to worry about with normal IOs and should not have to\nworry about for bsg requests.\n\nSigned-off-by: Mike Christie \u003cmichaelc@cs.wisc.edu\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "5757a6d76cdf6dda2a492c09b985c015e86779b1", "tree": "6356a6353639eb473dd917a1b2062f9e7e20de22", "parents": [ "ef3230880abd36553ab442363d3c9a0661f00769" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Sat Jul 23 20:44:25 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Jul 23 20:44:25 2011 +0200" }, "message": "block: strict rq_affinity\n\nSome systems benefit from completions always being steered to the strict\nrequester cpu rather than the looser \"per-socket\" steering that\nblk_cpu_to_group() attempts by default. This is because the first\nCPU in the group mask ends up being completely overloaded with work,\nwhile the others (including the original submitter) has power left\nto spare.\n\nAllow the strict mode to be set by writing \u00272\u0027 to the sysfs control\nfile. This is identical to the scheme used for the nomerges file,\nwhere \u00272\u0027 is a more aggressive setting than just being turned on.\n\necho 2 \u003e /sys/block/\u003cbdev\u003e/queue/rq_affinity\n\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nCc: Roland Dreier \u003croland@purestorage.com\u003e\nTested-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "d7b7630130e52361af66ce3b994696e2357ba7de", "tree": "52e76c86cd35ffb1e7656f391f4b730072a6193c", "parents": [ "7700fc4f675fa38094e78e345b594363a2fd895b" ], "author": { "name": "Richard Kennedy", "email": "richard@rsk.demon.co.uk", "time": "Wed Jul 13 21:17:23 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jul 13 21:17:49 2011 +0200" }, "message": "block: reorder request_queue to remove 64 bit alignment padding\n\nReorder request_queue to remove 16 bytes of alignment padding in 64 bit\nbuilds.\n\nOn my config this shrinks the size of this structure from 1608 to 1592\nbytes and therefore needs one fewer cachelines.\n\nAlso trivially move the open bracket { to be on the same line as the\nstructure name to make it easier to grep.\n\nSigned-off-by: Richard Kennedy \u003crichard@rsk.demon.co.uk\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "316cc67d5e03801a5ee4ac660a4dfe9e02aed475", "tree": "ddcac370cf4402127a9609cfe6bb86eb1ebb6c8d", "parents": [ "55c022bbddb2c056b5dff1bd1b1758d31b6d64c9" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jul 08 08:19:21 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jul 08 08:19:21 2011 +0200" }, "message": "block: document blk_plug list access\n\nI\u0027m often confused why not disable preempt when changing blk_plug list. It\nwould be better to add comments here in case others have the similar concerns.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "55c022bbddb2c056b5dff1bd1b1758d31b6d64c9", "tree": "fdd3aa29a1407bbd19b8efe47b2538544da85a70", "parents": [ "719c0c590609809365c6f3da2f923cd84dc99113" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Fri Jul 08 08:19:20 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jul 08 08:19:20 2011 +0200" }, "message": "block: avoid building too big plug list\n\nWhen I test fio script with big I/O depth, I found the total throughput drops\ncompared to some relative small I/O depth. The reason is the thread accumulates\nbig requests in its plug list and causes some delays (surely this depends\non CPU speed).\nI thought we\u0027d better have a threshold for requests. When a threshold reaches,\nthis means there is no request merge and queue lock contention isn\u0027t severe\nwhen pushing per-task requests to queue, so the main advantages of blk plug\ndon\u0027t exist. We can force a plug list flush in this case.\nWith this, my test throughput actually increases and almost equals to small\nI/O depth. Another side effect is irq off time decreases in blk_flush_plug_list()\nfor big I/O depth.\nThe BLK_MAX_REQUEST_COUNT is choosen arbitarily, but 16 is efficiently to\nreduce lock contention to me. But I\u0027m open here, 32 is ok in my test too.\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "04bf7869ca0fd12009aee301cac2264a36df4d98", "tree": "66cb81ebf8b76560a31433c2c493dc430c914af9", "parents": [ "d2f31a5fd60d168b00fc4f7617b68a1287b21e90", "7b28afe01ab6ffb5f152f47831b44933facd2328" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jul 01 16:17:13 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jul 01 16:17:13 2011 +0200" }, "message": "Merge branch \u0027for-linus\u0027 into for-3.1/core\n\nConflicts:\n\tblock/blk-throttle.c\n\tblock/cfq-iosched.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4d0d98b60eba726e0a4f3e6617628b070c444707", "tree": "fb83b8412ff2bf7602adabe317a1ca702aa683ec", "parents": [ "df4156569d4ace581bd1581e7aa4a9dd7d2f0775" ], "author": { "name": "Wanlong Gao", "email": "wanlong.gao@gmail.com", "time": "Mon Jun 13 10:45:38 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Jun 13 10:45:38 2011 +0200" }, "message": "block:fix the comment error in blkdev.h\n\nThere is not a function rq_init but blk_rq_init in block/blk-core.c.\n\nSigned-off-by: Wanlong Gao \u003cwanlong.gao@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "ea9d6553b3b3044e7374774cc33bb1b2eee19dd3", "tree": "9079017f7576f1df50d063c033c80c5a201dc555", "parents": [ "a1706ac4c0201ea0143dc0db0659001b26ceeabb" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Tue May 31 13:45:53 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue May 31 13:45:53 2011 +0200" }, "message": "block: remove unwanted semicolons\n\nSince those defined functions require additional semicolon\nfrom the caller, they could cause potential syntax errors\nwhen used in if-else statements.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nAcked-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "698567f3fa790fea37509a54dea855302dd88331", "tree": "7a1df976a0eb12cab03e82c18809a30d5482fee4", "parents": [ "d70d0711edd8076ec2ce0ed109106e2df950681b", "61c4f2c81c61f73549928dfd9f3e8f26aa36a8cf" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri May 20 20:33:15 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri May 20 20:33:15 2011 +0200" }, "message": "Merge commit \u0027v2.6.39\u0027 into for-2.6.40/core\n\nSince for-2.6.40/core was forked off the 2.6.39 devel tree, we\u0027ve\nhad churn in the core area that makes it difficult to handle\npatches for eg cfq or blk-throttle. Instead of requiring that they\nbe based in older versions with bugs that have been fixed later\nin the rc cycle, merge in 2.6.39 final.\n\nAlso fixes up conflicts in the below files.\n\nConflicts:\n\tdrivers/block/paride/pcd.c\n\tdrivers/cdrom/viocd.c\n\tdrivers/ide/ide-cd.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a934a00a69e940b126b9bdbf83e630ef5fe43523", "tree": "b3f82d7b0a6c34c4d449debd2d1d242bdbf84b56", "parents": [ "bbdd304cf66fbf2b4b2d28418dc619d443635e83" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed May 18 10:37:35 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed May 18 10:37:35 2011 +0200" }, "message": "block: Fix discard topology stacking and reporting\n\nIn some cases we would end up stacking discard_zeroes_data incorrectly.\nFix this by enabling the feature by default for stacking drivers and\nclearing it for low-level drivers. Incorporating a device that does not\nsupport dzd will then cause the feature to be disabled in the stacking\ndriver.\n\nAlso ensure that the maximum discard value does not overflow when\nexported in sysfs and return 0 in the alignment and dzd fields for\ndevices that don\u0027t support discard.\n\nReported-by: Lukas Czerner \u003clczerner@redhat.com\u003e\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "3ac0cc4508709d42ec9aa351086c7d38bfc0660c", "tree": "933d5ab6ff9b0763f36e614962abb5bdcc4a348a", "parents": [ "f3876930952390a31c3a7fd68dd621464a36eb80" ], "author": { "name": "shaohua.li@intel.com", "email": "shaohua.li@intel.com", "time": "Fri May 06 11:34:41 2011 -0600" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri May 06 11:36:25 2011 -0600" }, "message": "block: hold queue if flush is running for non-queueable flush drive\n\nIn some drives, flush requests are non-queueable. When flush request is\nrunning, normal read/write requests can\u0027t run. If block layer dispatches\nsuch request, driver can\u0027t handle it and requeue it. Tejun suggested we\ncan hold the queue when flush is running. This can avoid unnecessary\nrequeue. Also this can improve performance. For example, we have\nrequest flush1, write1, flush 2. flush1 is dispatched, then queue is\nhold, write1 isn\u0027t inserted to queue. After flush1 is finished, flush2\nwill be dispatched. Since disk cache is already clean, flush2 will be\nfinished very soon, so looks like flush2 is folded to flush1.\n\nIn my test, the queue holding completely solves a regression introduced by\ncommit 53d63e6b0dfb95882ec0219ba6bbd50cde423794:\n\n block: make the flush insertion use the tail of the dispatch list\n\n It\u0027s not a preempt type request, in fact we have to insert it\n behind requests that do specify INSERT_FRONT.\n\nwhich causes about 20% regression running a sysbench fileio\nworkload.\n\nStable: 2.6.39 only\n\nCc: stable@kernel.org\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "f3876930952390a31c3a7fd68dd621464a36eb80", "tree": "7fe2306a8dc2022ac7724b2d47629e38e3fb354b", "parents": [ "490b94be0282c3b67f56453628ff0aaae827a670" ], "author": { "name": "shaohua.li@intel.com", "email": "shaohua.li@intel.com", "time": "Fri May 06 11:34:32 2011 -0600" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri May 06 11:36:25 2011 -0600" }, "message": "block: add a non-queueable flush flag\n\nflush request isn\u0027t queueable in some drives. Add a flag to let driver\nnotify block layer about this. We can optimize flush performance with the\nknowledge.\n\nStable: 2.6.39 only\n\nCc: stable@kernel.org\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\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": "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": "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": "a237c1c5bc5dc5c76a21be922dca4826f3eca8ca", "tree": "a216c9a6d9e870b84424938e9e0b4722dc8634cd", "parents": [ "5853b4f06f7b9b56f37f457d7923f7b96496074e" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Apr 16 13:27:55 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Apr 16 13:27:55 2011 +0200" }, "message": "block: let io_schedule() flush the plug inline\n\nLinus correctly observes that the most important dispatch cases\nare now done from kblockd, this isn\u0027t ideal for latency reasons.\nThe original reason for switching dispatches out-of-line was to\navoid too deep a stack, so by _only_ letting the \"accidental\"\nflush directly in schedule() be guarded by offload to kblockd,\nwe should be able to get the best of both worlds.\n\nSo add a blk_schedule_flush_plug() that offloads to kblockd,\nand only use that from the schedule() path.\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": "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": "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": "1f940bdfc0d03265d178d9dfd840d854819f797d", "tree": "b45ef8482ef17ccaee2e754e872fb4047efb3448", "parents": [ "eba2ed9c9636d9e7ed203a2498ed230b48341709" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 11 20:17:08 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Mar 11 20:17:08 2011 +0100" }, "message": "block: fixup plugging stubs for !CONFIG_BLOCK\n\nThey used an older prototype, fix it up.\n\nReported-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4c63f5646e405b5010cc9499419060bf2e838f5b", "tree": "df91ba315032c8ec4aafeb3ab96fdfa7c6c656e1", "parents": [ "cafb0bfca1a73efd6d8a4a6a6a716e6134b96c24", "69d60eb96ae8a73cf9b79cf28051caf973006011" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:58:35 2011 +0100" }, "message": "Merge branch \u0027for-2.6.39/stack-plug\u0027 into for-2.6.39/core\n\nConflicts:\n\tblock/blk-core.c\n\tblock/blk-flush.c\n\tdrivers/md/raid1.c\n\tdrivers/md/raid10.c\n\tdrivers/md/raid5.c\n\tfs/nilfs2/btnode.c\n\tfs/nilfs2/mdt.c\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7eaceaccab5f40bbfda044629a6298616aeaed50", "tree": "33954d12f63e25a47eb6d86ef3d3d0a5e62bf752", "parents": [ "73c101011926c5832e6e141682180c4debe2cf45" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:52:07 2011 +0100" }, "message": "block: remove per-queue plugging\n\nCode has been converted over to the new explicit on-stack plugging,\nand delay users have been converted to use the new API for that.\nSo lets kill off the old plugging along with aops-\u003esync_page().\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "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": "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": "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": "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": "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": "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": "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": "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": "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": "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" }, { "commit": "e692cb668fdd5a712c6ed2a2d6f2a36ee83997b4", "tree": "accf682fe5e1388f305b5fc364a931dfda5f3fb9", "parents": [ "04a6b516cdc6efc2500b52a540cf65be8c5aaf9e" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Dec 01 19:41:49 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Dec 17 08:35:53 2010 +0100" }, "message": "block: Deprecate QUEUE_FLAG_CLUSTER and use queue_limits instead\n\nWhen stacking devices, a request_queue is not always available. This\nforced us to have a no_cluster flag in the queue_limits that could be\nused as a carrier until the request_queue had been set up for a\nmetadevice.\n\nThere were several problems with that approach. First of all it was up\nto the stacking device to remember to set queue flag after stacking had\ncompleted. Also, the queue flag and the queue limits had to be kept in\nsync at all times. We got that wrong, which could lead to us issuing\ncommands that went beyond the max scatterlist limit set by the driver.\n\nThe proper fix is to avoid having two flags for tracking the same thing.\nWe deprecate QUEUE_FLAG_CLUSTER and use the queue limit directly in the\nblock layer merging functions. The queue_limit \u0027no_cluster\u0027 is turned\ninto \u0027cluster\u0027 to avoid double negatives and to ease stacking.\nClustering defaults to being enabled as before. The queue flag logic is\nremoved from the stacking function, and explicitly setting the cluster\nflag is no longer necessary in DM and MD.\n\nReported-by: Ed Lin \u003ced.lin@promise.com\u003e\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "77ea887e433ad8389d416826936c110fa7910f80", "tree": "ac9d32aabcebf5a465acae2066b12c9335b5ca6f", "parents": [ "d2bf1b6723ed0eab378363649d15b7893bf14e91" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Dec 08 20:57:37 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Dec 16 17:53:38 2010 +0100" }, "message": "implement in-kernel gendisk events handling\n\nCurrently, media presence polling for removeable block devices is done\nfrom userland. There are several issues with this.\n\n* Polling is done by periodically opening the device. For SCSI\n devices, the command sequence generated by such action involves a\n few different commands including TEST_UNIT_READY. This behavior,\n while perfectly legal, is different from Windows which only issues\n single command, GET_EVENT_STATUS_NOTIFICATION. Unfortunately, some\n ATAPI devices lock up after being periodically queried such command\n sequences.\n\n* There is no reliable and unintrusive way for a userland program to\n tell whether the target device is safe for media presence polling.\n For example, polling for media presence during an on-going burning\n session can make it fail. The polling program can avoid this by\n opening the device with O_EXCL but then it risks making a valid\n exclusive user of the device fail w/ -EBUSY.\n\n* Userland polling is unnecessarily heavy and in-kernel implementation\n is lighter and better coordinated (workqueue, timer slack).\n\nThis patch implements framework for in-kernel disk event handling,\nwhich includes media presence polling.\n\n* bdops-\u003echeck_events() is added, which supercedes -\u003emedia_changed().\n It should check whether there\u0027s any pending event and return if so.\n Currently, two events are defined - DISK_EVENT_MEDIA_CHANGE and\n DISK_EVENT_EJECT_REQUEST. -\u003echeck_events() is guaranteed not to be\n called parallelly.\n\n* gendisk-\u003eevents and -\u003easync_events are added. These should be\n initialized by block driver before passing the device to add_disk().\n The former contains the mask of all supported events and the latter\n the mask of all events which the device can report without polling.\n /sys/block/*/events[_async] export these to userland.\n\n* Kernel parameter block.events_dfl_poll_msecs controls the system\n polling interval (default is 0 which means disable) and\n /sys/block/*/events_poll_msecs control polling intervals for\n individual devices (default is -1 meaning use system setting). Note\n that if a device can report all supported events asynchronously and\n its polling interval isn\u0027t explicitly set, the device won\u0027t be\n polled regardless of the system polling interval.\n\n* If a device is opened exclusively with write access, event checking\n is automatically disabled until all write exclusive accesses are\n released.\n\n* There are event \u0027clearing\u0027 events. For example, both of currently\n defined events are cleared after the device has been successfully\n opened. This information is passed to -\u003echeck_events() callback\n using @clearing argument as a hint.\n\n* Event checking is always performed from system_nrt_wq and timer\n slack is set to 25% for polling.\n\n* Nothing changes for drivers which implement -\u003emedia_changed() but\n not -\u003echeck_events(). Going forward, all drivers will be converted\n to -\u003echeck_events() and -\u003emedia_change() will be dropped.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nCc: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "d2bf1b6723ed0eab378363649d15b7893bf14e91", "tree": "4a4144b09ab747ad661605aa761753d9a805e77f", "parents": [ "dddd9dc340ae1a41d90e084529ca979c77c4ecfe" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Dec 08 20:57:36 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Dec 16 17:53:38 2010 +0100" }, "message": "block: move register_disk() and del_gendisk() to block/genhd.c\n\nThere\u0027s no reason for register_disk() and del_gendisk() to be in\nfs/partitions/check.c. Move both to genhd.c. While at it, collapse\nunlink_gendisk(), which was artificially in a separate function due to\ngenhd.c / check.c split, into del_gendisk().\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "02e031cbc843b010e72fcc05c76113c688b2860f", "tree": "9f68559ec3acc39bcc4ce2ff87043a094eaa2e8f", "parents": [ "00e375e7e962f938f6b3c93e4cd097a5e26cc788" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Wed Nov 10 14:54:09 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Nov 10 14:54:09 2010 +0100" }, "message": "block: remove REQ_HARDBARRIER\n\nREQ_HARDBARRIER is dead now, so remove the leftovers. What\u0027s left\nat this point is:\n\n - various checks inside the block layer.\n - sanity checks in bio based drivers.\n - now unused bio_empty_barrier helper.\n - Xen blockfront use of BLKIF_OP_WRITE_BARRIER - it\u0027s dead for a while,\n but Xen really needs to sort out it\u0027s barrier situaton.\n - setting of ordered tags in uas - dead code copied from old scsi\n drivers.\n - scsi different retry for barriers - it\u0027s dead and should have been\n removed when flushes were converted to FS requests.\n - blktrace handling of barriers - removed. Someone who knows blktrace\n better should add support for REQ_FLUSH and REQ_FUA, though.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a107e5a3a473a2ea62bd5af24e11b84adf1486ff", "tree": "d36c2cb38d8be88d4d75cdebc354aa140aa0e470", "parents": [ "e3e1288e86a07cdeb0aee5860a2dff111c6eff79", "a269029d0e2192046be4c07ed78a45022469ee4c" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Oct 27 23:44:47 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Oct 27 23:44:47 2010 -0400" }, "message": "Merge branch \u0027next\u0027 into upstream-merge\n\nConflicts:\n\tfs/ext4/inode.c\n\tfs/ext4/mballoc.c\n\tinclude/trace/events/ext4.h\n" }, { "commit": "e6fa0be699449d28a20e815bfe9ce26725ec4962", "tree": "2e979f57060c50a73eef51fdbc19d4666d463c4e", "parents": [ "5c2178e785244341d1e6f2bc3b62f20a337cc44f" ], "author": { "name": "Lukas Czerner", "email": "lczerner@redhat.com", "time": "Wed Oct 27 21:30:04 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Oct 27 21:30:04 2010 -0400" }, "message": "Add helper function for blkdev_issue_zeroout (sb_issue_discard)\n\nThis is done the same way as helper sb_issue_discard for\nblkdev_issue_discard.\n\nSigned-off-by: Lukas Czerner \u003clczerner@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "f253b86b4ad1b3220544e75880510fd455ebd23f", "tree": "cc2dd76b8ffc8df4356c1e95bd15276169dd335e", "parents": [ "35da7a307c535f9c2929cae277f3df425c9f9b1e" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sun Oct 24 22:06:02 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sun Oct 24 22:06:02 2010 +0200" }, "message": "Revert \"block: fix accounting bug on cross partition merges\"\n\nThis reverts commit 7681bfeeccff5efa9eb29bf09249a3c400b15327.\n\nConflicts:\n\n\tinclude/linux/genhd.h\n\nIt has numerous issues with the cleanup path and non-elevator\ndevices. Revert it for now so we can come up with a clean\nversion without rushing things.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a2887097f25cd38cadfc11d10769e2b349fb5eca", "tree": "cd4adcb305365d6ba9acd2c02d4eb9d0125c6f8d", "parents": [ "8abfc6e7a45eb74e51904bbae676fae008b11366", "005a1d15f5a6b2bb4ada80349513effbf22b4588" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:07:18 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:07:18 2010 -0700" }, "message": "Merge branch \u0027for-2.6.37/barrier\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.37/barrier\u0027 of git://git.kernel.dk/linux-2.6-block: (46 commits)\n xen-blkfront: disable barrier/flush write support\n Added blk-lib.c and blk-barrier.c was renamed to blk-flush.c\n block: remove BLKDEV_IFL_WAIT\n aic7xxx_old: removed unused \u0027req\u0027 variable\n block: remove the BH_Eopnotsupp flag\n block: remove the BLKDEV_IFL_BARRIER flag\n block: remove the WRITE_BARRIER flag\n swap: do not send discards as barriers\n fat: do not send discards as barriers\n ext4: do not send discards as barriers\n jbd2: replace barriers with explicit flush / FUA usage\n jbd2: Modify ASYNC_COMMIT code to not rely on queue draining on barrier\n jbd: replace barriers with explicit flush / FUA usage\n nilfs2: replace barriers with explicit flush / FUA usage\n reiserfs: replace barriers with explicit flush / FUA usage\n gfs2: replace barriers with explicit flush / FUA usage\n btrfs: replace barriers with explicit flush / FUA usage\n xfs: replace barriers with explicit flush / FUA usage\n block: pass gfp_mask and flags to sb_issue_discard\n dm: convey that all flushes are processed as empty\n ...\n" }, { "commit": "7681bfeeccff5efa9eb29bf09249a3c400b15327", "tree": "8557964a2df96e253dcf1a61734b98dbfbf192d6", "parents": [ "495d2b3883682fcd1c3dee3a45e38fd00154ae25" ], "author": { "name": "Yasuaki Ishimatsu", "email": "isimatu.yasuaki@jp.fujitsu.com", "time": "Tue Oct 19 09:05:00 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Oct 19 09:07:02 2010 +0200" }, "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\nWhen reloading partition tables, quiesce IO to ensure that no\nrequest references to the partition struct exists. When it is safe\nto free the partition table, the IO for that device is restarted\nagain.\n\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": "892b6f90db81cccb723d5d92f4fddc2d68b206e1", "tree": "a4cedbff388c2cc0659c76f429003b856ef17943", "parents": [ "c49c06e4960949a9bced708858433fcf6ca36a9c" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Oct 13 21:18:03 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Oct 13 21:19:12 2010 +0200" }, "message": "block: Ensure physical block size is unsigned int\n\nPhysical block size was declared unsigned int to accomodate the maximum\nsize reported by READ CAPACITY(16). Make sure we use the right type in\nthe related functions.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "dd3932eddf428571762596e17b65f5dc92ca361b", "tree": "57cec5ae2f862037f78b7e993323d77955bb6463", "parents": [ "8786fb70ccb36c7cff64680bb80c46d3a09d44db" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Thu Sep 16 20:51:46 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Sep 16 20:52:58 2010 +0200" }, "message": "block: remove BLKDEV_IFL_WAIT\n\nAll the blkdev_issue_* helpers can only sanely be used for synchronous\ncaller. To issue cache flushes or barriers asynchronously the caller needs\nto set up a bio by itself with a completion callback to move the asynchronous\nstate machine ahead. So drop the BLKDEV_IFL_WAIT flag that is always\nspecified when calling blkdev_issue_* and also remove the now unused flags\nargument to blkdev_issue_flush and blkdev_issue_zeroout. For\nblkdev_issue_discard we need to keep it for the secure discard flag, which\ngains a more descriptive name and loses the bitops vs flag confusion.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "e43473b7f223ec866f7db273697e76c337c390f9", "tree": "e90b52dbe4ec4ae37263a00e2bd9eaf5367cf72f", "parents": [ "4c9eefa16c6f124ffcc736cb719b24ea27f85017" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Sep 15 17:06:35 2010 -0400" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Sep 16 08:42:52 2010 +0200" }, "message": "blkio: Core implementation of throttle policy\n\no Actual implementation of throttling policy in block layer. Currently it\n implements READ and WRITE bytes per second throttling logic. IOPS throttling\n comes in later patches.\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "144177991ca624841ddbd1e7edff958fc0f6d1fe", "tree": "793a42427d46585c0acfcfe97d23e604174a2c10", "parents": [ "8dcbdc742fec9e6c542ff9cb599d2ee620753d07" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Wed Sep 15 13:08:27 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Sep 15 13:08:27 2010 +0200" }, "message": "block: fix an address space warning in blk-map.c\n\nChange type of 2nd parameter of blk_rq_aligned() into unsigned long\nand remove unnecessary casting. Now we can call it with \u0027uaddr\u0027\ninstead of \u0027ubuf\u0027 in __blk_rq_map_user() so that it can remove\nfollowing warnings from sparse:\n\n block/blk-map.c:57:31: warning: incorrect type in argument 2 (different address spaces)\n block/blk-map.c:57:31: expected void *addr\n block/blk-map.c:57:31: got void [noderef] \u003casn:1\u003e*ubuf\n\nHowever blk_rq_map_kern() needs one more local variable to handle it.\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "13f05c8d8e98bbdce89158bfdb2e380940695a88", "tree": "055215e7e2b1bdc684ead64daa61b30b35eaa3c5", "parents": [ "c8bf1336824ebd698d37b71763e1c43190f2229a" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Sep 10 20:50:10 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "axboe@carl.home.kernel.dk", "time": "Fri Sep 10 20:50:10 2010 +0200" }, "message": "block/scsi: Provide a limit on the number of integrity segments\n\nSome controllers have a hardware limit on the number of protection\ninformation scatter-gather list segments they can handle.\n\nIntroduce a max_integrity_segments limit in the block layer and provide\na new scsi_host_template setting that allows HBA drivers to provide a\nvalue suitable for the hardware.\n\nAdd support for honoring the integrity segment limit when merging both\nbios and requests.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003caxboe@carl.home.kernel.dk\u003e\n" }, { "commit": "8c5553678237b7121355108e03c36086037d8975", "tree": "7ecbc41837bf2dac6010230839e3312d6d33dd35", "parents": [ "31725e65c7214b52b607eba705fc4f306be4d5a5" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Aug 18 05:29:22 2010 -0400" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:40 2010 +0200" }, "message": "block: remove the BLKDEV_IFL_BARRIER flag\n\nRemove support for barriers on discards, which is unused now. Also\nremove the DISCARD_NOBARRIER I/O type in favour of just setting the\nrw flags up locally in blkdev_issue_discard.\n\ntj: Also remove DISCARD_SECURE and use REQ_SECURE directly.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "2cf6d26a354ab6362e301b5a323832b02867df47", "tree": "dcc84f3422df777baa85f54870cda613262e74ca", "parents": [ "b372d360df6deaf79a58a02fa0cc0d7e0aa3e92f" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Aug 18 05:29:10 2010 -0400" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:38 2010 +0200" }, "message": "block: pass gfp_mask and flags to sb_issue_discard\n\nWe\u0027ll need to get rid of the BLKDEV_IFL_BARRIER flag, and to facilitate\nthat and to make the interface less confusing pass all flags explicitly.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4fed947cb311e5aa51781d316cefca836352f6ce", "tree": "eada83d5bf503244628e3c190e97e8c7af847e35", "parents": [ "dd4c133f387c48f526022860ad70354637a80f4c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:17 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:37 2010 +0200" }, "message": "block: implement REQ_FLUSH/FUA based interface for FLUSH/FUA requests\n\nNow that the backend conversion is complete, export sequenced\nFLUSH/FUA capability through REQ_FLUSH/FUA flags. REQ_FLUSH means the\ndevice cache should be flushed before executing the request. REQ_FUA\nmeans that the data in the request should be on non-volatile media on\ncompletion.\n\nBlock layer will choose the correct way of implementing the semantics\nand execute it. The request may be passed to the device directly if\nthe device can handle it; otherwise, it will be sequenced using one or\nmore proxy requests. Devices will never see REQ_FLUSH and/or FUA\nwhich it doesn\u0027t support.\n\nAlso, unlike the original REQ_HARDBARRIER, REQ_FLUSH/FUA requests are\nnever failed with -EOPNOTSUPP. If the underlying device doesn\u0027t\nsupport FLUSH/FUA, the block layer simply make those noop. IOW, it no\nlonger distinguishes between writeback cache which doesn\u0027t support\ncache flush and writethrough/no cache. Devices which have WB cache\nw/o flush are very difficult to come by these days and there\u0027s nothing\nmuch we can do anyway, so it doesn\u0027t make sense to require everyone to\nimplement -EOPNOTSUPP handling. This will simplify filesystems and\nblock drivers as they can drop -EOPNOTSUPP retry logic for barriers.\n\n* QUEUE_ORDERED_* are removed and QUEUE_FSEQ_* are moved into\n blk-flush.c.\n\n* REQ_FLUSH w/o data can also be directly passed to drivers without\n sequencing but some drivers assume that zero length requests don\u0027t\n have rq-\u003ebio which isn\u0027t true for these requests requiring the use\n of proxy requests.\n\n* REQ_COMMON_MASK now includes REQ_FLUSH | REQ_FUA so that they are\n copied from bio to request.\n\n* WRITE_BARRIER is marked deprecated and WRITE_FLUSH, WRITE_FUA and\n WRITE_FLUSH_FUA are added.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "dd4c133f387c48f526022860ad70354637a80f4c", "tree": "7b741150d37d327b69e483468278d9de3a43a4e1", "parents": [ "8839a0e055d9abd6c011d533373a8dd266cad011" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "block: rename barrier/ordered to flush\n\nWith ordering requirements dropped, barrier and ordered are misnomers.\nNow all block layer does is sequencing FLUSH and FUA. Rename them to\nflush.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "28e7d1845216538303bb95d679d8fd4de50e2f1a", "tree": "0ef56dc0d7c894657c4ae71a3e8da6e1164fb933", "parents": [ "dd831006d5be7f74c3fe7aef82380c51c3637960" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "block: drop barrier ordering by queue draining\n\nFilesystems will take all the responsibilities for ordering requests\naround commit writes and will only indicate how the commit writes\nthemselves should be handled by block layers. This patch drops\nbarrier ordering by queue draining from block layer. Ordering by\ndraining implementation was somewhat invasive to request handling.\nList of notable changes follow.\n\n* Each queue has 1 bit color which is flipped on each barrier issue.\n This is used to track whether a given request is issued before the\n current barrier or not. REQ_ORDERED_COLOR flag and coloring\n implementation in __elv_add_request() are removed.\n\n* Requests which shouldn\u0027t be processed yet for draining were stalled\n by returning -EAGAIN from blk_do_ordered() according to the test\n result between blk_ordered_req_seq() and blk_blk_ordered_cur_seq().\n This logic is removed.\n\n* Draining completion logic in elv_completed_request() removed.\n\n* All barrier sequence requests were queued to request queue and then\n trckled to lower layer according to progress and thus maintaining\n request orders during requeue was necessary. This is replaced by\n queueing the next request in the barrier sequence only after the\n current one is complete from blk_ordered_complete_seq(), which\n removes the need for multiple proxy requests in struct request_queue\n and the request sorting logic in the ELEVATOR_INSERT_REQUEUE path of\n elv_insert().\n\n* As barriers no longer have ordering constraints, there\u0027s no need to\n dump the whole elevator onto the dispatch queue on each barrier.\n Insert barriers at the front instead.\n\n* If other barrier requests come to the front of the dispatch queue\n while one is already in progress, they are stored in\n q-\u003epending_barriers and restored to dispatch queue one-by-one after\n each barrier completion from blk_ordered_complete_seq().\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "dd831006d5be7f74c3fe7aef82380c51c3637960", "tree": "30a86749f0c4f7980b2982dbde8bbcfff37b08f9", "parents": [ "9cbbdca44ae1a6f512ea1e2be11ced8bbb9d430a" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "block: misc cleanups in barrier code\n\nMake the following cleanups in preparation of barrier/flush update.\n\n* blk_do_ordered() declaration is moved from include/linux/blkdev.h to\n block/blk.h.\n\n* blk_do_ordered() now returns pointer to struct request, with %NULL\n meaning \"try the next request\" and ERR_PTR(-EAGAIN) \"try again\n later\". The third case will be dropped with further changes.\n\n* In the initialization of proxy barrier request, data direction is\n already set by init_request_from_bio(). Drop unnecessary explicit\n REQ_WRITE setting and move init_request_from_bio() above REQ_FUA\n flag setting.\n\n* add_request() is collapsed into __make_request().\n\nThese changes don\u0027t make any functional difference.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "4913efe456c987057e5d36a3f0a55422a9072cae", "tree": "295f04a7214e1933df3301dd42c12ff3f282a22c", "parents": [ "6958f145459ca7ad9715024de97445addacb8510" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "block: deprecate barrier and replace blk_queue_ordered() with blk_queue_flush()\n\nBarrier is deemed too heavy and will soon be replaced by FLUSH/FUA\nrequests. Deprecate barrier. All REQ_HARDBARRIERs are failed with\n-EOPNOTSUPP and blk_queue_ordered() is replaced with simpler\nblk_queue_flush().\n\nblk_queue_flush() takes combinations of REQ_FLUSH and FUA. If a\ndevice has write cache and can flush it, it should set REQ_FLUSH. If\nthe device can handle FUA writes, it should also set REQ_FUA.\n\nAll blk_queue_ordered() users are converted.\n\n* ORDERED_DRAIN is mapped to 0 which is the default value.\n* ORDERED_DRAIN_FLUSH is mapped to REQ_FLUSH.\n* ORDERED_DRAIN_FLUSH_FUA is mapped to REQ_FLUSH | REQ_FUA.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Boaz Harrosh \u003cbharrosh@panasas.com\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nCc: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nCc: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nCc: Jeremy Fitzhardinge \u003cjeremy@xensource.com\u003e\nCc: Chris Wright \u003cchrisw@sous-sol.org\u003e\nCc: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nCc: Geert Uytterhoeven \u003cGeert.Uytterhoeven@sonycom.com\u003e\nCc: David S. Miller \u003cdavem@davemloft.net\u003e\nCc: Alasdair G Kergon \u003cagk@redhat.com\u003e\nCc: Pierre Ossman \u003cdrzeus@drzeus.cx\u003e\nCc: Stefan Weinhuber \u003cwein@de.ibm.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "6958f145459ca7ad9715024de97445addacb8510", "tree": "c8c945eb68ceb88bd34647d7bcaedd13a0d753ca", "parents": [ "589d7ed02ade0d06a3510da2e15a7edfdb2ef3d8" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "block: kill QUEUE_ORDERED_BY_TAG\n\nNobody is making meaningful use of ORDERED_BY_TAG now and queue\ndraining for barrier requests will be removed soon which will render\nthe advantage of tag ordering moot. Kill ORDERED_BY_TAG. The\nfollowing users are affected.\n\n* brd: converted to ORDERED_DRAIN.\n* virtio_blk: ORDERED_TAG path was already marked deprecated. Removed.\n* xen-blkfront: ORDERED_TAG case dropped.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nCc: Nick Piggin \u003cnpiggin@kernel.dk\u003e\nCc: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nCc: Jeremy Fitzhardinge \u003cjeremy@xensource.com\u003e\nCc: Chris Wright \u003cchrisw@sous-sol.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "0da2f50944976e890ccc9436ab88c0da87788d02", "tree": "d535494f58873f468b73aa39e21e2c0c116a8c62", "parents": [ "76be97c1fc945db08aae1f1b746012662d643e97" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 03 11:56:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Sep 10 12:35:36 2010 +0200" }, "message": "ide: remove unnecessary blk_queue_flushing() test in do_ide_request()\n\nUnplugging from a request function doesn\u0027t really help much (it\u0027s\nalready in the request_fn) and soon block layer will be updated to mix\nbarrier sequence with other commands, so there\u0027s no need to treat\nqueue flushing any differently.\n\nide was the only user of blk_queue_flushing(). Remove it.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: David S. Miller \u003cdavem@davemloft.net\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "8d57a98ccd0b4489003473979da8f5a1363ba7a3", "tree": "2982997ce66bb6a92c020b7189966c3097095fd7", "parents": [ "93caf8e69eac763f6a20cf253ace8e7fc1ab7953" ], "author": { "name": "Adrian Hunter", "email": "adrian.hunter@nokia.com", "time": "Wed Aug 11 14:17:49 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Aug 12 08:43:30 2010 -0700" }, "message": "block: add secure discard\n\nSecure discard is the same as discard except that all copies of the\ndiscarded sectors (perhaps created by garbage collection) must also be\nerased.\n\nSigned-off-by: Adrian Hunter \u003cadrian.hunter@nokia.com\u003e\nAcked-by: Jens Axboe \u003caxboe@kernel.dk\u003e\nCc: Kyungmin Park \u003ckmpark@infradead.org\u003e\nCc: Madhusudhan Chikkature \u003cmadhu.cr@ti.com\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Ben Gardiner \u003cbengardiner@nanometrics.ca\u003e\nCc: \u003clinux-mmc@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": "edca4a380584a65a16839bdee33ec82244f0f88e", "tree": "f0e24d4712cf98eb5876157ae1f49bebc6d701ef", "parents": [ "f6c4c8e19a087dae7dc651ccbd1ff8b843eedee2" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Tue Aug 03 12:54:51 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:53:04 2010 +0200" }, "message": "block: disallow FS recursion from sb_issue_discard allocation\n\nFilesystems can call sb_issue_discard on a memory reclaim path\n(e.g. ext4 calls sb_issue_discard during journal commit).\n\nUse GFP_NOFS in sb_issue_discard to avoid recursing back into the FS.\n\nReported-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "8a6cfeb6deca3a8fefd639d898b0d163c0b5d368", "tree": "9a633ad48c3b1ada0519ee7bade0602f940037f6", "parents": [ "34484062445fe905bf02c72f87ddda21881acda3" ], "author": { "name": "Arnd Bergmann", "email": "arnd@arndb.de", "time": "Thu Jul 08 10:18:46 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:25:00 2010 +0200" }, "message": "block: push down BKL into .locked_ioctl\n\nAs a preparation for the removal of the big kernel\nlock in the block layer, this removes the BKL\nfrom the common ioctl handling code, moving it\ninto every single driver still using it.\n\nSigned-off-by: Arnd Bergmann \u003carnd@arndb.de\u003e\nAcked-by: Christoph Hellwig \u003chch@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "a89f5c899db3c6be4bb426e4efb72ecee29a93b5", "tree": "b71bcb7885c1ff5dc2001800852b6144db640157", "parents": [ "82b6d57fb11644fe25c8a1346627ad0027673dae" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Tue Jul 06 09:03:18 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:24:21 2010 +0200" }, "message": "block: remove unused REQ_TYPE_LINUX_BLOCK\n\nNobody uses REQ_TYPE_LINUX_BLOCK (and its REQ_LB_OP_*).\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nAcked-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "00fff26539bfe3fad21c164fc4002d9ede056fb0", "tree": "4909c8eed03133b1d46f689e465ed1f579b8e282", "parents": [ "afc23068103ccfbf1917eb2a007bc15ab5418cc9" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Sat Jul 03 17:45:40 2010 +0900" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:24:15 2010 +0200" }, "message": "block: remove q-\u003eprepare_flush_fn completely\n\nThis removes q-\u003eprepare_flush_fn completely (changes the\nblk_queue_ordered API).\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "28018c242a4ec7017bbbf81d2d3952f820a27118", "tree": "03732bef20d0e4dad30d81d9b9ad9ffcf5a2e41c", "parents": [ "e597cd09f711b28b8466ebdc2f12e55b44fa81e4" ], "author": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Thu Jul 01 19:49:17 2010 +0900" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:23:47 2010 +0200" }, "message": "block: implement an unprep function corresponding directly to prep\n\nReviewed-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "66ac0280197981f88774e74b60c8e5f9f07c1dba", "tree": "d093ce493146779926df88b5831805c6f9ee14e1", "parents": [ "082439004b31adc146e96e5f1c574dd2b57dcd93" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Jun 18 16:59:42 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:23:08 2010 +0200" }, "message": "block: don\u0027t allocate a payload for discard request\n\nAllocating a fixed payload for discard requests always was a horrible hack,\nand it\u0027s not coming to byte us when adding support for discard in DM/MD.\n\nSo change the code to leave the allocation of a payload to the lowlevel\ndriver. Unfortunately that means we\u0027ll need another hack, which allows\nus to update the various block layer length fields indicating that we\nhave a payload. Instead of hiding this in sd.c, which we already partially\ndo for UNMAP support add a documented helper in the core block layer for it.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "7b6d91daee5cac6402186ff224c3af39d79f4a0e", "tree": "b1518cf0b6c301178e0a320f80610cd5b3aa7625", "parents": [ "33659ebbae262228eef4e0fe990f393d1f0ed941" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Sat Aug 07 18:20:39 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:20:39 2010 +0200" }, "message": "block: unify flags for struct bio and struct request\n\nRemove the current bio flags and reuse the request flags for the bio, too.\nThis allows to more easily trace the type of I/O from the filesystem\ndown to the block driver. There were two flags in the bio that were\nmissing in the requests: BIO_RW_UNPLUG and BIO_RW_AHEAD. Also I\u0027ve\nrenamed two request flags that had a superflous RW in them.\n\nNote that the flags are in bio.h despite having the REQ_ name - as\nblkdev.h includes bio.h that is the only way to go for now.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "33659ebbae262228eef4e0fe990f393d1f0ed941", "tree": "fcb537f09359c8dad3a6f6e16dc4319562dc42cc", "parents": [ "7e005f79791dcd58436c88ded4a7f5aed1b82147" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Sat Aug 07 18:17:56 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:17:56 2010 +0200" }, "message": "block: remove wrappers for request type/flags\n\nRemove all the trivial wrappers for the cmd_type and cmd_flags fields in\nstruct requests. This allows much easier grepping for different request\ntypes instead of unwinding through macros.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "bfe172310e58225f0d07f9354b683abacbd6a0d8", "tree": "6f50393419bd29dabb27316e6875bc5715816789", "parents": [ "b375a612ad931264b71cf162d692b4420f2578a9" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Mon May 31 15:59:03 2010 +0900" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:15:49 2010 +0200" }, "message": "block: kill ISA_DMA_THRESHOLD usage\n\nblock uses ISA_DMA_THRESHOLD for BLK_BOUNCE_ISA. Only SCSI uses\nISA_DMA_THRESHOLD for ancient drivers with non-zero\nunchecked_isa_dma. Nowadays drivers (and subsystems) use dma_mask\nproperly instead of ISA_DMA_THRESHOLD.\n\nDocumentation/scsi/scsi_mid_low_api.txt says:\n\nunchecked_isa_dma - 1\u003d\u003eonly use bottom 16 MB of ram (ISA DMA addressing\n restriction), 0\u003d\u003ecan use full 32 bit (or better) DMA\n address space\n\nSo block simply uses DMA_BIT_MASK(24) for BLK_BOUNCE_ISA for SCSI.\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nAcked-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "e2e1a148bc45855816ae6b4692ce29d0020fa22e", "tree": "fd5ec8a580d4333b471acfe50f6f92b4cc880087", "parents": [ "841fdffdd382722d33579a6aa1487e8a4e526dbd" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jun 09 10:42:09 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:13:00 2010 +0200" }, "message": "block: add sysfs knob for turning off disk entropy contributions\n\nThere are two reasons for doing this:\n\n- On SSD disks, the completion times aren\u0027t as random as they\n are for rotational drives. So it\u0027s questionable whether they\n should contribute to the random pool in the first place.\n\n- Calling add_disk_randomness() has a lot of overhead.\n\nThis adds /sys/block/\u003cdev\u003e/queue/add_random that will allow you to\nswitch off on a per-device basis. The default setting is on, so there\nshould be no functional changes from this patch.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "28f4197e5d4707311febeec8a0eb97cb5fd93c97", "tree": "9f7c5808e0c069b449b67856207c5ee4f740bd34", "parents": [ "713b686494a577b3c4f4f9f585a4705fc30d51c2" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jun 01 12:23:18 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jun 01 12:23:18 2010 +0200" }, "message": "block: disable preemption before using sched_clock()\n\nCommit 9195291e5f05e01d67f9a09c756b8aca8f009089 added calls to\nsched_clock() from preemptible code. sched_clock() is both the\nwrong interface AND cannot be called without preempt disabled.\n\nApply a temporary fix to get rid of the warnings, a real patch\nis in the works.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "d79df0b1eda0099a22cbcece01ce5e7d222450de", "tree": "eb9c938fa5faf285fb091d7446508a32229ceea4", "parents": [ "6e80e8ed5eb92d0112674aabe82951266a6a1051", "c8d1a126924fcbc1d61ceb830226e0c7afdcc841" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 21 15:26:46 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 21 15:26:46 2010 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging-2.6: (577 commits)\n Staging: ramzswap: Handler for swap slot free callback\n swap: Add swap slot free callback to block_device_operations\n swap: Add flag to identify block swap devices\n Staging: vt6655: use ETH_FRAME_LEN macro instead of custom one\n Staging: vt6655: use ETH_DATA_LEN macro instead of custom one\n Staging: vt6655: use ETH_FCS_LEN macro instead of custom one\n Staging: vt6656: use ETH_HLEN macro instead of custom one\n Staging: comedi: quatech_daqp_cs.c Replace eos semaphore with a completion.\n Staging: dt3155v4l: remove private memory allocator\n Staging: crystalhd: Remove typedefs from driver\n Staging: winbond: Fix for pointer name format issue in mds.c\n Staging: vt6656: removed custom UCHAR/USHORT/UINT/ULONG/ULONGLONG typedefs\n Staging: vt6656: removed custom CHAR/SHORT/INT/LONG typedefs\n Staging: comedi: Altered the way printk is used in 8255.c\n staging: iio: adis16350 and similar IMU driver\n Staging: iio: max1363 Fix two bugs in single_channel_from_ring\n Staging: iio: adis16220 extract bin_attribute structures from state\n Staging: iio: adis16220 vibration sensor driver\n Staging: comedi: Kconfig dependancy fixes\n Staging: comedi: fix up build error from last Kconfig changes\n ...\n" }, { "commit": "c3e33e043f5e9c583aa59d5591a614b2a8243d3a", "tree": "fe8fef91dafb670fad1f433ae48514472b8d23e5", "parents": [ "56bca01738733709bef076e2e97bbd01e5659f24" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat May 15 20:09:29 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri May 21 20:01:02 2010 +0200" }, "message": "block,ide: simplify bdops-\u003eset_capacity() to -\u003eunlock_native_capacity()\n\nbdops-\u003eset_capacity() is unnecessarily generic. All that\u0027s required\nis a simple one way notification to lower level driver telling it to\ntry to unlock native capacity. There\u0027s no reason to pass in target\ncapacity or return the new capacity. The former is always the\ninherent native capacity and the latter can be handled via the usual\ndevice resize / revalidation path. In fact, the current API is always\nused that way.\n\nReplace -\u003eset_capacity() with -\u003eunlock_native_capacity() which take\nonly @disk and doesn\u0027t return anything. IDE which is the only current\nuser of the API is converted accordingly.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Ben Hutchings \u003cben@decadent.org.uk\u003e\nCc: Bartlomiej Zolnierkiewicz \u003cbzolnier@gmail.com\u003e\nAcked-by: David S. Miller \u003cdavem@davemloft.net\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "b3a27d0529c6e5206f1b60f60263e3ecfd0d77cb", "tree": "50ce5c6ce06ed700160491f2cd0e93d318ccc7fd", "parents": [ "b27256439568950f30864ccecaeb6dfb588089d5" ], "author": { "name": "Nitin Gupta", "email": "ngupta@vflare.org", "time": "Mon May 17 11:02:43 2010 +0530" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Tue May 18 15:07:52 2010 -0700" }, "message": "swap: Add swap slot free callback to block_device_operations\n\nThis callback is required when RAM based devices are used as swap disks.\nOne such device is ramzswap which is used as compressed in-memory swap\ndisk. For such devices, we need a callback as soon as a swap slot is no\nlonger used to allow freeing memory allocated for this slot. Without this\ncallback, stale data can quickly accumulate in memory defeating the whole\npurpose of such devices.\n\nSigned-off-by: Nitin Gupta \u003cngupta@vflare.org\u003e\nAcked-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nAcked-by: Nigel Cunningham \u003cnigel@tuxonice.net\u003e\nAcked-by: Pekka Enberg \u003cpenberg@cs.helsinki.fi\u003e\nReviewed-by: Minchan Kim \u003cminchan.kim@gmail.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n" }, { "commit": "01effb0dc1451fad55925873ffbfb88fa4eadce0", "tree": "40fc70111217d23bbb5453fe08214d28ed4c7713", "parents": [ "0f3942a39ed768c967cb71ea0e9be7fc94112713" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Tue May 11 08:57:42 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue May 11 08:57:42 2010 +0200" }, "message": "block: allow initialization of previously allocated request_queue\n\nblk_init_queue() allocates the request_queue structure and then\ninitializes it as needed (request_fn, elevator, etc).\n\nSplit initialization out to blk_init_allocated_queue_node.\nIntroduce blk_init_allocated_queue wrapper function to model existing\nblk_init_queue and blk_init_queue_node interfaces.\n\nExport elv_register_queue to allow a newly added elevator to be\nregistered with sysfs. Export elv_unregister_queue for symmetry.\n\nThese changes allow DM to initialize a device\u0027s request_queue with more\nprecision. In particular, DM no longer unconditionally initializes a\nfull request_queue (elevator et al). It only does so for a\nrequest-based DM device.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "3f14d792f9a8fede64ce918dbb517f934497a4f8", "tree": "1eb1d1803ebe21c05f38bc75c70ee966701a1fe8", "parents": [ "f31e7e4022841c43c53b847b86b1bf97a08b2c94" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Wed Apr 28 17:55:09 2010 +0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Apr 28 19:47:36 2010 +0200" }, "message": "blkdev: add blkdev_issue_zeroout helper function\n\n- Add bio_batch helper primitive. This is rather generic primitive\n for submitting/waiting a complex request which consists of several\n bios.\n- blkdev_issue_zeroout() generate number of zero filed write bios.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "fbd9b09a177a481eda256447c881f014f29034fe", "tree": "ef7e213045382f82a1e3e3cf134d196a1045dd7a", "parents": [ "6b4517a7913a09d3259bb1d21c9cb300f12294bd" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Wed Apr 28 17:55:06 2010 +0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Apr 28 19:47:36 2010 +0200" }, "message": "blkdev: generalize flags for blkdev_issue_fn functions\n\nThe patch just convert all blkdev_issue_xxx function to common\nset of flags. Wait/allocation semantics preserved.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "7f1dc8a2d2f45fc557b27fd56115338b1d34fc24", "tree": "355619d9483aca33f5c45642ce0bf8a042ea3236", "parents": [ "8d2a91f8960b230b8bbcc4d97ed2015f5271c87d" ], "author": { "name": "Vivek Goyal", "email": "vgoyal@redhat.com", "time": "Wed Apr 21 17:44:16 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Apr 21 17:44:16 2010 +0200" }, "message": "blkio: Fix blkio crash during rq stat update\n\nblkio + cfq was crashing even when two sequential readers were put in two\nseparate cgroups (group_isolation\u003d0).\n\nThe reason being that cfqq can migrate across groups based on its being\nsync-noidle or not, it can happen that at request insertion time, cfqq\nbelonged to one cfqg and at request dispatch time, it belonged to root\ngroup. In this case request stats per cgroup can go wrong and it also runs\ninto BUG_ON().\n\nThis patch implements rq stashing away a cfq group pointer and not relying\non cfqq-\u003ecfqg pointer alone for rq stat accounting.\n\n[ 65.163523] ------------[ cut here ]------------\n[ 65.164301] kernel BUG at block/blk-cgroup.c:117!\n[ 65.164301] invalid opcode: 0000 [#1] SMP\n[ 65.164301] last sysfs file: /sys/devices/pci0000:00/0000:00:05.0/0000:60:00.1/host9/rport-9:0-0/target9:0:0/9:0:0:2/block/sde/stat\n[ 65.164301] CPU 1\n[ 65.164301] Modules linked in: dm_round_robin dm_multipath qla2xxx scsi_transport_fc dm_zero dm_mirror dm_region_hash dm_log dm_mod [last unloaded: scsi_wait_scan]\n[ 65.164301]\n[ 65.164301] Pid: 4505, comm: fio Not tainted 2.6.34-rc4-blk-for-35 #34 0A98h/HP xw8600 Workstation\n[ 65.164301] RIP: 0010:[\u003cffffffff8121924f\u003e] [\u003cffffffff8121924f\u003e] blkiocg_update_io_remove_stats+0x5b/0xaf\n[ 65.164301] RSP: 0018:ffff8800ba5a79e8 EFLAGS: 00010046\n[ 65.164301] RAX: 0000000000000096 RBX: ffff8800bb268d60 RCX: 0000000000000000\n[ 65.164301] RDX: ffff8800bb268eb8 RSI: 0000000000000000 RDI: ffff8800bb268e00\n[ 65.164301] RBP: ffff8800ba5a7a08 R08: 0000000000000064 R09: 0000000000000001\n[ 65.164301] R10: 0000000000079640 R11: ffff8800a0bd5bf0 R12: ffff8800bab4af01\n[ 65.164301] R13: ffff8800bab4af00 R14: ffff8800bb1d8928 R15: 0000000000000000\n[ 65.164301] FS: 00007f18f75056f0(0000) GS:ffff880001e40000(0000) knlGS:0000000000000000\n[ 65.164301] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033\n[ 65.164301] CR2: 000000000040e7f0 CR3: 00000000ba52b000 CR4: 00000000000006e0\n[ 65.164301] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000\n[ 65.164301] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400\n[ 65.164301] Process fio (pid: 4505, threadinfo ffff8800ba5a6000, task ffff8800ba45ae80)\n[ 65.164301] Stack:\n[ 65.164301] ffff8800ba5a7a08 ffff8800ba722540 ffff8800bab4af68 ffff8800bab4af68\n[ 65.164301] \u003c0\u003e ffff8800ba5a7a38 ffffffff8121d814 ffff8800ba722540 ffff8800bab4af68\n[ 65.164301] \u003c0\u003e ffff8800ba722540 ffff8800a08f6800 ffff8800ba5a7a68 ffffffff8121d8ca\n[ 65.164301] Call Trace:\n[ 65.164301] [\u003cffffffff8121d814\u003e] cfq_remove_request+0xe4/0x116\n[ 65.164301] [\u003cffffffff8121d8ca\u003e] cfq_dispatch_insert+0x84/0xe1\n[ 65.164301] [\u003cffffffff8121e833\u003e] cfq_dispatch_requests+0x767/0x8e8\n[ 65.164301] [\u003cffffffff8120e524\u003e] ? submit_bio+0xc3/0xcc\n[ 65.164301] [\u003cffffffff810ad657\u003e] ? sync_page_killable+0x0/0x35\n[ 65.164301] [\u003cffffffff8120ea8d\u003e] blk_peek_request+0x191/0x1a7\n[ 65.164301] [\u003cffffffffa000109c\u003e] ? dm_get_live_table+0x44/0x4f [dm_mod]\n[ 65.164301] [\u003cffffffffa0002799\u003e] dm_request_fn+0x38/0x14c [dm_mod]\n[ 65.164301] [\u003cffffffff810ad657\u003e] ? sync_page_killable+0x0/0x35\n[ 65.164301] [\u003cffffffff8120f600\u003e] __generic_unplug_device+0x32/0x37\n[ 65.164301] [\u003cffffffff8120f8a0\u003e] generic_unplug_device+0x2e/0x3c\n[ 65.164301] [\u003cffffffffa00011a6\u003e] dm_unplug_all+0x42/0x5b [dm_mod]\n[ 65.164301] [\u003cffffffff8120b063\u003e] blk_unplug+0x29/0x2d\n[ 65.164301] [\u003cffffffff8120b079\u003e] blk_backing_dev_unplug+0x12/0x14\n[ 65.164301] [\u003cffffffff81108a82\u003e] block_sync_page+0x35/0x39\n[ 65.164301] [\u003cffffffff810ad64e\u003e] sync_page+0x41/0x4a\n[ 65.164301] [\u003cffffffff810ad665\u003e] sync_page_killable+0xe/0x35\n[ 65.164301] [\u003cffffffff81589027\u003e] __wait_on_bit_lock+0x46/0x8f\n[ 65.164301] [\u003cffffffff810ad52d\u003e] __lock_page_killable+0x66/0x6d\n[ 65.164301] [\u003cffffffff81055fd4\u003e] ? wake_bit_function+0x0/0x33\n[ 65.164301] [\u003cffffffff810ad560\u003e] lock_page_killable+0x2c/0x2e\n[ 65.164301] [\u003cffffffff810aebfd\u003e] generic_file_aio_read+0x361/0x4f0\n[ 65.164301] [\u003cffffffff810e906c\u003e] do_sync_read+0xcb/0x108\n[ 65.164301] [\u003cffffffff811e32a3\u003e] ? security_file_permission+0x16/0x18\n[ 65.164301] [\u003cffffffff810e96d3\u003e] vfs_read+0xab/0x108\n[ 65.164301] [\u003cffffffff810e97f0\u003e] sys_read+0x4a/0x6e\n[ 65.164301] [\u003cffffffff81002b5b\u003e] system_call_fastpath+0x16/0x1b\n[ 65.164301] Code: 00 74 1c 48 8b 8b 60 01 00 00 48 85 c9 75 04 0f 0b eb fe 48 ff c9 48 89 8b 60 01 00 00 eb 1a 48 8b 8b 58 01 00 00 48 85 c9 75 04 \u003c0f\u003e 0b eb fe 48 ff c9 48 89 8b 58 01 00 00 45 84 e4 74 16 48 8b\n[ 65.164301] RIP [\u003cffffffff8121924f\u003e] blkiocg_update_io_remove_stats+0x5b/0xaf\n[ 65.164301] RSP \u003cffff8800ba5a79e8\u003e\n[ 65.164301] ---[ end trace 1b2b828753032e68 ]---\n\nSigned-off-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "84c124da9ff50bd71fab9c939ee5b7cd8bef2bd9", "tree": "f94ad03b56c71ebbe6b92265054c5e24af25b869", "parents": [ "31373d09da5b7fe21fe6f781e92bd534a3495f00" ], "author": { "name": "Divyesh Shah", "email": "dpshah@google.com", "time": "Fri Apr 09 08:31:19 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Apr 09 08:31:19 2010 +0200" }, "message": "blkio: Changes to IO controller additional stats patches\n\nthat include some minor fixes and addresses all comments.\n\nChangelog: (most based on Vivek Goyal\u0027s comments)\no renamed blkiocg_reset_write to blkiocg_reset_stats\no more clarification in the documentation on io_service_time and io_wait_time\no Initialize blkg-\u003estats_lock\no rename io_add_stat to blkio_add_stat and declare it static\no use bool for direction and sync\no derive direction and sync info from existing rq methods\no use 12 for major:minor string length\no define io_service_time better to cover the NCQ case\no add a separate reset_stats interface\no make the indexed stats a 2d array to simplify macro and function pointer code\no blkio.time now exports in jiffies as before\no Added stats description in patch description and\n Documentation/cgroup/blkio-controller.txt\no Prefix all stats functions with blkio and make them static as applicable\no replace IO_TYPE_MAX with IO_TYPE_TOTAL\no Moved #define constant to top of blk-cgroup.c\no Pass dev_t around instead of char *\no Add note to documentation file about resetting stats\no use BLK_CGROUP_MODULE in addition to BLK_CGROUP config option in #ifdef\n statements\no Avoid struct request specific knowledge in blk-cgroup. blk-cgroup.h now has\n rq_direction() and rq_sync() functions which are used by CFQ and when using\n io-controller at a higher level, bio_* functions can be added.\n\nSigned-off-by: Divyesh Shah\u003cdpshah@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "9195291e5f05e01d67f9a09c756b8aca8f009089", "tree": "e46a151fe39be2bc23d0683bedb2cbefe916fb5b", "parents": [ "303a3acb2362f16c7e7f4c53b40c2f4b396dc8d5" ], "author": { "name": "Divyesh Shah", "email": "dpshah@google.com", "time": "Thu Apr 01 15:01:41 2010 -0700" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Apr 02 08:44:37 2010 +0200" }, "message": "blkio: Increment the blkio cgroup stats for real now\n\nWe also add start_time_ns and io_start_time_ns fields to struct request\nhere to record the time when a request is created and when it is\ndispatched to device. We use ns uints here as ms and jiffies are\nnot very useful for non-rotational media.\n\nSigned-off-by: Divyesh Shah\u003cdpshah@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "181fdde3b4268cb7b4af76ba6337e7ec8accbb36", "tree": "6f2e3f177f21f6e299e84bc9075ac1ae257a301c", "parents": [ "b4b7a4ef097f288f724420b473dbf92a89c0ab7e" ], "author": { "name": "Richard Kennedy", "email": "richard@rsk.demon.co.uk", "time": "Fri Mar 19 08:58:16 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Mar 19 08:58:16 2010 +0100" }, "message": "block: remove 16 bytes of padding from struct request on 64bits\n\nRemove alignment padding to shrink struct request from 336 to 320 bytes\nso needing one fewer cacheline and therefore removing 48 bytes from\nstruct request_queue.\n\nSigned-off-by: Richard Kennedy \u003crichard@rsk.demon.co.uk\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "ee714f2dd33e726346e34f5cda12543162f4753e", "tree": "63252755cc38a81085191e6b851a6a04623cc9b9", "parents": [ "2cda2728aa1c8c006418a24f867b25e5eb7a32e2" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Mar 10 00:48:32 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Mar 15 12:47:59 2010 +0100" }, "message": "block: Finalize conversion of block limits functions\n\nRemove compatibility wrappers and update remaining drivers.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "8a78362c4eefc1deddbefe2c7f38aabbc2429d6b", "tree": "c095d95af1aec0f9cee5975b1dcdc6bc1d17d401", "parents": [ "086fa5ff0854c676ec333760f4c0154b3b242616" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Feb 26 00:20:39 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 26 13:58:08 2010 +0100" }, "message": "block: Consolidate phys_segment and hw_segment limits\n\nExcept for SCSI no device drivers distinguish between physical and\nhardware segment limits. Consolidate the two into a single segment\nlimit.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "086fa5ff0854c676ec333760f4c0154b3b242616", "tree": "ee63fb3c7c7d964bd799355b7cde18ba95f91f07", "parents": [ "eb28d31bc97e6374d81f404da309401ffaed467b" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Feb 26 00:20:38 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 26 13:58:08 2010 +0100" }, "message": "block: Rename blk_queue_max_sectors to blk_queue_max_hw_sectors\n\nThe block layer calling convention is blk_queue_\u003climit name\u003e.\nblk_queue_max_sectors predates this practice, leading to some confusion.\nRename the function to appropriately reflect that its intended use is to\nset max_hw_sectors.\n\nAlso introduce a temporary wrapper for backwards compability. This can\nbe removed after the merge window is closed.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "eb28d31bc97e6374d81f404da309401ffaed467b", "tree": "580d750832dd1da39a5c93b91b8e069db3395b1c", "parents": [ "e751e76a5f7adeee7438e68b0965559ad2864d0d" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Feb 26 00:20:37 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 26 13:58:08 2010 +0100" }, "message": "block: Add BLK_ prefix to definitions\n\nAdd a BLK_ prefix to block layer constants.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "e751e76a5f7adeee7438e68b0965559ad2864d0d", "tree": "06b1863a484203d7a217fa33964ef2d80a2ad712", "parents": [ "2800aac1114548a9b47b6e0d398117cc01b89685" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri Feb 26 00:20:36 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 26 13:58:07 2010 +0100" }, "message": "block: Remove unused accessor function\n\nblk_queue_max_hw_sectors is no longer called by any subsystem and can be\nremoved.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "7f03292ee16e68be96aa88904afc6152f8d37736", "tree": "5c32738f0348449c2fa9688d52e6ecf4445d76f0", "parents": [ "c5ecc484c528ff50bdbb16fbfbac758ee368b329", "baac35c4155a8aa826c70acee6553368ca5243a2" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Feb 25 08:48:05 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Feb 25 08:48:05 2010 +0100" }, "message": "Merge branch \u0027master\u0027 into for-2.6.34\n\nConflicts:\n\tinclude/linux/blkdev.h\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "79da0644a8e0838522828f106e4049639eea6baf", "tree": "4dbb8e1a3a4be78a79f14e8ebac5d6fe817a2fb4", "parents": [ "9f3a6284880ceea452903e2043c88d7226736318" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 23 08:40:43 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 23 08:40:43 2010 +0100" }, "message": "Revert \"block: improve queue_should_plug() by looking at IO depths\"\n\nThis reverts commit fb1e75389bd06fd5987e9cda1b4e0305c782f854.\n\n\"Benjamin S.\" \u003csbenni@gmx.de\u003e reports that the patch in question\ncauses a big drop in sequential throughput for him, dropping from\n200MB/sec down to only 70MB/sec.\n\nNeeds to be investigated more fully, for now lets just revert the\noffending commit.\n\nConflicts:\n\n\tinclude/linux/blkdev.h\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "488991e28e55b4fbca8067edf0259f69d1a6f92c", "tree": "fea5e0aca42e338137cc050e66aaeb5f539e3d21", "parents": [ "47483e25205f1f8d79784f0f7c733941bc080ec0" ], "author": { "name": "Alan D. Brunelle", "email": "Alan.Brunelle@hp.com", "time": "Fri Jan 29 09:04:08 2010 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Jan 29 09:04:08 2010 +0100" }, "message": "block: Added in stricter no merge semantics for block I/O\n\nUpdated \u0027nomerges\u0027 tunable to accept a value of \u00272\u0027 - indicating that _no_\nmerges at all are to be attempted (not even the simple one-hit cache).\n\nThe following table illustrates the additional benefit - 5 minute runs of\na random I/O load were applied to a dozen devices on a 16-way x86_64 system.\n\nnomerges Throughput %System Improvement (tput / %sys)\n-------- ------------ ----------- -------------------------\n0 12.45 MB/sec 0.669365609\n1 12.50 MB/sec 0.641519199 0.40% / 2.71%\n2 12.52 MB/sec 0.639849750 0.56% / 2.96%\n\nSigned-off-by: Alan D. Brunelle \u003calan.brunelle@hp.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "e03a72e13648ac6277bf2bab6b8324d51f89c0fa", "tree": "a01a2e072a5b225f22bd8bfa75f5270a527a4e4b", "parents": [ "ce289321b7dc1eb108e3df0dec872b7429ef49f7" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Mon Jan 11 03:21:51 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Jan 11 14:30:09 2010 +0100" }, "message": "block: Stop using byte offsets\n\nAll callers of the stacking functions use 512-byte sector units rather\nthan byte offsets. Simplify the code so the stacking functions take\nsectors when specifying data offsets.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "17be8c245054b9c7786545af3ba3ca4e54cd4ad9", "tree": "563687812bbb1b08643dfcc25997e34eb22a71b5", "parents": [ "dd3d145d49c5816b79acc6761ebbd842bc50b0ee" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Mon Jan 11 03:21:49 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Jan 11 14:29:20 2010 +0100" }, "message": "block: bdev_stack_limits wrapper\n\nDM does not want to know about partition offsets. Add a partition-aware\nwrapper that DM can use when stacking block devices.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nAcked-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nReviewed-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "dd3d145d49c5816b79acc6761ebbd842bc50b0ee", "tree": "107bdb2f46c6aa6bc482909d7ebdbb1da40b7b4f", "parents": [ "fe0b393f2c0a0d23a9bc9ed7dc51a1ee511098bd" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Mon Jan 11 03:21:48 2010 -0500" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Jan 11 14:29:19 2010 +0100" }, "message": "block: Fix discard alignment calculation and printing\n\nDiscard alignment reporting for partitions was incorrect. Update to\nmatch the algorithm used elsewhere.\n\nThe alignment can be negative (misaligned). Fix format string\naccordingly.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "9bd3f98821a83041e77ee25158b80b535d02d7b4", "tree": "cfe440e45ff404e8128388e4579ac16d52807665", "parents": [ "e79e95db5cffb2e01170d510686489c40937faa1" ], "author": { "name": "Gui Jianfeng", "email": "guijianfeng@cn.fujitsu.com", "time": "Wed Dec 30 08:41:07 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Dec 30 08:41:07 2009 +0100" }, "message": "block: blk_rq_err_sectors cleanup\n\nblk_rq_err_sectors() seems useless, get rid of it.\n\nSigned-off-by: Gui Jianfeng \u003cguijianfeng@cn.fujitsu.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "81744ee44ab2845c16ffd7d6f762f7b4a49a4750", "tree": "c3d0cca0e71080ecd8423cef8af1c632fc444764", "parents": [ "2f7a2d89a8b5915d89ad9ebeb0065db7d5831cea" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Tue Dec 29 08:35:35 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Dec 29 08:35:35 2009 +0100" }, "message": "block: Fix incorrect alignment offset reporting and update documentation\n\nqueue_sector_alignment_offset returned the wrong value which caused\npartitions to report an incorrect alignment_offset. Since offset\nalignment calculation is needed several places it has been split into a\nseparate helper function. The topology stacking function has been\nupdated accordingly.\n\nFurthermore, comments have been added to clarify how the stacking\nfunction works.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nTested-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "98262f2762f0067375f83824d81ea929e37e6bfe", "tree": "a26618d5c305eed975501c80cb02bc0f1b6e4524", "parents": [ "464191c65b85a8ec68a6e1a6293af625287c807e" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Thu Dec 03 09:24:48 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Dec 03 09:24:48 2009 +0100" }, "message": "block: Allow devices to indicate whether discarded blocks are zeroed\n\nThe discard ioctl is used by mkfs utilities to clear a block device\nprior to putting metadata down. However, not all devices return zeroed\nblocks after a discard. Some drives return stale data, potentially\ncontaining old superblocks. It is therefore important to know whether\ndiscarded blocks are properly zeroed.\n\nBoth ATA and SCSI drives have configuration bits that indicate whether\nzeroes are returned after a discard operation. Implement a block level\ninterface that allows this information to be bubbled up the stack and\nqueried via a new block device ioctl.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "2d4dc890b5c8fabd818a8586607e6843c4375e62", "tree": "9976ed7b0eed0056f8289aeb6a2b0abf8c940454", "parents": [ "3586e917f2c7df769d173c4ec99554cb40a911e5" ], "author": { "name": "Ilya Loginov", "email": "isloginov@gmail.com", "time": "Thu Nov 26 09:16:19 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Nov 26 09:16:19 2009 +0100" }, "message": "block: add helpers to run flush_dcache_page() against a bio and a request\u0027s pages\n\nMtdblock driver doesn\u0027t call flush_dcache_page for pages in request. So,\nthis causes problems on architectures where the icache doesn\u0027t fill from\nthe dcache or with dcache aliases. The patch fixes this.\n\nThe ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE symbol was introduced to avoid\npointless empty cache-thrashing loops on architectures for which\nflush_dcache_page() is a no-op. Every architecture was provided with this\nflush pages on architectires where ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE is\nequal 1 or do nothing otherwise.\n\nSee \"fix mtd_blkdevs problem with caches on some architectures\" discussion\non LKML for more information.\n\nSigned-off-by: Ilya Loginov \u003cisloginov@gmail.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: David Woodhouse \u003cdwmw2@infradead.org\u003e\nCc: Peter Horton \u003cphorton@bitbox.co.uk\u003e\nCc: \"Ed L. Cashin\" \u003cecashin@coraid.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "86b37281411cf1e9bc0a6b5406c45edb7bd9ea5d", "tree": "729db57dd52054af1bc16b4afb131093dfc9d255", "parents": [ "cf7c25cf91f632a3528669fc0876e1fc8355ff9b" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Tue Nov 10 11:50:21 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Nov 10 11:50:21 2009 +0100" }, "message": "block: Expose discard granularity\n\nWhile SSDs track block usage on a per-sector basis, RAID arrays often\nhave allocation blocks that are bigger. Allow the discard granularity\nand alignment to be set and teach the topology stacking logic how to\nhandle them.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "b9d128f1088ea5245109dfc9bbceb128b6371a77", "tree": "63e85f281dd372e77af4022e2235a4e5a46287de", "parents": [ "a870a3a485ddf7c0dec549269ed71d169556d61c" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 29 13:59:26 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 29 13:59:26 2009 +0100" }, "message": "block: move bdi/address_space unplug functions to backing-dev.h\n\nThere\u0027s nothing block related about them, the backing device\nis used by things like NFS etc as well. This gets rid of the\nneed to protect such calls by CONFIG_BLOCK.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" } ], "next": "23e018a1b083ecb4b8bb2fb43d58e7c19b5d7959" }