)]}' { "log": [ { "commit": "318ae2edc3b29216abd8a2510f3f80b764f06858", "tree": "ce595adde342f57f379d277b25e4dd206988a052", "parents": [ "25cf84cf377c0aae5dbcf937ea89bc7893db5176", "3e58974027b04e84f68b964ef368a6cd758e2f84" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Mar 08 16:55:37 2010 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Mar 08 16:55:37 2010 +0100" }, "message": "Merge branch \u0027for-next\u0027 into for-linus\n\nConflicts:\n\tDocumentation/filesystems/proc.txt\n\tarch/arm/mach-u300/include/mach/debug-macro.S\n\tdrivers/net/qlge/qlge_ethtool.c\n\tdrivers/net/qlge/qlge_main.c\n\tdrivers/net/typhoon.c\n" }, { "commit": "86963918965eb8fe0c8ae009e7c1b4c630f533d5", "tree": "333371098928c834b0bda0c2fe5bfd591a3806ef", "parents": [ "e5472147e1c0712d95d973acfdbd862957c77add" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Tue Feb 16 20:37:12 2010 +0100" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Mar 05 00:20:26 2010 +0100" }, "message": "jbd: Delay discarding buffers in journal_unmap_buffer\n\nDelay discarding buffers in journal_unmap_buffer until\nwe know that \"add to orphan\" operation has definitely been\ncommitted, otherwise the log space of committing transation\nmay be freed and reused before truncate get committed, updates\nmay get lost if crash happens.\n\nThis patch is a backport of JBD2 fix by dingdinghua \u003cdingdinghua@nrchpc.ac.cn\u003e.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "3ad2f3fbb961429d2aa627465ae4829758bc7e07", "tree": "f365c513e8f5b477a61336a600ff54f32b7ad6e1", "parents": [ "1537a3638cbf741d3826c1002026cce487a6bee0" ], "author": { "name": "Daniel Mack", "email": "daniel@caiaq.de", "time": "Wed Feb 03 08:01:28 2010 +0800" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Tue Feb 09 11:13:56 2010 +0100" }, "message": "tree-wide: Assorted spelling fixes\n\nIn particular, several occurances of funny versions of \u0027success\u0027,\n\u0027unknown\u0027, \u0027therefore\u0027, \u0027acknowledge\u0027, \u0027argument\u0027, \u0027achieve\u0027, \u0027address\u0027,\n\u0027beginning\u0027, \u0027desirable\u0027, \u0027separate\u0027 and \u0027necessary\u0027 are fixed.\n\nSigned-off-by: Daniel Mack \u003cdaniel@caiaq.de\u003e\nCc: Joe Perches \u003cjoe@perches.com\u003e\nCc: Junio C Hamano \u003cgitster@pobox.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "3adae9da0b35d2ca908039f42a1e90395c335181", "tree": "bc7d971ffb673dab926a4c3d02411a2b78b58f6a", "parents": [ "9c28cbccec66a5ca292c6659bf5a0fe0c8459fa7" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Tue Aug 11 17:27:21 2009 +0200" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Sep 16 17:44:10 2009 +0200" }, "message": "jbd: Annotate transaction start also for journal_restart()\n\nlockdep annotation for a transaction start has been at the end of\njournal_start(). But a transaction is also started from journal_restart(). Move\nthe lockdep annotation to start_this_handle() which covers both cases.\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "b449fc6fcc07a392c69f3c1db9a4ad4dda8cbcba", "tree": "cc748936233e62cb8ae3325505e84805fb08b5ad", "parents": [ "ab86e5765d41a5eb4239a1c04d613db87bea5ed8" ], "author": { "name": "Andreas Dilger", "email": "adilger@sun.com", "time": "Thu Jul 30 20:09:46 2009 +0200" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Sep 16 17:44:10 2009 +0200" }, "message": "JBD: round commit timer up to avoid uncommitted transaction\n\nFix jiffie rounding in jbd commit timer setup code. Rounding down could cause\nthe timer to be fired before the corresponding transaction has expired. That\ntransaction can stay not committed forever if no new transaction is created or\nexplicit sync/umount happens.\n\nSigned-off-by: Andreas Dilger \u003cadilger@sun.com\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "1e9fd53b783ea646de3ee09a4574afeb6778d504", "tree": "0b87e9ca83c612a1c3c5b91a359f8604cde48c45", "parents": [ "9eaaa2d5759837402ec5eee13b2a97921808c3eb" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Jun 24 17:31:40 2009 +0200" }, "committer": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Wed Jul 15 21:30:07 2009 +0200" }, "message": "jbd: Fix a race between checkpointing code and journal_get_write_access()\n\nThe following race can happen:\n\n CPU1 CPU2\n checkpointing code checks the buffer, adds\n it to an array for writeback\ndo_get_write_access()\n ...\n lock_buffer()\n unlock_buffer()\n flush_batch() submits the buffer for IO\n __jbd_journal_file_buffer()\n\n So a buffer under writeout is returned from do_get_write_access(). Since\nthe filesystem code relies on the fact that journaled buffers cannot be\nwritten out, it does not take the buffer lock and so it can modify buffer\nwhile it is under writeout. That can lead to a filesystem corruption\nif we crash at the right moment. The similar problem can happen with\nthe journal_get_create_access() path.\n We fix the problem by clearing the buffer dirty bit under buffer_lock\neven if the buffer is on BJ_None list. Actually, we clear the dirty bit\nregardless the list the buffer is in and warn about the fact if\nthe buffer is already journalled.\n\nThanks for spotting the problem goes to dingdinghua \u003cdingdinghua85@gmail.com\u003e.\n\nReported-by: dingdinghua \u003cdingdinghua85@gmail.com\u003e\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "6f3f1cb21f08fbf757bbbbb0709ee515a7a7c6ad", "tree": "c6874d850ab2347dfeb58ea95a05422880fc0892", "parents": [ "e8ef7aaea79a899be4d7f50e829900c0ce15e52f" ], "author": { "name": "Hisashi Hifumi", "email": "hifumi.hisashi@oss.ntt.co.jp", "time": "Wed Jun 17 16:26:23 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 18 13:03:45 2009 -0700" }, "message": "jbd: clean up journal_try_to_free_buffers()\n\nI delete the following patch\n\"commit 3f31fddfa26b7594b44ff2b34f9a04ba409e0f91\nAuthor: Mingming Cao \u003ccmm@us.ibm.com\u003e\nDate: Fri Jul 25 01:46:22 2008 -0700\n\n jbd: fix race between free buffer and commit transaction\n\nThis patch is no longer needed because if race between freeing buffer and\ncommitting transaction functionality occurs and dio gets error, currently\ndio falls back to buffered IO by the following patch.\n\n\tcommit 6ccfa806a9cfbbf1cd43d5b6aa47ef2c0eb518fd\n\tAuthor: Hisashi Hifumi \u003chifumi.hisashi@oss.ntt.co.jp\u003e\n\tDate: Tue Sep 2 14:35:40 2008 -0700\n\n \tVFS: fix dio write returning EIO when try_to_release_page fails\n\nSigned-off-by: Hisashi Hifumi \u003chifumi.hisashi@oss.ntt.co.jp\u003e\nCc: Theodore Tso \u003ctytso@mit.edu\u003e\nCc: Mingming Cao \u003ccmm@us.ibm.com\u003e\nAcked-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "512a004382f2c60d5c4f855476ba965adc00250c", "tree": "e0ecfce01aab892a415b95d6d4c8c5a8bd033a49", "parents": [ "a64c8610bd3b753c6aff58f51c04cdf0ae478c18" ], "author": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Mar 27 22:14:27 2009 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Fri Mar 27 22:14:27 2009 -0400" }, "message": "ext3: Use WRITE_SYNC for commits which are caused by fsync()\n\nIf a commit is triggered by fsync(), set a flag indicating the journal\nblocks associated with the transaction should be flushed out using\nWRITE_SYNC.\n\nSigned-off-by: \"Theodore Ts\u0027o\" \u003ctytso@mit.edu\u003e\nAcked-by: Jan Kara \u003cjack@suse.cz\u003e\n" }, { "commit": "1579c3a15c06055713b42b077b805f818638302c", "tree": "8898416461b4451f095ea0a426dd8a9d91dfd95d", "parents": [ "04143e2fb9d512c21e1dcfb561dbb0445dcfdc8c" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Wed Jan 07 18:07:27 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 08 08:31:01 2009 -0800" }, "message": "jbd: remove excess kernel-doc notation\n\nRemove excess kernel-doc from fs/jbd/transaction.c:\n\nWarning(linux-2.6.28-git5//fs/jbd/transaction.c:764): Excess function parameter \u0027credits\u0027 description in \u0027journal_get_write_access\u0027\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "f420d4dc4272fd223986762df2ad06056ddebada", "tree": "2ae50476e901dc5c2e5d189d44785e27234bcce9", "parents": [ "ef8b646183868b2d042fa6cde0eef2a31263ff85" ], "author": { "name": "Josef Bacik", "email": "jbacik@redhat.com", "time": "Wed Jan 07 18:07:24 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 08 08:31:00 2009 -0800" }, "message": "jbd: improve fsync batching\n\nThere is a flaw with the way jbd handles fsync batching. If we fsync() a\nfile and we were not the last person to run fsync() on this fs then we\nautomatically sleep for 1 jiffie in order to wait for new writers to join\ninto the transaction before forcing the commit. The problem with this is\nthat with really fast storage (ie a Clariion) the time it takes to commit\na transaction to disk is way faster than 1 jiffie in most cases, so\nsleeping means waiting longer with nothing to do than if we just committed\nthe transaction and kept going. Ric Wheeler noticed this when using\nfs_mark with more than 1 thread, the throughput would plummet as he added\nmore threads.\n\nThis patch attempts to fix this problem by recording the average time in\nnanoseconds that it takes to commit a transaction to disk, and what time\nwe started the transaction. If we run an fsync() and we have been running\nfor less time than it takes to commit the transaction to disk, we sleep\nfor the delta amount of time and then commit to disk. We acheive\nsub-jiffie sleeping using schedule_hrtimeout. This means that the wait\ntime is auto-tuned to the speed of the underlying disk, instead of having\nthis static timeout. I weighted the average according to somebody\u0027s\ncomments (Andreas Dilger I think) in order to help normalize random\noutliers where we take way longer or way less time to commit than the\naverage. I also have a min() check in there to make sure we don\u0027t sleep\nlonger than a jiffie in case our storage is super slow, this was requested\nby Andrew.\n\nI unfortunately do not have access to a Clariion, so I had to use a\nramdisk to represent a super fast array. I tested with a SATA drive with\nbarrier\u003d1 to make sure there was no regression with local disks, I tested\nwith a 4 way multipathed Apple Xserve RAID array and of course the\nramdisk. I ran the following command\n\nfs_mark -d /mnt/ext3-test -s 4096 -n 2000 -D 64 -t $i\n\nwhere $i was 2, 4, 8, 16 and 32. I mkfs\u0027ed the fs each time. Here are my\nresults\n\ntype\tthreads\t\twith patch\twithout patch\nsata\t2\t\t24.6\t\t26.3\nsata\t4\t\t49.2\t\t48.1\nsata\t8\t\t70.1\t\t67.0\nsata\t16\t\t104.0\t\t94.1\nsata\t32\t\t153.6\t\t142.7\n\nxserve\t2\t\t246.4\t\t222.0\nxserve\t4\t\t480.0\t\t440.8\nxserve\t8\t\t829.5\t\t730.8\nxserve\t16\t\t1172.7\t\t1026.9\nxserve\t32\t\t1816.3\t\t1650.5\n\nramdisk\t2\t\t2538.3\t\t1745.6\nramdisk\t4\t\t2942.3\t\t661.9\nramdisk\t8\t\t2882.5\t\t999.8\nramdisk\t16\t\t2738.7\t\t1801.9\nramdisk\t32\t\t2541.9\t\t2394.0\n\nSigned-off-by: Josef Bacik \u003cjbacik@redhat.com\u003e\nCc: Andreas Dilger \u003cadilger@sun.com\u003e\nCc: Arjan van de Ven \u003carjan@infradead.org\u003e\nCc: Ric Wheeler \u003crwheeler@redhat.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e74481e23283fb080d4591c258de20785cc3b6c3", "tree": "bfc5076aa40d62beeba319fdb0fe55ed53f4b335", "parents": [ "e99c97ade53fb6f5e665f2960eb86c624a532d7b" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Wed Oct 29 14:01:10 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Oct 30 11:38:46 2008 -0700" }, "message": "fs: remove excess kernel-doc\n\nDelete excess kernel-doc notation in fs/ subdirectory:\n\nWarning(linux-2.6.27-git10//fs/jbd/transaction.c:886): Excess function parameter or struct member \u0027credits\u0027 description in \u0027journal_get_undo_access\u0027\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "960a22ae60c8a723bd17da3b929fe0bcea6d007e", "tree": "844e3726fb6bdedd433854966e90116eb1d03c99", "parents": [ "0e4fb5e283870757024294bc4567a7c59d936f0b" ], "author": { "name": "Hidehiro Kawai", "email": "hidehiro.kawai.ez@hitachi.com", "time": "Sat Oct 18 20:27:58 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 20 08:52:37 2008 -0700" }, "message": "jbd: ordered data integrity fix\n\nIn ordered mode, if a file data buffer being dirtied exists in the\ncommitting transaction, we write the buffer to the disk, move it from the\ncommitting transaction to the running transaction, then dirty it. But we\ndon\u0027t have to remove the buffer from the committing transaction when the\nbuffer couldn\u0027t be written out, otherwise it would miss the error and the\ncommitting transaction would not abort.\n\nThis patch adds an error check before removing the buffer from the\ncommitting transaction.\n\nSigned-off-by: Hidehiro Kawai \u003chidehiro.kawai.ez@hitachi.com\u003e\nAcked-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3295f0ef9ff048a4619ede597ad9ec9cab725654", "tree": "f39a8ecf1958130a0b86c554399d23a65b1c3991", "parents": [ "8bfe0298f7a04952d19f4a2cf510d7a6311eeed0" ], "author": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Aug 11 10:30:30 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Aug 11 10:30:30 2008 +0200" }, "message": "lockdep: rename map_[acquire|release]() \u003d\u003e lock_map_[acquire|release]()\n\nthe names were too generic:\n\n drivers/uio/uio.c:87: error: expected identifier or \u0027(\u0027 before \u0027do\u0027\n drivers/uio/uio.c:87: error: expected identifier or \u0027(\u0027 before \u0027while\u0027\n drivers/uio/uio.c:113: error: \u0027map_release\u0027 undeclared here (not in a function)\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "4f3e7524b2e703d9f8b02ac338153a53dd7ede66", "tree": "0033ade94f230d6044ad318e3791c55ad611069e", "parents": [ "f82b217e3513fe3af342c0f3ee1494e86250c21c" ], "author": { "name": "Peter Zijlstra", "email": "a.p.zijlstra@chello.nl", "time": "Mon Aug 11 09:30:23 2008 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Mon Aug 11 09:30:23 2008 +0200" }, "message": "lockdep: map_acquire\n\nMost the free-standing lock_acquire() usages look remarkably similar, sweep\nthem into a new helper.\n\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "3f31fddfa26b7594b44ff2b34f9a04ba409e0f91", "tree": "88994baf22f65dc4da0bef17ce61eda09c59db2a", "parents": [ "9ebfbe9f926553eabc21b4400918d1216b27ed0c" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Fri Jul 25 01:46:22 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jul 25 10:53:32 2008 -0700" }, "message": "jbd: fix race between free buffer and commit transaction\n\njournal_try_to_free_buffers() could race with jbd commit transaction when\nthe later is holding the buffer reference while waiting for the data\nbuffer to flush to disk. If the caller of journal_try_to_free_buffers()\nrequest tries hard to release the buffers, it will treat the failure as\nerror and return back to the caller. We have seen the directo IO failed\ndue to this race. Some of the caller of releasepage() also expecting the\nbuffer to be dropped when passed with GFP_KERNEL mask to the\nreleasepage()-\u003ejournal_try_to_free_buffers().\n\nWith this patch, if the caller is passing the __GFP_WAIT and __GFP_FS to\nindicating this call could wait, in case of try_to_free_buffers() failed,\nlet\u0027s waiting for journal_commit_transaction() to finish commit the\ncurrent committing transaction, then try to free those buffers again.\n\n[akpm@linux-foundation.org: coding-style fixes]\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nReviewed-by: Badari Pulavarty \u003cpbadari@us.ibm.com\u003e\nAcked-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "08fc99bfc39d5b394aa27498ac5f14df0ef8e52a", "tree": "3749ce01f8cb60e3cf08c95f3f4076e21ad62ee7", "parents": [ "e05b6b524bd5c5c2bae1b64a7cbe08d46d57a6fe" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Mon Apr 28 02:16:16 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 28 08:58:45 2008 -0700" }, "message": "jbd: replace remaining __FUNCTION__ occurrences\n\n__FUNCTION__ is gcc-specific, use __func__\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5b9a499d77e9dd39c9e6611ea10c56a31604f274", "tree": "a3a2099c3abe67623aa52b803889d65855b28414", "parents": [ "5bc833feaa8b2236265764e7e81f44937be46eda" ], "author": { "name": "Josef Bacik", "email": "jbacik@redhat.com", "time": "Mon Apr 28 02:16:12 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 28 08:58:44 2008 -0700" }, "message": "jbd: fix possible journal overflow issues\n\nThere are several cases where the running transaction can get buffers added to\nits BJ_Metadata list which it never dirtied, which makes its t_nr_buffers\ncounter end up larger than its t_outstanding_credits counter.\n\nThis will cause issues when starting new transactions as while we are logging\nbuffers we decrement t_outstanding_buffers, so when t_outstanding_buffers goes\nnegative, we will report that we need less space in the journal than we\nactually need, so transactions will be started even though there may not be\nenough room for them. In the worst case scenario (which admittedly is almost\nimpossible to reproduce) this will result in the journal running out of space.\n\nThe fix is to only\nrefile buffers from the committing transaction to the running transactions\nBJ_Modified list when b_modified is set on that journal, which is the only way\nto be sure if the running transaction has modified that buffer.\n\nThis patch also fixes an accounting error in journal_forget, it is possible\nthat we can call journal_forget on a buffer without having modified it, only\ngotten write access to it, so instead of freeing a credit, we only do so if\nthe buffer was modified. The assert will help catch if this problem occurs.\nWithout these two patches I could hit this assert within minutes of running\npostmark, with them this issue no longer arises. Thank you,\n\nSigned-off-by: Josef Bacik \u003cjbacik@redhat.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nAcked-by: Jan Kara \u003cjack@ucw.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5bc833feaa8b2236265764e7e81f44937be46eda", "tree": "9aa34ee4e898d268fade3e6e2b9b5678f49caa54", "parents": [ "269b26191650be46ce6c91dec24cf20f59650529" ], "author": { "name": "Josef Bacik", "email": "jbacik@redhat.com", "time": "Mon Apr 28 02:16:10 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 28 08:58:44 2008 -0700" }, "message": "jbd: fix the way the b_modified flag is cleared\n\nCurrently at the start of a journal commit we loop through all of the buffers\non the committing transaction and clear the b_modified flag (the flag that is\nset when a transaction modifies the buffer) under the j_list_lock.\n\nThe problem is that everywhere else this flag is modified only under the jbd\nlock buffer flag, so it will race with a running transaction who could\npotentially set it, and have it unset by the committing transaction.\n\nThis is also a big waste, you can have several thousands of buffers that you\nare clearing the modified flag on when you may not need to. This patch\nremoves this code and instead clears the b_modified flag upon entering\ndo_get_write_access/journal_get_create_access, so if that transaction does\nindeed use the buffer then it will be accounted for properly, and if it does\nnot then we know we didn\u0027t use it.\n\nThat will be important for the next patch in this series. Tested thoroughly\nby myself using postmark/iozone/bonnie++.\n\nSigned-off-by: Josef Bacik \u003cjbacik@redhat.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nAcked-by: Jan Kara \u003cjack@ucw.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a6b91919e0881a0d0a4ae5211d5c879a8c7ca92b", "tree": "148aabc0a92a297f9220174435e822f30c951215", "parents": [ "5366dc9fd1cb74023790a8edf29e4fe42c886dcf" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Wed Mar 19 17:01:00 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 19 18:53:36 2008 -0700" }, "message": "fs: fix kernel-doc notation warnings\n\nFix kernel-doc notation warnings in fs/.\n\nWarning(mmotm-2008-0314-1449//fs/super.c:560): missing initial short description on line:\n *\tmark_files_ro\nWarning(mmotm-2008-0314-1449//fs/locks.c:1277): missing initial short description on line:\n *\tlease_get_mtime\nWarning(mmotm-2008-0314-1449//fs/locks.c:1277): missing initial short description on line:\n *\tlease_get_mtime\nWarning(mmotm-2008-0314-1449//fs/namei.c:1368): missing initial short description on line:\n * lookup_one_len: filesystem helper to lookup single pathname component\nWarning(mmotm-2008-0314-1449//fs/buffer.c:3221): missing initial short description on line:\n * bh_uptodate_or_lock: Test whether the buffer is uptodate\nWarning(mmotm-2008-0314-1449//fs/buffer.c:3240): missing initial short description on line:\n * bh_submit_read: Submit a locked buffer for reading\nWarning(mmotm-2008-0314-1449//fs/fs-writeback.c:30): missing initial short description on line:\n * writeback_acquire: attempt to get exclusive writeback access to a device\nWarning(mmotm-2008-0314-1449//fs/fs-writeback.c:47): missing initial short description on line:\n * writeback_in_progress: determine whether there is writeback in progress\nWarning(mmotm-2008-0314-1449//fs/fs-writeback.c:58): missing initial short description on line:\n * writeback_release: relinquish exclusive writeback access against a device.\nWarning(mmotm-2008-0314-1449//include/linux/jbd.h:351): contents before sections\nWarning(mmotm-2008-0314-1449//include/linux/jbd.h:561): contents before sections\nWarning(mmotm-2008-0314-1449//fs/jbd/transaction.c:1935): missing initial short description on line:\n * void journal_invalidatepage()\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "0cf01f6685bef41dea37bbec606f375b7a98fd98", "tree": "554cd3af3f58b107c543b5eaae9b8be6db6ec794", "parents": [ "0e82989d95cc46cc58622381eafa54f7428ee679" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Wed Mar 19 17:00:44 2008 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 19 18:53:35 2008 -0700" }, "message": "jbd: fix jbd kernel-doc notation\n\nFix kernel-doc notation in jbd.\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "78a4a50a86b0a54f7ecbc164267b6c762760254c", "tree": "a50503b14fc4f21abdcc0e2c6e49380029bb5e1c", "parents": [ "64e6269071fb1888e5ddc9ad557368cc5fa4f538" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Fri Feb 29 22:02:31 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Mar 03 10:47:13 2008 -0800" }, "message": "docbook: fix filesystems.tmpl source files\n\nFix docbook problems in filesystems.tmpl.\nThese cause the generated docbook to be incorrect.\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "f63dcda197bd71c6565c2121bf70e3d371539f90", "tree": "d0c64161017788d0dddd59687859ac0a67d0ad07", "parents": [ "1d6f4e60e736a00b50ec668ba1a9fe27afb083a3" ], "author": { "name": "Jonas Bonn", "email": "jonas.bonn@gmail.com", "time": "Thu Jan 17 15:21:13 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Jan 17 15:38:59 2008 -0800" }, "message": "jbd: do not try lock_acquire after handle made invalid\n\nThis likely fixes the oops in __lock_acquire reported as:\n\nhttp://www.kerneloops.org/raw.php?rawid\u003d2753\u0026msgid\u003d\nhttp://www.kerneloops.org/raw.php?rawid\u003d2749\u0026msgid\u003d\n\nIn these reported oopses, start_this_handle is returning -EROFS.\n\nSigned-off-by: Jonas Bonn \u003cjonas.bonn@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "8c3478a523a48470eb11a23f01249250684677d9", "tree": "5818d64427bcc1128fc74cc99e8f1fd1c9a54766", "parents": [ "345225c8e4a4adad9eb261db26aebcd3b87055ad" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Thu Oct 18 23:39:20 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 19 11:53:34 2007 -0700" }, "message": "JBD/ext3 cleanups: convert to kzalloc\n\nConvert kmalloc to kzalloc() and get rid of the memset().\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c80544dc0b87bb65038355e7aafdc30be16b26ab", "tree": "176349304bec88a9de16e650c9919462e0dd453c", "parents": [ "0e9663ee452ffce0d429656ebbcfe69417a30e92" ], "author": { "name": "Stephen Hemminger", "email": "shemminger@linux-foundation.org", "time": "Thu Oct 18 03:07:05 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Oct 18 14:37:31 2007 -0700" }, "message": "sparse pointer use of zero as null\n\nGet rid of sparse related warnings from places that use integer as NULL\npointer.\n\n[akpm@linux-foundation.org: coding-style fixes]\nSigned-off-by: Stephen Hemminger \u003cshemminger@linux-foundation.org\u003e\nCc: Andi Kleen \u003cak@suse.de\u003e\nCc: Jeff Garzik \u003cjeff@garzik.org\u003e\nCc: Matt Mackall \u003cmpm@selenic.com\u003e\nCc: Ian Kent \u003craven@themaw.net\u003e\nCc: Arnd Bergmann \u003carnd@arndb.de\u003e\nCc: Davide Libenzi \u003cdavidel@xmailserver.org\u003e\nCc: Stephen Smalley \u003csds@tycho.nsa.gov\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a5005da204289ce01ca37be59e902100ef247a4d", "tree": "4f754d5936a3b8b2d462a343bdd2342967170ac8", "parents": [ "af1e76d6b3f37cb89d9192eaf83588adaf4728eb" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Tue Oct 16 18:38:25 2007 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Oct 17 18:49:57 2007 -0400" }, "message": "JBD: replace jbd_kmalloc with kmalloc directly\n\nThis patch cleans up jbd_kmalloc and replace it with kmalloc directly\n\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\n" }, { "commit": "c089d490dfbf53bc0893dc9ef57cf3ee6448314d", "tree": "8faffea3bdcfdd48ce175ac92d5088ced4f1c969", "parents": [ "d85714d81cc0408daddb68c10f7fd69eafe7c213" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Tue Oct 16 18:38:25 2007 -0400" }, "committer": { "name": "Theodore Ts\u0027o", "email": "tytso@mit.edu", "time": "Wed Oct 17 18:49:56 2007 -0400" }, "message": "JBD: JBD slab allocation cleanups\n\nJBD: Replace slab allocations with page allocations\n\nJBD allocate memory for committed_data and frozen_data from slab. However\nJBD should not pass slab pages down to the block layer. Use page allocator pages instead. This will also prepare JBD for the large blocksize patchset.\n\nSigned-off-by: Christoph Lameter \u003cclameter@sgi.com\u003e\nSigned-off-by: Mingming Cao \u003ccmm@us.ibm.com\u003e\n" }, { "commit": "34a3d1e83708702ac6cb872215e68cd07dae298b", "tree": "bed36f9a4b64a98676776ccff820b7897b1dcb74", "parents": [ "523b44cff279c42c79f7bda709e2fefc30f20a59" ], "author": { "name": "Peter Zijlstra", "email": "a.p.zijlstra@chello.nl", "time": "Thu Oct 11 22:11:12 2007 +0200" }, "committer": { "name": "Peter Zijlstra", "email": "a.p.zijlstra@chello.nl", "time": "Thu Oct 11 22:11:12 2007 +0200" }, "message": "lockdep: annotate journal_start()\n\nOn Fri, 2007-07-13 at 02:05 -0700, Andrew Morton wrote:\n\n\u003e Except lockdep doesn\u0027t know about journal_start(), which has ranking\n\u003e requirements similar to a semaphore. \n\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "5886269962f94fa9185c32db3ec936c612503235", "tree": "2f24dabcf5791319ebb1fbdfb5dbc58284714275", "parents": [ "01afd80626e98c2347bc25be92ee4a3faf314514" ], "author": { "name": "Uwe Kleine-König", "email": "ukleinek@informatik.uni-freiburg.de", "time": "Wed May 09 07:51:49 2007 +0200" }, "committer": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Wed May 09 08:58:16 2007 +0200" }, "message": "fix file specification in comments\n\nMany files include the filename at the beginning, serveral used a wrong one.\n\nSigned-off-by: Uwe Kleine-König \u003cukleinek@informatik.uni-freiburg.de\u003e\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\n" }, { "commit": "e63340ae6b6205fef26b40a75673d1c9c0c8bb90", "tree": "8d3212705515edec73c3936bb9e23c71d34a7b41", "parents": [ "04c9167f91e309c9c4ea982992aa08e83b2eb42e" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Tue May 08 00:28:08 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue May 08 11:15:07 2007 -0700" }, "message": "header cleaning: don\u0027t include smp_lock.h when not used\n\nRemove includes of \u003clinux/smp_lock.h\u003e where it is not used/needed.\nSuggested by Al Viro.\n\nBuilds cleanly on x86_64, i386, alpha, ia64, powerpc, sparc,\nsparc64, and arm (all 59 defconfigs).\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "44d306e1508fef6fa7a6eb15a1aba86ef68389a6", "tree": "84edb1f4d6e1e2d3926f152bad132e037da9024c", "parents": [ "4c36a5dec25fb344ad76b11860da3a8b50bd1248" ], "author": { "name": "Arjan van de Ven", "email": "arjan@linux.intel.com", "time": "Sun Dec 10 02:21:26 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Sun Dec 10 09:57:22 2006 -0800" }, "message": "[PATCH] user of the jiffies rounding code: JBD\n\nThis patch introduces a user: of the round_jiffies() function; the \"5 second\"\next3/jbd wakeup.\n\nWhile \"every 5 seconds\" doesn\u0027t sound as a problem, there can be many of these\n(and these timers do add up over all the kernel). The \"5 second\" wakeup isn\u0027t\nreally timing sensitive; in addition even with rounding it\u0027ll still happen\nevery 5 seconds (with the exception of the very first time, which is likely to\nbe rounded up to somewhere closer to 6 seconds)\n\nSigned-off-by: Arjan van de Ven \u003carjan@linux.intel.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "d394e122bc1adba0f3eb1ebec1cedb8a8c524741", "tree": "3dec0f71b7c2b5f910013ccbeaadd9427737dc4b", "parents": [ "8487f2e4067839ad3d009c240d51b682264320ae" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Wed Dec 06 20:38:26 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Thu Dec 07 08:39:40 2006 -0800" }, "message": "[PATCH] make fs/jbd/transaction.c:__journal_temp_unlink_buffer() static\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "f58a74dca88d48b0669609b4957f3dd757bdc898", "tree": "bfd9a7f078d3d017e92fbd75659f35b619ccf188", "parents": [ "1939e49a0cb9d73785857bf312f4f65661b4b513" ], "author": { "name": "Eric Sandeen", "email": "sandeen@sandeen.net", "time": "Sat Oct 28 10:38:27 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Oct 28 11:30:51 2006 -0700" }, "message": "[PATCH] jbd: journal_dirty_data re-check for unmapped buffers\n\nWhen running several fsx\u0027s and other filesystem stress tests, we found\ncases where an unmapped buffer was still being sent to submit_bh by the\next3 dirty data journaling code.\n\nI saw this happen in two ways, both related to another thread doing a\ntruncate which would unmap the buffer in question.\n\nEither we would get into journal_dirty_data with a bh which was already\nunmapped (although journal_dirty_data_fn had checked for this earlier, the\nstate was not locked at that point), or it would get unmapped in the middle\nof journal_dirty_data when we dropped locks to call sync_dirty_buffer.\n\nBy re-checking for mapped state after we\u0027ve acquired the bh state lock, we\nshould avoid these races. If we find a buffer which is no longer mapped,\nwe essentially ignore it, because journal_unmap_buffer has already decided\nthat this buffer can go away.\n\nI\u0027ve also added tracepoints in these two cases, and made a couple other\ntracepoint changes that I found useful in debugging this.\n\nSigned-off-by: Eric Sandeen \u003cesandeen@redhat.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "3e2a532b26b491706bd8b5c7cfc8d767b43b8f36", "tree": "641db5da4c1adbeee04ce6594746cec51f581ff0", "parents": [ "82591e6ea234762eeaa8b2337fe060ed438c18dc" ], "author": { "name": "OGAWA Hirofumi", "email": "hirofumi@mail.parknet.co.jp", "time": "Thu Oct 19 23:29:11 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Oct 20 10:26:44 2006 -0700" }, "message": "[PATCH] ext3/4: fix J_ASSERT(transaction-\u003et_updates \u003e 0) in journal_stop()\n\nA disk generated some I/O error, after it, I hitted\nJ_ASSERT(transaction-\u003et_updates \u003e 0) in journal_stop().\n\nIt seems to happened on ext3_truncate() path from stack trace. Then,\nmaybe the following case may trigger J_ASSERT(transaction-\u003et_updates \u003e 0).\n\next3_truncate()\n -\u003e ext3_free_branches()\n -\u003e ext3_journal_test_restart()\n\t -\u003e ext3_journal_restart()\n -\u003e journal_restart()\n transaction-\u003et_updates--;\n /* another process aborted journal */\n -\u003e start_this_handle()\n\t\t returns -EROFS without transaction-\u003et_updates++;\n\n -\u003e ext3_journal_stop()\n -\u003e journal_stop()\n\tJ_ASSERT(transaction-\u003et_updates \u003e 0)\n\nIf journal was aborted in middle of journal_restart(), ext3_truncate()\nmay trigger J_ASSERT().\n\nSigned-off-by: OGAWA Hirofumi \u003chirofumi@mail.parknet.co.jp\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "e9ad5620bfb901df8a7a2603c88689ededeecaf1", "tree": "82a2b22e1b7401b98fc2f201b8df2db05764332a", "parents": [ "7543fc7b3abfee8c6cd6349ebd5e5fde02fac984" ], "author": { "name": "Dave Kleikamp", "email": "shaggy@austin.ibm.com", "time": "Wed Sep 27 01:49:35 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Wed Sep 27 08:26:10 2006 -0700" }, "message": "[PATCH] ext3: More whitespace cleanups\n\nMore white space cleanups in preparation of cloning ext4 from ext3.\nRemoving spaces that precede a tab.\n\nSigned-off-by: Dave Kleikamp \u003cshaggy@austin.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "ae6ddcc5f24d6b06ae9231dc128904750a4155e0", "tree": "93c6e20b513f39b616af101dabe9b756f7300d0d", "parents": [ "e7ab8d65055e9b9dfc131d0467cfc5a8368d7ee4" ], "author": { "name": "Mingming Cao", "email": "cmm@us.ibm.com", "time": "Wed Sep 27 01:49:27 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Wed Sep 27 08:26:09 2006 -0700" }, "message": "[PATCH] ext3 and jbd cleanup: remove whitespace\n\nRemove whitespace from ext3 and jbd, before we clone ext4.\n\nSigned-off-by: Mingming Cao\u003ccmm@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "4cfb04a9d3d6865d932d02616b27b89d3a634026", "tree": "c4e50178d972ca9c4c9a90a23db413ed3ef66617", "parents": [ "3b98b087fc2daab67518d2baa8aef19a6ad82723" ], "author": { "name": "Badari Pulavarty", "email": "pbadari@us.ibm.com", "time": "Thu Aug 31 21:27:54 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Sep 01 11:39:10 2006 -0700" }, "message": "[PATCH] manage-jbd-its-own-slab fix\n\nMissed a place where I forgot to convert kfree() to kmem_cache_free() as\npart of jbd-manage-its-own-slab changes.\n\nSigned-off-by: Badari Pulavarty \u003cpbadari@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "ea817398e68dfa25612229fda7fc74580cf915fb", "tree": "f511458f277c30b74e26bed69fe9d36dd2fe857c", "parents": [ "4c4d50f7b39cc58f1064b93a61ad617451ae41df" ], "author": { "name": "Badari Pulavarty", "email": "pbadari@us.ibm.com", "time": "Sun Aug 27 01:23:52 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Aug 27 11:01:32 2006 -0700" }, "message": "[PATCH] Manage jbd allocations from its own slabs\n\nJBD currently allocates commit and frozen buffers from slabs. With\nCONFIG_SLAB_DEBUG, its possible for an allocation to cross the page\nboundary causing IO problems.\n\nhttps://bugzilla.redhat.com/bugzilla/show_bug.cgi?id\u003d200127\n\nSo, instead of allocating these from regular slabs - manage allocation from\nits own slabs and disable slab debug for these slabs.\n\n[akpm@osdl.org: cleanups]\nSigned-off-by: Badari Pulavarty \u003cpbadari@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "304c4c841a31c780a45d65e389b07706babf5d36", "tree": "d5fed361a127e46c1a99d72ef907e1c6b03a1abe", "parents": [ "22722051fb6bcbb95c895e68dad10d34a9db7e4d" ], "author": { "name": "Andrew Morton", "email": "akpm@osdl.org", "time": "Fri Jun 23 02:05:31 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Jun 23 07:43:05 2006 -0700" }, "message": "[PATCH] jbd: avoid kfree(NULL)\n\nThere are a couple of places where JBD has to check to see whether an unneeded\nmemory allocation was performed. Usually it _was_ needed, so we end up\ncalling kfree(NULL). We can micro-optimise that by checking the pointer\nbefore calling kfree().\n\nThanks to Steven Rostedt \u003crostedt@goodmis.org\u003e for identifying this.\n\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "9ada7340987aa24395809570840c7c6847044f52", "tree": "e2a57c184d15cbd2415f4fce35534cf635e9fa9f", "parents": [ "8e0a43d8fa953179505869ec28de78550246e795" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Fri Jun 23 02:05:25 2006 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Jun 23 07:43:04 2006 -0700" }, "message": "[PATCH] jbd: fix BUG in journal_commit_transaction()\n\nFix possible assertion failure in journal_commit_transaction() on\njh-\u003eb_next_transaction \u003d\u003d NULL (when we are processing BJ_Forget list and\nbuffer is not jbddirty).\n\n!jbddirty buffers can be placed on BJ_Forget list for example by\njournal_forget() or by __dispose_buffer() - generally such buffer means\nthat it has been freed by this transaction.\n\nFreed buffers should not be reallocated until the transaction has committed\n(that\u0027s why we have the assertion there) but they *can* be reallocated when\nthe transaction has already been committed to disk and we are just\nprocessing the BJ_Forget list (as soon as we remove b_committed_data from\nthe bitmap bh, ext3 will be able to reallocate buffers freed by the\ncommitting transaction). So we have to also count with the case that the\nbuffer has been reallocated and b_next_transaction has been already set.\n\nAnd one more subtle point: it can happen that we manage to reallocate the\nbuffer and also mark it jbddirty. Then we also add the freed buffer to the\ncheckpoint list of the committing trasaction. But that should do no harm.\n\nNon-jbddirty buffers should be filed to BJ_Reserved and not BJ_Metadata\nlist. It can actually happen that we refile such buffers during the commit\nphase when we reallocate in the running transaction blocks deleted in\ncommitting transaction (and that can happen if the committing transaction\nalready wrote all the data and is just cleaning up BJ_Forget list).\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nAcked-by: \"Stephen C. Tweedie\" \u003csct@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "2ff28e22bdb8727fbc7d7889807bc5a73aae56c5", "tree": "f7418aa963d729bf9fe8bd44d6c9b6e424a6c6bf", "parents": [ "3978d7179d3849848df8a37dd0a5acc20bcb8750" ], "author": { "name": "NeilBrown", "email": "neilb@suse.de", "time": "Sun Mar 26 01:37:18 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Mar 26 08:56:55 2006 -0800" }, "message": "[PATCH] Make address_space_operations-\u003einvalidatepage return void\n\nThe return value of this function is never used, so let\u0027s be honest and\ndeclare it as void.\n\nSome places where invalidatepage returned 0, I have inserted comments\nsuggesting a BUG_ON.\n\n[akpm@osdl.org: JBD BUG fix]\n[akpm@osdl.org: rework for git-nfs]\n[akpm@osdl.org: don\u0027t go BUG in block_invalidate_page()]\nSigned-off-by: Neil Brown \u003cneilb@suse.de\u003e\nAcked-by: Dave Kleikamp \u003cshaggy@austin.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "e3df18983ea090a2e00dd5c2c6167bb431a0e0a2", "tree": "99f7944da7c8c85eed6738c1ef9f357f7dcae928", "parents": [ "1d9b7d97d6661edb44ce08f17e47c66d4ac20e34" ], "author": { "name": "Andrew Morton", "email": "akpm@osdl.org", "time": "Sat Mar 25 03:06:53 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Mar 25 08:22:50 2006 -0800" }, "message": "[PATCH] jbd: embed j_commit_timer in journal struct\n\nThe kjournald timer is currently on the kernel thread\u0027s stack and the journal\nstructure points at it. Save a pointer hop by moving the timer into the\njournal structure.\n\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "2c68ee754c40099c59828e59618a54726f76126a", "tree": "b656a0caa9b31749b5b6f9a3f7a1f3ffc482552c", "parents": [ "f24075bd0c1cd1cc2cf86d394f960aa0401de573" ], "author": { "name": "Arjan van de Ven", "email": "arjan@infradead.org", "time": "Thu Mar 23 03:00:35 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Thu Mar 23 07:38:12 2006 -0800" }, "message": "[PATCH] sem2mutex: jbd, j_checkpoint_mutex\n\nSemaphore to mutex conversion.\n\nThe conversion was generated via scripts, and the result was validated\nautomatically via a script as well.\n\nSigned-off-by: Arjan van de Ven \u003carjan@infradead.org\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "fe1dcbc4f311c2e6c23b33c0fa8572461618ab3e", "tree": "189e935ff275bba20629e46e2832bd523acf6cff", "parents": [ "bc5e483da61eb5ab8d24b4a919fb512e5886d02c" ], "author": { "name": "Andrew Morton", "email": "akpm@osdl.org", "time": "Sat Feb 04 23:27:54 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sun Feb 05 11:06:53 2006 -0800" }, "message": "[PATCH] jbd: fix transaction batching\n\nBen points out that:\n\n When writing files out using O_SYNC, jbd\u0027s 1 jiffy delay results in a\n significant drop in throughput as the disk sits idle. The patch below\n results in a 4-5x performance improvement (from 6.5MB/s to ~24-30MB/s on my\n IDE test box) when writing out files using O_SYNC.\n\nSo optimise the batching code by omitting it entirely if the process which is\ndoing a sync write is the same as the one which did the most recent sync\nwrite. If that\u0027s true, we\u0027re unlikely to get any other processes joining the\ntransaction.\n\n(Has been in -mm for ages - it took me a long time to get on to performance\ntesting it)\n\nNumbers, on write-cache-disabled IDE:\n\n/usr/bin/time -p synctest -n 10 -uf -t 1 -p 1 dir-name\n\nUnpatched:\n\t40 seconds\nPatched:\n\t35 seconds\nBatching disabled:\n\t35 seconds\n\nThis is the problematic single-process-doing-fsync case. With multiple\nfsyncing processes the numbers are AFACIT unaltered by the patch.\n\nAside: performance testing and instrumentation shows that the transaction\nbatching almost doesn\u0027t help (testing with synctest -n 1 -uf -t 100 -p 10\ndir-name on non-writeback-caching IDE). This is because by the time one\nprocess is running a synchronous commit, a bunch of other processes already\nhave a transaction handle open, so they\u0027re all going to batch into the same\ntransaction anyway.\n\nThe batching seems to offer maybe 5-10% speedup with this workload, but I\u0027m\npretty sure it was more important than that when it was first developed 4-odd\nyears ago...\n\nCc: \"Stephen C. Tweedie\" \u003csct@redhat.com\u003e\nCc: Benjamin LaHaise \u003cbcrl@kvack.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "f99d49adf527fa6f7a9c42257fa76bca6b8df1e3", "tree": "41dddbc336016f9dc9557cdb15300de5e599dac1", "parents": [ "6044ec8882c726e325017bd948aa0cd94ad33abc" ], "author": { "name": "Jesper Juhl", "email": "jesper.juhl@gmail.com", "time": "Mon Nov 07 01:01:34 2005 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Mon Nov 07 07:54:06 2005 -0800" }, "message": "[PATCH] kfree cleanup: fs\n\nThis is the fs/ part of the big kfree cleanup patch.\n\nRemove pointless checks for NULL prior to calling kfree() in fs/.\n\nSigned-off-by: Jesper Juhl \u003cjesper.juhl@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "27496a8c67bef4d789d8e3c8317ca35813a507ae", "tree": "fe99a34fe5a800e41af61853e7444ddddf45d014", "parents": [ "7d877f3bda870ab5f001bd92528654471d5966b3" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Oct 21 03:20:48 2005 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Fri Oct 28 08:16:47 2005 -0700" }, "message": "[PATCH] gfp_t: fs/*\n\n - -\u003ereleasepage() annotated (s/int/gfp_t), instances updated\n - missing gfp_t in fs/* added\n - fixed misannotation from the original sweep caught by bitwise checks:\n XFS used __nocast both for gfp_t and for flags used by XFS allocator.\n The latter left with unsigned int __nocast; we might want to add a\n different type for those but for now let\u0027s leave them alone. That,\n BTW, is a case when __nocast use had been actively confusing - it had\n been used in the same code for two different and similar types, with\n no way to catch misuses. Switch of gfp_t to bitwise had caught that\n immediately...\n\nOne tricky bit is left alone to be dealt with later - mapping-\u003eflags is\na mix of gfp_t and error indications. Left alone for now.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "041e0e3b1970c508dc9a95b7dd9dc86271a7d7ac", "tree": "41ff880a87412cf55eb12425e916fda57955ee5c", "parents": [ "373016e9e1353f2af871993d27d00768f08cc883" ], "author": { "name": "Nishanth Aravamudan", "email": "nacc@us.ibm.com", "time": "Sat Sep 10 00:27:23 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Sat Sep 10 10:06:36 2005 -0700" }, "message": "[PATCH] fs: fix-up schedule_timeout() usage\n\nUse schedule_timeout_{,un}interruptible() instead of\nset_current_state()/schedule_timeout() to reduce kernel size. Also use helper\nfunctions to convert between human time units and jiffies rather than constant\nHZ division to avoid rounding errors.\n\nSigned-off-by: Nishanth Aravamudan \u003cnacc@us.ibm.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "4407c2b6b297339e296facf62e020cf66e55053d", "tree": "485d60b1cb5c6013d09a0327355e216b202bd8ed", "parents": [ "e39f07c83bac96850265b87a69dfc5c90ed4f1f5" ], "author": { "name": "Jan Kara", "email": "jack@suse.cz", "time": "Tue Sep 06 15:19:17 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@g5.osdl.org", "time": "Wed Sep 07 16:57:57 2005 -0700" }, "message": "[PATCH] Fix race in do_get_write_access()\n\n attached patch should fix the following race:\n Proc 1 Proc 2\n\n __flush_batch()\n ll_rw_block()\n do_get_write_access()\n\t\t\t\t\t lock_buffer\n jh is only waiting for checkpoint\n\t\t\t\t\t -\u003e b_transaction \u003d\u003d NULL -\u003e\n\t\t\t\t\t do nothing\n unlock_buffer\n test_set_buffer_locked()\n test_clear_buffer_dirty()\n __journal_file_buffer()\n change the data\n submit_bh()\n\nand we have sent wrong data to disk... We now clean the dirty buffer flag\nunder buffer lock in all cases and hence we know that whenever a buffer is\nstarting to be journaled we either finish the pending write-out before\nattaching a buffer to a transaction or we won\u0027t write the buffer until the\ntransaction is going to be committed.\n\nThe test in jbd_unexpected_dirty_buffer() is redundant - remove it.\nFurthermore we have to clear the buffer dirty bit under the buffer lock to\nprevent races with buffer write-out (and hence prevent returning a buffer with\nIO happening).\n\nSigned-off-by: Jan Kara \u003cjack@suse.cz\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "d13df84ff7f3f3e26a9643c1d3cbf94cef9b5b59", "tree": "9531a56973381aa48a10fbc7073d1109f380d550", "parents": [ "19272d4385126c2ac369c9f6137a27a08aee50d1" ], "author": { "name": "akpm@osdl.org", "email": "akpm@osdl.org", "time": "Sat Apr 16 15:26:36 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@ppc970.osdl.org", "time": "Sat Apr 16 15:26:36 2005 -0700" }, "message": "[PATCH] jbd dirty buffer leak fix\n\nThis fixes the lots-of-fsx-linux-instances-cause-a-slow-leak bug.\n\nIt\u0027s been there since 2.6.6, caused by:\n\nftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.5/2.6.5-mm4/broken-out/jbd-move-locked-buffers.patch\n\nThat patch moves under-writeout ordered-data buffers onto a separate journal\nlist during commit. It took out the old code which was based on a single\nlist.\n\nThe old code (necessarily) had logic which would restart I/O against buffers\nwhich had been redirtied while they were on the committing transaction\u0027s\nt_sync_datalist list. The new code only writes buffers once, ignoring\nredirtyings by a later transaction, which is good.\n\nBut over on the truncate side of things, in journal_unmap_buffer(), we\u0027re\ntreating buffers on the t_locked_list as inviolable things which belong to the\ncommitting transaction, and we just leave them alone during concurrent\ntruncate-vs-commit.\n\nThe net effect is that when truncate tries to invalidate a page whose buffers\nare on t_locked_list and have been redirtied, journal_unmap_buffer() just\nleaves those buffers alone. truncate will remove the page from its mapping\nand we end up with an anonymous clean page with dirty buffers, which is an\nillegal state for a page. The JBD commit will not clean those buffers as they\nare removed from t_locked_list. The VM (try_to_free_buffers) cannot reclaim\nthese pages.\n\nThe patch teaches journal_unmap_buffer() about buffers which are on the\ncommitting transaction\u0027s t_locked_list. These buffers have been written and\nI/O has completed. We can take them off the transaction and undirty them\nwithin the context of journal_invalidatepage()-\u003ejournal_unmap_buffer().\n\nAcked-by: \"Stephen C. Tweedie\" \u003csct@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "1da177e4c3f41524e886b7f1b8a0c1fc7321cac2", "tree": "0bba044c4ce775e45a88a51686b5d9f90697ea9d", "parents": [], "author": { "name": "Linus Torvalds", "email": "torvalds@ppc970.osdl.org", "time": "Sat Apr 16 15:20:36 2005 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@ppc970.osdl.org", "time": "Sat Apr 16 15:20:36 2005 -0700" }, "message": "Linux-2.6.12-rc2\n\nInitial git repository build. I\u0027m not bothering with the full history,\neven though we have it. We can create a separate \"historical\" git\narchive of that later if we want to, and in the meantime it\u0027s about\n3.2GB when imported into git - space that would just make the early\ngit days unnecessarily complicated, when we don\u0027t have a lot of good\ninfrastructure for it.\n\nLet it rip!\n" } ] }