* [PATCH] video/console/vgacon: Print big fat warning with nomodeset @ 2018-07-02 21:04 ` Lyude Paul 2018-07-03 7:34 ` Daniel Vetter 2018-07-24 15:24 ` Bartlomiej Zolnierkiewicz 0 siblings, 2 replies; 4+ messages in thread From: Lyude Paul @ 2018-07-02 21:04 UTC (permalink / raw) To: dri-devel Cc: Bartlomiej Zolnierkiewicz, Kees Cook, Jan H. Schönherr, Bjorn Helgaas, linux-fbdev, linux-kernel It's been a pretty good while since kernel modesetting was introduced. It has almost entirely replaced previous solutions which required userspace modesetting, and I can't even recall any drivers off the top of my head for modern day hardware that don't only support one or the other. Even nvidia's ugly blob does not require the use of nomodeset, and only requires that nouveau be blacklisted. Effectively, the only thing nomodeset does in the year 2018 is disable your graphics drivers. Since VESA is a thing, this will give many users the false impression that they've actually fixed an issue they were having with their machine simply because the laptop will boot up to a degraded GUI. This of course, is never actually the case. Things get even worse when you consider that there's still an enormous amount of tutorials users find on the internet that still suggest adding nomodeset, along with various users who have been around long enough to still suggest it. There really isn't any legitimate reason I can see for this to be an option that's used by anyone else other then developers, or properly informed users. So, let's end the confusion and start printing warnings whenever it's enabled. Signed-off-by: Lyude Paul <lyude@redhat.com> --- drivers/video/console/vgacon.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/video/console/vgacon.c b/drivers/video/console/vgacon.c index f09e17b60e45..09731b2f6815 100644 --- a/drivers/video/console/vgacon.c +++ b/drivers/video/console/vgacon.c @@ -112,6 +112,11 @@ EXPORT_SYMBOL(vgacon_text_force); static int __init text_mode(char *str) { vgacon_text_mode_force = true; + + pr_warning("You have booted with nomodeset. This means your GPU drivers are DISABLED\n"); + pr_warning("Any video related functionality will be severely degraded, and you may not even be able to suspend the system properly\n"); + pr_warning("Unless you actually understand what nomodeset does, you should reboot without enabling it\n"); + return 1; } -- 2.17.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] video/console/vgacon: Print big fat warning with nomodeset 2018-07-02 21:04 ` [PATCH] video/console/vgacon: Print big fat warning with nomodeset Lyude Paul @ 2018-07-03 7:34 ` Daniel Vetter 2018-07-03 11:57 ` Jani Nikula 2018-07-24 15:24 ` Bartlomiej Zolnierkiewicz 1 sibling, 1 reply; 4+ messages in thread From: Daniel Vetter @ 2018-07-03 7:34 UTC (permalink / raw) To: Lyude Paul Cc: linux-fbdev, Kees Cook, Bartlomiej Zolnierkiewicz, Jan H. Schönherr, linux-kernel, dri-devel, Bjorn Helgaas On Mon, Jul 02, 2018 at 05:04:40PM -0400, Lyude Paul wrote: > It's been a pretty good while since kernel modesetting was introduced. > It has almost entirely replaced previous solutions which required > userspace modesetting, and I can't even recall any drivers off the top > of my head for modern day hardware that don't only support one or the > other. Even nvidia's ugly blob does not require the use of nomodeset, > and only requires that nouveau be blacklisted. > > Effectively, the only thing nomodeset does in the year 2018 is disable > your graphics drivers. Since VESA is a thing, this will give many users > the false impression that they've actually fixed an issue they were > having with their machine simply because the laptop will boot up to a > degraded GUI. This of course, is never actually the case. > > Things get even worse when you consider that there's still an enormous > amount of tutorials users find on the internet that still suggest adding > nomodeset, along with various users who have been around long enough to > still suggest it. > > There really isn't any legitimate reason I can see for this to be an > option that's used by anyone else other then developers, or properly > informed users. So, let's end the confusion and start printing warnings > whenever it's enabled. > > Signed-off-by: Lyude Paul <lyude@redhat.com> Yeah makes tons of sense. I think we should also perhaps mark the option as a tainting one, but I think that's only wired up for module option. Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/video/console/vgacon.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/video/console/vgacon.c b/drivers/video/console/vgacon.c > index f09e17b60e45..09731b2f6815 100644 > --- a/drivers/video/console/vgacon.c > +++ b/drivers/video/console/vgacon.c > @@ -112,6 +112,11 @@ EXPORT_SYMBOL(vgacon_text_force); > static int __init text_mode(char *str) > { > vgacon_text_mode_force = true; > + > + pr_warning("You have booted with nomodeset. This means your GPU drivers are DISABLED\n"); > + pr_warning("Any video related functionality will be severely degraded, and you may not even be able to suspend the system properly\n"); > + pr_warning("Unless you actually understand what nomodeset does, you should reboot without enabling it\n"); > + > return 1; > } > > -- > 2.17.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] video/console/vgacon: Print big fat warning with nomodeset 2018-07-03 7:34 ` Daniel Vetter @ 2018-07-03 11:57 ` Jani Nikula 0 siblings, 0 replies; 4+ messages in thread From: Jani Nikula @ 2018-07-03 11:57 UTC (permalink / raw) To: Daniel Vetter, Lyude Paul Cc: linux-fbdev, Kees Cook, Bartlomiej Zolnierkiewicz, Jan H. Schönherr, linux-kernel, dri-devel, Bjorn Helgaas On Tue, 03 Jul 2018, Daniel Vetter <daniel@ffwll.ch> wrote: > On Mon, Jul 02, 2018 at 05:04:40PM -0400, Lyude Paul wrote: >> It's been a pretty good while since kernel modesetting was introduced. >> It has almost entirely replaced previous solutions which required >> userspace modesetting, and I can't even recall any drivers off the top >> of my head for modern day hardware that don't only support one or the >> other. Even nvidia's ugly blob does not require the use of nomodeset, >> and only requires that nouveau be blacklisted. >> >> Effectively, the only thing nomodeset does in the year 2018 is disable >> your graphics drivers. Since VESA is a thing, this will give many users >> the false impression that they've actually fixed an issue they were >> having with their machine simply because the laptop will boot up to a >> degraded GUI. This of course, is never actually the case. >> >> Things get even worse when you consider that there's still an enormous >> amount of tutorials users find on the internet that still suggest adding >> nomodeset, along with various users who have been around long enough to >> still suggest it. >> >> There really isn't any legitimate reason I can see for this to be an >> option that's used by anyone else other then developers, or properly >> informed users. So, let's end the confusion and start printing warnings >> whenever it's enabled. >> >> Signed-off-by: Lyude Paul <lyude@redhat.com> > > Yeah makes tons of sense. I think we should also perhaps mark the option > as a tainting one, but I think that's only wired up for module option. You could do that by adding add_taint(TAINT_USER, LOCKDEP_STILL_OK); in text_mode(). If you do that, you should probably amend the warning message accordingly. Either way, Acked-by: Jani Nikula <jani.nikula@intel.com> > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > >> --- >> drivers/video/console/vgacon.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/drivers/video/console/vgacon.c b/drivers/video/console/vgacon.c >> index f09e17b60e45..09731b2f6815 100644 >> --- a/drivers/video/console/vgacon.c >> +++ b/drivers/video/console/vgacon.c >> @@ -112,6 +112,11 @@ EXPORT_SYMBOL(vgacon_text_force); >> static int __init text_mode(char *str) >> { >> vgacon_text_mode_force = true; >> + >> + pr_warning("You have booted with nomodeset. This means your GPU drivers are DISABLED\n"); >> + pr_warning("Any video related functionality will be severely degraded, and you may not even be able to suspend the system properly\n"); >> + pr_warning("Unless you actually understand what nomodeset does, you should reboot without enabling it\n"); >> + >> return 1; >> } >> >> -- >> 2.17.1 >> >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] video/console/vgacon: Print big fat warning with nomodeset 2018-07-02 21:04 ` [PATCH] video/console/vgacon: Print big fat warning with nomodeset Lyude Paul 2018-07-03 7:34 ` Daniel Vetter @ 2018-07-24 15:24 ` Bartlomiej Zolnierkiewicz 1 sibling, 0 replies; 4+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2018-07-24 15:24 UTC (permalink / raw) To: Lyude Paul Cc: linux-fbdev, Kees Cook, Jan H. Schönherr, linux-kernel, dri-devel, Bjorn Helgaas On Monday, July 02, 2018 05:04:40 PM Lyude Paul wrote: > It's been a pretty good while since kernel modesetting was introduced. > It has almost entirely replaced previous solutions which required > userspace modesetting, and I can't even recall any drivers off the top > of my head for modern day hardware that don't only support one or the > other. Even nvidia's ugly blob does not require the use of nomodeset, > and only requires that nouveau be blacklisted. > > Effectively, the only thing nomodeset does in the year 2018 is disable > your graphics drivers. Since VESA is a thing, this will give many users > the false impression that they've actually fixed an issue they were > having with their machine simply because the laptop will boot up to a > degraded GUI. This of course, is never actually the case. > > Things get even worse when you consider that there's still an enormous > amount of tutorials users find on the internet that still suggest adding > nomodeset, along with various users who have been around long enough to > still suggest it. > > There really isn't any legitimate reason I can see for this to be an > option that's used by anyone else other then developers, or properly > informed users. So, let's end the confusion and start printing warnings > whenever it's enabled. > > Signed-off-by: Lyude Paul <lyude@redhat.com> Patch queued for 4.19, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-07-24 15:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20180702210516epcas3p151aefd4a0217c4cd30143147f0bafe1d@epcas3p1.samsung.com>
2018-07-02 21:04 ` [PATCH] video/console/vgacon: Print big fat warning with nomodeset Lyude Paul
2018-07-03 7:34 ` Daniel Vetter
2018-07-03 11:57 ` Jani Nikula
2018-07-24 15:24 ` Bartlomiej Zolnierkiewicz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).