[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <87bjlxokvx.fsf@yellow.woof>
Date: Thu, 23 Oct 2025 14:36:18 +0200
From: Nam Cao <namcao@...utronix.de>
To: Gabriele Monaco <gmonaco@...hat.com>, linux-kernel@...r.kernel.org, Steven
Rostedt <rostedt@...dmis.org>, linux-trace-kernel@...r.kernel.org
Cc: Tomas Glozar <tglozar@...hat.com>, Juri Lelli <jlelli@...hat.com>, Clark
Williams <williams@...hat.com>, John Kacur <jkacur@...hat.com>
Subject: Re: [PATCH v2 18/20] rv: Add support for per-object monitors in DA/HA
Gabriele Monaco <gmonaco@...hat.com> writes:
> Ignoring the vagueness of the name, the two implementations of this hook are to
> call an allocation (yes, always conditionally) or just assign the target to a
> pre-allocated storage.
> Your suggestion of da_monitor_prepare_storage might fit both descriptions.
Now that I have grasped (or at least I think so) the patches, it is not
that important to me anymore. But still, having function names which
precisely describe what they do would help new people understand the code.
Nam
Powered by blists - more mailing lists