From: Geert Uytterhoeven <geert@linux-m68k.org> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Helge Deller <deller@gmx.de>, Thomas Zimmermann <tzimmermann@suse.de>, Linux Fbdev development list <linux-fbdev@vger.kernel.org>, Sven Schnelle <svens@stackframe.org>, Linus Torvalds <torvalds@linux-foundation.org>, Javier Martinez Canillas <javierm@redhat.com>, Daniel Vetter <daniel.vetter@intel.com>, Ilia Mirkin <imirkin@alum.mit.edu>, Tomi Valkeinen <tomi.valkeinen@ti.com>, DRI Development <dri-devel@lists.freedesktop.org>, Jani Nikula <jani.nikula@linux.intel.com>, Pavel Machek <pavel@ucw.cz>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Gerd Hoffmann <kraxel@redhat.com>, Sam Ravnborg <sam@ravnborg.org>, Claudio Suarez <cssk@net-c.es> Subject: Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling" Date: Wed, 19 Jan 2022 12:47:50 +0100 [thread overview] Message-ID: <CAMuHMdVWFJEDwjf-htZ_D1484efmuPnz_L-qhcTeUE-GVpvZXA@mail.gmail.com> (raw) In-Reply-To: <Yef15k2GtC40aJEu@kroah.com> Hi Greg, On Wed, Jan 19, 2022 at 12:28 PM Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > On Wed, Jan 19, 2022 at 12:22:55PM +0100, Greg Kroah-Hartman wrote: > > On Wed, Jan 19, 2022 at 12:08:39PM +0100, Helge Deller wrote: > > > This reverts commit 39aead8373b3c20bb5965c024dfb51a94e526151. > > > > > > Revert this patch. This patch started to introduce the regression that > > > all hardware acceleration of more than 35 existing fbdev drivers were > > > bypassed and thus fbcon console output for those was dramatically slowed > > > down by factor of 10 and more. > > > > > > Reverting this commit has no impact on DRM, since none of the DRM drivers are > > > tagged with the acceleration flags FBINFO_HWACCEL_COPYAREA, > > > FBINFO_HWACCEL_FILLRECT or others. > > > > > > Signed-off-by: Helge Deller <deller@gmx.de> > > As for "why", I think there was a number of private bugs that were > > reported in this code, which is why it was removed. I do not think it > > can be safely added back in without addressing them first. Let me go > > dig through my email to see if I can find them... > > Ah, no, that was just the soft scrollback code I was thinking of, which So the bugs argument is moot. > was a different revert and is still gone, thankfully :) FTR, not everyone else was thankful about that one... > This one was just removed because Daniel noticed that only 3 drivers > used this (nouveau, omapdrm, and gma600), so this shouldn't have caused > any regressions in any other drivers like you are reporting here. > > So perhaps this regression is caused by something else? 1. Daniel's patch was not CCed to linux-fbdev, 2. When I discovered the patch, I pointed out that the premise of 3 drivers was not true, and that it affects 32 more fbdev drivers[1] . The patch was applied regardless. 3. When the patch was suggested for backporting, I pointed out the same[2]. The patch was backported regardless. [1] https://lore.kernel.org/r/alpine.DEB.2.22.394.2010311116530.379363@ramsan.of.borg/ [2] https://lore.kernel.org/r/CAMuHMdXRgam2zahPEGcw8+76Xm-0AO-Ci9-YmVa5JpTKVHphRw@mail.gmail.com/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
WARNING: multiple messages have this Message-ID (diff)
From: Geert Uytterhoeven <geert@linux-m68k.org> To: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Linux Fbdev development list <linux-fbdev@vger.kernel.org>, Pavel Machek <pavel@ucw.cz>, Sam Ravnborg <sam@ravnborg.org>, Sven Schnelle <svens@stackframe.org>, Helge Deller <deller@gmx.de>, Javier Martinez Canillas <javierm@redhat.com>, DRI Development <dri-devel@lists.freedesktop.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Tomi Valkeinen <tomi.valkeinen@ti.com>, Claudio Suarez <cssk@net-c.es>, Thomas Zimmermann <tzimmermann@suse.de>, Daniel Vetter <daniel.vetter@intel.com>, Linus Torvalds <torvalds@linux-foundation.org>, Gerd Hoffmann <kraxel@redhat.com> Subject: Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling" Date: Wed, 19 Jan 2022 12:47:50 +0100 [thread overview] Message-ID: <CAMuHMdVWFJEDwjf-htZ_D1484efmuPnz_L-qhcTeUE-GVpvZXA@mail.gmail.com> (raw) In-Reply-To: <Yef15k2GtC40aJEu@kroah.com> Hi Greg, On Wed, Jan 19, 2022 at 12:28 PM Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > On Wed, Jan 19, 2022 at 12:22:55PM +0100, Greg Kroah-Hartman wrote: > > On Wed, Jan 19, 2022 at 12:08:39PM +0100, Helge Deller wrote: > > > This reverts commit 39aead8373b3c20bb5965c024dfb51a94e526151. > > > > > > Revert this patch. This patch started to introduce the regression that > > > all hardware acceleration of more than 35 existing fbdev drivers were > > > bypassed and thus fbcon console output for those was dramatically slowed > > > down by factor of 10 and more. > > > > > > Reverting this commit has no impact on DRM, since none of the DRM drivers are > > > tagged with the acceleration flags FBINFO_HWACCEL_COPYAREA, > > > FBINFO_HWACCEL_FILLRECT or others. > > > > > > Signed-off-by: Helge Deller <deller@gmx.de> > > As for "why", I think there was a number of private bugs that were > > reported in this code, which is why it was removed. I do not think it > > can be safely added back in without addressing them first. Let me go > > dig through my email to see if I can find them... > > Ah, no, that was just the soft scrollback code I was thinking of, which So the bugs argument is moot. > was a different revert and is still gone, thankfully :) FTR, not everyone else was thankful about that one... > This one was just removed because Daniel noticed that only 3 drivers > used this (nouveau, omapdrm, and gma600), so this shouldn't have caused > any regressions in any other drivers like you are reporting here. > > So perhaps this regression is caused by something else? 1. Daniel's patch was not CCed to linux-fbdev, 2. When I discovered the patch, I pointed out that the premise of 3 drivers was not true, and that it affects 32 more fbdev drivers[1] . The patch was applied regardless. 3. When the patch was suggested for backporting, I pointed out the same[2]. The patch was backported regardless. [1] https://lore.kernel.org/r/alpine.DEB.2.22.394.2010311116530.379363@ramsan.of.borg/ [2] https://lore.kernel.org/r/CAMuHMdXRgam2zahPEGcw8+76Xm-0AO-Ci9-YmVa5JpTKVHphRw@mail.gmail.com/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
next prev parent reply other threads:[~2022-01-19 11:48 UTC|newest] Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-01-19 11:08 [PATCH 0/2] Fix regression introduced by disabling accelerated scrolling in fbcon Helge Deller 2022-01-19 11:08 ` [PATCH 1/2] Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)" Helge Deller 2022-01-19 11:08 ` [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling" Helge Deller 2022-01-19 11:22 ` Greg Kroah-Hartman 2022-01-19 11:22 ` Greg Kroah-Hartman 2022-01-19 11:28 ` Greg Kroah-Hartman 2022-01-19 11:28 ` Greg Kroah-Hartman 2022-01-19 11:47 ` Geert Uytterhoeven [this message] 2022-01-19 11:47 ` Geert Uytterhoeven 2022-01-19 12:28 ` Helge Deller 2022-01-19 12:28 ` Helge Deller 2022-01-19 14:01 ` Linus Torvalds 2022-01-19 14:01 ` Linus Torvalds 2022-01-19 14:11 ` Helge Deller 2022-01-19 14:11 ` Helge Deller 2022-01-19 14:34 ` Daniel Vetter 2022-01-19 14:34 ` Daniel Vetter 2022-01-19 15:05 ` Sven Schnelle 2022-01-19 15:05 ` Sven Schnelle 2022-01-19 15:37 ` Thomas Zimmermann 2022-01-19 15:37 ` Thomas Zimmermann 2022-01-19 16:55 ` Helge Deller 2022-01-19 16:55 ` Helge Deller 2022-01-19 15:42 ` Daniel Vetter 2022-01-19 15:42 ` Daniel Vetter 2022-01-19 15:54 ` Helge Deller 2022-01-19 15:54 ` Helge Deller 2022-01-19 16:13 ` Jani Nikula 2022-01-19 16:13 ` Jani Nikula 2022-01-19 16:31 ` Daniel Vetter 2022-01-19 16:31 ` Daniel Vetter 2022-01-19 15:18 ` Helge Deller 2022-01-19 15:18 ` Helge Deller 2022-01-19 13:01 ` Sven Schnelle 2022-01-19 13:01 ` Sven Schnelle 2022-01-19 13:35 ` Greg Kroah-Hartman 2022-01-19 13:35 ` Greg Kroah-Hartman 2022-01-20 14:30 ` Daniel Vetter 2022-01-20 14:30 ` Daniel Vetter 2022-01-20 17:01 ` Helge Deller 2022-01-21 7:20 ` Gerd Hoffmann 2022-01-21 7:20 ` Gerd Hoffmann 2022-01-24 11:10 ` Helge Deller 2022-01-24 11:10 ` Helge Deller 2022-01-24 11:28 ` Javier Martinez Canillas 2022-01-24 11:28 ` Javier Martinez Canillas 2022-01-24 11:33 ` Thomas Zimmermann 2022-01-24 11:33 ` Thomas Zimmermann 2022-01-24 11:50 ` Javier Martinez Canillas 2022-01-24 11:50 ` Javier Martinez Canillas 2022-01-24 15:29 ` Helge Deller 2022-01-24 15:29 ` Helge Deller 2022-01-24 15:45 ` Thomas Zimmermann 2022-01-24 15:45 ` Thomas Zimmermann 2022-01-24 15:50 ` Geert Uytterhoeven 2022-01-24 15:50 ` Geert Uytterhoeven 2022-01-24 15:58 ` Daniel Vetter 2022-01-24 15:58 ` Daniel Vetter 2022-01-24 15:50 ` Geert Uytterhoeven 2022-01-24 15:50 ` Geert Uytterhoeven 2022-01-24 16:11 ` Thomas Zimmermann 2022-01-24 16:11 ` Thomas Zimmermann 2022-01-19 12:45 ` [PATCH 0/2] Fix regression introduced by disabling accelerated scrolling in fbcon Geert Uytterhoeven 2022-01-19 12:45 ` Geert Uytterhoeven 2022-01-19 13:02 ` Sven Schnelle 2022-01-19 13:02 ` Sven Schnelle
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAMuHMdVWFJEDwjf-htZ_D1484efmuPnz_L-qhcTeUE-GVpvZXA@mail.gmail.com \ --to=geert@linux-m68k.org \ --cc=cssk@net-c.es \ --cc=daniel.vetter@intel.com \ --cc=deller@gmx.de \ --cc=dri-devel@lists.freedesktop.org \ --cc=gregkh@linuxfoundation.org \ --cc=imirkin@alum.mit.edu \ --cc=jani.nikula@linux.intel.com \ --cc=javierm@redhat.com \ --cc=kraxel@redhat.com \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=pavel@ucw.cz \ --cc=sam@ravnborg.org \ --cc=svens@stackframe.org \ --cc=tomi.valkeinen@ti.com \ --cc=torvalds@linux-foundation.org \ --cc=tzimmermann@suse.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.