public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Thomas Renninger <trenn@suse.de>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
	Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	Len Brown <lenb@kernel.org>,
	linux-acpi <linux-acpi@vger.kernel.org>,
	Zhao Yakui <yakui.zhao@intel.com>,
	me@markdoughty.co.uk,
	linux-thinkpad <linux-thinkpad@linux-thinkpad.org>,
	"devel@acpica.org" <devel@acpica.org>
Subject: Re: [RESEND] [PATCH 2/3] Introduce acpi_root_table=rsdt boot param and dmi list to force rsdt
Date: Tue, 21 Oct 2008 16:01:55 +0200	[thread overview]
Message-ID: <200810211601.56527.rjw@sisk.pl> (raw)
In-Reply-To: <200810211534.44226.trenn@suse.de>

On Tuesday, 21 of October 2008, Thomas Renninger wrote:
> On Tuesday 21 October 2008 15:08:36 Matthew Garrett wrote:
> > On Tue, Oct 21, 2008 at 03:05:33PM +0200, Thomas Renninger wrote:
> > > Even if this is not proven, the patchset should still go in to get the
> > > machines fixed for now and to easily test others, there was enough
> > > discussion around it.
> >
> > What do you mean, "fixed"? They all already work, even if it's in a
> > suboptimal way.
> 
> If C-states don't work and do work with a patch. I call the patch a fix.
> I also call the machine broken (even if it does boot) before and fixed 
> afterwards.

If not a fix, this certainly is improvement.

> I also call a machine broken which takes minutes to boot and fixed as soon as 
> the problem is solved and the machine is booting in a reasonable amount of 
> time.

I haven't followed the discussion closely, but IMO it's better to have the
machines in a usable state right now than wait for a "real fix" forever.

Moreover, as soon as the real fix is developed we can remove the DMI list.
Quite frankly, I don't see any downsides.

Thanks,
Rafael

  reply	other threads:[~2008-10-21 13:57 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-19 21:50 [RESEND] [PATCH 2/3] Introduce acpi_root_table=rsdt boot param and dmi list to force rsdt Thomas Renninger
2008-10-20  1:01 ` Matthew Garrett
2008-10-20 15:31   ` Henrique de Moraes Holschuh
2008-10-20 16:23     ` Thomas Renninger
2008-10-20 16:27       ` Matthew Garrett
2008-10-20 16:48         ` Thomas Renninger
2008-10-20 16:54           ` Matthew Garrett
2008-10-20 17:51             ` Henrique de Moraes Holschuh
2008-10-20 17:58               ` Matthew Garrett
2008-10-20 18:16                 ` Henrique de Moraes Holschuh
2008-10-20 18:24                   ` Matthew Garrett
2008-10-20 18:47                     ` Henrique de Moraes Holschuh
2008-10-20 18:52                       ` Matthew Garrett
2008-10-20 19:25                         ` Henrique de Moraes Holschuh
2008-10-21  8:14                         ` Thomas Renninger
2008-10-21  9:53                         ` Thomas Renninger
2008-10-21  9:57                 ` Thomas Renninger
2008-10-21 12:46                   ` Matthew Garrett
2008-10-21 13:05                     ` Thomas Renninger
2008-10-21 13:08                       ` Matthew Garrett
2008-10-21 13:34                         ` Thomas Renninger
2008-10-21 14:01                           ` Rafael J. Wysocki [this message]
2008-10-21 14:00                             ` Matthew Garrett
2008-10-21 14:12                               ` Thomas Renninger
2008-10-21 14:19                                 ` Matthew Garrett
2008-10-21 14:25                               ` Rafael J. Wysocki
2008-10-21 14:29                                 ` Matthew Garrett
2008-10-21 14:45                                   ` Thomas Renninger
2008-10-21 14:49                                     ` Matthew Garrett
2008-10-21 15:10                                   ` Rafael J. Wysocki
2008-10-21 15:27                                     ` Matthew Garrett
2008-10-21 15:46                                       ` Henrique de Moraes Holschuh
2008-10-21 15:50                                         ` Matthew Garrett
2008-10-21 16:58                                           ` [ltp] " Henrique de Moraes Holschuh
2008-10-21 17:02                                             ` Matthew Garrett
2008-10-21 19:04                                             ` Rafael J. Wysocki
2008-10-20 16:43   ` Thomas Renninger
2008-10-20 18:05     ` Henrique de Moraes Holschuh
2008-10-20 15:46 ` Henrique de Moraes Holschuh
2008-10-21 11:07   ` Thomas Renninger
2008-11-11  0:58 ` Matthew Garrett
2008-11-12 23:58   ` Thomas Renninger
2008-11-13  0:56     ` Matthew Garrett
2008-11-13  2:21     ` [Devel] " Zhang Rui
2008-11-13  2:24       ` Matthew Garrett
2008-11-13  8:27         ` Zhang Rui
2008-11-13 11:13           ` Matthew Garrett

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=200810211601.56527.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=devel@acpica.org \
    --cc=hmh@hmh.eng.br \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-thinkpad@linux-thinkpad.org \
    --cc=me@markdoughty.co.uk \
    --cc=mjg59@srcf.ucam.org \
    --cc=trenn@suse.de \
    --cc=yakui.zhao@intel.com \
    /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