linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steven Newbury <steve@snewbury.org.uk>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Matthew Garrett <matthew.garrett@nebula.com>,
	James Hogan <james@albanarts.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Kamal Mostafa <kamal@canonical.com>,
	"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>
Subject: Re: Linux 3.11-rc2
Date: Tue, 23 Jul 2013 23:49:29 +0100	[thread overview]
Message-ID: <1374619769.316.1.camel@artifact> (raw)
In-Reply-To: <2081090.JJVTShTh7K@vostro.rjw.lan>

On Tue, 2013-07-23 at 23:24 +0200, Rafael J. Wysocki wrote:
> On Tuesday, July 23, 2013 06:47:55 PM Steven Newbury wrote:
> > On Mon, 2013-07-22 at 21:56 +0200, Rafael J. Wysocki wrote:
> > > On Monday, July 22, 2013 06:16:20 PM Matthew Garrett wrote:
> > > > On Mon, 2013-07-22 at 15:02 +0200, Rafael J. Wysocki wrote:
> > > > 
> > > > > In the meantime I received a report from Steven Newbury that these changes
> > > > > broke things for him too, so we need to revert commits 8c5bd7a and efaa14c.
> > > > > The other two commits in the series should be benign.
> > > > 
> > > > Could you let me know the details of this problem?
> > > 
> > > Steven, can you please describe the problem you're seeing to Matthew and
> > > the other people on the list?
> > > 
> > > Rafael
> > > 
> > 
> > Before the changes backlight was working fine using the ACPI method: 
> > /sys/class/backlight/acpi_video0/ is present and the keyboard function keys
> > control brightness with notifications working in GNOME.
> > 
> > In the code as was present in the linux-pm/bleeding-edge tree I would
> > encounter a hard lockup on keyboard brightness trigger.  This also occurred with
> > the code as it initially hit mainline, but a later commit fixed the crash*, but
> > resulted in no backlight controls being available at all.
> > /sys/class/backlight  is empty.
> > 
> > *not actually sure if /sys/class/backlight contained anything before this
> 
> Hmm.  Which commit fixed the crash for you?
> 
> Rafael
> 

I'll see if I can build a broken kernel tomorrow, after backing up! ;-)

      reply	other threads:[~2013-07-23 22:49 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CA+55aFy330bSRjK5HT1n0wXioPykL3RquHXyBPjm5GpTz_Y=DQ@mail.gmail.com>
     [not found] ` <51EC4D77.2010502@mni.thm.de>
2013-07-22 13:08   ` Linux 3.11-rc2 Rafael J. Wysocki
2013-07-22 15:43     ` Tobias Klausmann
     [not found] ` <5728692.GlhGgHY92H@vostro.rjw.lan>
     [not found]   ` <CA+55aFyrMRDD8qe9DYXixaSLJXEURMYYku1azABA72m71_X+cA@mail.gmail.com>
2013-07-22 19:54     ` Rafael J. Wysocki
2013-07-23 18:46       ` Linux 3.11-rc2 (acpi backlight) Kamal Mostafa
2013-07-24  0:05         ` Rafael J. Wysocki
2013-07-24  4:54           ` Steven Newbury
2013-07-24  6:49           ` James Hogan
2013-07-24  7:14             ` Mike Galbraith
2013-07-24  7:19             ` Igor Gnatenko
2013-07-24 12:06               ` Rafael J. Wysocki
2013-07-24 11:45           ` Jörg Otte
2013-07-25 13:00       ` Linux 3.11-rc2 (acpi backlight, revert) Rafael J. Wysocki
2013-07-25 14:13         ` Aaron Lu
2013-07-25 14:43         ` Kamal Mostafa
2013-07-25 14:46           ` Daniel Vetter
2013-07-25 14:59             ` Kamal Mostafa
2013-07-25 14:52         ` Jörg Otte
2013-07-25 15:52           ` Jörg Otte
2013-07-25 19:14         ` James Hogan
2013-07-25 19:47           ` Rafael J. Wysocki
2013-07-26  4:23             ` Joerg Platte
2013-07-26 11:22             ` Igor Gnatenko
2013-07-26  7:43         ` Steven Newbury
2013-07-26 12:09         ` Martin Steigerwald
2013-07-26 12:40           ` Rafael J. Wysocki
2013-08-04 19:33             ` Martin Steigerwald
2013-08-04 22:15               ` Rafael J. Wysocki
2013-07-27  5:34         ` Kalle Valo
2013-07-27 12:18           ` Rafael J. Wysocki
     [not found]   ` <1374516979.2641.0.camel@x230>
2013-07-22 19:56     ` Linux 3.11-rc2 Rafael J. Wysocki
2013-07-23 17:47       ` Steven Newbury
2013-07-23 19:51         ` Matthew Garrett
2013-07-23 21:10           ` Steven Newbury
2013-07-23 21:24         ` Rafael J. Wysocki
2013-07-23 22:49           ` Steven Newbury [this message]

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=1374619769.316.1.camel@artifact \
    --to=steve@snewbury.org.uk \
    --cc=james@albanarts.com \
    --cc=kamal@canonical.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matthew.garrett@nebula.com \
    --cc=rafael.j.wysocki@intel.com \
    --cc=rjw@sisk.pl \
    --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).