diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2019-05-20 18:14:07 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-05-20 18:14:07 +0200 |
| commit | 33a268bd3bf77735e47b6a32f8a1a037af9274c4 (patch) | |
| tree | 2c35e55fbd660dd9780591249663631fed01065a /docs/syzbot.md | |
| parent | b5641f32bf42f02cf749ac7ec0c8fc5d7817d802 (diff) | |
docs: clarify syzbot handling of rebased trees
Diffstat (limited to 'docs/syzbot.md')
| -rw-r--r-- | docs/syzbot.md | 34 |
1 files changed, 31 insertions, 3 deletions
diff --git a/docs/syzbot.md b/docs/syzbot.md index 8c8e63c57..f24e28b1a 100644 --- a/docs/syzbot.md +++ b/docs/syzbot.md @@ -25,9 +25,8 @@ crashes create a new bug). ## Communication with syzbot If you fix a bug reported by `syzbot`, please add the provided `Reported-by` -tag to the commit (`Reported-and-tested-by` and `Tested-by` tags with the -`syzbot+HASH` address are recognized as well). You can also communicate with -`syzbot` by replying to its emails. The commands are: +tag to the commit. You can also communicate with `syzbot` by replying +to its emails. The commands are: - to attach a fixing commit to the bug (if you forgot to add `Reported-by` tag): ``` @@ -90,6 +89,35 @@ Note: see [below](#kmsan-bugs) for `KMSAN` bugs testing. Note: see [below](#usb-bugs) for `USB` bugs testing. +<div id="amend"/> +<div id="linux-next"/> + +## Rebuilt trees/amended patches + +There are several additional aspects if the tree is rebuilt/rebased or contains +amended/folded patches (namely, `linux-next`). + +First, adding `Reported-by` tags to amended commits may be misleading. +A `Reported-by` tag suggests that the commit fixes a bug in some previous +commit, but here it's not the case (it only fixes a bug in a previous version +of itself which is not in the tree). In such case it's recommended to include +a `Tested-by` or a `Reviewed-by` tag with the hash instead. Technically, +`syzbot` accepts any tag, so `With-inputs-from` would work too. + +Then, if the guilty commit is completely dropped (actually removed from the +tree during rebuild), then there is effectively no fixing commit. There is no +good way to handle such cases at the moment. One possibility is to mark them +with `#syz invalid`. However this needs to be done only when `syzbot` picks up +the new tree in all builds (current kernel commits can be seen on dashboard). +Otherwise, the occurrences of the crash that are still happening in the current +build will immediately create a new bug report. Another possibility is to mark +it as fixed with some unrelated later commit using +`#syz fix: some-later-commit`. This way `syzbot` will wait until the commit +propagates to all tested trees automatically. + +In any case the relation between the report and the fix can later be fixed up +using `#syz fix: commit-title` commands. + <div id="bisection"/> ## Bisection |
