<div dir="ltr"><div><br></div><div>>> For Issue 5: Can you provide me (and the CC: the list) the list of<br>>> false positives (the commit hashes) you found for issue 5 on<br>>> EXPORT_SYMBOL?<br>><br>></div><div>> Here are the commit hashes for which the warning is issued:<br>> 54505a1e2083<br>> 75d75b7a4d54<br>> 8084c99b9af6<br>> bfdaf029c9c9<br>> dfd402a4c4ba<br>><br>>> Can you also provide a short rationale/explanation for<br>>> each case that you considered a false positive?<br>><br>><br>> In each case the `EXPORT_SYMBOL()` is correctly written and the variable/function to be exported<br>> is also inside the parentheses, still, we get the warning. Please let me know if I am wrong here.<br><br></div><div>I checked those warnings. Some of the patches are good cases to check if we can improve the heuristics on EXPORT_SYMBOL().</div><div><br></div><div>E.g., commit bfdaf029c9c9 ("ia64: turn csum_partial_copy_from_user() into csum_and_copy_from_user()") looks sound to me, and <a href="http://checkpatch.pl">checkpatch.pl</a> should not really warn about that.</div><div><br></div><div>Mrinal, maybe you can find out why <a href="http://checkpatch.pl">checkpatch.pl</a> believes that this case is wrong and needs to be warned about?</div><div><br></div><div><br></div><div>Lukas</div></div>