[PATCH] mi: Remove unused overlay support
aplattner at nvidia.com
Tue Mar 29 10:40:29 PDT 2011
On Tue, Mar 29, 2011 at 09:47:19AM -0700, Mikhail Gusarov wrote:
> Twas brillig at 09:26:19 29.03.2011 UTC-07 when aplattner at nvidia.com did gyre and gimble:
> AP> NAK. This code is not NVIDIA-specific just because only the NVIDIA
> AP> driver currently uses it.
> I did not claim it. I just said it's unused.
And I'm disagreeing about it being unused. I'm actually surprised that
none of the other drivers use it since workstation overlays are still
> AP> We can move it into the driver if you really think it's necessary,
> AP> but from a software design standpoint it's the wrong thing to do.
> Keeping unused code in mi is wrong from software engineering standpoint
> as well.
I disagree. mi is a toolkit of machine-independent driver support
routines. While I understand that X needs to advance forward and gain
features, I don't think it should turn its back on its loyal professional
workstation heritage either. NVIDIA is committed to supporting both the
forward advancement of the window system and traditional X features like
overlays. It is disappointing if no other driver shares both those
The mi overlay code is not hardware-specific, and architecturally makes the
most sense in the X server's mi layer. You can sign me up as the
maintainer if you think it's a maintenance burden.
More information about the xorg-devel