On Thu, 9 Sep 2010 13:16:55 -0500 shirishpargaonkar@xxxxxxxxx wrote: > Three major functionality changes > Maybe this should be 3 separate patchsets then? It would be best if these major functionality changes were bisectable. Also, these patches mix a lot of trivial cleanup code in with the deltas that actually make the major changes. Those should be separated. IOW, more patches that are smaller and targeted would be ideal. > > Make NTLMv2 as a default authentication mechanism within NTLMSSP > > Enable cifs/smb signing > > Use Crypto sync hash APIs instead of cifs crypto functions to genereate > various hashes such as arc4/rc4, md5, and hmac-md5 used during authentication > and smb/cifs signature generation > It seems like this set should be a net reduction in code or at least something closer to 0. This set should not be merged without an accompanying patch to remove the unused crypto code after the conversion to the kernel crypto APIs. -- Jeff Layton <jlayton@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
