blob: eceab1308a8c2fbde6722232db18bbb57a6e7f2e [file] [log] [blame]
Linus Torvalds1da177e2005-04-16 15:20:36 -07001Copyright 2004 Linus Torvalds
Pavel Macheka2531292010-07-18 14:27:13 +02002Copyright 2004 Pavel Machek <pavel@ucw.cz>
Bob Copelande8331952006-06-23 02:06:09 -07003Copyright 2006 Bob Copeland <me@bobcopeland.com>
Linus Torvalds1da177e2005-04-16 15:20:36 -07004
5Using sparse for typechecking
6~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
7
8"__bitwise" is a type attribute, so you have to do something like this:
9
10 typedef int __bitwise pm_request_t;
11
12 enum pm_request {
13 PM_SUSPEND = (__force pm_request_t) 1,
14 PM_RESUME = (__force pm_request_t) 2
15 };
16
17which makes PM_SUSPEND and PM_RESUME "bitwise" integers (the "__force" is
18there because sparse will complain about casting to/from a bitwise type,
19but in this case we really _do_ want to force the conversion). And because
20the enum values are all the same type, now "enum pm_request" will be that
21type too.
22
23And with gcc, all the __bitwise/__force stuff goes away, and it all ends
24up looking just like integers to gcc.
25
26Quite frankly, you don't need the enum there. The above all really just
27boils down to one special "int __bitwise" type.
28
29So the simpler way is to just do
30
31 typedef int __bitwise pm_request_t;
32
33 #define PM_SUSPEND ((__force pm_request_t) 1)
34 #define PM_RESUME ((__force pm_request_t) 2)
35
36and you now have all the infrastructure needed for strict typechecking.
37
38One small note: the constant integer "0" is special. You can use a
39constant zero as a bitwise integer type without sparse ever complaining.
40This is because "bitwise" (as the name implies) was designed for making
41sure that bitwise types don't get mixed up (little-endian vs big-endian
42vs cpu-endian vs whatever), and there the constant "0" really _is_
43special.
44
Sam Ravnborg20375bf2009-04-10 13:18:08 +020045__bitwise__ - to be used for relatively compact stuff (gfp_t, etc.) that
46is mostly warning-free and is supposed to stay that way. Warnings will
47be generated without __CHECK_ENDIAN__.
48
49__bitwise - noisy stuff; in particular, __le*/__be* are that. We really
50don't want to drown in noise unless we'd explicitly asked for it.
51
Ed Cashin6e976632012-12-17 16:03:25 -080052Using sparse for lock checking
53~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
54
55The following macros are undefined for gcc and defined during a sparse
56run to use the "context" tracking feature of sparse, applied to
57locking. These annotations tell sparse when a lock is held, with
58regard to the annotated function's entry and exit.
59
60__must_hold - The specified lock is held on function entry and exit.
61
62__acquires - The specified lock is held on function exit, but not entry.
63
64__releases - The specified lock is held on function entry, but not exit.
65
66If the function enters and exits without the lock held, acquiring and
67releasing the lock inside the function in a balanced way, no
68annotation is needed. The tree annotations above are for cases where
69sparse would otherwise report a context imbalance.
Sam Ravnborg20375bf2009-04-10 13:18:08 +020070
Bob Copelande8331952006-06-23 02:06:09 -070071Getting sparse
72~~~~~~~~~~~~~~
Linus Torvalds1da177e2005-04-16 15:20:36 -070073
Dave Jonesa55028f2007-03-08 19:45:26 -050074You can get latest released versions from the Sparse homepage at
Bill Pemberton05be7a82010-04-27 16:20:15 -040075https://sparse.wiki.kernel.org/index.php/Main_Page
Linus Torvalds1da177e2005-04-16 15:20:36 -070076
Dave Jonesa55028f2007-03-08 19:45:26 -050077Alternatively, you can get snapshots of the latest development version
78of sparse using git to clone..
Linus Torvalds1da177e2005-04-16 15:20:36 -070079
Bill Pemberton05be7a82010-04-27 16:20:15 -040080 git://git.kernel.org/pub/scm/devel/sparse/sparse.git
Dave Jonesa55028f2007-03-08 19:45:26 -050081
82DaveJ has hourly generated tarballs of the git tree available at..
Linus Torvalds1da177e2005-04-16 15:20:36 -070083
Bob Copelande8331952006-06-23 02:06:09 -070084 http://www.codemonkey.org.uk/projects/git-snapshots/sparse/
Linus Torvalds1da177e2005-04-16 15:20:36 -070085
86
87Once you have it, just do
88
89 make
90 make install
91
Bob Copelande8331952006-06-23 02:06:09 -070092as a regular user, and it will install sparse in your ~/bin directory.
93
94Using sparse
95~~~~~~~~~~~~
96
97Do a kernel make with "make C=1" to run sparse on all the C files that get
98recompiled, or use "make C=2" to run sparse on the files whether they need to
99be recompiled or not. The latter is a fast way to check the whole tree if you
100have already built it.
101
Geert Uytterhoevena887a072008-06-20 15:45:12 +0200102The optional make variable CF can be used to pass arguments to sparse. The
103build system passes -Wbitwise to sparse automatically. To perform endianness
104checks, you may define __CHECK_ENDIAN__:
Bob Copelande8331952006-06-23 02:06:09 -0700105
Geert Uytterhoevena887a072008-06-20 15:45:12 +0200106 make C=2 CF="-D__CHECK_ENDIAN__"
Bob Copelande8331952006-06-23 02:06:09 -0700107
108These checks are disabled by default as they generate a host of warnings.