Hi Michael,
On 10-09-2025 20:55, Michael Kelley wrote:
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.
Is deprecating the driver a mandatory step?
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
