Documentation: Coccinelle: fix various typos etc.
Fix various typos etc. in dev-tools/coccinelle.rst: - punctuation, grammar, wording Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Acked-by: Julia Lawall <julia.lawall@inria.fr> Link: https://lore.kernel.org/r/def28907-18b9-5b7a-e743-79b0418c946c@infradead.org Signed-off-by: Jonathan Corbet <corbet@lwn.net>
This commit is contained in:
committed by
Jonathan Corbet
parent
7303515ae4
commit
7d087d02cf
@@ -85,7 +85,7 @@ Four basic modes are defined: ``patch``, ``report``, ``context``, and
|
|||||||
file:line:column-column: message
|
file:line:column-column: message
|
||||||
|
|
||||||
- ``context`` highlights lines of interest and their context in a
|
- ``context`` highlights lines of interest and their context in a
|
||||||
diff-like style.Lines of interest are indicated with ``-``.
|
diff-like style. Lines of interest are indicated with ``-``.
|
||||||
|
|
||||||
- ``org`` generates a report in the Org mode format of Emacs.
|
- ``org`` generates a report in the Org mode format of Emacs.
|
||||||
|
|
||||||
@@ -119,7 +119,7 @@ For each semantic patch, a commit message is proposed. It gives a
|
|||||||
description of the problem being checked by the semantic patch, and
|
description of the problem being checked by the semantic patch, and
|
||||||
includes a reference to Coccinelle.
|
includes a reference to Coccinelle.
|
||||||
|
|
||||||
As any static code analyzer, Coccinelle produces false
|
As with any static code analyzer, Coccinelle produces false
|
||||||
positives. Thus, reports must be carefully checked, and patches
|
positives. Thus, reports must be carefully checked, and patches
|
||||||
reviewed.
|
reviewed.
|
||||||
|
|
||||||
@@ -135,18 +135,18 @@ the parallelism, set the J= variable. For example, to run across 4 CPUs::
|
|||||||
|
|
||||||
make coccicheck MODE=report J=4
|
make coccicheck MODE=report J=4
|
||||||
|
|
||||||
As of Coccinelle 1.0.2 Coccinelle uses Ocaml parmap for parallelization,
|
As of Coccinelle 1.0.2 Coccinelle uses Ocaml parmap for parallelization;
|
||||||
if support for this is detected you will benefit from parmap parallelization.
|
if support for this is detected you will benefit from parmap parallelization.
|
||||||
|
|
||||||
When parmap is enabled coccicheck will enable dynamic load balancing by using
|
When parmap is enabled coccicheck will enable dynamic load balancing by using
|
||||||
``--chunksize 1`` argument, this ensures we keep feeding threads with work
|
``--chunksize 1`` argument. This ensures we keep feeding threads with work
|
||||||
one by one, so that we avoid the situation where most work gets done by only
|
one by one, so that we avoid the situation where most work gets done by only
|
||||||
a few threads. With dynamic load balancing, if a thread finishes early we keep
|
a few threads. With dynamic load balancing, if a thread finishes early we keep
|
||||||
feeding it more work.
|
feeding it more work.
|
||||||
|
|
||||||
When parmap is enabled, if an error occurs in Coccinelle, this error
|
When parmap is enabled, if an error occurs in Coccinelle, this error
|
||||||
value is propagated back, the return value of the ``make coccicheck``
|
value is propagated back, and the return value of the ``make coccicheck``
|
||||||
captures this return value.
|
command captures this return value.
|
||||||
|
|
||||||
Using Coccinelle with a single semantic patch
|
Using Coccinelle with a single semantic patch
|
||||||
---------------------------------------------
|
---------------------------------------------
|
||||||
@@ -183,7 +183,7 @@ To check only newly edited code, use the value 2 for the C flag, i.e.::
|
|||||||
|
|
||||||
make C=2 CHECK="scripts/coccicheck"
|
make C=2 CHECK="scripts/coccicheck"
|
||||||
|
|
||||||
In these modes, which works on a file basis, there is no information
|
In these modes, which work on a file basis, there is no information
|
||||||
about semantic patches displayed, and no commit message proposed.
|
about semantic patches displayed, and no commit message proposed.
|
||||||
|
|
||||||
This runs every semantic patch in scripts/coccinelle by default. The
|
This runs every semantic patch in scripts/coccinelle by default. The
|
||||||
@@ -198,12 +198,12 @@ Debugging Coccinelle SmPL patches
|
|||||||
|
|
||||||
Using coccicheck is best as it provides in the spatch command line
|
Using coccicheck is best as it provides in the spatch command line
|
||||||
include options matching the options used when we compile the kernel.
|
include options matching the options used when we compile the kernel.
|
||||||
You can learn what these options are by using V=1, you could then
|
You can learn what these options are by using V=1; you could then
|
||||||
manually run Coccinelle with debug options added.
|
manually run Coccinelle with debug options added.
|
||||||
|
|
||||||
Alternatively you can debug running Coccinelle against SmPL patches
|
Alternatively you can debug running Coccinelle against SmPL patches
|
||||||
by asking for stderr to be redirected to stderr, by default stderr
|
by asking for stderr to be redirected to stderr. By default stderr
|
||||||
is redirected to /dev/null, if you'd like to capture stderr you
|
is redirected to /dev/null; if you'd like to capture stderr you
|
||||||
can specify the ``DEBUG_FILE="file.txt"`` option to coccicheck. For
|
can specify the ``DEBUG_FILE="file.txt"`` option to coccicheck. For
|
||||||
instance::
|
instance::
|
||||||
|
|
||||||
@@ -211,8 +211,8 @@ instance::
|
|||||||
make coccicheck COCCI=scripts/coccinelle/free/kfree.cocci MODE=report DEBUG_FILE=cocci.err
|
make coccicheck COCCI=scripts/coccinelle/free/kfree.cocci MODE=report DEBUG_FILE=cocci.err
|
||||||
cat cocci.err
|
cat cocci.err
|
||||||
|
|
||||||
You can use SPFLAGS to add debugging flags, for instance you may want to
|
You can use SPFLAGS to add debugging flags; for instance you may want to
|
||||||
add both --profile --show-trying to SPFLAGS when debugging. For instance
|
add both --profile --show-trying to SPFLAGS when debugging. For example
|
||||||
you may want to use::
|
you may want to use::
|
||||||
|
|
||||||
rm -f err.log
|
rm -f err.log
|
||||||
@@ -229,7 +229,7 @@ DEBUG_FILE support is only supported when using coccinelle >= 1.0.2.
|
|||||||
--------------------
|
--------------------
|
||||||
|
|
||||||
Coccinelle supports reading .cocciconfig for default Coccinelle options that
|
Coccinelle supports reading .cocciconfig for default Coccinelle options that
|
||||||
should be used every time spatch is spawned, the order of precedence for
|
should be used every time spatch is spawned. The order of precedence for
|
||||||
variables for .cocciconfig is as follows:
|
variables for .cocciconfig is as follows:
|
||||||
|
|
||||||
- Your current user's home directory is processed first
|
- Your current user's home directory is processed first
|
||||||
@@ -237,7 +237,7 @@ variables for .cocciconfig is as follows:
|
|||||||
- The directory provided with the --dir option is processed last, if used
|
- The directory provided with the --dir option is processed last, if used
|
||||||
|
|
||||||
Since coccicheck runs through make, it naturally runs from the kernel
|
Since coccicheck runs through make, it naturally runs from the kernel
|
||||||
proper dir, as such the second rule above would be implied for picking up a
|
proper dir; as such the second rule above would be implied for picking up a
|
||||||
.cocciconfig when using ``make coccicheck``.
|
.cocciconfig when using ``make coccicheck``.
|
||||||
|
|
||||||
``make coccicheck`` also supports using M= targets. If you do not supply
|
``make coccicheck`` also supports using M= targets. If you do not supply
|
||||||
@@ -260,13 +260,13 @@ If not using the kernel's coccicheck target, keep the above precedence
|
|||||||
order logic of .cocciconfig reading. If using the kernel's coccicheck target,
|
order logic of .cocciconfig reading. If using the kernel's coccicheck target,
|
||||||
override any of the kernel's .coccicheck's settings using SPFLAGS.
|
override any of the kernel's .coccicheck's settings using SPFLAGS.
|
||||||
|
|
||||||
We help Coccinelle when used against Linux with a set of sensible defaults
|
We help Coccinelle when used against Linux with a set of sensible default
|
||||||
options for Linux with our own Linux .cocciconfig. This hints to coccinelle
|
options for Linux with our own Linux .cocciconfig. This hints to coccinelle
|
||||||
git can be used for ``git grep`` queries over coccigrep. A timeout of 200
|
that git can be used for ``git grep`` queries over coccigrep. A timeout of 200
|
||||||
seconds should suffice for now.
|
seconds should suffice for now.
|
||||||
|
|
||||||
The options picked up by coccinelle when reading a .cocciconfig do not appear
|
The options picked up by coccinelle when reading a .cocciconfig do not appear
|
||||||
as arguments to spatch processes running on your system, to confirm what
|
as arguments to spatch processes running on your system. To confirm what
|
||||||
options will be used by Coccinelle run::
|
options will be used by Coccinelle run::
|
||||||
|
|
||||||
spatch --print-options-only
|
spatch --print-options-only
|
||||||
@@ -290,7 +290,7 @@ given to it when options are in conflict. ::
|
|||||||
|
|
||||||
Coccinelle supports idutils as well but requires coccinelle >= 1.0.6.
|
Coccinelle supports idutils as well but requires coccinelle >= 1.0.6.
|
||||||
When no ID file is specified coccinelle assumes your ID database file
|
When no ID file is specified coccinelle assumes your ID database file
|
||||||
is in the file .id-utils.index on the top level of the kernel, coccinelle
|
is in the file .id-utils.index on the top level of the kernel. Coccinelle
|
||||||
carries a script scripts/idutils_index.sh which creates the database with::
|
carries a script scripts/idutils_index.sh which creates the database with::
|
||||||
|
|
||||||
mkid -i C --output .id-utils.index
|
mkid -i C --output .id-utils.index
|
||||||
@@ -317,7 +317,7 @@ SmPL patch specific options
|
|||||||
---------------------------
|
---------------------------
|
||||||
|
|
||||||
SmPL patches can have their own requirements for options passed
|
SmPL patches can have their own requirements for options passed
|
||||||
to Coccinelle. SmPL patch specific options can be provided by
|
to Coccinelle. SmPL patch-specific options can be provided by
|
||||||
providing them at the top of the SmPL patch, for instance::
|
providing them at the top of the SmPL patch, for instance::
|
||||||
|
|
||||||
// Options: --no-includes --include-headers
|
// Options: --no-includes --include-headers
|
||||||
@@ -327,7 +327,7 @@ SmPL patch Coccinelle requirements
|
|||||||
|
|
||||||
As Coccinelle features get added some more advanced SmPL patches
|
As Coccinelle features get added some more advanced SmPL patches
|
||||||
may require newer versions of Coccinelle. If an SmPL patch requires
|
may require newer versions of Coccinelle. If an SmPL patch requires
|
||||||
at least a version of Coccinelle, this can be specified as follows,
|
a minimum version of Coccinelle, this can be specified as follows,
|
||||||
as an example if requiring at least Coccinelle >= 1.0.5::
|
as an example if requiring at least Coccinelle >= 1.0.5::
|
||||||
|
|
||||||
// Requires: 1.0.5
|
// Requires: 1.0.5
|
||||||
|
|||||||
Reference in New Issue
Block a user