)]}' { "log": [ { "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": "c36258b5925e6cf6bf72904635100593573bfcff", "tree": "565f1ce29a7f8a2cd1c25f2d36c932727adbdbc2", "parents": [ "b434eda6fda5bcdcc2dd918e5ffbf7184f2d4e17" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Thu Sep 27 15:53:38 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:38 2007 +0100" }, "message": "[DLM] block dlm_recv in recovery transition\n\nIntroduce a per-lockspace rwsem that\u0027s held in read mode by dlm_recv\nthreads while working in the dlm. This allows dlm_recv activity to be\nsuspended when the lockspace transitions to, from and between recovery\ncycles.\n\nThe specific bug prompting this change is one where an in-progress\nrecovery cycle is aborted by a new recovery cycle. While dlm_recv was\nprocessing a recovery message, the recovery cycle was aborted and\ndlm_recoverd began cleaning up. dlm_recv decremented recover_locks_count\non an rsb after dlm_recoverd had reset it to zero. This is fixed by\nsuspending dlm_recv (taking write lock on the rwsem) before aborting the\ncurrent recovery.\n\nThe transitions to/from normal and recovery modes are simplified by using\nthis new ability to block dlm_recv. The switch from normal to recovery\nmode means dlm_recv goes from processing locking messages, to saving them\nfor later, and vice versa. Races are avoided by blocking dlm_recv when\nsetting the flag that switches between modes.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b434eda6fda5bcdcc2dd918e5ffbf7184f2d4e17", "tree": "e1931dffbbf294fe1d087e9215ee7143517636e2", "parents": [ "5a60c532c9224babc172fafccc9e2fec6937af6f" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Oct 01 15:28:42 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:36 2007 +0100" }, "message": "[DLM] don\u0027t overwrite castparam if it\u0027s NULL\n\nIf the castaddr passed to the userland API is NULL then don\u0027t overwrite the\nexisting castparam. This allows a different thread to cancel a lock request and\nthe CANCEL AST gets delivered to the original thread.\n\nbz#306391 (for RHEL4) refers.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "d66f8277f53407754f50ae6bada68f1b68d04d48", "tree": "8725f182541eb98e1947d8a224e61d66c2c959da", "parents": [ "49e61f2ef6f7d1d0296e3e30d366b28e0ca595c2" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Fri Sep 14 08:49:21 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:56:19 2007 +0100" }, "message": "[DLM] Make dlm_sendd cond_resched more\n\nUnder high recovery loads dlm_sendd can monopolise the CPU and cause soft lockups.\n\nThis one extra and one moved cond_resched() make it yield a little more during\nsuch times keeping work moving.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "61d96be0f474df354c2ff4a2b2bf410b23a5cd60", "tree": "b40a06b75f7ff59b3925dcabbbf63605f7b18c18", "parents": [ "a13b8c5f2381495879e6facd3b3ada51c9e68194" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Aug 20 15:13:38 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Oct 10 08:55:39 2007 +0100" }, "message": "[DLM] Fix lowcomms socket closing\n\nThis patch fixes the slight mess made in lowcomms closing by previous patches\nand fixes all sorts of DLM hangs.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3650925893469ccb03dbcc6a440c5d363350f591", "tree": "8f0b9d6ce880547723cb6590c27799821b9c2f96", "parents": [ "9e5f2825a8b721360b291f14f42cd7a25781156b" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Aug 07 09:44:48 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:31:02 2007 +0100" }, "message": "[DLM] fix basts for granted PR waiting CW\n\nFix a long standing bug where a blocking callback would be missed\nwhen there\u0027s a granted lock in PR mode and waiting locks in both\nPR and CW modes (and the PR lock was added to the waiting queue\nbefore the CW lock). The logic simply compared the numerical values\nof the modes to determine if a blocking callback was required, but in\nthe one case of PR and CW, the lower valued CW mode blocks the higher\nvalued PR mode. We just need to add a special check for this PR/CW\ncase in the tests that decide when a blocking callback is needed.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9e5f2825a8b721360b291f14f42cd7a25781156b", "tree": "7d8e54a91dc46edce1c62f4991a7bef2267b5402", "parents": [ "1a2bf2eefb63a267aea7f3f80d6ac59160e20810" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Thu Aug 02 14:58:14 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:30:36 2007 +0100" }, "message": "[DLM] More othercon fixes\n\nThe last patch to clean out \u0027othercon\u0027 structures only fixed half the problem.\nThe attached addresses the other situations too, and fixes bz#238490\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1a2bf2eefb63a267aea7f3f80d6ac59160e20810", "tree": "eb80d5d70ed84d445890db1f4907b013e3bbaf1c", "parents": [ "01c8cab25863de007fe8c598d0033919ea8ae65e" ], "author": { "name": "Jesper Juhl", "email": "jesper.juhl@gmail.com", "time": "Thu Jul 19 00:27:43 2007 +0200" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:30:04 2007 +0100" }, "message": "[DLM] Fix memory leak in dlm_add_member() when dlm_node_weight() returns less than zero\n\nThere\u0027s a memory leak in fs/dlm/member.c::dlm_add_member().\n\nIf \"dlm_node_weight(ls-\u003els_name, nodeid)\" returns \u003c 0, then\nwe\u0027ll return without freeing the memory allocated to the (at\nthat point yet unused) \u0027memb\u0027.\nThis patch frees the allocated memory in that case and thus\navoids the leak.\n\nSigned-off-by: Jesper Juhl \u003cjesper.juhl@gmail.com\u003e\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "01c8cab25863de007fe8c598d0033919ea8ae65e", "tree": "b7945a4766743ad4dd559681b65bb9e3d230a4a8", "parents": [ "41684f9547455b395ffd65e5b7961067d20a2872" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Jul 17 16:53:15 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:29:27 2007 +0100" }, "message": "[DLM] zero unused parts of sockaddr_storage\n\nWhen we build a sockaddr_storage for an IP address, clear the unused parts as\nthey could be used for node comparisons.\n\nI have seen this occasionally make sctp connections fail.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "41684f9547455b395ffd65e5b7961067d20a2872", "tree": "8afb8cae66afe67246666c694d5731a876672f1b", "parents": [ "25720c2d73058f4f929f16093f60817ed52a285c" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri Jul 13 14:49:06 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:28:44 2007 +0100" }, "message": "[DLM] fix NULL ls usage\n\nFix regression in recent patch \"[DLM] variable allocation\" which\nattempts to dereference an \"ls\" struct when it\u0027s NULL.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "25720c2d73058f4f929f16093f60817ed52a285c", "tree": "09b14797656f31596c3b061608eeba7b8ded3701", "parents": [ "39d3520c92cf7a28c07229ca00cc35a1e8026c77" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Jul 11 13:39:43 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue Aug 14 10:28:05 2007 +0100" }, "message": "[DLM] Clear othercon pointers when a connection is closed\n\nThis patch clears the othercon pointer and frees the memory when a connnection\nis closed. This could cause a small memory leak when nodes leave the cluster.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "20c2df83d25c6a95affe6157a4c9cac4cf5ffaac", "tree": "415c4453d2b17a50abe7a3e515177e1fa337bd67", "parents": [ "64fb98fc40738ae1a98bcea9ca3145b89fb71524" ], "author": { "name": "Paul Mundt", "email": "lethal@linux-sh.org", "time": "Fri Jul 20 10:11:58 2007 +0900" }, "committer": { "name": "Paul Mundt", "email": "lethal@linux-sh.org", "time": "Fri Jul 20 10:11:58 2007 +0900" }, "message": "mm: Remove slab destructors from kmem_cache_create().\n\nSlab destructors were no longer supported after Christoph\u0027s\nc59def9f222d44bb7e2f0a559f2906191a0862d7 change. They\u0027ve been\nBUGs for both slab and slub, and slob never supported them\neither.\n\nThis rips out support for the dtor pointer from kmem_cache_create()\ncompletely and fixes up every single callsite in the kernel (there were\nabout 224, not including the slab allocator definitions themselves,\nor the documentation references).\n\nSigned-off-by: Paul Mundt \u003clethal@linux-sh.org\u003e\n" }, { "commit": "dd00cc486ab1c17049a535413d1751ef3482141c", "tree": "d90ff69ea06792b9284f2f2665c96624f121b88a", "parents": [ "3b5ad0797c0e4049001f961a8b58f1d0ce532072" ], "author": { "name": "Yoann Padioleau", "email": "padator@wanadoo.fr", "time": "Thu Jul 19 01:49:03 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Thu Jul 19 10:04:50 2007 -0700" }, "message": "some kmalloc/memset -\u003ekzalloc (tree wide)\n\nTransform some calls to kmalloc/memset to a single kzalloc (or kcalloc).\n\nHere is a short excerpt of the semantic patch performing\nthis transformation:\n\n@@\ntype T2;\nexpression x;\nidentifier f,fld;\nexpression E;\nexpression E1,E2;\nexpression e1,e2,e3,y;\nstatement S;\n@@\n\n x \u003d\n- kmalloc\n+ kzalloc\n (E1,E2)\n ... when !\u003d \\(x-\u003efld\u003dE;\\|y\u003df(...,x,...);\\|f(...,x,...);\\|x\u003dE;\\|while(...) S\\|for(e1;e2;e3) S\\)\n- memset((T2)x,0,E1);\n\n@@\nexpression E1,E2,E3;\n@@\n\n- kzalloc(E1 * E2,E3)\n+ kcalloc(E1,E2,E3)\n\n[akpm@linux-foundation.org: get kcalloc args the right way around]\nSigned-off-by: Yoann Padioleau \u003cpadator@wanadoo.fr\u003e\nCc: Richard Henderson \u003crth@twiddle.net\u003e\nCc: Ivan Kokshaysky \u003cink@jurassic.park.msu.ru\u003e\nAcked-by: Russell King \u003crmk@arm.linux.org.uk\u003e\nCc: Bryan Wu \u003cbryan.wu@analog.com\u003e\nAcked-by: Jiri Slaby \u003cjirislaby@gmail.com\u003e\nCc: Dave Airlie \u003cairlied@linux.ie\u003e\nAcked-by: Roland Dreier \u003crolandd@cisco.com\u003e\nCc: Jiri Kosina \u003cjkosina@suse.cz\u003e\nAcked-by: Dmitry Torokhov \u003cdtor@mail.ru\u003e\nCc: Benjamin Herrenschmidt \u003cbenh@kernel.crashing.org\u003e\nAcked-by: Mauro Carvalho Chehab \u003cmchehab@infradead.org\u003e\nAcked-by: Pierre Ossman \u003cdrzeus-list@drzeus.cx\u003e\nCc: Jeff Garzik \u003cjeff@garzik.org\u003e\nCc: \"David S. Miller\" \u003cdavem@davemloft.net\u003e\nAcked-by: Greg KH \u003cgreg@kroah.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@steeleye.com\u003e\nCc: \"Antonino A. Daplas\" \u003cadaplas@pol.net\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e6bd07aee739566803425acdbf5cdb29919164e1", "tree": "e085a5065d06af2b7c0cab8bcd8fb4eb289344be", "parents": [ "3fe6c5ce1176cf661dbe71fc43b627c1a742a89a" ], "author": { "name": "Joel Becker", "email": "joel.becker@oracle.com", "time": "Fri Jul 06 23:33:17 2007 -0700" }, "committer": { "name": "Mark Fasheh", "email": "mark.fasheh@oracle.com", "time": "Tue Jul 10 17:10:56 2007 -0700" }, "message": "configfs: Convert subsystem semaphore to mutex\n\nConvert the su_sem member of struct configfs_subsystem to a struct\nmutex, as that\u0027s what it is. Also convert all the users and update\nDocumentation/configfs.txt and Documentation/configfs_example.c\naccordingly.\n\n[ Conflict in fs/dlm/config.c with commit\n 3168b0780d06ace875696f8a648d04d6089654e5 manually resolved. --Mark ]\n\nInspired-by: Satyam Sharma \u003cssatyam@cse.iitk.ac.in\u003e\nSigned-off-by: Joel Becker \u003cjoel.becker@oracle.com\u003e\nSigned-off-by: Mark Fasheh \u003cmark.fasheh@oracle.com\u003e\n" }, { "commit": "3fe6c5ce1176cf661dbe71fc43b627c1a742a89a", "tree": "be94563e2ba49f7116e866e57253b5ed82fb85ee", "parents": [ "9b1d9aa4e9c5cafe73b9df21d758b50b5d75264d" ], "author": { "name": "Satyam Sharma", "email": "ssatyam@cse.iitk.ac.in", "time": "Wed Jul 04 16:37:16 2007 +0530" }, "committer": { "name": "Mark Fasheh", "email": "mark.fasheh@oracle.com", "time": "Tue Jul 10 17:02:31 2007 -0700" }, "message": "[PATCH] configfs+dlm: Rename config_group_find_obj and state semantics clearly\n\nConfigfs being based upon sysfs code, config_group_find_obj() is probably\nso named because of the similar kset_find_obj() in sysfs. However,\n\"kobject\"s in sysfs become \"config_item\"s in configfs, so let\u0027s call it\nconfig_group_find_item() instead, for sake of uniformity, and make\ncorresponding change in the users of this function.\n\nBTW a crucial difference between kset_find_obj and config_group_find_item\nis in locking expectations. kset_find_obj does its locking by itself, but\nconfig_group_find_item expects the *caller* to do the locking. The reason\nfor this: kset\u0027s have their own locks, config_group\u0027s don\u0027t but instead\nrely on the subsystem mutex. And, subsystem needn\u0027t necessarily be around\nwhen config_group_find_item() is called.\n\nSo let\u0027s state these locking semantics explicitly, and rectify the comment,\notherwise bugs could continue to occur in future, as they did in the past\n(refer commit d82b8191e238 in gfs2-2.6-fixes.git).\n\n[ I also took the opportunity to fix some bad whitespace and\ndouble-empty lines. --Joel ]\n\n[ Conflict in fs/dlm/config.c with commit\n 3168b0780d06ace875696f8a648d04d6089654e5 manually resolved. --Mark ]\n\nSigned-off-by: Satyam Sharma \u003cssatyam@cse.iitk.ac.in\u003e\nCc: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Joel Becker \u003cjoel.becker@oracle.com\u003e\nSigned-off-by: Mark Fasheh \u003cmark.fasheh@oracle.com\u003e\n" }, { "commit": "9b1d9aa4e9c5cafe73b9df21d758b50b5d75264d", "tree": "fdb043638b15b429fb3b04be8d56667c3d9963be", "parents": [ "4c62b53454a83178676e5ecae6665447d363c7b4" ], "author": { "name": "Satyam Sharma", "email": "ssatyam@cse.iitk.ac.in", "time": "Wed Jul 04 16:37:06 2007 +0530" }, "committer": { "name": "Mark Fasheh", "email": "mark.fasheh@oracle.com", "time": "Tue Jul 10 16:52:27 2007 -0700" }, "message": "[PATCH] configfs+dlm: Separate out __CONFIGFS_ATTR into configfs.h\n\nfs/dlm/config.c contains a useful generic macro called __CONFIGFS_ATTR\nthat is similar to sysfs\u0027 __ATTR macro that makes defining attributes\neasy for any user of configfs. Separate it out into configfs.h so that\nother users (forthcoming in dynamic netconsole patchset) can use it too.\n\nSigned-off-by: Satyam Sharma \u003cssatyam@cse.iitk.ac.in\u003e\nCc: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Joel Becker \u003cjoel.becker@oracle.com\u003e\nSigned-off-by: Mark Fasheh \u003cmark.fasheh@oracle.com\u003e\n" }, { "commit": "ac90a2552500996c529d5f0ddc16a9bf60bf670d", "tree": "fb546addd48fa75f8f4ea8d7b7929d6c0a3c89ee", "parents": [ "35dcc52e3a916184b145fd840250244b81004200" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri Jul 06 09:47:08 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:24:13 2007 +0100" }, "message": "[DLM] dump more lock values\n\nAdd two more output fields (lkb_flags and rsb nodeid) to the new debugfs\nfile that dumps one lock per line. Also, dump all locks instead of just\nmastered locks. Accordingly, use a suffix of _locks instead of _master.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "f4fadb23ca49abd2f1387a0b7e78b385ebc760ce", "tree": "8e1d7a958694356ab0793309cf5b41e878fe6e59", "parents": [ "b3657629249eba0b3b61ff964d6e1539b469d117" ], "author": { "name": "akpm@linux-foundation.org", "email": "akpm@linux-foundation.org", "time": "Wed Jun 27 14:43:37 2007 -0700" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:24:06 2007 +0100" }, "message": "[GFS2] git-gfs2-nmw-build-fix\n\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "97d848365e603def43c69e160937f073bf9cf02e", "tree": "04aecce7e6dfcccd6afaefce80abf6bf9f08126f", "parents": [ "1875f2f31b3955dff8c3712a56ae61836c8b90fe" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Jun 27 11:36:23 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:23:57 2007 +0100" }, "message": "[DLM] Telnet to port 21064 can stop all lockspaces\n\nThis patch fixes Red Hat bz#245892\n\nOpening a tcp connection from a cluster member to another cluster member\ntargeting the dlm port it is enough to stop every dlm operation in the cluster.\nThis means that GFS and rgmanager will hang.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "fad59c1390045b5adb7c7249ec4e77e0f868aca5", "tree": "d04380e0f72b87ebcc8efc6da68e27ece846c8ea", "parents": [ "d93cfa9884354dac2d8ccd894594a43e0b962b6f" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Jun 11 10:47:18 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:23:38 2007 +0100" }, "message": "[DLM] don\u0027t require FS flag on all nodes\n\nMask off the recently added DLM_LSFL_FS flag when setting the exflags.\nThis way all the nodes in the lockspace aren\u0027t required to have the FS\nflag set, since we later check that exflags matches among all nodes.\n\nSigned-off-by: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "44f487a5536a3afd96a9f571de24c36559e9ae82", "tree": "1bbdb85a23c75a29e49ec0b7a1cbb6834a5b55e6", "parents": [ "292e539e9386823df8aab556f3da09667f78da8c" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Jun 06 09:21:22 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:23:17 2007 +0100" }, "message": "[DLM] variable allocation\n\nAdd a new flag, DLM_LSFL_FS, to be used when a file system creates a lockspace.\nThis flag causes the dlm to use GFP_NOFS for allocations instead of GFP_KERNEL.\n(This updated version of the patch uses gfp_t for ls_allocation.)\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-Off-By: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "292e539e9386823df8aab556f3da09667f78da8c", "tree": "1a4f5354b7f4cf006503885a44273550c482459d", "parents": [ "4bd91ba18198eee42c39d4c334c825d1a0a4b445" ], "author": { "name": "Josef Bacik", "email": "jwhiter@redhat.com", "time": "Tue Jun 05 17:36:38 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:23:15 2007 +0100" }, "message": "[DLM] fix reference counting\n\nThis is a fix for the patch\n\n021d2ff3a08019260a1dc002793c92d6bf18afb6\n\nI left off a dlm_hold_rsb which causes the box to panic if you try to use\ndebugfs. This patch fixes the problem. Sorry about that,\n\nSigned-off-by: Josef Bacik \u003cjwhiter@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "afb853fb4eec380b492a3c369f837359359c28e8", "tree": "ee589e832761f6915db48a222288f68164874a05", "parents": [ "89918647a445fddfe223b097e29f775dcfa81eab" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Fri Jun 01 10:07:26 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:23:05 2007 +0100" }, "message": "[DLM] fix socket shutdown\n\nThis patch clears the user_data of active sockets as part of cleanup.\nThis prevents any late-arriving data from trying to add jobs to the work\nqueue while we are tidying up.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-Off-By: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "0b7cac0fb0e541a7f54d0ba55b31d829ce3dd899", "tree": "4319c315564972493c5d1cebefd01ccc2fd9bb8b", "parents": [ "9dd592d70be0db6fa8e4e19d7642cfaa424b200e" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue May 29 08:47:51 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:59 2007 +0100" }, "message": "[DLM] show default protocol\n\nDisplay the initial value of the \"protocol\" config value in configfs.\nThe default value has always been 0 in the past anyway, so it\u0027s always\nappeared to be correct.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9dd592d70be0db6fa8e4e19d7642cfaa424b200e", "tree": "6694d26986949a0bc8aac3895c7b61f94af97452", "parents": [ "8b4021fa436f7c76a2299e6d85d4d4a619724e9a" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue May 29 08:47:04 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:56 2007 +0100" }, "message": "[DLM] dumping master locks\n\nAdd a new debugfs file that dumps a compact list of mastered locks.\nThis will be used by a userland daemon to collect state for deadlock\ndetection.\n\nAlso, for the existing function that prints all lock state, lock the rsb\nbefore going through the lock lists since they can be changing in the\ncourse of normal dlm activity.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8b4021fa436f7c76a2299e6d85d4d4a619724e9a", "tree": "4e3be51910ede8a36d640229bb1d20d3f7547652", "parents": [ "84d8cd69a8e7f1c9962f46bc79850c9f1f663806" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue May 29 08:46:00 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:54 2007 +0100" }, "message": "[DLM] canceling deadlocked lock\n\nAdd a function that can be used through libdlm by a system daemon to cancel\nanother process\u0027s deadlocked lock. A completion ast with EDEADLK is returned\nto the process waiting for the lock.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "84d8cd69a8e7f1c9962f46bc79850c9f1f663806", "tree": "4c0cedc62e7b94ba2aaa1ea9faeb7de074e08e08", "parents": [ "b3cab7b9a34a6e65c1ca8f80fb57b256d57e8555" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue May 29 08:44:23 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:52 2007 +0100" }, "message": "[DLM] timeout fixes\n\nVarious fixes related to the new timeout feature:\n- add_timeout() missed setting TIMEWARN flag on lkb\u0027s when the\n TIMEOUT flag was already set\n- clear_proc_locks should remove a dead process\u0027s locks from the\n timeout list\n- the end-of-life calculation for user locks needs to consider that\n ETIMEDOUT is equivalent to -DLM_ECANCEL\n- make initial default timewarn_cs config value visible in configfs\n- change bit position of TIMEOUT_CANCEL flag so it\u0027s not copied to\n a remote master node\n- set timestamp on remote lkb\u0027s so a lock dump will display the time\n they\u0027ve been waiting\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b3cab7b9a34a6e65c1ca8f80fb57b256d57e8555", "tree": "571298c30e9e85afa847822488d532b5172f9110", "parents": [ "518bbde33f7a6b4e252ea2591c77ef796065afa8" ], "author": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 29 11:14:21 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:49 2007 +0100" }, "message": "[DLM] Compile fix\n\nA one liner fix which got missed from the earlier patches.\n\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Fabio Massimo Di Nitto \u003cfabbione@ubuntu.com\u003e\nCc: David Teigland \u003cteigland@redhat.com\u003e\n" }, { "commit": "639aca417d91ebba1077a6084e4423af1c1dd811", "tree": "b542aa0720998f1f64fe629bdc950668f33efdf4", "parents": [ "8b0e7b2cf35aa827ed5efb508c1879481b970496" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 16:02:57 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:45 2007 +0100" }, "message": "[DLM] fix compile breakage\n\nIn the rush to get the previous patch set sent, a compilation bug I fixed\nshortly before sending somehow got clobbered, probably by a missed quilt\nrefresh or something.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8b0e7b2cf35aa827ed5efb508c1879481b970496", "tree": "c1304a8b65cd0e1b474623903ba0907bc209f439", "parents": [ "79d72b54483bf81b9f9de0dd555c710ac7267986" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 09:03:35 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:42 2007 +0100" }, "message": "[DLM] wait for config check during join [6/6]\n\nJoining the lockspace should wait for the initial round of inter-node\nconfig checks to complete before returning. This way, if there\u0027s a\nconfiguration mismatch between the joining node and the existing nodes,\nthe join can fail and return an error to the application.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "79d72b54483bf81b9f9de0dd555c710ac7267986", "tree": "271298ddf4864631e7afe0c54239b46e7c4a81dc", "parents": [ "c85d65e91430db94ae9ce0cf38b56e496658b642" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 09:02:20 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:40 2007 +0100" }, "message": "[DLM] fix new_lockspace error exit [5/6]\n\nFix the error path when exiting new_lockspace(). It was kfree\u0027ing the\nlockspace struct at the end, but that\u0027s only valid if it exits before\nkobject_register occured. After kobject_register we have to let the\nkobject do the freeing.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c85d65e91430db94ae9ce0cf38b56e496658b642", "tree": "5ab6e0399594df5fa53793ca8f922e0a55f0aca3", "parents": [ "d7db923ea4990edb5583bf54af868ba687a1bc84" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 09:01:26 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:38 2007 +0100" }, "message": "[DLM] cancel in conversion deadlock [4/6]\n\nWhen conversion deadlock is detected, cancel the conversion and return\nEDEADLK to the application. This is a new default behavior where before\nthe dlm would allow the deadlock to exist indefinately.\n\nThe DLM_LKF_NODLCKWT flag can now be used in a conversion to prevent the\ndlm from performing conversion deadlock detection/cancelation on it.\nThe DLM_LKF_CONVDEADLK flag can continue to be used as before to tell the\ndlm to demote the granted mode of the lock being converted if it gets into\na conversion deadlock.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "d7db923ea4990edb5583bf54af868ba687a1bc84", "tree": "3efa0db00a3a6400f43a65aef2726fa813f9dfa2", "parents": [ "3ae1acf93a21512512f8a78430fcde5992dd208e" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 09:00:32 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:36 2007 +0100" }, "message": "[DLM] dlm_device interface changes [3/6]\n\nChange the user/kernel device interface used by libdlm:\n- Add ability for userspace to check the version of the interface. libdlm\n can now adapt to different versions of the kernel interface.\n- Increase the size of the flags passed in a lock request so all possible\n flags can be used from userspace.\n- Add an opaque \"xid\" value for each lock. This \"transaction id\" will be\n used later to associate locks with each other during deadlock detection.\n- Add a \"timeout\" value for each lock. This is used along with the\n DLM_LKF_TIMEOUT flag.\n\nAlso, remove a fragment of unused code in device_read().\n\nThis patch requires updating libdlm which is backward compatible with\nolder kernels.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3ae1acf93a21512512f8a78430fcde5992dd208e", "tree": "878ed3c619530c6acf862ecf663063f66fc47a06", "parents": [ "85e86edf951a8a39954c0ba1edbe4a58827dcd5c" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 08:59:31 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:33 2007 +0100" }, "message": "[DLM] add lock timeouts and warnings [2/6]\n\nNew features: lock timeouts and time warnings. If the DLM_LKF_TIMEOUT\nflag is set, then the request/conversion will be canceled after waiting\nthe specified number of centiseconds (specified per lock). This feature\nis only available for locks requested through libdlm (can be enabled for\nkernel dlm users if there\u0027s a use for it.)\n\nIf the new DLM_LSFL_TIMEWARN flag is set when creating the lockspace, then\na warning message will be sent to userspace (using genetlink) after a\nrequest/conversion has been waiting for a given number of centiseconds\n(configurable per node). The time warnings will be used in the future\nto do deadlock detection in userspace.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "85e86edf951a8a39954c0ba1edbe4a58827dcd5c", "tree": "0d605cdea61d8301c20a87784456a660aba89017", "parents": [ "916297aad5de2363dccd531873eda55d4d6afb57" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri May 18 08:58:15 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:31 2007 +0100" }, "message": "[DLM] block scand during recovery [1/6]\n\nDon\u0027t let dlm_scand run during recovery since it may try to do a resource\ndirectory removal while the directory nodes are changing.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "916297aad5de2363dccd531873eda55d4d6afb57", "tree": "e8e5eabf529bd4a41d9ee225087e237581ce5ffd", "parents": [ "2a87ab080607d009b8b2a8706f4e27d70402ca9c" ], "author": { "name": "Josef Bacik", "email": "jwhiter@redhat.com", "time": "Wed May 16 15:56:13 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:29 2007 +0100" }, "message": "[DLM] keep dlm from panicing when traversing rsb list in debugfs\n\nThis problem was originally reported against GFS6.1, but the same issue exists\nin upstream DLM. This patch keeps the rsb iterator assigning under the rsbtbl\nlist lock. Each time we process an rsb we grab a reference to it to make sure\nit is not freed out from underneath us, and then put it when we get the next rsb\nin the list or move onto another list.\n\nSigned-off-by: Josef Bacik \u003cjwhiter@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3168b0780d06ace875696f8a648d04d6089654e5", "tree": "b93814b13155cce67432e1ed11bee58c9a5d5a50", "parents": [ "b524fe646c9a226a847e30ca1221dc22e952f16b" ], "author": { "name": "Satyam Sharma", "email": "ssatyam@cse.iitk.ac.in", "time": "Tue May 08 09:18:58 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Jul 09 08:22:10 2007 +0100" }, "message": "[DLM] fix a couple of races\n\nFix two races in fs/dlm/config.c:\n\n(1) Grab the configfs subsystem semaphore before calling\nconfig_group_find_obj() in get_space(). This solves a potential race\nbetween get_space() and concurrent mkdir(2) or rmdir(2).\n\n(2) Grab a reference on the found config_item _while_ holding the configfs\nsubsystem semaphore in get_comm(), and not after it. This solves a\npotential race between get_comm() and concurrent rmdir(2).\n\nSigned-off-by: Satyam Sharma \u003cssatyam@cse.iitk.ac.in\u003e\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "95511ad4342cd094e62c807f6631b9a19cc6b129", "tree": "560bfd2f278e7bd9b67134662750523a2e5171cd", "parents": [ "38377be88ab878f44c687dbd704d4d044b000c97" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Sat Jul 07 06:14:02 2007 +0200" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Sat Jul 07 14:17:43 2007 -0700" }, "message": "DLM must depend on SYSFS\n\nThe dependency of DLM on SYSFS got lost in\ncommit 6ed7257b46709e87d79ac2b6b819b7e0c9184998 resulting in the\nfollowing compile error with CONFIG_DLM\u003dy, CONFIG_SYSFS\u003dn:\n\n\u003c-- snip --\u003e\n\n...\n LD .tmp_vmlinux1\nfs/built-in.o: In function `dlm_lockspace_init\u0027:\n/home/bunk/linux/kernel-2.6/linux-2.6.22-rc6-mm1/fs/dlm/lockspace.c:231: undefined reference to `kernel_subsys\u0027\nfs/built-in.o: In function `configfs_init\u0027:\n/home/bunk/linux/kernel-2.6/linux-2.6.22-rc6-mm1/fs/configfs/mount.c:143: undefined reference to `kernel_subsys\u0027\nmake[1]: *** [.tmp_vmlinux1] Error 1\n\n\u003c-- snip --\u003e\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5cefcab3db2b13093480f2a42bf081574dd72d3d", "tree": "c3755a241553436a1b84d65ad3c00f77ce6d02ad", "parents": [ "5f757f91e70a97eda8f0cc13bddc853209b2d173", "37fde8ca6c60ea61f5e9d7cb877c25ac60e74167" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon May 07 12:26:27 2007 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon May 07 12:26:27 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: (34 commits)\n [GFS2] Uncomment sprintf_symbol calling code\n [DLM] lowcomms style\n [GFS2] printk warning fixes\n [GFS2] Patch to fix mmap of stuffed files\n [GFS2] use lib/parser for parsing mount options\n [DLM] Lowcomms nodeid range \u0026 initialisation fixes\n [DLM] Fix dlm_lowcoms_stop hang\n [DLM] fix mode munging\n [GFS2] lockdump improvements\n [GFS2] Patch to detect corrupt number of dir entries in leaf and/or inode blocks\n [GFS2] bz 236008: Kernel gpf doing cat /debugfs/gfs2/xxx (lock dump)\n [DLM] fs/dlm/ast.c should #include \"ast.h\"\n [DLM] Consolidate transport protocols\n [DLM] Remove redundant assignment\n [GFS2] Fix bz 234168 (ignoring rgrp flags)\n [DLM] change lkid format\n [DLM] interface for purge (2/2)\n [DLM] add orphan purging code (1/2)\n [DLM] split create_message function\n [GFS2] Set drop_count to 0 (off) by default\n ...\n" }, { "commit": "823bccfc4002296ba88c3ad0f049e1abd8108d30", "tree": "5338ae0b32409446af4cd00c5107d9405d5bf0b6", "parents": [ "2609e7b9bebfd433254c02538ba803dc516ff674" ], "author": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Fri Apr 13 13:15:19 2007 -0700" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Wed May 02 18:57:59 2007 -0700" }, "message": "remove \"struct subsystem\" as it is no longer needed\n\nWe need to work on cleaning up the relationship between kobjects, ksets and\nktypes. The removal of \u0027struct subsystem\u0027 is the first step of this,\nespecially as it is not really needed at all.\n\nThanks to Kay for fixing the bugs in this patch.\n\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "617e82e10ccf96a13eb2efd5eac4abef44a87d02", "tree": "9528cdfae4fd8a03ff4edb4acc76d4e9d9e4b71c", "parents": [ "f391a4ead61e4510ff385815ddaf3c0777fbad1b" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Thu Apr 26 13:46:49 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:51 2007 +0100" }, "message": "[DLM] lowcomms style\n\nReplace some printk with log_print, and fix some simple cases of lines\nover 80. Also, return -ENOTCONN if lowcomms_start fails due to no local\nIP address being available.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "30d3a2373f171e62e4032819f55fed2ec887d0b8", "tree": "44b8168d9ae30a89e2fd394696efdc70ec57ae01", "parents": [ "2439fe50724e8693e8b933b3f8125d870bfbdb25" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Apr 23 16:26:21 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:41 2007 +0100" }, "message": "[DLM] Lowcomms nodeid range \u0026 initialisation fixes\n\nFix a few range \u0026 initialization bugs in lowcomms.\n- max_nodeid is really the highest nodeid encountered, so all loops must include\nit in their iterations.\n- clean dlm_local_count \u0026 connection_idr so we can do a clean restart.\n- Remove a spurious BUG_ON\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "2439fe50724e8693e8b933b3f8125d870bfbdb25", "tree": "7d14d4a09aefb1fcd9fe773a093aa4ac710af492", "parents": [ "7d3c1feb80913ba4253c3517d48b9b3741c44fc9" ], "author": { "name": "Josef Bacik", "email": "jwhiter@redhat.com", "time": "Thu Apr 19 17:59:05 2007 -0400" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:38 2007 +0100" }, "message": "[DLM] Fix dlm_lowcoms_stop hang\n\nWhen you attempt to release a lockspace in DLM, it will hang trying to down a\nsemaphore that has already been downed. The attached patch fixes the problem.\n\nSigned-off-by: Josef Bacik \u003cjwhiter@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\n" }, { "commit": "7d3c1feb80913ba4253c3517d48b9b3741c44fc9", "tree": "e52a741e4148c7193617e17d34dc68dc735e0392", "parents": [ "5f8820960cf4fb621483d4a37c24939ad831bfe7" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Thu Apr 19 10:30:41 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:36 2007 +0100" }, "message": "[DLM] fix mode munging\n\nThere are flags to enable two specialized features in the dlm:\n1. CONVDEADLK causes the dlm to resolve conversion deadlocks internally by\n changing the granted mode of locks to NL.\n2. ALTPR/ALTCW cause the dlm to change the requested mode of locks to PR\n or CW to grant them if the normal requested mode can\u0027t be granted.\n\nGFS direct i/o exercises both of these features, especially when mixed\nwith buffered i/o. The dlm has problems with them.\n\nThe first problem is on the master node. If it demotes a lock as a part of\nconverting it, the actual step of converting the lock isn\u0027t being done\nafter the demotion, the lock is just left sitting on the granted queue\nwith a granted mode of NL. I think the mistaken assumption was that the\ncall to grant_pending_locks() would grant it, but that function naturally\ndoesn\u0027t look at locks on the granted queue.\n\nThe second problem is on the process node. If the master either demotes\nor gives an altmode, the munging of the gr/rq modes is never done in the\nprocess copy of the lock, leaving the master/process copies out of sync.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8fa1de386f4d72f0710b389ccf96308fef87df78", "tree": "316233a16c3185fc5f5164c66f20058c5954c739", "parents": [ "6ed7257b46709e87d79ac2b6b819b7e0c9184998" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Wed Apr 04 17:25:29 2007 +0200" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:25 2007 +0100" }, "message": "[DLM] fs/dlm/ast.c should #include \"ast.h\"\n\nEvery file should include the headers containing the prototypes for\nit\u0027s global functions.\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "6ed7257b46709e87d79ac2b6b819b7e0c9184998", "tree": "502f68849175f8fb52bb141501df2df9efc8e06c", "parents": [ "fc7c44f03d95f20b5446d06f5bb9605cddd53203" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Apr 17 15:39:57 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:23 2007 +0100" }, "message": "[DLM] Consolidate transport protocols\n\nThis patch consolidates the TCP \u0026 SCTP protocols for the DLM into a single file\nand makes it switchable at run-time (well, at least before the DLM actually\nstarts up!)\n\nFor RHEL5 this patch requires Neil Horman\u0027s patch that expands the in-kernel\nsocket API but that has already been twice ACKed so it should be OK.\n\nThe patch adds a new lowcomms.c file that replaces the existing lowcomms-sctp.c\n\u0026 lowcomms-tcp.c files.\n\nSigned-off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "fc7c44f03d95f20b5446d06f5bb9605cddd53203", "tree": "28bb0e7ad56e74a49bc7544e8a8c680afe520e93", "parents": [ "a43a49066d36612f3bb46653cdb265a89c235eff" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Apr 10 09:40:19 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:20 2007 +0100" }, "message": "[DLM] Remove redundant assignment\n\nThis patch removes a redundant (and incorrect) assignment from compat_output\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "ce03f12b37c0bd81ad707d3642241cc55c944711", "tree": "11f6eba9b44a29d0c203c99c47d351152d25529a", "parents": [ "72c2be776bd6eec5186e316e6d9dd4aab78d314d" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Apr 02 12:12:55 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:15 2007 +0100" }, "message": "[DLM] change lkid format\n\nA lock id is a uint32 and is used as an opaque reference to the lock. For\nuserland apps, the lkid is passed up, through libdlm, as the return value\nfrom a write() on the dlm device. This created a problem when the high\nbit was 1, making the lkid look like an error. This is fixed by changing\nhow the lkid is composed. The low 16 bits identified the hash bucket for\nthe lock and the high 16 bits were a per-bucket counter (which eventually\nhit 0x8000 causing the problem). These are simply swapped around; the\nnumber of hash table buckets is far below 0x8000, making all lkid\u0027s\npositive when viewed as signed.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "72c2be776bd6eec5186e316e6d9dd4aab78d314d", "tree": "378338f229887375efbb3388068a148227b72b23", "parents": [ "8499137d4ef1829281e04838113b6b09a0bf1269" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri Mar 30 15:06:16 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:12 2007 +0100" }, "message": "[DLM] interface for purge (2/2)\n\nAdd code to accept purge commands from userland.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8499137d4ef1829281e04838113b6b09a0bf1269", "tree": "9083983db5099f0a4764194aabc27f81342f13ea", "parents": [ "7e4dac33594468153c38b5c94d8ebcafb0e0a68d" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri Mar 30 15:02:40 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:10 2007 +0100" }, "message": "[DLM] add orphan purging code (1/2)\n\nAdd code for purging orphan locks. A process can also purge all of its\nown non-orphan locks by passing a pid of zero. Code already exists for\nprocesses to create persistent locks that become orphans when the process\nexits, but the complimentary capability for another process to then purge\nthese orphans has been missing.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "7e4dac33594468153c38b5c94d8ebcafb0e0a68d", "tree": "162910796a02f05843cafaf8a90a81330e66a122", "parents": [ "f01963f2648cfd708ee8d521b3737cfa55ea8795" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Apr 02 09:06:41 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:07 2007 +0100" }, "message": "[DLM] split create_message function\n\nThis splits the current create_message() function into two parts so that\nlater patches can call the new lower-level _create_message() function when\nthey don\u0027t have an rsb struct. No functional change in this patch.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "ef0c2bb05f40f9a0cd2deae63e199bfa62faa7fa", "tree": "df73645f93cfec29fe5b854ff5990a69b03d5c1d", "parents": [ "032067270295cfca11975c0f7b467244aa170c14" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Mar 28 09:56:46 2007 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:11:00 2007 +0100" }, "message": "[DLM] overlapping cancel and unlock\n\nFull cancel and force-unlock support. In the past, cancel and force-unlock\nwouldn\u0027t work if there was another operation in progress on the lock. Now,\nboth cancel and unlock-force can overlap an operation on a lock, meaning there\nmay be 2 or 3 operations in progress on a lock in parallel. This support is\nimportant not only because cancel and force-unlock are explicit operations\nthat an app can use, but both are used implicitly when a process exits while\nholding locks.\n\nSummary of changes:\n\n- add-to and remove-from waiters functions were rewritten to handle situations\n with more than one remote operation outstanding on a lock\n\n- validate_unlock_args detects when an overlapping cancel/unlock-force\n can be sent and when it needs to be delayed until a request/lookup\n reply is received\n\n- processing request/lookup replies detects when cancel/unlock-force\n occured during the op, and carries out the delayed cancel/unlock-force\n\n- manipulation of the \"waiters\" (remote operation) state of a lock moved under\n the standard rsb mutex that protects all the other lock state\n\n- the two recovery routines related to locks on the waiters list changed\n according to the way lkb\u0027s are now locked before accessing waiters state\n\n- waiters recovery detects when lkb\u0027s being recovered have overlapping\n cancel/unlock-force, and may not recover such locks\n\n- revert_lock (cancel) returns a value to distinguish cases where it did\n nothing vs cases where it actually did a cancel; the cancel completion ast\n should only be done when cancel did something\n\n- orphaned locks put on new list so they can be found later for purging\n\n- cancel must be called on a lock when making it an orphan\n\n- flag user locks (ENDOFLIFE) at the end of their useful life (to the\n application) so we can return an error for any further cancel/unlock-force\n\n- we weren\u0027t setting COMP/BAST ast flags if one was already set, so we\u0027d lose\n either a completion or blocking ast\n\n- clear an unread bast on a lock that\u0027s become unlocked\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "032067270295cfca11975c0f7b467244aa170c14", "tree": "8e72e40bf68a0ff7960228ec44f49aafdd691040", "parents": [ "04b933f27bc8e7f3f6423020cec58a4eab3bb7a7" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Mar 26 09:56:00 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:10:57 2007 +0100" }, "message": "[DLM] fix coverity-spotted stupidity\n\nReplacement patch to remove redundant code rather than moving it around.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "254da030dfb1b13d42d07e4292a4790d88c6874f", "tree": "8c8f34d6b33232f8081fff6842930d55b2b41140", "parents": [ "420d2a1028b906f24e836e37089a6ad55ab1848f" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Mar 21 09:23:53 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:10:44 2007 +0100" }, "message": "[DLM] Don\u0027t delete misc device if lockspace removal fails\n\nCurrently if the lockspace removal fails the misc device associated with a\nlockspace is left deleted. After that there is no way to access the orphaned\nlockspace from userland.\n\nThis patch recreates the misc device if th dlm_release_lockspace fails. I\nbelieve this is better than attempting to remove the lockspace first because\nthat leaves an unattached device lying around. The potential gap in which there\nis no access to the lockspace between removing the misc device and recreating it\nis acceptable ... after all the application is trying to remove it, and only new\nusers of the lockspace will be affected.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "89adc934f3f96600e7f31447426c7e99d62c5460", "tree": "7bf6c34cf9898b8c78f76cecd170f946bd0516a9", "parents": [ "5c7342d894973636f03270673e1fb7b908a421a8" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Mar 13 17:08:45 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Tue May 01 09:10:34 2007 +0100" }, "message": "[DLM] Fix uninitialised variable in receiving\n\nThe length of the second element of the kvec array was not initialised before\nbeing added to the first one. This could cause invalid lengths to be passed to\nkernel_recvmsg\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "84c6e8cd359adc34d21e40efcafe09297510b3c8", "tree": "0f90995bf4a558030e0777ba41289134b0cdb8ed", "parents": [ "a13cbe375303585fec1425135ed54adb62be41fc" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Mon Feb 26 00:18:42 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Wed Mar 07 13:58:21 2007 -0500" }, "message": "[DLM] fs/dlm/user.c should #include \"user.h\"\n\nEvery file should include the headers containing the prototypes for\nit\u0027s global functions.\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "00977a59b951207d38380c75f03a36829950265c", "tree": "26933feafebffca95df02c19df03f5e56aada47e", "parents": [ "d54b1fdb1d9f82e375a299e22bd366aad52d4c34" ], "author": { "name": "Arjan van de Ven", "email": "arjan@linux.intel.com", "time": "Mon Feb 12 00:55:34 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Mon Feb 12 09:48:45 2007 -0800" }, "message": "[PATCH] mark struct file_operations const 6\n\nMany struct file_operations in the kernel can be \"const\". Marking them const\nmoves these to the .rodata section, which avoids false sharing with potential\ndirty data. In addition it\u0027ll catch accidental writes at compile time to\nthese shared resources.\n\nSigned-off-by: Arjan van de Ven \u003carjan@linux.intel.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c376222960ae91d5ffb9197ee36771aaed1d9f90", "tree": "7f431c42529fec77433d33490bd9f2a8c47ba091", "parents": [ "1b135431abf5ea92e61bf4e91d93726c7b96da5f" ], "author": { "name": "Robert P. J. Day", "email": "rpjday@mindspring.com", "time": "Sat Feb 10 01:45:03 2007 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Sun Feb 11 10:51:27 2007 -0800" }, "message": "[PATCH] Transform kmem_cache_alloc()+memset(0) -\u003e kmem_cache_zalloc().\n\nReplace appropriate pairs of \"kmem_cache_alloc()\" + \"memset(0)\" with the\ncorresponding \"kmem_cache_zalloc()\" call.\n\nSigned-off-by: Robert P. J. Day \u003crpjday@mindspring.com\u003e\nCc: \"Luck, Tony\" \u003ctony.luck@intel.com\u003e\nCc: Andi Kleen \u003cak@muc.de\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@steeleye.com\u003e\nCc: Greg KH \u003cgreg@kroah.com\u003e\nAcked-by: Joel Becker \u003cJoel.Becker@oracle.com\u003e\nCc: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Jan Kara \u003cjack@ucw.cz\u003e\nCc: Michael Halcrow \u003cmhalcrow@us.ibm.com\u003e\nCc: \"David S. Miller\" \u003cdavem@davemloft.net\u003e\nCc: Stephen Smalley \u003csds@tycho.nsa.gov\u003e\nCc: James Morris \u003cjmorris@namei.org\u003e\nCc: Chris Wright \u003cchrisw@sous-sol.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "58addbffdde731da25a2f9eaa1353fc434f3f3c4", "tree": "875b23f974e17a7221c878ad47bc33313a56377a", "parents": [ "9340b0d356ee52783121af398fa6a332e19e37e2" ], "author": { "name": "Al Viro", "email": "viro@ftp.linux.org.uk", "time": "Fri Feb 09 16:38:45 2007 +0000" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Fri Feb 09 09:14:06 2007 -0800" }, "message": "[PATCH] dlm: use kern_recvmsg()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a34fbc6363256387372331000462691bc4b3f5a9", "tree": "d211465adf24f06d772cfbc78b92269771f46b07", "parents": [ "62a0f62369b0fece37f6652d69b918c89d53c3b3" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Thu Feb 01 16:46:33 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:38:27 2007 -0500" }, "message": "[DLM] fix softlockup in dlm_recv\n\nThis patch stops the dlm_recv workqueue from busy-waiting when a node\ndisconnects. This can cause soft lockup errors on debug systems and bad\nperformance generally.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "62a0f62369b0fece37f6652d69b918c89d53c3b3", "tree": "1b32f039b103dbb34af458ec593728682913505f", "parents": [ "9beeb9f3c5b0401491f59b71521ab2678f584b09" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Jan 31 13:25:00 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:38:24 2007 -0500" }, "message": "[DLM] zero new user lvbs\n\nA new lvb for a userland lock wasn\u0027t being initialized to zero.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9beeb9f3c5b0401491f59b71521ab2678f584b09", "tree": "15e46c270892a7864be1078f5ce962ca8d7503d9", "parents": [ "ddee76089cc9bcbd8ae9ec6c26e726a8ab2fe675" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Tue Jan 30 14:30:08 2007 -0800" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:38:20 2007 -0500" }, "message": "[DLM/GFS2] indent help text\n\nIndent help text as expected.\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "001172778543c6997d3339f43085e43460e5883a", "tree": "73795627b092435cd9f254e5e12439ee16e31340", "parents": [ "67f55897ee5ffa16ca00ed39d176dc52b5066679" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Sun Jan 28 17:19:50 2007 +0100" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:38:08 2007 -0500" }, "message": "[GFS2/DLM] fix GFS2 circular dependency\n\nOn Sun, Jan 28, 2007 at 11:08:18AM +0100, Jiri Slaby wrote:\n\u003e Andrew Morton napsal(a):\n\u003e \u003eTemporarily at\n\u003e \u003e\n\u003e \u003e\thttp://userweb.kernel.org/~akpm/2.6.20-rc6-mm1/\n\u003e\n\u003e Unable to select IPV6. Menuconfig doesn\u0027t offer it when INET is selected.\n\u003e When it\u0027s not it appears in the menu, but after state change it gets away.\n\u003e The same behaviour in xconfig, gconfig.\n\u003e\n\u003e $ mkdir ../a/tst\n\u003e $ make O\u003d../a/tst menuconfig\n\u003e HOSTCC scripts/basic/fixdep\n\u003e [...]\n\u003e HOSTLD scripts/kconfig/mconf\n\u003e scripts/kconfig/mconf arch/i386/Kconfig\n\u003e Warning! Found recursive dependency: INET GFS2_FS_LOCKING_DLM SYSFS\n\u003e OCFS2_FS INET\n\u003e\n\u003e Maybe this is the problem?\n\nYes, patch below.\n\n\u003e regards,\n\ncu\nAdrian\n\n\u003c-- snip --\u003e\n\nThis patch fixes a circular dependency by letting GFS2_FS_LOCKING_DLM\nand DLM depend on instead of select SYSFS.\n\nSince SYSFS depends on EMBEDDED this change shouldn\u0027t cause any problems\nfor users.\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nAcked-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "67f55897ee5ffa16ca00ed39d176dc52b5066679", "tree": "5660a77dce80c07d90f677febc8bbb7e9b12fe29", "parents": [ "ee32e4f3d347e4b562de0bd70be99e622d7d1a9f" ], "author": { "name": "Randy Dunlap", "email": "randy.dunlap@oracle.com", "time": "Thu Jan 25 18:42:39 2007 -0800" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:38:05 2007 -0500" }, "message": "[GFS2/DLM] use sysfs\n\nWith CONFIG_DLM\u003dm, CONFIG_PROC_FS\u003dn, and CONFIG_SYSFS\u003dn, kernel build\nfails with:\n\nWARNING: \"kernel_subsys\" [fs/gfs2/locking/dlm/lock_dlm.ko] undefined!\nWARNING: \"kernel_subsys\" [fs/dlm/dlm.ko] undefined!\nWARNING: \"kernel_subsys\" [fs/configfs/configfs.ko] undefined!\nmake[1]: *** [__modpost] Error 1\nmake: *** [modules] Error 2\n\nSince fs/dlm/lockspace.c and fs/gfs2/locking/dlm/sysfs.c use\nkernel_subsys, they should either DEPEND on it or SELECT it.\n\nSigned-off-by: Randy Dunlap \u003crandy.dunlap@oracle.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b790c3b7c38aae28c497bb363a6fe72f7c96568f", "tree": "e4dda1c5c775b4ae164a997f1216147066de31e3", "parents": [ "8fd3a98f2c22982aff4d29e4ee72959d3032c123" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Jan 24 10:21:33 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:37:50 2007 -0500" }, "message": "[DLM] can miss clearing resend flag\n\nA long, complicated sequence of events, beginning with the RESEND flag not\nbeing cleared on an lkb, can result in an unlock never completing.\n\n- lkb on waiters list for remote lookup\n- the remote node is both the dir node and the master node, so\n it optimizes the lookup into a request and sends a request\n reply back\n- the request reply is saved on the requestqueue to be processed\n after recovery\n- recovery runs dlm_recover_waiters_pre() which sets RESEND flag\n so the lookup will be resent after recovery\n- end of recovery: process_requestqueue takes saved request reply\n which removes the lkb off the waitesr list, _without_ clearing\n the RESEND flag\n- end of recovery: dlm_recover_waiters_post() doesn\u0027t do anything\n with the now completed lookup lkb (would usually clear RESEND)\n- later, the node unmounts, unlocks this lkb that still has RESEND\n flag set\n- the lkb is on the waiters list again, now for unlock, when recovery\n occurs, dlm_recover_waiters_pre() shows the lkb for unlock with RESEND\n set, doesn\u0027t do anything since the master still exists\n- end of recovery: dlm_recover_waiters_post() takes this lkb off\n the waiters list because it has the RESEND flag set, then reports\n an error because unlocks are never supposed to be handled in\n recover_waiters_post().\n- later, the unlock reply is received, doesn\u0027t find the lkb on\n the waiters list because recover_waiters_post() has wrongly\n removed it.\n- the unlock operation has been lost, and we\u0027re left with a\n stray granted lock\n- unmount spins waiting for the unlock to complete\n\nThe visible evidence of this problem will be a node where gfs umount is\nspinning, the dlm waiters list will be empty, and the dlm locks list will\nshow a granted lock.\n\nThe fix is simply to clear the RESEND flag when taking an lkb off the\nwaiters list.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8fd3a98f2c22982aff4d29e4ee72959d3032c123", "tree": "9d4b75dc24d506572d7f2700f0d68f568969e62d", "parents": [ "f1f1c1ccf7848a6e25db30ee9216e1a1e7eb6bef" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Jan 24 10:11:45 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:37:47 2007 -0500" }, "message": "[DLM] saved dlm message can be dropped\n\ndlm_receive_message() returns 0 instead of returning \u0027error\u0027. What would\nhappen is that process_requestqueue would take a saved message off the\nrequestqueue and call receive_message on it. receive_message would then\nsee that recovery had been aborted, set error to EINTR, and \u0027goto out\u0027,\nexpecting that the error would be returned. Instead, 0 was always\nreturned, so process_requestqueue would think that the message had been\nprocessed and delete it instead of saving it to process next time. This\nmeans the message (usually an unlock in my tests) would be lost.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "f1f1c1ccf7848a6e25db30ee9216e1a1e7eb6bef", "tree": "38b5f6ba90143ceea49c62fb824a7b46b1f03636", "parents": [ "d043e1900c97f7282b71844c8530279913b6ec5a" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Jan 24 11:17:59 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:37:44 2007 -0500" }, "message": "[DLM] Make sock_sem into a mutex\n\nNow that there can be multiple dlm_recv threads running we need to prevent two\nrecvs running for the same connection - it\u0027s unlikely but it can happen and it\ncauses message corruption.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "bd44e2b007bc9024bce3357c185b38c73f87c3dd", "tree": "6359c8eb02e4cb1ce26dd8da88e1f032c187ec04", "parents": [ "b5d32bead1578afc5ca817d40c320764d50a8600" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Jan 22 14:51:33 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:37:29 2007 -0500" }, "message": "[DLM] fix lowcomms receiving\n\nThis patch fixes a bug whereby data on a newly accepted connection would be\nignored if it arrived soon after the accept.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "f2f5095f9e63db57faa7cb082e958910ecdd7ad4", "tree": "42344ef03a046bd6c9cf8ca5b14a693094a70e3f", "parents": [ "1c0f4872dc4bbeb2223a300517099786211fce83" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Jan 22 14:50:10 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:37:23 2007 -0500" }, "message": "[DLM] lowcomms tidy\n\nThis patch removes some redundant fields from the connection structure and adds\nsome lockdep annotation to remove spurious warnings.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "222d396092acc11b4af03bede309aa066945e920", "tree": "ec8e0ac0183faad1167af24ff33ba23ba0db85b1", "parents": [ "a1bc86e6bddd34362ca08a3a4d898eb4b5c15215" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Jan 15 10:28:22 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:58 2007 -0500" }, "message": "[DLM] fix master recovery\n\nIf master recovery happens on an rsb in one recovery sequence, then that\nsequence is aborted before lock recovery happens, then in the next\nsequence, we rely on the previous master recovery (which may now be\ninvalid due to another node ignoring a lookup result) and go on do to the\nlock recovery where we get stuck due to an invalid master value.\n\n recovery cycle begins: master of rsb X has left\n nodes A and B send node C an rcom lookup for X to find the new master\n C gets lookup from B first, sets B as new master, and sends reply back to B\n C gets lookup from A next, and sends reply back to A saying B is master\n A gets lookup reply from C and sets B as the new master in the rsb\n recovery cycle on A, B and C is aborted to start a new recovery\n B gets lookup reply from C and ignores it since there\u0027s a new recovery\n recovery cycle begins: some other node has joined\n B doesn\u0027t think it\u0027s the master of X so it doesn\u0027t rebuild it in the directory\n C looks up the master of X, no one is master, so it becomes new master\n B looks up the master of X, finds it\u0027s C\n A believes that B is the master of X, so it sends its lock to B\n B sends an error back to A\n A resends\n this repeats forever, the incorrect master value on A is never corrected\n\nThe fix is to do master recovery on an rsb that still has the NEW_MASTER\nflag set from an earlier recovery sequence, and therefore didn\u0027t complete\nlock recovery.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "a1bc86e6bddd34362ca08a3a4d898eb4b5c15215", "tree": "92b30f8f9400c5a1b71a2e3a17397b9d0b2cc2f2", "parents": [ "1d6e8131cf0064ef5ab5f3411a82b800afbfadee" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Jan 15 10:34:52 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:55 2007 -0500" }, "message": "[DLM] fix user unlocking\n\nWhen a user process exits, we clear all the locks it holds. There is a\nproblem, though, with locks that the process had begun unlocking before it\nexited. We couldn\u0027t find the lkb\u0027s that were in the process of being\nunlocked remotely, to flag that they are DEAD. To solve this, we move\nlkb\u0027s being unlocked onto a new list in the per-process structure that\ntracks what locks the process is holding. We can then go through this\nlist to flag the necessary lkb\u0027s when clearing locks for a process when it\nexits.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1d6e8131cf0064ef5ab5f3411a82b800afbfadee", "tree": "d165b0210a2c12de8848b2b8bc30237183b611dc", "parents": [ "03dc6a538e42bcc8d5dfabcee208b639db85a80c" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Mon Jan 15 14:33:34 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:52 2007 -0500" }, "message": "[DLM] Use workqueues for dlm lowcomms\n\nThis patch converts the DLM TCP lowcomms to use workqueues rather than using its\nown daemon functions. Simultaneously removing a lot of code and making it more\nscalable on multi-processor machines.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "d200778e1257eeb92242355de6f191a0a5ad43c4", "tree": "ff517f0e4af571951df015cf3ceefebc73e017d0", "parents": [ "99fc64874aad1ee0aea5c4d8c07e3529f9d03497" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Jan 09 09:46:02 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:43 2007 -0500" }, "message": "[DLM] expose dlm_config_info fields in configfs\n\nMake the dlm_config_info values readable and writeable via configfs\nentries.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "99fc64874aad1ee0aea5c4d8c07e3529f9d03497", "tree": "46d36b4d11d920e31bbbf04f5d62e676945550e6", "parents": [ "68c817a1c4e21b893672ac73d8a498e6647453aa" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Jan 09 09:44:01 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:40 2007 -0500" }, "message": "[DLM] add config entry to enable log_debug\n\nAdd a new dlm_config_info field to enable log_debug output and change\nlog_debug() to use it.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "68c817a1c4e21b893672ac73d8a498e6647453aa", "tree": "12c0fe9bd087ac1a0189048a7f4535993d6cb697", "parents": [ "8ec6886748443bec53ce9b9bf50cec92bc417a1b" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Jan 09 09:41:48 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:37 2007 -0500" }, "message": "[DLM] rename dlm_config_info fields\n\nAdd a \"ci_\" prefix to the fields in the dlm_config_info struct so that we\ncan use macros to add configfs functions to access them (in a later\npatch). No functional changes in this patch, just naming changes.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8ec6886748443bec53ce9b9bf50cec92bc417a1b", "tree": "f51bdd8cc92931c5f50d1a15181444651ceeeacb", "parents": [ "87d21e07f3880b8d489f0b4a639deb1362101838" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Jan 09 09:38:39 2007 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:34 2007 -0500" }, "message": "[DLM] change some log_error to log_debug\n\nSome common, non-error messages should use log_debug instead of log_error\nso they can be turned off.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "4edde74eedb8bc4c03adc3602b114b72a7ccd13f", "tree": "636a9c5836a6ea132d7316ae101616039e8d768f", "parents": [ "3fb4a251febe70e4c65ea8250545b391fd414d5a" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Jan 02 17:08:54 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:21 2007 -0500" }, "message": "[DLM] Fix spin lock already unlocked bug\n\nI just noticed this message when testing some other changes I\u0027d made to\nlowcomms (to use workqueues) but the problem seems to be in the current\ngit trees too. I\u0027m amazed no-one has seen it.\n\n BUG: spinlock already unlocked on CPU#1, dlm_recoverd/16868\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "3fb4a251febe70e4c65ea8250545b391fd414d5a", "tree": "28de58dc7a76c3e8c00a3cc4d1005196e55fe475", "parents": [ "5509826f1e548d14bb888c1cb6e3bbf23f855770" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Tue Jan 02 17:01:05 2007 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:18 2007 -0500" }, "message": "[DLM] Fix schedule() calls\n\nI was a little over-enthusiastic turning schedule() calls int cond_sched() when fixing the DLM for Andrew Morton.\n\nThese four should really be calls to schedule() or the dlm can busy-wait.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "927255f0383342f5d49b82adb6689b9cba52a6f5", "tree": "128aa611a70e73664d38344ae4da0251635c1699", "parents": [ "075529b5e1ffa8c9864d23930b71b5306a13d9f8" ], "author": { "name": "Adrian Bunk", "email": "bunk@stusta.de", "time": "Tue Dec 19 13:04:03 2006 -0800" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:05 2007 -0500" }, "message": "[DLM] fs/dlm/lowcomms-tcp.c: remove 2 functions\n\nRemove the following unused functions:\n\n- lowcomms_send_message()\n- lowcomms_max_buffer_size()\n\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "075529b5e1ffa8c9864d23930b71b5306a13d9f8", "tree": "414833bf6041a82f9e0cd2f813b05fe2da37ead8", "parents": [ "8d07fd509e9c82a59e37b8b18a2fd0e8ef8fc837" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:40:26 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:36:02 2007 -0500" }, "message": "[DLM] fix lost flags in stub replies\n\nWhen the dlm fakes an unlock/cancel reply from a failed node using a stub\nmessage struct, it wasn\u0027t setting the flags in the stub message. So, in\nthe process of receiving the fake message the lkb flags would be updated\nand cleared from the zero flags in the message. The problem observed in\ntests was the loss of the USER flag which caused the dlm to think a user\nlock was a kernel lock and subsequently fail an assertion checking the\nvalidity of the ast/callback field.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "8d07fd509e9c82a59e37b8b18a2fd0e8ef8fc837", "tree": "165b88e0208e00301a5069507e84df597d636de8", "parents": [ "da49f36f4f64feb281d7663be99e779b2aecc607" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:39:20 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:35:59 2007 -0500" }, "message": "[DLM] fix receive_request() lvb copying\n\nLVB\u0027s are not sent as part of new requests, but the code receiving the\nrequest was copying data into the lvb anyway. The space in the message\nwhere it mistakenly thought the lvb lived actually contained the resource\nname, so it wound up incorrectly copying this name data into the lvb. Fix\nis to just create the lvb, not copy junk into it.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "da49f36f4f64feb281d7663be99e779b2aecc607", "tree": "53e2a66d70eac2d2d3ab4ff9ced6a2d674c0c815", "parents": [ "9e971b715dcc3cd5f4383f2815aaa7e5853d1f7b" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:38:45 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:35:56 2007 -0500" }, "message": "[DLM] fix send_args() lvb copying\n\nThe send_args() function is used to copy parameters into a message for a\nnumber different message types. Only some of those types are set up\nbeforehand (in create_message) to include space for sending lvb data.\nsend_args was wrongly copying the lvb for all message types as long as the\nlock had an lvb. This means that the lvb data was being written past the\nend of the message into unknown space.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "9e971b715dcc3cd5f4383f2815aaa7e5853d1f7b", "tree": "9f8ad2435aef01e725c1b7bc94ef5b787ab9bdf2", "parents": [ "38aa8b0c59c35d10d15ebf00ceee641f9ed7acba" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:37:55 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:35:53 2007 -0500" }, "message": "[DLM] add version check\n\nCheck if we receive a message from another lockspace member running a\nversion of the dlm with an incompatible inter-node message protocol.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "38aa8b0c59c35d10d15ebf00ceee641f9ed7acba", "tree": "17444ed0f0e195677a6faaac31ba296f37b5e148", "parents": [ "dc200a8848cca8b0e99012996c66f4b379a390ed" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:37:16 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:35:50 2007 -0500" }, "message": "[DLM] fix old rcom messages\n\nA reply to a recovery message will often be received after the relevant\nrecovery sequence has aborted and the next recovery sequence has begun.\nWe need to ignore replies to these old messages from the previous\nrecovery. There\u0027s already a way to do this for synchronous recovery\nrequests using the rc_id number, but not for async.\n\nEach recovery sequence already has a locally unique sequence number\nassociated with it. This patch adds a field to the rcom (recovery\nmessage) structure where this recovery sequence number can be placed,\nrc_seq. When a node sends a reply to a recovery request, it copies the\nrc_seq number it received into rc_seq_reply. When the first node receives\nthe reply to its recovery message, it will check whether rc_seq_reply\nmatches the current recovery sequence number, ls_recover_seq, and if not\nthen it ignores the old reply.\n\nAn old, inadequate approach to filtering out old replies (checking if the\ncurrent stage of recovery has moved back to the start) has been removed\nfrom two spots.\n\nThe protocol version number is changed to reflect the different rcom\nstructures.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "dc200a8848cca8b0e99012996c66f4b379a390ed", "tree": "8237c1e9d972beabbc674136d33479eddd396e7d", "parents": [ "c378051177dce4421428fd1691ffdf15ad57c161" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Dec 13 10:36:37 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Mon Feb 05 13:35:47 2007 -0500" }, "message": "[DLM] fix resend rcom lock\n\nThere\u0027s a chance the new master of resource hasn\u0027t learned it\u0027s the new\nmaster before another node sends it a lock during recovery. The node\nsending the lock needs to resend if this happens.\n\n- A sends a master lookup for resource R to C\n- B sends a master lookup for resource R to C\n- C receives A\u0027s lookup, assigns A to be master of R and\n sends a reply back to A\n- C receives B\u0027s lookup and sends a reply back to B saying\n that A is the master\n- B receives lookup reply from C and sends its lock for R to A\n- A receives lock from B, doesn\u0027t think it\u0027s the master of R\n and sends an error back to B\n- A receives lookup reply from C and becomes master of R\n- B gets error back from A and resends its lock back to A\n (this resending is what this patch does)\n- A receives lock from B, it now sees it\u0027s the master of R\n and takes the lock\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "c80e7c83d56866a735236b45441f024b589f9e88", "tree": "071fd93450d534c20594ab8b72def41ea5c61270", "parents": [ "d1998ef38a13c4e74c69df55ccd38b0440c429b2" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Fri Dec 08 14:31:12 2006 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Fri Dec 15 12:51:22 2006 -0500" }, "message": "[DLM] fix compile warning\n\nThis patch fixes a compile warning in lowcomms-tcp.c indicating that\nkmem_cache_t is deprecated.\n\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1c1afa3c053d4ccdf44e5a4e159005cdfd48bfc6", "tree": "3e686ad4cf1ae2300e7190ff83afc3f3dd4ba740", "parents": [ "0a01707b289853f56d1c000057b27e243c039722", "ac33d0710595579e3cfca42dde2257eb0b123f6d" ], "author": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Thu Dec 07 09:13:20 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Thu Dec 07 09:13:20 2006 -0800" }, "message": "Merge master.kernel.org:/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw\n\n* master.kernel.org:/pub/scm/linux/kernel/git/steve/gfs2-2.6-nmw: (73 commits)\n [DLM] Clean up lowcomms\n [GFS2] Change gfs2_fsync() to use write_inode_now()\n [GFS2] Fix indent in recovery.c\n [GFS2] Don\u0027t flush everything on fdatasync\n [GFS2] Add a comment about reading the super block\n [GFS2] Mount problem with the GFS2 code\n [GFS2] Remove gfs2_check_acl()\n [DLM] fix format warnings in rcom.c and recoverd.c\n [GFS2] lock function parameter\n [DLM] don\u0027t accept replies to old recovery messages\n [DLM] fix size of STATUS_REPLY message\n [GFS2] fs/gfs2/log.c:log_bmap() fix printk format warning\n [DLM] fix add_requestqueue checking nodes list\n [GFS2] Fix recursive locking in gfs2_getattr\n [GFS2] Fix recursive locking in gfs2_permission\n [GFS2] Reduce number of arguments to meta_io.c:getbuf()\n [GFS2] Move gfs2_meta_syncfs() into log.c\n [GFS2] Fix journal flush problem\n [GFS2] mark_inode_dirty after write to stuffed file\n [GFS2] Fix glock ordering on inode creation\n ...\n" }, { "commit": "e18b890bb0881bbab6f4f1a6cd20d9c60d66b003", "tree": "4828be07e1c24781c264b42c5a75bcd968223c3f", "parents": [ "441e143e95f5aa1e04026cb0aa71c801ba53982f" ], "author": { "name": "Christoph Lameter", "email": "clameter@sgi.com", "time": "Wed Dec 06 20:33:20 2006 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.osdl.org", "time": "Thu Dec 07 08:39:25 2006 -0800" }, "message": "[PATCH] slab: remove kmem_cache_t\n\nReplace all uses of kmem_cache_t with struct kmem_cache.\n\nThe patch was generated using the following script:\n\n\t#!/bin/sh\n\t#\n\t# Replace one string by another in all the kernel sources.\n\t#\n\n\tset -e\n\n\tfor file in `find * -name \"*.c\" -o -name \"*.h\"|xargs grep -l $1`; do\n\t\tquilt add $file\n\t\tsed -e \"1,\\$s/$1/$2/g\" $file \u003e/tmp/$$\n\t\tmv /tmp/$$ $file\n\t\tquilt refresh\n\tdone\n\nThe script was run like this\n\n\tsh replace kmem_cache_t \"struct kmem_cache\"\n\nSigned-off-by: Christoph Lameter \u003cclameter@sgi.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n" }, { "commit": "ac33d0710595579e3cfca42dde2257eb0b123f6d", "tree": "ebb3050be68aa49666ee03f51ffe2667f5b18c74", "parents": [ "34126f9f41901ca9d7d0031c2b11fc0d6c07b72d" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Dec 06 15:10:37 2006 +0000" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Dec 07 09:25:13 2006 -0500" }, "message": "[DLM] Clean up lowcomms\n\nThis fixes up most of the things pointed out by akpm and Pavel Machek\nwith comments below indicating why some things have been left:\n\nAndrew Morton wrote:\n\u003e\n\u003e\u003e +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)\n\u003e\u003e +{\n\u003e\u003e +\tstruct nodeinfo *ni;\n\u003e\u003e +\tint r;\n\u003e\u003e +\tint n;\n\u003e\u003e +\n\u003e\u003e +\tdown_read(\u0026nodeinfo_lock);\n\u003e\n\u003e Given that this function can sleep, I wonder if `alloc\u0027 is useful.\n\u003e\n\u003e I see lots of callers passing in a literal \"0\" for `alloc\u0027. That\u0027s in fact\n\u003e a secret (GFP_ATOMIC \u0026 ~__GFP_HIGH). I doubt if that\u0027s what you really\n\u003e meant. Particularly as the code could at least have used __GFP_WAIT (aka\n\u003e GFP_NOIO) which is much, much more reliable than \"0\". In fact \"0\" is the\n\u003e least reliable mode possible.\n\u003e\n\u003e IOW, this is all bollixed up.\n\nWhen 0 is passed into nodeid2nodeinfo the function does not try to allocate a\nnew structure at all. it\u0027s an indication that the caller only wants the nodeinfo\nstruct for that nodeid if there actually is one in existance.\nI\u0027ve tidied the function itself so it\u0027s more obvious, (and tidier!)\n\n\u003e\u003e +/* Data received from remote end */\n\u003e\u003e +static int receive_from_sock(void)\n\u003e\u003e +{\n\u003e\u003e +\tint ret \u003d 0;\n\u003e\u003e +\tstruct msghdr msg;\n\u003e\u003e +\tstruct kvec iov[2];\n\u003e\u003e +\tunsigned len;\n\u003e\u003e +\tint r;\n\u003e\u003e +\tstruct sctp_sndrcvinfo *sinfo;\n\u003e\u003e +\tstruct cmsghdr *cmsg;\n\u003e\u003e +\tstruct nodeinfo *ni;\n\u003e\u003e +\n\u003e\u003e +\t/* These two are marginally too big for stack allocation, but this\n\u003e\u003e +\t * function is (currently) only called by dlm_recvd so static should be\n\u003e\u003e +\t * OK.\n\u003e\u003e +\t */\n\u003e\u003e +\tstatic struct sockaddr_storage msgname;\n\u003e\u003e +\tstatic char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];\n\u003e\n\u003e whoa. This is globally singly-threaded code??\n\nYes. it is only ever run in the context of dlm_recvd.\n\u003e\u003e\n\u003e\u003e +static void initiate_association(int nodeid)\n\u003e\u003e +{\n\u003e\u003e +\tstruct sockaddr_storage rem_addr;\n\u003e\u003e +\tstatic char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];\n\u003e\n\u003e Another static buffer to worry about. Globally singly-threaded code?\n\nYes. Only ever called by dlm_sendd.\n\n\u003e\u003e +\n\u003e\u003e +/* Send a message */\n\u003e\u003e +static int send_to_sock(struct nodeinfo *ni)\n\u003e\u003e +{\n\u003e\u003e +\tint ret \u003d 0;\n\u003e\u003e +\tstruct writequeue_entry *e;\n\u003e\u003e +\tint len, offset;\n\u003e\u003e +\tstruct msghdr outmsg;\n\u003e\u003e +\tstatic char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];\n\u003e\n\u003e Singly-threaded?\n\nYep.\n\n\u003e\u003e\n\u003e\u003e +static void dealloc_nodeinfo(void)\n\u003e\u003e +{\n\u003e\u003e +\tint i;\n\u003e\u003e +\n\u003e\u003e +\tfor (i\u003d1; i\u003c\u003dmax_nodeid; i++) {\n\u003e\u003e +\t\tstruct nodeinfo *ni \u003d nodeid2nodeinfo(i, 0);\n\u003e\u003e +\t\tif (ni) {\n\u003e\u003e +\t\t\tidr_remove(\u0026nodeinfo_idr, i);\n\u003e\n\u003e Didn\u0027t that need locking?\n\nNot. it\u0027s only ever called at DLM shutdown after all the other threads\nhave been stopped.\n\n\u003e\u003e\n\u003e\u003e +static int write_list_empty(void)\n\u003e\u003e +{\n\u003e\u003e +\tint status;\n\u003e\u003e +\n\u003e\u003e +\tspin_lock_bh(\u0026write_nodes_lock);\n\u003e\u003e +\tstatus \u003d list_empty(\u0026write_nodes);\n\u003e\u003e +\tspin_unlock_bh(\u0026write_nodes_lock);\n\u003e\u003e +\n\u003e\u003e +\treturn status;\n\u003e\u003e +}\n\u003e\n\u003e This function\u0027s return value is meaningless. As soon as the lock gets\n\u003e dropped, the return value can get out of sync with reality.\n\u003e\n\u003e Looking at the caller, this _might_ happen to be OK, but it\u0027s a nasty and\n\u003e dangerous thing. Really the locking should be moved into the caller.\n\nIt\u0027s just an optimisation to allow the caller to schedule if there is no work\nto do. if something arrives immediately afterwards then it will get picked up\nwhen the process re-awakes (and it will be woken by that arrival).\n\nThe \u0027accepting\u0027 atomic has gone completely. as Andrew pointed out it didn\u0027t\nreally achieve much anyway. I suspect it was a plaster over some other\nstartup or shutdown bug to be honest.\n\n\nSigned-off-by: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\nCc: Andrew Morton \u003cakpm@osdl.org\u003e\nCc: Pavel Machek \u003cpavel@ucw.cz\u003e\n\n" }, { "commit": "57adf7eede38d315e0e328c52484d6a596e9a238", "tree": "6e08a97b0de41ecc46e465d01f100bb32eeffecc", "parents": [ "0ac230699a0f3f0d15ad4e4ad99446dac5b4a21f" ], "author": { "name": "Ryusuke Konishi", "email": "ryusuke@osrg.net", "time": "Wed Nov 29 09:33:48 2006 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:37:22 2006 -0500" }, "message": "[DLM] fix format warnings in rcom.c and recoverd.c\n\nThis fixes the following gcc warnings generated on\nthe architectures where uint64_t !\u003d unsigned long long (e.g. ppc64).\n\nfs/dlm/rcom.c:154: warning: format \u0027%llx\u0027 expects type \u0027long long unsigned int\u0027, but argument 4 has type \u0027uint64_t\u0027\nfs/dlm/rcom.c:154: warning: format \u0027%llx\u0027 expects type \u0027long long unsigned int\u0027, but argument 5 has type \u0027uint64_t\u0027\nfs/dlm/recoverd.c:48: warning: format \u0027%llx\u0027 expects type \u0027long long unsigned int\u0027, but argument 3 has type \u0027uint64_t\u0027\nfs/dlm/recoverd.c:202: warning: format \u0027%llx\u0027 expects type \u0027long long unsigned int\u0027, but argument 3 has type \u0027uint64_t\u0027\nfs/dlm/recoverd.c:210: warning: format \u0027%llx\u0027 expects type \u0027long long unsigned int\u0027, but argument 3 has type \u0027uint64_t\u0027\n\nSigned-off-by: Ryusuke Konishi \u003cryusuke@osrg.net\u003e\nSigned-off-by: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "98f176fb32f33795b6d0f83856008b932123ab38", "tree": "0565bd70a23546469a985b93c34509f7938fbd5b", "parents": [ "1babdb453138f17b8ed3d1d5711089c4e2fa5ace" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Nov 27 13:19:28 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:37:14 2006 -0500" }, "message": "[DLM] don\u0027t accept replies to old recovery messages\n\nWe often abort a recovery after sending a status request to a remote node.\nWe want to ignore any potential status reply we get from the remote node.\nIf we get one of these unwanted replies, we\u0027ve often moved on to the next\nrecovery message and incremented the message sequence counter, so the\nreply will be ignored due to the seq number. In some cases, we\u0027ve not\nmoved on to the next message so the seq number of the reply we want to\nignore is still correct, causing the reply to be accepted. The next\nrecovery message will then mistake this old reply as a new one.\n\nTo fix this, we add the flag RCOM_WAIT to indicate when we can accept a\nnew reply. We clear this flag if we abort recovery while waiting for a\nreply. Before the flag is set again (to allow new replies) we know that\nany old replies will be rejected due to their sequence number. We also\ninitialize the recovery-message sequence number to a random value when a\nlockspace is first created. This makes it clear when messages are being\nrejected from an old instance of a lockspace that has since been\nrecreated.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "1babdb453138f17b8ed3d1d5711089c4e2fa5ace", "tree": "cff82b8349a633cbf770b13522fb180d39f163c2", "parents": [ "aed3255f2267e2d1d95b9cf7f2995ce24e6c873b" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Nov 27 13:18:41 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:37:08 2006 -0500" }, "message": "[DLM] fix size of STATUS_REPLY message\n\nWhen the not_ready routine sends a \"fake\" status reply with blank status\nflags, it needs to use the correct size for a normal STATUS_REPLY by\nincluding the size of the would-be config parameters. We also fill in the\nnon-existant config parameters with an invalid lvblen value so it\u0027s easier\nto notice if these invalid paratmers are ever being used.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "2896ee37ccc1f9acb244c9b02becb74a43661009", "tree": "c7129fd39b33c9c61516419b595f109bff500749", "parents": [ "dcf3dd852f554bb0016aa23892596717cc123a26" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Mon Nov 27 11:31:22 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:37:00 2006 -0500" }, "message": "[DLM] fix add_requestqueue checking nodes list\n\nRequests that arrive after recovery has started are saved in the\nrequestqueue and processed after recovery is done. Some of these requests\nare purged during recovery if they are from nodes that have been removed.\nWe move the purging of the requests (dlm_purge_requestqueue) to later in\nthe recovery sequence which allows the routine saving requests\n(dlm_add_requestqueue) to avoid filtering out requests by nodeid since the\nsame will be done by the purge. The current code has add_requestqueue\nfiltering by nodeid but doesn\u0027t hold any locks when accessing the list of\ncurrent nodes. This also means that we need to call the purge routine\nwhen the lockspace is being shut down since the add routine will not be\nrejecting requests itself any more.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "b98c95af01c10827e3443157651eb469071391a3", "tree": "81e72ef5265cb2f20643a6250b94d3069bc9e789", "parents": [ "6f90a8b1b87f97144911790390d56f695b59db9b" ], "author": { "name": "Patrick Caulfield", "email": "pcaulfie@redhat.com", "time": "Wed Nov 15 12:29:24 2006 -0500" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:41 2006 -0500" }, "message": "[DLM] Fix DLM config\n\nThe attached patch fixes the DLM config so that it selects the chosen network\ntransport. It should fix the bug where DLM can be left selected when NET gets\nunselected. This incorporates all the comments received about this patch.\n\nCc: Adrian Bunk \u003cbunk@stusta.de\u003e\nCc: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-Off-By: Patrick Caulfield \u003cpcaulfie@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "6f90a8b1b87f97144911790390d56f695b59db9b", "tree": "1a6a4d972984554535aec02880af159f45128668", "parents": [ "4b77f2c93d052adca8cc8690b9b5e7f8798f4ddd" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Fri Nov 10 14:16:27 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:27 2006 -0500" }, "message": "[DLM] clear sbflags on lock master\n\nRH BZ 211622\n\nThe ALTMODE flag can be set in the lock master\u0027s copy of the lock but\nnever cleared, so ALTMODE will also be returned in a subsequent conversion\nof the lock when it shouldn\u0027t be. This results in lock_dlm incorrectly\nswitching to the alternate lock mode when returning the result to gfs\nwhich then asserts when it sees the wrong lock state. The fix is to\npropagate the cleared sbflags value to the master node when the lock is\nrequested. QA\u0027s d_rwrandirectlarge test triggers this bug very quickly.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "4b77f2c93d052adca8cc8690b9b5e7f8798f4ddd", "tree": "b61c4923c355d36875bf878212dfc1b2f1f0f7ba", "parents": [ "2cdc98aaf072d573df10c503d3b3b0b74e2a6d06" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Wed Nov 01 09:31:48 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:24 2006 -0500" }, "message": "[DLM] do full recover_locks barrier\n\nRed Hat BZ 211914\n\nThe previous patch \"[DLM] fix aborted recovery during\nnode removal\" was incomplete as discovered with further testing. It set\nthe bit for the RS_LOCKS barrier but did not then wait for the barrier.\nThis is often ok, but sometimes it will cause yet another recovery hang.\nIf it\u0027s a new node that also has the lowest nodeid that skips the barrier\nwait, then it misses the important step of collecting and reporting the\nbarrier status from the other nodes (which is the job of the low nodeid in\nthe barrier wait routine).\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "2cdc98aaf072d573df10c503d3b3b0b74e2a6d06", "tree": "c8f0ed478c6384da504026ce2668ed916b01a6af", "parents": [ "91c0dc93a1a6bbdd79707ed311e48b4397df177f" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Oct 31 11:56:08 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:16 2006 -0500" }, "message": "[DLM] fix stopping unstarted recovery\n\nRed Hat BZ 211914\n\nWhen many nodes are joining a lockspace simultaneously, the dlm gets a\nquick sequence of stop/start events, a pair for adding each node.\ndlm_controld in user space sends dlm_recoverd in the kernel each stop and\nstart event. dlm_controld will sometimes send the stop before\ndlm_recoverd has had a chance to take up the previously queued start. The\nstop aborts the processing of the previous start by setting the\nRECOVERY_STOP flag. dlm_recoverd is erroneously clearing this flag and\nignoring the stop/abort if it happens to take up the start after the stop\nmeant to abort it. The fix is to check the sequence number that\u0027s\nincremented for each stop/start before clearing the flag.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "91c0dc93a1a6bbdd79707ed311e48b4397df177f", "tree": "ec78620714a49fc41244afb667404b0ff825b1a2", "parents": [ "d4400156d415540086c34a06e5d233122d6bf56a" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Oct 31 11:56:01 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:13 2006 -0500" }, "message": "[DLM] fix aborted recovery during node removal\n\nRed Hat BZ 211914\n\nWith the new cluster infrastructure, dlm recovery for a node removal can\nbe aborted and restarted for a node addition. When this happens, the\nrestarted recovery isn\u0027t aware that it\u0027s doing recovery for the earlier\nremoval as well as the addition. So, it then skips the recovery steps\nonly required when nodes are removed. This can result in locks not being\npurged for failed/removed nodes. The fix is to check for removed nodes\nfor which recovery has not been completed at the start of a new recovery\nsequence.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" }, { "commit": "d4400156d415540086c34a06e5d233122d6bf56a", "tree": "747e4d270fb453d57926d6b6cab564664d9d2c0f", "parents": [ "435618b75b82b5ee511cc01fcdda9c44adb2f4bd" ], "author": { "name": "David Teigland", "email": "teigland@redhat.com", "time": "Tue Oct 31 11:55:56 2006 -0600" }, "committer": { "name": "Steven Whitehouse", "email": "swhiteho@redhat.com", "time": "Thu Nov 30 10:35:10 2006 -0500" }, "message": "[DLM] fix requestqueue race\n\nRed Hat BZ 211914\n\nThere\u0027s a race between dlm_recoverd (1) enabling locking and (2) clearing\nout the requestqueue, and dlm_recvd (1) checking if locking is enabled and\n(2) adding a message to the requestqueue. An order of recoverd(1),\nrecvd(1), recvd(2), recoverd(2) will result in a message being left on the\nrequestqueue. The fix is to have dlm_recvd check if dlm_recoverd has\nenabled locking after taking the mutex for the requestqueue and if it has\nprocessing the message instead of queueing it.\n\nSigned-off-by: David Teigland \u003cteigland@redhat.com\u003e\nSigned-off-by: Steven Whitehouse \u003cswhiteho@redhat.com\u003e\n" } ], "next": "435618b75b82b5ee511cc01fcdda9c44adb2f4bd" }