Hello, On Tue, Aug 29, 2017 at 03:35:34PM +0900, sioh Lee wrote: > Hello, > Thank you for the reply and for being supportive. > First of all, I made a mistake in that I typed crc32 incorrectly. All the experiments were done using crc32c-intel, not crc32 (PCLMULQDQ). So the fuzzy search in __crypto_alg_lookup gave you crc32c-intel because you didn't enable crc32 PCLMULQDQ in the kernel config? >From source it looks like an explicit load of crc32c-intel would work too, instead of checking the priority. We can load in order crc32c-intel, crc32-pclmul and fallback in "crc32c" which must be then forced enabled in the kernel config. > Second, the reason for (priority < 200) is because the priority of crc32c-intel is 200 so that if the priority is less than 200, jhash2 is used. > Also, I have a question about implementation. Do you want to exclude jhash2 from ksm and go only with crc32 ? Could you please give me guidance about it? Yes, the idea about excluding jhash2 from KSM is that if one almost certain crc32c hash collision once every 200k modifications to the page truly isn't a concern with sse4.2, it's still not a concern if crc32 is implemented in C, but still faster than jhash2. I don't like the behavior of the hash to change depending on hw or kernel config, as that decreases the testing and it would generate different behavior depending on kernel config or arch. I don't like non reproducible bugs or to fragment testing across the userbase depending on kernel config or arch. If crc32 creates problems with hash collisions, it's better everyone is testing it so we find out sooner than later. The arch code can choose which of the crc32* variants to use, but it shouldn't change the hash to something completely different. It's more robust if the default hash algorithm is the same (only with minor variations allowed like crc32c vs crc32 vs crc32be). > Then, I will implement it and send you a new patch. > Once again, thank you so much for your reply. Thanks! Andrea -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
