)]}' { "log": [ { "commit": "c9059598ea8981d02356eead3188bf7fa4d717b8", "tree": "03e73b20a30e988da7c6a3e0ad93b2dc5843274d", "parents": [ "0a33f80a8373eca7f4bea3961d1346c3815fa5ed", "b0fd271d5fba0b2d00888363f3869e3f9b26caa9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 11 10:52:27 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 11 11:10:35 2009 -0700" }, "message": "Merge branch \u0027for-2.6.31\u0027 of git://git.kernel.dk/linux-2.6-block\n\n* \u0027for-2.6.31\u0027 of git://git.kernel.dk/linux-2.6-block: (153 commits)\n block: add request clone interface (v2)\n floppy: fix hibernation\n ramdisk: remove long-deprecated \"ramdisk\u003d\" boot-time parameter\n fs/bio.c: add missing __user annotation\n block: prevent possible io_context-\u003erefcount overflow\n Add serial number support for virtio_blk, V4a\n block: Add missing bounce_pfn stacking and fix comments\n Revert \"block: Fix bounce limit setting in DM\"\n cciss: decode unit attention in SCSI error handling code\n cciss: Remove no longer needed sendcmd reject processing code\n cciss: change SCSI error handling routines to work with interrupts enabled.\n cciss: separate error processing and command retrying code in sendcmd_withirq_core()\n cciss: factor out fix target status processing code from sendcmd functions\n cciss: simplify interface of sendcmd() and sendcmd_withirq()\n cciss: factor out core of sendcmd_withirq() for use by SCSI error handling code\n cciss: Use schedule_timeout_uninterruptible in SCSI error handling code\n block: needs to set the residual length of a bidi request\n Revert \"block: implement blkdev_readpages\"\n block: Fix bounce limit setting in DM\n Removed reference to non-existing file Documentation/PCI/PCI-DMA-mapping.txt\n ...\n\nManually fix conflicts with tracing updates in:\n\tblock/blk-sysfs.c\n\tdrivers/ide/ide-atapi.c\n\tdrivers/ide/ide-cd.c\n\tdrivers/ide/ide-floppy.c\n\tdrivers/ide/ide-tape.c\n\tinclude/trace/events/block.h\n\tkernel/trace/blktrace.c\n" }, { "commit": "862366118026a358882eefc70238dbcc3db37aac", "tree": "4eb62bc10327a5afac064a95a091ea05ecd2acc1", "parents": [ "57eee9ae7bbcfb692dc96c739a5184adb6349733", "511b01bdf64ad8a38414096eab283c7784aebfc4" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 10 19:53:40 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 10 19:53:40 2009 -0700" }, "message": "Merge branch \u0027tracing-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip\n\n* \u0027tracing-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip: (244 commits)\n Revert \"x86, bts: reenable ptrace branch trace support\"\n tracing: do not translate event helper macros in print format\n ftrace/documentation: fix typo in function grapher name\n tracing/events: convert block trace points to TRACE_EVENT(), fix !CONFIG_BLOCK\n tracing: add protection around module events unload\n tracing: add trace_seq_vprint interface\n tracing: fix the block trace points print size\n tracing/events: convert block trace points to TRACE_EVENT()\n ring-buffer: fix ret in rb_add_time_stamp\n ring-buffer: pass in lockdep class key for reader_lock\n tracing: add annotation to what type of stack trace is recorded\n tracing: fix multiple use of __print_flags and __print_symbolic\n tracing/events: fix output format of user stack\n tracing/events: fix output format of kernel stack\n tracing/trace_stack: fix the number of entries in the header\n ring-buffer: discard timestamps that are at the start of the buffer\n ring-buffer: try to discard unneeded timestamps\n ring-buffer: fix bug in ring_buffer_discard_commit\n ftrace: do not profile functions when disabled\n tracing: make trace pipe recognize latency format flag\n ...\n" }, { "commit": "55782138e47d9baf2f7d3a7af9e7cf42adf72c56", "tree": "c7ccabae20e27bbeb08b69a358e8b86c98d1d9f3", "parents": [ "f57a8a1911342265e7acdc190333c4e9235a6632" ], "author": { "name": "Li Zefan", "email": "lizf@cn.fujitsu.com", "time": "Tue Jun 09 13:43:05 2009 +0800" }, "committer": { "name": "Steven Rostedt", "email": "rostedt@goodmis.org", "time": "Tue Jun 09 12:34:23 2009 -0400" }, "message": "tracing/events: convert block trace points to TRACE_EVENT()\n\nTRACE_EVENT is a more generic way to define tracepoints. Doing so adds\nthese new capabilities to this tracepoint:\n\n - zero-copy and per-cpu splice() tracing\n - binary tracing without printf overhead\n - structured logging records exposed under /debug/tracing/events\n - trace events embedded in function tracer output and other plugins\n - user-defined, per tracepoint filter expressions\n ...\n\nCons:\n\n - no dev_t info for the output of plug, unplug_timer and unplug_io events.\n no dev_t info for getrq and sleeprq events if bio \u003d\u003d NULL.\n no dev_t info for rq_abort,...,rq_requeue events if rq-\u003erq_disk \u003d\u003d NULL.\n\n This is mainly because we can\u0027t get the deivce from a request queue.\n But this may change in the future.\n\n - A packet command is converted to a string in TP_assign, not TP_print.\n While blktrace do the convertion just before output.\n\n Since pc requests should be rather rare, this is not a big issue.\n\n - In blktrace, an event can have 2 different print formats, but a TRACE_EVENT\n has a unique format, which means we have some unused data in a trace entry.\n\n The overhead is minimized by using __dynamic_array() instead of __array().\n\nI\u0027ve benchmarked the ioctl blktrace vs the splice based TRACE_EVENT tracing:\n\n dd dd + ioctl blktrace dd + TRACE_EVENT (splice)\n1 7.36s, 42.7 MB/s 7.50s, 42.0 MB/s 7.41s, 42.5 MB/s\n2 7.43s, 42.3 MB/s 7.48s, 42.1 MB/s 7.43s, 42.4 MB/s\n3 7.38s, 42.6 MB/s 7.45s, 42.2 MB/s 7.41s, 42.5 MB/s\n\nSo the overhead of tracing is very small, and no regression when using\nthose trace events vs blktrace.\n\nAnd the binary output of TRACE_EVENT is much smaller than blktrace:\n\n # ls -l -h\n -rw-r--r-- 1 root root 8.8M 06-09 13:24 sda.blktrace.0\n -rw-r--r-- 1 root root 195K 06-09 13:24 sda.blktrace.1\n -rw-r--r-- 1 root root 2.7M 06-09 13:25 trace_splice.out\n\nFollowing are some comparisons between TRACE_EVENT and blktrace:\n\nplug:\n kjournald-480 [000] 303.084981: block_plug: [kjournald]\n kjournald-480 [000] 303.084981: 8,0 P N [kjournald]\n\nunplug_io:\n kblockd/0-118 [000] 300.052973: block_unplug_io: [kblockd/0] 1\n kblockd/0-118 [000] 300.052974: 8,0 U N [kblockd/0] 1\n\nremap:\n kjournald-480 [000] 303.085042: block_remap: 8,0 W 102736992 + 8 \u003c- (8,8) 33384\n kjournald-480 [000] 303.085043: 8,0 A W 102736992 + 8 \u003c- (8,8) 33384\n\nbio_backmerge:\n kjournald-480 [000] 303.085086: block_bio_backmerge: 8,0 W 102737032 + 8 [kjournald]\n kjournald-480 [000] 303.085086: 8,0 M W 102737032 + 8 [kjournald]\n\ngetrq:\n kjournald-480 [000] 303.084974: block_getrq: 8,0 W 102736984 + 8 [kjournald]\n kjournald-480 [000] 303.084975: 8,0 G W 102736984 + 8 [kjournald]\n\n bash-2066 [001] 1072.953770: 8,0 G N [bash]\n bash-2066 [001] 1072.953773: block_getrq: 0,0 N 0 + 0 [bash]\n\nrq_complete:\n konsole-2065 [001] 300.053184: block_rq_complete: 8,0 W () 103669040 + 16 [0]\n konsole-2065 [001] 300.053191: 8,0 C W 103669040 + 16 [0]\n\n ksoftirqd/1-7 [001] 1072.953811: 8,0 C N (5a 00 08 00 00 00 00 00 24 00) [0]\n ksoftirqd/1-7 [001] 1072.953813: block_rq_complete: 0,0 N (5a 00 08 00 00 00 00 00 24 00) 0 + 0 [0]\n\nrq_insert:\n kjournald-480 [000] 303.084985: block_rq_insert: 8,0 W 0 () 102736984 + 8 [kjournald]\n kjournald-480 [000] 303.084986: 8,0 I W 102736984 + 8 [kjournald]\n\nChangelog from v2 -\u003e v3:\n\n- use the newly introduced __dynamic_array().\n\nChangelog from v1 -\u003e v2:\n\n- use __string() instead of __array() to minimize the memory required\n to store hex dump of rq-\u003ecmd().\n\n- support large pc requests.\n\n- add missing blk_fill_rwbs_rq() in block_rq_requeue TRACE_EVENT.\n\n- some cleanups.\n\nSigned-off-by: Li Zefan \u003clizf@cn.fujitsu.com\u003e\nLKML-Reference: \u003c4A2DF669.5070905@cn.fujitsu.com\u003e\nSigned-off-by: Steven Rostedt \u003crostedt@goodmis.org\u003e\n" }, { "commit": "0e6e0271a210817e202c8a4bfffbde3e3c0616d1", "tree": "0452e29d393b7688f09f38c98f113703d1b5cc3d", "parents": [ "a8c906ca3f63d05f0d25490cf82276f73c6fe095" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 16:32:22 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 16:32:22 2009 +1000" }, "message": "md/raid5: fix bug in reshape code when chunk_size decreases.\n\nNow that we support changing the chunksize, we calculate\n\"reshape_sectors\" to be the max of number of sectors in old\nand new chunk size.\nHowever there is one please where we still use \u0027chunksize\u0027\nrather than \u0027reshape_sectors\u0027.\nThis causes a reshape that reduces the size of chunks to freeze.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "a8c906ca3f63d05f0d25490cf82276f73c6fe095", "tree": "993249ef5865a2ec81ed21e8934ed9c0eb9f33dd", "parents": [ "f001a70cdc61c01452d42e8b32fd7c7842ef62d5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 14:39:59 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 14:39:59 2009 +1000" }, "message": "md/raid5 - avoid deadlocks in get_active_stripe during reshape\n\nmd has functionality to \u0027quiesce\u0027 and array so that all pending\nIO completed and no new IO starts. This is used to achieve a\nstable state before making internal changes.\n\nCurrently this quiescing applies equally to normal IO, resync\nIO, and reshape IO.\nHowever there is a problem with applying it to reshape IO.\nReshape can have multiple \u0027stripe_heads\u0027 that must be active together.\nIf the quiesce come between allocating the first and the last of\nsuch a collection, then we deadlock, as the last will not be allocated\nuntil the quiesce is lifted, the quiesce will not be lifted until the\nfirst (which has been allocated) gets used, and that first cannot be\nused until the last is allocated.\n\nIt is not necessary to inhibit reshape IO when a quiesce is\nrequested. Those places in the code that require a full quiesce will\nensure the reshape thread is not running at all.\n\nSo allow reshape requests to get access to new stripe_heads without\nbeing blocked by a \u0027quiesce\u0027.\n\nThis only affects in-place reshapes (i.e. where the array does not\ngrow or shrink) and these are only newly supported. So this patch is\nnot needed in earlier kernels.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "f001a70cdc61c01452d42e8b32fd7c7842ef62d5", "tree": "6427a1baca4b818aa2081cbeb5534f2e6b17cf5e", "parents": [ "ccc0d38ec13d4649d4168c1db590137df53ad783" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 14:30:31 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Jun 09 14:30:31 2009 +1000" }, "message": "md/raid5: use conf-\u003eraid_disks in preference to mddev-\u003eraid_disk\n\nmddev-\u003eraid_disks can be changed and any time by a request from\nuser-space. It is a suggestion as to what number of raid_disks is\ndesired.\n\nconf-\u003eraid_disks can only be changed by the raid5 module with suitable\nlocks in place. It is a statement as to the current number of\nraid_disks.\n\nThere are two places where the latter should be used, but the former\nis used. This can lead to a crash when reshaping an array.\n\nThis patch changes to mddev-\u003e to conf-\u003e\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "9df1bb9b516daeece159ab7fb262d01a0359247c", "tree": "602c7b02e85b031d0269a1f70ece81bac524c0ac", "parents": [ "3969251b80a7b143d01576780073fe0cc9ef6253" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jun 09 06:22:57 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Tue Jun 09 06:22:57 2009 +0200" }, "message": "Revert \"block: Fix bounce limit setting in DM\"\n\nThis reverts commit a05c0205ba031c01bba33a21bf0a35920eb64833.\n\nDM doesn\u0027t need to access the bounce_pfn directly.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "a05c0205ba031c01bba33a21bf0a35920eb64833", "tree": "517c7682fc415cdb122ee7fcfc75eff674cc7b78", "parents": [ "dbdc9dd342f0a7e32f40f0d4ade662bdfe057484" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Wed Jun 03 09:33:18 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Jun 03 09:33:18 2009 +0200" }, "message": "block: Fix bounce limit setting in DM\n\nblk_queue_bounce_limit() is more than a wrapper about the request queue\nlimits.bounce_pfn variable. Introduce blk_queue_bounce_pfn() which can\nbe called by stacking drivers that wish to set the bounce limit\nexplicitly.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "ed37d83e6aa218192fb28bb6b82498d2a8c74070", "tree": "bfa8462c7be84539deae430355228305e5516a7a", "parents": [ "b492b852cd8c99505708152c29a5e09a787af9de" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 27 21:39:05 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Wed May 27 21:39:05 2009 +1000" }, "message": "md: raid5: change incorrect usage of \u0027min\u0027 macro to \u0027min_t\u0027\n\nA recent patch to raid5.c use min on an int and a sector_t.\nThis isn\u0027t allowed.\nSo change it to min_t(sector_t,x,y).\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b492b852cd8c99505708152c29a5e09a787af9de", "tree": "f26aa9676231a583e209a9397acfc4512061325e", "parents": [ "7a91ee1f628ef6bfe3f13067c0ddf9db520cb86b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:36 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:36 2009 +1000" }, "message": "md: don\u0027t use locked_ioctl.\n\nmd has no need for the BKL - it does its own locking.\nSo md_ioctl doesn\u0027t need to be a locked_ioctl.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7a91ee1f628ef6bfe3f13067c0ddf9db520cb86b", "tree": "b6b317a2dfb067784fccae113001cb3957e25337", "parents": [ "848b3182365fdf5a05bcd5ed949071cac2c894b3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:21 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:57:21 2009 +1000" }, "message": "md: don\u0027t update curr_resync_completed without also updating reshape_position.\n\nIn order for the metadata to always be consistent, we mustn\u0027t updated\ncurr_resync_completed without also updating reshape_position.\n\nThe reshape code updates both at the same time. However since\ncommit 97e4f42d62badb0f9fbc27c013e89bc1336a03bc\nthe common md_do_sync will sometimes update curr_resync_completed\nbut is not in a position to update reshape_position.\nSo if MD_RECOVERY_RESHAPE is set (indicating that a reshape is\nhappening, so reshape_position might change), don\u0027t update\ncurr_resync_completed in md_do_sync, leave it to the per-personality\nreshape code.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "848b3182365fdf5a05bcd5ed949071cac2c894b3", "tree": "cfb09457f8d8d582d443a92c2a47f1f9862a2036", "parents": [ "b6a9ce688f613e2ee5f15e6720e0bb8520efc36e" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:41:08 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 12:41:08 2009 +1000" }, "message": "md: raid5: avoid sector values going negative when testing reshape progress.\n\nAs sector_t in unsigned, we cannot afford to let \u0027safepos\u0027 etc go\nnegative.\nSo replace\n a -\u003d b;\nby\n a -\u003d min(b,a);\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b6a9ce688f613e2ee5f15e6720e0bb8520efc36e", "tree": "7fc4c8ab5ea14df5e544e740ec3154b707766a08", "parents": [ "be512691036cc989c11d0f418187efbbf14468e6" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "message": "md: export \u0027frozen\u0027 resync state through sysfs\n\nThe md resync engine has a \u0027frozen\u0027 state which ensures that\nno resync/recovery. This is used to avoid races.\n\nExport this state through the \u0027sync_action\u0027 sysfs attribute\nso that user-space can benefit and also avoid some races.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "be512691036cc989c11d0f418187efbbf14468e6", "tree": "557b303a058fc3effdfc15a5cc08867fb0ee6889", "parents": [ "2b69c83924396ad1eda36fdd267c9d2f360f5555" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "message": "md: bitmap: improve bitmap maintenance code.\n\nThe code for checking which bits in the bitmap can be cleared\nhas 2 problems:\n 1/ it repeatedly takes and drops a spinlock, where it would make\n more sense to just hold on to it most of the time.\n 2/ it doesn\u0027t make use of some opportunities to skip large sections\n of the bitmap\n\nThis patch fixes those. It will only affect CPU consumption, not\ncorrectness.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2b69c83924396ad1eda36fdd267c9d2f360f5555", "tree": "14c01b6dd6b596859835b3f0d176bf02129d2c45", "parents": [ "62e1e389f87a8839ad83b08c44691d1df8320846" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:41:17 2009 +1000" }, "message": "md: improve errno return when setting array_size\n\nInstead of always returns EINVAL if anything goes wrong\nwhen setting the array size, add the option of\n E2BIG\nif the size requested is too large. This makes it easier\nfor user-space to be sure what went wrong.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "62e1e389f87a8839ad83b08c44691d1df8320846", "tree": "d8915df95d7164188334052b31ffe7ac40aa4e94", "parents": [ "59a3759d0fe8d969888c741bb33f4946e4d3750d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:40:59 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue May 26 09:40:59 2009 +1000" }, "message": "md: always update level / chunk_size / layout when writing v1.x metadata.\n\nWe previously didn\u0027t update these fields when writing the metadata\nbecause they could never change. They can now, so we better write\nthem.\nv0.90 metadata always updated these fields.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "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": "e1defc4ff0cf57aca6c5e3ff99fa503f5943c1f1", "tree": "d60d15a082171c58ac811d547d51a9c3119f23e3", "parents": [ "9bd7de51ee8537094656149eaf45338cadb7d7d4" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Fri May 22 17:17:49 2009 -0400" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Fri May 22 23:22:54 2009 +0200" }, "message": "block: Do away with the notion of hardsect_size\n\nUntil now we have had a 1:1 mapping between storage device physical\nblock size and the logical block sized used when addressing the device.\nWith SATA 4KB drives coming out that will no longer be the case. The\nsector size will be 4KB but the logical block size will remain\n512-bytes. Hence we need to distinguish between the physical block size\nand the logical ditto.\n\nThis patch renames hardsect_size to logical_block_size.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "1079cac0f4eb7d968395378b1625979d4c818dd6", "tree": "d261058d95d27fe7250511d5fad69db30dc17731", "parents": [ "5872144f64b34a5942f6b4acedc90b02de72c58b", "1406de8e11eb043681297adf86d6892ff8efc27a" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon May 18 10:15:09 2009 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon May 18 10:15:35 2009 +0200" }, "message": "Merge commit \u0027v2.6.30-rc6\u0027 into tracing/core\n\nMerge reason: we were on an -rc4 base, sync up to -rc6\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "44347d947f628060b92449702071bfe1d31dfb75", "tree": "c6ed74610d5b3295df4296659f80f5feb94b28cc", "parents": [ "d94fc523f3c35bd8013f04827e94756cbc0212f4", "413f81eba35d6ede9289b0c8a920c013a84fac71" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Thu May 07 11:17:13 2009 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Thu May 07 11:17:34 2009 +0200" }, "message": "Merge branch \u0027linus\u0027 into tracing/core\n\nMerge reason: tracing/core was on a .30-rc1 base and was missing out on\n on a handful of tracing fixes present in .30-rc5-almost.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "c4647292fda0833bebe45be27f04453b736981fa", "tree": "85c89a4161b06b1c591a76c2a68ff51fc02ff6f4", "parents": [ "5bf295975416f8e97117bbbcfb0191c00bc3e2b4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:51:06 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:51:06 2009 +1000" }, "message": "md: remove rd%d links immediately after stopping an array.\n\nmd maintains link in sys/mdXX/md/ to identify which device has\nwhich role in the array. e.g.\n rd2 -\u003e dev-sda\n\nindicates that the device with role \u00272\u0027 in the array is sda.\n\nThese links are only present when the array is active. They are\ncreated immediately after -\u003erun is called, and so should be removed\nimmediately after -\u003estop is called.\nHowever they are currently removed a little bit later, and it is\npossible for -\u003erun to be called again, thus adding these links, before\nthey are removed.\n\nSo move the removal earlier so they are consistently only present when\nthe array is active.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "5bf295975416f8e97117bbbcfb0191c00bc3e2b4", "tree": "9d7ddfe9d4632c8d6cacb6c4dd7b9cb49f837432", "parents": [ "110518bccf076726cc93bf604527d8019aae50ba" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:50:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:50:57 2009 +1000" }, "message": "md: remove ability to explicit set an inactive array to \u0027clean\u0027.\n\nBeing able to write \u0027clean\u0027 to an \u0027array_state\u0027 of an inactive array\nto activate it in \u0027clean\u0027 mode is both unnecessary and inconvenient.\n\nIt is unnecessary because the same can be achieved by writing\n\u0027active\u0027. This activates and array, but it still remains \u0027clean\u0027\nuntil the first write.\n\nIt is inconvenient because writing \u0027clean\u0027 is more often used to\ncause an \u0027active\u0027 array to revert to \u0027clean\u0027 mode (thus blocking\nany writes until a \u0027write-pending\u0027 is promoted to \u0027active\u0027).\n\nAllowing \u0027clean\u0027 to both activate an array and mark an active array as\nclean can lead to races: One program writes \u0027clean\u0027 to mark the\nactive array as clean at the same time as another program writes\n\u0027inactive\u0027 to deactivate (stop) and active array. Depending on which\nwrites first, the array could be deactivated and immediately\nreactivated which isn\u0027t what was desired.\n\nSo just disable the use of \u0027clean\u0027 to activate an array.\n\nThis avoids a race that can be triggered with mdadm-3.0 and external\nmetadata, so it suitable for -stable.\n\nReported-by: Rafal Marszewski \u003crafal.marszewski@intel.com\u003e\nAcked-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "110518bccf076726cc93bf604527d8019aae50ba", "tree": "f2df7daf983640c64f66bcd76aea475b35f9f1d3", "parents": [ "dd71cf6b2773310b01c6fe6c773064c80fd2476b" ], "author": { "name": "Jan Engelhardt", "email": "jengelh@medozas.de", "time": "Thu May 07 12:49:37 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:37 2009 +1000" }, "message": "md: constify VFTs\n\n\nSigned-off-by: Jan Engelhardt \u003cjengelh@medozas.de\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "dd71cf6b2773310b01c6fe6c773064c80fd2476b", "tree": "33a4cc4fe4db45ec4f1a8f477bb294a0b685efa4", "parents": [ "db305e507d554430a69ede901a6308e6ecb72349" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:35 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:35 2009 +1000" }, "message": "md: tidy up status_resync to handle large arrays.\n\nTwo problems in status_resync.\n1/ It still used Kilobytes as the basic block unit, while most code\n now uses sectors uniformly.\n2/ It doesn\u0027t allow for the possibility that max_sectors exceeds\n the range of \"unsigned long\".\n\nSo\n - change \"max_blocks\" to \"max_sectors\", and store sector numbers\n in there and in \u0027resync\u0027\n - Make \u0027rt\u0027 a \u0027sector_t\u0027 so it can temporarily hold the number of\n remaining sectors.\n - use sector_div rather than normal division.\n - change the magic \u0027100\u0027 used to preserve precision to \u002732\u0027.\n + making it a power of 2 makes division easier\n + it doesn\u0027t need to be as large as it was chosen when we averaged\n speed over the entire run. Now we average speed over the last 30\n seconds or so.\n\nReported-by: \"Mario \u0027BitKoenig\u0027 Holbe\" \u003cMario.Holbe@TU-Ilmenau.DE\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "db305e507d554430a69ede901a6308e6ecb72349", "tree": "b3ad8266ff05ae9059de551cdf8a8965bd27bcb1", "parents": [ "18055569127253755d01733f6ecc004ed02f88d0" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:06 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:49:06 2009 +1000" }, "message": "md: fix some (more) errors with bitmaps on devices larger than 2TB.\n\nIf a write intent bitmap covers more than 2TB, we sometimes work with\nvalues beyond 32bit, so these need to be sector_t. This patches\nadd the required casts to some unsigned longs that are being shifted\nup.\n\nThis will affect any raid10 larger than 2TB, or any raid1/4/5/6 with\nmember devices that are larger than 2TB.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\nReported-by: \"Mario \u0027BitKoenig\u0027 Holbe\" \u003cMario.Holbe@TU-Ilmenau.DE\u003e\nCc: stable@kernel.org\n" }, { "commit": "18055569127253755d01733f6ecc004ed02f88d0", "tree": "5f1fa83f98d8a6d08cd1916106a2c5b9f9f6ec43", "parents": [ "b74fd2826c5acce20e6f691437b2d19372bc2057" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:48:10 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:48:10 2009 +1000" }, "message": "md/raid10: don\u0027t clear bitmap during recovery if array will still be degraded.\n\nIf we have a raid10 with multiple missing devices, and we recover just\none of these to a spare, then we risk (depending on the bitmap and\narray chunk size) clearing bits of the bitmap for which recovery isn\u0027t\ncomplete (because a device is still missing).\n\nThis can lead to a subsequent \"re-add\" being recovered without\nany IO happening, which would result in loss of data.\n\nThis patch takes the safe approach of not clearing bitmap bits\nif the array will still be degraded.\n\nThis patch is suitable for all active -stable kernels.\n\nCc: stable@kernel.org\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b74fd2826c5acce20e6f691437b2d19372bc2057", "tree": "818e1fa7a7f91b1ca37279f19ce215f19256d7f0", "parents": [ "b4348f32dae3cb6eb4bc21c7ed8f76c0b11e9d6a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:47:19 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Thu May 07 12:47:19 2009 +1000" }, "message": "md: fix loading of out-of-date bitmap.\n\nWhen md is loading a bitmap which it knows is out of date, it fills\neach page with 1s and writes it back out again. However the\nwrite_page call makes used of bitmap-\u003efile_pages and\nbitmap-\u003elast_page_size which haven\u0027t been set correctly yet. So this\ncan sometimes fail.\n\nMove the setting of file_pages and last_page_size to before the call\nto write_page.\n\nThis bug can cause the assembly on an array to fail, thus making the\ndata inaccessible. Hence I think it is a suitable candidate for\n-stable.\n\nCc: stable@kernel.org\nReported-by: Vojtech Pavlik \u003cvojtech@suse.cz\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "22a7c31a9659deaddafbbcec6562d44141e84474", "tree": "0fe1c64e82d673597c78c6ce5f3569d9101b5281", "parents": [ "a42aaa3bbce85ac487ad4fad5db99e8e91b7aac1" ], "author": { "name": "Alan D. Brunelle", "email": "Alan.Brunelle@hp.com", "time": "Mon May 04 16:35:08 2009 -0400" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Wed May 06 14:13:01 2009 +0200" }, "message": "blktrace: from-sector redundant in trace_block_remap\n\nRemove redundant from-sector parameter: it\u0027s /always/ the bio\u0027s sector\npassed in.\n\n[ Impact: cleanup ]\n\nSigned-off-by: Alan D. Brunelle \u003calan.brunelle@hp.com\u003e\nReviewed-by: Li Zefan \u003clizf@cn.fujitsu.com\u003e\nReviewed-by: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nCc: Arnaldo Carvalho de Melo \u003cacme@redhat.com\u003e\nLKML-Reference: \u003c49FF517C.7000503@hp.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "2edbdd1266784495979576c4ab4ef294c29bad27", "tree": "fd4909b58e783145f19cde650e7d4d16daa36b21", "parents": [ "a5432f5ad43822802ba6f8ba8662ca6361712044", "1f59390339f263c0fe7908fbe54466dbf3a64b58" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 20 08:37:37 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 20 08:37:37 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md:\n md: support bitmaps on RAID10 arrays larger then 2 terabytes\n md: update sync_completed and reshape_position even more often.\n md: improve usefulness and accuracy of sysfs file md/sync_completed.\n md: allow setting newly added device to \u0027in_sync\u0027 via sysfs.\n md: tiny md.h cleanups\n" }, { "commit": "1f59390339f263c0fe7908fbe54466dbf3a64b58", "tree": "334f287c819c1876f29cd5833222eacd910e0ada", "parents": [ "c03f6a19699fce4389888a45db8245969311d868" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 20 11:50:24 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 20 11:50:24 2009 +1000" }, "message": "md: support bitmaps on RAID10 arrays larger then 2 terabytes\n\n.. and other arrays with components larger than 2 terabytes.\n\nWe use a \"long\" rather than a \"sector_t\" in part of the bitmap\nsize calculations, which is sad.\n\nReported-by: \"Mario \u0027BitKoenig\u0027 Holbe\" \u003cMario.Holbe@TU-Ilmenau.DE\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "c03f6a19699fce4389888a45db8245969311d868", "tree": "2a3be3033d5fad42f25fe2678e98efcf6abb04bc", "parents": [ "acb180b0e335ad88acfed6c8a33e39c05b95dc49" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Apr 17 11:06:30 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Fri Apr 17 11:06:30 2009 +1000" }, "message": "md: update sync_completed and reshape_position even more often.\n\nThere are circumstances when a user-space process might need to\n\"oversee\" a resync/reshape process. For example when doing an\nin-place reshape of a raid5, it is prudent to take a backup of each\nsection before reshaping it as this is the only way to provide\nsafety against an unplanned shutdown (i.e. crash/power failure).\n\nThe sync_max sysfs value can be used to stop the resync from\nadvancing beyond a particular point.\nSo user-space can:\n suspend IO to the first section and back it up\n set \u0027sync_max\u0027 to the end of the section\n wait for \u0027sync_completed\u0027 to reach that point\n resume IO on the first section and move on to the next section.\n\nHowever this process requires the kernel and user-space to run in\nlock-step which could introduce unnecessary delays.\n\nIt would be better if a \u0027double buffered\u0027 approach could be used with\nuserspace and kernel space working on different sections with the\n\u0027next\u0027 section always ready when the \u0027current\u0027 section is finished.\n\nOne problem with implementing this is that sync_completed is only\nguaranteed to be updated when the sync process reaches sync_max.\n(it is updated on a time basis at other times, but it is hard to rely\non that). This defeats some of the double buffering.\n\nWith this patch, sync_completed (and reshape_position) get updated as\nthe current position approaches sync_max, so there is room for\nuserspace to advance sync_max early without losing updates.\n\nTo be precise, sync_completed is updated when the current sync\nposition reaches half way between the current value of sync_completed\nand the value of sync_max. This will usually be a good time for user\nspace to update sync_max.\n\nIf sync_max does not get updated, the updates to sync_completed\n(together with associated metadata updates) will occur at an\nexponentially increasing frequency which will get unreasonably fast\n(one update every page) immediately before the process hits sync_max\nand stops. So the update rate will be unreasonably fast only for an\ninsignificant period of time.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "8f3d8ba20e67991b531e9c0227dcd1f99271a32c", "tree": "337efd46f067ce3b7eeab0c3627b02a7edef4afd", "parents": [ "0882e8dd3aad33eca41696d463bb896e6c8817eb" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Apr 07 19:55:13 2009 +0200" }, "committer": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Apr 15 08:28:09 2009 +0200" }, "message": "block: move bio list helpers into bio.h\n\nIt\u0027s used by DM and MD and generally useful, so move the bio list\nhelpers into bio.h.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\n" }, { "commit": "acb180b0e335ad88acfed6c8a33e39c05b95dc49", "tree": "fa980978bb66b178fd8c2f474194754139c0921d", "parents": [ "6d56e278444bc8323b1bcfcada126b8e4dbba0f4" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 16:28:34 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 16:28:34 2009 +1000" }, "message": "md: improve usefulness and accuracy of sysfs file md/sync_completed.\n\nThe sync_completed file reports how much of a resync (or recovery or\nreshape) has been completed.\nHowever due to the possibility of out-of-order completion of writes,\nit is not certain to be accurate.\n\nWe have an internal value - mddev-\u003ecurr_resync_completed - which is an\naccurate value (though it might not always be quite so uptodate).\n\nSo:\n - make curr_resync_completed be uptodate a little more often,\n particularly when raid5 reshape updates status in the metadata\n - report curr_resync_completed in the sysfs file\n - allow poll/select to report all updates to md/sync_completed.\n\nThis makes sync_completed completed usable by any external metadata\nhandler that wants to record this status information in its metadata.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "6d56e278444bc8323b1bcfcada126b8e4dbba0f4", "tree": "4a1e838b3bb46ce0650f08daa4cdf976eb49317e", "parents": [ "63fe08177f92ce21929df8af6361491b98ad12cd" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 12:01:57 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 12:01:57 2009 +1000" }, "message": "md: allow setting newly added device to \u0027in_sync\u0027 via sysfs.\n\nWhen adding devices to an active array via sysfs, there is currently\nno way to mark a device as \u0027in-sync\u0027 which is useful when\nincrementally assembling an array.\n\nSo add that option.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "63fe08177f92ce21929df8af6361491b98ad12cd", "tree": "7d6f5aa9251f3120e0911483463e14e1137ed81c", "parents": [ "b0cbc861a3c05e634520b049b5cc27ad6febb51f" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Apr 14 12:01:53 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Apr 14 12:01:53 2009 +1000" }, "message": "md: tiny md.h cleanups\n\n- update inclusion guard and make sure it covers the whole file\n - remove superflous #ifdef CONFIG_BLOCK\n - make sure all required headers are included so that new users aren\u0027t\n required to include others before\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "340cd44451fb0bfa542365e6b4b565bbd44836e2", "tree": "bd5e2d1436c91a3365d427552fc5a497234c6b50", "parents": [ "73830857bca6f6c9dbd48e906daea50bea42d676" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:17 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:17 2009 +0100" }, "message": "dm kcopyd: fix callback race\n\nIf the thread calling dm_kcopyd_copy is delayed due to scheduling inside\nsplit_job/segment_complete and the subjobs complete before the loop in\nsplit_job completes, the kcopyd callback could be invoked from the\nthread that called dm_kcopyd_copy instead of the kcopyd workqueue.\n\ndm_kcopyd_copy -\u003e split_job -\u003e segment_complete -\u003e job-\u003efn()\n\nSnapshots depend on the fact that callbacks are called from the singlethreaded\nkcopyd workqueue and expect that there is no racing between individual\ncallbacks. The racing between callbacks can lead to corruption of exception\nstore and it can also mean that exception store callbacks are called twice\nfor the same exception - a likely reason for crashes reported inside\npending_complete() / remove_exception().\n\nThis patch fixes two problems:\n\n1. job-\u003efn being called from the thread that submitted the job (see above).\n\n- Fix: hand over the completion callback to the kcopyd thread.\n\n2. job-\u003efn(read_err, write_err, job-\u003econtext); in segment_complete\nreports the error of the last subjob, not the union of all errors.\n\n- Fix: pass job-\u003ewrite_err to the callback to report all error bits\n (it is done already in run_complete_job)\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "73830857bca6f6c9dbd48e906daea50bea42d676", "tree": "1d68ff2f2da69fea4ba3fe720bbfc32507a5f7e3", "parents": [ "af7e466a1acededbc10beaba9eec8531d561c566" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:16 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:16 2009 +0100" }, "message": "dm kcopyd: prepare for callback race fix\n\nUse a variable in segment_complete() to point to the dm_kcopyd_client\nstruct and only release job-\u003epages in run_complete_job() if any are\ndefined. These changes are needed by the next patch.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "af7e466a1acededbc10beaba9eec8531d561c566", "tree": "1d1d0655e2fc22e5b440202be4050fd083a66c04", "parents": [ "92c639021ca6e962645114f02e356e7feb131d0b" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:16 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:16 2009 +0100" }, "message": "dm: implement basic barrier support\n\nBarriers are submitted to a worker thread that issues them in-order.\n\nThe thread is modified so that when it sees a barrier request it waits\nfor all pending IO before the request then submits the barrier and\nwaits for it. (We must wait, otherwise it could be intermixed with\nfollowing requests.)\n\nErrors from the barrier request are recorded in a per-device barrier_error\nvariable. There may be only one barrier request in progress at once.\n\nFor now, the barrier request is converted to a non-barrier request when\nsending it to the underlying device.\n\nThis patch guarantees correct barrier behavior if the underlying device\ndoesn\u0027t perform write-back caching. The same requirement existed before\nbarriers were supported in dm.\n\nBottom layer barrier support (sending barriers by target drivers) and\nhandling devices with write-back caches will be done in further patches.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "92c639021ca6e962645114f02e356e7feb131d0b", "tree": "eee01e9394b09fb3929fd4f3682815e91b87c764", "parents": [ "3b00b2036fac7a7667d0676a0f80eee575b8c32b" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:15 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:15 2009 +0100" }, "message": "dm: remove dm_request loop\n\nRemove queue_io return value and a loop in dm_request.\n\nIO may be submitted to a worker thread with queue_io(). queue_io() sets\nDMF_QUEUE_IO_TO_THREAD so that all further IO is queued for the thread. When\nthe thread finishes its work, it clears DMF_QUEUE_IO_TO_THREAD and from this\npoint on, requests are submitted from dm_request again. This will be used\nfor processing barriers.\n\nRemove the loop in dm_request. queue_io() can submit I/Os to the worker thread\neven if DMF_QUEUE_IO_TO_THREAD was not set.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "3b00b2036fac7a7667d0676a0f80eee575b8c32b", "tree": "cd893a75900ee81550230711ea4a0005a6f2c2ce", "parents": [ "54d9a1b4513b96cbd835ca6866c6a604d194b2ae" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:15 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:15 2009 +0100" }, "message": "dm: rework queueing and suspension\n\nRework shutting down on suspend and document the associated rules.\n\nDrop write lock in __split_and_process_bio to allow more processing\nconcurrency.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "54d9a1b4513b96cbd835ca6866c6a604d194b2ae", "tree": "42813374e137f5ac6d9d7aedfae02a089b6443a6", "parents": [ "1eb787ec183d1267cac95aae632e92dee05ed50a" ], "author": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:14 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:14 2009 +0100" }, "message": "dm: simplify dm_request loop\n\nRefactor the code in dm_request().\n\nRequire the new DMF_BLOCK_FOR_SUSPEND flag on readahead bios we will\ndiscard so we don\u0027t drop such bios while processing a barrier.\n\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "1eb787ec183d1267cac95aae632e92dee05ed50a", "tree": "ad8f50d995afc72fb4751fdd2436ebc40b648b0f", "parents": [ "df12ee996378a5917e9555169fe278ecca0612d4" ], "author": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:14 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:14 2009 +0100" }, "message": "dm: split DMF_BLOCK_IO flag into two\n\nSplit the DMF_BLOCK_IO flag into two.\n\nDMF_BLOCK_IO_FOR_SUSPEND is set when I/O must be blocked while suspending a\ndevice. DMF_QUEUE_IO_TO_THREAD is set when I/O must be queued to a\nworker thread for later processing.\n\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "df12ee996378a5917e9555169fe278ecca0612d4", "tree": "ea6200205c68aab575b078e3a9e22d9b3466718f", "parents": [ "692d0eb9e02cf81fb387ff891f53840db2f3110a" ], "author": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:13 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:13 2009 +0100" }, "message": "dm: rearrange dm_wq_work\n\nRefactor dm_wq_work() to make later patch more readable.\n\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "692d0eb9e02cf81fb387ff891f53840db2f3110a", "tree": "0e6aa42d81cd47c55de37b55d2ffad419fe57cd0", "parents": [ "9c47008d13add50ec4597a8b9eee200c515282c8" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 09 00:27:13 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:13 2009 +0100" }, "message": "dm: remove limited barrier support\n\nPrepare for full barrier implementation: first remove the restricted support.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "9c47008d13add50ec4597a8b9eee200c515282c8", "tree": "debd8ebb2fef2687d4548a177505ec30bda95503", "parents": [ "577c9c456f0e1371cbade38eaf91ae8e8a308555" ], "author": { "name": "Martin K. Petersen", "email": "martin.petersen@oracle.com", "time": "Thu Apr 09 00:27:12 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 09 00:27:12 2009 +0100" }, "message": "dm: add integrity support\n\nThis patch provides support for data integrity passthrough in the device\nmapper.\n\n - If one or more component devices support integrity an integrity\n profile is preallocated for the DM device.\n\n - If all component devices have compatible profiles the DM device is\n flagged as capable.\n\n - Handle integrity metadata when splitting and cloning bios.\n\nSigned-off-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "91a9e99d76d71b1b16f422fae4bd844245f14638", "tree": "4b38dfbb004bb39a670bebdcb1d99ed452f2de49", "parents": [ "bf538fe4689694b0b90d358dda91cde7b438097d" ], "author": { "name": "Alexander Beregalov", "email": "a.beregalov@gmail.com", "time": "Tue Apr 07 01:35:56 2009 +0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 06 14:40:07 2009 -0700" }, "message": "md/raid1: fix build breakage\n\nFix this build error:\n\n drivers/md/raid1.c: In function \u0027raid1_congested\u0027:\n drivers/md/raid1.c:589: error: \u0027BDI_write_congested\u0027 undeclared\n\nBDI_write_congested was changed in commit 1faa16d228 (\"block: change the\nrequest allocation/congestion logic to be sync/async based\")\n\nSigned-off-by: Alexander Beregalov \u003ca.beregalov@gmail.com\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "303a0e11d0ee136ad8f53f747f3c377daece763b", "tree": "197f4a00ec3eba9e00cdc264c25749914667c389", "parents": [ "6bb597507f9839b13498781e481f5458aea33620" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 06 14:40:38 2009 +1000" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Mon Apr 06 14:40:38 2009 +1000" }, "message": "md/raid1 - don\u0027t assume newly allocated bvecs are initialised.\n\nSince commit d3f761104b097738932afcc310fbbbbfb007ef92\nnewly allocated bvecs aren\u0027t initialised to NULL, so we have\nto be more careful about freeing a bio which only managed\nto get a few pages allocated to it. Otherwise the resync\nprocess crashes.\n\nThis patch is appropriate for 2.6.29-stable.\n\nCc: stable@kernel.org\nCc: \"Jens Axboe\" \u003cjens.axboe@oracle.com\u003e\nReported-by: Gabriele Tozzi \u003cgabriele@tozzi.eu\u003e\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d9b9be024a6628a01d8730d1fd0b5f25658a2794", "tree": "9f8e606f975f6dff4213747e85fedaccd148eb60", "parents": [ "9b59f0316bc556a1b63518f0b1224cf9be48467b", "99360b4c18f7675b50d283301d46d755affe75fd" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 10:02:45 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 10:02:45 2009 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/agk/linux-2.6-dm\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/agk/linux-2.6-dm: (36 commits)\n dm: set queue ordered mode\n dm: move wait queue declaration\n dm: merge pushback and deferred bio lists\n dm: allow uninterruptible wait for pending io\n dm: merge __flush_deferred_io into caller\n dm: move bio_io_error into __split_and_process_bio\n dm: rename __split_bio\n dm: remove unnecessary struct dm_wq_req\n dm: remove unnecessary work queue context field\n dm: remove unnecessary work queue type field\n dm: bio list add bio_list_add_head\n dm snapshot: persistent fix dtr cleanup\n dm snapshot: move status to exception store\n dm snapshot: move ctr parsing to exception store\n dm snapshot: use DMEMIT macro for status\n dm snapshot: remove dm_snap header\n dm snapshot: remove dm_snap header use\n dm exception store: move cow pointer\n dm exception store: move chunk_fields\n dm exception store: move dm_target pointer\n ...\n" }, { "commit": "223cdea4c4b5af5181b2da00ac85711d1e0c737c", "tree": "dfe7226c70ddabbf2e2e63924ba636345278e79c", "parents": [ "31e6e2dac575c9d21a6ec56ca52ae89086baa705", "c8f517c444e4f9f55b5b5ca202b8404691a35805" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 09:08:19 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Apr 03 09:08:19 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://neil.brown.name/md\n\n* \u0027for-linus\u0027 of git://neil.brown.name/md: (53 commits)\n md/raid5 revise rules for when to update metadata during reshape\n md/raid5: minor code cleanups in make_request.\n md: remove CONFIG_MD_RAID_RESHAPE config option.\n md/raid5: be more careful about write ordering when reshaping.\n md: don\u0027t display meaningless values in sysfs files resync_start and sync_speed\n md/raid5: allow layout and chunksize to be changed on active array.\n md/raid5: reshape using largest of old and new chunk size\n md/raid5: prepare for allowing reshape to change layout\n md/raid5: prepare for allowing reshape to change chunksize.\n md/raid5: clearly differentiate \u0027before\u0027 and \u0027after\u0027 stripes during reshape.\n Documentation/md.txt update\n md: allow number of drives in raid5 to be reduced\n md/raid5: change reshape-progress measurement to cope with reshaping backwards.\n md: add explicit method to signal the end of a reshape.\n md/raid5: enhance raid5_size to work correctly with negative delta_disks\n md/raid5: drop qd_idx from r6_state\n md/raid6: move raid6 data processing to raid6_pq.ko\n md: raid5 run(): Fix max_degraded for raid level 4.\n md: \u0027array_size\u0027 sysfs attribute\n md: centralize -\u003earray_sectors modifications\n ...\n" }, { "commit": "99360b4c18f7675b50d283301d46d755affe75fd", "tree": "1ad78057b086aa1571cc8dcf2443c819f75ca30a", "parents": [ "b44ebeb017b8a5fe5439e1259708b68cf83a8921" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "message": "dm: set queue ordered mode\n\nSet queue ordered mode. It doesn\u0027t really matter what we set here\nbecause we don\u0027t ever put any requests on the queue. But we need to set\nsomething other than QUEUE_ORDERED_NONE so that __generic_make_request\npasses barrier requests to us.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "b44ebeb017b8a5fe5439e1259708b68cf83a8921", "tree": "ea138d44aa80302002b3e67cd6049a18c2302734", "parents": [ "022c261100e15652d720395b17ce76304fb2f97f" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "message": "dm: move wait queue declaration\n\nMove wait queue declaration and unplug to dm_wait_for_completion.\n\nThe purpose is to minimize duplicate code in the further patches.\n\nThe patch reorders functions a little bit. It doesn\u0027t change any\nfunctionality. For proper non-deadlock operation, add_wait_queue must\nhappen before set_current_state(interruptible) and before the test for\n!atomic_read(\u0026md-\u003epending).\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "022c261100e15652d720395b17ce76304fb2f97f", "tree": "cbab417b11242d4561ba45e60b3838f1672013cb", "parents": [ "401600dfd368305e641d79db16d514f55c084544" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:39 2009 +0100" }, "message": "dm: merge pushback and deferred bio lists\n\nMerge pushback and deferred lists into one list - use deferred list\nfor both deferred and pushed-back bios.\n\nThis will be needed for proper support of barrier bios: it is impossible to\nsupport ordering correctly with two lists because the requests on both lists\nwill be mixed up.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "401600dfd368305e641d79db16d514f55c084544", "tree": "4e1c3f19c52d2bb7d78ec01689d17659960732ab", "parents": [ "ef2085870ea448b3c19160d899cf4f948da6a384" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "message": "dm: allow uninterruptible wait for pending io\n\nAllow uninterruptible wait for pending IOs.\n\nAdd argument \"interruptible\" to dm_wait_for_completion that specifies\neither interruptible or uninterruptible waiting.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "ef2085870ea448b3c19160d899cf4f948da6a384", "tree": "f8e6e4bdeecc41d1b0cd8608a7460b0d130f43a8", "parents": [ "f0b9a4502baa18f8a255a2866bb4e0655fb35974" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "message": "dm: merge __flush_deferred_io into caller\n\nMerge __flush_deferred_io() into the only caller, dm_wq_work().\n\nThere\u0027s no need to have a function that has only one caller.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "f0b9a4502baa18f8a255a2866bb4e0655fb35974", "tree": "28d45275783b68d608d632a84371aa6e625c08f5", "parents": [ "8a53c28db42853591edbe8103e2ce3c4f2917f42" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:38 2009 +0100" }, "message": "dm: move bio_io_error into __split_and_process_bio\n\nMove the bio_io_error() calls directly into __split_and_process_bio().\n\nThis avoids some code duplication in later patches.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "8a53c28db42853591edbe8103e2ce3c4f2917f42", "tree": "fe09d07b660847c37fa8cd6ee67ee043f4302ba4", "parents": [ "53d5914f288b67ddc4d594d6a09568fe114bb909" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:37 2009 +0100" }, "message": "dm: rename __split_bio\n\nRename __split_bio() to __split_and_process_bio() because it not only splits\nthe bio to serveral parts, but also submits them to target drivers.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "53d5914f288b67ddc4d594d6a09568fe114bb909", "tree": "e4f8e2442b85564a3f8f90f32228d5efeb3bd033", "parents": [ "9a1fb46448cac50e93115322ad28f417936f7852" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:37 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:37 2009 +0100" }, "message": "dm: remove unnecessary struct dm_wq_req\n\nRemove struct dm_wq_req and move \"work\" directly into struct mapped_device.\n\nIn the revised implementation, the thread will do just one type of work\n(processing the queue).\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "9a1fb46448cac50e93115322ad28f417936f7852", "tree": "120e3e431dd98044a7620e4095da4bcc37ae4407", "parents": [ "143773965b4677bd72dbbf71f52bea0df2ed4e18" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "message": "dm: remove unnecessary work queue context field\n\nRemove the context field from struct dm_wq_req because we will no longer\nneed it.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "143773965b4677bd72dbbf71f52bea0df2ed4e18", "tree": "42cbaca9b881d1e84d555d9ced120f4e78babc9a", "parents": [ "99c75e3130cff0c869c657ee97fb4a68958eeffc" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "message": "dm: remove unnecessary work queue type field\n\nRemove \"type\" field from struct dm_wq_req because we no longer need it\nto have more than one value.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "99c75e3130cff0c869c657ee97fb4a68958eeffc", "tree": "cd3aaee7a0aeab15a3cc345046ef2020e62f21cd", "parents": [ "a32079ce17899192a7b98a42753be467605a8b2f" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:36 2009 +0100" }, "message": "dm: bio list add bio_list_add_head\n\nIntroduce a function that adds a bio to the head of the list for\nuse by the patch that will support barriers.\n\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a32079ce17899192a7b98a42753be467605a8b2f", "tree": "116fefba16427c16cb792e218889aa971eab755b", "parents": [ "1e302a929e2da6e8448e2058e4b07b07252b57fe" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:35 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:35 2009 +0100" }, "message": "dm snapshot: persistent fix dtr cleanup\n\nThe persistent exception store destructor does not properly\naccount for all conditions in which it can be called. If it\nis called after \u0027ctr\u0027 but before \u0027read_metadata\u0027 (e.g. if\nsomething else in \u0027snapshot_ctr\u0027 fails) then it will attempt\nto free areas of memory that haven\u0027t been allocated yet.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "1e302a929e2da6e8448e2058e4b07b07252b57fe", "tree": "3345d31926a2254a2041d62d270cba76b2590bae", "parents": [ "fee1998e9c690f9920671e1e0ef187a48cfbbde4" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:35 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:35 2009 +0100" }, "message": "dm snapshot: move status to exception store\n\nLet the exception store types print out their status through\nthe new API, rather than having the snapshot code do it.\n\nAdjust the buffer position to allow for the preceding DMEMIT in the\narguments to type-\u003estatus().\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "fee1998e9c690f9920671e1e0ef187a48cfbbde4", "tree": "a217e9bc2142350d9e8065c6e581be0524636498", "parents": [ "2e4a31df2b10cbcaf43c333112f6f7440a035c69" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "message": "dm snapshot: move ctr parsing to exception store\n\nFirst step of having the exception stores parse their own arguments -\ngeneralizing the interface.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "2e4a31df2b10cbcaf43c333112f6f7440a035c69", "tree": "fbfcde3d358e615b500c22219190654825e48a26", "parents": [ "ccc45ea8aeffec49fa5985efc3649aa67bb4fcb7" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "message": "dm snapshot: use DMEMIT macro for status\n\nUse DMEMIT in place of snprintf. This makes it easier later when\nother modules are helping to populate our status output.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "ccc45ea8aeffec49fa5985efc3649aa67bb4fcb7", "tree": "69a72e1a1599d3495224dcd425f0bb230a48108e", "parents": [ "71fab00a6bef7fb53119271a8abdbaf40970d28a" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:34 2009 +0100" }, "message": "dm snapshot: remove dm_snap header\n\nMove some of the last bits from dm-snap.h into dm-snap.c where they\nbelong and remove dm-snap.h.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "71fab00a6bef7fb53119271a8abdbaf40970d28a", "tree": "4881f8f6d1b1c42528889d7a56fb8674d2aa7723", "parents": [ "49beb2b87a972a994ff77633234ca3bf0d30a1d8" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:33 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:33 2009 +0100" }, "message": "dm snapshot: remove dm_snap header use\n\nMove useful functions out of dm-snap.h and stop using dm-snap.h.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "49beb2b87a972a994ff77633234ca3bf0d30a1d8", "tree": "4b281ebd48624d0fbe74b93fa72baa834211ca2f", "parents": [ "d0216849519bec8dc96301a3cd80316e71243839" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:33 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:33 2009 +0100" }, "message": "dm exception store: move cow pointer\n\nMove COW device from snapshot to exception store.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "d0216849519bec8dc96301a3cd80316e71243839", "tree": "06e7f1d2028e784213e17f1a8ea1abae9ffb0ab9", "parents": [ "0cea9c78270cdf1d2ad74ce0e083d5555a0842e8" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:32 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:32 2009 +0100" }, "message": "dm exception store: move chunk_fields\n\nMove chunk fields from snapshot to exception store.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "0cea9c78270cdf1d2ad74ce0e083d5555a0842e8", "tree": "783785811de9ed558ec1179a6bc201dd6161cae6", "parents": [ "493df71c6420b211a68ae82b889c1e8a5fe701be" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:32 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:32 2009 +0100" }, "message": "dm exception store: move dm_target pointer\n\nMove target pointer from snapshot to exception store.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "493df71c6420b211a68ae82b889c1e8a5fe701be", "tree": "2738295190c1b3c5f72d85059aae3c96f162afa6", "parents": [ "7513c2a761d69d2a93f17146b3563527d3618ba0" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:31 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:31 2009 +0100" }, "message": "dm exception store: introduce registry\n\nMove exception stores into a registry.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "7513c2a761d69d2a93f17146b3563527d3618ba0", "tree": "375a14998a60a47b83962d5a497139112ec6e8e4", "parents": [ "b2a114652940ccf7e9668ad447ca78bf16a31139" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "message": "dm raid1: add is_remote_recovering hook for clusters\n\nThe logging API needs an extra function to make cluster mirroring\npossible. This new function allows us to check whether a mirror\nregion is being recovered on another machine in the cluster. This\nhelps us prevent simultaneous recovery I/O and process I/O to the\nsame locations on disk.\n\nCluster-aware log modules will implement this function. Single\nmachine log modules will not. So, there is no performance\npenalty for single machine mirrors.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nAcked-by: Heinz Mauelshagen \u003cheinzm@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "b2a114652940ccf7e9668ad447ca78bf16a31139", "tree": "f86e851ff752a8ae5ce1b5571cde1918c86a156a", "parents": [ "ec44ab9d6681ddf9026b593e866bec9c0e075e1d" ], "author": { "name": "Jonathan Brassow", "email": "jbrassow@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "message": "dm exception store: separate type from instance\n\nIntroduce struct dm_exception_store_type.\n\nSigned-off-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "ec44ab9d6681ddf9026b593e866bec9c0e075e1d", "tree": "56c0f99068c7bf2a474e8094c1fa9d14a7861ae8", "parents": [ "84e67c9319eb2232757a022c24f6a461291eaee5" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:30 2009 +0100" }, "message": "dm log: remove struct dm_dirty_log_internal\n\nRemove the \u0027dm_dirty_log_internal\u0027 structure. The resulting cleanup\neliminates extra memory allocations. Therefore exposing the internal\nlist_head to the external \u0027dm_dirty_log_type\u0027 structure is a worthwhile\ncompromise.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "84e67c9319eb2232757a022c24f6a461291eaee5", "tree": "581e427fa8bc914089de24f70235e471c5bd0c5b", "parents": [ "b81d6cf79b57f6123a5d4f7a4932cc359995484d" ], "author": { "name": "Mike Snitzer", "email": "snitzer@redhat.com", "time": "Thu Apr 02 19:55:29 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:29 2009 +0100" }, "message": "dm log: use standard kernel module refcount\n\nAvoid private module usage accounting by removing \u0027use\u0027 from\ndm_dirty_log_internal. The standard module reference counting is\nsufficient.\n\nSigned-off-by: Mike Snitzer \u003csnitzer@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "b81d6cf79b57f6123a5d4f7a4932cc359995484d", "tree": "ac3df19b3800c4ecb9e5725ada90c606d542dfbb", "parents": [ "45194e4f89fbdd97a2b7d2698c05f0b00c19e820" ], "author": { "name": "Johannes Weiner", "email": "hannes@cmpxchg.org", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "message": "dm crypt: use kzfree\n\nUse kzfree() instead of memset() + kfree().\n\nSigned-off-by: Johannes Weiner \u003channes@cmpxchg.org\u003e\nReviewed-by: Pekka Enberg \u003cpenberg@cs.helsinki.fi\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "45194e4f89fbdd97a2b7d2698c05f0b00c19e820", "tree": "70d35747228b84c1d7b7d76c5e18f72b16f412a3", "parents": [ "570b9d968bf9b16974252ef7cbce73fa6dac34f3" ], "author": { "name": "Cheng Renquan", "email": "crquan@gmail.com", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "message": "dm target: remove struct tt_internal\n\nThe tt_internal is really just a list_head to manage registered target_type\nin a double linked list,\n\nHere embed the list_head into target_type directly,\n1. to avoid kmalloc/kfree;\n2. then tt_internal is really unneeded;\n\nCc: stable@kernel.org\nSigned-off-by: Cheng Renquan \u003ccrquan@gmail.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nReviewed-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "570b9d968bf9b16974252ef7cbce73fa6dac34f3", "tree": "1a356ada71b373b363f9fc2204769f8eee920f30", "parents": [ "aea9058801c0acfa2831af1714da412dfb0018c2" ], "author": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:28 2009 +0100" }, "message": "dm table: fix upgrade mode race\n\nupgrade_mode() sets bdev to NULL temporarily, and does not have any\nlocking to exclude anything from seeing that NULL.\n\nIn dm_table_any_congested() bdev_get_queue() can dereference that NULL and\ncause a reported oops.\n\nFix this by not changing that field during the mode upgrade.\n\nCc: stable@kernel.org\nCc: Neil Brown \u003cneilb@suse.de\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "aea9058801c0acfa2831af1714da412dfb0018c2", "tree": "3a51498e150b84ef60580f1d2f5620785853a41a", "parents": [ "5642b8a61a15436231adf27b2b1bd96901b623dd" ], "author": { "name": "Jun\u0027ichi Nomura", "email": "j-nomura@ce.jp.nec.com", "time": "Thu Apr 02 19:55:27 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:27 2009 +0100" }, "message": "dm: path selector use module refcount directly\n\nFix refcount corruption in dm-path-selector\n\nRefcounting with non-atomic ops under shared lock will corrupt the counter\nin multi-processor system and may trigger BUG_ON().\nUse module refcount.\n# same approach as dm-target-use-module-refcount-directly.patch here\n# https://www.redhat.com/archives/dm-devel/2008-December/msg00075.html\n\nTypical oops:\n kernel BUG at linux-2.6.29-rc3/drivers/md/dm-path-selector.c:90!\n Pid: 11148, comm: dmsetup Not tainted 2.6.29-rc3-nm #1\n dm_put_path_selector+0x4d/0x61 [dm_multipath]\n Call Trace:\n [\u003cffffffffa031d3f9\u003e] free_priority_group+0x33/0xb3 [dm_multipath]\n [\u003cffffffffa031d4aa\u003e] free_multipath+0x31/0x67 [dm_multipath]\n [\u003cffffffffa031d50d\u003e] multipath_dtr+0x2d/0x32 [dm_multipath]\n [\u003cffffffffa015d6c2\u003e] dm_table_destroy+0x64/0xd8 [dm_mod]\n [\u003cffffffffa015b73a\u003e] __unbind+0x46/0x4b [dm_mod]\n [\u003cffffffffa015b79f\u003e] dm_swap_table+0x60/0x14d [dm_mod]\n [\u003cffffffffa015f963\u003e] dev_suspend+0xfd/0x177 [dm_mod]\n [\u003cffffffffa0160250\u003e] dm_ctl_ioctl+0x24c/0x29c [dm_mod]\n [\u003cffffffff80288cd3\u003e] ? get_page_from_freelist+0x49c/0x61d\n [\u003cffffffffa015f866\u003e] ? dev_suspend+0x0/0x177 [dm_mod]\n [\u003cffffffff802bf05c\u003e] vfs_ioctl+0x2a/0x77\n [\u003cffffffff802bf4f1\u003e] do_vfs_ioctl+0x448/0x4a0\n [\u003cffffffff802bf5a0\u003e] sys_ioctl+0x57/0x7a\n [\u003cffffffff8020c05b\u003e] system_call_fastpath+0x16/0x1b\n\nCc: stable@kernel.org\nSigned-off-by: Jun\u0027ichi Nomura \u003cj-nomura@ce.jp.nec.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "5642b8a61a15436231adf27b2b1bd96901b623dd", "tree": "2108366dbec955f71bc49795e3cf894d4dcd991c", "parents": [ "35bf659b008e83e725dcd30f542e38461dbb867c" ], "author": { "name": "Cheng Renquan", "email": "crquan@gmail.com", "time": "Thu Apr 02 19:55:27 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:27 2009 +0100" }, "message": "dm target: use module refcount directly\n\nThe tt_internal\u0027s \u0027use\u0027 field is superfluous: the module\u0027s refcount can do\nthe work properly. An acceptable side-effect is that this increases the\nreference counts reported by \u0027lsmod\u0027.\n\nRemove the superfluous test when removing a target module.\n\n[Crash possible without this on SMP - agk]\n\nCc: stable@kernel.org\nSigned-off-by: Cheng Renquan \u003ccrquan@gmail.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nReviewed-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\nReviewed-by: Jonathan Brassow \u003cjbrassow@redhat.com\u003e\n" }, { "commit": "35bf659b008e83e725dcd30f542e38461dbb867c", "tree": "40be2f357f0905573bd4d21ff59e0db3cda26442", "parents": [ "c66213921c816f6b1b16a84911618ba9a363b134" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:26 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:26 2009 +0100" }, "message": "dm snapshot: avoid having two exceptions for the same chunk\n\nWe need to check if the exception was completed after dropping the lock.\n\nAfter regaining the lock, __find_pending_exception checks if the exception\nwas already placed into \u0026s-\u003epending hash.\n\nBut we don\u0027t check if the exception was already completed and placed into\n\u0026s-\u003ecomplete hash. If the process waiting in alloc_pending_exception was\ndelayed at this point because of a scheduling latency and the exception\nwas meanwhile completed, we\u0027d miss that and allocate another pending\nexception for already completed chunk.\n\nIt would lead to a situation where two records for the same chunk exist\nand potential data corruption because multiple snapshot I/Os to the\naffected chunk could be redirected to different locations in the\nsnapshot.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "c66213921c816f6b1b16a84911618ba9a363b134", "tree": "2b99c1ef34eea840fe3d517d8a20fb59a5a84044", "parents": [ "2913808eb56a6445a7b277eb8d17651c8defb035" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:25 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:25 2009 +0100" }, "message": "dm snapshot: avoid dropping lock in __find_pending_exception\n\nIt is uncommon and bug-prone to drop a lock in a function that is called with\nthe lock held, so this is moved to the caller.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "2913808eb56a6445a7b277eb8d17651c8defb035", "tree": "a6d5392acc09cba9658071ca80a81b5a3a725a6c", "parents": [ "b64b6bf4fd8b678a9f8477c11773c38a0a246a6d" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:25 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:25 2009 +0100" }, "message": "dm snapshot: refactor __find_pending_exception\n\nMove looking-up of a pending exception from __find_pending_exception to another\nfunction.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "b64b6bf4fd8b678a9f8477c11773c38a0a246a6d", "tree": "26e12749b51ce21f0f59b8d7ee45a3716d2a96d8", "parents": [ "95f8fac8dc6139fedfb87746e0c8fda9b803cb46" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:24 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:24 2009 +0100" }, "message": "dm io: make sync_io uninterruptible\n\nIf someone sends signal to a process performing synchronous dm-io call,\nthe kernel may crash.\n\nThe function sync_io attempts to exit with -EINTR if it has pending signal,\nhowever the structure \"io\" is allocated on stack, so already submitted io\nrequests end up touching unallocated stack space and corrupting kernel memory.\n\nsync_io sets its state to TASK_UNINTERRUPTIBLE, so the signal can\u0027t break out\nof io_schedule() --- however, if the signal was pending before sync_io entered\nwhile (1) loop, the corruption of kernel memory will happen.\n\nThere is no way to cancel in-progress IOs, so the best solution is to ignore\nsignals at this point.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "95f8fac8dc6139fedfb87746e0c8fda9b803cb46", "tree": "36c980967ee188865d0a58be2e1281b5bf77cbae", "parents": [ "a920f6b3accc77d9dddbc98a7426be23ee479625" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:24 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:24 2009 +0100" }, "message": "dm raid1: switch read_record from kmalloc to slab to save memory\n\nWith my previous patch to save bi_io_vec, the size of dm_raid1_read_record\nis significantly increased (the vector list takes 3072 bytes on 32-bit machines\nand 4096 bytes on 64-bit machines).\n\nThe structure dm_raid1_read_record used to be allocated with kmalloc,\nbut kmalloc aligns the size on the next power-of-two so an object\nslightly greater than 4096 will allocate 8192 bytes of memory and half of\nthat memory will be wasted.\n\nThis patch turns kmalloc into a slab cache which doesn\u0027t have this\npadding so it will reduce the memory consumed.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "a920f6b3accc77d9dddbc98a7426be23ee479625", "tree": "bff633328b5f1362d99a04f7e513a6a4b2084ad8", "parents": [ "833bb3046b6cb320e775ea2160ddca87d53260d5" ], "author": { "name": "Mikulas Patocka", "email": "mpatocka@redhat.com", "time": "Thu Apr 02 19:55:23 2009 +0100" }, "committer": { "name": "Alasdair G Kergon", "email": "agk@redhat.com", "time": "Thu Apr 02 19:55:23 2009 +0100" }, "message": "dm: preserve bi_io_vec when resubmitting bios\n\nDevice mapper saves and restores various fields in the bio, but it doesn\u0027t save\nbi_io_vec. If the device driver modifies this after a partially successful\nrequest, dm-raid1 and dm-multipath may attempt to resubmit a bio that has\nbi_size inconsistent with the size of vector.\n\nTo make requests resubmittable in dm-raid1 and dm-multipath, we must save\nand restore the bio vector as well.\n\nTo reduce the memory overhead involved in this, we do not save the pages in a\nvector and use a 16-bit field size if the page size is less than 65536.\n\nCc: stable@kernel.org\nSigned-off-by: Mikulas Patocka \u003cmpatocka@redhat.com\u003e\nSigned-off-by: Alasdair G Kergon \u003cagk@redhat.com\u003e\n" }, { "commit": "c8f517c444e4f9f55b5b5ca202b8404691a35805", "tree": "e679ae13a07e1a2644da0dfc4a4d66bf73f83626", "parents": [ "b0f9ec047b79a92e8b8a9dfbf97537c8fbef234a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:28:40 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:28:40 2009 +1100" }, "message": "md/raid5 revise rules for when to update metadata during reshape\n\nWe currently update the metadata :\n 1/ every 3Megabytes\n 2/ When the place we will write new-layout data to is recorded in\n the metadata as still containing old-layout data.\n\nRule one exists to avoid having to re-do too much reshaping in the\nface of a crash/restart. So it should really be time based rather\nthan size based. So change it to \"every 10 seconds\".\n\nRule two turns out to be too harsh when restriping an array\n\u0027in-place\u0027, as in that case the metadata much be updates for every\nstripe.\nFor the in-place update, it can only possibly be safe from a crash if\nsome user-space program data a backup of every e.g. few hundred\nstripes before allowing them to be reshaped. In that case, the\nconstant metadata update is pointless.\nSo only update the metadata if the new metadata will report that the\nend of the \u0027old-layout\u0027 data is beyond where we are currently\nwriting \u0027new-layout\u0027 data.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "b0f9ec047b79a92e8b8a9dfbf97537c8fbef234a", "tree": "2761193e869ed51d08674c39ed3df105cd91f4de", "parents": [ "2cffc4a01dd90a502324e3453d7b245d6d16e1c2" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:27:18 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:27:18 2009 +1100" }, "message": "md/raid5: minor code cleanups in make_request.\n\n... and to be certain the that make_request doesn\u0027t wait forever,\nadd a \u0027wake_up\u0027 when -\u003ereshape_progress has been set to MaxSector\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "2cffc4a01dd90a502324e3453d7b245d6d16e1c2", "tree": "bcaf6e9fb8169c4e863173d33624de4c9a181246", "parents": [ "ab69ae12ceef7f23c578a3c230144e94a167a821" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:27:05 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:27:05 2009 +1100" }, "message": "md: remove CONFIG_MD_RAID_RESHAPE config option.\n\nThis was only needed when the code was experimental. Most of it\nis well tested now, so the option is no longer useful.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ab69ae12ceef7f23c578a3c230144e94a167a821", "tree": "01bee8e4c4a91a1714596c4f85769bfe72e328ce", "parents": [ "d1a7c50369835f9ecbd7752016cd9302ecfae678" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:26:47 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:26:47 2009 +1100" }, "message": "md/raid5: be more careful about write ordering when reshaping.\n\nWhen we are reshaping an array, it is very important that we read\nthe data from a particular sector offset before writing new data\nat that offset.\n\nIn most cases when growing or shrinking an array we read long before\nwe even consider writing. But when restriping an array without\nchanging it size, there is a small possibility that we might have\nsome data to available write before the read has happened at the same\nlocation. This would require some stripes to be in cache already.\n\nTo guard against this small possibility, we check, before writing,\nthat the \u0027old\u0027 stripe at the same location is not in the process of\nbeing read. And we ensure that we mark all \u0027source\u0027 stripes as such\nbefore allowing new \u0027destination\u0027 stripes to proceed.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "d1a7c50369835f9ecbd7752016cd9302ecfae678", "tree": "2c6bcc8c5cb4d045d555e0d030321cb887a4e05a", "parents": [ "88ce4930e2b80378d45506ce2c3bb5820e156e85" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:32 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:32 2009 +1100" }, "message": "md: don\u0027t display meaningless values in sysfs files resync_start and sync_speed\n\nWhen no resync if happening, both of these files currently have\nmeaningless values (is slightly different ways).\nChange them to \"none\" in that case.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "88ce4930e2b80378d45506ce2c3bb5820e156e85", "tree": "75847293ede7cd396b5470c0d1c4f50dc79ff031", "parents": [ "7a6613810785872b7c028fba22fc0bae1c91733d" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:23 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:24:23 2009 +1100" }, "message": "md/raid5: allow layout and chunksize to be changed on active array.\n\nIf an array has 3 or more devices, we allow the chunksize or layout\nto be changed and when a reshape starts, we use these as the \u0027new\u0027\nvalues.\n\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7a6613810785872b7c028fba22fc0bae1c91733d", "tree": "b716e9278d42e83c82888d82671062d64fbf0bea", "parents": [ "e183eaedd53807e33f02ee80573e2833890e1f21" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:21:40 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:21:40 2009 +1100" }, "message": "md/raid5: reshape using largest of old and new chunk size\n\nThis ensures that even when old and new stripes are overlapping,\nwe will try to read all of the old before having to write any\nof the new.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "e183eaedd53807e33f02ee80573e2833890e1f21", "tree": "ba648feacbc930936e5a66006a19ee8822ac28f5", "parents": [ "784052ecc6ade6b6acf4f67e4ada8e5f2e6df446" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:20:22 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:20:22 2009 +1100" }, "message": "md/raid5: prepare for allowing reshape to change layout\n\nAdd prev_algo to raid5_conf_t along the same lines as prev_chunk\nand previous_raid_disks.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "784052ecc6ade6b6acf4f67e4ada8e5f2e6df446", "tree": "f0b1060215922ba30a4d676a00dbdad8ddb724c9", "parents": [ "86b42c713be3e5f6807aa14b4cbdb005d35c64d5" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:19:07 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:19:07 2009 +1100" }, "message": "md/raid5: prepare for allowing reshape to change chunksize.\n\nAdd \"prev_chunk\" to raid5_conf_t, similar to \"previous_raid_disks\", to\nremember what the chunk size was before the reshape that is currently\nunderway.\n\nThis seems like duplication with \"chunk_size\" and \"new_chunk\" in\nmddev_t, and to some extent it is, but there are differences.\nThe values in mddev_t are always defined and often the same.\nThe prev* values are only defined if a reshape is underway.\n\nAlso (and more significantly) the raid5_conf_t values will be changed\nat the same time (inside an appropriate lock) that the reshape is\nstarted by setting reshape_position. In contrast, the new_chunk value\nis set when the sysfs file is written which could be well before the\nreshape starts.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "86b42c713be3e5f6807aa14b4cbdb005d35c64d5", "tree": "d6c70a966a6bda99e6cd4ce055efa84ab0ded701", "parents": [ "11373542344bdc35be1e6e68b0baadd1b6f7acbb" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:19:03 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:19:03 2009 +1100" }, "message": "md/raid5: clearly differentiate \u0027before\u0027 and \u0027after\u0027 stripes during reshape.\n\nDuring a raid5 reshape, we have some stripes in the cache that are\n\u0027before\u0027 the reshape (and are still to be processed) and some that are\n\u0027after\u0027. They are currently differentiated by having different\n-\u003edisks values as the only reshape current supported involves changing\nthe number of disks.\n\nHowever we will soon support reshapes that do not change the number\nof disks (chunk parity or chunk size). So make the difference more\nexplicit with a \u0027generation\u0027 number.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "ec32a2bd35bd6b933a5db6542c48210ce069a376", "tree": "32d30344202025cf1c6cd202394e65ff7e99753f", "parents": [ "fef9c61fdfabf97a307c2cf3621a6949f0a4b995" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:17:38 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:17:38 2009 +1100" }, "message": "md: allow number of drives in raid5 to be reduced\n\nWhen reshaping a raid5 to have fewer devices, we work from the end of\nthe array to the beginning.\nmd_do_sync gives addresses to sync_request that go from the beginning\nto the end. So largely ignore them use the internal state variable\n\"reshape_progress\" to keep track of what to do next.\n\nNever allow the size to be reduced below the minimum (4 for raid6,\n3 otherwise).\n\nWe require that the size of the array has already been reduced before\nthe array is reshaped to a smaller size. This is because simply\nreducing the size is an easily reversible operation, while the reshape\nis immediately destructive and so is not reversible for the blocks at\nthe ends of the devices.\nThus to reshape an array to have fewer devices, you must first write\nan appropriately small size to md/array_size.\n\nWhen reshape finished, we remove any drives that are no longer\nneeded and fix up -\u003edegraded.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "fef9c61fdfabf97a307c2cf3621a6949f0a4b995", "tree": "82b128341c12205db62fe092d692d32103a7ea9f", "parents": [ "cea9c22800773cecb1d41f4a6139f9eb6a95368b" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:16:46 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:16:46 2009 +1100" }, "message": "md/raid5: change reshape-progress measurement to cope with reshaping backwards.\n\nWhen reducing the number of devices in a raid4/5/6, the reshape\nprocess has to start at the end of the array and work down to the\nbeginning. So we need to handle expand_progress and expand_lo\ndifferently.\n\nThis patch renames \"expand_progress\" and \"expand_lo\" to avoid the\nimplication that anything is getting bigger (expand-\u003ereshape) and\nevery place they are used, we make sure that they are used the right\nway depending on whether delta_disks is positive or negative.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "cea9c22800773cecb1d41f4a6139f9eb6a95368b", "tree": "d070d22390b86768fefb8ba04c64d74993ed28e9", "parents": [ "7ec0547838976d088dfb9cb0adb073e6e8a15aa3" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:15:05 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:15:05 2009 +1100" }, "message": "md: add explicit method to signal the end of a reshape.\n\nCurrently raid5 (the only module that supports restriping)\nnotices that the reshape has finished be sync_request being\ngiven a large value, and handles any cleanup them.\n\nThis patch changes it so md_check_recovery calls into an\nexplicit finish_reshape method as well.\n\nThe clean-up from sync_request can do things that need to be\ndone promptly, typically things local to the raid5_conf_t\nstructure.\n\nThe \"finish_reshape\" method is called under the mddev_lock\nso it can do things involving reconfiguring the device.\n\nThis allows us to get rid of md_set_array_sectors_locked, which\nwould have caused a deadlock if you tried to stop and array\nwhile a reshape was happening.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "7ec0547838976d088dfb9cb0adb073e6e8a15aa3", "tree": "32f2d5aa5d112e99bebbfb9bfde49ba82b66809d", "parents": [ "34e04e87fb8b2c62c9e8868f41c8179d0e15f51a" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:10:36 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:10:36 2009 +1100" }, "message": "md/raid5: enhance raid5_size to work correctly with negative delta_disks\n\nThis is the first of four patches which combine to allow md/raid5 to\nreduce the number of devices in the array by restriping the data over\na subset of the devices.\n\nIf the number of disks in a raid4/5/6 is being reduced, then the\ndefault size must be based on the new number, not the old number\nof devices.\nIn general, it should be based on the smaller of new and old.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" }, { "commit": "34e04e87fb8b2c62c9e8868f41c8179d0e15f51a", "tree": "19d62e6b77f6f786902cc5ab5f8f068fb889efe1", "parents": [ "f701d589aa34d7531183c9ac6f7713ba14212b02" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:10:16 2009 +1100" }, "committer": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Tue Mar 31 15:10:16 2009 +1100" }, "message": "md/raid5: drop qd_idx from r6_state\n\nWe now have this value in stripe_head so we don\u0027t need to duplicate\nit.\n\nSigned-off-by: NeilBrown \u003cneilb@suse.de\u003e\n" } ], "next": "f701d589aa34d7531183c9ac6f7713ba14212b02" }