Bug reassigned.

Updated package uploaded to Mentors. Verified to work on a Geode host.

Help for getting this cleared for Testing is welcome.

debdiff attached.

Thanks!
Martin-Éric

ma 17. toukok. 2021 klo 21.39 Martin-Éric Racine
([email protected]) kirjoitti:
>
> ma 17. toukok. 2021 klo 21.13 Ben Hutchings ([email protected]) kirjoitti:
> >
> > On Mon, 2021-05-17 at 20:01 +0200, Ben Hutchings wrote:
> > > Control: tag -1 wontfix
> > >
> > > On Wed, 2021-05-12 at 06:16 +0300, Martin-Éric Racine wrote:
> > > > This still applies to current kernels.
> > > >
> > > > The problem seems to be that kernels after version 3 implement a
> > > > memory protection scheme that prevents the framebuffer from being
> > > > accessed by both vesafb and X drivers. The Geode X driver does that.
> > >
> > > Then this is not a kernel bug.  The change was intentional and
> > > documented in the NEWS file along with the option to revert it:
> > >
> > >     - On most architectures, the /dev/mem device can no longer be used to
> > >       access devices that also have a kernel driver.  This breaks dosemu
> > >       and some old user-space graphics drivers.  To allow this, set the
> > >       kernel parameter: iomem=relaxed
> >
> > That said, this was originally reported before that change was made, so
> > possibly there are two different issues here.  Let us know if the
> > kernel parameter change doesn't allow the X driver to work again.
>
> Using this cmdline parameter indeed restores X driver operation. Feel
> free to reassign.
>
> I'll prepare a small patch to the Geode driver's own README to mention
> this and upload the result to unstable.
>
> Thanks for pointing out the fix.
>
> Cheers!
> Martin-Éric
diff -Nru xserver-xorg-video-geode-2.11.20/debian/changelog 
xserver-xorg-video-geode-2.11.20/debian/changelog
--- xserver-xorg-video-geode-2.11.20/debian/changelog   2021-01-03 
17:11:43.000000000 +0200
+++ xserver-xorg-video-geode-2.11.20/debian/changelog   2021-05-17 
21:58:21.000000000 +0300
@@ -1,3 +1,18 @@
+xserver-xorg-video-geode (2.11.20-6) unstable; urgency=medium
+
+  * Merged patch from Git (Closes: #822112):
+    03_Mention-iomem-relaxed.patch
+
+    Fixes a long-standing issue that prevents this legacy X driver from booting
+    with stock Linux kernels version 4 or newer. Merely adding a cmdline option
+    makes vesafb and legacy X drivers like this one coexist peacefully again.
+
+    This short patch to the upstream README mentions iomem=relaxed as the fix.
+
+    Thanks to Ben Hutchings for figuring this one out.
+
+ -- Martin-Éric Racine <[email protected]>  Mon, 17 May 2021 21:58:21 
+0300
+
 xserver-xorg-video-geode (2.11.20-5) unstable; urgency=medium
 
   * [watch]
diff -Nru 
xserver-xorg-video-geode-2.11.20/debian/patches/03_Mention-iomem-relaxed.patch 
xserver-xorg-video-geode-2.11.20/debian/patches/03_Mention-iomem-relaxed.patch
--- 
xserver-xorg-video-geode-2.11.20/debian/patches/03_Mention-iomem-relaxed.patch  
    1970-01-01 02:00:00.000000000 +0200
+++ 
xserver-xorg-video-geode-2.11.20/debian/patches/03_Mention-iomem-relaxed.patch  
    2021-05-17 21:55:02.000000000 +0300
@@ -0,0 +1,37 @@
+From ae82f4b2746eb24295389f184bc18a9cb6c8f31c Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Martin-=C3=89ric=20Racine?= <[email protected]>
+Date: Mon, 17 May 2021 21:50:23 +0300
+Subject: [PATCH] Mention iomem=relaxed in the README FAQ for Linux 4.x+
+ requirements
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Thanks to Ben Hutchings for pointing out that vesafb and legacy X
+drivers can peacefully coexist if iomem=relaxed gets added to the
+Linux kernel cmdline options.
+
+Signed-off-by: Martin-Éric Racine <[email protected]>
+---
+ README | 5 ++++-
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+diff --git a/README b/README
+index e3ec6ee..111f9a6 100644
+--- a/README
++++ b/README
+@@ -94,7 +94,10 @@ A: Since kernel 4.x Linux has strong memory protection. If 
the kernel is
+ 
+    GRUB_GFXPAYLOAD_LINUX=text
+ 
+-   Once this option has been added and the GRUB configuration refreshed,
++   Alternately, adding iomem=relaxed to GRUB_CMDLINE_LINUX_DEFAULT will
++   allow the video memory to be accessed by vesafb and the GEODE driver.
++
++   Once either option has been added and the GRUB configuration refreshed,
+    the GEODE driver will launch on recent kernels as previously.
+ 
+ Q: Why doesn't the GEODE driver work at WXGA (wide screen) resolutions?
+-- 
+2.20.1
+
diff -Nru xserver-xorg-video-geode-2.11.20/debian/patches/series 
xserver-xorg-video-geode-2.11.20/debian/patches/series
--- xserver-xorg-video-geode-2.11.20/debian/patches/series      2020-12-15 
20:27:27.000000000 +0200
+++ xserver-xorg-video-geode-2.11.20/debian/patches/series      2021-05-17 
21:57:57.000000000 +0300
@@ -1,2 +1,3 @@
 01_fno-common.patch
 02_Updated-the-README.patch
+03_Mention-iomem-relaxed.patch

Reply via email to