From: Tobias Klausmann <tobias.johannes.klausmann@mni.thm.de>
To: Dave Airlie <airlied@gmail.com>
Cc: Ilia Mirkin <imirkin@alum.mit.edu>,
nouveau <nouveau@lists.freedesktop.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [Nouveau] 3.12rc1-pre Nouveau? oops
Date: Mon, 09 Sep 2013 01:02:36 +0200 [thread overview]
Message-ID: <522D020C.6070402@mni.thm.de> (raw)
In-Reply-To: <CAPM=9twkyQGbnPEerqCpJNfXxSRo1pw7MBvGMuz8zs1pQ+Fcng@mail.gmail.com>
On 09.09.2013 00:29, Dave Airlie wrote:
> On Mon, Sep 9, 2013 at 8:01 AM, Tobias Klausmann
> <tobias.johannes.klausmann@mni.thm.de> wrote:
>> On 08.09.2013 23:33, Dave Airlie wrote:
>>>>> Looks like you have Optimus (intel + nvidia), and the backtrace has
>>>>> runtime pm in it, which is something new Dave added for 3.12, adding
>>>>> him in explicitly. The simplest explanation is that disp->init is
>>>>> NULL. And it seems like there are no outputs from the earlier nouveau
>>>>> init prints. I guess that the call to nouveau_display_resume from
>>>>> nouveau_pmops_runtime_resume should be guarded by a if
>>>>> (dev->mode_config.num_crtc) like it is everywhere else.
>>>>>
>>>>> -ilia
>>>> Your guess was right, this (hopefully attached patch) fixes it for me!
>>> Does it look like this one?
>>>
>>> Dave.
>> No, mine was quick and dirty, reverted it and took yours. But i'm a little
>> bit confused that this is a suspend/resume problem, i booted the kernel for
>> the first time while seeing the oops. But anyway i tested it and it works.
> It's runtime suspend/resume - so it turns the nvidia gpu off at boot since it
> isn't being used.
>
> you should see longer battery life.
> Dave.
Ah thanks for the explanation.
Can we see the state in /sys somewhere? I looked around but did not find
something to determine the state?
/sys/bus/pci/drivers/nouveau/0000:01:00.0/drm/card0/power/runtime_status
gives me "unsupported".
But i suspect thats because of nouveaus lack to properly reclock my
nvidia gpu. Anyway i'm better of asking you for the right anyswer.
Thanks for your time,
Tobias
prev parent reply other threads:[~2013-09-08 23:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-08 16:53 3.12rc1-pre Nouveau? oops Tobias Klausmann
2013-09-08 19:39 ` Ilia Mirkin
2013-09-08 21:21 ` Tobias Klausmann
2013-09-08 21:21 ` [PATCH] Bail in nouveau_display_resume() if there are no output available! Tobias Klausmann
2013-09-08 21:33 ` [Nouveau] 3.12rc1-pre Nouveau? oops Dave Airlie
2013-09-08 22:01 ` Tobias Klausmann
2013-09-08 22:29 ` Dave Airlie
2013-09-08 23:02 ` Tobias Klausmann [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=522D020C.6070402@mni.thm.de \
--to=tobias.johannes.klausmann@mni.thm.de \
--cc=airlied@gmail.com \
--cc=imirkin@alum.mit.edu \
--cc=linux-kernel@vger.kernel.org \
--cc=nouveau@lists.freedesktop.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