Messages in this thread |  | | Date | Fri, 06 Feb 2015 09:15:51 -0800 | From | Stephen Boyd <> | Subject | Re: [Cocci] [PATCH v13 3/6] clk: Make clk API return per-user struct clk instances |
| |
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
|  |