)]}' { "log": [ { "commit": "050c8ea80e3e90019d9e981c6a117ef614e882ed", "tree": "e1c53f17a93ba48b9aedb0c1560dfb022733845f", "parents": [ "4e8670e26135d8fbfd5e084fddc1a8ed9f8eb4cb" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Feb 08 09:19:38 2012 +0100" }, "committer": { "name": "Jens Axboe", "email": "axboe@kernel.dk", "time": "Wed Feb 08 09:19:38 2012 +0100" }, "message": "block: separate out blk_rq_merge_ok() and blk_try_merge() from elevator functions\n\nblk_rq_merge_ok() is the elevator-neutral part of merge eligibility\ntest. blk_try_merge() determines merge direction and expects the\ncaller to have tested elv_rq_merge_ok() previously.\n\nelv_rq_merge_ok() now wraps blk_rq_merge_ok() and then calls\nelv_iosched_allow_merge(). elv_try_merge() is removed and the two\ncallers are updated to call elv_rq_merge_ok() explicitly followed by\nblk_try_merge(). While at it, make rq_merge_ok() functions return\nbool.\n\nThis is to prepare for plug merge update and doesn\u0027t introduce any\nbehavior change.\n\nThis is based on Jens\u0027 patch to skip elevator_allow_merge_fn() from\nplug merge.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nLKML-Reference: \u003c4F16F3CA.90904@kernel.dk\u003e\nOriginal-patch-by: Jens Axboe \u003caxboe@kernel.dk\u003e\nSigned-off-by: Jens Axboe \u003caxboe@kernel.dk\u003e\n" }, { "commit": "5e84ea3a9c662dc2d7a48703a4468fad954a3b7f", "tree": "3fa0fb26a7c8a970213584104cc2498ef46d60a3", "parents": [ "4345caba340f051e10847924fc078ae18ed6695c" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 21 10:14:27 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Mon Mar 21 10:14:27 2011 +0100" }, "message": "block: attempt to merge with existing requests on plug flush\n\nOne of the disadvantages of on-stack plugging is that we potentially\nlose out on merging since all pending IO isn\u0027t always visible to\neverybody. When we flush the on-stack plugs, right now we don\u0027t do\nany checks to see if potential merge candidates could be utilized.\n\nCorrect this by adding a new insert variant, ELEVATOR_INSERT_SORT_MERGE.\nIt works just ELEVATOR_INSERT_SORT, but first checks whether we can\nmerge with an existing request before doing the insertion (if we fail\nmerging).\n\nThis fixes a regression with multiple processes issuing IO that\ncan be merged.\n\nThanks to Shaohua Li \u003cshaohua.li@intel.com\u003e for testing and fixing\nan accounting bug.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "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": "6c23a9681c0fe7fb7dd331b39dda11926f43746e", "tree": "bf113a475a17faa866e7e59806378107dcb3aa40", "parents": [ "09e099d4bafea3b15be003d548bdf94b4b6e0e17" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:43:37 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Fri Jan 07 08:43:37 2011 +0100" }, "message": "block: add internal hd part table references\n\nWe can\u0027t use krefs since it\u0027s apparently restricted to very basic\nreference counting.\n\nThis reverts commit e4a683c8.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "09e099d4bafea3b15be003d548bdf94b4b6e0e17", "tree": "a4199338ad73e88c0863bbfc6604c4972055f16d", "parents": [ "e4a683c899cd5a49f8d684a054c95bd115a0c005" ], "author": { "name": "Jerome Marchand", "email": "jmarchan@redhat.com", "time": "Wed Jan 05 16:57:38 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Wed Jan 05 16:57:38 2011 +0100" }, "message": "block: fix accounting bug on cross partition merges\n\n/proc/diskstats would display a strange output as follows.\n\n$ cat /proc/diskstats |grep sda\n 8 0 sda 90524 7579 102154 20464 0 0 0 0 0 14096 20089\n 8 1 sda1 19085 1352 21841 4209 0 0 0 0 4294967064 15689 4293424691\n ~~~~~~~~~~\n 8 2 sda2 71252 3624 74891 15950 0 0 0 0 232 23995 1562390\n 8 3 sda3 54 487 2188 92 0 0 0 0 0 88 92\n 8 4 sda4 4 0 8 0 0 0 0 0 0 0 0\n 8 5 sda5 81 2027 2130 138 0 0 0 0 0 87 137\n\nIts reason is the wrong way of accounting hd_struct-\u003ein_flight. When a bio is\nmerged into a request belongs to different partition by ELEVATOR_FRONT_MERGE.\n\nThe detailed root cause is as follows.\n\nAssuming that there are two partition, sda1 and sda2.\n\n1. A request for sda2 is in request_queue. Hence sda1\u0027s hd_struct-\u003ein_flight\n is 0 and sda2\u0027s one is 1.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | 0\n sda2 | 1\n ---------------------------\n\n2. A bio belongs to sda1 is issued and is merged into the request mentioned on\n step1 by ELEVATOR_BACK_MERGE. The first sector of the request is changed\n from sda2 region to sda1 region. However the two partition\u0027s\n hd_struct-\u003ein_flight are not changed.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | 0\n sda2 | 1\n ---------------------------\n\n3. The request is finished and blk_account_io_done() is called. In this case,\n sda2\u0027s hd_struct-\u003ein_flight, not a sda1\u0027s one, is decremented.\n\n | hd_struct-\u003ein_flight\n ---------------------------\n sda1 | -1\n sda2 | 1\n ---------------------------\n\nThe patch fixes the problem by caching the partition lookup\ninside the request structure, hence making sure that the increment\nand decrement will always happen on the same partition struct. This\nalso speeds up IO with accounting enabled, since it cuts down on\nthe number of lookups we have to do.\n\nAlso add a refcount to struct hd_struct to keep the partition in\nmemory as long as users exist. We use kref_test_and_get() to ensure\nwe don\u0027t add a reference to a partition which is going away.\n\nSigned-off-by: Jerome Marchand \u003cjmarchan@redhat.com\u003e\nSigned-off-by: Yasuaki Ishimatsu \u003cisimatu.yasuaki@jp.fujitsu.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "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": "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": "e9dd2b6837e26fe202708cce5ea4bb4ee3e3482e", "tree": "f42fd892495bfc4cbb740d06b016d267c9c42d00", "parents": [ "4f3a29dadaf999a273f1e7fe2476595d0283eef3", "b4627321e18582dcbdeb45d77df29d3177107c65" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:00:32 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Oct 22 17:00:32 2010 -0700" }, "message": "Merge branch \u0027for-2.6.37/core\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.37/core\u0027 of git://git.kernel.dk/linux-2.6-block: (39 commits)\n cfq-iosched: Fix a gcc 4.5 warning and put some comments\n block: Turn bvec_k{un,}map_irq() into static inline functions\n block: fix accounting bug on cross partition merges\n block: Make the integrity mapped property a bio flag\n block: Fix double free in blk_integrity_unregister\n block: Ensure physical block size is unsigned int\n blkio-throttle: Fix possible multiplication overflow in iops calculations\n blkio-throttle: limit max iops value to UINT_MAX\n blkio-throttle: There is no need to convert jiffies to milli seconds\n blkio-throttle: Fix link failure failure on i386\n blkio: Recalculate the throttled bio dispatch time upon throttle limit change\n blkio: Add root group to td-\u003etg_list\n blkio: deletion of a cgroup was causes oops\n blkio: Do not export throttle files if CONFIG_BLK_DEV_THROTTLING\u003dn\n block: set the bounce_pfn to the actual DMA limit rather than to max memory\n block: revert bad fix for memory hotplug causing bounces\n Fix compile error in blk-exec.c for !CONFIG_DETECT_HUNG_TASK\n block: set the bounce_pfn to the actual DMA limit rather than to max memory\n block: Prevent hang_check firing during long I/O\n cfq: improve fsync performance for small files\n ...\n\nFix up trivial conflicts due to __rcu sparse annotation in include/linux/genhd.h\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": "f281fb5fe54e15a7ab802945e42f8e24fceb56b2", "tree": "51a7dd2e900b8ce4118d1357ab6e187edfc12e25", "parents": [ "a850ea30374ebed32a0724742601861853fde869" ], "author": { "name": "Adrian Hunter", "email": "adrian.hunter@nokia.com", "time": "Sat Sep 25 12:42:55 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Sep 25 12:42:55 2010 +0200" }, "message": "block: prevent merges of discard and write requests\n\nAdd logic to prevent two I/O requests being merged if\nonly one of them is a discard. Ditto secure discard.\n\nWithout this fix, it is possible for write requests\nto transform into discard requests. For example:\n\n Submit bio 1 to discard 8 sectors from sector n\n Submit bio 2 to write 8 sectors from sector n + 16\n Submit bio 3 to write 8 sectors from sector n + 8\n\nBio 1 becomes request 1. Bio 2 becomes request 2.\nBio 3 is merged with request 2, and then subsequently\nrequest 2 is merged with request 1 resulting in just\none I/O request which discards all 24 sectors.\n\nSigned-off-by: Adrian Hunter \u003cadrian.hunter@nokia.com\u003e\n\n(Moved the checks above the position checks /Jens)\n\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": "2c8919dee659928d66cc13333d4e7a5bdd2206d5", "tree": "c60a8c1208cd89b0f5bdb7fc8cdc082b4569f289", "parents": [ "66ac0280197981f88774e74b60c8e5f9f07c1dba" ], "author": { "name": "Andi Kleen", "email": "andi@firstfloor.org", "time": "Mon Jun 21 11:02:47 2010 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Sat Aug 07 18:23:10 2010 +0200" }, "message": "gcc-4.6: block: fix unused but set variables in blk-merge\n\nJust some dead code.\n\nSigned-off-by: Andi Kleen \u003cak@linux.intel.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\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": "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": "316d315bffa4026f28085f6b24ebcebede370ac7", "tree": "10b6b057fec2382536371d2e14f9d4c0d6cf9eea", "parents": [ "23e018a1b083ecb4b8bb2fb43d58e7c19b5d7959" ], "author": { "name": "Nikanth Karthikesan", "email": "knikanth@suse.de", "time": "Tue Oct 06 20:16:55 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Oct 06 20:16:55 2009 +0200" }, "message": "block: Seperate read and write statistics of in_flight requests v2\n\nCommit a9327cac440be4d8333bba975cbbf76045096275 added seperate read\nand write statistics of in_flight requests. And exported the number\nof read and write requests in progress seperately through sysfs.\n\nBut Corrado Zoccolo \u003cczoccolo@gmail.com\u003e reported getting strange\noutput from \"iostat -kx 2\". Global values for service time and\nutilization were garbage. For interval values, utilization was always\n100%, and service time is higher than normal.\n\nSo this was reverted by commit 0f78ab9899e9d6acb09d5465def618704255963b\n\nThe problem was in part_round_stats_single(), I missed the following:\n if (now \u003d\u003d part-\u003estamp)\n return;\n\n- if (part-\u003ein_flight) {\n+ if (part_in_flight(part)) {\n __part_stat_add(cpu, part, time_in_queue,\n part_in_flight(part) * (now - part-\u003estamp));\n __part_stat_add(cpu, part, io_ticks, (now - part-\u003estamp));\n\nWith this chunk included, the reported regression gets fixed.\n\nSigned-off-by: Nikanth Karthikesan \u003cknikanth@suse.de\u003e\n\n--\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "0f78ab9899e9d6acb09d5465def618704255963b", "tree": "b92832da61ae4f4d712a33ae524c482e979be5ba", "parents": [ "e00c54c36ac2024c3a8a37432e2e2698ff849594" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Sun Oct 04 21:04:38 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Sun Oct 04 21:04:38 2009 +0200" }, "message": "Revert \"Seperate read and write statistics of in_flight requests\"\n\nThis reverts commit a9327cac440be4d8333bba975cbbf76045096275.\n\nCorrado Zoccolo \u003cczoccolo@gmail.com\u003e reports:\n\n\"with 2.6.32-rc1 I started getting the following strange output from\n\"iostat -kx 2\":\nLinux 2.6.31bisect (et2) \t04/10/2009 \t_i686_\t(2 CPU)\n\navg-cpu: %user %nice %system %iowait %steal %idle\n 10,70 0,00 3,16 15,75 0,00 70,38\n\nDevice: rrqm/s wrqm/s r/s w/s rkB/s wkB/s\navgrq-sz avgqu-sz await svctm %util\nsda 18,22 0,00 0,67 0,01 14,77 0,02\n43,94 0,01 10,53 39043915,03 2629219,87\nsdb 60,89 9,68 50,79 3,04 1724,43 50,52\n65,95 0,70 13,06 488437,47 2629219,87\n\navg-cpu: %user %nice %system %iowait %steal %idle\n 2,72 0,00 0,74 0,00 0,00 96,53\n\nDevice: rrqm/s wrqm/s r/s w/s rkB/s wkB/s\navgrq-sz avgqu-sz await svctm %util\nsda 0,00 0,00 0,00 0,00 0,00 0,00\n0,00 0,00 0,00 0,00 100,00\nsdb 0,00 0,00 0,00 0,00 0,00 0,00\n0,00 0,00 0,00 0,00 100,00\n\navg-cpu: %user %nice %system %iowait %steal %idle\n 6,68 0,00 0,99 0,00 0,00 92,33\n\nDevice: rrqm/s wrqm/s r/s w/s rkB/s wkB/s\navgrq-sz avgqu-sz await svctm %util\nsda 0,00 0,00 0,00 0,00 0,00 0,00\n0,00 0,00 0,00 0,00 100,00\nsdb 0,00 0,00 0,00 0,00 0,00 0,00\n0,00 0,00 0,00 0,00 100,00\n\navg-cpu: %user %nice %system %iowait %steal %idle\n 4,40 0,00 0,73 1,47 0,00 93,40\n\nDevice: rrqm/s wrqm/s r/s w/s rkB/s wkB/s\navgrq-sz avgqu-sz await svctm %util\nsda 0,00 0,00 0,00 0,00 0,00 0,00\n0,00 0,00 0,00 0,00 100,00\nsdb 0,00 4,00 0,00 3,00 0,00 28,00\n18,67 0,06 19,50 333,33 100,00\n\nGlobal values for service time and utilization are garbage. For\ninterval values, utilization is always 100%, and service time is\nhigher than normal.\n\nI bisected it down to:\n[a9327cac440be4d8333bba975cbbf76045096275] Seperate read and write\nstatistics of in_flight requests\nand verified that reverting just that commit indeed solves the issue\non 2.6.32-rc1.\"\n\nSo until this is debugged, revert the bad commit.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "a9327cac440be4d8333bba975cbbf76045096275", "tree": "83fd5dc45dc925c5b4eb0264c3add96cfaa4cae6", "parents": [ "18d8217bc441630c3c5ec7416c5a65c69e8a0979" ], "author": { "name": "Nikanth Karthikesan", "email": "knikanth@suse.de", "time": "Fri Sep 11 09:18:54 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Sep 14 08:24:52 2009 +0200" }, "message": "Seperate read and write statistics of in_flight requests\n\nCurrently, there is a single in_flight counter measuring the number of\nrequests in the request_queue. But some monitoring tools would like to\nknow how many read requests and write requests are in progress. Split the\ncurrent in_flight counter into two seperate counters for read and write.\n\nThis information is exported as a sysfs attribute, as changing the\ncurrently available stat files would break the existing tools.\n\nSigned-off-by: Nikanth Karthikesan \u003cknikanth@suse.de\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "da6c5c720c52cc717124f8f0830b710ea6a092fd", "tree": "f54c880aec9289895721e707d4b22e456b277127", "parents": [ "80a761fd33cf812f771e212139157bf8f58d4b3f" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Sep 11 14:26:40 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Sep 11 14:33:30 2009 +0200" }, "message": "scsi,block: update SCSI to handle mixed merge failures\n\nUpdate scsi_io_completion() such that it only fails requests till the\nnext error boundary and retry the leftover. This enables block layer\nto merge requests with different failfast settings and still behave\ncorrectly on errors. Allow merge of requests of different failfast\nsettings.\n\nAs SCSI is currently the only subsystem which follows failfast status,\nthere\u0027s no need to worry about other block drivers for now.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Niel Lambrechts \u003cniel.lambrechts@gmail.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "80a761fd33cf812f771e212139157bf8f58d4b3f", "tree": "21ea67ad749c15f25cb8a9278fe9bd7643c0ba31", "parents": [ "a82afdfcb8c0df09776b6458af6b68fc58b2e87b" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jul 03 17:48:17 2009 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Sep 11 14:33:30 2009 +0200" }, "message": "block: implement mixed merge of different failfast requests\n\nFailfast has characteristics from other attributes. When issuing,\nexecuting and successuflly completing requests, failfast doesn\u0027t make\nany difference. It only affects how a request is handled on failure.\nAllowing requests with different failfast settings to be merged cause\nnormal IOs to fail prematurely while not allowing has performance\npenalties as failfast is used for read aheads which are likely to be\nlocated near in-flight or to-be-issued normal IOs.\n\nThis patch introduces the concept of \u0027mixed merge\u0027. A request is a\nmixed merge if it is merge of segments which require different\nhandling on failure. Currently the only mixable attributes are\nfailfast ones (or lack thereof).\n\nWhen a bio with different failfast settings is added to an existing\nrequest or requests of different failfast settings are merged, the\nmerged request is marked mixed. Each bio carries failfast settings\nand the request always tracks failfast state of the first bio. When\nthe request fails, blk_rq_err_bytes() can be used to determine how\nmany bytes can be safely failed without crossing into an area which\nrequires further retrials.\n\nThis allows request merging regardless of failfast settings while\nkeeping the failure handling correct.\n\nThis patch only implements mixed merge but doesn\u0027t enable it. The\nnext one will update SCSI to make use of mixed merge.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Niel Lambrechts \u003cniel.lambrechts@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "ab0fd1debe730ec9998678a0c53caefbd121ed10", "tree": "c44de6ef7d876a32c4f733fdc26d40cdcbf7d374", "parents": [ "b59e64d0ddb756af57ea032383bfd393a286a8e8" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jul 03 12:56:18 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "axboe@carl.(none)", "time": "Fri Jul 03 21:06:45 2009 +0200" }, "message": "block: don\u0027t merge requests of different failfast settings\n\nBlock layer used to merge requests and bios with different failfast\nsettings. This caused regular IOs to fail prematurely when they were\nmerged into failfast requests for readahead.\n\nNiel Lambrechts could trigger the problem semi-reliably on ext4 when\nresuming from STR. ext4 uses readahead when reading inodes and\ncombined with the deterministic extra SATA PHY exception cycle during\nresume on the specific configuration, non-readahead inode read would\nfail causing ext4 errors. Please read the following thread for\ndetails.\n\n http://lkml.org/lkml/2009/5/23/21\n\nThis patch makes block layer reject merging if the failfast settings\ndon\u0027t match. This is correct but likely to lower IO performance by\npreventing regular IOs from mingling into surrounding readahead\nrequests. Changes to allow such mixed merges and handle errors\ncorrectly will be added later.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Niel Lambrechts \u003cniel.lambrechts@gmail.com\u003e\nCc: Theodore Tso \u003ctytso@mit.edu\u003e\nSigned-off-by: Jens Axboe \u003caxboe@carl.(none)\u003e\n" }, { "commit": "ae03bf639a5027d27270123f5f6e3ee6a412781d", "tree": "d705f41a188ad656b1f47f7952626a9f992e3b8f", "parents": [ "e1defc4ff0cf57aca6c5e3ff99fa503f5943c1f1" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri May 22 17:17:50 2009 -0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri May 22 23:22:54 2009 +0200" }, "message": "block: Use accessor functions for queue limits\n\nConvert all external users of queue limits to using wrapper functions\ninstead of poking the request queue variables directly.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "a2dec7b36364a5cc564c4d76cf16d2e7d33f5c05", "tree": "76dfb84ba5fa097a929ab81cf5718c6fcbc9d720", "parents": [ "34b7d2c957199834c474c9d46739265643f4d9c7" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu May 07 22:24:44 2009 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon May 11 09:50:55 2009 +0200" }, "message": "block: hide request sector and data_len\n\nBlock low level drivers for some reason have been pretty good at\nabusing block layer API. Especially struct request\u0027s fields tend to\nget violated in all possible ways. Make it clear that low level\ndrivers MUST NOT access or manipulate rq-\u003esector and rq-\u003edata_len\ndirectly by prefixing them with double underscores.\n\nThis change is also necessary to break build of out-of-tree codes\nwhich assume the previous block API where internal fields can be\nmanipulated and rq-\u003edata_len carries residual count on completion.\n\n[ Impact: hide internal fields, block API change ]\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "2e46e8b27aa57c6bd34b3102b40ee4d0144b4fab", "tree": "134f560f66c2af277f0f25e5b85f6c1acaddfc06", "parents": [ "9780e2dd8254351f6cbe11304849126b51dbd561" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu May 07 22:24:41 2009 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon May 11 09:50:54 2009 +0200" }, "message": "block: drop request-\u003ehard_* and *nr_sectors\n\nstruct request has had a few different ways to represent some\nproperties of a request. -\u003ehard_* represent block layer\u0027s view of the\nrequest progress (completion cursor) and the ones without the prefix\nare supposed to represent the issue cursor and allowed to be updated\nas necessary by the low level drivers. The thing is that as block\nlayer supports partial completion, the two cursors really aren\u0027t\nnecessary and only cause confusion. In addition, manual management of\nrequest detail from low level drivers is cumbersome and error-prone at\nthe very least.\n\nAnother interesting duplicate fields are rq-\u003e[hard_]nr_sectors and\nrq-\u003e{hard_cur|current}_nr_sectors against rq-\u003edata_len and\nrq-\u003ebio-\u003ebi_size. This is more convoluted than the hard_ case.\n\nrq-\u003e[hard_]nr_sectors are initialized for requests with bio but\nblk_rq_bytes() uses it only for !pc requests. rq-\u003edata_len is\ninitialized for all request but blk_rq_bytes() uses it only for pc\nrequests. This causes good amount of confusion throughout block layer\nand its drivers and determining the request length has been a bit of\nblack magic which may or may not work depending on circumstances and\nwhat the specific LLD is actually doing.\n\nrq-\u003e{hard_cur|current}_nr_sectors represent the number of sectors in\nthe contiguous data area at the front. This is mainly used by drivers\nwhich transfers data by walking request segment-by-segment. This\nvalue always equals rq-\u003ebio-\u003ebi_size \u003e\u003e 9. However, data length for\npc requests may not be multiple of 512 bytes and using this field\nbecomes a bit confusing.\n\nIn general, having multiple fields to represent the same property\nleads only to confusion and subtle bugs. With recent block low level\ndriver cleanups, no driver is accessing or manipulating these\nduplicate fields directly. Drop all the duplicates. Now rq-\u003esector\nmeans the current sector, rq-\u003edata_len the current total length and\nrq-\u003ebio-\u003ebi_size the current segment length. Everything else is\ndefined in terms of these three and available only through accessors.\n\n* blk_recalc_rq_sectors() is collapsed into blk_update_request() and\n now handles pc and fs requests equally other than rq-\u003esector update.\n This means that now pc requests can use partial completion too (no\n in-kernel user yet tho).\n\n* bio_cur_sectors() is replaced with bio_cur_bytes() as block layer\n now uses byte count as the primary data length.\n\n* blk_rq_pos() is now guranteed to be always correct. In-block users\n converted.\n\n* blk_rq_bytes() is now guaranteed to be always valid as is\n blk_rq_sectors(). In-block users converted.\n\n* blk_rq_sectors() is now guaranteed to equal blk_rq_bytes() \u003e\u003e 9.\n More convenient one is used.\n\n* blk_rq_bytes() and blk_rq_cur_bytes() are now inlined and take const\n pointer to request.\n\n[ Impact: API cleanup, single way to represent one property of a request ]\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Boaz Harrosh \u003cbharrosh@panasas.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "83096ebf1263b2c1ee5e653ba37d993d02e3eb7b", "tree": "2226b71a616ec1cb2d37511c6a09ba9507a1cd69", "parents": [ "5b93629b4509c03ffa87a9316412fedf6f58cb37" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu May 07 22:24:39 2009 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon May 11 09:50:54 2009 +0200" }, "message": "block: convert to pos and nr_sectors accessors\n\nWith recent cleanups, there is no place where low level driver\ndirectly manipulates request fields. This means that the \u0027hard\u0027\nrequest fields always equal the !hard fields. Convert all\nrq-\u003esectors, nr_sectors and current_nr_sectors references to\naccessors.\n\nWhile at it, drop superflous blk_rq_pos() \u003c 0 test in swim.c.\n\n[ Impact: use pos and nr_sectors accessors ]\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Geert Uytterhoeven \u003cGeert.Uytterhoeven@sonycom.com\u003e\nTested-by: Grant Likely \u003cgrant.likely@secretlab.ca\u003e\nAcked-by: Grant Likely \u003cgrant.likely@secretlab.ca\u003e\nTested-by: Adrian McMenamin \u003cadrian@mcmen.demon.co.uk\u003e\nAcked-by: Adrian McMenamin \u003cadrian@mcmen.demon.co.uk\u003e\nAcked-by: Mike Miller \u003cmike.miller@hp.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nCc: Bartlomiej Zolnierkiewicz \u003cbzolnier@gmail.com\u003e\nCc: Borislav Petkov \u003cpetkovbb@googlemail.com\u003e\nCc: Sergei Shtylyov \u003csshtylyov@ru.mvista.com\u003e\nCc: Eric Moore \u003cEric.Moore@lsi.com\u003e\nCc: Alan Stern \u003cstern@rowland.harvard.edu\u003e\nCc: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nCc: Pete Zaitcev \u003czaitcev@redhat.com\u003e\nCc: Stephen Rothwell \u003csfr@canb.auug.org.au\u003e\nCc: Paul Clements \u003cpaul.clements@steeleye.com\u003e\nCc: Tim Waugh \u003ctim@cyberelk.net\u003e\nCc: Jeff Garzik \u003cjgarzik@pobox.com\u003e\nCc: Jeremy Fitzhardinge \u003cjeremy@xensource.com\u003e\nCc: Alex Dubov \u003coakad@yahoo.com\u003e\nCc: David Woodhouse \u003cdwmw2@infradead.org\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Dario Ballabio \u003cballabio_dario@emc.com\u003e\nCc: David S. Miller \u003cdavem@davemloft.net\u003e\nCc: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: unsik Kim \u003cdonari75@gmail.com\u003e\nCc: Laurent Vivier \u003cLaurent@lvivier.info\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "42dad7647aec49b3ad20dd0cb832b232a6ae514f", "tree": "b70d4cb7706f2647e65426e24f078ddf14d6e139", "parents": [ "097102c2d04974bdfcfa16a5f3062d499842139c" ], "author": { "name": "Jerome Marchand", "email": "jmarchan@redhat.com", "time": "Wed Apr 22 14:01:49 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Apr 24 08:54:21 2009 +0200" }, "message": "block: simplify I/O stat accounting\n\nThis simplifies I/O stat accounting switching code and separates it\ncompletely from I/O scheduler switch code.\n\nRequests are accounted according to the state of their request queue\nat the time of the request allocation. There is no need anymore to\nflush the request queue when switching I/O accounting state.\n\nSigned-off-by: Jerome Marchand \u003cjmarchan@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "26308eab69aa193f7b3fb50764a64ae14544a39b", "tree": "758f6100395a23228d042c3a9d6f7dc1922d7530", "parents": [ "6c7e8cee6a9128eeb7f83c3ad1cb243f77f5cb16" ], "author": { "name": "Jerome Marchand", "email": "jmarchan@redhat.com", "time": "Fri Mar 27 10:31:51 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Apr 07 08:12:38 2009 +0200" }, "message": "block: fix inconsistency in I/O stat accounting code\n\nThis forces in_flight to be zero when turning off or on the I/O stat\naccounting and stops updating I/O stats in attempt_merge() when\naccounting is turned off.\n\nSigned-off-by: Jerome Marchand \u003cjmarchan@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "1cd96c242a829d52f7a5ae98f554ca9775429685", "tree": "018d3ab9fa0d6e2bcd739483ac2693f6f71db96b", "parents": [ "f028f3b2f987ebc61cef382ab7a5c449917b728e" ], "author": { "name": "Boaz Harrosh", "email": "bharrosh@panasas.com", "time": "Tue Mar 24 12:35:07 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Mar 26 11:01:23 2009 +0100" }, "message": "block: WARN in __blk_put_request() for potential bio leak\n\nPut a WARN_ON in __blk_put_request if it is about to\nleak bio(s). This is a serious bug that can happen in error\nhandling code paths.\n\nFor this to work I have fixed a couple of places in block/ where\nrequest-\u003ebio !\u003d NULL ownership was not honored. And a small cleanup\nat sg_io() while at it.\n\nSigned-off-by: Boaz Harrosh \u003cbharrosh@panasas.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "59247eaea50cc68cc6ce3d3fd3855f3301b65c96", "tree": "76548e4273bb26568810095b14d62a4feaaf86a4", "parents": [ "a3941ec101a5ec54c1e929730afeb196441a171e" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Mar 06 08:55:24 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Mar 06 08:55:24 2009 +0100" }, "message": "block: fix missing bio back/front segment size setting in blk_recount_segments()\n\nCommit 1e42807918d17e8c93bf14fbb74be84b141334c1 introduced a bug where we\ndon\u0027t get front/back segment sizes in the bio in blk_recount_segments().\nFix this by tracking the back bio as well as the front bio in\n__blk_recalc_rq_segments(), this also cleans up the interface by getting\nrid of the segment size pointer passing.\n\nTested-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nTested-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "1e42807918d17e8c93bf14fbb74be84b141334c1", "tree": "55b15f0f6b2c666efeb938d064f24fdd268faf40", "parents": [ "5e4c91c84b194b26cf592779e451f4b5be777cba" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Feb 23 09:03:10 2009 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Feb 26 10:45:48 2009 +0100" }, "message": "block: reduce stack footprint of blk_recount_segments()\n\nblk_recalc_rq_segments() requires a request structure passed in, which\nwe don\u0027t have from blk_recount_segments(). So the latter allocates one on\nthe stack, using \u003e 400 bytes of stack for that. This can cause us to spill\nover one page of stack from ext4 at least:\n\n 0) 4560 400 blk_recount_segments+0x43/0x62\n 1) 4160 32 bio_phys_segments+0x1c/0x24\n 2) 4128 32 blk_rq_bio_prep+0x2a/0xf9\n 3) 4096 32 init_request_from_bio+0xf9/0xfe\n 4) 4064 112 __make_request+0x33c/0x3f6\n 5) 3952 144 generic_make_request+0x2d1/0x321\n 6) 3808 64 submit_bio+0xb9/0xc3\n 7) 3744 48 submit_bh+0xea/0x10e\n 8) 3696 368 ext4_mb_init_cache+0x257/0xa6a [ext4]\n 9) 3328 288 ext4_mb_regular_allocator+0x421/0xcd9 [ext4]\n10) 3040 160 ext4_mb_new_blocks+0x211/0x4b4 [ext4]\n11) 2880 336 ext4_ext_get_blocks+0xb61/0xd45 [ext4]\n12) 2544 96 ext4_get_blocks_wrap+0xf2/0x200 [ext4]\n13) 2448 80 ext4_da_get_block_write+0x6e/0x16b [ext4]\n14) 2368 352 mpage_da_map_blocks+0x7e/0x4b3 [ext4]\n15) 2016 352 ext4_da_writepages+0x2ce/0x43c [ext4]\n16) 1664 32 do_writepages+0x2d/0x3c\n17) 1632 144 __writeback_single_inode+0x162/0x2cd\n18) 1488 96 generic_sync_sb_inodes+0x1e3/0x32b\n19) 1392 16 sync_sb_inodes+0xe/0x10\n20) 1376 48 writeback_inodes+0x69/0xb3\n21) 1328 208 balance_dirty_pages_ratelimited_nr+0x187/0x2f9\n22) 1120 224 generic_file_buffered_write+0x1d4/0x2c4\n23) 896 176 __generic_file_aio_write_nolock+0x35f/0x393\n24) 720 80 generic_file_aio_write+0x6c/0xc8\n25) 640 80 ext4_file_write+0xa9/0x137 [ext4]\n26) 560 320 do_sync_write+0xf0/0x137\n27) 240 48 vfs_write+0xb3/0x13c\n28) 192 64 sys_write+0x4c/0x74\n29) 128 128 system_call_fastpath+0x16/0x1b\n\nSplit the segment counting out into a __blk_recalc_rq_segments() helper\nto avoid allocating an onstack request just for checking the physical\nsegment count.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "43381785a5ba1cb424b36812373a6a04054b5c3c", "tree": "46917d4723f5cf167913597346c0377ab8da59a6", "parents": [ "75fa67706cce5272bcfc51ed646f2da21f3bdb6e" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Mon Oct 20 15:43:43 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Nov 06 08:41:55 2008 +0100" }, "message": "block: remove unused ll_new_mergeable()\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "8677142710516d986d932d6f1fba7be8382c1fec", "tree": "5221f48b4e7e62590f8bbb832b4065886681d6cf", "parents": [ "0fc71e3d6520ba7abad5cfbc9a33db0190e4d5b8" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Mon Oct 13 14:19:05 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Oct 17 08:46:56 2008 +0200" }, "message": "block: fix nr_phys_segments miscalculation bug\n\nThis fixes the bug reported by Nikanth Karthikesan \u003cknikanth@suse.de\u003e:\n\nhttp://lkml.org/lkml/2008/10/2/203\n\nThe root cause of the bug is that blk_phys_contig_segment\nmiscalculates q-\u003emax_segment_size.\n\nblk_phys_contig_segment checks:\n\nreq-\u003ebiotail-\u003ebi_size + next_req-\u003ebio-\u003ebi_size \u003e q-\u003emax_segment_size\n\nBut blk_recalc_rq_segments might expect that req-\u003ebiotail and the\nprevious bio in the req are supposed be merged into one\nsegment. blk_recalc_rq_segments might also expect that next_req-\u003ebio\nand the next bio in the next_req are supposed be merged into one\nsegment. In such case, we merge two requests that can\u0027t be merged\nhere. Later, blk_rq_map_sg gives more segments than it should.\n\nWe need to keep track of segment size in blk_recalc_rq_segments and\nuse it to see if two requests can be merged. This patch implements it\nin the similar way that we used to do for hw merging (virtual\nmerging).\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "ab780f1ece0dc8d5e8e8e85435acc5e4747ccda3", "tree": "4085b6c929a44de229eb5ef13fa2079211ea5512", "parents": [ "c7c22e4d5c1fdebfac4dba76de7d0338c2b0d832" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Aug 26 10:25:02 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:09 2008 +0200" }, "message": "block: inherit CPU completion on bio-\u003erq and rq-\u003erq merges\n\nSomewhat incomplete, as we do allow merges of requests and bios\nthat have different completion CPUs given. This is done on the\nassumption that a larger IO is still more beneficial than CPU\nlocality.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "074a7aca7afa6f230104e8e65eba3420263714a5", "tree": "f418313e45bd55be8156c8a3e8f9a216cf63058d", "parents": [ "eddb2e26b5ee3c5da68ba4bf1921ba20e2097bff" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:56:14 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:08 2008 +0200" }, "message": "block: move stats from disk to part0\n\nMove stats related fields - stamp, in_flight, dkstats - from disk to\npart0 and unify stat handling such that...\n\n* part_stat_*() now updates part0 together if the specified partition\n is not part0. ie. part_stat_*() are now essentially all_stat_*().\n\n* {disk|all}_stat_*() are gone.\n\n* part_round_stats() is updated similary. It handles part0 stats\n automatically and disk_round_stats() is killed.\n\n* part_{inc|dec}_in_fligh() is implemented which automatically updates\n part0 stats for parts other than part0.\n\n* disk_map_sector_rcu() is updated to return part0 if no part matches.\n Combined with the above changes, this makes NULL special case\n handling in callers unnecessary.\n\n* Separate stats show code paths for disk are collapsed into part\n stats show code paths.\n\n* Rename disk_stat_lock/unlock() to part_stat_lock/unlock()\n\nWhile at it, reposition stat handling macros a bit and add missing\nparentheses around macro parameters.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "c9959059161ddd7bf4670cf47367033d6b2f79c4", "tree": "6454db55f8e34361fe472358e10e0c5cfac1e366", "parents": [ "e71bf0d0ee89e51b92776391c5634938236977d5" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:47:21 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:06 2008 +0200" }, "message": "block: fix diskstats access\n\nThere are two variants of stat functions - ones prefixed with double\nunderbars which don\u0027t care about preemption and ones without which\ndisable preemption before manipulating per-cpu counters. It\u0027s unclear\nwhether the underbarred ones assume that preemtion is disabled on\nentry as some callers don\u0027t do that.\n\nThis patch unifies diskstats access by implementing disk_stat_lock()\nand disk_stat_unlock() which take care of both RCU (for partition\naccess) and preemption (for per-cpu counter access). diskstats access\nshould always be enclosed between the two functions. As such, there\u0027s\nno need for the versions which disables preemption. They\u0027re removed\nand double underbars ones are renamed to drop the underbars. As an\nextra argument is added, there\u0027s no danger of using the old version\nunconverted.\n\ndisk_stat_lock() uses get_cpu() and returns the cpu index and all\ndiskstat functions which access per-cpu counters now has @cpu\nargument to help RT.\n\nThis change adds RCU or preemption operations at some places but also\ncollapses several preemption ops into one at others. Overall, the\nperformance difference should be negligible as all involved ops are\nvery lightweight per-cpu ones.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "e71bf0d0ee89e51b92776391c5634938236977d5", "tree": "9fc62352a40ad388deebdd8ed497cab926cf0470", "parents": [ "f331c0296f2a9fee0d396a70598b954062603015" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Sep 03 09:03:02 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:06 2008 +0200" }, "message": "block: fix disk-\u003epart[] dereferencing race\n\ndisk-\u003epart[] is protected by its matching bdev\u0027s lock. However,\nnon-critical accesses like collecting stats and printing out sysfs and\nproc information used to be performed without any locking. As\npartitions can come and go dynamically, partitions can go away\nunderneath those non-critical accesses. As some of those accesses are\nwrites, this theoretically can lead to silent corruption.\n\nThis patch fixes the race by using RCU for the partition array and dev\nreference counter to hold partitions.\n\n* Rename disk-\u003epart[] to disk-\u003e__part[] to make sure no one outside\n genhd layer proper accesses it directly.\n\n* Use RCU for disk-\u003e__part[] dereferencing.\n\n* Implement disk_{get|put}_part() which can be used to get and put\n partitions from gendisk respectively.\n\n* Iterators are implemented to help iterate through all partitions\n safely.\n\n* Functions which require RCU readlock are marked with _rcu suffix.\n\n* Use disk_put_part() in __blkdev_put() instead of directly putting\n the contained kobject.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "310a2c1012934f590192377f65940cad4aa72b15", "tree": "ed30346abf07c5a7e94719f567368d5642af1f95", "parents": [ "88e341261ca4d39eec21b212961c77eff51105f7" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Aug 25 19:47:17 2008 +0900" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:04 2008 +0200" }, "message": "block: misc updates\n\nThis patch makes the following misc updates in preparation for\ndisk-\u003epart dereference fix and extended block devt support.\n\n* implment part_to_disk()\n\n* fix comment about gendisk-\u003epart indexing\n\n* rename get_part() to disk_map_sector()\n\n* don\u0027t use n which is always zero while printing disk information in\n diskstats_show()\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "5df97b91b5d7ed426034fcc84cb6e7cf682b8838", "tree": "727b9fb778f72d2b1ff44c007fb5209bacf67f4a", "parents": [ "b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Aug 15 10:20:02 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:03 2008 +0200" }, "message": "drop vmerge accounting\n\nRemove hw_segments field from struct bio and struct request. Without virtual\nmerge accounting they have no purpose.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5", "tree": "5832535c112c0504590256cb8a0bcabc6e282be3", "parents": [ "6a421c1dc94b12923294a359822346f12492de5e" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Fri Aug 15 10:15:19 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:03 2008 +0200" }, "message": "block: drop virtual merging accounting\n\nRemove virtual merge accounting.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "e17fc0a1ccf88f6d4dcb363729f3141b0958c325", "tree": "0a7c2dc1c3159c2af14d87c67ca83e158b2c78b5", "parents": [ "d30a2605be9d5132d95944916e8f578fcfe4f976" ], "author": { "name": "David Woodhouse", "email": "David.Woodhouse@intel.com", "time": "Sat Aug 09 16:42:20 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Oct 09 08:56:02 2008 +0200" }, "message": "Allow elevators to sort/merge discard requests\n\nBut blkdev_issue_discard() still emits requests which are interpreted as\nsoft barriers, because naïve callers might otherwise issue subsequent\nwrites to those same sectors, which might cross on the queue (if they\u0027re\nreallocated quickly enough).\n\nCallers still _can_ issue non-barrier discard requests, but they have to\ntake care of queue ordering for themselves.\n\nSigned-off-by: David Woodhouse \u003cDavid.Woodhouse@intel.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "7ba1ba12eeef0aa7113beb16410ef8b7c748e18b", "tree": "4629aabe88bf095d58eabd2f451207695bb35b08", "parents": [ "51d654e1d885607a6edd02b337105fa5c28b6d33" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Mon Jun 30 20:04:41 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Jul 03 13:21:13 2008 +0200" }, "message": "block: Block layer data integrity support\n\nSome block devices support verifying the integrity of requests by way\nof checksums or other protection information that is submitted along\nwith the I/O.\n\nThis patch implements support for generating and verifying integrity\nmetadata, as well as correctly merging, splitting and cloning bios and\nrequests that have this extra information attached.\n\nSee Documentation/block/data-integrity.txt for more information.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "2cdf79cafbd11580f5b63cd4993b45c1c4952415", "tree": "c0f4b7eea00127675785174a79ff0ac4c6d2669a", "parents": [ "7f3d4ee108c184ab215036051087aaaaa8de7661" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed May 07 09:33:55 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed May 07 09:33:55 2008 +0200" }, "message": "block: get rid of likely/unlikely predictions in merge logic\n\nThey tend to depend a lot on the workload, so not a clear-cut\nlikely or unlikely fit.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "75ad23bc0fcb4f992a5d06982bf0857ab1738e9e", "tree": "8668ef63b1f420252ae41aed9e13737d49fd8054", "parents": [ "68154e90c9d1492d570671ae181d9a8f8530da55" ], "author": { "name": "Nick Piggin", "email": "npiggin@suse.de", "time": "Tue Apr 29 14:48:33 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Apr 29 14:48:33 2008 +0200" }, "message": "block: make queue flags non-atomic\n\nWe can save some atomic ops in the IO path, if we clearly define\nthe rules of how to modify the queue flags.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "f18573abcc57844a7c3c12699d40eead8728cd8a", "tree": "99d5dcfdfe29578bb23900e63f226022f5e87281", "parents": [ "afdc1a780ef84a54b613dae6f971407748aab61c" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Fri Apr 11 12:56:52 2008 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Mon Apr 21 09:50:08 2008 +0200" }, "message": "block: move the padding adjustment to blk_rq_map_sg\n\nblk_rq_map_user adjusts bi_size of the last bio. It breaks the rule\nthat req-\u003edata_len (the true data length) is equal to sum(bio). It\nbroke the scsi command completion code.\n\ncommit e97a294ef6938512b655b1abf17656cf2b26f709 was introduced to fix\nthe above issue. However, the partial completion code doesn\u0027t work\nwith it. The commit is also a layer violation (scsi mid-layer should\nnot know about the block layer\u0027s padding).\n\nThis patch moves the padding adjustment to blk_rq_map_sg (suggested by\nJames). The padding works like the drain buffer. This patch breaks the\nrule that req-\u003edata_len is equal to sum(sg), however, the drain buffer\nalready broke it. So this patch just restores the rule that\nreq-\u003edata_len is equal to sub(bio) without breaking anything new.\n\nNow when a low level driver needs padding, blk_rq_map_user and\nblk_rq_map_user_iov guarantee there\u0027s enough room for padding.\nblk_rq_map_sg can safely extend the last entry of a scatter list.\n\nblk_rq_map_sg must extend the last entry of a scatter list only for a\nrequest that got through bio_copy_user_iov. This patches introduces\nnew REQ_COPY_USER flag.\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nCc: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: Mike Christie \u003cmichaelc@cs.wisc.edu\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "7a85f8896f4b4a4a0249563b92af9e3161a6b467", "tree": "616c62e3c96237e874fc0f47133fbca2160913b5", "parents": [ "89b6e743788516491846724d7ef89bcac7ac9c99" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Tue Mar 04 11:17:11 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "axboe@carl.home.kernel.dk", "time": "Tue Mar 04 11:17:11 2008 +0100" }, "message": "block: restore the meaning of rq-\u003edata_len to the true data length\n\nThe meaning of rq-\u003edata_len was changed to the length of an allocated\nbuffer from the true data length. It breaks SG_IO friends and\nbsg. This patch restores the meaning of rq-\u003edata_len to the true data\nlength and adds rq-\u003eextra_len to store an extended length (due to\ndrain buffer and padding).\n\nThis patch also removes the code to update bio in blk_rq_map_user\nintroduced by the commit 40b01b9bbdf51ae543a04744283bf2d56c4a6afa.\nThe commit adjusts bio according to memory alignment\n(queue_dma_alignment). However, memory alignment is NOT padding\nalignment. This adjustment also breaks SG_IO friends and bsg. Padding\nalignment needs to be fixed in a proper way (by a separate patch).\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nSigned-off-by: Jens Axboe \u003caxboe@carl.home.kernel.dk\u003e\n" }, { "commit": "db0a2e0099be3a1cff55879225881465f16c67d3", "tree": "297f788296f1722b4a6b2e5947fa9a3aa63dc36b", "parents": [ "2fb98e8414c42cb14698833aac640b143b9ade4f" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Feb 19 11:36:55 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 19 11:36:55 2008 +0100" }, "message": "block: clear drain buffer if draining for write command\n\nClear drain buffer before chaining if the command in question is a\nwrite.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "2fb98e8414c42cb14698833aac640b143b9ade4f", "tree": "ab241305b5ce3ea5ef3e76df7abe04b831cd9e34", "parents": [ "6b00769fe1502b4ad97bb327ef7ac971b208bfb5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Feb 19 11:36:53 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 19 11:36:53 2008 +0100" }, "message": "block: implement request_queue-\u003edma_drain_needed\n\nDraining shouldn\u0027t be done for commands where overflow may indicate\ndata integrity issues. Add dma_drain_needed callback to\nrequest_queue. Drain buffer is appened iff this function returns\nnon-zero.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "6b00769fe1502b4ad97bb327ef7ac971b208bfb5", "tree": "2ffc51176437ebf4f8a4ab13de59a32431043f15", "parents": [ "40b01b9bbdf51ae543a04744283bf2d56c4a6afa" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Feb 19 11:36:35 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Feb 19 11:36:35 2008 +0100" }, "message": "block: add request-\u003eraw_data_len\n\nWith padding and draining moved into it, block layer now may extend\nrequests as directed by queue parameters, so now a request has two\nsizes - the original request size and the extended size which matches\nthe size of area pointed to by bios and later by sgs. The latter size\nis what lower layers are primarily interested in when allocating,\nfilling up DMA tables and setting up the controller.\n\nBoth padding and draining extend the data area to accomodate\ncontroller characteristics. As any controller which speaks SCSI can\nhandle underflows, feeding larger data area is safe.\n\nSo, this patch makes the primary data length field, request-\u003edata_len,\nindicate the size of full data area and add a separate length field,\nrequest-\u003eraw_data_len, for the unmodified request size. The latter is\nused to report to higher layer (userland) and where the original\nrequest size should be fed to the controller or device.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "6f2576af5ba5913538fda7dfb7c6a17771025477", "tree": "ff55bc30686ccec8b0ce22eb3844e78a52aea875", "parents": [ "ea5c48ab2a76559d4af39e1f7de137c0851ac0a5" ], "author": { "name": "Jerome Marchand", "email": "jmarchan@redhat.com", "time": "Fri Feb 08 11:04:35 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 08 12:41:56 2008 +0100" }, "message": "Enhanced partition statistics: update partition statitics\n\nUpdates the enhanced partition statistics in generic block layer\nbesides the disk statistics.\n\nSigned-off-by: Jerome Marchand \u003cjmarchan@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "6728cb0e6343d4068ccec13f07212e6382d3ff33", "tree": "5a7826dc91cf2d9cf54e8c24b40aa3c4d892c797", "parents": [ "22b132102f1540dd40f3e41df88796829b685f1a" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Thu Jan 31 13:03:55 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri Feb 01 09:26:33 2008 +0100" }, "message": "block: make core bits checkpatch compliant\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "d6d48196961729b79509dcaeeb0f0fb4fbb47d29", "tree": "e80f5cdc7f0d682a46b816f5a489357f7d622d62", "parents": [ "db1d08c646e47f3c7835ecdabb51a906ec311f69" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jan 29 14:04:06 2008 +0100" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jan 29 21:55:12 2008 +0100" }, "message": "block: ll_rw_blk.c split, add blk-merge.c\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" } ] }