Hi Alexander, > On Wed, Jun 3, 2026 at 7:43 PM Yunseong Kim <[email protected]> wrote: >> >> Introduces a new KCOV exetened feature that captures function arguments and >> return values at kernel function boundaries, enabling per-process visibility >> into runtime dataflow. > > Some high-level comments: > - Make sure your code can run on every platform supported by kcov (namely > ARM64) > - Check out Sashiko findings: > https://sashiko.dev/#/patchset/20260603-kcov-dataflow-next-20260603-v2-0-fee0939de2c4%40est.tech,
I handled those parts that seemed problematic from sashiko's review. > at least some of them seem to make sense > - Please consolidate changes to the same file into a single patch > - There seem to be two tools (one in C and one in Python) with > overlapping functionality, can you keep only one? I revised this part in v2. > - The test modules seem to be used only in manual testing. Can you > convert them to kselftests or remove them? Thanks again for yout guide, I've updated it to v2. > - At this point, long dashes in the kernel codebase are quite rare, > and I don't see a reason to add more. I checked that the v2 series patchset was removed, using long dashes. Best regards, Yunseong
