Nicolas Palix | 9e39555 | 2013-03-02 22:36:26 +0100 | [diff] [blame] | 1 | #!/bin/bash |
Luis R. Rodriguez | c100d53 | 2016-06-29 15:14:58 -0700 | [diff] [blame] | 2 | # Linux kernel coccicheck |
| 3 | # |
Markus Elfring | 1e01892 | 2016-10-07 16:06:15 +0200 | [diff] [blame] | 4 | # Read Documentation/dev-tools/coccinelle.rst |
Nicolas Palix | ec97946 | 2013-07-03 16:41:01 +0200 | [diff] [blame] | 5 | # |
| 6 | # This script requires at least spatch |
| 7 | # version 1.0.0-rc11. |
Nicolas Palix | ec97946 | 2013-07-03 16:41:01 +0200 | [diff] [blame] | 8 | |
Luis R. Rodriguez | a9e064c | 2016-06-29 15:14:57 -0700 | [diff] [blame] | 9 | DIR="$(dirname $(readlink -f $0))/.." |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 10 | SPATCH="`which ${SPATCH:=spatch}`" |
| 11 | |
Luis R. Rodriguez | 13d9486 | 2016-06-29 15:14:51 -0700 | [diff] [blame] | 12 | if [ ! -x "$SPATCH" ]; then |
| 13 | echo 'spatch is part of the Coccinelle project and is available at http://coccinelle.lip6.fr/' |
| 14 | exit 1 |
| 15 | fi |
| 16 | |
Luis R. Rodriguez | a9e064c | 2016-06-29 15:14:57 -0700 | [diff] [blame] | 17 | SPATCH_VERSION=$($SPATCH --version | head -1 | awk '{print $3}') |
| 18 | SPATCH_VERSION_NUM=$(echo $SPATCH_VERSION | ${DIR}/scripts/ld-version.sh) |
| 19 | |
Luis R. Rodriguez | c930a1b | 2016-06-29 15:14:53 -0700 | [diff] [blame] | 20 | USE_JOBS="no" |
| 21 | $SPATCH --help | grep "\-\-jobs" > /dev/null && USE_JOBS="yes" |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 22 | |
Bernd Schubert | 26e5672 | 2013-01-29 17:03:37 +0100 | [diff] [blame] | 23 | # The verbosity may be set by the environmental parameter V= |
| 24 | # as for example with 'make V=1 coccicheck' |
| 25 | |
| 26 | if [ -n "$V" -a "$V" != "0" ]; then |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 27 | VERBOSE="$V" |
Bernd Schubert | 26e5672 | 2013-01-29 17:03:37 +0100 | [diff] [blame] | 28 | else |
| 29 | VERBOSE=0 |
| 30 | fi |
| 31 | |
Luis R. Rodriguez | 8e826ad | 2016-06-29 15:14:52 -0700 | [diff] [blame] | 32 | FLAGS="--very-quiet" |
Nicolas Palix | 9e39555 | 2013-03-02 22:36:26 +0100 | [diff] [blame] | 33 | |
Luis R. Rodriguez | 5c384db | 2016-06-29 15:14:55 -0700 | [diff] [blame] | 34 | # You can use SPFLAGS to append extra arguments to coccicheck or override any |
| 35 | # heuristics done in this file as Coccinelle accepts the last options when |
| 36 | # options conflict. |
| 37 | # |
| 38 | # A good example for use of SPFLAGS is if you want to debug your cocci script, |
| 39 | # you can for instance use the following: |
| 40 | # |
| 41 | # $ export COCCI=scripts/coccinelle/misc/irqf_oneshot.cocci |
| 42 | # $ make coccicheck MODE=report DEBUG_FILE="all.err" SPFLAGS="--profile --show-trying" M=./drivers/mfd/arizona-irq.c |
| 43 | # |
| 44 | # "--show-trying" should show you what rule is being processed as it goes to |
| 45 | # stdout, you do not need a debug file for that. The profile output will be |
| 46 | # be sent to stdout, if you provide a DEBUG_FILE the profiling data can be |
| 47 | # inspected there. |
| 48 | # |
| 49 | # --profile will not output if --very-quiet is used, so avoid it. |
| 50 | echo $SPFLAGS | egrep -e "--profile|--show-trying" 2>&1 > /dev/null |
| 51 | if [ $? -eq 0 ]; then |
| 52 | FLAGS="--quiet" |
| 53 | fi |
| 54 | |
Nicolas Palix | 9e39555 | 2013-03-02 22:36:26 +0100 | [diff] [blame] | 55 | # spatch only allows include directories with the syntax "-I include" |
| 56 | # while gcc also allows "-Iinclude" and "-include include" |
| 57 | COCCIINCLUDE=${LINUXINCLUDE//-I/-I } |
Andrzej Hajda | 5b16910 | 2015-09-22 15:15:30 +0200 | [diff] [blame] | 58 | COCCIINCLUDE=${COCCIINCLUDE// -include/ --include} |
Nicolas Palix | 9e39555 | 2013-03-02 22:36:26 +0100 | [diff] [blame] | 59 | |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 60 | if [ "$C" = "1" -o "$C" = "2" ]; then |
| 61 | ONLINE=1 |
| 62 | |
Nicolas Palix | 9e39555 | 2013-03-02 22:36:26 +0100 | [diff] [blame] | 63 | # Take only the last argument, which is the C file to test |
| 64 | shift $(( $# - 1 )) |
| 65 | OPTIONS="$COCCIINCLUDE $1" |
Masahiro Yamada | b96d06e | 2017-11-14 20:38:07 +0900 | [diff] [blame] | 66 | |
| 67 | # No need to parallelize Coccinelle since this mode takes one input file. |
| 68 | NPROC=1 |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 69 | else |
| 70 | ONLINE=0 |
Greg Dietsche | d0bc1fb | 2011-11-05 20:59:43 -0500 | [diff] [blame] | 71 | if [ "$KBUILD_EXTMOD" = "" ] ; then |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 72 | OPTIONS="--dir $srctree $COCCIINCLUDE" |
Greg Dietsche | d0bc1fb | 2011-11-05 20:59:43 -0500 | [diff] [blame] | 73 | else |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 74 | OPTIONS="--dir $KBUILD_EXTMOD $COCCIINCLUDE" |
Greg Dietsche | d0bc1fb | 2011-11-05 20:59:43 -0500 | [diff] [blame] | 75 | fi |
Masahiro Yamada | b96d06e | 2017-11-14 20:38:07 +0900 | [diff] [blame] | 76 | |
| 77 | if [ -z "$J" ]; then |
| 78 | NPROC=$(getconf _NPROCESSORS_ONLN) |
| 79 | else |
| 80 | NPROC="$J" |
| 81 | fi |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 82 | fi |
| 83 | |
Nicolas Palix | bad6a40 | 2013-03-02 22:36:28 +0100 | [diff] [blame] | 84 | if [ "$KBUILD_EXTMOD" != "" ] ; then |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 85 | OPTIONS="--patch $srctree $OPTIONS" |
Nicolas Palix | bad6a40 | 2013-03-02 22:36:28 +0100 | [diff] [blame] | 86 | fi |
| 87 | |
Luis R. Rodriguez | c930a1b | 2016-06-29 15:14:53 -0700 | [diff] [blame] | 88 | # You can override by using SPFLAGS |
| 89 | if [ "$USE_JOBS" = "no" ]; then |
| 90 | trap kill_running SIGTERM SIGINT |
| 91 | declare -a SPATCH_PID |
| 92 | elif [ "$NPROC" != "1" ]; then |
| 93 | # Using 0 should work as well, refer to _SC_NPROCESSORS_ONLN use on |
| 94 | # https://github.com/rdicosmo/parmap/blob/master/setcore_stubs.c |
| 95 | OPTIONS="$OPTIONS --jobs $NPROC --chunksize 1" |
| 96 | fi |
| 97 | |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 98 | if [ "$MODE" = "" ] ; then |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 99 | if [ "$ONLINE" = "0" ] ; then |
Nicolas Palix | 1f0a674 | 2013-06-06 23:39:52 +0200 | [diff] [blame] | 100 | echo 'You have not explicitly specified the mode to use. Using default "report" mode.' |
| 101 | echo 'Available modes are the following: patch, report, context, org' |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 102 | echo 'You can specify the mode with "make coccicheck MODE=<mode>"' |
Nicolas Palix | 1f0a674 | 2013-06-06 23:39:52 +0200 | [diff] [blame] | 103 | echo 'Note however that some modes are not implemented by some semantic patches.' |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 104 | fi |
Nicolas Palix | 1f0a674 | 2013-06-06 23:39:52 +0200 | [diff] [blame] | 105 | MODE="report" |
| 106 | fi |
| 107 | |
| 108 | if [ "$MODE" = "chain" ] ; then |
| 109 | if [ "$ONLINE" = "0" ] ; then |
| 110 | echo 'You have selected the "chain" mode.' |
| 111 | echo 'All available modes will be tried (in that order): patch, report, context, org' |
| 112 | fi |
Nicolas Palix | 03ee0c4 | 2010-10-08 21:27:41 +0200 | [diff] [blame] | 113 | elif [ "$MODE" = "report" -o "$MODE" = "org" ] ; then |
Deepa Dinamani | 7a2358b | 2016-06-12 12:04:39 -0700 | [diff] [blame] | 114 | FLAGS="--no-show-diff $FLAGS" |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 115 | fi |
| 116 | |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 117 | if [ "$ONLINE" = "0" ] ; then |
| 118 | echo '' |
| 119 | echo 'Please check for false positives in the output before submitting a patch.' |
| 120 | echo 'When using "patch" mode, carefully review the patch before submitting it.' |
| 121 | echo '' |
| 122 | fi |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 123 | |
Luis R. Rodriguez | c930a1b | 2016-06-29 15:14:53 -0700 | [diff] [blame] | 124 | run_cmd_parmap() { |
| 125 | if [ $VERBOSE -ne 0 ] ; then |
| 126 | echo "Running ($NPROC in parallel): $@" |
| 127 | fi |
Luis R. Rodriguez | be1fa90 | 2016-06-29 15:14:54 -0700 | [diff] [blame] | 128 | if [ "$DEBUG_FILE" != "/dev/null" -a "$DEBUG_FILE" != "" ]; then |
| 129 | if [ -f $DEBUG_FILE ]; then |
| 130 | echo "Debug file $DEBUG_FILE exists, bailing" |
| 131 | exit |
| 132 | fi |
| 133 | else |
| 134 | DEBUG_FILE="/dev/null" |
| 135 | fi |
| 136 | $@ 2>$DEBUG_FILE |
Luis R. Rodriguez | c930a1b | 2016-06-29 15:14:53 -0700 | [diff] [blame] | 137 | if [[ $? -ne 0 ]]; then |
| 138 | echo "coccicheck failed" |
| 139 | exit $? |
| 140 | fi |
| 141 | } |
| 142 | |
| 143 | run_cmd_old() { |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 144 | local i |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 145 | if [ $VERBOSE -ne 0 ] ; then |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 146 | echo "Running ($NPROC in parallel): $@" |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 147 | fi |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 148 | for i in $(seq 0 $(( NPROC - 1)) ); do |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 149 | eval "$@ --max $NPROC --index $i &" |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 150 | SPATCH_PID[$i]=$! |
| 151 | if [ $VERBOSE -eq 2 ] ; then |
| 152 | echo "${SPATCH_PID[$i]} running" |
| 153 | fi |
| 154 | done |
| 155 | wait |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 156 | } |
| 157 | |
Luis R. Rodriguez | c930a1b | 2016-06-29 15:14:53 -0700 | [diff] [blame] | 158 | run_cmd() { |
| 159 | if [ "$USE_JOBS" = "yes" ]; then |
| 160 | run_cmd_parmap $@ |
| 161 | else |
| 162 | run_cmd_old $@ |
| 163 | fi |
| 164 | } |
| 165 | |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 166 | kill_running() { |
Kees Cook | 2552a39 | 2016-05-16 05:55:58 -0700 | [diff] [blame] | 167 | for i in $(seq 0 $(( NPROC - 1 )) ); do |
Kees Cook | 90d06a4 | 2013-06-18 14:49:29 -0700 | [diff] [blame] | 168 | if [ $VERBOSE -eq 2 ] ; then |
| 169 | echo "Killing ${SPATCH_PID[$i]}" |
| 170 | fi |
| 171 | kill ${SPATCH_PID[$i]} 2>/dev/null |
| 172 | done |
| 173 | } |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 174 | |
Luis R. Rodriguez | 8e826ad | 2016-06-29 15:14:52 -0700 | [diff] [blame] | 175 | # You can override heuristics with SPFLAGS, these must always go last |
| 176 | OPTIONS="$OPTIONS $SPFLAGS" |
| 177 | |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 178 | coccinelle () { |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 179 | COCCI="$1" |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 180 | |
| 181 | OPT=`grep "Option" $COCCI | cut -d':' -f2` |
Luis R. Rodriguez | a9e064c | 2016-06-29 15:14:57 -0700 | [diff] [blame] | 182 | REQ=`grep "Requires" $COCCI | cut -d':' -f2 | sed "s| ||"` |
| 183 | REQ_NUM=$(echo $REQ | ${DIR}/scripts/ld-version.sh) |
| 184 | if [ "$REQ_NUM" != "0" ] ; then |
| 185 | if [ "$SPATCH_VERSION_NUM" -lt "$REQ_NUM" ] ; then |
| 186 | echo "Skipping coccinele SmPL patch: $COCCI" |
| 187 | echo "You have coccinelle: $SPATCH_VERSION" |
| 188 | echo "This SmPL patch requires: $REQ" |
| 189 | return |
| 190 | fi |
| 191 | fi |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 192 | |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 193 | # The option '--parse-cocci' can be used to syntactically check the SmPL files. |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 194 | # |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 195 | # $SPATCH -D $MODE $FLAGS -parse_cocci $COCCI $OPT > /dev/null |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 196 | |
Nicolas Palix | 35d88a3 | 2013-03-02 22:36:25 +0100 | [diff] [blame] | 197 | if [ $VERBOSE -ne 0 -a $ONLINE -eq 0 ] ; then |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 198 | |
| 199 | FILE=`echo $COCCI | sed "s|$srctree/||"` |
| 200 | |
Nicolas Palix | 3c90841 | 2010-10-08 21:27:38 +0200 | [diff] [blame] | 201 | echo "Processing `basename $COCCI`" |
| 202 | echo "with option(s) \"$OPT\"" |
| 203 | echo '' |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 204 | echo 'Message example to submit a patch:' |
| 205 | |
Nicolas Palix | 3c90841 | 2010-10-08 21:27:38 +0200 | [diff] [blame] | 206 | sed -ne 's|^///||p' $COCCI |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 207 | |
Nicolas Palix | 062c182 | 2010-10-24 23:37:34 +0200 | [diff] [blame] | 208 | if [ "$MODE" = "patch" ] ; then |
| 209 | echo ' The semantic patch that makes this change is available' |
| 210 | elif [ "$MODE" = "report" ] ; then |
| 211 | echo ' The semantic patch that makes this report is available' |
| 212 | elif [ "$MODE" = "context" ] ; then |
| 213 | echo ' The semantic patch that spots this code is available' |
| 214 | elif [ "$MODE" = "org" ] ; then |
| 215 | echo ' The semantic patch that makes this Org report is available' |
| 216 | else |
| 217 | echo ' The semantic patch that makes this output is available' |
| 218 | fi |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 219 | echo " in $FILE." |
| 220 | echo '' |
| 221 | echo ' More information about semantic patching is available at' |
| 222 | echo ' http://coccinelle.lip6.fr/' |
| 223 | echo '' |
| 224 | |
Nicolas Palix | 3c90841 | 2010-10-08 21:27:38 +0200 | [diff] [blame] | 225 | if [ "`sed -ne 's|^//#||p' $COCCI`" ] ; then |
| 226 | echo 'Semantic patch information:' |
| 227 | sed -ne 's|^//#||p' $COCCI |
| 228 | echo '' |
| 229 | fi |
Nicolas Palix | 2c1160c8 | 2010-10-08 21:27:40 +0200 | [diff] [blame] | 230 | fi |
Nicolas Palix | 3c90841 | 2010-10-08 21:27:38 +0200 | [diff] [blame] | 231 | |
Nicolas Palix | 2c1160c8 | 2010-10-08 21:27:40 +0200 | [diff] [blame] | 232 | if [ "$MODE" = "chain" ] ; then |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 233 | run_cmd $SPATCH -D patch \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 234 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS || \ |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 235 | run_cmd $SPATCH -D report \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 236 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS --no-show-diff || \ |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 237 | run_cmd $SPATCH -D context \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 238 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS || \ |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 239 | run_cmd $SPATCH -D org \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 240 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS --no-show-diff || exit 1 |
Nicolas Palix | c05cd6d | 2012-09-20 22:30:46 +0200 | [diff] [blame] | 241 | elif [ "$MODE" = "rep+ctxt" ] ; then |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 242 | run_cmd $SPATCH -D report \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 243 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS --no-show-diff && \ |
Bernd Schubert | 5303265 | 2013-01-29 17:03:42 +0100 | [diff] [blame] | 244 | run_cmd $SPATCH -D context \ |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 245 | $FLAGS --cocci-file $COCCI $OPT $OPTIONS || exit 1 |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 246 | else |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 247 | run_cmd $SPATCH -D $MODE $FLAGS --cocci-file $COCCI $OPT $OPTIONS || exit 1 |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 248 | fi |
| 249 | |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 250 | } |
| 251 | |
| 252 | if [ "$COCCI" = "" ] ; then |
| 253 | for f in `find $srctree/scripts/coccinelle/ -name '*.cocci' -type f | sort`; do |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 254 | coccinelle $f |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 255 | done |
| 256 | else |
Nicolas Palix | 1e9dea2 | 2010-06-13 09:26:34 +0200 | [diff] [blame] | 257 | coccinelle $COCCI |
Nicolas Palix | 74425ee | 2010-06-06 17:15:01 +0200 | [diff] [blame] | 258 | fi |