)]}' { "log": [ { "commit": "2cebaa58b7de775386732bbd6cd11c3f5b73faf0", "tree": "aaa392677870ded6c3790e1d327fa12f584e767f", "parents": [ "cf6ace16a3cd8b728fb0afa68368fd40bbeae19f" ], "author": { "name": "Pavel Shilovsky", "email": "piastryyy@gmail.com", "time": "Wed Jul 20 18:24:09 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 21 00:48:05 2011 +0000" }, "message": "CIFS: Fix wrong length in cifs_iovec_read\n\nSigned-off-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e501f29c727dea74e2124c584cbab5fa805d489b", "tree": "3a5c3611e7fc3ac454fd498b93a521d7a1e2be05", "parents": [ "3a5c3743f15f27237ab025736a981e2d0c9fdfed", "59430262401bec02d415179c43dbe5b8819c09ce" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jul 19 21:50:21 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jul 19 21:50:21 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6:\n vfs: fix race in rcu lookup of pruned dentry\n Fix cifs_get_root()\n\n[ Edited the last commit to get rid of a \u0027unused variable \"seq\"\u0027\n warning due to Al editing the patch. - Linus ]\n" }, { "commit": "fec11dd9a0109fe52fd631e5c510778d6cbff6cc", "tree": "2a59d8e2cb0325862740fc24ecb246aa02c8413f", "parents": [ "0916a5e45fbd2604a303c8cc18e6b2b7c815e4c9" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jul 18 13:50:40 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jul 18 13:51:58 2011 -0400" }, "message": "Fix cifs_get_root()\n\nAdd missing -\u003ei_mutex, convert to lookup_one_len() instead of\n(broken) open-coded analog, cope with getting something like\na//b as relative pathname. Simplify the hell out of it, while\nwe are there...\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\n" }, { "commit": "d36c30181c4cf6ead34ae30fa2c777b871225c87", "tree": "8a2476c0eb6bb83ed5a8b493d79458d0e114a146", "parents": [ "a406721dff91a9a5297d140dbb90327966cf9bc0", "0916a5e45fbd2604a303c8cc18e6b2b7c815e4c9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 18 09:03:15 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 18 09:03:15 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6:\n hppfs_lookup(): don\u0027t open-code lookup_one_len()\n hppfs: fix dentry leak\n cramfs: get_cramfs_inode() returns ERR_PTR() on failure\n ufs should use d_splice_alias()\n fix exofs -\u003eget_parent()\n ceph analog of cifs build_path_from_dentry() race fix\n cifs: build_path_from_dentry() race fix\n" }, { "commit": "dc137bf553dbb6855bd7efc34fedcd03102455f7", "tree": "ab1fefc7da26bb86655a28651c4eafb56d741ea1", "parents": [ "3110df800c4de2724624d46e6bed27efc5e9a707" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jul 16 23:37:20 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jul 16 23:37:20 2011 -0400" }, "message": "cifs: build_path_from_dentry() race fix\n\ndeal with d_move() races properly; rename_lock read-retry loop,\nrcu_read_lock() held while walking to root, d_lock held over\nsubtraction from namelen and copying the component to stabilize\n-\u003ed_name.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "c2ec9471b5b1307429aef1cfaa2b3ae453a61d6f", "tree": "ca216e0c2793c56047644496501923d0491cdff2", "parents": [ "ea1be1a3c3c4b2bbc32aeb7995e18336c8060e0e" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 12 19:15:02 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 12 19:15:02 2011 +0000" }, "message": "[CIFS] update cifs to version 1.74\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ea1be1a3c3c4b2bbc32aeb7995e18336c8060e0e", "tree": "b9f057cb63be2fb16a48c5e1a0214dd4089d84c1", "parents": [ "62411ab2fe5f002dff27417630ddf02cc40ca404" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 12 18:24:43 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 12 19:14:24 2011 +0000" }, "message": "[CIFS] update limit for snprintf in cifs_construct_tcon\n\nIn 34c87901e113 \"Shrink stack space usage in cifs_construct_tcon\" we\nchange the size of the username name buffer from MAX_USERNAME_SIZE\n(256) to 28. This call to snprintf() needs to be updated as well.\nReported by Dan Carpenter.\n\nReviewed-by: Dan Carpenter \u003cerror27@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "62411ab2fe5f002dff27417630ddf02cc40ca404", "tree": "0265e2d9cf87d4226ffe3ad909dd46eb6c079823", "parents": [ "620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Sun Jul 10 06:55:32 2011 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 12 19:14:23 2011 +0000" }, "message": "cifs: Fix signing failure when server mandates signing for NTLMSSP\n\nWhen using NTLMSSP authentication mechanism, if server mandates\nsigning, keep the flags in type 3 messages of the NTLMSSP exchange\nsame as in type 1 messages (i.e. keep the indicated capabilities same).\n\nSome of the servers such as Samba, expect the flags such as\nNegotiate_Key_Exchange in type 3 message of NTLMSSP exchange as well.\nSome servers like Windows do not.\n\nhttps://bugzilla.samba.org/show_bug.cgi?id\u003d8212\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "71a1b44b036457169e7974eea0b5b37f64176952", "tree": "f6756944ee46e2cb9225c46941a08db824094140", "parents": [ "c891f2cd89c7a2b4d51d009e94c74a1e31626528", "f484b5d001a972a42129570e98086a2a6d216ce0" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 11 12:48:24 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 11 12:48:24 2011 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6:\n cifs: drop spinlock before calling cifs_put_tlink\n cifs: fix expand_dfs_referral\n cifs: move bdi_setup_and_register outside of CONFIG_CIFS_DFS_UPCALL\n cifs: factor smb_vol allocation out of cifs_setup_volume_info\n cifs: have cifs_cleanup_volume_info not take a double pointer\n cifs: fix build_unc_path_to_root to account for a prefixpath\n cifs: remove bogus call to cifs_cleanup_volume_info\n" }, { "commit": "f484b5d001a972a42129570e98086a2a6d216ce0", "tree": "1345cdda657b020acc0ca5da7645119d6740cbb3", "parents": [ "b9bce2e9f9936cfd12fbc62ead11edcdd46dec7e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 11 10:16:34 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jul 11 18:40:52 2011 +0000" }, "message": "cifs: drop spinlock before calling cifs_put_tlink\n\n...as that function can sleep.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b9bce2e9f9936cfd12fbc62ead11edcdd46dec7e", "tree": "dc2c4677b3530b612d11489800128e53d27b27ba", "parents": [ "20547490c12b0ee3d32152b85e9f9bd183aa7224" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jul 06 08:10:39 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jul 09 21:25:57 2011 +0000" }, "message": "cifs: fix expand_dfs_referral\n\nRegression introduced in commit 724d9f1cfba.\n\nPrior to that, expand_dfs_referral would regenerate the mount data string\nand then call cifs_parse_mount_options to re-parse it (klunky, but it\nworked). The above commit moved cifs_parse_mount_options out of cifs_mount,\nso the re-parsing of the new mount options no longer occurred. Fix it by\nmaking expand_dfs_referral re-parse the mount options.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "20547490c12b0ee3d32152b85e9f9bd183aa7224", "tree": "98cff812d83763a35e1820faad54e01612947771", "parents": [ "04db79b015dafcb79371fda6b5c32ffdbd31a2ff" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Jul 09 12:21:07 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jul 09 20:29:51 2011 +0000" }, "message": "cifs: move bdi_setup_and_register outside of CONFIG_CIFS_DFS_UPCALL\n\nThis needs to be done regardless of whether that KConfig option is set\nor not.\n\nReported-by: Sven-Haegar Koch \u003chaegar@sdinet.de\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "04db79b015dafcb79371fda6b5c32ffdbd31a2ff", "tree": "eef264d76b4f352d56be896b950631d857ec61b0", "parents": [ "f9e59bcba2cff580a3ccf62e89460f9eed295d89" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jul 06 08:10:38 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jul 08 03:51:23 2011 +0000" }, "message": "cifs: factor smb_vol allocation out of cifs_setup_volume_info\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c902ce1bfb40d8b049bd2319b388b4b68b04bc27", "tree": "7dcb1e8378f8ec8e7ad7684cd26e9d5a1b5b22d1", "parents": [ "075d9db13183c102770dc6cefabfee1b832f9614" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Thu Jul 07 12:19:48 2011 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jul 07 13:21:56 2011 -0700" }, "message": "FS-Cache: Add a helper to bulk uncache pages on an inode\n\nAdd an FS-Cache helper to bulk uncache pages on an inode. This will\nonly work for the circumstance where the pages in the cache correspond\n1:1 with the pages attached to an inode\u0027s page cache.\n\nThis is required for CIFS and NFS: When disabling inode cookie, we were\nreturning the cookie and setting cifsi-\u003efscache to NULL but failed to\ninvalidate any previously mapped pages. This resulted in \"Bad page\nstate\" errors and manifested in other kind of errors when running\nfsstress. Fix it by uncaching mapped pages when we disable the inode\ncookie.\n\nThis patch should fix the following oops and \"Bad page state\" errors\nseen during fsstress testing.\n\n ------------[ cut here ]------------\n kernel BUG at fs/cachefiles/namei.c:201!\n invalid opcode: 0000 [#1] SMP\n Pid: 5, comm: kworker/u:0 Not tainted 2.6.38.7-30.fc15.x86_64 #1 Bochs Bochs\n RIP: 0010: cachefiles_walk_to_object+0x436/0x745 [cachefiles]\n RSP: 0018:ffff88002ce6dd00 EFLAGS: 00010282\n RAX: ffff88002ef165f0 RBX: ffff88001811f500 RCX: 0000000000000000\n RDX: 0000000000000000 RSI: 0000000000000100 RDI: 0000000000000282\n RBP: ffff88002ce6dda0 R08: 0000000000000100 R09: ffffffff81b3a300\n R10: 0000ffff00066c0a R11: 0000000000000003 R12: ffff88002ae54840\n R13: ffff88002ae54840 R14: ffff880029c29c00 R15: ffff88001811f4b0\n FS: 00007f394dd32720(0000) GS:ffff88002ef00000(0000) knlGS:0000000000000000\n CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b\n CR2: 00007fffcb62ddf8 CR3: 000000001825f000 CR4: 00000000000006e0\n DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000\n DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400\n Process kworker/u:0 (pid: 5, threadinfo ffff88002ce6c000, task ffff88002ce55cc0)\n Stack:\n 0000000000000246 ffff88002ce55cc0 ffff88002ce6dd58 ffff88001815dc00\n ffff8800185246c0 ffff88001811f618 ffff880029c29d18 ffff88001811f380\n ffff88002ce6dd50 ffffffff814757e4 ffff88002ce6dda0 ffffffff8106ac56\n Call Trace:\n cachefiles_lookup_object+0x78/0xd4 [cachefiles]\n fscache_lookup_object+0x131/0x16d [fscache]\n fscache_object_work_func+0x1bc/0x669 [fscache]\n process_one_work+0x186/0x298\n worker_thread+0xda/0x15d\n kthread+0x84/0x8c\n kernel_thread_helper+0x4/0x10\n RIP cachefiles_walk_to_object+0x436/0x745 [cachefiles]\n ---[ end trace 1d481c9af1804caa ]---\n\nI tested the uncaching by the following means:\n\n (1) Create a big file on my NFS server (104857600 bytes).\n\n (2) Read the file into the cache with md5sum on the NFS client. Look in\n /proc/fs/fscache/stats:\n\n\tPages : mrk\u003d25601 unc\u003d0\n\n (3) Open the file for read/write (\"bash 5\u003c\u003e/warthog/bigfile\"). Look in proc\n again:\n\n\tPages : mrk\u003d25601 unc\u003d25601\n\nReported-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nReviewed-and-Tested-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\ncc: stable@kernel.org\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "f9e59bcba2cff580a3ccf62e89460f9eed295d89", "tree": "0b1409174fa9e2be92b8b34b810749138f66dbee", "parents": [ "b2a0fa152072f0085fa8d8eb0dbf9b3b0c5952fc" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jul 06 08:10:37 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 06 20:03:05 2011 +0000" }, "message": "cifs: have cifs_cleanup_volume_info not take a double pointer\n\n...as that makes for a cumbersome interface. Make it take a regular\nsmb_vol pointer and rely on the caller to zero it out if needed.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b2a0fa152072f0085fa8d8eb0dbf9b3b0c5952fc", "tree": "59e205c8c98301f1465d7ff1bad45b8f833456b2", "parents": [ "677d8537d875832019fa989186f084ba47ecd93d" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jul 06 08:10:36 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 06 20:03:05 2011 +0000" }, "message": "cifs: fix build_unc_path_to_root to account for a prefixpath\n\nRegression introduced by commit f87d39d9513.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "677d8537d875832019fa989186f084ba47ecd93d", "tree": "28e5931d4c480c8a90e722cb0a0e6f1bef46cea9", "parents": [ "de3796e77a587b28f0c9d8e04c02894d3939dc4e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 05 17:37:37 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 06 20:03:04 2011 +0000" }, "message": "cifs: remove bogus call to cifs_cleanup_volume_info\n\nThis call to cifs_cleanup_volume_info is clearly wrong. As soon as it\u0027s\ncalled the following call to cifs_get_tcp_session will oops as the\nvolume_info pointer will then be NULL.\n\nThe caller of cifs_mount should clean up this data since it passed it\nin. There\u0027s no need for us to call this here.\n\nRegression introduced by commit 724d9f1cfba.\n\nReported-by: Adam Williamson \u003cawilliam@redhat.com\u003e\nCc: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ee1b3ea9e6171d7a842527a44873f9f51e6f239b", "tree": "550a4d42bcf0d3e7f70caada1605171fd6e832ad", "parents": [ "daf54f1f363a61c618662ef66d4bf09d2b090941" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jun 21 07:18:26 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jul 01 16:15:30 2011 +0000" }, "message": "cifs: set socket send and receive timeouts before attempting connect\n\nBenjamin S. reported that he was unable to suspend his machine while\nit had a cifs share mounted. The freezer caused this to spew when he\ntried it:\n\n-----------------------[snip]------------------\nPM: Syncing filesystems ... done.\nFreezing user space processes ... (elapsed 0.01 seconds) done.\nFreezing remaining freezable tasks ...\nFreezing of tasks failed after 20.01 seconds (1 tasks refusing to freeze, wq_busy\u003d0):\ncifsd S ffff880127f7b1b0 0 1821 2 0x00800000\n ffff880127f7b1b0 0000000000000046 ffff88005fe008a8 ffff8800ffffffff\n ffff880127cee6b0 0000000000011100 ffff880127737fd8 0000000000004000\n ffff880127737fd8 0000000000011100 ffff880127f7b1b0 ffff880127736010\nCall Trace:\n [\u003cffffffff811e85dd\u003e] ? sk_reset_timer+0xf/0x19\n [\u003cffffffff8122cf3f\u003e] ? tcp_connect+0x43c/0x445\n [\u003cffffffff8123374e\u003e] ? tcp_v4_connect+0x40d/0x47f\n [\u003cffffffff8126ce41\u003e] ? schedule_timeout+0x21/0x1ad\n [\u003cffffffff8126e358\u003e] ? _raw_spin_lock_bh+0x9/0x1f\n [\u003cffffffff811e81c7\u003e] ? release_sock+0x19/0xef\n [\u003cffffffff8123e8be\u003e] ? inet_stream_connect+0x14c/0x24a\n [\u003cffffffff8104485b\u003e] ? autoremove_wake_function+0x0/0x2a\n [\u003cffffffffa02ccfe2\u003e] ? ipv4_connect+0x39c/0x3b5 [cifs]\n [\u003cffffffffa02cd7b7\u003e] ? cifs_reconnect+0x1fc/0x28a [cifs]\n [\u003cffffffffa02cdbdc\u003e] ? cifs_demultiplex_thread+0x397/0xb9f [cifs]\n [\u003cffffffff81076afc\u003e] ? perf_event_exit_task+0xb9/0x1bf\n [\u003cffffffffa02cd845\u003e] ? cifs_demultiplex_thread+0x0/0xb9f [cifs]\n [\u003cffffffffa02cd845\u003e] ? cifs_demultiplex_thread+0x0/0xb9f [cifs]\n [\u003cffffffff810444a1\u003e] ? kthread+0x7a/0x82\n [\u003cffffffff81002d14\u003e] ? kernel_thread_helper+0x4/0x10\n [\u003cffffffff81044427\u003e] ? kthread+0x0/0x82\n [\u003cffffffff81002d10\u003e] ? kernel_thread_helper+0x0/0x10\n\nRestarting tasks ... done.\n-----------------------[snip]------------------\n\nWe do attempt to perform a try_to_freeze in cifs_reconnect, but the\nconnection attempt itself seems to be taking longer than 20s to time\nout. The connect timeout is governed by the socket send and receive\ntimeouts, so we can shorten that period by setting those timeouts\nbefore attempting the connect instead of after.\n\nAdam Williamson tested the patch and said that it seems to have fixed\nsuspending on his laptop when a cifs share is mounted.\n\nReported-by: Benjamin S \u003cda_joind@gmx.net\u003e\nTested-by: Adam Williamson \u003cawilliam@redhat.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "258e43fdb00c73d424b114ae3850accd55b27cbb", "tree": "62f252cc4cde2a816ad2d609d80616322b26da15", "parents": [ "804a007f5401f87fc6e7da1e8884dc32912e35d0", "daf6c4501e048eed56b256401d030d6b28e92ff1" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jun 26 19:40:31 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jun 26 19:40:31 2011 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6:\n cifs: mark CONFIG_CIFS_NFSD_EXPORT as BROKEN\n cifs: free blkcipher in smbhash\n" }, { "commit": "804a007f5401f87fc6e7da1e8884dc32912e35d0", "tree": "c94fda93a6c85b655bb8c13703a97295ce4314f6", "parents": [ "8abf55883431a91d4877933240c8419b7fc17274", "9403c9c598e91d473c0582066e47ed2289292e45" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jun 26 19:39:22 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jun 26 19:39:22 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6:\n cifs: propagate errors from cifs_get_root() to mount(2)\n cifs: tidy cifs_do_mount() up a bit\n cifs: more breakage on mount failures\n cifs: close sget() races\n cifs: pull freeing mountdata/dropping nls/freeing cifs_sb into cifs_umount()\n cifs: move cifs_umount() call into -\u003ekill_sb()\n cifs: pull cifs_mount() call up\n sanitize cifs_umount() prototype\n cifs: initialize -\u003etlink_tree in cifs_setup_cifs_sb()\n cifs: allocate mountdata earlier\n cifs: leak on mount if we share superblock\n cifs: don\u0027t pass superblock to cifs_mount()\n cifs: don\u0027t leak nls on mount failure\n cifs: double free on mount failure\n take bdi setup/destruction into cifs_mount/cifs_umount\n\nAcked-by: Steve French \u003csmfrench@gmail.com\u003e\n" }, { "commit": "9403c9c598e91d473c0582066e47ed2289292e45", "tree": "95f3da790bd27de1b4a16e1f1f6f3952d274b8ab", "parents": [ "5c4f1ad7c6aa3b729bd3a93b80f9417d7e978c32" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 10:02:59 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:43 2011 -0400" }, "message": "cifs: propagate errors from cifs_get_root() to mount(2)\n\n... instead of just failing with -EINVAL\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "5c4f1ad7c6aa3b729bd3a93b80f9417d7e978c32", "tree": "5cae9c65a9be45b7f01efdb10e5c41238f24f6d4", "parents": [ "fa18f1bdce898f0efd0c8639c901d826d01be04f" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:56:55 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: tidy cifs_do_mount() up a bit\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "fa18f1bdce898f0efd0c8639c901d826d01be04f", "tree": "1165e5c3172792bf9d9ce4340380b9e7c73b7499", "parents": [ "ee01a14d9ddcf3f832f9ceb837888501cb496e27" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:50:44 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: more breakage on mount failures\n\nif cifs_get_root() fails, we end up with -\u003emount() returning NULL,\nwhich is not what callers expect. Moreover, in case of superblock\nreuse we end up leaking a superblock reference...\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "ee01a14d9ddcf3f832f9ceb837888501cb496e27", "tree": "a0ae5b81a9b7b26ce4c4a8b412b85f779157b7c4", "parents": [ "d757d71bfc30669a500b72792067e8d1c5d401a5" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:47:23 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: close sget() races\n\nhave -\u003es_fs_info set by the set() callback passed to sget()\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "d757d71bfc30669a500b72792067e8d1c5d401a5", "tree": "faa2fc95514ad2629d89230d1b13f1c6f865c75e", "parents": [ "98ab494dd1d25388981114057cf9446250cc7dc7" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:42:43 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: pull freeing mountdata/dropping nls/freeing cifs_sb into cifs_umount()\n\nall callers of cifs_umount() proceed to do the same thing; pull it into\ncifs_umount() itself.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "98ab494dd1d25388981114057cf9446250cc7dc7", "tree": "504637660e92f4438ed9bd105ac2644ab021813d", "parents": [ "97d1152acec0647b72f8c6ecc57da0d6fed574de" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:32:10 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: move cifs_umount() call into -\u003ekill_sb()\n\ninstead of calling it manually in case if cifs_read_super() fails\nto set -\u003es_root, just call it from -\u003ekill_sb(). cifs_put_super()\nis gone now *and* we have cifs_sb shutdown and destruction done\nafter the superblock is gone from -\u003es_instances.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "97d1152acec0647b72f8c6ecc57da0d6fed574de", "tree": "a5d8fc4720ca10d65e6a56cd7281841c91acabe7", "parents": [ "2a9b99516c662d1713d58648e4a4c9aef72051bc" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:29:57 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: pull cifs_mount() call up\n\n... to the point prior to sget(). Now we have cifs_sb set up early\nenough.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "2a9b99516c662d1713d58648e4a4c9aef72051bc", "tree": "3c85e9cbe43af86d647b3c4d658955c82dc5fb7f", "parents": [ "2ced6f693581357b2a5bf8b031a702c624b12d0d" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:27:16 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "sanitize cifs_umount() prototype\n\na) superblock argument is unused\nb) it always returns 0\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "2ced6f693581357b2a5bf8b031a702c624b12d0d", "tree": "b689f09161fbaf870695f8dc3ded0b61b1c693ce", "parents": [ "5d3bc605cafe3f367b1c43b673bf643245c81626" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:20:04 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:42 2011 -0400" }, "message": "cifs: initialize -\u003etlink_tree in cifs_setup_cifs_sb()\n\nno need to wait until cifs_read_super() and we need it done\nby the time cifs_mount() will be called.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "5d3bc605cafe3f367b1c43b673bf643245c81626", "tree": "e85fb1ad187627a0722dd0b18fe435b43c81ba33", "parents": [ "d687ca380f1a8f3043f42efd2403cbe58c846e70" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:17:28 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "cifs: allocate mountdata earlier\n\npull mountdata allocation up, so that it won\u0027t stand in the way when\nwe lift cifs_mount() to location before sget().\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "d687ca380f1a8f3043f42efd2403cbe58c846e70", "tree": "c18d1ad493a08fbbe2ffcb03a7727f7aa454ff11", "parents": [ "2c6292ae4be00454882246d07f38cdf15a823c2a" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:14:27 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "cifs: leak on mount if we share superblock\n\ncifs_sb and nls end up leaked...\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "2c6292ae4be00454882246d07f38cdf15a823c2a", "tree": "40bfa681a0f94f6df2b3f8140452f0eab7809b1c", "parents": [ "ca171baaad1420a29cca98be5bdf5596cd70b294" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 09:05:48 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "cifs: don\u0027t pass superblock to cifs_mount()\n\nTo close sget() races we\u0027ll need to be able to set cifs_sb up before\nwe get the superblock, so we\u0027ll want to be able to do cifs_mount()\nearlier. Fortunately, it\u0027s easy to do - setting -\u003es_maxbytes can\nbe done in cifs_read_super(), ditto for -\u003es_time_gran and as for\nputting MS_POSIXACL into -\u003es_flags, we can mirror it in -\u003emnt_cifs_flags\nuntil cifs_read_super() is called. Kill unused \u0027devname\u0027 argument,\nwhile we are at it...\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "ca171baaad1420a29cca98be5bdf5596cd70b294", "tree": "0b689378dfa3b10002f60d57b68b4fee05b4e94b", "parents": [ "6d6861757dfadb7d6aec6bb34acd471210a755f9" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 08:49:12 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "cifs: don\u0027t leak nls on mount failure\n\nif cifs_sb allocation fails, we still need to drop nls we\u0027d stashed\ninto volume_info - the one we would\u0027ve copied to cifs_sb if we could\nallocate the latter.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6d6861757dfadb7d6aec6bb34acd471210a755f9", "tree": "6de2a3443afacc51cb307619e0a5d90ec22c0786", "parents": [ "dd8544661947ad6d8d87b3c9d4333bfa1583d1bc" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 08:34:57 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "cifs: double free on mount failure\n\nif we get to out_super with -\u003es_root already set (e.g. with\ncifs_get_root() failure), we\u0027ll end up with cifs_put_super()\ncalled and -\u003emountdata freed twice. We\u0027ll also get cifs_sb\nfreed twice and cifs_sb-\u003elocal_nls dropped twice. The problem\nis, we can get to out_super both with and without -\u003es_root,\nwhich makes -\u003eput_super() a bad place for such work.\n\nSwitch to -\u003ekill_sb(), have all that work done there after\nkill_anon_super(). Unlike -\u003eput_super(), -\u003ekill_sb() is\ncalled by deactivate_locked_super() whether we have -\u003es_root\nor not.\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "dd8544661947ad6d8d87b3c9d4333bfa1583d1bc", "tree": "e1d530ceb7de08402a42034d815492f941bce185", "parents": [ "56299378726d5f2ba8d3c8cbbd13cb280ba45e4f" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 17 08:24:42 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Fri Jun 24 18:39:41 2011 -0400" }, "message": "take bdi setup/destruction into cifs_mount/cifs_umount\n\nAcked-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9b8e072a31180eb5cd6991d08524d9c4fa235ade", "tree": "d5b7ec9e72cc2cce3b14a539e0719e44f9d13f01", "parents": [ "e4fb0edb7c03e5ec19b6f732f1dfbe911212dbde" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jun 21 07:18:26 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 24 17:33:30 2011 +0000" }, "message": "cifs: mark CONFIG_CIFS_NFSD_EXPORT as BROKEN\n\nThis does not work properly with CIFS as current servers do not\nenable support for the FILE_OPEN_BY_FILE_ID on SMB NTCreateX\nand not all NFS clients handle ESTALE.\n\nFor now, it just plain doesn\u0027t work. Mark it BROKEN to discourage\ndistros from enabling it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e4fb0edb7c03e5ec19b6f732f1dfbe911212dbde", "tree": "1ac0749688ab9e90bdf486db591a24536f9eba86", "parents": [ "5220cc9382e11ca955ce946ee6a5bac577bb14ff" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jun 20 14:33:16 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 24 17:03:55 2011 +0000" }, "message": "cifs: free blkcipher in smbhash\n\nThis is currently leaked in the rc \u003d\u003d 0 case.\n\nReported-by: J. Bruce Fields \u003cbfields@fieldses.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1190f6a067bf27b2ee7e06ec0776a17fe0f6c4d8", "tree": "7a754b292e2cac6f5fe808d3e66c90c9689593e9", "parents": [ "446b23a75804d7ffa4cca2d4d8f0afb822108c7e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 22 17:33:57 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 23 17:54:39 2011 +0000" }, "message": "cifs: fix wsize negotiation to respect max buffer size and active signing (try #4)\n\nHopefully last version. Base signing check on CAP_UNIX instead of\ntcon-\u003eunix_ext, also clean up the comments a bit more.\n\nAccording to Hongwei Sun\u0027s blog posting here:\n\n http://blogs.msdn.com/b/openspecification/archive/2009/04/10/smb-maximum-transmit-buffer-size-and-performance-tuning.aspx\n\nCAP_LARGE_WRITEX is ignored when signing is active. Also, the maximum\nsize for a write without CAP_LARGE_WRITEX should be the maxBuf that\nthe server sent in the NEGOTIATE request.\n\nFix the wsize negotiation to take this into account. While we\u0027re at it,\nalter the other wsize definitions to use sizeof(WRITE_REQ) to allow for\nslightly larger amounts of data to potentially be written per request.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "446b23a75804d7ffa4cca2d4d8f0afb822108c7e", "tree": "b39afe031f24ce3c79df35d32381f5331d6b68db", "parents": [ "2992c4bd5742b31a0ee00a76eee9c1c284507418" ], "author": { "name": "Pavel Shilovsky", "email": "piastryyy@gmail.com", "time": "Mon Jun 20 12:33:16 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 22 21:43:56 2011 +0000" }, "message": "CIFS: Fix problem with 3.0-rc1 null user mount failure\n\nFigured it out: it was broken by b946845a9dc523c759cae2b6a0f6827486c3221a commit - \"cifs: cifs_parse_mount_options: do not tokenize mount options in-place\". So, as a quick fix I suggest to apply this patch.\n\n[PATCH] CIFS: Fix kfree() with constant string in a null user case\n\nSigned-off-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ec12781f192568f7ea860f440f890389ba393df7", "tree": "5261d0f1e2f5c6e6a903838133f7bc8929c672a3", "parents": [ "1712c20dae7b770b62b2e3272100b3b40af0157c" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Jun 18 20:03:36 2011 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Mon Jun 20 10:44:07 2011 -0400" }, "message": "cifs_permission() doesn\u0027t need to bail out in RCU mode\n\nnothing potentially blocking except generic_permission(), which\nwill DTRT\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "1252b3013b790c77e1c4f077a40542f86df37fb4", "tree": "dd830c22c120672e80c1f4f69e6c7ab255ea3c9b", "parents": [ "040d15c86747cf44fcf6b8ee19d805d4ef20caf3" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 14 16:19:54 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 14 16:19:54 2011 +0000" }, "message": "[CIFS] update cifs version to 1.73\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "040d15c86747cf44fcf6b8ee19d805d4ef20caf3", "tree": "a4ef01d1071920975bc64f132abdabac9af56c8c", "parents": [ "8d1bca328b7c17af33bcf966d799c556ecbf370f" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 14 15:51:18 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 14 15:51:18 2011 +0000" }, "message": "[CIFS] trivial cleanup fscache cFYI and cERROR messages\n\n... for uniformity and cleaner debug logs.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8d1bca328b7c17af33bcf966d799c556ecbf370f", "tree": "734c3909bd750f8de39d04ee656eae6d31d2e24a", "parents": [ "3e715513643f0207c8f3c22010b54954cd697474" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Jun 11 21:17:10 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 13 20:34:34 2011 +0000" }, "message": "cifs: correctly handle NULL tcon pointer in CIFSTCon\n\nLong ago (in commit 00e485b0), I added some code to handle share-level\npasswords in CIFSTCon. That code ignored the fact that it\u0027s legit to\npass in a NULL tcon pointer when connecting to the IPC$ share on the\nserver.\n\nThis wasn\u0027t really a problem until recently as we only called CIFSTCon\nthis way when the server returned -EREMOTE. With the introduction of\ncommit c1508ca2 however, it gets called this way on every mount, causing\nan oops when share-level security is in effect.\n\nFix this by simply treating a NULL tcon pointer as if user-level\nsecurity were in effect. I\u0027m not aware of any servers that protect the\nIPC$ share with a specific password anyway. Also, add a comment to the\ntop of CIFSTCon to ensure that we don\u0027t make the same mistake again.\n\nCc: \u003cstable@kernel.org\u003e\nReported-by: Martijn Uffing \u003cmp3project@sarijopen.student.utwente.nl\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3e715513643f0207c8f3c22010b54954cd697474", "tree": "7f35f4bc196fff989bb2a17fb22a86e348b03cf9", "parents": [ "7fdbaa1b8daa1009b705985b903e3d2ebccad456" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jun 13 11:50:41 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 13 20:34:34 2011 +0000" }, "message": "cifs: show sec\u003d option in /proc/mounts\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7fdbaa1b8daa1009b705985b903e3d2ebccad456", "tree": "46097ff74b491655845753f8a09216822653c77d", "parents": [ "cd51875d53ae1459a2b09b4338166a218c0635a7" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Jun 10 16:14:57 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 13 20:34:33 2011 +0000" }, "message": "cifs: don\u0027t allow cifs_reconnect to exit with NULL socket pointer\n\nIt\u0027s possible for the following set of events to happen:\n\ncifsd calls cifs_reconnect which reconnects the socket. A userspace\nprocess then calls cifs_negotiate_protocol to handle the NEGOTIATE and\ngets a reply. But, while processing the reply, cifsd calls\ncifs_reconnect again. Eventually the GlobalMid_Lock is dropped and the\nreply from the earlier NEGOTIATE completes and the tcpStatus is set to\nCifsGood. cifs_reconnect then goes through and closes the socket and sets the\npointer to zero, but because the status is now CifsGood, the new socket\nis not created and cifs_reconnect exits with the socket pointer set to\nNULL.\n\nFix this by only setting the tcpStatus to CifsGood if the tcpStatus is\nCifsNeedNegotiate, and by making sure that generic_ip_connect is always\ncalled at least once in cifs_reconnect.\n\nNote that this is not a perfect fix for this issue. It\u0027s still possible\nthat the NEGOTIATE reply is handled after the socket has been closed and\nreconnected. In that case, the socket state will look correct but it no\nNEGOTIATE was performed on it be for the wrong socket. In that situation\nthough the server should just shut down the socket on the next attempted\nsend, rather than causing the oops that occurs today.\n\nCc: \u003cstable@kernel.org\u003e # .38.x: fd88ce9: [CIFS] cifs: clarify the meaning of tcpStatus \u003d\u003d CifsGood\nReported-and-Tested-by: Ben Greear \u003cgreearb@candelatech.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "cd51875d53ae1459a2b09b4338166a218c0635a7", "tree": "701511e617549ae84de010d6849ef544fac0afd0", "parents": [ "40779859de0f73b40390c6401a024d06cf024290" ], "author": { "name": "Pavel Shilovsky", "email": "piastryyy@gmail.com", "time": "Thu Jun 09 12:58:53 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 13 20:34:33 2011 +0000" }, "message": "CIFS: Fix sparse error\n\ncifs_sb_master_tlink was declared as inline, but without a definition.\nRemove the declaration and move the definition up.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastryyy@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "83fb086e0ecd879a4676cf12fc7afc1f9ecd1784", "tree": "58961f4e61c2eacac3fa3fb985e13b2ab5167e9e", "parents": [ "9c4843ea576107a3c1fb94f2f758f198e9fe9e54" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 08 07:35:24 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 08 16:03:29 2011 +0000" }, "message": "cifs: trivial: add space in fsc error message\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9c4843ea576107a3c1fb94f2f758f198e9fe9e54", "tree": "3860494ee8307b603a654e8f43685943974f7171", "parents": [ "243e2dd38e1b322b29a1714034cc60b84d3d5e07" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jun 06 15:40:23 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 07 00:57:05 2011 +0000" }, "message": "cifs: silence printk when establishing first session on socket\n\nWhen signing is enabled, the first session that\u0027s established on a\nsocket will cause a printk like this to pop:\n\n CIFS VFS: Unexpected SMB signature\n\nThis is because the key exchange hasn\u0027t happened yet, so the signature\nfield is bogus. Don\u0027t try to check the signature on the socket until the\nfirst session has been established. Also, eliminate the specific check\nfor SMB_COM_NEGOTIATE since this check covers that case too.\n\nCc: stable@kernel.org\nCc: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "243e2dd38e1b322b29a1714034cc60b84d3d5e07", "tree": "f459f208b78dd6ccb73ccd7f35dd4f0a0f1bc24a", "parents": [ "957df4535d06a8e009101239937ca5e50a6218c6" ], "author": { "name": "Darren Salt", "email": "linux@youmustbejoking.demon.co.uk", "time": "Mon Jun 06 16:58:16 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 06 16:58:16 2011 +0000" }, "message": "CIFS ACL support needs CONFIG_KEYS, so depend on it\n\nBuild fails if CONFIG_KEYS is not selected.\n\nSigned-off-by: Darren Salt \u003clinux@youmustbejoking.demon.co.uk\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "957df4535d06a8e009101239937ca5e50a6218c6", "tree": "a61f7cd789b4b43e49ebd0751ecc56858a51e117", "parents": [ "5f0b23eeba2d9105944148e5a85b0bfb34a8ecf5" ], "author": { "name": "Vasily Averin", "email": "vvs@parallels.com", "time": "Mon Jun 06 11:33:12 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 06 15:31:29 2011 +0000" }, "message": "possible memory corruption in cifs_parse_mount_options()\n\nerror path after mountdata check frees uninitialized mountdata_copy\n\nSigned-off-by: Vasily Averin \u003cvvs@sw.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5f0b23eeba2d9105944148e5a85b0bfb34a8ecf5", "tree": "5249d7e1ba44c4e8136b4617371cdd32b8f59389", "parents": [ "c592a7073796de1cd221f957ab693c19d383423f" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Fri Jun 03 14:19:01 2011 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 03 17:38:44 2011 +0000" }, "message": "cifs: make CIFS depend on CRYPTO_ECB\n\nWhen CONFIG_CRYPTO_ECB is not set, trying to mount a CIFS share with NTLM\nsecurity resulted in mount failure with the following error:\n \"CIFS VFS: could not allocate des crypto API\"\n\nSeems like a leftover from commit 43988d7.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nCC: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c592a7073796de1cd221f957ab693c19d383423f", "tree": "ca7df5d91211a3b15b6b7cbeada83cab6e8af277", "parents": [ "1fa7b6a29c61358cc2ca6f64cef4aa0e1a7ca74c" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Fri Jun 03 12:06:19 2011 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 03 15:31:23 2011 +0000" }, "message": "cifs: fix the kernel release version in the default security warning message\n\nWhen ntlm security mechanim is used, the message that warns about the upgrade\nto ntlmv2 got the kernel release version wrong (Blame it on Linus :). Fix it.\n\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ef1d57599dc904fdb31b8e9b5336350d21a1fde1", "tree": "4e3f7d49c5a8c27efd5a1009131f4b3f87eff503", "parents": [ "bc658c96037fc87463f0703ad2ea7c895344cb7e" ], "author": { "name": "Al Viro", "email": "viro@ZenIV.linux.org.uk", "time": "Sun May 29 13:46:08 2011 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun May 29 11:17:34 2011 -0700" }, "message": "cifs/ubifs: Fix shrinker API change fallout\n\nCommit 1495f230fa77 (\"vmscan: change shrinker API by passing\nshrink_control struct\") changed the API of -\u003eshrink(), but missed ubifs\nand cifs instances.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4f61258f6111e2afd56cf40989e5a43cba9e59c8", "tree": "ff30519ba0a2072970a774ff0225911f89d58a14", "parents": [ "e52e713ec30a31e9a4663d9aebbaae5ec07466a6" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 20:40:18 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 20:40:18 2011 +0000" }, "message": "[CIFS] Follow on to cifsacl endian patch (__constant_cpu_to_le32 was required)\n\nAs Jeff just pointed out, __constant_cpu_to_le32 was required instead of\ncpu_to_le32 in previous patch to cifsacl.c 383c55350fb4ab6bd08abfab82038ae0364f1f48\n(Fix endian error comparing authusers when cifsacl enabled)\n\nCC: Stable \u003cstable@kernel.org\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "383c55350fb4ab6bd08abfab82038ae0364f1f48", "tree": "1e390d83133414f08cf15490027c50f02c257d1a", "parents": [ "96daf2b09178d8ebde2b0d56b027de917c17dfdf" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 15:19:12 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 15:19:12 2011 +0000" }, "message": "[CIFS] Fix endian error comparing authusers when cifsacl enabled\n\nFix sparse warning:\n CHECK fs/cifs/cifsacl.c\nfs/cifs/cifsacl.c:41:36: warning: incorrect type in initializer\n(different base types)\nfs/cifs/cifsacl.c:41:36: expected restricted __le32\nfs/cifs/cifsacl.c:41:36: got int\nfs/cifs/cifsacl.c:461:52: warning: restricted __le32 degrades to integer\nfs/cifs/cifsacl.c:461:73: warning: restricted __le32 degrades to integer\n\nThe second one looks harmless but the first one (sid_authusers)\nwas added in commit 2fbc2f1729e785a7b2faf9d8d60926bb1ff62af0\nand only affects 2.6.38/2.6.39\n\nCC: Stable \u003cstable@kernel.org\u003e\nReviewed-and-Tested-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "96daf2b09178d8ebde2b0d56b027de917c17dfdf", "tree": "74802fa44e87925e49067d5106eb762cc27b9b97", "parents": [ "07cc6cf9ef84bcf6a60ee513332bcb0ad5d628d8" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 04:34:02 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 04:34:02 2011 +0000" }, "message": "[CIFS] Rename three structures to avoid camel case\n\nsecMode to sec_mode\nand\ncifsTconInfo to cifs_tcon\nand\ncifsSesInfo to cifs_ses\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "07cc6cf9ef84bcf6a60ee513332bcb0ad5d628d8", "tree": "f293d656be1875ebc1d1ff6aa84c2a356c23b891", "parents": [ "d4ffff1fa9695c5b5c0bf337e208d8833b88ff2d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 04:12:29 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 04:21:29 2011 +0000" }, "message": "Fix extended security auth failure\n\nFix authentication failures using extended security mechanisms.\ncifs client does not take into consideration extended security bit\nin capabilities field in negotiate protocol response from the server.\n\nPlease refer to Samba bugzilla 8046.\n\nReported-and-tested by: Werner Maes \u003cWerner.Maes@icts.kuleuven.be\u003e\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d4ffff1fa9695c5b5c0bf337e208d8833b88ff2d", "tree": "acd4b6cfa7962a1cee7e9c81f11bd9ccb0d3ff24", "parents": [ "25c7f41e9234f60af30e086278f1de7974f8816f" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 26 06:02:00 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 03:57:16 2011 +0000" }, "message": "CIFS: Add rwpidforward mount option\n\nAdd rwpidforward mount option that switches on a mode when we forward\npid of a process who opened a file to any read and write operation.\n\nThis can prevent applications like WINE from failing on read or write\noperation on a previously locked file region from the same netfd from\nanother process if we use mandatory brlock style.\n\nIt is actual for WINE because during a run of WINE program two processes\nwork on the same netfd - share the same file struct between several VFS\nfds:\n1) WINE-server does open and lock;\n2) WINE-application does read and write.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "25c7f41e9234f60af30e086278f1de7974f8816f", "tree": "41cb4da532da30418da2d33e5376ddbf9f65958a", "parents": [ "f87d39d951329cd8f462bf9007d334122c0599d0" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 26 23:35:47 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 03:53:23 2011 +0000" }, "message": "CIFS: Migrate to shared superblock model\n\nAdd cifs_match_super to use in sget to share superblock between mounts\nthat have the same //server/sharename, credentials and mount options.\nIt helps us to improve performance on work with future SMB2.1 leases.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f87d39d951329cd8f462bf9007d334122c0599d0", "tree": "ba4c4d9dee6958c7d8ee87070a58d80389be2d2f", "parents": [ "641a58d66d086327042e9d73c6606fd02c8f067c" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 03:50:55 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 03:50:55 2011 +0000" }, "message": "[CIFS] Migrate from prefixpath logic\n\nNow we point superblock to a server share root and set a root dentry\nappropriately. This let us share superblock between mounts like\n//server/sharename/foo/bar and //server/sharename/foo further.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "641a58d66d086327042e9d73c6606fd02c8f067c", "tree": "d34ce82a1971daef0735c398924e3161c22b8391", "parents": [ "6848b7334b24b47aa3d0e70342ff839ffa95d5fa" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 26 00:02:16 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 27 03:45:37 2011 +0000" }, "message": "CIFS: Fix memory leak in cifs_do_mount\n\nand simplify error handling code.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6848b7334b24b47aa3d0e70342ff839ffa95d5fa", "tree": "6e28dfc52d0625569293b02969416315af3046f6", "parents": [ "fa2989f4473413a86890066aa3a5676a53b541e4" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 26 18:38:54 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 26 18:38:54 2011 +0000" }, "message": "[CIFS] When mandatory encryption on share, fail mount\n\n When mandatory encryption is configured in samba server on a\n share (smb.conf parameter \"smb encrypt \u003d mandatory\") the\n server will hang up the tcp session when we try to send\n the first frame after the tree connect if it is not a\n QueryFSUnixInfo, this causes cifs mount to hang (it must\n be killed with ctl-c). Move the QueryFSUnixInfo call\n earlier in the mount sequence, and check whether the SetFSUnixInfo\n fails due to mandatory encryption so we can return a sensible\n error (EACCES) on mount.\n\n In a future patch (for 2.6.40) we will support mandatory\n encryption.\n\nCC: Stable \u003cstable@kernel.org\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fa2989f4473413a86890066aa3a5676a53b541e4", "tree": "a033c59307ba77ae0819e818aec7d68178d11d4d", "parents": [ "c28c89fc43e3f81436efc4748837534d4d46f90c" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 26 10:01:59 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 26 18:07:02 2011 +0000" }, "message": "CIFS: Use pid saved from cifsFileInfo in writepages and set_file_size\n\nWe need it to make them work with mandatory locking style because\nwe can fail in a situation like when kernel need to flush dirty pages\nand there is a lock held by a process who opened file.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c28c89fc43e3f81436efc4748837534d4d46f90c", "tree": "d7eec9d8aabf41d38dcecab0de6f91b6a8a037c6", "parents": [ "f7910cbd9fa319ee4501074f1f3b5ce23c4b1518" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:56 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 25 20:38:33 2011 +0000" }, "message": "cifs: add cifs_async_writev\n\nAdd the ability for CIFS to do an asynchronous write. The kernel will\nset the frame up as it would for a \"normal\" SMBWrite2 request, and use\ncifs_call_async to send it. The mid callback will then be configured to\nhandle the result.\n\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f7910cbd9fa319ee4501074f1f3b5ce23c4b1518", "tree": "940a9f9836771d2b8820dff93010457ed3e0444c", "parents": [ "c3d17b63e5eafcaf2678c11de801c189468631c8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:58 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 25 20:12:16 2011 +0000" }, "message": "cifs: clean up wsize negotiation and allow for larger wsize\n\nNow that we can handle larger wsizes in writepages, fix up the\nnegotiation of the wsize to allow for that. find_get_pages only seems to\ngive out a max of 256 pages at a time, so that gives us a reasonable\ndefault of 1M for the wsize.\n\nIf the server however does not support large writes via POSIX\nextensions, then we cap the wsize to (128k - PAGE_CACHE_SIZE). That\ngives us a size that goes up to the max frame size specified in RFC1001.\n\nFinally, if CAP_LARGE_WRITE_AND_X isn\u0027t set, then further cap it to the\nlargest size allowed by the protocol (USHRT_MAX).\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-and-Tested-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c3d17b63e5eafcaf2678c11de801c189468631c8", "tree": "ce9edbb87abc452bf7ae1a3b33e2b5cac42ab023", "parents": [ "b2e5cd33b598fb496b9366c445bd77c801efabb8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:57 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 25 20:05:03 2011 +0000" }, "message": "cifs: convert cifs_writepages to use async writes\n\nHave cifs_writepages issue asynchronous writes instead of waiting on\neach write call to complete before issuing another. This also allows us\nto return more quickly from writepages. It can just send out all of the\nI/Os and not wait around for the replies.\n\nIn the WB_SYNC_ALL case, if the write completes with a retryable error,\nthen the completion workqueue job will resend the write.\n\nThis also changes the page locking semantics a little bit. Instead of\nholding the page lock until the response is received, release it after\ndoing the send. This will reduce contention for the page lock and should\nprevent processes that have the file mmap\u0027ed from being blocked\nunnecessarily.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-and-Tested-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b2e5cd33b598fb496b9366c445bd77c801efabb8", "tree": "19f404a46cc637c7d6034deec5d7b1f9b0553924", "parents": [ "3c1105df699188a70f5c17dc0795affea388bca7" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Wed May 25 13:35:34 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 25 20:02:31 2011 +0000" }, "message": "CIFS: Fix undefined behavior when mount fails\n\nFix double kfree() calls on the same pointers and cleanup mount code.\n\nReviewed-and-Tested-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3c1105df699188a70f5c17dc0795affea388bca7", "tree": "bfbcfaaae2fdfd16b2fcdb57710b9affbf183c49", "parents": [ "724d9f1cfba0cb16a7151333b501e8f7885450d8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sun May 22 07:09:13 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 24 03:11:33 2011 +0000" }, "message": "cifs: don\u0027t call mid_q_entry-\u003ecallback under the Global_MidLock (try #5)\n\nMinor revision to the last version of this patch -- the only difference\nis the fix to the cFYI statement in cifs_reconnect.\n\nHolding the spinlock while we call this function means that it can\u0027t\nsleep, which really limits what it can do. Taking it out from under\nthe spinlock also means less contention for this global lock.\n\nChange the semantics such that the Global_MidLock is not held when\nthe callback is called. To do this requires that we take extra care\nnot to have sync_mid_result remove the mid from the list when the\nmid is in a state where that has already happened. This prevents\nlist corruption when the mid is sitting on a private list for\nreconnect or when cifsd is coming down.\n\nReviewed-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "724d9f1cfba0cb16a7151333b501e8f7885450d8", "tree": "d86b0e84a9475374cdbd05cc1e4f85fd0231162f", "parents": [ "37bb04e5a091a5330faef0cc09930326672b7061" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 05 09:55:12 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 24 03:07:42 2011 +0000" }, "message": "CIFS: Simplify mount code for further shared sb capability\n\nReorganize code to get mount option at first and when get a superblock.\nThis lets us use shared superblock model further for equal mounts.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "37bb04e5a091a5330faef0cc09930326672b7061", "tree": "cc7f143162ad2fb8cb22caf427ea7972d05dced6", "parents": [ "c02324a6ae6bd3e83533c59525111ffe03b7e791" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu May 05 09:55:11 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 23 19:05:09 2011 +0000" }, "message": "CIFS: Simplify connection structure search calls\n\nUse separate functions for comparison between existing structure\nand what we are requesting for to make server, session and tcon\nsearch code easier to use on next superblock match call.\n\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c02324a6ae6bd3e83533c59525111ffe03b7e791", "tree": "0c5c204b4c9a6167fdca73ad19eae27ccb41608b", "parents": [ "59ffd84141472c45a1729a739e4730f974d26e63" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Sun May 22 07:04:51 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 23 18:08:05 2011 +0000" }, "message": "cifs: remove unused SMB2 config and mount options\n\nThere\u0027s no SMB2 support in the CIFS filesystem driver, so there\u0027s no need to\nhave a config and mount option for it.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "59ffd84141472c45a1729a739e4730f974d26e63", "tree": "4790475b37a8130049ca6b33c2fe4115ee709e94", "parents": [ "fcc31cb6f1ae6c4e64a463b124d3c2e9cbd7dd70" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:55 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 23 02:59:16 2011 +0000" }, "message": "cifs: add ignore_pend flag to cifs_call_async\n\nThe current code always ignores the max_pending limit. Have it instead\nonly optionally ignore the pending limit. For CIFSSMBEcho, we need to\nignore it to make sure they always can go out. For async reads, writes\nand potentially other calls, we need to respect it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fcc31cb6f1ae6c4e64a463b124d3c2e9cbd7dd70", "tree": "cf4b73bc9d8d5e71e71b76d69550a70f58fb8278", "parents": [ "2c8f981d93f830c167c811f046b5107cc24b6e1b" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:53 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 23 02:58:26 2011 +0000" }, "message": "cifs: make cifs_send_async take a kvec array\n\nWe\u0027ll need this for async writes, so convert the call to take a kvec\narray. CIFSSMBEcho is changed to put a kvec on the stack and pass\nin the SMB buffer using that.\n\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2c8f981d93f830c167c811f046b5107cc24b6e1b", "tree": "8150cda7231acf1afc073b2eaaed0947d1937e16", "parents": [ "71a8638480eb8fb6cfabe2ee9ca3fbc6e3453a14" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 19 16:22:52 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 23 02:58:24 2011 +0000" }, "message": "cifs: consolidate SendReceive response checks\n\nFurther consolidate the SendReceive code by moving the checks run over\nthe packet into a separate function that all the SendReceive variants\ncan call.\n\nWe can also eliminate the check for a receive_len that\u0027s too big or too\nsmall. cifs_demultiplex_thread already checks that and disconnects the\nsocket if that occurs, while setting the midStatus to MALFORMED. It\u0027ll\nnever call this code if that\u0027s the case.\n\nFinally do a little cleanup. Use \"goto out\" on errors so that the flow\nof code in the normal case is more evident. Also switch the logErr\nvariable in map_smb_to_linux_error to a bool.\n\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "156ecb2d8b06589098f6ce3012e6a10fef07c416", "tree": "4008ea00d8c49ff2361ebe784a1e757c8cad9986", "parents": [ "ceec1e0fae6eecac876ec28cfb97786f87dfb1c6" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 20 17:00:01 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 20 17:00:01 2011 +0000" }, "message": "[CIFS] Fix to problem with getattr caused by invalidate simplification patch\n\nFix to earlier \"Simplify invalidate part (try #6)\" patch\nThat patch caused problems with connectathon test 5.\n\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ceec1e0fae6eecac876ec28cfb97786f87dfb1c6", "tree": "834a9d5566dc4f3703dce20d5ebf37df2f9a218f", "parents": [ "4e64fb33dea9c4cad0106760c49c7629d4c699a5" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Mar 14 19:01:59 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:56 2011 +0000" }, "message": "[CIFS] Remove sparse warning\n\nMove extern for cifsConvertToUCS to different header to prevent following warning:\n\nCHECK fs/cifs/cifs_unicode.c\nfs/cifs/cifs_unicode.c:267:1: warning: symbol \u0027cifsConvertToUCS\u0027 was not declared. Should it be static?\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4e64fb33dea9c4cad0106760c49c7629d4c699a5", "tree": "b898bcf8766b25d40472f0d5952458dd0d79443d", "parents": [ "c4aca0c09f80ca40dbcecb2370af9594fbe9051d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 18 18:08:28 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:56 2011 +0000" }, "message": "[CIFS] Update cifs to version 1.72\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c4aca0c09f80ca40dbcecb2370af9594fbe9051d", "tree": "27390f82234bf8ee4376bbececfc8a247fa0299e", "parents": [ "f14bcf71d1b802f6042b6c70a0c37120e47a0876" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Fri May 06 02:35:00 2011 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:56 2011 +0000" }, "message": "cifs: Change key name to cifs.idmap, misc. clean-up\n\nChange idmap key name from cifs.cifs_idmap to cifs.idmap.\nRemoved unused structure wksidarr and function match_sid().\nHandle errors correctly in function init_cifs().\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f14bcf71d1b802f6042b6c70a0c37120e47a0876", "tree": "9852d20c31ec19caaccec30a0b06b89542681ac7", "parents": [ "5167f11ec962690ecf926fab00f1d0524cd78664" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:35 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:55 2011 +0000" }, "message": "cifs: Unconditionally copy mount options to superblock info\n\nPreviously mount options were copied and updated in the cifs_sb_info\nstruct only when CONFIG_CIFS_DFS_UPCALL was enabled. Making this\ninformation generally available allows us to remove a number of ifdefs,\nextra function params, and temporary variables.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5167f11ec962690ecf926fab00f1d0524cd78664", "tree": "06797f7f000ad72df4127fa78c673772d93ddede", "parents": [ "046462abca2576cc1c71f501c148798fac4ea314" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:34 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:55 2011 +0000" }, "message": "cifs: Use kstrndup for cifs_sb-\u003emountdata\n\nA relatively minor nit, but also clarified the \"consensus\" from the\npreceding comments that it is in fact better to try for the kstrdup\nearly and cleanup while cleaning up is still a simple thing to do.\n\nReviewed-By: Steve French \u003csmfrench@gmail.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "046462abca2576cc1c71f501c148798fac4ea314", "tree": "1bd1148f9b6534b43767dd42a4b4ea27205c4cc9", "parents": [ "b946845a9dc523c759cae2b6a0f6827486c3221a" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:33 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:55 2011 +0000" }, "message": "cifs: Simplify handling of submount options in cifs_mount.\n\nWith CONFIG_DFS_UPCALL enabled, maintain the submount options in\ncifs_sb-\u003emountdata, simplifying the code just a bit as well as making\ncorner-case allocation problems less likely.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b946845a9dc523c759cae2b6a0f6827486c3221a", "tree": "15345fcb639fa2da178ade7fc704117ac0fb0ce4", "parents": [ "c1508ca23653245266e2e3ab69a8dad464f7a569" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:32 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:54 2011 +0000" }, "message": "cifs: cifs_parse_mount_options: do not tokenize mount options in-place\n\nTo keep strings passed to cifs_parse_mount_options re-usable (which is\nneeded to clean up the DFS referral handling), tokenize a copy of the\nmount options instead. If values are needed from this tokenized string,\nthey too must be duplicated (previously, some options were copied and\nothers duplicated).\n\nSince we are not on the critical path and any cleanup is relatively easy,\nthe extra memory usage shouldn\u0027t be a problem (and it is a bit simpler\nthan trying to implement something smarter).\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c1508ca23653245266e2e3ab69a8dad464f7a569", "tree": "31105f22f36808917ee218f196fb65f8676b3e4e", "parents": [ "dd61394586dbd9387fe53b325c6807f61734cf89" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:31 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:54 2011 +0000" }, "message": "cifs: Add support for mounting Windows 2008 DFS shares\n\nWindows 2008 CIFS servers do not always return PATH_NOT_COVERED when\nattempting to access a DFS share. Therefore, when checking for remote\nshares, unconditionally ask for a DFS referral for the UNC (w/out prepath)\nbefore continuing with previous behavior of attempting to access the UNC +\nprepath and checking for PATH_NOT_COVERED.\n\nBugzilla: https://bugzilla.kernel.org/show_bug.cgi?id\u003d31092\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "dd61394586dbd9387fe53b325c6807f61734cf89", "tree": "468fe19f775489e09cdd4dbb28c1dc5428ae7ab6", "parents": [ "460458ce8ec195a1902f0c742b76880fbd01dd96" ], "author": { "name": "Sean Finney", "email": "seanius@seanius.net", "time": "Mon Apr 11 13:19:30 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:54 2011 +0000" }, "message": "cifs: Extract DFS referral expansion logic to separate function\n\nThe logic behind the expansion of DFS referrals is now extracted from\ncifs_mount into a new static function, expand_dfs_referral. This will\nreduce duplicate code in upcoming commits.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Sean Finney \u003cseanius@seanius.net\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "460458ce8ec195a1902f0c742b76880fbd01dd96", "tree": "00ebdd1ce2f23cf0f0d5359befc3a7715d3dc8a8", "parents": [ "820a803ffac3ef591e597bc107f8e289a823a29c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Mar 31 21:18:15 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:53 2011 +0000" }, "message": "cifs: turn BCC into a static inlined function\n\nIt\u0027s a bad idea to have macro functions that reference variables more\nthan once, as the arguments could have side effects. Turn BCC() into\na static inlined function instead.\n\nWhile we\u0027re at it, make it return a void * to discourage anyone from\ndereferencing it as-is.\n\nReported-and-acked-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "820a803ffac3ef591e597bc107f8e289a823a29c", "tree": "246451259a7efc5027647de639a69cb121b889e0", "parents": [ "0e6e37a7a81f370d9aafafdf88aca13977f6fb5f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 04 08:05:26 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:53 2011 +0000" }, "message": "cifs: keep BCC in little-endian format\n\nThis is the same patch as originally posted, just with some merge\nconflicts fixed up...\n\nCurrently, the ByteCount is usually converted to host-endian on receive.\nThis is confusing however, as we need to keep two sets of routines for\naccessing it, and keep track of when to use each routine. Munging\nreceived packets like this also limits when the signature can be\ncalulated.\n\nSimplify the code by keeping the received ByteCount in little-endian\nformat. This allows us to eliminate a set of routines for accessing it\nand we can now drop the *_le suffixes from the accessor functions since\nthat\u0027s now implied.\n\nWhile we\u0027re at it, switch all of the places that read the ByteCount\ndirectly to use the get_bcc inline which should also clean up some\nunaligned accesses.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0e6e37a7a81f370d9aafafdf88aca13977f6fb5f", "tree": "5b622b01e5b6d1dfaa3fe70cb758aaff6f82692c", "parents": [ "6feb9891da4f8b04ffca69c00eb56bb7c1b64dc4" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 04 08:08:19 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:52 2011 +0000" }, "message": "cifs: fix some unused variable warnings in id_rb_search\n\nfs/cifs/cifsacl.c: In function ‘id_rb_search’:\nfs/cifs/cifsacl.c:215:19: warning: variable ‘linkto’ set but not used\n[-Wunused-but-set-variable]\nfs/cifs/cifsacl.c:214:18: warning: variable ‘parent’ set but not used\n[-Wunused-but-set-variable]\n\nReviewed-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6feb9891da4f8b04ffca69c00eb56bb7c1b64dc4", "tree": "647e083e7e9c78b7f1b63482c7c6aaa5185963ff", "parents": [ "0b81c1c405c063f3ecea66c2f5e9c3aefc5359c8" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu Apr 07 18:18:11 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:52 2011 +0000" }, "message": "CIFS: Simplify invalidate part (try #5)\n\nSimplify many places when we call cifs_revalidate/invalidate to make\nit do what it exactly needs.\n\nReviewed-by: Jeff Layton \u003cjlayton@samba.org\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0b81c1c405c063f3ecea66c2f5e9c3aefc5359c8", "tree": "b284b86c0e17c24affff9f6d1e1f2ab2d217021f", "parents": [ "be8e3b0044a68e1f1002c432f6b40d290cf0701d" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Thu Mar 10 10:11:05 2011 +0300" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:51 2011 +0000" }, "message": "CIFS: directio read/write cleanups\n\nRecently introduced strictcache mode brought a new code that can be\nefficiently used by directio part. That\u0027s let us add vectored operations\nand break unnecessary cifs_user_read and cifs_user_write.\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "be8e3b0044a68e1f1002c432f6b40d290cf0701d", "tree": "41f6a5e2ccf4bf03eb722030563490bbe46f0644", "parents": [ "9409ae58e0759d010b347e7b19ebc90ab5d4b98f" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Apr 29 05:40:20 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:51 2011 +0000" }, "message": "consistently use smb_buf_length as be32 for cifs (try 3)\n\n There is one big endian field in the cifs protocol, the RFC1001\n length, which cifs code (unlike in the smb2 code) had been handling as\n u32 until the last possible moment, when it was converted to be32 (its\n native form) before sending on the wire. To remove the last sparse\n endian warning, and to make this consistent with the smb2\n implementation (which always treats the fields in their\n native size and endianness), convert all uses of smb_buf_length to\n be32.\n\n This version incorporates Christoph\u0027s comment about\n using be32_add_cpu, and fixes a typo in the second\n version of the patch.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9409ae58e0759d010b347e7b19ebc90ab5d4b98f", "tree": "7b64bf5100528daaaf9f29819c78c8fe46b51af4", "parents": [ "4d79dba0e00749fa40de8ef13a9b85ce57a1603b" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Fri Apr 22 12:09:36 2011 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:51 2011 +0000" }, "message": "cifs: Invoke id mapping functions (try #17 repost)\n\nrb tree search and insertion routines.\n\nA SID which needs to be mapped, is looked up in one of the rb trees\ndepending on whether SID is either owner or group SID.\nIf found in the tree, a (mapped) id from that node is assigned to\nuid or gid as appropriate. If unmapped, an upcall is attempted to\nmap the SID to an id. If upcall is successful, node is marked as\nmapped. If upcall fails, node stays marked as unmapped and a mapping\nis attempted again only after an arbitrary time period has passed.\n\nTo map a SID, which can be either a Owner SID or a Group SID, key\ndescription starts with the string \"os\" or \"gs\" followed by SID converted\nto a string. Without \"os\" or \"gs\", cifs.upcall does not know whether\nSID needs to be mapped to either an uid or a gid.\n\nNodes in rb tree have fields to prevent multiple upcalls for\na SID. Searching, adding, and removing nodes is done within global locks.\nWhenever a node is either found or inserted in a tree, a reference\nis taken on that node.\nShrinker routine prunes a node if it has expired but does not prune\nan expired node if its refcount is not zero (i.e. sid/id of that node\nis_being/will_be accessed).\nThus a node, if its SID needs to be mapped by making an upcall,\ncan safely stay and its fields accessed without shrinker pruning it.\nA reference (refcount) is put on the node without holding the spinlock\nbut a reference is get on the node by holding the spinlock.\n\nEvery time an existing mapped node is accessed or mapping is attempted,\nits timestamp is updated to prevent it from getting erased or a\nto prevent multiple unnecessary repeat mapping retries respectively.\n\nFor now, cifs.upcall is only used to map a SID to an id (uid or gid) but\nit would be used to obtain an SID for an id.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4d79dba0e00749fa40de8ef13a9b85ce57a1603b", "tree": "55e377f53c761e70d5b90a1f901f70609ffcbcc7", "parents": [ "9ad1506b42c828dff0b9d8f3914e1f837734e91c" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Wed Apr 27 23:34:35 2011 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:51 2011 +0000" }, "message": "cifs: Add idmap key and related data structures and functions (try #17 repost)\n\nDefine (global) data structures to store ids, uids and gids, to which a\nSID maps. There are two separate trees, one for SID/uid and another one\nfor SID/gid.\n\nA new type of key, cifs_idmap_key_type, is used.\n\nKeys are instantiated and searched using credential of the root by\noverriding and restoring the credentials of the caller requesting the key.\n\nId mapping functions are invoked under config option of cifs acl.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9ad1506b42c828dff0b9d8f3914e1f837734e91c", "tree": "6108ad6564b013a978a717f52b06d5fff4d3ff0e", "parents": [ "1cb06d0b50536af177b2f2f7cab25546f3731d3e" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Fri Apr 08 05:29:10 2011 +0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:50 2011 +0000" }, "message": "CIFS: Add launder_page operation (try #3)\n\nAdd this let us drop filemap_write_and_wait from cifs_invalidate_mapping\nand simplify the code to properly process invalidate logic.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1cb06d0b50536af177b2f2f7cab25546f3731d3e", "tree": "c0fc587667f9b97d15ff13abbb0efeeba4faf0e7", "parents": [ "257fb1f15d72f89dad2d72fa467c189f2d7fdd71" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Feb 24 18:07:19 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:50 2011 +0000" }, "message": "Introduce smb2 mounts as vers\u003d2\n\nAs with Linux nfs client, which uses \"nfsvers\u003d\" or \"vers\u003d\" to\nindicate which protocol to use for mount, specifying\n\n\"vers\u003dsmb2\" or \"vers\u003d2\"\n\nwill force an smb2 mount. When vers is not specified cifs is used\n\nie \"vers\u003dcifs\" or \"vers\u003d1\"\n\nWe can eventually autonegotiate down from smb2 to cifs\nwhen smb2 is stable enough to make it the default, but this\nis for the future. At that time we could also implement a\n\"maxprotocol\" mount option as smbclient and Samba have today,\nbut that would be premature until smb2 is stable.\n\nIntially the smb2 Kconfig option will depend on \"BROKEN\"\nuntil the merge is complete, and then be \"EXPERIMENTAL\"\nWhen it is no longer experimental we can consider changing\nthe default protocol to attempt first.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "257fb1f15d72f89dad2d72fa467c189f2d7fdd71", "tree": "75069bb2801c4e9642d00b702c0c2e742826de33", "parents": [ "fd5707e1b44560d18b47d552c1871ea0bfc46688" ], "author": { "name": "Pavel Shilovsky", "email": "piastry@etersoft.ru", "time": "Wed Mar 16 01:55:32 2011 +0300" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:50 2011 +0000" }, "message": "CIFS: Use invalidate_inode_pages2 instead of invalidate_remote_inode (try #4)\n\nUse invalidate_inode_pages2 that don\u0027t leave pages even if shrink_page_list()\nhas a temp ref on them. It prevents a data coherency problem when\ncifs_invalidate_mapping didn\u0027t invalidate pages but the client thinks that a data\nfrom the cache is uptodate according to an oplock level (exclusive or II).\n\nSigned-off-by: Pavel Shilovsky \u003cpiastry@etersoft.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "fd5707e1b44560d18b47d552c1871ea0bfc46688", "tree": "49881bbf211161f7523f5fed61c6f3db720917a3", "parents": [ "4358b5678b27ffe81391d84ce150df8e81010f6a" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Mar 31 17:22:07 2011 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:50 2011 +0000" }, "message": "cifs: fix comment in validate_t2\n\nThe comment about checking the bcc is in the wrong place. Also make it\nmatch kernel coding style.\n\nReported-and-acked-by: David Howells \u003cdhowells@redhat.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b73b9a4ba753dfd7d304ee6ee4685b827524c533", "tree": "19a8362402e8d01d2571022b3a3fc4d4ba1fafb8", "parents": [ "43988d76851077d2945080665e3c4e2e636d700a" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Apr 19 18:27:10 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:49 2011 +0000" }, "message": "[CIFS] Allow to set extended attribute cifs_acl (try #2)\n\nAllow setting cifs_acl on the server.\nPass on to the server the ACL blob generated by an application.\ncifs is just a pass-through, it does not monitor or inspect the contents\nof the blob, server decides whether to enforce/apply the ACL blob composed\nby an application.\nIf setting of ACL is succeessful, mark the inode for revalidation.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "43988d76851077d2945080665e3c4e2e636d700a", "tree": "8fe4d82c49687c47f496803197f2a8736a8e2dfc", "parents": [ "257208736acc694def83627fa0de2892490a5d42" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Apr 19 18:23:31 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:49 2011 +0000" }, "message": "[CIFS] Use ecb des kernel crypto APIs instead of\n local cifs functions (repost)\n\nUsing kernel crypto APIs for DES encryption during LM and NT hash generation\ninstead of local functions within cifs.\nSource file smbdes.c is deleted sans four functions, one of which\nuses ecb des functionality provided by kernel crypto APIs.\n\nRemove function SMBOWFencrypt.\n\nAdd return codes to various functions such as calc_lanman_hash,\nSMBencrypt, and SMBNTencrypt. Includes fix noticed by Dan Carpenter.\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nCC: Dan Carpenter \u003cerror27@gmail.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "257208736acc694def83627fa0de2892490a5d42", "tree": "fefef8ec91edec3d38082977d079663e21032706", "parents": [ "b34cb85cc2d84c487afe2baa2d3c04d8b677bbd0" ], "author": { "name": "Shirish Pargaonkar", "email": "shirishpargaonkar@gmail.com", "time": "Fri Feb 25 10:48:55 2011 -0600" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:49 2011 +0000" }, "message": "cifs: cleanup: Rename and remove config flags\n\nRemove config flag CIFS_EXPERIMENTAL.\nDo export operations under new config flag CIFS_NFSD_EXPORT\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishpargaonkar@gmail.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b34cb85cc2d84c487afe2baa2d3c04d8b677bbd0", "tree": "67d117ee0bb631c443e32fa1402508958adf8f6d", "parents": [ "34c87901e113799a45423fdac29c7478c889a95d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Feb 24 17:58:00 2011 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 19 14:10:48 2011 +0000" }, "message": "Introduce SMB2 Kconfig option\n\nSMB2 is the followon to the CIFS (and SMB) protocols\nand the default for Windows since Windows Vista, and also\nnow implemented by various non-Windows servers. SMB2\nis more secure, has various performance advantages, including\nlarger i/o sizes, flow control, better caching model and more.\nSMB2 also resolves some scalability limits in the cifs\nprotocol and adds many new features while being much\nsimpler (only a few dozen commands instead of hundreds)\nand since the protocol is clearer it is\nalso more consistently implemented across servers\nand thus easier to optimize.\n\nAfter much discussion with Jeff Layton, Jeremy Allison\nand others at Connectathon, we decided to move the smb2\ncode from a distinct .ko and fstype into distinct\nC files that optionally build in cifs.ko. As a result\nthe Kconfig gets simpler.\n\nTo avoid destabilizing cifs, the smb2 code is going\nto be moved into its own experimental CONFIG_CIFS_SMB2 ifdef\nas it is merged and rereviewed. The changes to stable\ncifs (builds with the smb2 ifdef off) are expected to be\nfairly small.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" } ], "next": "34c87901e113799a45423fdac29c7478c889a95d" }