On 8 October 2014 20:00, Christopher Li <sparse@xxxxxxxxxxx> wrote: > On Mon, Oct 6, 2014 at 10:44 PM, Ard Biesheuvel > <ard.biesheuvel@xxxxxxxxxx> wrote: >>> we only address the case where the function pointer is the target of >>> an assignment. > > I apply the patch in my private branch but haven't push out yet. > > I just want to confirm, you didn't find this kind of assignment in the > kernel, right? > I noticed this patch https://git.kernel.org/cgit/linux/kernel/git/kvmarm/kvmarm.git/commit/?h=queue&id=de56fb1923ca11f428bf557870e0faa99f38762e and saw that it was bogus: changing the return type of the pointed-to function to 'const int' makes no sense at all, and yet it shuts up sparse. With my sparse patch, that patch could have been dropped (although I think it was merged after all) -- Ard. -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
