)]}' { "log": [ { "commit": "1d957f9bf87da74f420424d16ece005202bbebd3", "tree": "363d4770c0c74a536524c99ccd2762ce96ee9bbe", "parents": [ "4ac9137858e08a19f29feac4e1f4df7c268b0ba5" ], "author": { "name": "Jan Blunck", "email": "jblunck@suse.de", "time": "Thu Feb 14 19:34:35 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 14 21:13:33 2008 -0800" }, "message": "Introduce path_put()\n\n* Add path_put() functions for releasing a reference to the dentry and\n vfsmount of a struct path in the right order\n\n* Switch from path_release(nd) to path_put(\u0026nd-\u003epath)\n\n* Rename dput_path() to path_put_conditional()\n\n[akpm@linux-foundation.org: fix cifs]\nSigned-off-by: Jan Blunck \u003cjblunck@suse.de\u003e\nSigned-off-by: Andreas Gruenbacher \u003cagruen@suse.de\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: \u003clinux-fsdevel@vger.kernel.org\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Steven French \u003csfrench@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": "4ac9137858e08a19f29feac4e1f4df7c268b0ba5", "tree": "f5b5d84fd12fcc2b0ba0e7ce1a79ff381ad8f5dd", "parents": [ "c5e725f33b733a77de622e91b6ba5645fcf070be" ], "author": { "name": "Jan Blunck", "email": "jblunck@suse.de", "time": "Thu Feb 14 19:34:32 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 14 21:13:33 2008 -0800" }, "message": "Embed a struct path into struct nameidata instead of nd-\u003e{dentry,mnt}\n\nThis is the central patch of a cleanup series. In most cases there is no good\nreason why someone would want to use a dentry for itself. This series reflects\nthat fact and embeds a struct path into nameidata.\n\nTogether with the other patches of this series\n- it enforced the correct order of getting/releasing the reference count on\n \u003cdentry,vfsmount\u003e pairs\n- it prepares the VFS for stacking support since it is essential to have a\n struct path in every place where the stack can be traversed\n- it reduces the overall code size:\n\nwithout patch series:\n text data bss dec hex filename\n5321639 858418 715768 6895825 6938d1 vmlinux\n\nwith patch series:\n text data bss dec hex filename\n5320026 858418 715768 6894212 693284 vmlinux\n\nThis patch:\n\nSwitch from nd-\u003e{dentry,mnt} to nd-\u003epath.{dentry,mnt} everywhere.\n\n[akpm@linux-foundation.org: coding-style fixes]\n[akpm@linux-foundation.org: fix cifs]\n[akpm@linux-foundation.org: fix smack]\nSigned-off-by: Jan Blunck \u003cjblunck@suse.de\u003e\nSigned-off-by: Andreas Gruenbacher \u003cagruen@suse.de\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nCc: Casey Schaufler \u003ccasey@schaufler-ca.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "69840b0d065a031a2e5b3fcc3f30560229e312da", "tree": "8fc5f8934a71fd5e219a64462b6b72698e25e180", "parents": [ "aa7fa240c7d4ed28ee2d1afacd97be2d76e3cb49" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Thu Feb 07 00:15:29 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 07 08:42:27 2008 -0800" }, "message": "iget: use iget_failed() in GFS2\n\nUse iget_failed() in GFS2 to kill a failed inode.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nAcked-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e231c2ee64eb1c5cd3c63c31da9dac7d888dcf7f", "tree": "d4b17ef65960594681397a3acac02c2d248200b5", "parents": [ "d1bc8e95445224276d7896b8b08cbb0b28a0ca80" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Thu Feb 07 00:15:26 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 07 08:42:26 2008 -0800" }, "message": "Convert ERR_PTR(PTR_ERR(p)) instances to ERR_CAST(p)\n\nConvert instances of ERR_PTR(PTR_ERR(p)) to ERR_CAST(p) using:\n\nperl -spi -e \u0027s/ERR_PTR[(]PTR_ERR[(](.*)[)][)]/ERR_CAST(\\1)/\u0027 `grep -rl \u0027ERR_PTR[(]*PTR_ERR\u0027 fs crypto net security`\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "eccba068915feece2868c502787037e244db3376", "tree": "eac7790a497c1a33c167605a81ba62b22057b99c", "parents": [ "4cbc76eadf56399cd11fb736b33c53aec9caab8c" ], "author": { "name": "Pavel Emelyanov", "email": "xemul@openvz.org", "time": "Thu Feb 07 00:13:21 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 07 08:42:06 2008 -0800" }, "message": "gfs2: make gfs2_glock.gl_owner_pid be a struct pid *\n\nThe gl_owner_pid field is used to get the lock owning task by its pid, so make\nit in a proper manner, i.e. by using the struct pid pointer and pid_task()\nfunction.\n\nThe pid_task() becomes exported for the gfs2 module.\n\nSigned-off-by: Pavel Emelyanov \u003cxemul@openvz.org\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nAcked-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b1e058da50f7938e9c9e963e978b0730bba4ad32", "tree": "77dbff919c9c7f1d60d69426726c9aad6a2348a3", "parents": [ "488b5ec871191359b9b79262a3d48456dae7ea5f" ], "author": { "name": "Pavel Emelyanov", "email": "xemul@openvz.org", "time": "Thu Feb 07 00:13:19 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Feb 07 08:42:06 2008 -0800" }, "message": "gfs2: make gfs2_holder.gh_owner_pid be a struct pid *\n\nThe gl_owner_pid field is used to get the holder task by its pid and check\nwhether the current is a holder, so make it in a proper manner, i.e. via the\nstruct pid * manipulations.\n\nSigned-off-by: Pavel Emelyanov \u003cxemul@openvz.org\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nAcked-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "eebd2aa355692afaf9906f62118620f1a1c19dbb", "tree": "207eead3a736963c3e50942038c463f2f611ccce", "parents": [ "b98348bdd08dc4ec11828aa98a78edde15c53cfa" ], "author": { "name": "Christoph Lameter", "email": "clameter@sgi.com", "time": "Mon Feb 04 22:28:29 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue Feb 05 09:44:13 2008 -0800" }, "message": "Pagecache zeroing: zero_user_segment, zero_user_segments and zero_user\n\nSimplify page cache zeroing of segments of pages through 3 functions\n\nzero_user_segments(page, start1, end1, start2, end2)\n\n Zeros two segments of the page. It takes the position where to\n start and end the zeroing which avoids length calculations and\n\tmakes code clearer.\n\nzero_user_segment(page, start, end)\n\n Same for a single segment.\n\nzero_user(page, start, length)\n\n Length variant for the case where we know the length.\n\nWe remove the zero_user_page macro. Issues:\n\n1. Its a macro. Inline functions are preferable.\n\n2. The KM_USER0 macro is only defined for HIGHMEM.\n\n Having to treat this special case everywhere makes the\n code needlessly complex. The parameter for zeroing is always\n KM_USER0 except in one single case that we open code.\n\nAvoiding KM_USER0 makes a lot of code not having to be dealing\nwith the special casing for HIGHMEM anymore. Dealing with\nkmap is only necessary for HIGHMEM configurations. In those\nconfigurations we use KM_USER0 like we do for a series of other\nfunctions defined in highmem.h.\n\nSince KM_USER0 is depends on HIGHMEM the existing zero_user_page\nfunction could not be a macro. zero_user_* functions introduced\nhere can be be inline because that constant is not used when these\nfunctions are called.\n\nAlso extract the flushing of the caches to be outside of the kmap.\n\n[akpm@linux-foundation.org: fix nfs and ntfs build]\n[akpm@linux-foundation.org: fix ntfs build some more]\nSigned-off-by: Christoph Lameter \u003cclameter@sgi.com\u003e\nCc: Steven French \u003csfrench@us.ibm.com\u003e\nCc: Michael Halcrow \u003cmhalcrow@us.ibm.com\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nCc: Anton Altaparmakov \u003caia21@cantab.net\u003e\nCc: Mark Fasheh \u003cmark.fasheh@oracle.com\u003e\nCc: David Chinner \u003cdgc@sgi.com\u003e\nCc: Michael Halcrow \u003cmhalcrow@us.ibm.com\u003e\nCc: Steven French \u003csfrench@us.ibm.com\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Trond Myklebust \u003ctrond.myklebust@fys.uio.no\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c78bad11fbf1272ea021f56458025dc98486d6f4", "tree": "7ac1cc64d3429c3b3c52e707212d5a0711a9a9d1", "parents": [ "ee0fc097ef47a4a6ff6b4800f2391030131b7828" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Sun Feb 03 17:33:42 2008 +0200" }, "committer": { "name": "Adrian Bunk", "email": "bunk@kernel.org", "time": "Sun Feb 03 17:33:42 2008 +0200" }, "message": "fs/: Spelling fixes\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: Adrian Bunk \u003cbunk@kernel.org\u003e\n" }, { "commit": "e07dd2ad305f6b29b47d713600aa8b722ef2a9f7", "tree": "4815808e538ec625bf2766b1eb9d91c7b3beaead", "parents": [ "eba0e319c12fb098d66316a8eafbaaa9174a07c3", "7bc5c414fe6627ec518c82d154c796f0981f5b02" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 25 08:39:18 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 25 08:39:18 2008 -0800" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw: (56 commits)\n [GFS2] Allow journal recovery on read-only mount\n [GFS2] Lockup on error\n [GFS2] Fix page_mkwrite truncation race path\n [GFS2] Fix typo\n [GFS2] Fix write alloc required shortcut calculation\n [GFS2] gfs2_alloc_required performance\n [GFS2] Remove unneeded i_spin\n [GFS2] Reduce inode size by moving i_alloc out of line\n [GFS2] Fix assert in log code\n [GFS2] Fix problems relating to execution of files on GFS2\n [GFS2] Initialize extent_list earlier\n [GFS2] Allow page migration for writeback and ordered pages\n [GFS2] Remove unused variable\n [GFS2] Fix log block mapper\n [GFS2] Minor correction\n [GFS2] Eliminate the no longer needed sd_statfs_mutex\n [GFS2] Incremental patch to fix compiler warning\n [GFS2] Function meta_read optimization\n [GFS2] Only fetch the dinode once in block_map\n [GFS2] Reorganize function gfs2_glmutex_lock\n ...\n" }, { "commit": "7bc5c414fe6627ec518c82d154c796f0981f5b02", "tree": "affd4857934df6c86b6fde572de0370700e6fe9a", "parents": [ "1b8177ec1e779bcc3ed89419ff7c80dbc3dcc489" ], "author": { "name": "Abhijith Das", "email": "adas@redhat.com", "time": "Fri Jan 18 14:06:37 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:21:22 2008 +0000" }, "message": "[GFS2] Allow journal recovery on read-only mount\n\nThis patch allows gfs2 to perform journal recovery even if it is mounted\nread-only. Strictly speaking, a read-only mount should not be writing to\nthe filesystem, but we do this only to perform journal recovery. A\nread-only mount will fail if we don\u0027t recover the dirty journal. Also,\nwhen gfs2 is used as a root filesystem, it will be mounted read-only\nbefore being mounted read-write during the boot sequence. A failed\nread-only mount will panic the machine during bootup.\n\nSigned-off-by: Abhijith Das \u003cadas@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1b8177ec1e779bcc3ed89419ff7c80dbc3dcc489", "tree": "f0bd7d9446dd5d6a90eada749ebb129caa4c6c0a", "parents": [ "b7fe2e391ee7b711d6dfd6a694d60c4f21113cbb" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Sat Jan 19 21:50:24 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:21:04 2008 +0000" }, "message": "[GFS2] Lockup on error\n\nI spotted this bug while I was digging around. Looks like it could cause\na lockup in some rare error condition.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b7fe2e391ee7b711d6dfd6a694d60c4f21113cbb", "tree": "4a5d141ffd4ae77dc6ce1a47b0bb8ce9d6d7c685", "parents": [ "3e5cd0877e6d2f059dc36b8206cb7e93938151db" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Jan 17 15:12:03 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:20:15 2008 +0000" }, "message": "[GFS2] Fix page_mkwrite truncation race path\n\nThere was a bug in the truncation/invalidation race path for\n-\u003epage_mkwrite for gfs2. It ought to return 0 so that the effect is the\nsame as if the page was truncated at any of the other points at which\nthe page_lock is dropped. This will result in the restart of the whole\npage fault path. If it was due to a real truncation (as opposed to an\ninvalidate because we let a glock go) then the -\u003efault path will pick\nthat up when it gets called again.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3e5cd0877e6d2f059dc36b8206cb7e93938151db", "tree": "ac4de94b84c94d06864c7256bbc50dd264a62998", "parents": [ "1af535727bbf68e1da7ac232de47315da4c66ade" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Wed Jan 16 08:45:39 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:19:51 2008 +0000" }, "message": "[GFS2] Fix typo\n\nThis patch fixes a minor typo. Surprisingly, it still compiled.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1af535727bbf68e1da7ac232de47315da4c66ade", "tree": "ab24a9bc1271179fdae558d4f66e7ee0e401ce4c", "parents": [ "05220535196d413db434527a3edcba79b7187df8" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Jan 16 14:24:05 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:19:28 2008 +0000" }, "message": "[GFS2] Fix write alloc required shortcut calculation\n\nThe comparison was being made against the wrong quantity.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "05220535196d413db434527a3edcba79b7187df8", "tree": "5811b6f1caeac55102afb993f0b1538bbe9ed2b8", "parents": [ "598278bd4808ed81b0e6fa445458a7d549f72a32" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Fri Jan 11 13:44:50 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:19:03 2008 +0000" }, "message": "[GFS2] gfs2_alloc_required performance\n\nThis is a small I/O performance enhancement to gfs2. (Actually, it is a rework of\nan earlier version I got wrong). The idea here is to check if the write extends\npast the last block in the file. If so, the function can save itself a lot of\ntime and trouble because it knows an allocate will be required. Benchmarks like\niozone should see better performance.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "598278bd4808ed81b0e6fa445458a7d549f72a32", "tree": "92a49c8fc86b2d87e853cc5a5c5407d1c8d6ae7e", "parents": [ "6dbd822487d0a9f14432cb4680415b80656b63a2" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Fri Jan 11 13:31:12 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:18:44 2008 +0000" }, "message": "[GFS2] Remove unneeded i_spin\n\nThis patch removes a vestigial variable \"i_spin\" from the gfs2_inode\nstructure. This not only saves us memory (\u003e300000 of these in memory\nfor the oom test) it also saves us time because we don\u0027t have to\nspend time initializing it (i.e. slightly better performance).\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "6dbd822487d0a9f14432cb4680415b80656b63a2", "tree": "f0391d598c27cd7c39c67cfa13799a784f4c389a", "parents": [ "ac39aadd0440ae696e6dacaa8006ce1737b17008" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Jan 10 15:18:55 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:18:25 2008 +0000" }, "message": "[GFS2] Reduce inode size by moving i_alloc out of line\n\nIt is possible to reduce the size of GFS2 inodes by taking the i_alloc\nstructure out of the gfs2_inode. This patch allocates the i_alloc\nstructure whenever its needed, and frees it afterward. This decreases\nthe amount of low memory we use at the expense of requiring a memory\nallocation for each page or partial page that we write. A quick test\nwith postmark shows that the overhead is not measurable and I also note\nthat OCFS2 use the same approach.\n\nIn the future I\u0027d like to solve the problem by shrinking down the size\nof the members of the i_alloc structure, but for now, this reduces the\nimmediate problem of using too much low-memory on x86 and doesn\u0027t add\ntoo much overhead.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "ac39aadd0440ae696e6dacaa8006ce1737b17008", "tree": "d2a6784809c6bec674879dc865de4150e8b0fb01", "parents": [ "9656b2c14c6ee0806c90a6be41dec71117fc8f50" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Jan 10 14:49:43 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:18:03 2008 +0000" }, "message": "[GFS2] Fix assert in log code\n\nAlthough the values were all being calculated correctly, there was a\nrace in the assert due to the way it was using atomic variables. This\nchanges the value we assert on so that we get the same effect by testing\na different variable. This prevents the assert triggering when it shouldn\u0027t.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9656b2c14c6ee0806c90a6be41dec71117fc8f50", "tree": "3e093571e695bf780885f36caec9594392da31fc", "parents": [ "0811a127cb83ad2e0355e5e3e30164d7ef0f2d65" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Jan 08 08:14:30 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:17:31 2008 +0000" }, "message": "[GFS2] Fix problems relating to execution of files on GFS2\n\nThis patch fixes a couple of problems which affected the execution of files\non GFS2. The first is that there was a corner case where inodes were not\nalways uptodate at the point at which permissions checks were being carried\nout, this was resulting in refusal of execute permission, but only on the\nfirst lookup, subsequent requests worked correctly. The second was a problem\nrelating to incorrect updating of file sizes which was introduced with the\nwrite_begin/end code for GFS2 a little while back.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Abhijith Das \u003cadas@redhat.com\u003e\n" }, { "commit": "0811a127cb83ad2e0355e5e3e30164d7ef0f2d65", "tree": "f1e7eacb1a8a30e47299137db25be673621b9ee0", "parents": [ "e5d9dc278c7f79c220e4506cc1ade2efa2ca73fd" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Thu Jan 03 09:24:53 2008 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:17:04 2008 +0000" }, "message": "[GFS2] Initialize extent_list earlier\n\nHere is a patch for the latest upstream GFS2 code:\nThe journal extent map needs to be initialized sooner than it\ncurrently is. Otherwise failed mount attempts (e.g. not enough\njournals, etc.) may panic trying to access the uninitialized list.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "e5d9dc278c7f79c220e4506cc1ade2efa2ca73fd", "tree": "f984562ccb3b29f31fa1e63d4dfe1f01d8e74be7", "parents": [ "65a6290998f3d38b5c5e84423ae9e08bdd957095" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Jan 03 11:31:38 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:16:41 2008 +0000" }, "message": "[GFS2] Allow page migration for writeback and ordered pages\n\nTo improve performance on NUMA, we use the VM\u0027s standard page\nmigration for writeback and ordered pages. Probably we could\nalso do the same for journaled data, but that would need a\ncareful audit of the code, so will be the subject of a later\npatch.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "65a6290998f3d38b5c5e84423ae9e08bdd957095", "tree": "2263af7252e61281429115d63eaa8e019224e678", "parents": [ "ff91cc9bb41b62bc4ea7d5ced396fabf97539df9" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Jan 02 10:16:56 2008 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:16:19 2008 +0000" }, "message": "[GFS2] Remove unused variable\n\nThe go_drop_th function is never called or referenced.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "ff91cc9bb41b62bc4ea7d5ced396fabf97539df9", "tree": "8e14975bba539c796cddb30853d76f0a0c2d060e", "parents": [ "fa3742fa8545df20e54aa0953a1873cca3a9bd92" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Dec 14 14:04:34 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:15:58 2008 +0000" }, "message": "[GFS2] Fix log block mapper\n\nA missing offset in the calculation.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "fa3742fa8545df20e54aa0953a1873cca3a9bd92", "tree": "12a52441aa753408af536919583ad3cfe59848ec", "parents": [ "c3f60b6e3a7667f78a63b15cf09655ecfca757fc" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Wed Dec 12 17:52:13 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:15:37 2008 +0000" }, "message": "[GFS2] Minor correction\n\nThis is a small correction to my previously posted patch1.\nIt just changes a divide to a shift. It\u0027s faster and doesn\u0027t\nintroduce odd dependencies on 32-bit compiles.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c3f60b6e3a7667f78a63b15cf09655ecfca757fc", "tree": "8dbe2656098c1fa69a9e149248533e1fba131ac5", "parents": [ "b3513fca7e41965d85125c9770ce5f8fd4ff509a" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Wed Dec 12 11:44:41 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:15:16 2008 +0000" }, "message": "[GFS2] Eliminate the no longer needed sd_statfs_mutex\n\nThis patch eliminates the unneeded sd_statfs_mutex mutex but preserves\nthe ordering as discussed.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b3513fca7e41965d85125c9770ce5f8fd4ff509a", "tree": "3747019335ccd0f91bf30bd4a85609db455d7214", "parents": [ "15c7cee7995a9013f1b2f31a15b70e1d2e8ae501" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Wed Dec 12 09:24:08 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:14:53 2008 +0000" }, "message": "[GFS2] Incremental patch to fix compiler warning\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "15c7cee7995a9013f1b2f31a15b70e1d2e8ae501", "tree": "22646442ffd942d77fc34f168d270e7bd2668643", "parents": [ "b0d5fd307463405fe1f57494fbb37f810715ed6d" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 19:29:17 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:14:33 2008 +0000" }, "message": "[GFS2] Function meta_read optimization\n\nThis patch optimizes function gfs2_meta_read. Basically, gfs2_meta_wait\nwas being called regardless of whether a disk read was requested.\nThis just pulls that wait into the if that triggers the read.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b0d5fd307463405fe1f57494fbb37f810715ed6d", "tree": "2a627d7362a6db7eea4c388b0e0f0ceed8362fb5", "parents": [ "398bbe68321947f6763fbc259a01eb548ce19408" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 19:16:09 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:14:13 2008 +0000" }, "message": "[GFS2] Only fetch the dinode once in block_map\n\nFunction gfs2_block_map was often looking up the disk inode twice.\nThis optimizes it so that only does it once.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "398bbe68321947f6763fbc259a01eb548ce19408", "tree": "64cf07eb0f461aa5d26c2abdc873c8727d8a2bad", "parents": [ "5fdc2eeb5d1d3800367f471690b01fcd1fd5b963" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 19:13:54 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:13:52 2008 +0000" }, "message": "[GFS2] Reorganize function gfs2_glmutex_lock\n\nThis patch optimizes the function gfs2_glmutex_lock.\nThe basic theory is: Why bother initializing a holder, setting up\nwait bits and then waiting on them, if you know the glock can be\nyours. So the holder stuff is placed inside the if checking if the\nglock is locked. This one needs careful scrutiny because changing\nanything to do with locking should strike terror into one\u0027s heart.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "5fdc2eeb5d1d3800367f471690b01fcd1fd5b963", "tree": "d1e138b18a74541570736f933d938aa7f6e2d376", "parents": [ "0d0868bde33273a200b33e54f4fad6099ad0c566" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 19:00:16 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:13:31 2008 +0000" }, "message": "[GFS2] Run through full bitmaps quicker in gfs2_bitfit\n\nI eliminated the passing of an unused parameter into gfs2_bitfit called rgd.\n\nThis also changes the gfs2_bitfit code that searches for free (or used) blocks.\nBefore, the code was trying to check for bytes that indicated 4 blocks in\nthe undesired state. The problem is, it was spending more time trying to\ndo this than it actually was saving. This version only optimizes the case\nwhere we\u0027re looking for free blocks, and it checks a machine word at a time.\nSo on 32-bit machines, it will check 32-bits (16 blocks) and on 64-bit\nmachines, it will check 64-bits (32 blocks) at a time. The compiler\noptimizes that quite well and we save some time, especially when running\nthrough full bitmaps (like the bitmaps allocated for the journals).\n\nThere\u0027s probably a more elegant or optimized way to do this, but I haven\u0027t\nthought of it yet. I\u0027m open to suggestions.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "0d0868bde33273a200b33e54f4fad6099ad0c566", "tree": "482f3c72e582ce48b54c67760ca21e190f278ed4", "parents": [ "da6dd40d59fa9617ed697b90114e197036901632" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 18:51:25 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:13:01 2008 +0000" }, "message": "[GFS2] Get rid of useless \"found\" variable in quota.c\n\nThis just eliminates an unused variable from the quota code.\nNot likely to be a time saver.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "da6dd40d59fa9617ed697b90114e197036901632", "tree": "49e869021ed1f911bf3cdf185e9c4ce75c67f42a", "parents": [ "e9e1ef2b6ee401d7c1e1eb38052857b4b206d172" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Tue Dec 11 18:49:21 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:11:46 2008 +0000" }, "message": "[GFS2] Journal extent mapping\n\nThis patch saves a little time when gfs2 writes to the journals by\nkeeping a mapping between logical and physical blocks on disk.\nThat\u0027s better than constantly looking up indirect pointers in\nbuffers, when the journals are several levels of indirection\n(which they typically are).\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "e9e1ef2b6ee401d7c1e1eb38052857b4b206d172", "tree": "68dff93cf4058eb06b162d800f08ff7752cd1a55", "parents": [ "2066b58b0a038d7aedd24133677efb8856cac3a1" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Mon Dec 10 14:13:27 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:25 2008 +0000" }, "message": "[GFS2] Remove function gfs2_get_block\n\nThis patch is just a cleanup. Function gfs2_get_block() just calls\nfunction gfs2_block_map reversing the last two parameters. By\nreversing the parameters, gfs2_block_map() may be called directly\nand function gfs2_get_block may be eliminated altogether.\nSince this function is done for every block operation,\nthis streamlines the code and makes it a little bit more efficient.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "2066b58b0a038d7aedd24133677efb8856cac3a1", "tree": "0afb8a5ee7a9c1a0e4f618b818f8bd05ccd8c3a9", "parents": [ "dbee2199c37336e89060fbe9abdfd1ca8454372a" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Thu Dec 06 09:35:25 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:23 2008 +0000" }, "message": "[GFS2] use pid for plock owner for nfs clients\n\nThe fl_owner is that of lockd when posix locks arrive from nfs\nclients, so it can\u0027t be used to distinguish between lock holders.\nUse fl_pid as owner instead; it\u0027s the pid of the process on the\nnfs client.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "dbee2199c37336e89060fbe9abdfd1ca8454372a", "tree": "9005d78b7e6064016b8bc6c8ab8ae79442eb80ff", "parents": [ "292c8c14cace19c94c6abe25506310239daf949e" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Nov 30 08:17:15 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:20 2008 +0000" }, "message": "[GFS2] Remove unused variable\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "292c8c14cace19c94c6abe25506310239daf949e", "tree": "3b1b1407e00abc154768dc2f5a684b0fcf0cbd1f", "parents": [ "c97bfe4351771675963e02f34d31e206fd2d7150" ], "author": { "name": "Abhijith Das", "email": "adas@redhat.com", "time": "Thu Nov 29 14:13:54 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:18 2008 +0000" }, "message": "[GFS2] patch to check for recursive lock requests in gfs2_rename code path\n\nA certain scenario in the rename code path triggers a kernel BUG()\nbecause it accidentally does recursive locking The first lock is\nrequested to unlink an already existing inode (replacing a file) and the\nsecond lock is requested when the destination directory needs to alloc\nsome space. It is rare that these two\nevents happen during the same rename call, and even more rare that these\ntwo instances try to lock the same rgrp. It is, however, possible.\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d404711\n\nSigned-off-by: Abhijith Das \u003cadas@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c97bfe4351771675963e02f34d31e206fd2d7150", "tree": "9ee4de86cacc54f3e49135a4ce39cf7a236efb31", "parents": [ "bcd405599faa16cf32a3d3f1ce6a1e12cb37fede" ], "author": { "name": "Wendy Cheng", "email": "wcheng@redhat.com", "time": "Thu Nov 29 17:56:51 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:15 2008 +0000" }, "message": "[GFS2] Remove lock methods for lock_nolock protocol\n\nGFS2 supports two modes of locking - lock_nolock for single node filesystem\nand lock_dlm for cluster mode locking. The gfs2 lock methods are removed from\nfile operation table for lock_nolock protocol. This would allow VFS to handle\nposix lock and flock logics just like other in-tree filesystems without\nduplication.\n\nSigned-off-by: S. Wendy Cheng \u003cwcheng@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "bcd405599faa16cf32a3d3f1ce6a1e12cb37fede", "tree": "216771f3a69086c8ccafa58a6d4449926f455336", "parents": [ "6a69a23f7df18f39e4a084e10b62ff4a144b05a5" ], "author": { "name": "Fabio M. Di Nitto", "email": "fabbione@ubuntu.com", "time": "Wed Nov 28 16:22:09 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:13 2008 +0000" }, "message": "[GFS2] Remove unrequired code\n\nSigned-off-by: Fabio M. Di Nitto \u003cfabbione@ubuntu.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "6a69a23f7df18f39e4a084e10b62ff4a144b05a5", "tree": "d5bc744a43f2f05aea7a419b3c93ea314397d311", "parents": [ "002ef1dc63ded14507c110d3cf83d0c3f51374ab" ], "author": { "name": "Fabio Massimo Di Nitto", "email": "fabbione@ubuntu.com", "time": "Tue Nov 27 06:16:42 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:11 2008 +0000" }, "message": "[GFS2] Fix build warnings\n\nHi Steven,\n\nSteven Whitehouse wrote:\n\u003e Hi,\n\u003e\n\u003e Now in the -nmw git tree. Thanks,\n\u003e\n\u003e Steve.\n\u003e\n\u003e On Wed, 2007-11-21 at 11:54 -0600, Ryan O\u0027Hara wrote:\n\nthis patch introduces a bunch of build warnings by leaving around\n\nstruct inode *inode \u003d \u0026ip-\u003ei_inode;\n\nThe patch in attachment cleans them up. Please apply.\n\nSigned-off-by: Fabio Massimo Di Nitto \u003cfabbione@ubuntu.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "002ef1dc63ded14507c110d3cf83d0c3f51374ab", "tree": "c81a66cdd0b8bf49a515141c669a2600e7b01140", "parents": [ "1a2781cfa5ed8eb82bb311d684f268c1822dae69" ], "author": { "name": "Ryan O\u0027Hara", "email": "rohara@redhat.com", "time": "Wed Nov 21 11:54:54 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:08 2008 +0000" }, "message": "[GFS2] remove unnecessary permission checks\n\nRemove read/write permission() checks from xattr operations.\nVFS layer is already handling permission for xattrs via the\nxattr_permission() call, so there is no need for gfs2 to\ncheck permissions. Futhermore, using permission() for SELinux\nxattrs ops is incorrect.\n\nSigned-off-by: Ryan O\u0027Hara \u003crohara@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1a2781cfa5ed8eb82bb311d684f268c1822dae69", "tree": "d65a551d4e0b8b12365fd150fe70cfd9f14f80e2", "parents": [ "00c134756c5ad570a1ad3d6f93a67fc9c25a67ea" ], "author": { "name": "Fabio Massimo Di Nitto", "email": "fabbione@ubuntu.com", "time": "Fri Nov 16 09:50:40 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:06 2008 +0000" }, "message": "[GFS2] Fix runtime issue with UP kernels\n\nThe issue is indeed UP vs SMP and it is totally random.\n\nspin_is_locked() is a bad assertion because there is no correct answer on UP.\non UP spin_is_locked() has to return either one value or another, always.\n\nThis means that in my setup I am lucky enough to trigger the issue and your you\nare lucky enough not to.\n\nthe patch in attachment removes the bogus calls to BUG_ON and according to David\n(in CC and thanks for the long explanation on the problem) we can rely upon\nthings like lockdep to find problem that might be trying to catch.\n\nSigned-off-by: Fabio M. Di Nitto \u003cfabbione@ubuntu.com\u003e\nCc: David S. Miller \u003cdavem@davemloft.net\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "00c134756c5ad570a1ad3d6f93a67fc9c25a67ea", "tree": "36176240919582d17d4655a1838e1b934ede9cb8", "parents": [ "0b7580c786a5feda6291fe68ead3a1b92b6b35b8" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Thu Nov 15 09:01:13 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:04 2008 +0000" }, "message": "[GFS2] tidy up error message\n\nPrint error with log_error() to be consistent with others.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Fabio M. Di Nitto \u003cfabbione@ubuntu.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "0b7580c786a5feda6291fe68ead3a1b92b6b35b8", "tree": "baa74404abb5166fd7a3f3601c724ff1d9fc94db", "parents": [ "e35b921185728850c5db3b5d5b356178f931a157" ], "author": { "name": "Fabio Massimo Di Nitto", "email": "fabbione@ubuntu.com", "time": "Thu Nov 15 13:48:52 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:08:01 2008 +0000" }, "message": "[GFS2] Check for installation of mount helpers for DLM mounts\n\nThe patch is a fix to abort mount if the mount.gfs* and possible\numount.* are missing from /sbin.\n\nWhile we do what we can to guarantee that they are installed properly in\nuserland (CVS HEAD), we want to make sure that mount still aborts properly.\n\nThe only sign of missing helpers is that lock_dlm will receive no mount options\nat all. According to David the problem does not exist for lock_nolock as the\nhelpers are not required.\n\nThe patch has been tested for both gfs and gfs2 and it works as expected. The\nlack of mount.gfs* will generate an error that is propagated to mount:\n\noot@node1:~# mount -t gfs2 /dev/nbd2 /mnt/\nmount: wrong fs type, bad option, bad superblock on /dev/nbd2,\n missing codepage or helper program, or other error\n In some cases useful info is found in syslog - try\n dmesg | tail or so\n\n[ 3513.303346] GFS2: fsid\u003d: Trying to join cluster \"lock_dlm\", \"gutsy:gfs2\"\n[ 3513.304546] DLM/GFS2/GFS ERROR: (u)mount helpers are not installed properly!\n[ 3513.306290] GFS2: fsid\u003d: can\u0027t mount proto\u003dlock_dlm, table\u003dgutsy:gfs2, hostdata\u003d\n\nYou might want to notice that it will also avoid mount to hang or fail silently\nor with strange errors that will require the cluster to reboot/restart before\nyou can actually mount the filesystem again.\n\nSigned-off-by: Fabio M. Di Nitto \u003cfabbione@ubuntu.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "e35b921185728850c5db3b5d5b356178f931a157", "tree": "1f5586886b1a81fe961425da3ccc9fde8f40b538", "parents": [ "ec69b188837a347769e187997d040e84a683b38a" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Nov 09 10:07:21 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:59 2008 +0000" }, "message": "[GFS2] Don\u0027t periodically update the jindex\n\nWe only care about the content of the jindex in two cases,\none is when we mount the fs and the other is when we need\nto recover another journal. In both cases we have to update\nthe jindex anyway, so there is no point in updating it\nperiodically between times, so this removes it to simplify\ngfs2_logd.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "ec69b188837a347769e187997d040e84a683b38a", "tree": "d16ebc9b83807f99a114efe3a093315f9839b81c", "parents": [ "fd041f0b4045db8646b36d393cbb274db60649f5" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Nov 09 10:01:41 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:56 2008 +0000" }, "message": "[GFS2] Move gfs2_logd into log.c\n\nThis means that we can mark gfs2_ail1_empty static and prepares\nthe way for further changes.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "fd041f0b4045db8646b36d393cbb274db60649f5", "tree": "dd09e7317eb471d45c685e3269bd3c20dc686f3e", "parents": [ "2bcd610d2fdea608a8fdac32788fc35a32a2327c" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 08 14:55:03 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:54 2008 +0000" }, "message": "[GFS2] Use atomic_t for journal free blocks counter\n\nThis patch changes the counter which keeps track of the free\nblocks in the journal to an atomic_t in preparation for the\nfollowing patch which will update the log reservation code.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "2bcd610d2fdea608a8fdac32788fc35a32a2327c", "tree": "5b3753ff18c1da54bb860dbd67211e6abea78ca7", "parents": [ "8cbc4342478311c2a85260a7ca54d96cb7f71f7b" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 08 14:25:12 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:52 2008 +0000" }, "message": "[GFS2] Don\u0027t add glocks to the journal\n\nThe only reason for adding glocks to the journal was to keep track\nof which locks required a log flush prior to release. We add a\nflag to the glock to allow this check to be made in a simpler way.\n\nThis reduces the size of a glock (by 12 bytes on i386, 24 on x86_64)\nand means that we can avoid extra work during the journal flush.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8cbc4342478311c2a85260a7ca54d96cb7f71f7b", "tree": "23a8ab6a4a2c154274d01cfe0e69715cd7ad4d86", "parents": [ "c7227e46423a57b4df27a2d75b5869bd3ae654d0" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Nov 07 09:03:56 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:49 2008 +0000" }, "message": "[GFS2] check kthread_should_stop when waiting\n\nUse wait_event_interruptible() in the lock_dlm thread instead\nof an open coded equivalent, and include a kthread_should_stop()\ncheck in the wait test so we don\u0027t miss a kthread_stop().\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c7227e46423a57b4df27a2d75b5869bd3ae654d0", "tree": "e0d1ce8a49d766f119358a721c38837776dcdaf5", "parents": [ "e589665eb97b297412fb16b4c1737a01a91db903" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Fri Nov 02 09:37:15 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:47 2008 +0000" }, "message": "[GFS2] Given device ID rather than s_id in \"id\" sysfs file\n\nThis patch changes the /sys/fs/gfs2/\u003cs_id\u003e/id file to give the device\nid \"major:minor\" rather than the s_id. That enables gfs2_tool to\nmatch devices properly (by id, not name) when locating the tuning files.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "e589665eb97b297412fb16b4c1737a01a91db903", "tree": "139a302a47bda6241d40a61bec1d7239de0e7108", "parents": [ "3042a2ccd68d2b609d283219e51cba363aa35c1d" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Nov 02 09:14:31 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:44 2008 +0000" }, "message": "[GFS2] Remove flags no longer required\n\nThe HIF_MUTEX and HIF_PROMOTE flags were set on the glock holders\ndepending upon which of the two waiters lists they were going to\nbe queued upon. They were then tested when the holders were taken\noff the lists to ensure that the right type of holder was being\ndequeued.\n\nSince we are already using separate lists, there doesn\u0027t seem a\nlot of point having these flags as well, and since setting them\nand testing them is in the fast path for locking and unlocking\nglock, this patch removes them.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3042a2ccd68d2b609d283219e51cba363aa35c1d", "tree": "032653f2111bf20c1f4610d3801c42020c3a1abd", "parents": [ "52d4c74b08bf859f698ddb4e8a43c0dc8d4a0685" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Nov 02 08:39:34 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:42 2008 +0000" }, "message": "[GFS2] Reorder writeback for glock sync\n\nPreviously we were doing (write data, wait for data, write metadata, wait\nfor metadata). After this patch we so (write metadata, write data, wait for\ndata, wait for metadata) which should be more efficient.\n\nAlso I noticed that the drop_bh and xmote_bh functions were almost\nidentical. In fact the only difference was a single test, and that\ntest is such that in the drop_bh case, it would always evaluate to\nthe correct result. As such we can use the xmote_bh functions in\nall the places where we were using the drop_bh function and remove\nthe drop_bh functions.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "52d4c74b08bf859f698ddb4e8a43c0dc8d4a0685", "tree": "3454c250947bc4b19ee5410345d00fef20c98c8d", "parents": [ "c2932e03dbcfe7ea9052953dbd5f3157183c1e9b" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 01 09:34:14 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:40 2008 +0000" }, "message": "[GFS2] Add sync_page to metadata address space operations\n\nThis set of address space operations was missing a sync_page\noperation.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c2932e03dbcfe7ea9052953dbd5f3157183c1e9b", "tree": "4d72603f5fc136a0ad0fd65b423045d5d30e69ef", "parents": [ "60b0d0877986b8fa70148f06055422d2ed858e88" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 01 09:26:54 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:37 2008 +0000" }, "message": "[GFS2] Remove \"reclaim limit\"\n\nThis call to reclaim glocks is not needed, and in particular we don\u0027t want it\nin the fast path for locking glocks. The limit was entirely arbitrary anyway\nand we can\u0027t expect users to adjust things like this, the remaining code will\ndo the right thing on its own.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "60b0d0877986b8fa70148f06055422d2ed858e88", "tree": "a3046f15ed803570554bad905ce690332dae8da8", "parents": [ "47e83b509127f5e83ae5d93afd5c7cb9241acc38" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 31 14:24:33 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:35 2008 +0000" }, "message": "[GFS2] Remove unused variables\n\nThese haven\u0027t been used for some time, remove them.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "47e83b509127f5e83ae5d93afd5c7cb9241acc38", "tree": "8f4412df56f8d65c355e553e7437035d2386b963", "parents": [ "c41d4f09f13671f98ba4b82fdc94420cdc09be08" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Oct 18 11:15:50 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:32 2008 +0000" }, "message": "[GFS2] Use correct include file in ops_address.c\n\nSomething changed in the upstream kernel, and it needs this\none-liner to allow ops_address.c to build.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c41d4f09f13671f98ba4b82fdc94420cdc09be08", "tree": "d1a9701517d56673c869077520ea59be9d6dce64", "parents": [ "b8e7cbb65bcc99630e123422c6829ce3c0fcdf14" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 17 14:05:41 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:30 2008 +0000" }, "message": "[GFS2] Don\u0027t hold page lock when starting transaction\n\nThis is an addendum to the new AOPs work which moves the point\nat which we take the page lock so that we don\u0027t get it until\nthe last possible moment. This resolves a conflict between\nstarting transactions and the page lock.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b8e7cbb65bcc99630e123422c6829ce3c0fcdf14", "tree": "a9f68259b90e9e65ea7f0369f448d580a8944f06", "parents": [ "9ff8ec32e58875022447af619bec6e5aee7c77e4" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 17 09:04:24 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:28 2008 +0000" }, "message": "[GFS2] Add writepages for GFS2 jdata\n\nThis patch resolves a lock ordering issue where we had been getting\na transaction lock in the wrong order with respect to the page lock.\nBy using writepages rather than just writepage, it is then possible\nto start a transaction before locking the page, and thus matching the\nlocking order elsewhere in the code.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9ff8ec32e58875022447af619bec6e5aee7c77e4", "tree": "0c626762f334d1cc066113b3e47a2fa02a72af0c", "parents": [ "5561093e2cac9f7d2a77e39cc689b8d2b7f9b2bc" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Sep 28 13:49:05 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:25 2008 +0000" }, "message": "[GFS2] Split gfs2_writepage into three cases\n\nThis patch splits gfs2_writepage into separate functions for each of\nthe three cases: writeback, ordered and journalled. As a result\nit becomes a lot easier to see what each one is doing. The common\ncode is moved into gfs2_writepage_common.\n\nThis fixes a performance bug where we were doing more work than\nstrictly required in the ordered write case.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "5561093e2cac9f7d2a77e39cc689b8d2b7f9b2bc", "tree": "aac1fa3e47604f2d7bb1e4d9e97f41b8ffebe91e", "parents": [ "bf36a713169432643d4fc7eeb4e0ace96d791d26" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 17 08:47:38 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:23 2008 +0000" }, "message": "[GFS2] Introduce gfs2_set_aops()\n\nJust like ext3 we now have three sets of address space operations\nto cover the cases of writeback, ordered and journalled data\nwrites. This means that the individual operations can now become\nless complicated as we are able to remove some of the tests for\nfile data mode from the code.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "bf36a713169432643d4fc7eeb4e0ace96d791d26", "tree": "2cb69c9716b9ee8c687ef807bb88d5e635db3462", "parents": [ "e7e36f143565d14950055c893cfaf4400ad64d34" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 17 08:35:19 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:21 2008 +0000" }, "message": "[GFS2] Add gfs2_is_writeback()\n\nThis adds a function \"gfs2_is_writeback()\" along the lines of the\nexisting \"gfs2_is_jdata()\" in order to clean up the code and make\nthe various tests for the inode mode more obvious. It also fixes\nthe PageChecked() logic where we were resetting the flag too early\nin the case of an error path.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "e7e36f143565d14950055c893cfaf4400ad64d34", "tree": "972889a8329afc80d4bef454cf2800e042fa5cc6", "parents": [ "f91a0d3e24e4b0198be5fae20d45a35c40d1efce" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Oct 16 11:47:04 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:18 2008 +0000" }, "message": "[GFS2] Remove unused field in struct gfs2_inode\n\nRemoves a field that is not used.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "f91a0d3e24e4b0198be5fae20d45a35c40d1efce", "tree": "cda8095f9befd25cbfaf5f63a4c8ca26870d45ca", "parents": [ "3cc3f710ce0effe397b830826a1a081fa81f11c7" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Oct 15 16:29:05 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:16 2008 +0000" }, "message": "[GFS2] Remove useless i_cache from inodes\n\nThe i_cache was designed to keep references to the indirect blocks\nused during block mapping so that they didn\u0027t have to be looked\nup continually. The idea failed because there are too many places\nwhere the i_cache needs to be freed, and this has in the past been\nthe cause of many bugs.\n\nIn addition there was no performance benefit being gained since the\ndisk blocks in question were cached anyway. So this patch removes\nit in order to simplify the code to prepare for other changes which\nwould otherwise have had to add further support for this feature.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3cc3f710ce0effe397b830826a1a081fa81f11c7", "tree": "53f69f1b8d1cbc2849c6bac08ce7786f3ecd7447", "parents": [ "51ff87bdd9f21a5d3672517b75d25ab5842d94a8" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Oct 15 15:40:33 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:13 2008 +0000" }, "message": "[GFS2] Use -\u003epage_mkwrite() for mmap()\n\nThis cleans up the mmap() code path for GFS2 by implementing the\npage_mkwrite function for GFS2. We are thus able to use the\ngeneric filemap_fault function for our -\u003efault() implementation.\n\nThis now means that shared writable mappings will be much more\nefficiently shared across the cluster if there is a reasonable\nproportion of read activity (the greater proportion, the better).\n\nAs a side effect, it also reduces the size of the code, removes\nspecial cases from readpage and readpages, and makes the code\npath easier to follow.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "51ff87bdd9f21a5d3672517b75d25ab5842d94a8", "tree": "7a8de5720b2a63e8d7d03a940a2b06196b2a3776", "parents": [ "cc7e79b168a552152299bd8a8254dc099aacc993" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Oct 15 14:42:35 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:11 2008 +0000" }, "message": "[GFS2] Clean up internal read function\n\nAs requested by Christoph, this patch cleans up GFS2\u0027s internal\nread function so that it no longer uses the do_generic_mapping_read\nfunction. This function is obsolete and GFS2 is the last user of it.\n\nAs a side effect the internal read code gets smaller and easier\nto read and gfs2_readpage is split into two. One function has the locking\nand the other function has the rest of the logic.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\n" }, { "commit": "cc7e79b168a552152299bd8a8254dc099aacc993", "tree": "65871c8ec495401846bc9e7030a89d3c6335f5ee", "parents": [ "49914084e797530d9baaf51df9eda77babc98fa8" ], "author": { "name": "Wendy Cheng", "email": "wcheng@redhat.com", "time": "Fri Oct 05 00:27:58 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Jan 25 08:07:09 2008 +0000" }, "message": "[GFS2] Handle multiple glock demote requests\n\nFix a race condition where multiple glock demote requests are sent to\na node back-to-back. This patch does a check inside handle_callback()\nto see whether a demote request is in progress. If true, it sets a flag\nto make sure run_queue() will loop again to handle the new request,\ninstead of erronously setting gl_demote_state to a different state.\n\nSigned-off-by: S. Wendy Cheng \u003cwcheng@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "197b12d6796a3bca187f22a8978a33d51e2bcd79", "tree": "706ff3454f03d4aa8ca4d76010479d7f9d2b36f4", "parents": [ "c10997f6575f476ff38442fa18fd4a0d80345f9d" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Dec 20 08:13:05 2007 -0800" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:40 2008 -0800" }, "message": "Kobject: convert fs/* from kobject_unregister() to kobject_put()\n\nThere is no need for kobject_unregister() anymore, thanks to Kay\u0027s\nkobject cleanup changes, so replace all instances of it with\nkobject_put().\n\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "901195ed7f4b2f30dc5a36271887939c5d7bfb9f", "tree": "1d56af3b3484ba97a98e3649247da25ac289ec7b", "parents": [ "43b98c4a66760d18d4056beed4078a8dbad193cc" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Dec 17 15:54:39 2007 -0400" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:26 2008 -0800" }, "message": "Kobject: change GFS2 to use kobject_init_and_add\n\nStop using kobject_register, as this way we can control the sending of\nthe uevent properly, after everything is properly initialized.\n\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "0ff21e46630abce11fdaaffabd72bbd4eed5ac2c", "tree": "cc49671622ef90775bf12a91d20b8286aa346e6f", "parents": [ "5c03c7ab886859eb195440dbb6ccb8c30c4e84cc" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Tue Nov 06 10:36:58 2007 -0800" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:24 2008 -0800" }, "message": "kobject: convert kernel_kset to be a kobject\n\nkernel_kset does not need to be a kset, but a much simpler kobject now\nthat we have kobj_attributes.\n\nWe also rename kernel_kset to kernel_kobj to catch all users of this\nsymbol with a build error instead of an easy-to-ignore build warning.\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "bd35b93d8049ab47b5bfaf6b10ba39badf21d1c3", "tree": "bac82e14d960b2c7011b7f660a93f07e922f8a97", "parents": [ "e5e38a86c0bbe8475543f10f0a48393a45df5182" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Oct 29 20:13:17 2007 +0100" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:14 2008 -0800" }, "message": "kset: convert kernel_subsys to use kset_create\n\nDynamically create the kset instead of declaring it statically. We also\nrename kernel_subsys to kernel_kset to catch all users of this symbol\nwith a build error instead of an easy-to-ignore build warning.\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "136a27507fd09006973f11b17ca971d4c176a06a", "tree": "9d1490f61cbc3aded5bfc7c161c46285d0ecc74f", "parents": [ "9bec101a0c38d559a8c95b44d850cd09a7b4edef" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Oct 29 20:13:17 2007 +0100" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:14 2008 -0800" }, "message": "kset: convert gfs2 dlm to use kset_create\n\nDynamically create the kset instead of declaring it statically.\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "9bec101a0c38d559a8c95b44d850cd09a7b4edef", "tree": "65161628aae7438635f183a6d070887a55033952", "parents": [ "00d2666623368ffd39afc875ff8a2eead2a0436c" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Oct 29 20:13:17 2007 +0100" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:13 2008 -0800" }, "message": "kset: convert gfs2 to use kset_create\n\nDynamically create the kset instead of declaring it statically.\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "00d2666623368ffd39afc875ff8a2eead2a0436c", "tree": "9b16b85167f1bd86c02c0ebeb74510d8029783a3", "parents": [ "917e865df7eb020f20ffc2b4204f282a587df94f" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Mon Oct 29 14:17:23 2007 -0600" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:13 2008 -0800" }, "message": "kobject: convert main fs kobject to use kobject_create\n\nThis also renames fs_subsys to fs_kobj to catch all current users with a\nbuild error instead of a build warning which can easily be missed.\n\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "3514faca19a6fdc209734431c509631ea92b094e", "tree": "f6d102e6dec276f8e8d1044b47c74a02b901554f", "parents": [ "c11c4154e7ff4cebfadad849b1e22689d759c3f4" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Tue Oct 16 10:11:44 2007 -0600" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Thu Jan 24 20:40:10 2008 -0800" }, "message": "kobject: remove struct kobj_type from struct kset\n\nWe don\u0027t need a \"default\" ktype for a kset. We should set this\nexplicitly every time for each kset. This change is needed so that we\ncan make ksets dynamic, and cleans up one of the odd, undocumented\nassumption that the kset/kobject/ktype model has.\n\nThis patch is based on a lot of help from Kay Sievers.\n\nNasty bug in the block code was found by Dave Young\n\u003chidave.darkstar@gmail.com\u003e\n\nCc: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nCc: Dave Young \u003chidave.darkstar@gmail.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "39655164405940d4818224a085e35420e2f97aed", "tree": "6b019b3bc77eecac1731fe64e5c031790c2b5223", "parents": [ "cfaea787c05822acbb4d8963baee5edd1cc0258f" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Sun Oct 21 16:42:17 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Oct 22 08:13:21 2007 -0700" }, "message": "exportfs: make struct export_operations const\n\nNow that nfsd has stopped writing to the find_exported_dentry member we an\nmark the export_operations const\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nCc: \u003clinux-ext4@vger.kernel.org\u003e\nCc: Dave Kleikamp \u003cshaggy@austin.ibm.com\u003e\nCc: Anton Altaparmakov \u003caia21@cantab.net\u003e\nCc: David Chinner \u003cdgc@sgi.com\u003e\nCc: Timothy Shimmin \u003ctes@sgi.com\u003e\nCc: OGAWA Hirofumi \u003chirofumi@mail.parknet.co.jp\u003e\nCc: Hugh Dickins \u003chugh@veritas.com\u003e\nCc: Chris Mason \u003cmason@suse.com\u003e\nCc: Jeff Mahoney \u003cjeffm@suse.com\u003e\nCc: \"Vladimir V. Saveliev\" \u003cvs@namesys.com\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Mark Fasheh \u003cmark.fasheh@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": "34c0d154243dd913c5690ae6ceb9557017429b9c", "tree": "8e795b5d0e0833010fe3be8cf4754f864b2d07cb", "parents": [ "be55caf177e14bc54d1498d599a78849b0b230bb" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Sun Oct 21 16:42:14 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Oct 22 08:13:20 2007 -0700" }, "message": "gfs2: new export ops\n\nConvert gfs2 to the new ops. Uses a similar structure to the generic helpers,\nbut gfs2 has it\u0027s own file handle formats.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Neil Brown \u003cneilb@suse.de\u003e\nCc: \"J. Bruce Fields\" \u003cbfields@fieldses.org\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a9c62a18a291499d15a370d08771e781fbaf91e6", "tree": "5d608ba9dfefce643060c1d9ab18d9f4bff18360", "parents": [ "28e3fed8b79c76450f264e77c58d0f5fbd2dd739" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Tue Oct 16 23:30:22 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Oct 17 08:43:01 2007 -0700" }, "message": "fs: correct SuS compliance for open of large file without options\n\nThe early LFS work that Linux uses favours EFBIG in various places. SuSv3\nspecifically uses EOVERFLOW for this as noted by Michael (Bug 7253)\n\n[EOVERFLOW]\n The named file is a regular file and the size of the file cannot be\nrepresented correctly in an object of type off_t. We should therefore\ntransition to the proper error return code\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nCc: Theodore Tso \u003ctytso@mit.edu\u003e\nCc: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nCc: Arjan van de Ven \u003carjan@infradead.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4ba9b9d0ba0a49d91fa6417c7510ee36f48cf957", "tree": "191b4f45f926e44b882b1e87a9a85dc12230b892", "parents": [ "b811c202a0edadaac7242ab834fe7ba409978ae7" ], "author": { "name": "Christoph Lameter", "email": "clameter@sgi.com", "time": "Tue Oct 16 23:25:51 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Oct 17 08:42:45 2007 -0700" }, "message": "Slab API: remove useless ctor parameter and reorder parameters\n\nSlab constructors currently have a flags parameter that is never used. And\nthe order of the arguments is opposite to other slab functions. The object\npointer is placed before the kmem_cache pointer.\n\nConvert\n\n ctor(void *object, struct kmem_cache *s, unsigned long flags)\n\nto\n\n ctor(struct kmem_cache *s, void *object)\n\nthroughout the kernel\n\n[akpm@linux-foundation.org: coupla fixes]\nSigned-off-by: Christoph Lameter \u003cclameter@sgi.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "7765ec26ae1c01bb29bedf910e4efcced8cc81d2", "tree": "8ce0e877daf805e5e546f10b389cd1c686ec0365", "parents": [ "d79689c7038ea07182e8d7340786f7fcf8c77780" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Oct 16 01:25:07 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Tue Oct 16 09:42:55 2007 -0700" }, "message": "gfs2: convert to new aops\n\nCc: Nick Piggin \u003cnickpiggin@yahoo.com.au\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "541010e4b8921cd781ff02ae68028501457045b6", "tree": "58bd529d4c6e69899a0aa20afa2d7f1c23326417", "parents": [ "e457f790d8b05977853aa238bbc667b3bb375671", "5e7fc436426b1f9e106f511a049de91c82ec2c53" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Oct 15 16:07:40 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Oct 15 16:07:40 2007 -0700" }, "message": "Merge branch \u0027locks\u0027 of git://linux-nfs.org/~bfields/linux\n\n* \u0027locks\u0027 of git://linux-nfs.org/~bfields/linux:\n nfsd: remove IS_ISMNDLCK macro\n Rework /proc/locks via seq_files and seq_list helpers\n fs/locks.c: use list_for_each_entry() instead of list_for_each()\n NFS: clean up explicit check for mandatory locks\n AFS: clean up explicit check for mandatory locks\n 9PFS: clean up explicit check for mandatory locks\n GFS2: clean up explicit check for mandatory locks\n Cleanup macros for distinguishing mandatory locks\n Documentation: move locks.txt in filesystems/\n locks: add warning about mandatory locking races\n Documentation: move mandatory locking documentation to filesystems/\n locks: Fix potential OOPS in generic_setlease()\n Use list_first_entry in locks_wake_up_blocks\n locks: fix flock_lock_file() comment\n Memory shortage can result in inconsistent flocks state\n locks: kill redundant local variable\n locks: reverse order of posix_locks_conflict() arguments\n" }, { "commit": "efefc6eb38d43b8e5daef482f575d767b002004e", "tree": "8a3933db1f8721f9bcc9912c800dc8406f4bdf94", "parents": [ "117494a1b65183f0e3fcc817b07944bc5c465050", "cd59abfcc441b2abb4cf2cd62c1eb0f02a60e8dd" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 12 15:49:37 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 12 15:49:37 2007 -0700" }, "message": "Merge master.kernel.org:/pub/scm/linux/kernel/git/gregkh/driver-2.6\n\n* master.kernel.org:/pub/scm/linux/kernel/git/gregkh/driver-2.6: (75 commits)\n PM: merge device power-management source files\n sysfs: add copyrights\n kobject: update the copyrights\n kset: add some kerneldoc to help describe what these strange things are\n Driver core: rename ktype_edd and ktype_efivar\n Driver core: rename ktype_driver\n Driver core: rename ktype_device\n Driver core: rename ktype_class\n driver core: remove subsystem_init()\n sysfs: move sysfs file poll implementation to sysfs_open_dirent\n sysfs: implement sysfs_open_dirent\n sysfs: move sysfs_dirent-\u003es_children into sysfs_dirent-\u003es_dir\n sysfs: make sysfs_root a regular directory dirent\n sysfs: open code sysfs_attach_dentry()\n sysfs: make s_elem an anonymous union\n sysfs: make bin attr open get active reference of parent too\n sysfs: kill unnecessary NULL pointer check in sysfs_release()\n sysfs: kill unnecessary sysfs_get() in open paths\n sysfs: reposition sysfs_dirent-\u003es_mode.\n sysfs: kill sysfs_update_file()\n ...\n" }, { "commit": "34980ca8faebfcce31094eba6ffbb0113950361f", "tree": "1339b5a059fdc34e688d5571749577e8f3f96f09", "parents": [ "7e7654a92a5a75a350cc3458389bc76273498f6a" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Wed Sep 12 15:06:57 2007 -0700" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Fri Oct 12 14:51:02 2007 -0700" }, "message": "Drivers: clean up direct setting of the name of a kset\n\nA kset should not have its name set directly, so dynamically set the\nname at runtime.\n\nThis is needed to remove the static array in the kobject structure which\nwill be changed in a future patch.\n\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n\n" }, { "commit": "f26e51f67ae6a75ffc57b96cf5fe096f75e778cb", "tree": "1e848187885426430cc93bffaadc539312ce636d", "parents": [ "1462222b76a09a24b240563a51d5f9fbea8bd3e1", "c36258b5925e6cf6bf72904635100593573bfcff" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 12 09:14:51 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 12 09:14:51 2007 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw: (51 commits)\n [DLM] block dlm_recv in recovery transition\n [DLM] don\u0027t overwrite castparam if it\u0027s NULL\n [GFS2] Get superblock a different way\n [GFS2] Don\u0027t try to remove buffers that don\u0027t exist\n [GFS2] Alternate gfs2_iget to avoid looking up inodes being freed\n [GFS2] Data corruption fix\n [GFS2] Clean up journaled data writing\n [GFS2] GFS2: chmod hung - fix race in thread creation\n [DLM] Make dlm_sendd cond_resched more\n [GFS2] Move inode deletion out of blocking_cb\n [GFS2] flocks from same process trip kernel BUG at fs/gfs2/glock.c:1118!\n [GFS2] Clean up gfs2_trans_add_revoke()\n [GFS2] Use slab operations for all gfs2_bufdata allocations\n [GFS2] Replace revoke structure with bufdata structure\n [GFS2] Fix ordering of dirty/journal for ordered buffer unstuffing\n [GFS2] Clean up ordered write code\n [GFS2] Move pin/unpin into lops.c, clean up locking\n [GFS2] Don\u0027t mark jdata dirty in gfs2_unstuffer_page()\n [GFS2] Introduce gfs2_remove_from_ail\n [GFS2] Correct lock ordering in unlink\n ...\n" }, { "commit": "782e3b3b3804c38d5130c7f21d7ec7bf6709023f", "tree": "54e790dfd52f0451189fa1591dc97bc5ff5f5866", "parents": [ "cbd09dbbb62096c1da627eca865f988d2ed0a84e" ], "author": { "name": "Al Viro", "email": "viro@ftp.linux.org.uk", "time": "Fri Oct 12 07:17:47 2007 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Oct 12 00:29:50 2007 -0700" }, "message": "Fix up more bio fallout\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5a60c532c9224babc172fafccc9e2fec6937af6f", "tree": "1acf07fea71d33fbce3ae0404b05a94087406984", "parents": [ "891ba6d4a5f9e6302bb6542592d73feb4d0d3687" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Sep 26 09:39:31 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:34 2007 +0100" }, "message": "[GFS2] Get superblock a different way\n\nThe mapping may be NULL by the time the I/O has completed, so\nwe now get the superblock by a different route (via the bd and glock)\nto avoid this problem.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Wendy Cheng \u003cwcheng@redhat.com\u003e\n" }, { "commit": "891ba6d4a5f9e6302bb6542592d73feb4d0d3687", "tree": "a99d059aa2f1803111b063ce99beb809f384e44a", "parents": [ "7a9f53b3c1875bef22ad4588e818bc046ef183da" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Sep 20 15:26:33 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:31 2007 +0100" }, "message": "[GFS2] Don\u0027t try to remove buffers that don\u0027t exist\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "7a9f53b3c1875bef22ad4588e818bc046ef183da", "tree": "0b23b5b8a667d0d57c695c47f6b4eb6d50c2d661", "parents": [ "de986e859a29097fb9211b052d86a9a2c868f6cd" ], "author": { "name": "Benjamin Marzinski", "email": "bmarzins@redhat.com", "time": "Tue Sep 18 13:33:18 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:29 2007 +0100" }, "message": "[GFS2] Alternate gfs2_iget to avoid looking up inodes being freed\n\nThere is a possible deadlock between two processes on the same node, where one\nprocess is deleting an inode, and another process is looking for allocated but\nunused inodes to delete in order to create more space.\n\nprocess A does an iput() on inode X, and it\u0027s i_count drops to 0. This causes\niput_final() to be called, which puts an inode into state I_FREEING at\ngeneric_delete_inode(). There no point between when iput_final() is called, and\nwhen I_FREEING is set where GFS2 could acquire any glocks. Once I_FREEING is\nset, no other process on that node can successfully look up that inode until\nthe delete finishes.\n\nprocess B locks the the resource group for the same inode in get_local_rgrp(),\nwhich is called by gfs2_inplace_reserve_i()\n\nprocess A tries to lock the resource group for the inode in\ngfs2_dinode_dealloc(), but it\u0027s already locked by process B\n\nprocess B waits in find_inode for the inode to have the I_FREEING state cleared.\n\nDeadlock.\n\nThis patch solves the problem by adding an alternative to gfs2_iget(),\ngfs2_iget_skip(), that simply skips any inodes that are in the I_FREEING\nstate.o The alternate test function is just like the original one, except that\nit fails if the inode is being freed, and sets a skipped flag. The alternate\nset function is just like the original, except that it fails if the skipped\nflag is set. Only try_rgrp_unlink() calls gfs2_iget_skip() instead of\ngfs2_iget().\n\nSigned-off-by: Benjamin E. Marzinski \u003cbmarzins@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "de986e859a29097fb9211b052d86a9a2c868f6cd", "tree": "85c97fe22e0eafb3c7440c8ff21fe73d8c456960", "parents": [ "16615be18cadf53ee6f8a4f0bdd647f0753421b1" ], "author": { "name": "Wendy Cheng", "email": "wcheng@redhat.com", "time": "Tue Sep 18 09:19:13 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:26 2007 +0100" }, "message": "[GFS2] Data corruption fix\n\n* GFS2 has been using i_cache array to store its indirect meta blocks.\nIts flush routine doesn\u0027t correctly clean up all the entries. The\nproblem would show while multiple nodes do simultaneous writes to the\nsame file. Upon glock exclusive lock transfer, if the file is a sparse\nfile with large file size where the indirect meta blocks span multiple\narray entries with \"zero\" entries in between. The flush routine\nprematurely stops the flushing that leaves old (stale) entries around.\nThis leads to several nasty issues, including data corruption.\n* Fix gfs2_get_block_noalloc checking to correctly return EIO upon\nunmapped buffer.\n\nSigned-off-by: Wendy Cheng \u003cwcheng@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "16615be18cadf53ee6f8a4f0bdd647f0753421b1", "tree": "670c75e931e6d606211f338ee5e8b1d603c96521", "parents": [ "55c0c4ac0be144014651b19e77c9b77f367955de" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Sep 17 10:59:52 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:24 2007 +0100" }, "message": "[GFS2] Clean up journaled data writing\n\nThis patch cleans up the code for writing journaled data into the log.\nIt also removes the need to allocate a small \"tag\" structure for each\nblock written into the log. Instead we just keep count of the outstanding\nI/O so that we can be sure that its all been written at the correct time.\nAnother result of this patch is that a number of ll_rw_block() calls\nhave become submit_bh() calls, closing some races at the same time.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "55c0c4ac0be144014651b19e77c9b77f367955de", "tree": "dbea7f24657f8e99f4ff94519cb4f24373930320", "parents": [ "d66f8277f53407754f50ae6bada68f1b68d04d48" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Fri Sep 14 09:27:59 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:22 2007 +0100" }, "message": "[GFS2] GFS2: chmod hung - fix race in thread creation\n\nThe problem boiled down to a race between the gdlm_init_threads()\nfunction initializing thread1 and its setting of blist \u003d 1.\nEssentially, \"if (current \u003d\u003d ls-\u003ethread1)\" was checked by the thread\nbefore the thread creator set ls-\u003ethread1.\n\nSince thread1 is the only thread who is allowed to work on the\nblocking queue, and since neither thread thought it was thread1, no one\nwas working on the queue. So everything just sat.\n\nThis patch reuses the ls-\u003easync_lock spin_lock to fix the race,\nand it fixes the problem. I\u0027ve done more than 2000 iterations of the\nloop that was recreating the failure and it seems to work.\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n\n--\n" }, { "commit": "49e61f2ef6f7d1d0296e3e30d366b28e0ca595c2", "tree": "7f2efb6c0dce87f3f16809e8cd1072137b12271c", "parents": [ "b4c20166dcfca106f0f416bfce200099ed76ab18" ], "author": { "name": "Wendy Cheng", "email": "wcheng@redhat.com", "time": "Thu Sep 13 17:52:42 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:17 2007 +0100" }, "message": "[GFS2] Move inode deletion out of blocking_cb\n\nMove inode deletion code out of blocking_cb handle_callback route to\navoid racy conditions that end up blocking lock_dlm1 thread. Fix\nbugzilla 286821.\n\nSigned-off-by: Wendy Cheng \u003cwcheng@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b4c20166dcfca106f0f416bfce200099ed76ab18", "tree": "474e8c3bda161953202793c0801ab00a5b5433de", "parents": [ "1ad38c437fa33f85ba4b6a85ea8c5478ee72d5bd" ], "author": { "name": "Abhijith Das", "email": "adas@redhat.com", "time": "Thu Sep 13 23:35:27 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:14 2007 +0100" }, "message": "[GFS2] flocks from same process trip kernel BUG at fs/gfs2/glock.c:1118!\n\nThis patch adds a new flag to the gfs2_holder structure GL_FLOCK.\nIt is set on holders of glocks representing flocks. This flag is\nchecked in add_to_queue() and a process is permitted to queue more\nthan one holder onto a glock if it is set. This solves the issue\nof a process not being able to do multiple flocks on the same file.\nThrough a single descriptor, a process can now promote and demote\nflocks. Through multiple descriptors a process can now queue\nmultiple flocks on the same file. There\u0027s still the problem of\na process deadlocking itself (because gfs2 blocking locks are not\ninterruptible) by queueing incompatible deadlock.\n\nSigned-off-by: Abhijith Das \u003cadas@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1ad38c437fa33f85ba4b6a85ea8c5478ee72d5bd", "tree": "11a5fc7993ba9ae343fc72f03f9a11f312fd6128", "parents": [ "0820ab517e1b100ee3f9584ec27f93309689ebe7" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Sep 03 11:01:33 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:12 2007 +0100" }, "message": "[GFS2] Clean up gfs2_trans_add_revoke()\n\nThe following alters gfs2_trans_add_revoke() to take a struct\ngfs2_bufdata as an argument. This eliminates the memory allocation which\nwas previously required by making use of the already existing struct\ngfs2_bufdata. It makes some sanity checks to ensure that the\ngfs2_bufdata has been removed from all the lists before its recycled as\na revoke structure. This saves one memory allocation and one free per\nrevoke structure.\n\nAlso as a result, and to simplify the locking, since there is no longer\nany blocking code in gfs2_trans_add_revoke() we must hold the log lock\nwhenever this function is called. This reduces the amount of times we\ntake and unlock the log lock.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "0820ab517e1b100ee3f9584ec27f93309689ebe7", "tree": "db2baf9e6d2ac545c344a412fe31371a09bf9b68", "parents": [ "82e86087bb774cd54d47db4a7c771b5b29bea9ed" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Sun Sep 02 16:47:38 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:10 2007 +0100" }, "message": "[GFS2] Use slab operations for all gfs2_bufdata allocations\n\nThe old revoke structure was allocated using kalloc/kfree but\nthere is a slab cache for gfs2_bufdata, so we should use that\nnow that the structures have been converted.\n\nThis is part two of the patch series to merge the revoke\nand gfs2_bufdata structures.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "82e86087bb774cd54d47db4a7c771b5b29bea9ed", "tree": "c067773861203becfa6c8cf9e4ee449787cf65f6", "parents": [ "8475487befb29eeb038fef374a7433d276336a25" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Sun Sep 02 15:39:43 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:07 2007 +0100" }, "message": "[GFS2] Replace revoke structure with bufdata structure\n\nBoth the revoke structure and the bufdata structure are quite similar.\nThey are basically small tags which are put on lists. In addition to\nwhich the revoke structure is always allocated when there is a bufdata\nstructure which is (or can be) freed. As such it should be possible to\nreduce the number of frees and allocations by using the same structure\nfor both purposes.\n\nThis patch is the first step along that path. It replaces existing uses\nof the revoke structure with the bufdata structure.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8475487befb29eeb038fef374a7433d276336a25", "tree": "1fb635fc8880475ab2042f4030b2643b31a63654", "parents": [ "d7b616e252b125f12b007c392f7644053bb6f140" ], "author": { "name": "Bob Peterson", "email": "rpeterso@redhat.com", "time": "Sun Sep 02 10:55:29 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:05 2007 +0100" }, "message": "[GFS2] Fix ordering of dirty/journal for ordered buffer unstuffing\n\nSigned-off-by: Bob Peterson \u003crpeterso@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "d7b616e252b125f12b007c392f7644053bb6f140", "tree": "0794272905a1876ef74144a993f7a76400893813", "parents": [ "9b9107a5a8b190e6cf09bbdf893869c6a9c482cc" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Sun Sep 02 10:48:13 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:03 2007 +0100" }, "message": "[GFS2] Clean up ordered write code\n\nThe following patch removes the ordered write processing from\ndatabuf_lo_before_commit() and moves it to log.c. This has the effect of\ngreatly simplyfying databuf_lo_before_commit() and well as potentially\nmaking the ordered write code more efficient.\n\nAs a side effect of this, its now possible to remove ordered buffers\nfrom the ordered buffer list at any time, so we now make use of this in\ninvalidatepage and releasepage to ensure timely release of these\nbuffers.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9b9107a5a8b190e6cf09bbdf893869c6a9c482cc", "tree": "77016dd5476f6ba72605ac0cdcbccec3ce794057", "parents": [ "eaf965270ffff3086ef929e660ace45e862cfd2d" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Aug 27 13:54:05 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:00 2007 +0100" }, "message": "[GFS2] Move pin/unpin into lops.c, clean up locking\n\ngfs2_pin and gfs2_unpin are only used in lops.c, despite being\ndefined in meta_io.c, so this patch moves them into lops.c and\nmakes them static. At the same time, its possible to clean up\nthe locking in the buf and databuf _lo_add() functions so that\nwe only need to grab the spinlock once. Also we have to move\nlock_buffer() around the _lo_add() functions since we can\u0027t\ndo that in gfs2_pin() any more since we hold the spinlock\nfor the duration of that function.\n\nAs a result, the code shrinks by 12 lines and we do far fewer\noperations when adding buffers to the log. It also makes the\ncode somewhat easier to read \u0026 understand.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "eaf965270ffff3086ef929e660ace45e862cfd2d", "tree": "7fda326324431424573ba5d62170637aa8f41d59", "parents": [ "1e1a3d03e927d39282208aed676e49d25129feea" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Aug 27 09:49:37 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:55:58 2007 +0100" }, "message": "[GFS2] Don\u0027t mark jdata dirty in gfs2_unstuffer_page()\n\nJournaled data is marked dirty by gfs2_unpin and should not be marked\ndirty here.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1e1a3d03e927d39282208aed676e49d25129feea", "tree": "731cca9b4df1a6164c809f002290a3d7fd6af1f4", "parents": [ "8497a46e178addb27ad1c981befaa17ca788b5c3" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Aug 27 09:45:26 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:55:55 2007 +0100" }, "message": "[GFS2] Introduce gfs2_remove_from_ail\n\nThis collects together the operations required to remove a gfs2_bufdata\nfrom the ail lists. Its only called from two places to start with, but\nexpect to see more of this function in future.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8497a46e178addb27ad1c981befaa17ca788b5c3", "tree": "a2793792fb63da1aa582b66e9ae1f05dbb90c00c", "parents": [ "e9bd2b3bafd29bf75522546207f0bba0ec4515c2" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Sun Aug 26 14:23:56 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:55:53 2007 +0100" }, "message": "[GFS2] Correct lock ordering in unlink\n\nThis patch corrects the lock ordering in unlink to be the same as\nthat in the rest of GFS2, i.e. parent -\u003e child -\u003e rgrp.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" } ], "next": "e9bd2b3bafd29bf75522546207f0bba0ec4515c2" }