Skip to content

Commit a9ea80a

Browse files
haxtibalhaxtibal
committed
Add README
Signed-off-by: Tobias Deiminger <tobias.deiminger@linutronix.de>
1 parent 1b93067 commit a9ea80a

1 file changed

Lines changed: 185 additions & 0 deletions

File tree

‎README.md‎

Lines changed: 185 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,185 @@
1+
# Exploring possibilities for integrating StrictDoc with ELISA’s requirements template approach for the Linux kernel
2+
3+
This demonstrates how to realize the tool-agnostic
4+
[ELISA Kernel Requirements Template](https://docs.google.com/document/d/1c7S7YAledHP2EEQ2nh26Ibegij-XPNuUFkrFLtJPlzs/edit?tab=t.0)
5+
proposal by using [StrictDoc](https://strictdoc.readthedocs.io) as requirements processing tool. The repository contains a filtered (for brevity)
6+
copy of the Linux kernel with requirements and tests from Alessandro Carminat and Gabriele Paoloni ([^1], [^2]) applied on top. ELISA's
7+
[`SPDX-*` tagging scheme]((https://docs.google.com/document/d/1c7S7YAledHP2EEQ2nh26Ibegij-XPNuUFkrFLtJPlzs/edit?tab=t.0#heading=h.9dudo2y6dlhf))
8+
was [added](https://github.com/strictdoc-project/linux-strictdoc/commit/9df9595b) along
9+
with a minimal StrictDoc project configuration.
10+
11+
[^1]: https://lore.kernel.org/all/20250910170000.6475-1-gpaoloni@redhat.com/
12+
[^2]: https://lore.kernel.org/linux-trace-kernel/20250814122206.109096-1-gpaoloni@redhat.com/#r
13+
14+
Go to [rendered requirements](https://strictdoc-project.github.io/linux-strictdoc/).
15+
16+
#### Demonstrated features
17+
18+
- Use `strictdoc export .` to generate a nice
19+
[static HTML document tree](https://strictdoc-project.github.io/linux-strictdoc/linux-strictdoc/Documentation/requirements/charmisc.html)
20+
with visual representation of the
21+
[traceability graph](https://strictdoc-project.github.io/linux-strictdoc/linux-strictdoc/Documentation/requirements/charmisc-TRACE.html#DOC-SUBSYS-CHARMISC),
22+
validation results and full-text search. Other output formats as e.g. PDF are available.
23+
- Compile and validate requirements
24+
[in CI](https://github.com/strictdoc-project/linux-strictdoc/blob/strictdoc/.github/workflows/ci.yaml).
25+
- Parses source code
26+
[SPDX-Req-* tags proposed by ELISA](https://docs.google.com/document/d/1c7S7YAledHP2EEQ2nh26Ibegij-XPNuUFkrFLtJPlzs/edit?tab=t.0#heading=h.9dudo2y6dlhf)
27+
and translates them to StrictDocs internal model.
28+
- Sidecar: Proposed by ELISA to hold additional requirement meta data outside source code. Realized as
29+
[separate sdoc file](https://github.com/strictdoc-project/linux-strictdoc/blob/strictdoc/Documentation/requirements/tracing.sdoc)s
30+
containing requirement stubs. Stubs are merged with source code tags by matching on `SPDX-Req-ID`.
31+
- Use `strictdoc manage auto-assign` to generate SPDX-Req-ID and SPDX-Req-HKey as suggested by Linux kernel
32+
requirements template. The hash generation method is `echo -nE "${PROJECT}${FILE_PATH}${INSTANCE}${CODE}" | sha256sum`.
33+
See [commit 86e7810d](https://github.com/strictdoc-project/linux-strictdoc/commit/86e7810d)
34+
for the auto-generated changes.
35+
- Tracing: Requirements, tests and functions become individual nodes in the traceability graph and are connected
36+
by their stable IDs.
37+
- Custom validations: Use plugin API to
38+
[provide a check](https://github.com/strictdoc-project/linux-strictdoc/blob/strictdoc/tools/requirements/validation_plugin.py#L28)
39+
to see if each requirement has at least one associated test, and each function expectations has at least one dedicated
40+
test.
41+
- Drift detection: As kernel development goes on, occasionally rerun `strictdoc manage auto-assign`. If `SPDX-Req-HKey`
42+
changes, this means that some semantic aspect of the requirement may have changed.
43+
44+
For a thorough documentation of StrictDocs features see
45+
[StrictDoc User Guide ](https://strictdoc.readthedocs.io/en/stable/stable/docs/strictdoc_01_user_guide.html)
46+
47+
#### Tutorial: Add a Requirement
48+
49+
Install StrictDoc
50+
```sh
51+
pipx install strictdoc # note: requires strictdoc >= 0.15.1
52+
git clone https://github.com/strictdoc-project/linux-strictdoc
53+
cd linux-strictdoc
54+
```
55+
56+
Add requirement statement and temporary identifier to source code comment
57+
58+
```c
59+
# kernel/trace/trace_events.c
60+
61+
/*
62+
* SPDX-Req-ID: TMP-trace_events_enabled
63+
* SPDX-Req-Text:
64+
* This function shall...
65+
*/
66+
int trace_events_enabled(struct trace_array *tr, const char *system)
67+
```
68+
69+
Add corresponding requirement stub in sidecar file
70+
```
71+
# Documentation/requirements/tracing.sdoc
72+
73+
[REQUIREMENT]
74+
MID: TMP-trace_events_enabled
75+
TITLE: trace_events_enabled
76+
```
77+
78+
Calculate stable identifier and hash value, will be replaced inline
79+
```sh
80+
strictdoc manage auto-uid .
81+
```
82+
83+
Verify new hash values were added and no existing requirement was changed
84+
```sh
85+
git diff
86+
```
87+
88+
```diff
89+
diff --git a/Documentation/requirements/tracing.sdoc b/Documentation/requirements/tracing.sdoc
90+
index 8d1dd2b5..2d86384a 100644
91+
--- a/Documentation/requirements/tracing.sdoc
92+
+++ b/Documentation/requirements/tracing.sdoc
93+
@@ -22,6 +22,11 @@ TITLE: Event Tracing
94+
MID: 1ac497acf75d497f893006853f85fe86
95+
TITLE: Requirements
96+
97+
+[REQUIREMENT]
98+
+MID: b12884ce9b5b3258f1d28026c8aa1526f94030fd9f61ba583560f472015b1abb
99+
+HASH: 5949e5bf7ec43ed2c665d4ffe614dfaa285aafbe73a77df55aef0c099637f65b
100+
+TITLE: trace_events_enabled
101+
+
102+
[REQUIREMENT]
103+
MID: 77958d2a51762caa727e5751d8dfec127c07cb5385f542d7b2fdf26b2a07c8b3
104+
HASH: e8ee84ca42f5626ca9636abb53ded027708fdaabc99c8b935c016dda53130d81
105+
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c
106+
index 16dabd1f..d07db4fa 100644
107+
--- a/kernel/trace/trace_events.c
108+
+++ b/kernel/trace/trace_events.c
109+
@@ -2062,6 +2062,9 @@ event_enable_write(struct file *filp, const char __user *ubuf, size_t cnt,
110+
}
111+
112+
/*
113+
+ * SPDX-Req-ID: b12884ce9b5b3258f1d28026c8aa1526f94030fd9f61ba583560f472015b1abb
114+
+ * SPDX-Req-Text: This function shall...
115+
+ *
116+
* Returns:
117+
* 0 : no events exist?
118+
* 1 : all events are disabled
119+
```
120+
121+
Verify and validate
122+
```sh
123+
strictdoc export .
124+
```
125+
126+
Send for review
127+
```sh
128+
git add -u && git commit -m "docs: Add LLR for trace_events_enabled"
129+
git format-patch -n1
130+
```
131+
132+
#### Explanation of Content and Processing
133+
134+
```
135+
.
136+
├── Documentation
137+
│ └── requirements
138+
│ ├── charmisc.sdoc # sidecar
139+
│ └── tracing.sdoc # sidecar
140+
├── drivers
141+
│ └── char
142+
│ └── mem.c # Linux code with inlined LLRs
143+
├── kernel
144+
│ └── trace
145+
│ └── trace_events.c # Linux code with inlined LLRs
146+
├── strictdoc_config.py # StrictDoc project configuration
147+
└── tools
148+
├── requirements
149+
│ └── validation_plugin.py # custom requirement validations
150+
└── testing
151+
└── selftests
152+
└── devmem
153+
└── tests.c # tests for /dev/mem LLRs
154+
```
155+
156+
StrictDoc performs the following notable process steps:
157+
- parse \*.sdoc files to create the initial traceability index (a DAG structure)
158+
- parse \*.c files using tree-sitter, read SPDX tags from it and merge it into the DAG
159+
- perform built-in validations and calculate built-in statistics
160+
- perform custom validations
161+
* check if all requirements have at least one related test
162+
* check if all function expectations are mentioned by one related test
163+
- render the DAG into a HTML document tree where all nodes are traceable, including
164+
requirements text, visual graph representation and source code view
165+
166+
#### Handling Fields with Special Meaning but Different Name in StrictDoc / ELISA
167+
168+
The StrictDoc model assigns special meaning to some reserved field names:
169+
- `UID` Unique, human-readable. May change during requirement life-cycle. Used to refer to child/parents by default.
170+
- `MID`: Unique, not human-readable, stable. Optionally used to refer to child/parents.
171+
Supports changing the human-readable UID during the requirement life-cycle.
172+
- `HASH`: Hash sum over predefined requirement content. Can be auto-calculated.
173+
- `STATEMENT`: Some document views and import/export formats require to select a "most important"
174+
field from the many fields.
175+
- `COMMENT`: Can occur multiple times within one requirement.
176+
177+
The ELISA requirements template defines similar special meaning for fields, but under different name.
178+
This is solved by two StrictDoc features:
179+
- `ProjectConfig(source_nodes=[SourceNodesEntry(sdoc_to_source_map={<sdoc_name>: <elisa_name>, ...})])` let's you define
180+
a mapping for fields that appearing under a different name in source code tags.
181+
Example: The stable ID appears as `SPDX-Req-ID` in source code comments, but must be named `MID` in sdoc.
182+
- Setting `HUMAN_TITLE` in the grammar lets you define a different display name for a field that has special
183+
StrictDoc meaning. Example: ELISA wants `HASH` to be named `SPDX-Req-HKey`. Since the field appears only in sdoc,
184+
but not in source code, it's enough to define `HUMAN_TITLE: SPDX-Req-HKey` for the `HASH` field.
185+
`sdoc_to_source_map` is not needed in this case.

0 commit comments

Comments
 (0)