From: Mikulas Patocka <mpatocka@redhat.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Dave Airlie <airlied@gmail.com>, Andreas Mohr <andi@lisas.de>,
Linus Torvalds <torvalds@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
DRI <dri-devel@lists.freedesktop.org>
Subject: Re: 3.14-rc7 crashes in drm ([PATCH] a crash in mga_driver_irq_uninstall)
Date: Tue, 25 Mar 2014 18:42:13 -0400 (EDT) [thread overview]
Message-ID: <alpine.LRH.2.02.1403251836160.28109@file01.intranet.prod.int.rdu2.redhat.com> (raw)
In-Reply-To: <CAKMK7uFW9NSF1WhENn9_3GHNvZtEjn=o+NSpY5g41Qg_pqMz8A@mail.gmail.com>
On Mon, 24 Mar 2014, Daniel Vetter wrote:
> >> Like I've said the entire teardown sequence for legacy drm drivers is
> >> terminally busted, so the only hope we have is to reapply this missing
> >> duct-tape which made your X crash. But if that itself isn't a regression
> >> there's no way to fix the current drm/mga driver without a complete
> >> rewrite as a new-style kernel modesetting driver.
> >> -Daniel
> >
> > If someone understands the locking issues I pointed out above, it could be
> > easy to fix.
>
> The locking issue isn't your problem, the real issue is that putting a
> irq_uninstall into core code will break all the new (properly working)
> drivers. And you can't really fix this in mga itself since the
> lifetime rules of the register mappings are totally broken. It's a
> fundamental misdesign of the legacy drm driver architecture and the
> _only_ way to fix this bug for real is to rewrite this all. Which was
> done for all the still used drivers like i915, radeon, nouveau, ...
> -Daniel
When I tried Radeon AGP card with the KMS driver, it lacked the
possibility to set video mode with fbset and the framebuffer console was
very slow because it wasn't accelerated.
So, Radeon with the new driver is much less useable than Matrox.
Did I misconfigure something? Or, is console acceleration and modesetting
deliberately unsupported in KMS drivers?
Mikulas
next prev parent reply other threads:[~2014-03-25 22:42 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-26 21:25 [PATCH] a crash in mga_driver_irq_uninstall Mikulas Patocka
2014-03-22 23:43 ` 3.14-rc7 crashes in drm ([PATCH] a crash in mga_driver_irq_uninstall) Andreas Mohr
2014-03-23 12:15 ` Andreas Mohr
2014-03-23 16:39 ` Linus Torvalds
2014-03-23 16:39 ` Linus Torvalds
2014-03-23 21:27 ` Andreas Mohr
2014-03-23 21:45 ` Dave Airlie
2014-03-23 21:45 ` Dave Airlie
2014-03-24 8:56 ` Daniel Vetter
2014-03-24 8:56 ` Daniel Vetter
2014-03-24 17:17 ` Mikulas Patocka
2014-03-24 20:26 ` Daniel Vetter
2014-03-24 20:26 ` Daniel Vetter
2014-03-24 20:40 ` Mikulas Patocka
2014-03-24 21:46 ` Daniel Vetter
2014-03-24 21:46 ` Daniel Vetter
2014-03-24 23:11 ` Andreas Mohr
2014-03-25 9:13 ` Daniel Vetter
2014-03-25 9:13 ` Daniel Vetter
2014-03-25 22:42 ` Mikulas Patocka [this message]
2014-03-26 7:15 ` Daniel Vetter
2014-03-26 7:15 ` Daniel Vetter
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=alpine.LRH.2.02.1403251836160.28109@file01.intranet.prod.int.rdu2.redhat.com \
--to=mpatocka@redhat.com \
--cc=airlied@gmail.com \
--cc=andi@lisas.de \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
/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: link
Be 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.