All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.