On 02/06/15 01:12, Julia Lawall wrote: > > On Fri, 6 Feb 2015, Quentin Lambert wrote: > >> On 06/02/2015 03:15, Stephen Boyd wrote: >>> Thanks for the coccinelle patch. Thinking more about it, I don't think >>> we care if the pointer is dereferenced because that would require a >>> definition of struct clk and that is most likely not the case outside of >>> the clock framework. Did you scan the entire kernel? >> No I haven't. >>> I'm running it now >>> but it seems to be taking a while. >>> >> Yes, that's why, as a first step, I chose to limit the scan to the arm >> directory. > Are you sure to be using all of the options provided: > > // Options: --recursive-includes --relax-include-path > // Options: --include-headers-for-types > > And are you using 1.0.0-rc23 or 1.0.0-rc24? Those should save parsed > header files so that they don't have to be parsed over and over. > > If you are using rc24, then you can use the -j option for parallelism. > But then you should also use an option like --chunksize 10 (I don't know > what number would be good), because the default is chunksize 1, and in > that case the saved parsed header files are not reused, because the fies > are all processed individually. In general, it is only the files within a > chunk that will share parsed header files. Thanks for the info. $ spatch --version spatch version 1.0.0-rc22 with Python support and with PCRE support so I guess I need to update. I tried throwing it into scripts/coccinelle/misc and then did make O=../obj/ coccicheck COCCI=../kernel/scripts/coccinelle/misc/structclk.cocci at the toplevel but it didn't find anything. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
