From: Jesse Barnes <jbarnes@virtuousgeek.org>
To: Eugeni Dodonov <eugeni@dodonov.net>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>,
intel-gfx@lists.freedesktop.org,
Eugeni Dodonov <eugeni.dodonov@intel.com>
Subject: Re: [PATCH 1/2] drm/i915: re-enable semaphores by default
Date: Wed, 16 Nov 2011 11:23:50 -0800 [thread overview]
Message-ID: <20111116112350.202a34bb@jbarnes-desktop> (raw)
In-Reply-To: <CAC7LmntYrPuf0o8GdakHYKRrTb7bJq58ws+Bk3f+YPOOdH2e+g@mail.gmail.com>
On Wed, 16 Nov 2011 17:05:37 -0200
Eugeni Dodonov <eugeni@dodonov.net> wrote:
> On Wed, Nov 16, 2011 at 15:24, Andrew Lutomirski <luto@mit.edu> wrote:
>
> > AFAICT my snb laptop has always been stable with semaphores and VT-d
> > enabled. Is this problem possibly restricted to just desktop
> > machines? I'm happy to test, since my box that can reproduce the hang
> > instantly is still around.
> >
>
> I cannot reproduce those issues on any mobile SNB, but I am out of desktop
> ones. So perhaps it is a clue.
>
> Jesse, Keith, Daniel, what if we add something like:
>
> ...
> .has_semaphores=1, .has_rc6=1
> ...
>
> to i915_drv.c intel_sandybridge_m_info, and take them into account when
> using default values for those settings?
Something like that is fine with me, or just doing it in modeset_init
for certain chipsets.
--
Jesse Barnes, Intel Open Source Technology Center
next prev parent reply other threads:[~2011-11-16 19:23 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-14 20:39 [PATCH 1/2] drm/i915: re-enable semaphores by default Eugeni Dodonov
2011-11-14 20:39 ` [PATCH 2/2] drm/i915: re-enable rc6 " Eugeni Dodonov
2011-11-14 21:06 ` Lukas Hejtmanek
2011-11-14 23:04 ` Eugeni Dodonov
2011-11-14 23:12 ` Lukas Hejtmanek
2011-11-14 23:17 ` Eugeni Dodonov
2011-11-14 23:23 ` Lukas Hejtmanek
2011-11-14 21:22 ` Jesse Barnes
2011-11-16 15:53 ` Daniel Vetter
[not found] ` <CAObL_7HxAt7B7v+CqgTff+ox5eg-=qLkd8jpZGsGAFFHLx73hw@mail.gmail.com>
2011-11-16 20:22 ` Daniel Vetter
2011-11-14 21:23 ` [PATCH 1/2] drm/i915: re-enable semaphores " Jesse Barnes
2011-11-16 15:49 ` Daniel Vetter
2011-11-16 17:16 ` Keith Packard
2011-11-16 17:24 ` Andrew Lutomirski
2011-11-16 19:05 ` Eugeni Dodonov
2011-11-16 19:23 ` Jesse Barnes [this message]
2011-11-16 20:18 ` Daniel Vetter
2011-11-16 21:37 ` Keith Packard
-- strict thread matches above, loose matches on Subject: below --
2011-11-16 20:56 Nicolas Kalkhof
2011-11-16 21:14 ` Daniel Vetter
2011-11-16 22:15 ` Keith Packard
2011-11-17 8:57 Nicolas Kalkhof
2011-11-17 9:22 ` Daniel Vetter
2011-11-22 5:41 ` Andrew Lutomirski
2011-11-22 11:23 ` Daniel Vetter
2011-11-29 16:39 ` Daniel Vetter
2011-11-17 18:31 Nicolas Kalkhof
2011-11-18 15:26 ` Daniel Vetter
2011-11-18 18:36 Nicolas Kalkhof
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=20111116112350.202a34bb@jbarnes-desktop \
--to=jbarnes@virtuousgeek.org \
--cc=daniel.vetter@ffwll.ch \
--cc=eugeni.dodonov@intel.com \
--cc=eugeni@dodonov.net \
--cc=intel-gfx@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