linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Dave Airlie <airlied@gmail.com>, Dave Airlie <airlied@linux.ie>,
	torvalds@linux-foundation.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.sf.net
Subject: Re: hung bootup with "drm/radeon/kms: move radeon KMS on/off switch out of staging."
Date: Thu, 4 Feb 2010 20:19:35 +0100	[thread overview]
Message-ID: <20100204191935.GC6665@elte.hu> (raw)
In-Reply-To: <20100204190012.GA21950@srcf.ucam.org>


* Matthew Garrett <mjg59@srcf.ucam.org> wrote:

> On Thu, Feb 04, 2010 at 07:56:03PM +0100, Ingo Molnar wrote:
> 
> > Do you see my argument why any user who is hit by this would categorize 
> > this as a kernel regression in an existing driver?
> 
> No. If a user changes configuration and gets a hang, that's a bug but not a 
> regression.

Only if it's some brand-new driver or a brand-new kernel feature for which 
no-one can have any prior expectations of stability. Especially if it's added 
in the merge window when many new drivers are added.

But isnt it a regression to a user if it's shipped in -rc7 appearing as a new 
sub-option of an existing driver?

I'd wager that most main-street Linux users would consider that a regression.
 
As i see it is that you are trying to have it both ways: claim it's a new 
driver when it comes to handling regressions, but also try to have the 
benefits (and adoption flux) of an old driver when it comes to facing it to 
users.

Some info about that in the Kconfig would be helpful IMO - so that people are 
less surprised if it happens to break while the radeon driver worked fine for 
them before.

Thanks,

	Ingo

  reply	other threads:[~2010-02-04 19:19 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-01  2:20 [git pull] drm fixes Dave Airlie
2010-02-01  2:53 ` Dave Airlie
2010-02-02  8:17   ` [crash, PATCH] Revert "drm/radeon/kms: move radeon KMS on/off switch out of staging." Ingo Molnar
2010-02-02  8:25     ` Dave Airlie
2010-02-02 15:44       ` Ingo Molnar
2010-02-04 20:39         ` Dave Airlie
2010-02-04 20:46           ` Ingo Molnar
2010-02-04 21:14             ` Dave Airlie
2010-02-05  0:43               ` Dave Airlie
2010-02-05  7:32                 ` Ingo Molnar
2010-02-02  8:35     ` Dave Airlie
2010-02-02  8:37       ` Dave Airlie
2010-02-02 15:42         ` Ingo Molnar
2010-02-02 15:46       ` Ingo Molnar
2010-02-02 20:34         ` Dave Airlie
2010-02-04  6:26           ` Ingo Molnar
2010-02-04  6:39             ` Dave Airlie
2010-02-04  7:36               ` Ingo Molnar
2010-02-04  7:49                 ` Dave Airlie
2010-02-04  7:17             ` hung bootup with " Ingo Molnar
2010-02-04 16:48               ` Matthew Garrett
2010-02-04 17:08                 ` Ingo Molnar
2010-02-04 17:15                   ` Linus Torvalds
2010-02-04 17:36                     ` Ingo Molnar
2010-02-04 17:36                   ` Matthew Garrett
2010-02-04 17:54                     ` Ingo Molnar
2010-02-04 17:59                       ` Matthew Garrett
2010-02-04 18:12                         ` Ingo Molnar
2010-02-04 18:15                           ` Matthew Garrett
2010-02-04 18:56                             ` Ingo Molnar
2010-02-04 19:00                               ` Matthew Garrett
2010-02-04 19:19                                 ` Ingo Molnar [this message]
2010-02-04 19:28                                   ` Jerome Glisse
2010-02-04 20:34                                     ` Ingo Molnar
2010-02-04 18:30                           ` Alex Deucher
2010-02-04 19:06                             ` Ingo Molnar
2010-02-04 19:18                               ` Alex Deucher
2010-02-04 19:24                                 ` Linus Torvalds
2010-02-04 19:34                                   ` Dave Airlie
2010-02-04 20:27                                     ` Ingo Molnar
2010-02-04 19:32                                 ` Ingo Molnar
2010-02-04 19:53                                   ` Jesse Barnes
2010-02-04 20:22                                     ` Ingo Molnar
2010-02-04 20:27                                       ` david
2010-02-04 20:33                                       ` Jesse Barnes
2010-02-04 20:57                                         ` Ingo Molnar
2010-02-04 20:48                                       ` Matthew Garrett
2010-02-04 21:05                                         ` Ingo Molnar
2010-02-04 21:09                                           ` Matthew Garrett
2010-02-05  7:56                                             ` Ingo Molnar
2010-02-05  8:34                                               ` Dave Airlie
2010-02-05  9:00                                                 ` Ingo Molnar
2010-02-05  9:18                                                   ` Dave Airlie
2010-02-05 10:47                                                     ` Ingo Molnar
2010-02-04 21:23                                           ` Andrew Morton
2010-02-04 21:34                                             ` Jesse Barnes
2010-02-04 21:35                                             ` Dave Airlie
2010-02-06 11:10                                             ` Felipe Contreras
2010-02-02  8:58     ` [crash, PATCH] Revert " Domenico Andreoli
2010-02-02 11:59       ` Jerome Glisse
2010-02-02 15:11         ` Domenico Andreoli
2010-02-02 11:56     ` Jerome Glisse
2010-02-02 15:42       ` Ingo Molnar
2010-02-02 23:15         ` Jerome Glisse

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=20100204191935.GC6665@elte.hu \
    --to=mingo@elte.hu \
    --cc=airlied@gmail.com \
    --cc=airlied@linux.ie \
    --cc=dri-devel@lists.sf.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.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 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).