From: Thomas Zimmermann <tzimmerm...@suse.de> Sent: Wednesday, September 10, 2025 2:36 AM > > Hi > > Am 09.09.25 um 18:58 schrieb Prasanna Kumar T S M: > > The Hyper-V DRM driver is available since kernel version 5.14 and > > provides full KMS support along with fbdev emulation via the DRM fbdev > > helpers. This makes the hyperv_fb driver redundant, remove it. > > I'm all for removing obsolete drivers. But hyperv_drm likely first needs > to merge the patch at > https://lore.kernel.org/dri-devel/20250904145806.430568-5-tzimmerm...@suse.de/ > It's been tested and works well. If maintainers from Microsoft have a > look at the patch first, we could possibly land it fairly soon.
Thomas -- My testing of your v3 patch series for vblank timers ended up getting a WARN_ON after about 3 days of usage. See [1]. So I don't think it's 100% ready yet. But I agree we need your synthetic vblank timer support to address the Hyper-V DRM driver performance issue, before removing the Hyper-V fbdev driver. (See [2] for a description of the performance issue.) Second, isn't it customary to mark a driver as deprecated for a period of time, before removing it entirely? I don't see any documentation on the deprecation process, but I've seen it done in other cases. If you grep through all the kernel Kconfig files, you'll see entries tagged with DEPRECATED. Also the driver should be updated to output a deprecated message when it loads. Michael [1] https://lore.kernel.org/dri-devel/bn7pr02mb4148e80c13605f6ead2b0a03d4...@bn7pr02mb4148.namprd02.prod.outlook.com/ [2] https://lore.kernel.org/dri-devel/sn6pr02mb415702b00d6d52b0ee962c98d4...@sn6pr02mb4157.namprd02.prod.outlook.com/ > > Best regards > Thomas > > > > > Signed-off-by: Prasanna Kumar T S M <p...@linux.microsoft.com> > > --- > > MAINTAINERS | 1 - > > drivers/video/fbdev/Kconfig | 8 - > > drivers/video/fbdev/Makefile | 1 - > > drivers/video/fbdev/hyperv_fb.c | 1386 ------------------------------- > > 4 files changed, 1396 deletions(-) > > delete mode 100644 drivers/video/fbdev/hyperv_fb.c > >
