blob: 1a3bdc27d95e98ce49c07791b02515e38ef678a5 [file] [log] [blame]
Linus Torvalds1da177e2005-04-16 15:20:36 -07001Copyright 2004 Linus Torvalds
2Copyright 2004 Pavel Machek <pavel@suse.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
Bob Copelande8331952006-06-23 02:06:09 -070045Getting sparse
46~~~~~~~~~~~~~~
Linus Torvalds1da177e2005-04-16 15:20:36 -070047
Dave Jonesa55028f2007-03-08 19:45:26 -050048You can get latest released versions from the Sparse homepage at
49http://www.kernel.org/pub/linux/kernel/people/josh/sparse/
Linus Torvalds1da177e2005-04-16 15:20:36 -070050
Dave Jonesa55028f2007-03-08 19:45:26 -050051Alternatively, you can get snapshots of the latest development version
52of sparse using git to clone..
Linus Torvalds1da177e2005-04-16 15:20:36 -070053
Dave Jonesa55028f2007-03-08 19:45:26 -050054 git://git.kernel.org/pub/scm/linux/kernel/git/josh/sparse.git
55
56DaveJ has hourly generated tarballs of the git tree available at..
Linus Torvalds1da177e2005-04-16 15:20:36 -070057
Bob Copelande8331952006-06-23 02:06:09 -070058 http://www.codemonkey.org.uk/projects/git-snapshots/sparse/
Linus Torvalds1da177e2005-04-16 15:20:36 -070059
60
61Once you have it, just do
62
63 make
64 make install
65
Bob Copelande8331952006-06-23 02:06:09 -070066as a regular user, and it will install sparse in your ~/bin directory.
67
68Using sparse
69~~~~~~~~~~~~
70
71Do a kernel make with "make C=1" to run sparse on all the C files that get
72recompiled, or use "make C=2" to run sparse on the files whether they need to
73be recompiled or not. The latter is a fast way to check the whole tree if you
74have already built it.
75
Robert P. J. Day1c7bafe2006-09-13 07:57:50 -040076The optional make variable CHECKFLAGS can be used to pass arguments to sparse.
77The build system passes -Wbitwise to sparse automatically. To perform
78endianness checks, you may define __CHECK_ENDIAN__:
Bob Copelande8331952006-06-23 02:06:09 -070079
Robert P. J. Day1c7bafe2006-09-13 07:57:50 -040080 make C=2 CHECKFLAGS="-D__CHECK_ENDIAN__"
Bob Copelande8331952006-06-23 02:06:09 -070081
82These checks are disabled by default as they generate a host of warnings.