Re: [PATCH] drm/mgag200: Increase bandwidth for G200se A rev1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Thomas,

On Wed, Jul 26, 2023 at 05:36:15PM +0200, Thomas Zimmermann wrote:
> > I've already sent a patch to use internally 24bit colors, when userspace
> > can use 32bit that would solve this issue as well. In the end, on the
> > VGA link, 24 or 32 bit colors are the same. That would allow modern
> > userspace to work on par with Xorg. So maybe we can reconsider it ?
> 
> No way. We've had IRC flamewars over this topic already. People didn't get
> work done that day; others ragequit-ed in frustration. Ask Javier, he'll
> remember. :)
> 
> The consent in DRM-land is: we're not going to mess with color formats
> behind the back of userspace. The only exception is the emulation of
> XRGB8888 iff the hardware does not support that. And only because it's the
> fallback format that is universally supported.

I'm aware that I might be reviving old heated debates here, but I'm not
sure what is the problem here.

I guess part of the problem is that due to the memcpy call, we would
have it in software.

But it's not clear to me how we're messing with color formats there: the
memcpy would drop the alpha part that was doing to be dropped by the
hardware anyway (and likely would have done so transparently if it
wasn't for the memcpy).

It doesn't affect the userspace at all, it doesn't change the way we
interpret the format either, it just ignores a part of the format that
is supposed to be ignored anyway. And doing so frees up a bunch of
resources?

So, AFAIU, it doesn't have any side effect except for the fact that it
consumes less memory bandwidth and allows for more pixels to go through.
That's not really "messing with the formats" in my book.

Maxime

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux