Thread (37 messages) 37 messages, 4 authors, 1d ago

Re: [PATCH v3 03/17] verification/rvgen: Improve rv_dir discovery in RVGenerator

From: Thomas Weissschuh <hidden>
Date: 2026-07-02 14:00:35
Also in: lkml

On Thu, Jul 02, 2026 at 03:53:35PM +0200, Nam Cao wrote:
Gabriele Monaco [off-list ref] writes:
quoted
The RVGenerator class can find the RV directory (kernel/trace/rv) in the
kernel tree to do some auto patching. This works by assuming PWD is
either the kernel tree or tools/verification, which isn't always the
case (e.g. when running from selftests).

Make discovery more robust by relying on the absolute path of the
current script and traversing backwards the right number of times.
This should work from any location if rvgen is in the kernel tree.
Agree.
quoted
+        # find the kernel tree root relative to this file's location
+        current_dir = os.path.dirname(os.path.abspath(__file__))
+        kernel_root = os.path.abspath(os.path.join(current_dir, "../../../.."))

The  "../../../.." makes me sad.

We can find the git project root instead. For example:

def getGitRoot():
    return subprocess.Popen(['git', 'rev-parse', '--show-toplevel'], stdout=subprocess.PIPE).communicate()[0].rstrip().decode('utf-8')

(stolen from https://stackoverflow.com/questions/22081209/find-the-root-of-the-git-repository-where-the-file-lives)

But that's not important, up to you.
Relying on git is problematic as the git directory might be missing.
People might use the sources from a tarball or use funky container setups.


Thomas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help