From: Hamza Mahfooz <[email protected]> Sent: Wednesday, May 27, 2026 6:04 AM > > On Tue, May 26, 2026 at 01:52:39PM -0700, Michael Kelley wrote: > > From: Michael Kelley <[email protected]> > > > > Function and structure names in the Hyper-V DRM driver currently > > use "hyperv_" as the prefix. This conflicts with usage in core Hyper-V > > and VMBus code, and incorrectly implies that functions and structures > > in this driver apply generically to Hyper-V. A specific conflict arises > > for "hyperv_init", which is an initcall for generic Hyper-V > > initialization on arm64. The conflict prevents the use of > > initcall_blacklist on the kernel boot line to skip loading this driver. > > > > Fix this by substituting "hvdrm_" as the prefix for all functions and > > I would personally prefer "hv_drm_", since it seems clearer.
My choice of "hvdrm" mimics the old Hyper-V FBdev driver, which uses "hvfb" as the prefix. However, looking through everything that starts with "hv" in /proc/kallsyms, I also see prefixes with the additional underscore. "hv_kbd_" in the Hyper-V keyboard driver is an example. The Hyper-V utils drivers have both forms -- I see "hv_vss_", "hv_ptp_", and "hv_kvp_", but also "hvt" (for Hyper-V Transport). So the historical practice is inconsistent. I'm OK going either way. Does anyone else want to express a preference? > > > structures in this driver. This prefix marries the existing "hv" prefix > > for Hyper-V related code with "drm" to indicate this driver. > > > > The changes are all mechanical text substitution in symbol names. > > There are no other code or functional changes. > > > > Signed-off-by: Michael Kelley <[email protected]> > > --- > > This patch is built against linux-next20260526. > > > > drivers/gpu/drm/hyperv/hyperv_drm.h | 20 ++-- > > drivers/gpu/drm/hyperv/hyperv_drm_drv.c | 88 ++++++++-------- > > drivers/gpu/drm/hyperv/hyperv_drm_modeset.c | 110 ++++++++++---------- > > drivers/gpu/drm/hyperv/hyperv_drm_proto.c | 70 ++++++------- > > 4 files changed, 144 insertions(+), 144 deletions(-) > > > > diff --git a/drivers/gpu/drm/hyperv/hyperv_drm.h > b/drivers/gpu/drm/hyperv/hyperv_drm.h > > index 9e776112c03e..66bd8730aad2 100644 > > --- a/drivers/gpu/drm/hyperv/hyperv_drm.h > > +++ b/drivers/gpu/drm/hyperv/hyperv_drm.h > > @@ -8,7 +8,7 @@ > > > > #define VMBUS_MAX_PACKET_SIZE 0x4000 > > > > -struct hyperv_drm_device { > > +struct hvdrm_drm_device { > > "hvdrm_drm_device" looks kinda redundant, perhaps > s/hyperv_drm_device/hv_drm_device would be more sensible. Yes, I'll make this change. And in looking through kallsyms, I see that the Hyper-V DRM driver has "hv_fops", which did not get changed in the mechanical substitution because it doesn't start with "hyperv_". I'll change it to hv_drm_fops. Michael
