)]}' { "log": [ { "commit": "5f248c9c251c60af3403902b26e08de43964ea0b", "tree": "6d3328e72a7e4015a64017eb30be18095c6a3c64", "parents": [ "f6cec0ae58c17522a7bc4e2f39dae19f199ab534", "dca332528bc69e05f67161e1ed59929633d5e63d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 10 11:26:52 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 10 11:26:52 2010 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6: (96 commits)\n no need for list_for_each_entry_safe()/resetting with superblock list\n Fix sget() race with failing mount\n vfs: don\u0027t hold s_umount over close_bdev_exclusive() call\n sysv: do not mark superblock dirty on remount\n sysv: do not mark superblock dirty on mount\n btrfs: remove junk sb_dirt change\n BFS: clean up the superblock usage\n AFFS: wait for sb synchronization when needed\n AFFS: clean up dirty flag usage\n cifs: truncate fallout\n mbcache: fix shrinker function return value\n mbcache: Remove unused features\n add f_flags to struct statfs(64)\n pass a struct path to vfs_statfs\n update VFS documentation for method changes.\n All filesystems that need invalidate_inode_buffers() are doing that explicitly\n convert remaining -\u003eclear_inode() to -\u003eevict_inode()\n Make -\u003edrop_inode() just return whether inode needs to be dropped\n fs/inode.c:clear_inode() is gone\n fs/inode.c:evict() doesn\u0027t care about delete vs. non-delete paths now\n ...\n\nFix up trivial conflicts in fs/nilfs2/super.c\n" }, { "commit": "0930fcc1ee2f0a810b938bc283a3a262d7adccbb", "tree": "5e5d10894f1e31a7aede75110b43d7a41347631d", "parents": [ "7da08fd17a6e42d80f0f3897a5cbd682e77bcdb4" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jun 07 13:16:22 2010 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Aug 09 16:48:30 2010 -0400" }, "message": "convert ext4 to -\u003eevict_inode()\n\npretty much brute-force...\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "1025774ce411f2bd4b059ad7b53f0003569b74fa", "tree": "2be221c205cb5579652a6063e8ee27d1c72d1bbd", "parents": [ "eef2380c187890816b73b1a4cb89a09203759469" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Jun 04 11:30:02 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Aug 09 16:47:37 2010 -0400" }, "message": "remove inode_setattr\n\nReplace inode_setattr with opencoded variants of it in all callers. This\nmoves the remaining call to vmtruncate into the filesystem methods where it\ncan be replaced with the proper truncate sequence.\n\nIn a few cases it was obvious that we would never end up calling vmtruncate\nso it was left out in the opencoded variant:\n\n spufs: explicitly checks for ATTR_SIZE earlier\n btrfs,hugetlbfs,logfs,dlmfs: explicitly clears ATTR_SIZE earlier\n ufs: contains an opencoded simple_seattr + truncate that sets the filesize just above\n\nIn addition to that ncpfs called inode_setattr with handcrafted iattrs,\nwhich allowed to trim down the opencoded variant.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6e1db88d536adcbbfe562b2d4b7d6425784fff12", "tree": "8cfcb5a6190722db6249b2e4978f39247975abcf", "parents": [ "f4e420dc423148fba637af1ab618fa8896dfb2d6" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Jun 04 11:29:57 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Aug 09 16:47:32 2010 -0400" }, "message": "introduce __block_write_begin\n\nSplit up the block_write_begin implementation - __block_write_begin is a new\ntrivial wrapper for block_prepare_write that always takes an already\nallocated page and can be either called from block_write_begin or filesystem\ncode that already has a page allocated. Remove the handling of already\nallocated pages from block_write_begin after switching all callers that\ndo it to __block_write_begin.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "eafdc7d190a944c755a9fe68573c193e6e0217e7", "tree": "915c3cc8004706fabf88eae1ab123a3fa0147516", "parents": [ "256249584bda1a9357e2d29987a37f5b2df035f6" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Jun 04 11:29:53 2010 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Aug 09 16:47:29 2010 -0400" }, "message": "sort out blockdev_direct_IO variants\n\nMove the call to vmtruncate to get rid of accessive blocks to the callers\nin prepearation of the new truncate calling sequence. This was only done\nfor DIO_LOCKING filesystems, so the __blockdev_direct_IO_newtrunc variant\nwas not needed anyway. Get rid of blockdev_direct_IO_no_locking and\nits _newtrunc variant while at it as just opencoding the two additional\nparamters is shorted than the name suffix.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "56d35a4cd13e7bc5eca5b2dba5a41794afb17e11", "tree": "8173b2de9f0f2b596812a725ac1897efa0ba38fc", "parents": [ "0cfc9255a1efb0467de2162950197750570ecec0" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Thu Aug 05 14:41:42 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Thu Aug 05 14:41:42 2010 -0400" }, "message": "ext4: Fix dirtying of journalled buffers in data\u003djournal mode\n\nIn data\u003djournal mode, we still use block_write_begin() to prepare\npage for writing. This function can occasionally mark buffer dirty\nwhich violates journalling assumptions - when a buffer is part of\na transaction, it should be dirty and a buffer can be already part\nof a forget list of some transaction when block_write_begin()\ngets called. This violation of journalling assumptions then results\nin \"JBD: Spotted dirty metadata buffer...\" warnings.\n\nIn fact, temporary dirtying the buffer while the page is still locked\ndoes not really cause problems to the journalling because we won\u0027t write\nthe buffer until the page gets unlocked. So we just have to make sure\nto clear dirty bits before unlocking the page.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "a931da6ac9331a6c80dd91c199105806f2336188", "tree": "2d4ea766def9a98d21c2379f41e796b0e57ccd6b", "parents": [ "a51dca9cd3bb4ec5a05bfb6feabf024a5c808a37" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Aug 03 21:35:12 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Aug 03 21:35:12 2010 -0400" }, "message": "jbd2: Change j_state_lock to be a rwlock_t\n\nLockstat reports have shown that j_state_lock is a major source of\nlock contention, especially on systems with more than 4 CPU cores. So\nchange it to be a read/write spinlock.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "4538821993f4486c76090dfb377c60c0a0e71ba3", "tree": "d196075a963896c25df9ceb983a19ce82f5eab9b", "parents": [ "f613dfcb3345dacb8cf99b7bb359acc1c18a1157" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Thu Jul 29 15:06:10 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Thu Jul 29 15:06:10 2010 -0400" }, "message": "ext4: drop inode from orphan list if ext4_delete_inode() fails\n\nThere were some error paths in ext4_delete_inode() which was not\ndropping the inode from the orphan list. This could lead to a BUG_ON\non umount when the orphan list is discovered to be non-empty.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n\n" }, { "commit": "e3570639c8b5f2c6a5018a2649c2b7c276af76d7", "tree": "07c497204ba370c5e64b5868aa704e269e5893a3", "parents": [ "d889dc8382c4d71b6d538b7b13777bc1ec51df10" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Tue Jul 27 11:56:08 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:08 2010 -0400" }, "message": "ext4: don\u0027t print scary messages for allocation failures post-abort\n\nI often get emails containing the \"This should not happen!!\" message,\nconveniently trimmed to remove things like:\n\nsd 0:0:0:0: [sda] Unhandled error code\nsd 0:0:0:0: [sda] Result: hostbyte\u003dDID_OK driverbyte\u003dDRIVER_TIMEOUT\nsd 0:0:0:0: [sda] CDB: Write(10): 2a 00 03 13 c9 70 00 00 28 00\nend_request: I/O error, dev sda, sector 51628400\nAborting journal on device dm-0-8.\nEXT4-fs error (device dm-0): ext4_journal_start_sb: Detected aborted journal\nEXT4-fs (dm-0): Remounting filesystem read-only\n\nI don\u0027t think there is any value to the verbosity if the reason is\ndue to a filesystem abort; it just obfuscates the root cause.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "79e8303677fc15f508b9877e0fea1925c4add6f3", "tree": "85891ab05b2a9e5dd47966fd16bc372df6d4bb88", "parents": [ "62d2b5f2dcd3707b070efb16bbfdf6947c38c194" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Tue Jul 27 11:56:07 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:07 2010 -0400" }, "message": "ext4: fix ext4_get_blocks references\n\next4_get_blocks got renamed to ext4_map_blocks, but left stale\ncomments and a prototype littered around.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "0c095c7f113e9fd05913d6e1b2cccbe356be039e", "tree": "f23477719c99d5c20b4f1817c0ebe302a74eb870", "parents": [ "506bf2d82165c09b179a5077e01037f6270a4db3" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "message": "ext4: Don\u0027t error out the fs if the user tries to make a file too big\n\nIf the user attempts to make a non-extent-mapped file to be too large,\nreturn EFBIG, but don\u0027t call ext4_std_err() which will end up marking\nthe file system as containing an error.\n\nThanks to Toshiyuki Okajima-san at Fujitsu for pointing this out.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5b3ff237bef43b9e7fb7d1eb858e29b73fd664f9", "tree": "e3810d974a0e51a8c4b9046abdd36a5a4022abb3", "parents": [ "552ef8024f909d9b3a7442d0ab0d48a22de24e9e" ], "author": { "name": "jiayingz@google.com (Jiaying Zhang)", "email": "", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "message": "ext4: move aio completion after unwritten extent conversion\n\nThis patch is to be applied upon Christoph\u0027s \"direct-io: move aio_complete\ninto -\u003eend_io\" patch. It adds iocb and result fields to struct ext4_io_end_t,\nso that we can call aio_complete from ext4_end_io_nolock() after the extent\nconversion has finished.\n\nI have verified with Christoph\u0027s aio-dio test that used to fail after a few\nruns on an original kernel but now succeeds on the patched kernel.\n\nSee http://thread.gmane.org/gmane.comp.file-systems.ext4/19659 for details.\n\nSigned-off-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "552ef8024f909d9b3a7442d0ab0d48a22de24e9e", "tree": "868af331b76e12c8d17b8449094065a069e0d759", "parents": [ "5c521830cf3dfcf7638d409d8e02ed21020c064f" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:06 2010 -0400" }, "message": "direct-io: move aio_complete into -\u003eend_io\n\nFilesystems with unwritten extent support must not complete an AIO request\nuntil the transaction to convert the extent has been commited. That means\nthe aio_complete calls needs to be moved into the -\u003eend_io callback so\nthat the filesystem can control when to call it exactly.\n\nThis makes a bit of a mess out of dio_complete and the -\u003eend_io callback\nprototype even more complicated. \n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nReviewed-by: Jan Kara \u003cjack@suse.cz\u003e \nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "40389687382bf0ae71458e7c0f828137a438a956", "tree": "d2a49d494754e7fa28d33c2389f78a5b5da0db4f", "parents": [ "a271fe8527fe9637bdd82c97123b1356940dd84b" ], "author": { "name": "Amir G", "email": "amir73il@users.sourceforge.net", "time": "Tue Jul 27 11:56:05 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:05 2010 -0400" }, "message": "ext4: Fix block bitmap inconsistencies after a crash when deleting files\n\nWe have experienced bitmap inconsistencies after crash during file\ndelete under heavy load. The crash is not file system related and I\nthe following patch in ext4_free_branches() fixes the recovery\nproblem.\n\nIf the transaction is restarted and there is a crash before the new\ntransaction is committed, then after recovery, the blocks that this\nindirect block points to have been freed, but the indirect block\nitself has not been freed and may still point to some of the free\nblocks (because of the ext4_forget()).\n\nSo ext4_forget() should be called inside ext4_free_blocks() to avoid\nthis problem.\n\nSigned-off-by: Amir Goldstein \u003camir73il@users.sf.net\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "1c13d5c0872870cca3e612aa045d492ead9ab004", "tree": "6e3dd0d3f49ff56dda9fc6cd72c233759bc24e09", "parents": [ "c398eda0e43a791be0fca6f197a1e2bbb9f16070" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:03 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:03 2010 -0400" }, "message": "ext4: Save error information to the superblock for analysis\n\nSave number of file system errors, and the time function name, line\nnumber, block number, and inode number of the first and most recent\nerrors reported on the file system in the superblock.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "c398eda0e43a791be0fca6f197a1e2bbb9f16070", "tree": "d7b2b95490f96a75e116a3aa13c17767aa630342", "parents": [ "60fd4da34d55a9cc0d857fc76dc12cf8cab4ed02" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:40 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jul 27 11:56:40 2010 -0400" }, "message": "ext4: Pass line numbers to ext4_error() and friends\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "40e2e97316af6e62affab7a392e792494b8d9dde", "tree": "981ce3b464467893683d47f52ae5d35fdd32d46a", "parents": [ "696123fca877905696591829c97a2cef11c8d048" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Sun Jul 18 21:17:09 2010 +0000" }, "committer": { "name": "Alex Elder", "email": "aelder@sgi.com", "time": "Mon Jul 26 16:09:02 2010 -0500" }, "message": "direct-io: move aio_complete into -\u003eend_io\n\nFilesystems with unwritten extent support must not complete an AIO request\nuntil the transaction to convert the extent has been commited. That means\nthe aio_complete calls needs to be moved into the -\u003eend_io callback so\nthat the filesystem can control when to call it exactly.\n\nThis makes a bit of a mess out of dio_complete and the -\u003eend_io callback\nprototype even more complicated.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nReviewed-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Alex Elder \u003caelder@sgi.com\u003e\n" }, { "commit": "e29136f80e775b0310273932b4297a62f5574a29", "tree": "d10d56b15b7968744ea0becdf61440aae7e609c5", "parents": [ "c67d859e39896e4286249da89c4ca0ef8bd949cb" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jun 29 12:54:28 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Jun 29 12:54:28 2010 -0400" }, "message": "ext4: Enhance ext4_grp_locked_error() to take block and function numbers\n\nAlso use a macro definition so that __func__ and __LINE__ is implicit.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "206f7ab4f49a2021fcb8687f25395be77711ddee", "tree": "19764d36a25ec1e088d32572a1c0936eeaf19636", "parents": [ "5a0790c2c4a18435759a70e1562450035d778339" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Mon Jun 14 14:42:49 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Jun 14 14:42:49 2010 -0400" }, "message": "ext4: remove vestiges of nobh support\n\nThe nobh option was only supported for writeback mode, but given that all\nwrite paths actually create buffer heads it effectively was a no-op already.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5a0790c2c4a18435759a70e1562450035d778339", "tree": "80acb12d0e37196cf60ae4ca150c6b556115f302", "parents": [ "07a038245b28df9196ffb2e8cc626e9b956a4e23" ], "author": { "name": "Andi Kleen", "email": "andi@firstfloor.org", "time": "Mon Jun 14 13:28:03 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Jun 14 13:28:03 2010 -0400" }, "message": "ext4: remove initialized but not read variables\n\nNo real bugs found, just removed some dead code.\n\nFound by gcc 4.6\u0027s new warnings.\n\nSigned-off-by: Andi Kleen \u003cak@linux.intel.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "07a038245b28df9196ffb2e8cc626e9b956a4e23", "tree": "33faae4d5785d5cdd83381f0feff5798845feafc", "parents": [ "a0375156ca1041574b5d47cc7e32f10b891151b0" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Jun 14 09:54:48 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Jun 14 09:54:48 2010 -0400" }, "message": "ext4: Convert more i_flags references to use accessor functions\n\nThese changes are not ones which are likely to result in races, but\nthey should be fixed.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "84a8dce2710cc425089a2b92acc354d4fbb5788d", "tree": "2d83f638d52326b6a8080a26aa1423fa28b38ff1", "parents": [ "1f5a81e41f8b1a782c68d3843e9ec1bfaadf7d72" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Sat Jun 05 11:51:27 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sat Jun 05 11:51:27 2010 -0400" }, "message": "ext4: Fix remaining racy updates of EXT4_I(inode)-\u003ei_flags\n\nA few functions were still modifying i_flags in a racy manner.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "e4ce30f3779c2ddaa7dfaa4042209e5dbacbada5", "tree": "cc64c1dcd16b5dbf71ebc8338b339e6fb04abaee", "parents": [ "b899ebeb05da4287ce845976727e3e83dadd25d5", "14ece1028b3ed53ffec1b1213ffc6acaf79ad77c" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 10:26:37 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 10:26:37 2010 -0700" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4: (40 commits)\n ext4: Make fsync sync new parent directories in no-journal mode\n ext4: Drop whitespace at end of lines\n ext4: Fix compat EXT4_IOC_ADD_GROUP\n ext4: Conditionally define compat ioctl numbers\n tracing: Convert more ext4 events to DEFINE_EVENT\n ext4: Add new tracepoints to track mballoc\u0027s buddy bitmap loads\n ext4: Add a missing trace hook\n ext4: restart ext4_ext_remove_space() after transaction restart\n ext4: Clear the EXT4_EOFBLOCKS_FL flag only when warranted\n ext4: Avoid crashing on NULL ptr dereference on a filesystem error\n ext4: Use bitops to read/modify i_flags in struct ext4_inode_info\n ext4: Convert calls of ext4_error() to EXT4_ERROR_INODE()\n ext4: Convert callers of ext4_get_blocks() to use ext4_map_blocks()\n ext4: Add new abstraction ext4_map_blocks() underneath ext4_get_blocks()\n ext4: Use our own write_cache_pages()\n ext4: Show journal_checksum option\n ext4: Fix for ext4_mb_collect_stats()\n ext4: check for a good block group before loading buddy pages\n ext4: Prevent creation of files larger than RLIMIT_FSIZE using fallocate\n ext4: Remove extraneous newlines in ext4_msg() calls\n ...\n\nFixed up trivial conflict in fs/ext4/fsync.c\n" }, { "commit": "12755627bdcddcdb30a1bfb9a09395a52b1d6838", "tree": "b2d69696975d5457648020324f683a45302fbf3c", "parents": [ "fcbc59f96e38a0999e827be9d04d46b62b53b20a" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Thu Apr 08 22:04:20 2010 +0400" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri May 21 19:30:45 2010 +0200" }, "message": "quota: unify quota init condition in setattr\n\nQuota must being initialized if size or uid/git changes requested.\nBut initialization performed in two different places:\nin case of i_size file system is responsible for dquot init\n, but in case of uid/gid init will be called internally in\ndquot_transfer().\nThis ambiguity makes code harder to understand.\nLet\u0027s move this logic to one common helper function.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "60e6679e28518ccd67169c4a539d8cc7490eb8a6", "tree": "65f39d1e9da3006df5bc5532d0c8a7ec365599f0", "parents": [ "4d92dc0f00a775dc2e1267b0e00befb783902fe7" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon May 17 07:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon May 17 07:00:00 2010 -0400" }, "message": "ext4: Drop whitespace at end of lines\n\nThis patch was generated using:\n\n#!/usr/bin/perl -i\nwhile (\u003c\u003e) {\n s/[ \t]+$//;\n print;\n}\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5a58ec8766e0ce98fd585eb404b3e56935afafe6", "tree": "191f9457a05ffe7803712f62d40d848d58987765", "parents": [ "0617b83fa239db9743a18ce6cc0e556f4d0fd567" ], "author": { "name": "Li Zefan", "email": "lizf@cn.fujitsu.com", "time": "Mon May 17 02:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon May 17 02:00:00 2010 -0400" }, "message": "ext4: Add a missing trace hook\n\nCommit f8ec9d6837241865cf99bed97bb99f4399fd5a03 added a\ntrace event ext4_da_release_space, but didn\u0027t add some\ncorresponding trace hook.\n\nSigned-off-by: Li Zefan \u003clizf@cn.fujitsu.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "12e9b892002d9af057655d35b44db8ee9243b0dc", "tree": "c5831b4bcf98eebdd39158d08dab97c198f5c683", "parents": [ "24676da469f50f433baa347845639662c561d1f6" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Sun May 16 22:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 22:00:00 2010 -0400" }, "message": "ext4: Use bitops to read/modify i_flags in struct ext4_inode_info\n\nAt several places we modify EXT4_I(inode)-\u003ei_flags without holding\ni_mutex (ext4_do_update_inode, ...). These modifications are racy and\nwe can lose updates to i_flags. So convert handling of i_flags to use\nbitops which are atomic.\n\nhttps://bugzilla.kernel.org/show_bug.cgi?id\u003d15792\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "24676da469f50f433baa347845639662c561d1f6", "tree": "b4b8205f8b50376af286193d0dcbe76f2bc2d1e1", "parents": [ "2ed886852adfcb070bf350e66a0da0d98b2f3ab5" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 21:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 21:00:00 2010 -0400" }, "message": "ext4: Convert calls of ext4_error() to EXT4_ERROR_INODE()\n\nEXT4_ERROR_INODE() tends to provide better error information and in a\nmore consistent format. Some errors were not even identifying the inode\nor directory which was corrupted, which made them not very useful.\n\nAddresses-Google-Bug: #2507977\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "2ed886852adfcb070bf350e66a0da0d98b2f3ab5", "tree": "bcec0a1004f413b70087e2c43097892f87f21cc3", "parents": [ "e35fd6609b2fee54484d520deccb8f18bf7d38f3" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 20:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 20:00:00 2010 -0400" }, "message": "ext4: Convert callers of ext4_get_blocks() to use ext4_map_blocks()\n\nThis saves a huge amount of stack space by avoiding unnecesary struct\nbuffer_head\u0027s from being allocated on the stack.\n\nIn addition, to make the code easier to understand, collapse and\nrefactor ext4_get_block(), ext4_get_block_write(),\nnoalloc_get_block_write(), into a single function.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n\n" }, { "commit": "e35fd6609b2fee54484d520deccb8f18bf7d38f3", "tree": "9b786445602819074f599c282b31bead166e8c03", "parents": [ "8e48dcfbd7c0892b4cfd064d682cc4c95a29df32" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 19:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 19:00:00 2010 -0400" }, "message": "ext4: Add new abstraction ext4_map_blocks() underneath ext4_get_blocks()\n\nJack up ext4_get_blocks() and add a new function, ext4_map_blocks()\nwhich uses a much smaller structure, struct ext4_map_blocks which is\n20 bytes, as opposed to a struct buffer_head, which nearly 5 times\nbigger on an x86_64 machine. By switching things to use\next4_map_blocks(), we can save stack space by using ext4_map_blocks()\nsince we can avoid allocating a struct buffer_head on the stack.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "8e48dcfbd7c0892b4cfd064d682cc4c95a29df32", "tree": "653f12cc4eea9606c80142752612ad28f1ccdb45", "parents": [ "39a4bade8c1826b658316d66ee81c09b0a4d7d42" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 18:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 18:00:00 2010 -0400" }, "message": "ext4: Use our own write_cache_pages()\n\nMake a copy of write_cache_pages() for the benefit of\next4_da_writepages(). This allows us to simplify the code some, and\nwill allow us to further customize the code in future patches.\n\nThere are some nasty hacks in write_cache_pages(), which Linus has\n(correctly) characterized as vile. I\u0027ve just copied it into\nwrite_cache_pages_da(), without trying to clean those bits up lest I\nbreak something in the ext4\u0027s delalloc implementation, which is a bit\nfragile right now. This will allow Dave Chinner to clean up\nwrite_cache_pages() in mm/page-writeback.c, without worrying about\nbreaking ext4. Eventually write_cache_pages_da() will go away when I\nrewrite ext4\u0027s delayed allocation and create a general\next4_writepages() which is used for all of ext4\u0027s writeback. Until\nnow this is the lowest risk way to clean up the core\nwrite_cache_pages() function.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: Dave Chinner \u003cdavid@fromorbit.com\u003e\n" }, { "commit": "fbe845ddf368f77f86aa7500f8fd2690f54c66a8", "tree": "81e62af0c35794a4c0a2802d0528cacb43f2b4de", "parents": [ "d4c402d9fd97a54bb7ed89f9e9869897eb4f6e1f" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Sun May 16 13:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 13:00:00 2010 -0400" }, "message": "ext4: Remove extraneous newlines in ext4_msg() calls\n\nAddresses-Google-Bug: #2562325\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "72b8ab9dde211ea518ff27e631b2046ef90c29a2", "tree": "f63fc1894bcc39c81060e2fd8b21bedd07a1e958", "parents": [ "0e05842bc117ea70ceb979cca798fd026879951b" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Sun May 16 11:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 11:00:00 2010 -0400" }, "message": "ext4: don\u0027t use quota reservation for speculative metadata\n\nBecause we can badly over-reserve metadata when we\ncalculate worst-case, it complicates things for quota, since\nwe must reserve and then claim later, retry on EDQUOT, etc.\nQuota is also a generally smaller pool than fs free blocks,\nso this over-reservation hurts more, and more often.\n\nI\u0027m of the opinion that it\u0027s not the worst thing to allow\nmetadata to push a user slightly over quota. This simplifies\nthe code and avoids the false quota rejections that result\nfrom worst-case speculation.\n\nThis patch stops the speculative quota-charging for\nworst-case metadata requirements, and just charges quota\nwhen the blocks are allocated at writeout. It also is\nable to remove the try-again loop on EDQUOT.\n\nThis patch has been tested indirectly by running the xfstests\nsuite with a hack to mount \u0026 enable quota prior to the test.\n\nI also did a more specific test of fragmenting freespace\nand then doing a large delalloc write under quota; quota\nstopped me at the right amount of file IO, and then the\nwriteout generated enough metadata (due to the fragmentation)\nthat it put me slightly over quota, as expected.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "c445e3e0a5c2804524dec6e55f66d63f6bc5bc3e", "tree": "e4839825628b6d3aa3a9141e32370d5edebcc546", "parents": [ "a30eec2a8650a77f754e84b2e15f062fe652baa7" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Sun May 16 04:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 04:00:00 2010 -0400" }, "message": "ext4: don\u0027t scan/accumulate more pages than mballoc will allocate\n\nThere was a bug reported on RHEL5 that a 10G dd on a 12G box\nhad a very, very slow sync after that.\n\nAt issue was the loop in write_cache_pages scanning all the way\nto the end of the 10G file, even though the subsequent call\nto mpage_da_submit_io would only actually write a smallish amt; then\nwe went back to the write_cache_pages loop ... wasting tons of time\nin calling __mpage_da_writepage for thousands of pages we would\njust revisit (many times) later.\n\nUpstream it\u0027s not such a big issue for sys_sync because we get\nto the loop with a much smaller nr_to_write, which limits the loop.\n\nHowever, talking with Aneesh he realized that fsync upstream still\ngets here with a very large nr_to_write and we face the same problem.\n\nThis patch makes mpage_add_bh_to_extent stop the loop after we\u0027ve\naccumulated 2048 pages, by setting mpd-\u003eio_done \u003d 1; which ultimately\ncauses the write_cache_pages loop to break.\n\nRepeating the test with a dirty_ratio of 80 (to leave something for\nfsync to do), I don\u0027t see huge IO performance gains, but the reduction\nin cpu usage is striking: 80% usage with stock, and 2% with the\nbelow patch. Instrumenting the loop in write_cache_pages clearly\nshows that we are wasting time here.\n\nEventually we need to change mpage_da_map_pages() also submit its I/O\nto the block layer, subsuming mpage_da_submit_io(), and then change it\ncall ext4_get_blocks() multiple times.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "35121c9860316d7799cea0fbc359a9186e7c2747", "tree": "6deb327895e42708a491a3b5d7a66b95cfb18003", "parents": [ "b684b2ee9409f2890a8b3aea98525bbe5f84e276" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Sun May 16 00:00:00 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun May 16 00:00:00 2010 -0400" }, "message": "ext4: fix quota accounting in case of fallocate\n\nallocated_meta_data is already included in \u0027used\u0027 variable.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "202f2bb0708e968099a23fead4598a4c3f8f8637", "tree": "d014d6222ea90af157cfb0fcbf5677447353b95e", "parents": [ "ddc9b34c3b3bc063a7d9eb891ae09b8f49cfb27e", "b90f687018e6d6c77d981b09203780f7001407e5" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Apr 25 10:01:51 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Apr 25 10:01:51 2010 -0700" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4:\n ext4: Issue the discard operation *before* releasing the blocks to be reused\n ext4: Fix buffer head leaks after calls to ext4_get_inode_loc()\n ext4: Fix possible lost inode write in no journal mode\n" }, { "commit": "fd2dd9fbaf9e498ec63eef298921e36556f7214c", "tree": "fddd099cf65844f2d7d3498e25f83ec4c55fac52", "parents": [ "8b472d739b2ddd8ab7fb278874f696cd95b25a5e" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Sat Apr 03 17:44:16 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sat Apr 03 17:44:16 2010 -0400" }, "message": "ext4: Fix buffer head leaks after calls to ext4_get_inode_loc()\n\nCalls to ext4_get_inode_loc() returns with a reference to a buffer\nhead in iloc-\u003ebh. The callers of this function in ext4_write_inode()\nwhen in no journal mode and in ext4_xattr_fiemap() don\u0027t release the\nbuffer head after using it.\n\nAddresses-Google-Bug: #2548165\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "8b472d739b2ddd8ab7fb278874f696cd95b25a5e", "tree": "c820b44837a7e6130bbf7710e09ce5c06118e764", "parents": [ "2eaa9cfdf33b8d7fb7aff27792192e0019ae8fc6" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Sat Apr 03 16:45:06 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sat Apr 03 16:45:06 2010 -0400" }, "message": "ext4: Fix possible lost inode write in no journal mode\n\nIn the no-journal case, ext4_write_inode() will fetch the bh and call\nsync_dirty_buffer() on it. However, if the bh has already been\nwritten and the bh reclaimed for some other purpose, AND if the inode\nis the only one in the inode table block in use, then\next4_get_inode_loc() will not read the inode table block from disk,\nbut as an optimization, fill the block with zero\u0027s assuming that its\ncaller will copy in the on-disk version of the inode. This is not\ndone by ext4_write_inode(), so the contents of the inode can simply\nget lost. The fix is to use __ext4_get_inode_loc() with in_mem set to\n0, instead of ext4_get_inode_loc(). Long term the API needs to be\nfixed so it\u0027s obvious why latter is not safe.\n\nAddresses-Google-Bug: #2526446\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "d330a5befb88875a9b3d2db62f9b74dadf660b13", "tree": "1d3d59cd0e35cf08519ecac6f086eb69f157cb20", "parents": [ "57d54889cd00db2752994b389ba714138652e60c" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Sun Mar 14 18:17:54 2010 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Mar 14 18:17:54 2010 -0400" }, "message": "ext4: Fix estimate of # of blocks needed to write indirect-mapped files\n\nhttp://bugzilla.kernel.org/show_bug.cgi?id\u003d15420\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "e213e26ab3988c516c06eba4dcd030ac052f6dc9", "tree": "6e26fbdbb842b387697d73daf6e70cf718269a77", "parents": [ "c812a51d11bbe983f4c24e32b59b265705ddd3c2", "efd8f0e6f6c1faa041f228d7113bd3a9db802d49" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 13:20:53 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 13:20:53 2010 -0800" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs-2.6: (33 commits)\n quota: stop using QUOTA_OK / NO_QUOTA\n dquot: cleanup dquot initialize routine\n dquot: move dquot initialization responsibility into the filesystem\n dquot: cleanup dquot drop routine\n dquot: move dquot drop responsibility into the filesystem\n dquot: cleanup dquot transfer routine\n dquot: move dquot transfer responsibility into the filesystem\n dquot: cleanup inode allocation / freeing routines\n dquot: cleanup space allocation / freeing routines\n ext3: add writepage sanity checks\n ext3: Truncate allocated blocks if direct IO write fails to update i_size\n quota: Properly invalidate caches even for filesystems with blocksize \u003c pagesize\n quota: generalize quota transfer interface\n quota: sb_quota state flags cleanup\n jbd: Delay discarding buffers in journal_unmap_buffer\n ext3: quota_write cross block boundary behaviour\n quota: drop permission checks from xfs_fs_set_xstate/xfs_fs_set_xquota\n quota: split out compat_sys_quotactl support from quota.c\n quota: split out netlink notification support from quota.c\n quota: remove invalid optimization from quota_sync_all\n ...\n\nFixed trivial conflicts in fs/namei.c and fs/ufs/inode.c\n" }, { "commit": "9467c4fdd66f6810cecef0f1173330f3c6e67d45", "tree": "5fea180a10127c893b288dff2c8788b72d2eaea3", "parents": [ "35c2e967d067ff02dc944f2434f024419c2fe83a", "a9185b41a4f84971b930c519f0c63bd450c4810d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 11:53:53 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 05 11:53:53 2010 -0800" }, "message": "Merge branch \u0027write_inode2\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6\n\n* \u0027write_inode2\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6:\n pass writeback_control to -\u003ewrite_inode\n make sure data is on disk before calling -\u003ewrite_inode\n" }, { "commit": "a9185b41a4f84971b930c519f0c63bd450c4810d", "tree": "268cf4e206cca12fb9e1dd68984e7c190e465b46", "parents": [ "26821ed40b4230259e770c9911180f38fcaa6f59" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Fri Mar 05 09:21:37 2010 +0100" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Mar 05 13:25:52 2010 -0500" }, "message": "pass writeback_control to -\u003ewrite_inode\n\nThis gives the filesystem more information about the writeback that\nis happening. Trond requested this for the NFS unstable write handling,\nand other filesystems might benefit from this too by beeing able to\ndistinguish between the different callers in more detail.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "871a293155a24554e153538d36e3a80fa169aefb", "tree": "7e38f5a2f9e87f63cbc4bc1077a4bb49dde441b0", "parents": [ "907f4554e2521cb28b0009d17167760650a9561c" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Mar 03 09:05:07 2010 -0500" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:30 2010 +0100" }, "message": "dquot: cleanup dquot initialize routine\n\nGet rid of the initialize dquot operation - it is now always called from\nthe filesystem and if a filesystem really needs it\u0027s own (which none\ncurrently does) it can just call into it\u0027s own routine directly.\n\nRename the now static low-level dquot_initialize helper to __dquot_initialize\nand vfs_dq_init to dquot_initialize to have a consistent namespace.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "907f4554e2521cb28b0009d17167760650a9561c", "tree": "68dc49163fd34331f8efbd63592c8f1baa387031", "parents": [ "9f7547580263d4a55efe06ce5cfd567f568be6e8" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Mar 03 09:05:06 2010 -0500" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:30 2010 +0100" }, "message": "dquot: move dquot initialization responsibility into the filesystem\n\nCurrently various places in the VFS call vfs_dq_init directly. This means\nwe tie the quota code into the VFS. Get rid of that and make the\nfilesystem responsible for the initialization. For most metadata operations\nthis is a straight forward move into the methods, but for truncate and\nopen it\u0027s a bit more complicated.\n\nFor truncate we currently only call vfs_dq_init for the sys_truncate case\nbecause open already takes care of it for ftruncate and open(O_TRUNC) - the\nnew code causes an additional vfs_dq_init for those which is harmless.\n\nFor open the initialization is moved from do_filp_open into the open method,\nwhich means it happens slightly earlier now, and only for regular files.\nThe latter is fine because we don\u0027t need to initialize it for operations\non special files, and we already do it as part of the namespace operations\nfor directories.\n\nAdd a dquot_file_open helper that filesystems that support generic quotas\ncan use to fill in -\u003eopen.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "b43fa8284d7790d9cca32c9c55e24f29be2fa33b", "tree": "acb7f205fe2a1ae324e0e11c50f43baaa6b8dfab", "parents": [ "759bfee658beab14af7b357156461d0eb852be2c" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Mar 03 09:05:03 2010 -0500" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:29 2010 +0100" }, "message": "dquot: cleanup dquot transfer routine\n\nGet rid of the transfer dquot operation - it is now always called from\nthe filesystem and if a filesystem really needs it\u0027s own (which none\ncurrently does) it can just call into it\u0027s own routine directly.\n\nRename the now static low-level dquot_transfer helper to __dquot_transfer\nand vfs_dq_transfer to dquot_transfer to have a consistent namespace,\nand make the new dquot_transfer return a normal negative errno value\nwhich all callers expect.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "5dd4056db84387975140ff2568eaa0406f07985e", "tree": "03c26d7f6e3367b167bfeeb1a01654c6619573f4", "parents": [ "49792c806d0bfd53afc789dcdf50dc9bed2c5b83" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed Mar 03 09:05:00 2010 -0500" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:28 2010 +0100" }, "message": "dquot: cleanup space allocation / freeing routines\n\nGet rid of the alloc_space, free_space, reserve_space, claim_space and\nrelease_rsv dquot operations - they are always called from the filesystem\nand if a filesystem really needs their own (which none currently does)\nit can just call into it\u0027s own routine directly.\n\nMove shared logic into the common __dquot_alloc_space,\ndquot_claim_space_nodirty and __dquot_free_space low-level methods,\nand rationalize the wrappers around it to move as much as possible\ncode into the common block for CONFIG_QUOTA vs not. Also rename\nall these helpers to be named dquot_* instead of vfs_dq_*.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "9b1d0998d24f9c207d5fbdd0b8bac07284e0eda7", "tree": "0670c100d74086884c772e72d3e4135b57118515", "parents": [ "67eeb5685d2a211c0252ac7884142e503c759500" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Mar 03 16:19:32 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Mar 03 16:19:32 2010 -0500" }, "message": "ext4: Release page references acquired in ext4_da_block_invalidatepages\n\nWe forget to release page references we acquire in\next4_da_block_invalidatepages. Luckily, this function gets called only if we\nare not able to allocate blocks for delay-allocated data so that function\nshould better never be called.\n\nAlso cleanup handling of index variable.\n\nReported-by: Wu Fengguang \u003cfengguang.wu@intel.com\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "273df556b6ee2065bfe96edab5888d3dc9b108d8", "tree": "19c73685fce581e4ed85ff845e0b2fc485cedf9c", "parents": [ "b7adc1f363e72e9131a582cc2cb00eaf83f51a39" ], "author": { "name": "Frank Mayhar", "email": "fmayhar@google.com", "time": "Tue Mar 02 11:46:09 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Mar 02 11:46:09 2010 -0500" }, "message": "ext4: Convert BUG_ON checks to use ext4_error() instead\n\nConvert a bunch of BUG_ONs to emit a ext4_error() message and return\nEIO. This is a first pass and most notably does _not_ cover\nmballoc.c, which is a morass of void functions.\n\nSigned-off-by: Frank Mayhar \u003cfmayhar@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "b7adc1f363e72e9131a582cc2cb00eaf83f51a39", "tree": "9316acb7855ef64f051b4ee4c01f431c15cb4c1e", "parents": [ "744692dc059845b2a3022119871846e74d4f6e11" ], "author": { "name": "Jiaying Zhang", "email": "jiayingz@google.com", "time": "Tue Mar 02 13:26:36 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Mar 02 13:26:36 2010 -0500" }, "message": "ext4: Use direct_IO_no_locking in ext4 dio read\n\nSigned-off-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "744692dc059845b2a3022119871846e74d4f6e11", "tree": "ed246651aebcb8dae57de8c58dc20983064ee017", "parents": [ "c7064ef13b2181a489836349f9baf87df0dab28f" ], "author": { "name": "Jiaying Zhang", "email": "jiayingz@google.com", "time": "Thu Mar 04 16:14:02 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Thu Mar 04 16:14:02 2010 -0500" }, "message": "ext4: use ext4_get_block_write in buffer write\n\nAllocate uninitialized extent before ext4 buffer write and\nconvert the extent to initialized after io completes.\nThe purpose is to make sure an extent can only be marked\ninitialized after it has been written with new data so\nwe can safely drop the i_mutex lock in ext4 DIO read without\nexposing stale data. This helps to improve multi-thread DIO\nread performance on high-speed disks.\n\nSkip the nobh and data\u003djournal mount cases to make things simple for now.\n\nSigned-off-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "c7064ef13b2181a489836349f9baf87df0dab28f", "tree": "433d86d9ed4285e5c5e7f24fbbaa7f48fad09381", "parents": [ "b8b8afe236e97b6359d46d3a3f8c46455e192271" ], "author": { "name": "Jiaying Zhang", "email": "jiayingz@google.com", "time": "Tue Mar 02 13:28:44 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Mar 02 13:28:44 2010 -0500" }, "message": "ext4: mechanical rename some of the direct I/O get_block\u0027s identifiers\n\nThis commit renames some of the direct I/O\u0027s block allocation flags,\nvariables, and functions introduced in Mingming\u0027s \"Direct IO for holes\nand fallocate\" patches so that they can be used by ext4\u0027s buffered\nwrite path as well. Also changed the related function comments\naccordingly to cover both direct write and buffered write cases.\n\nSigned-off-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "da1dafca84413145f5ac59998b4cdd06fb89f721", "tree": "60582ee94c6867f234bb15bb18af8dd9be4f1693", "parents": [ "f39490bcd1691d65dc33689222a12e1fc13dd824" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Mon Mar 01 23:15:02 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Mar 01 23:15:02 2010 -0500" }, "message": "ext4: explicitly remove inode from orphan list after failed direct io\n\nOtherwise non-empty orphan list will be triggered on umount.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "c8d46e41bc744c8fa0092112af3942fcd46c8b18", "tree": "fd8bad55c3c9903b6c798b838396bc832bbf7f4b", "parents": [ "73b50c1c92666d326b5fa2c945d46509f2f6d91f" ], "author": { "name": "Jiaying Zhang", "email": "jiayingz@google.com", "time": "Wed Feb 24 09:52:53 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Feb 24 09:52:53 2010 -0500" }, "message": "ext4: Add flag to files with blocks intentionally past EOF\n\nfallocate() may potentially instantiate blocks past EOF, depending\non the flags used when it is called.\n\ne2fsck currently has a test for blocks past i_size, and it\nsometimes trips up - noticeably on xfstests 013 which runs fsstress.\n\nThis patch from Jiayang does fix it up - it (along with\ne2fsprogs updates and other patches recently from Aneesh) has\nsurvived many fsstress runs in a row.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: Jiaying Zhang \u003cjiayingz@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "73b50c1c92666d326b5fa2c945d46509f2f6d91f", "tree": "87e2cc6fed3259a83364ede33d8f4d68173f8175", "parents": [ "ba869023eac8354b17acdcff82b851ea8e7b1809" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Tue Feb 16 15:06:29 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Feb 16 15:06:29 2010 -0500" }, "message": "ext4: Fix BUG_ON at fs/buffer.c:652 in no journal mode\n\nCalls to ext4_handle_dirty_metadata should only pass in an inode\npointer for inode-specific metadata, and not for shared metadata\nblocks such as inode table blocks, block group descriptors, the\nsuperblock, etc.\n\nThe BUG_ON can get tripped when updating a special device (such as a\nblock device) that is opened (so that i_mapping is set in\nfs/block_dev.c) and the file system is mounted in no journal mode.\n\nAddresses-Google-Bug: #2404870\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "12062dddda450976b129dcb1bacd91acaf4d8030", "tree": "e64590b1147639cd3629f8a977b269410cd6bd13", "parents": [ "f710b4b96ba292dfed2153afc47e9063b0abfd89" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Mon Feb 15 14:19:27 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Feb 15 14:19:27 2010 -0500" }, "message": "ext4: move __func__ into a macro for ext4_warning, ext4_error\n\nJust a pet peeve of mine; we had a mishash of calls with either __func__\nor \"function_name\" and the latter tends to get out of sync.\n\nI think it\u0027s easier to just hide the __func__ in a macro, and it\u0027ll\nbe consistent from then on.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "19f5fb7ad679bb361222c7916086435020c37cce", "tree": "9e301163075c4faaf340cf50afd51855c76acd8c", "parents": [ "d2eecb03936878ec574ade5532fa83df7d75dde7" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Jan 24 14:34:07 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Jan 24 14:34:07 2010 -0500" }, "message": "ext4: Use bitops to read/modify EXT4_I(inode)-\u003ei_state\n\nAt several places we modify EXT4_I(inode)-\u003ei_state without holding\ni_mutex (ext4_release_file, ext4_bmap, ext4_journalled_writepage,\next4_do_update_inode, ...). These modifications are racy and we can\nlose updates to i_state. So convert handling of i_state to use bitops\nwhich are atomic.\n\nCc: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "1296cc85c26e94eb865d03f82140f27d598de467", "tree": "b8a2bc06ecd992a86179ebcd9c087994c676c291", "parents": [ "5f634d064c709ea02c3cdaa850a08323a4a4bf28" ], "author": { "name": "Aneesh Kumar K.V", "email": "aneesh.kumar@linux.vnet.ibm.com", "time": "Fri Jan 15 01:27:59 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 15 01:27:59 2010 -0500" }, "message": "ext4: Drop EXT4_GET_BLOCKS_UPDATE_RESERVE_SPACE flag\n\nWe should update reserve space if it is delalloc buffer\nand that is indicated by EXT4_GET_BLOCKS_DELALLOC_RESERVE flag.\nSo use EXT4_GET_BLOCKS_DELALLOC_RESERVE in place of\nEXT4_GET_BLOCKS_UPDATE_RESERVE_SPACE\n\nSigned-off-by: Aneesh Kumar K.V \u003caneesh.kumar@linux.vnet.ibm.com\u003e\n" }, { "commit": "5f634d064c709ea02c3cdaa850a08323a4a4bf28", "tree": "3f81e9c56cd8348b7bb94f1d54efff696374c929", "parents": [ "1db913823c0f8360fccbd24ca67eb073966a5ffd" ], "author": { "name": "Aneesh Kumar K.V", "email": "aneesh.kumar@linux.vnet.ibm.com", "time": "Mon Jan 25 04:00:31 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Jan 25 04:00:31 2010 -0500" }, "message": "ext4: Fix quota accounting error with fallocate\n\nWhen we fallocate a region of the file which we had recently written,\nand which is still in the page cache marked as delayed allocated blocks\nwe need to make sure we don\u0027t do the quota update on writepage path.\nThis is because the needed quota updated would have already be done\nby fallocate.\n\nSigned-off-by: Aneesh Kumar K.V \u003caneesh.kumar@linux.vnet.ibm.com\u003e\n" }, { "commit": "1db913823c0f8360fccbd24ca67eb073966a5ffd", "tree": "ec5f7be2cfa884b04ca3d26c2dee53283fe16ca7", "parents": [ "74d2e4f8d79ae0c4b6ec027958d5b18058662eea" ], "author": { "name": "Aneesh Kumar K.V", "email": "aneesh.kumar@linux.vnet.ibm.com", "time": "Fri Jan 22 17:06:20 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 22 17:06:20 2010 -0500" }, "message": "ext4: Handle -EDQUOT error on write\n\nWe need to release the journal before we do a write_inode. Otherwise\nwe could deadlock.\n\nSigned-off-by: Aneesh Kumar K.V \u003caneesh.kumar@linux.vnet.ibm.com\u003e\n" }, { "commit": "9d0be50230b333005635967f7ecd4897dbfd181b", "tree": "59aefe29e33284e1d904b23eaf2cc98994431374", "parents": [ "ee5f4d9cdf32fd99172d11665c592a288c2b1ff4" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 02:41:30 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 02:41:30 2010 -0500" }, "message": "ext4: Calculate metadata requirements more accurately\n\nIn the past, ext4_calc_metadata_amount(), and its sub-functions\next4_ext_calc_metadata_amount() and ext4_indirect_calc_metadata_amount()\nbadly over-estimated the number of metadata blocks that might be\nrequired for delayed allocation blocks. This didn\u0027t matter as much\nwhen functions which managed the reserved metadata blocks were more\naggressive about dropping reserved metadata blocks as delayed\nallocation blocks were written, but unfortunately they were too\naggressive. This was fixed in commit 0637c6f, but as a result the\nover-estimation by ext4_calc_metadata_amount() would lead to reserving\n2-3 times the number of pending delayed allocation blocks as\npotentially required metadata blocks. So if there are 1 megabytes of\nblocks which have been not yet been allocation, up to 3 megabytes of\nspace would get reserved out of the user\u0027s quota and from the file\nsystem free space pool until all of the inode\u0027s data blocks have been\nallocated.\n\nThis commit addresses this problem by much more accurately estimating\nthe number of metadata blocks that will be required. It will still\nsomewhat over-estimate the number of blocks needed, since it must make\na worst case estimate not knowing which physical blocks will be\nneeded, but it is much more accurate than before.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "ee5f4d9cdf32fd99172d11665c592a288c2b1ff4", "tree": "a1ae2af609d49524d6b807890852bc549889821f", "parents": [ "0637c6f4135f592f094207c7c21e7c0fc5557834" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 02:36:15 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 02:36:15 2010 -0500" }, "message": "ext4: Fix accounting of reserved metadata blocks\n\nCommit 0637c6f had a typo which caused the reserved metadata blocks to\nnot be released correctly. Fix this.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "0637c6f4135f592f094207c7c21e7c0fc5557834", "tree": "ee76fc861dffa902e80d0d11c681f5dfa2fcc569", "parents": [ "515f41c33a9d44a964264c9511ad2c869af1fac3" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Dec 30 14:20:45 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Dec 30 14:20:45 2009 -0500" }, "message": "ext4: Patch up how we claim metadata blocks for quota purposes\n\nAs reported in Kernel Bugzilla #14936, commit d21cd8f triggered a BUG\nin the function ext4_da_update_reserve_space() found in\nfs/ext4/inode.c. The root cause of this BUG() was caused by the fact\nthat ext4_calc_metadata_amount() can severely over-estimate how many\nmetadata blocks will be needed, especially when using direct\nblock-mapped files.\n\nIn addition, it can also badly *under* estimate how much space is\nneeded, since ext4_calc_metadata_amount() assumes that the blocks are\ncontiguous, and this is not always true. If the application is\nwriting blocks to a sparse file, the number of metadata blocks\nnecessary can be severly underestimated by the functions\next4_da_reserve_space(), ext4_da_update_reserve_space() and\next4_da_release_space(). This was the cause of the dq_claim_space\nreports found on kerneloops.org.\n\nUnfortunately, doing this right means that we need to massively\nover-estimate the amount of free space needed. So in some cases we\nmay need to force the inode to be written to disk asynchronously in\nto avoid spurious quota failures.\n\nhttp://bugzilla.kernel.org/show_bug.cgi?id\u003d14936\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "2faf2e19dd0e060eeb32442858ef495ac3083277", "tree": "cd8c28414036f2d8ad45663371e1544d4efad5d4", "parents": [ "6d3b82f2d31f22085e5711b28dddcb9fb3d97a25" ], "author": { "name": "Richard Kennedy", "email": "richard@rsk.demon.co.uk", "time": "Fri Dec 25 15:46:07 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Dec 25 15:46:07 2009 -0500" }, "message": "ext4: return correct wbc.nr_to_write in ext4_da_writepages\n\nWhen ext4_da_writepages increases the nr_to_write in writeback_control\nthen it must always re-base the return value. Originally there was a\n(misguided) attempt prevent wbc.nr_to_write from going negative. In\nfact, it\u0027s necessary to allow nr_to_write to be negative so that\nwb_writeback() can correctly calculate how many pages were actually\nwritten. \n\nSigned-off-by: Richard Kennedy \u003crichard@rsk.demon.co.uk\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "c8afb44682fcef6273e8b8eb19fab13ddd05b386", "tree": "44c170427e54b611d7f02a31bbd5733cc9cf1dd0", "parents": [ "17bd55d037a02b04d9119511cfd1a4b985d20f63" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Wed Dec 23 07:58:12 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Dec 23 07:58:12 2009 -0500" }, "message": "ext4: flush delalloc blocks when space is low\n\nCreating many small files in rapid succession on a small\nfilesystem can lead to spurious ENOSPC; on a 104MB filesystem:\n\nfor i in `seq 1 22500`; do\n echo -n \u003e $SCRATCH_MNT/$i\n echo XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX \u003e $SCRATCH_MNT/$i\ndone\n\nleads to ENOSPC even though after a sync, 40% of the fs is free\nagain.\n\nThis is because we reserve worst-case metadata for delalloc writes,\nand when data is allocated that worst-case reservation is not\nusually needed.\n\nWhen freespace is low, kicking off an async writeback will start\nconverting that worst-case space usage into something more realistic,\nalmost always freeing up space to continue.\n\nThis resolves the testcase for me, and survives all 4 generic\nENOSPC tests in xfstests.\n\nWe\u0027ll still need a hard synchronous sync to squeeze out the last bit,\nbut this fixes things up to a large degree.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "39bc680a8160bb9d6743f7873b535d553ff61058", "tree": "6aac2c11b09870bffb877dcaf6ae44b34b56d4db", "parents": [ "d21cd8f163ac44b15c465aab7306db931c606908" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Thu Dec 10 16:36:27 2009 +0000" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Dec 23 13:44:12 2009 +0100" }, "message": "ext4: fix sleep inside spinlock issue with quota and dealloc (#14739)\n\nUnlock i_block_reservation_lock before vfs_dq_reserve_block().\nThis patch fixes http://bugzilla.kernel.org/show_bug.cgi?id\u003d14739\n\nCC: Theodore Ts\u0027o \u003ctytso@mit.edu\u003e\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "d21cd8f163ac44b15c465aab7306db931c606908", "tree": "72cbe30f358c93ed7cdae7d4061fc4fdb317959d", "parents": [ "82fdfa928cfa19d9627526b2ce164a27f8e9d34b" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Thu Dec 10 03:31:45 2009 +0000" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Dec 23 13:44:12 2009 +0100" }, "message": "ext4: Fix potential quota deadlock\n\nWe have to delay vfs_dq_claim_space() until allocation context destruction.\nCurrently we have following call-trace:\next4_mb_new_blocks()\n /* task is already holding ac-\u003ealloc_semp */\n -\u003eext4_mb_mark_diskspace_used\n -\u003evfs_dq_claim_space() /* acquire dqptr_sem here. Possible deadlock */\n -\u003eext4_mb_release_context() /* drop ac-\u003ealloc_semp here */\n\nLet\u0027s move quota claiming to ext4_da_update_reserve_space()\n\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n [ INFO: possible circular locking dependency detected ]\n 2.6.32-rc7 #18\n -------------------------------------------------------\n write-truncate-/3465 is trying to acquire lock:\n (\u0026s-\u003es_dquot.dqptr_sem){++++..}, at: [\u003cc025e73b\u003e] dquot_claim_space+0x3b/0x1b0\n\n but task is already holding lock:\n (\u0026meta_group_info[i]-\u003ealloc_sem){++++..}, at: [\u003cc02ce962\u003e] ext4_mb_load_buddy+0xb2/0x370\n\n which lock already depends on the new lock.\n\n the existing dependency chain (in reverse order) is:\n\n -\u003e #3 (\u0026meta_group_info[i]-\u003ealloc_sem){++++..}:\n [\u003cc017d04b\u003e] __lock_acquire+0xd7b/0x1260\n [\u003cc017d5ea\u003e] lock_acquire+0xba/0xd0\n [\u003cc0527191\u003e] down_read+0x51/0x90\n [\u003cc02ce962\u003e] ext4_mb_load_buddy+0xb2/0x370\n [\u003cc02d0c1c\u003e] ext4_mb_free_blocks+0x46c/0x870\n [\u003cc029c9d3\u003e] ext4_free_blocks+0x73/0x130\n [\u003cc02c8cfc\u003e] ext4_ext_truncate+0x76c/0x8d0\n [\u003cc02a8087\u003e] ext4_truncate+0x187/0x5e0\n [\u003cc01e0f7b\u003e] vmtruncate+0x6b/0x70\n [\u003cc022ec02\u003e] inode_setattr+0x62/0x190\n [\u003cc02a2d7a\u003e] ext4_setattr+0x25a/0x370\n [\u003cc022ee81\u003e] notify_change+0x151/0x340\n [\u003cc021349d\u003e] do_truncate+0x6d/0xa0\n [\u003cc0221034\u003e] may_open+0x1d4/0x200\n [\u003cc022412b\u003e] do_filp_open+0x1eb/0x910\n [\u003cc021244d\u003e] do_sys_open+0x6d/0x140\n [\u003cc021258e\u003e] sys_open+0x2e/0x40\n [\u003cc0103100\u003e] sysenter_do_call+0x12/0x32\n\n -\u003e #2 (\u0026ei-\u003ei_data_sem){++++..}:\n [\u003cc017d04b\u003e] __lock_acquire+0xd7b/0x1260\n [\u003cc017d5ea\u003e] lock_acquire+0xba/0xd0\n [\u003cc0527191\u003e] down_read+0x51/0x90\n [\u003cc02a5787\u003e] ext4_get_blocks+0x47/0x450\n [\u003cc02a74c1\u003e] ext4_getblk+0x61/0x1d0\n [\u003cc02a7a7f\u003e] ext4_bread+0x1f/0xa0\n [\u003cc02bcddc\u003e] ext4_quota_write+0x12c/0x310\n [\u003cc0262d23\u003e] qtree_write_dquot+0x93/0x120\n [\u003cc0261708\u003e] v2_write_dquot+0x28/0x30\n [\u003cc025d3fb\u003e] dquot_commit+0xab/0xf0\n [\u003cc02be977\u003e] ext4_write_dquot+0x77/0x90\n [\u003cc02be9bf\u003e] ext4_mark_dquot_dirty+0x2f/0x50\n [\u003cc025e321\u003e] dquot_alloc_inode+0x101/0x180\n [\u003cc029fec2\u003e] ext4_new_inode+0x602/0xf00\n [\u003cc02ad789\u003e] ext4_create+0x89/0x150\n [\u003cc0221ff2\u003e] vfs_create+0xa2/0xc0\n [\u003cc02246e7\u003e] do_filp_open+0x7a7/0x910\n [\u003cc021244d\u003e] do_sys_open+0x6d/0x140\n [\u003cc021258e\u003e] sys_open+0x2e/0x40\n [\u003cc0103100\u003e] sysenter_do_call+0x12/0x32\n\n -\u003e #1 (\u0026sb-\u003es_type-\u003ei_mutex_key#7/4){+.+...}:\n [\u003cc017d04b\u003e] __lock_acquire+0xd7b/0x1260\n [\u003cc017d5ea\u003e] lock_acquire+0xba/0xd0\n [\u003cc0526505\u003e] mutex_lock_nested+0x65/0x2d0\n [\u003cc0260c9d\u003e] vfs_load_quota_inode+0x4bd/0x5a0\n [\u003cc02610af\u003e] vfs_quota_on_path+0x5f/0x70\n [\u003cc02bc812\u003e] ext4_quota_on+0x112/0x190\n [\u003cc026345a\u003e] sys_quotactl+0x44a/0x8a0\n [\u003cc0103100\u003e] sysenter_do_call+0x12/0x32\n\n -\u003e #0 (\u0026s-\u003es_dquot.dqptr_sem){++++..}:\n [\u003cc017d361\u003e] __lock_acquire+0x1091/0x1260\n [\u003cc017d5ea\u003e] lock_acquire+0xba/0xd0\n [\u003cc0527191\u003e] down_read+0x51/0x90\n [\u003cc025e73b\u003e] dquot_claim_space+0x3b/0x1b0\n [\u003cc02cb95f\u003e] ext4_mb_mark_diskspace_used+0x36f/0x380\n [\u003cc02d210a\u003e] ext4_mb_new_blocks+0x34a/0x530\n [\u003cc02c83fb\u003e] ext4_ext_get_blocks+0x122b/0x13c0\n [\u003cc02a5966\u003e] ext4_get_blocks+0x226/0x450\n [\u003cc02a5ff3\u003e] mpage_da_map_blocks+0xc3/0xaa0\n [\u003cc02a6ed6\u003e] ext4_da_writepages+0x506/0x790\n [\u003cc01de272\u003e] do_writepages+0x22/0x50\n [\u003cc01d766d\u003e] __filemap_fdatawrite_range+0x6d/0x80\n [\u003cc01d7b9b\u003e] filemap_flush+0x2b/0x30\n [\u003cc02a40ac\u003e] ext4_alloc_da_blocks+0x5c/0x60\n [\u003cc029e595\u003e] ext4_release_file+0x75/0xb0\n [\u003cc0216b59\u003e] __fput+0xf9/0x210\n [\u003cc0216c97\u003e] fput+0x27/0x30\n [\u003cc02122dc\u003e] filp_close+0x4c/0x80\n [\u003cc014510e\u003e] put_files_struct+0x6e/0xd0\n [\u003cc01451b7\u003e] exit_files+0x47/0x60\n [\u003cc0146a24\u003e] do_exit+0x144/0x710\n [\u003cc0147028\u003e] do_group_exit+0x38/0xa0\n [\u003cc0159abc\u003e] get_signal_to_deliver+0x2ac/0x410\n [\u003cc0102849\u003e] do_notify_resume+0xb9/0x890\n [\u003cc01032d2\u003e] work_notifysig+0x13/0x21\n\n other info that might help us debug this:\n\n 3 locks held by write-truncate-/3465:\n #0: (jbd2_handle){+.+...}, at: [\u003cc02e1f8f\u003e] start_this_handle+0x38f/0x5c0\n #1: (\u0026ei-\u003ei_data_sem){++++..}, at: [\u003cc02a57f6\u003e] ext4_get_blocks+0xb6/0x450\n #2: (\u0026meta_group_info[i]-\u003ealloc_sem){++++..}, at: [\u003cc02ce962\u003e] ext4_mb_load_buddy+0xb2/0x370\n\n stack backtrace:\n Pid: 3465, comm: write-truncate- Not tainted 2.6.32-rc7 #18\n Call Trace:\n [\u003cc0524cb3\u003e] ? printk+0x1d/0x22\n [\u003cc017ac9a\u003e] print_circular_bug+0xca/0xd0\n [\u003cc017d361\u003e] __lock_acquire+0x1091/0x1260\n [\u003cc016bca2\u003e] ? sched_clock_local+0xd2/0x170\n [\u003cc0178fd0\u003e] ? trace_hardirqs_off_caller+0x20/0xd0\n [\u003cc017d5ea\u003e] lock_acquire+0xba/0xd0\n [\u003cc025e73b\u003e] ? dquot_claim_space+0x3b/0x1b0\n [\u003cc0527191\u003e] down_read+0x51/0x90\n [\u003cc025e73b\u003e] ? dquot_claim_space+0x3b/0x1b0\n [\u003cc025e73b\u003e] dquot_claim_space+0x3b/0x1b0\n [\u003cc02cb95f\u003e] ext4_mb_mark_diskspace_used+0x36f/0x380\n [\u003cc02d210a\u003e] ext4_mb_new_blocks+0x34a/0x530\n [\u003cc02c601d\u003e] ? ext4_ext_find_extent+0x25d/0x280\n [\u003cc02c83fb\u003e] ext4_ext_get_blocks+0x122b/0x13c0\n [\u003cc016bca2\u003e] ? sched_clock_local+0xd2/0x170\n [\u003cc016be60\u003e] ? sched_clock_cpu+0x120/0x160\n [\u003cc016beef\u003e] ? cpu_clock+0x4f/0x60\n [\u003cc0178fd0\u003e] ? trace_hardirqs_off_caller+0x20/0xd0\n [\u003cc052712c\u003e] ? down_write+0x8c/0xa0\n [\u003cc02a5966\u003e] ext4_get_blocks+0x226/0x450\n [\u003cc016be60\u003e] ? sched_clock_cpu+0x120/0x160\n [\u003cc016beef\u003e] ? cpu_clock+0x4f/0x60\n [\u003cc017908b\u003e] ? trace_hardirqs_off+0xb/0x10\n [\u003cc02a5ff3\u003e] mpage_da_map_blocks+0xc3/0xaa0\n [\u003cc01d69cc\u003e] ? find_get_pages_tag+0x16c/0x180\n [\u003cc01d6860\u003e] ? find_get_pages_tag+0x0/0x180\n [\u003cc02a73bd\u003e] ? __mpage_da_writepage+0x16d/0x1a0\n [\u003cc01dfc4e\u003e] ? pagevec_lookup_tag+0x2e/0x40\n [\u003cc01ddf1b\u003e] ? write_cache_pages+0xdb/0x3d0\n [\u003cc02a7250\u003e] ? __mpage_da_writepage+0x0/0x1a0\n [\u003cc02a6ed6\u003e] ext4_da_writepages+0x506/0x790\n [\u003cc016beef\u003e] ? cpu_clock+0x4f/0x60\n [\u003cc016bca2\u003e] ? sched_clock_local+0xd2/0x170\n [\u003cc016be60\u003e] ? sched_clock_cpu+0x120/0x160\n [\u003cc016be60\u003e] ? sched_clock_cpu+0x120/0x160\n [\u003cc02a69d0\u003e] ? ext4_da_writepages+0x0/0x790\n [\u003cc01de272\u003e] do_writepages+0x22/0x50\n [\u003cc01d766d\u003e] __filemap_fdatawrite_range+0x6d/0x80\n [\u003cc01d7b9b\u003e] filemap_flush+0x2b/0x30\n [\u003cc02a40ac\u003e] ext4_alloc_da_blocks+0x5c/0x60\n [\u003cc029e595\u003e] ext4_release_file+0x75/0xb0\n [\u003cc0216b59\u003e] __fput+0xf9/0x210\n [\u003cc0216c97\u003e] fput+0x27/0x30\n [\u003cc02122dc\u003e] filp_close+0x4c/0x80\n [\u003cc014510e\u003e] put_files_struct+0x6e/0xd0\n [\u003cc01451b7\u003e] exit_files+0x47/0x60\n [\u003cc0146a24\u003e] do_exit+0x144/0x710\n [\u003cc017b163\u003e] ? lock_release_holdtime+0x33/0x210\n [\u003cc0528137\u003e] ? _spin_unlock_irq+0x27/0x30\n [\u003cc0147028\u003e] do_group_exit+0x38/0xa0\n [\u003cc017babb\u003e] ? trace_hardirqs_on+0xb/0x10\n [\u003cc0159abc\u003e] get_signal_to_deliver+0x2ac/0x410\n [\u003cc0102849\u003e] do_notify_resume+0xb9/0x890\n [\u003cc0178fd0\u003e] ? trace_hardirqs_off_caller+0x20/0xd0\n [\u003cc017b163\u003e] ? lock_release_holdtime+0x33/0x210\n [\u003cc0165b50\u003e] ? autoremove_wake_function+0x0/0x50\n [\u003cc017ba54\u003e] ? trace_hardirqs_on_caller+0x134/0x190\n [\u003cc017babb\u003e] ? trace_hardirqs_on+0xb/0x10\n [\u003cc0300ba4\u003e] ? security_file_permission+0x14/0x20\n [\u003cc0215761\u003e] ? vfs_write+0x131/0x190\n [\u003cc0214f50\u003e] ? do_sync_write+0x0/0x120\n [\u003cc0103115\u003e] ? sysenter_do_call+0x27/0x32\n [\u003cc01032d2\u003e] work_notifysig+0x13/0x21\n\nCC: Theodore Ts\u0027o \u003ctytso@mit.edu\u003e\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "a9e7f4472075fb6937c545af3f6329e9946bbe66", "tree": "399e294982e1c9691332eca72942910a7f74e9f8", "parents": [ "fd8fbfc1709822bd94247c5b2ab15a5f5041e103" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Mon Dec 14 15:21:14 2009 +0300" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Dec 23 13:33:55 2009 +0100" }, "message": "ext4: Convert to generic reserved quota\u0027s space management.\n\nThis patch also fixes write vs chown race condition.\n\nAcked-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "4515c3069da5bfb6f08dbfca499464b4cbdfcb50", "tree": "0f067045a0e31f126aebcf3087028639ceb237b4", "parents": [ "a5eba3f66f812cbc076a1170b3f888ad63f850b2", "fab3a549e204172236779f502eccb4f9bf0dc87d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Dec 10 09:33:29 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Dec 10 09:33:29 2009 -0800" }, "message": "Merge branch \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4\n\n* \u0027for_linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4: (47 commits)\n ext4: Fix potential fiemap deadlock (mmap_sem vs. i_data_sem)\n ext4: Do not override ext2 or ext3 if built they are built as modules\n jbd2: Export jbd2_log_start_commit to fix ext4 build\n ext4: Fix insufficient checks in EXT4_IOC_MOVE_EXT\n ext4: Wait for proper transaction commit on fsync\n ext4: fix incorrect block reservation on quota transfer.\n ext4: quota macros cleanup\n ext4: ext4_get_reserved_space() must return bytes instead of blocks\n ext4: remove blocks from inode prealloc list on failure\n ext4: wait for log to commit when umounting\n ext4: Avoid data / filesystem corruption when write fails to copy data\n ext4: Use ext4 file system driver for ext2/ext3 file system mounts\n ext4: Return the PTR_ERR of the correct pointer in setup_new_group_blocks()\n jbd2: Add ENOMEM checking in and for jbd2_journal_write_metadata_buffer()\n ext4: remove unused parameter wbc from __ext4_journalled_writepage()\n ext4: remove encountered_congestion trace\n ext4: move_extent_per_page() cleanup\n ext4: initialize moved_len before calling ext4_move_extents()\n ext4: Fix double-free of blocks with EXT4_IOC_MOVE_EXT\n ext4: use ext4_data_block_valid() in ext4_free_blocks()\n ...\n" }, { "commit": "d014d043869cdc591f3a33243d3481fa4479c2d0", "tree": "63626829498e647ba058a1ce06419fe7e4d5f97d", "parents": [ "6ec22f9b037fc0c2e00ddb7023fad279c365324d", "6070d81eb5f2d4943223c96e7609a53cdc984364" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Dec 07 18:36:35 2009 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Dec 07 18:36:35 2009 +0100" }, "message": "Merge branch \u0027for-next\u0027 into for-linus\n\nConflicts:\n\n\tkernel/irq/chip.c\n" }, { "commit": "f8ec9d6837241865cf99bed97bb99f4399fd5a03", "tree": "f1198366ae4c9c2db3c8869813f061ee85d3b243", "parents": [ "71f2be213a0009098819e5c04f75ff19f84f2122" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 01:00:21 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 01 01:00:21 2010 -0500" }, "message": "ext4: Add new tracepoints to debug delayed allocation space functions\n\nAdd tracepoints for ext4_da_reserve_space(),\next4_da_update_reserve_space(), and ext4_da_release_space().\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "1f2acb6017d8528135ec3b01ab7cd2be6ea0630b", "tree": "7263a5c1402237166cf425bc8f5336426159622c", "parents": [ "15121c18a22ae483279f76dc9e554334b800d0f7" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 22 17:40:42 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Jan 22 17:40:42 2010 -0500" }, "message": "ext4: Add block validity check when truncating indirect block mapped inodes\n\nAdd checks to ext4_free_branches() to make sure a block number found\nin an indirect block are valid before trying to free it. If a bad\nblock number is found, stop freeing the indirect block immediately,\nsince the file system is corrupt and we will need to run fsck anyway.\nThis also avoids spamming the logs, and specifically avoids\ndriver-level \"attempt to access beyond end of device\" errors obscure\nwhat is really going on.\n\nIf you get *really*, *really*, *really* unlucky, without this patch, a\nsupposed indirect block containing garbage might contain a reference\nto a primary block group descriptor, in which case\next4_free_branches() could end up zero\u0027ing out a block group\ndescriptor block, and if then one of the block bitmaps for a block\ngroup described by that bg descriptor block is not in memory, and is\nread in by ext4_read_block_bitmap(). This function calls\next4_valid_block_bitmap(), which assumes that bg_inode_table() was\nvalidated at mount time and hasn\u0027t been modified since. Since this\nassumption is no longer valid, it\u0027s possible for the value\n(ext4_inode_table(sb, desc) - group_first_block) to go negative, which\nwill cause ext4_find_next_zero_bit() to trigger a kernel GPF.\n\nAddresses-Google-Bug: #2220436\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "a1de02dccf906faba2ee2d99cac56799bda3b96a", "tree": "9871a11252e9a7d39882206c2057ec3080517365", "parents": [ "724e6d3fe8003c3f60bf404bf22e4e331327c596" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Thu Feb 04 23:58:38 2010 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Thu Feb 04 23:58:38 2010 -0500" }, "message": "ext4: fix async i/o writes beyond 4GB to a sparse file\n\nThe \"offset\" member in ext4_io_end holds bytes, not blocks, so\next4_lblk_t is wrong - and too small (u32).\n\nThis caused the async i/o writes to sparse files beyond 4GB to fail\nwhen they wrapped around to 0.\n\nAlso fix up the type of arguments to ext4_convert_unwritten_extents(),\nit gets ssize_t from ext4_end_aio_dio_nolock() and\next4_ext_direct_IO().\n\nReported-by: Giel de Nijs \u003cgiel@vectorwise.com\u003e\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\n" }, { "commit": "b436b9bef84de6893e86346d8fbf7104bc520645", "tree": "50fb9ae167bcd622e9adf47646bcf3b4c7dd111d", "parents": [ "194074acacebc169ded90a4657193f5180015051" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Tue Dec 08 23:51:10 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Dec 08 23:51:10 2009 -0500" }, "message": "ext4: Wait for proper transaction commit on fsync\n\nWe cannot rely on buffer dirty bits during fsync because pdflush can come\nbefore fsync is called and clear dirty bits without forcing a transaction\ncommit. What we do is that we track which transaction has last changed\nthe inode and which transaction last changed allocation and force it to\ndisk on fsync.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "194074acacebc169ded90a4657193f5180015051", "tree": "6661f85c6ef4548abe348c2611ad8a43b17ec320", "parents": [ "5aca07eb7d8f14d90c740834d15ca15277f4820c" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Tue Dec 08 22:42:28 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Dec 08 22:42:28 2009 -0500" }, "message": "ext4: fix incorrect block reservation on quota transfer.\n\nInside -\u003esetattr() call both ATTR_UID and ATTR_GID may be valid\nThis means that we may end-up with transferring all quotas. Add\nwe have to reserve QUOTA_DEL_BLOCKS for all quotas, as we do in\ncase of QUOTA_INIT_BLOCKS.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nReviewed-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5aca07eb7d8f14d90c740834d15ca15277f4820c", "tree": "362ab7bb56cf0920ee27bbaf97e2657c022eb4b4", "parents": [ "8aa6790f876e81f5a2211fe1711a5fe3fe2d7b20" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Tue Dec 08 22:42:15 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Dec 08 22:42:15 2009 -0500" }, "message": "ext4: quota macros cleanup\n\nCurrently all quota block reservation macros contains hard-coded \"2\"\naka MAXQUOTAS value. This is no good because in some places it is not\nobvious to understand what does this digit represent. Let\u0027s introduce\nnew macro with self descriptive name.\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nAcked-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "8aa6790f876e81f5a2211fe1711a5fe3fe2d7b20", "tree": "d82eef7f6753858a08640b226bece35dfca3c52d", "parents": [ "b844167edc7fcafda9623955c05e4c1b3c32ebc7" ], "author": { "name": "Dmitry Monakhov", "email": "dmonakhov@openvz.org", "time": "Tue Dec 08 22:41:52 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Dec 08 22:41:52 2009 -0500" }, "message": "ext4: ext4_get_reserved_space() must return bytes instead of blocks\n\nSigned-off-by: Dmitry Monakhov \u003cdmonakhov@openvz.org\u003e\nReviewed-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nAcked-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "b9a4207d5e911b938f73079a83cc2ae10524ec7f", "tree": "6926354a2c6cc8e5f0e930845774ffe73d3766b8", "parents": [ "24b584240a0006ea7436cd35f5e8983eb76f1e6f" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Tue Dec 08 21:24:33 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Dec 08 21:24:33 2009 -0500" }, "message": "ext4: Avoid data / filesystem corruption when write fails to copy data\n\nWhen ext4_write_begin fails after allocating some blocks or\ngeneric_perform_write fails to copy data to write, we truncate blocks\nalready instantiated beyond i_size. Although these blocks were never\ninside i_size, we have to truncate the pagecache of these blocks so\nthat corresponding buffers get unmapped. Otherwise subsequent\n__block_prepare_write (called because we are retrying the write) will\nfind the buffers mapped, not call -\u003eget_block, and thus the page will\nbe backed by already freed blocks leading to filesystem and data\ncorruption.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "af901ca181d92aac3a7dc265144a9081a86d8f39", "tree": "380054af22521144fbe1364c3bcd55ad24c9bde4", "parents": [ "972b94ffb90ea6d20c589d9a47215df103388ddd" ], "author": { "name": "André Goddard Rosa", "email": "andre.goddard@gmail.com", "time": "Sat Nov 14 13:09:05 2009 -0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Dec 04 15:39:55 2009 +0100" }, "message": "tree-wide: fix assorted typos all over the place\n\nThat is \"success\", \"unknown\", \"through\", \"performance\", \"[re|un]mapping\"\n, \"access\", \"default\", \"reasonable\", \"[con]currently\", \"temperature\"\n, \"channel\", \"[un]used\", \"application\", \"example\",\"hierarchy\", \"therefore\"\n, \"[over|under]flow\", \"contiguous\", \"threshold\", \"enough\" and others.\n\nSigned-off-by: André Goddard Rosa \u003candre.goddard@gmail.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "bf48aabb894fd0639ab72a26e8abbd7683ef23c2", "tree": "cd7ba4504558ed518126966fa79f18adf2f37fd6", "parents": [ "fb3d38b9904888aa8e36d88b2388dc70934b2169" ], "author": { "name": "Uwe Kleine-König", "email": "u.kleine-koenig@pengutronix.de", "time": "Wed Oct 28 20:11:03 2009 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Dec 04 15:39:50 2009 +0100" }, "message": "tree-wide: fix typos \"offest\" -\u003e \"offset\"\n\nThis patch was generated by\n\n\tgit grep -E -i -l \u0027offest\u0027 | xargs -r perl -p -i -e \u0027s/offest/offset/\u0027\n\nSigned-off-by: Uwe Kleine-König \u003cu.kleine-koenig@pengutronix.de\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "3f0ca309858ee186435c608ee9eaafd1c8dcb53a", "tree": "747897214d2fda0ba36c36dc7be6352dacb3f5a0", "parents": [ "b4d7241596ffb6398ac5535ae8cf80d845b0c254" ], "author": { "name": "Wu Fengguang", "email": "fengguang.wu@intel.com", "time": "Tue Nov 24 11:15:44 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Nov 24 11:15:44 2009 -0500" }, "message": "ext4: remove unused parameter wbc from __ext4_journalled_writepage()\n\nCC: Jan Kara \u003cjack@suse.cz\u003e \nSigned-off-by: Wu Fengguang \u003cfengguang.wu@intel.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "e6362609b6c71c5b802026be9cf263bbdd67a50e", "tree": "be908b6b0566f70d31378bf11b548a0d59ae0218", "parents": [ "4433871130f36585fde38e7dd817433296648945" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:17:05 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:17:05 2009 -0500" }, "message": "ext4: call ext4_forget() from ext4_free_blocks()\n\nAdd the facility for ext4_forget() to be called from\next4_free_blocks(). This simplifies the code in a large number of\nplaces, and centralizes most of the work of calling ext4_forget() into\na single place.\n\nAlso fix a bug in the extents migration code; it wasn\u0027t calling\next4_forget() when releasing the indirect blocks during the\nconversion. As a result, if the system cashed during or shortly after\nthe extents migration, and the released indirect blocks get reused as\ndata blocks, the journal replay would corrupt the data blocks. With\nthis new patch, fixing this bug was as simple as adding the\nEXT4_FREE_BLOCKS_FORGET flags to the call to ext4_free_blocks().\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: \"Aneesh Kumar K.V\" \u003caneesh.kumar@linux.vnet.ibm.com\u003e\n" }, { "commit": "b7e57e7c2a41826e51fe060fae5158bfc7a04e81", "tree": "2c7331402c731a9c6bb98827a4f6e5926fd90055", "parents": [ "e4684b3fbb848446683feecb4aee133344c93933" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 22 21:00:13 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 22 21:00:13 2009 -0500" }, "message": "ext4: fold ext4_journal_forget() into ext4_forget()\n\nConvert the last two callers of ext4_journal_forget() to use\next4_forget() instead, and then fold ext4_journal_forget() into\next4_forget(). This reduces are code complexity and shortens our call\nstack.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "d6797d14b1640d088652c72508b529a3aea479e3", "tree": "2e608c3e362b94439a2e7503b7d97cb3891bb101", "parents": [ "e3bb52ae2bb9573e84c17b8e3560378d13a5c798" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 22 20:52:12 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 22 20:52:12 2009 -0500" }, "message": "ext4: move ext4_forget() to ext4_jbd2.c\n\nThe ext4_forget() function better belongs in ext4_jbd2.c. This will\nallow us to do some cleanup of the ext4_journal_revoke() and\next4_journal_forget() functions, as well as giving us better error\nreporting since we can report the caller of ext4_forget() when things\ngo wrong.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "2bba702d4f88d7b010ec37e2527b552588404ae7", "tree": "74f46ab1e70328f23f1abbeaf3bf1a49c15a10e8", "parents": [ "6b17d902fdd241adfa4ce780df20547b28bf5801" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Mon Nov 23 07:24:48 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:24:48 2009 -0500" }, "message": "ext4: fix error handling in ext4_ind_get_blocks()\n\nWhen an error happened in ext4_splice_branch we failed to notice that\nin ext4_ind_get_blocks and mapped the buffer anyway. Fix the problem\nby checking for error properly.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: stable@kernel.org\t \n" }, { "commit": "1032988c71f3f85483b2b4319684d1205a704c02", "tree": "ace311067686bf5b93e10ab793ee62b593d8660f", "parents": [ "8dadb198cb70ef811916668fe67eeec82e8858dd" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 15 15:29:56 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 15 15:29:56 2009 -0500" }, "message": "ext4: fix block validity checks so they work correctly with meta_bg\n\nThe block validity checks used by ext4_data_block_valid() wasn\u0027t\ncorrectly written to check file systems with the meta_bg feature. Fix\nthis.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: stable@kernel.org\n" }, { "commit": "30c6e07a92ea4cb87160d32ffa9bce172576ae4c", "tree": "5799d5f074bfe2dbde001f767bf675e6c11e8335", "parents": [ "50689696867d95b38d9c7be640a311494a04fb86" ], "author": { "name": "Julia Lawall", "email": "julia@diku.dk", "time": "Sun Nov 15 15:30:58 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sun Nov 15 15:30:58 2009 -0500" }, "message": "ext4: fix i_flags access in ext4_da_writepages_trans_blocks()\n\nWe need to be testing the i_flags field in the ext4 specific portion\nof the inode, instead of the (confusingly aliased) i_flags field in\nthe generic struct inode.\n\nSigned-off-by: Julia Lawall \u003cjulia@diku.dk\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: stable@kernel.org\n" }, { "commit": "50689696867d95b38d9c7be640a311494a04fb86", "tree": "69e582bb6c95d8d0ac59bcad97b5f7f1d79e7126", "parents": [ "beac2da7565e42be59963824899825d0cc624295" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:17:34 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:17:34 2009 -0500" }, "message": "ext4: make sure directory and symlink blocks are revoked\n\nWhen an inode gets unlinked, the functions ext4_clear_blocks() and\next4_remove_blocks() call ext4_forget() for all the buffer heads\ncorresponding to the deleted inode\u0027s data blocks. If the inode is a\ndirectory or a symlink, the is_metadata parameter must be non-zero so\next4_forget() will revoke them via jbd2_journal_revoke(). Otherwise,\nif these blocks are reused for a data file, and the system crashes\nbefore a journal checkpoint, the journal replay could end up\ncorrupting these data blocks.\n\nThanks to Curt Wohlgemuth for pointing out potential problems in this\narea.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nCc: stable@kernel.org\n" }, { "commit": "beac2da7565e42be59963824899825d0cc624295", "tree": "22b38198349bd000cd146243271f604708fbc3fb", "parents": [ "cf40db137cc2b2a1b3f6850247ac2b181d9d3847" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:25:08 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 23 07:25:08 2009 -0500" }, "message": "ext4: add tracepoint for ext4_forget()\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "567f3e9a70d71e5c9be03701b8578be77857293b", "tree": "1985907b6d9deffc9d38275c000d7f82d158a93f", "parents": [ "92c28159dce22913aef6aa811ce6fb0f7f3790b1" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sat Nov 14 08:19:05 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Sat Nov 14 08:19:05 2009 -0500" }, "message": "ext4: plug a buffer_head leak in an error path of ext4_iget()\n\nOne of the invalid error paths in ext4_iget() forgot to brelse() the\ninode buffer head. Fix it by adding a brelse() in the common error\nreturn path, which also simplifies function.\n\nThanks to Andi Kleen \u003cak@linux.intel.com\u003e reporting the problem.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "4b70df181611012a3556f017b57dfcef7e1d279f", "tree": "95da7ccaea96227b597078c1c5f2a7571d959e99", "parents": [ "5f5249507e4b5c4fc0f9c93f33d133d8c95f47e1" ], "author": { "name": "Mingming", "email": "cmm@us.ibm.com", "time": "Tue Nov 03 14:44:54 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Nov 03 14:44:54 2009 -0500" }, "message": "ext4: code clean up for dio fallocate handling\n\nThe ext4_debug() call in ext4_end_io_dio() should be moved after the\ncheck to make sure that io_end is non-NULL.\n\nThe comment above ext4_get_block_dio_write() (\"Maximum number of\nblocks...\") is a duplicate; the original and correct comment is above\nthe #define DIO_MAX_BLOCKS up above.\n\nBased on review comments from Curt Wohlgemuth.\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "5f5249507e4b5c4fc0f9c93f33d133d8c95f47e1", "tree": "0b0a790f568c07298c3a4122572e84392c787648", "parents": [ "109f55651954def97fa41ee71c464d268c512ab0" ], "author": { "name": "Mingming", "email": "cmm@us.ibm.com", "time": "Tue Nov 10 10:48:04 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Nov 10 10:48:04 2009 -0500" }, "message": "ext4: skip conversion of uninit extents after direct IO if there isn\u0027t any\n\nAt the end of direct I/O operation, ext4_ext_direct_IO() always called\next4_convert_unwritten_extents(), regardless of whether there were any\nunwritten extents involved in the I/O or not.\n\nThis commit adds a state flag so that ext4_ext_direct_IO() only calls\next4_convert_unwritten_extents() when necessary.\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "109f55651954def97fa41ee71c464d268c512ab0", "tree": "7c4aa2d8cb10c100bbd252e657ac410d30844bac", "parents": [ "fa5d11133b07053270e18fa9c18560e66e79217e" ], "author": { "name": "Mingming", "email": "cmm@us.ibm.com", "time": "Tue Nov 10 10:48:08 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Nov 10 10:48:08 2009 -0500" }, "message": "ext4: fix ext4_ext_direct_IO()\u0027s return value after converting uninit extents\n\nAfter a direct I/O request covering an uninitalized extent (i.e.,\ncreated using the fallocate system call) or a hole in a file, ext4\nwill convert the uninitialized extent so it is marked as initialized\nby calling ext4_convert_unwritten_extents(). This function returns\nzero on success.\n\nThis return value was getting returned by ext4_direct_IO(); however\nthe file system\u0027s direct_IO function is supposed to return the number\nof bytes read or written on a success. By returning zero, it confused\nthe direct I/O code into falling back to buffered I/O unnecessarily.\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "fa5d11133b07053270e18fa9c18560e66e79217e", "tree": "1018ea7d0a2f511ae58934175dbc3815b7e63b5f", "parents": [ "b419148e567728f6af0c3b01965c1cc141e3e13a" ], "author": { "name": "Aneesh Kumar K.V", "email": "aneesh.kumar@linux.vnet.ibm.com", "time": "Mon Nov 02 18:50:49 2009 -0500" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Nov 02 18:50:49 2009 -0500" }, "message": "ext4: discard preallocation when restarting a transaction during truncate\n\nWhen restart a transaction during a truncate operation, we drop and\nreacquire i_data_sem. After reacquiring i_data_sem, we need to\ndiscard any inode-based preallocation that might have been grabbed\nwhile we released i_data_sem (for example, if pdflush is allocating\nblocks and racing against the truncate).\n\nSigned-off-by: Aneesh Kumar K.V \u003caneesh.kumar@linux.vnet.ibm.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "fbbf69456619de5d251cb9f1df609069178c62d5", "tree": "e0a5bc75a02dda7f46ba4d0e838c54f39b1fe4f6", "parents": [ "74072d0a63553720dd3c70a8b8e9407eb2027dbe" ], "author": { "name": "Eric Sandeen", "email": "sandeen@redhat.com", "time": "Fri Oct 02 21:20:55 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Oct 02 21:20:55 2009 -0400" }, "message": "[PATCH] ext4: retry failed direct IO allocations\n\nOn a 256M filesystem, doing this in a loop:\n\n xfs_io -F -f -d -c \u0027pwrite 0 64m\u0027 test\n rm -f test\n\neventually leads to ENOSPC. (the xfs_io command does a\n64m direct IO write to the file \"test\")\n\nAs with other block allocation callers, it looks like we need to\npotentially retry the allocations on the initial ENOSPC.\n\nSigned-off-by: Eric Sandeen \u003csandeen@redhat.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "74072d0a63553720dd3c70a8b8e9407eb2027dbe", "tree": "9895fcb17d44980a82b4daa7688c17da082b4d88", "parents": [ "f0e2dfa7f3e1361ca8fc91c25e67fc4e92613cc9" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Fri Oct 02 21:08:32 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Oct 02 21:08:32 2009 -0400" }, "message": "ext4: Fix build warning in ext4_dirty_inode()\n\nThis fixes the following warning:\n\nfs/ext4/inode.c: In function \u0027ext4_dirty_inode\u0027:\nfs/ext4/inode.c:5615: warning: unused variable \u0027current_handle\u0027\n\nWe remove the jbd_debug() statement which does use current_handle, as\nit\u0027s not terribly important in the grand scheme of things.\n\nThanks to Stephen Rothwell for pointing this out.\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "1f94533d9cd75f6d2826018d54a971b9cc085992", "tree": "16c85061eabfcdece2487909bc08c45d2c01d84f", "parents": [ "c1fccc0696bcaff6008c11865091f5ec4b0937ab" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Sep 30 22:57:41 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Sep 30 22:57:41 2009 -0400" }, "message": "ext4: fix a BUG_ON crash by checking that page has buffers attached to it\n\nIn ext4_num_dirty_pages() we were calling page_buffers() before\nchecking to see if the page actually had pages attached to it; this\nwould cause a BUG check crash in the inline function page_buffers().\n\nThanks to Markus Trippelsdorf for reporting this bug.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "f3dc272fd5e2ae08244796bb39e7e1ce4b25d3b3", "tree": "795edae4030352dfea7cb79ed5a5b0b5ef3d3afb", "parents": [ "830156c79b0a99ddf0f62496bcf4de640f9f52cd" ], "author": { "name": "Curt Wohlgemuth", "email": "curtw@google.com", "time": "Tue Sep 29 16:06:01 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Sep 29 16:06:01 2009 -0400" }, "message": "ext4: Make sure ext4_dirty_inode() updates the inode in no journal mode\n\nThis patch a problem that ext4_dirty_inode() was not calling\next4_mark_inode_dirty() if the current_handle is not valid, which it\nis the case in no journal mode.\n\nIt also removes a test for non-matching transaction which can never\nhappen.\n\nSigned-off-by: Curt Wohlgemuth \u003ccurtw@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "830156c79b0a99ddf0f62496bcf4de640f9f52cd", "tree": "7af8e969f359afd8a4738e22b0374b567041a1fd", "parents": [ "f3ce8064b388ccf420012c5a4907aae4f13fe9d0" ], "author": { "name": "Frank Mayhar", "email": "fmayhar@google.com", "time": "Tue Sep 29 10:07:47 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Tue Sep 29 10:07:47 2009 -0400" }, "message": "ext4: Avoid updating the inode table bh twice in no journal mode\n\nThis is a cleanup of commit 91ac6f4. Since ext4_mark_inode_dirty()\nhas already called ext4_mark_iloc_dirty(), which in turn calls\next4_do_update_inode(), it\u0027s not necessary to have ext4_write_inode()\ncall ext4_do_update_inode() in no journal mode. Indeed, it would be\nduplicated work.\n\nReviewed-by: \"Aneesh Kumar K.V\" \u003caneesh.kumar@linux.vnet.ibm.com\u003e\nSigned-off-by: Frank Mayhar \u003cfmayhar@google.com\u003e\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\n" }, { "commit": "8d5d02e6b176565c77ff03604908b1453a22044d", "tree": "0d29e4f28233f24960c7921c1c0a7608077bf713", "parents": [ "4c0425ff68b1b87b802ffeda7b6a46ff7da7241c" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Mon Sep 28 15:48:29 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Mon Sep 28 15:48:29 2009 -0400" }, "message": "ext4: async direct IO for holes and fallocate support\n\nFor async direct IO that covers holes or fallocate, the end_io\ncallback function now queued the convertion work on workqueue but\ndon\u0027t flush the work rightaway as it might take too long to afford.\n\nBut when fsync is called after all the data is completed, user expects\nthe metadata also being updated before fsync returns.\n\nThus we need to flush the conversion work when fsync() is called.\nThis patch keep track of a listed of completed async direct io that\nhas a work queued on workqueue. When fsync() is called, it will go\nthrough the list and do the conversion.\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\n" } ], "next": "4c0425ff68b1b87b802ffeda7b6a46ff7da7241c" }