Re: [PATCH v2] kcov: update documentation on remote coverage collection
From: Jann Horn
Date: Tue May 26 2026 - 10:12:58 EST
On Fri, May 22, 2026 at 1:18 AM Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Wed, 20 May 2026 20:21:29 +0200 Jann Horn <jannh@xxxxxxxxxx> wrote:
> > Adjust the docs on remote coverage collection to reflect the changes made
> > in "kcov: refactor common handle ID into kcov_common_handle_id" and
> > "kcov: allow simultaneous KCOV_ENABLE/KCOV_REMOTE_ENABLE".
>
> This has all become rather confusing, with fixes and new versions of
> fixes under different titles, etc.
glider@ suggested in
<https://lore.kernel.org/all/CAG_fn=U+V9wzvJJsRh3aB1uCm5zfWTFv3UHeBaPU_E_KXbDKFQ@xxxxxxxxxxxxxx/>:
| Could you also update Documentation/dev-tools/kcov.rst (a separate
| commit is also fine - it can cover the kcov_common_handle_id change as
| well)?
So that was my intent - to have these three separate patches:
"kcov: refactor common handle ID into kcov_common_handle_id"
"kcov: allow simultaneous KCOV_ENABLE/KCOV_REMOTE_ENABLE"
"kcov: update documentation on remote coverage collection"
(where the documentation patch covers changes by both the other changes)
All the patches I sent were meant as full patches, not incremental fixups.
> I did my best to piece it all together then I joined everything into a
> single patch, below. Please check it for accuracy, up-to-dateness and
> changelog truthfulness, thanks.
Yes, that is an accurate combination of "[PATCH v2] kcov: allow
simultaneous KCOV_ENABLE/KCOV_REMOTE_ENABLE" and "[PATCH v2] kcov:
update documentation on remote coverage collection", and it is
up-to-date, and the changelog looks accurate.