Re: [PATCH 12/43] drm/mgag200: Use fbdev-shmem

2024-03-18 Thread Jocelyn Falempe




On 18/03/2024 08:56, Thomas Zimmermann wrote:

Hi

Am 13.03.24 um 15:03 schrieb Jocelyn Falempe:

Hi,

Thanks, it looks good to me.

Reviewed-by: Jocelyn Falempe 


Thanks. Do you still have access to that broken realtime system? I 
wonder if this patch makes a difference, as there's now one large 
memcpy() less.


Hi,

Sure, I'll do some latency tests if I can get access to that server again.

Best regards,



Best regards
Thomas





Re: [PATCH 12/43] drm/mgag200: Use fbdev-shmem

2024-03-18 Thread Thomas Zimmermann

Hi

Am 13.03.24 um 15:03 schrieb Jocelyn Falempe:

Hi,

Thanks, it looks good to me.

Reviewed-by: Jocelyn Falempe 


Thanks. Do you still have access to that broken realtime system? I 
wonder if this patch makes a difference, as there's now one large 
memcpy() less.


Best regards
Thomas

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)



Re: [PATCH 12/43] drm/mgag200: Use fbdev-shmem

2024-03-13 Thread Jocelyn Falempe

Hi,

Thanks, it looks good to me.

Reviewed-by: Jocelyn Falempe 

--

Jocelyn

On 12/03/2024 16:45, Thomas Zimmermann wrote:

Implement fbdev emulation with fbdev-shmem. Avoids the overhead of
fbdev-generic's additional shadow buffering. No functional changes.

Signed-off-by: Thomas Zimmermann 
Cc: Dave Airlie 
Cc: Thomas Zimmermann 
Cc: Jocelyn Falempe 
---
  drivers/gpu/drm/mgag200/mgag200_drv.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c 
b/drivers/gpu/drm/mgag200/mgag200_drv.c
index 573dbe256aa8b..65f2ed18b31c5 100644
--- a/drivers/gpu/drm/mgag200/mgag200_drv.c
+++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
@@ -12,7 +12,7 @@
  #include 
  #include 
  #include 
-#include 
+#include 
  #include 
  #include 
  #include 
@@ -285,7 +285,7 @@ mgag200_pci_probe(struct pci_dev *pdev, const struct 
pci_device_id *ent)
 * FIXME: A 24-bit color depth does not work with 24 bpp on
 * G200ER. Force 32 bpp.
 */
-   drm_fbdev_generic_setup(dev, 32);
+   drm_fbdev_shmem_setup(dev, 32);
  
  	return 0;

  }




[PATCH 12/43] drm/mgag200: Use fbdev-shmem

2024-03-12 Thread Thomas Zimmermann
Implement fbdev emulation with fbdev-shmem. Avoids the overhead of
fbdev-generic's additional shadow buffering. No functional changes.

Signed-off-by: Thomas Zimmermann 
Cc: Dave Airlie 
Cc: Thomas Zimmermann 
Cc: Jocelyn Falempe 
---
 drivers/gpu/drm/mgag200/mgag200_drv.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c 
b/drivers/gpu/drm/mgag200/mgag200_drv.c
index 573dbe256aa8b..65f2ed18b31c5 100644
--- a/drivers/gpu/drm/mgag200/mgag200_drv.c
+++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
@@ -12,7 +12,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -285,7 +285,7 @@ mgag200_pci_probe(struct pci_dev *pdev, const struct 
pci_device_id *ent)
 * FIXME: A 24-bit color depth does not work with 24 bpp on
 * G200ER. Force 32 bpp.
 */
-   drm_fbdev_generic_setup(dev, 32);
+   drm_fbdev_shmem_setup(dev, 32);
 
return 0;
 }
-- 
2.44.0