On Sat, Nov 19, 2011 at 16:32, Keith Packard wrote:
> On Sat, 19 Nov 2011 07:25:09 -0200, Eugeni Dodonov wrote:
>
>> Just one question I caught on 2nd read. Shouldn't we have #else within
>> this #ifdef block, to return 1? Otherwise, if CONFIG_INTEL_IOMMU is
>> not
arnes
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42696
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=40564
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41353
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38862
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm
p.org/show_bug.cgi?id=41353
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38862
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c|4 +-
drivers/gpu/drm/i915/i915_drv.h|2 +-
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 29 ++
The change which added support for default settings for fbc has not
changed the variable type of i915_enable_fbc from unsigned int to int. So
we were unable to get into the default ('-1') state with the default
settings.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_dr
ndom gpu hangs and
crashes when both are enabled :(.
When the root cause will be discovered, we should allow both of course. But
so far, all the attempts on this weren't successful.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
__
is that we're not using forcewake, ie. the software isn't
> forcing the hardware to stay awake. The downside is that in doing this
> we may wait a while and that causes an unnaturally idle state on the
> GPU.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42578
Those are needed by the rc6 and semaphores support in i915 driver.
In i915 driver, we do not enable either rc6 or semaphores on SNB when dmar
is enabled. So we use those variables to check the io remapping and iommu
status.
CC: Keith Packard
CC: Daniel Vetter
Signed-off-by: Eugeni Dodonov
>
> This fixes spurious gpu hang detection when running
> tests/gem_hangcheck_forcewake on snb/ivb.
>
> Signed-Off-by: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Int
always be enabled
by the i915.i915_enable_rc6=1 kernel parameter though.
CC: Daniel Vetter
CC: Ben Widawsky
CC: Keith Packard
CC: Jesse Barnes
CC: Chris Wilson
Signed-off-by: Keith Packard
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |4 ++--
drivers/gpu/drm/i915/
The change which added support for default settings for fbc has not
changed the variable type of i915_enable_fbc from unsigned int to int. So
we were unable to get into the default ('-1') state with the default
settings.
Reviewed-by: Chris Wilson
Signed-off-by: Eugeni Dodonov
---
d
/show_bug.cgi?id=38862
Reviewed-by: Chris Wilson
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c|4 ++--
drivers/gpu/drm/i915/i915_drv.h|2 +-
drivers/gpu/drm/i915/i915_gem_execbuffer.c | 18 +-
3 files changed, 20 insertions(+), 4
Fix small typos in comments.
Signed-off-by: Eugeni Dodonov
---
drivers/platform/x86/intel_ips.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c
index 809a3ae..8dec469 100644
--- a/drivers/platform
Fix function name in comments, a left-over from when i965_reset was
renamed to i915_reset.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915
: Jesse Barnes
Acked-by: Keith Packard
Reviewed-by: Chris Wilson
Cc:
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_dma.c | 10 ++
drivers/gpu/drm/i915/i915_drv.h |1 +
2 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_dma.c b
27; command:
Machine 4: from 3.210s to 1.060s
Reviewed-by: Chris Wilson
Reviewed-by: Keith Packard
Tested-by: Sean Finney
Tested-by: Soren Hansen
Tested-by: Hernando Torque
Tested-by: Mike Lothian
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41059
Signed-off-by: Eugeni Dodonov
After checking the specs and discussing with Jesse, turns out CxSR is not
available on Ironlake and gen5, and its advertisement on the device
description is misleading.
Acked-by: Jesse Barnes
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |2 +-
1 files changed, 1
, there is no
reason to not have them on by default there.
Eugeni Dodonov (8):
drm/i915: there is no pipe CxSR on ironlake
drm: give up on edid retries when i2c bus is not responding
drm/i915: prevent division by zero when asking for chipset power
drm/i915: fix typo in function name
platform
always be enabled
by the i915.i915_enable_rc6=1 kernel parameter though.
CC: Daniel Vetter
CC: Ben Widawsky
CC: Keith Packard
CC: Jesse Barnes
CC: Chris Wilson
Signed-off-by: Keith Packard
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |4 ++--
drivers/gpu/drm/i915/
This should allow even more energy saving when GPU is not in use.
According to the testing, this state results in around 0.1 - 0.4 W better
power usage.
No issues or regressions observed so far, but additional testing is
certainly welcome.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915
, to allow the module to know the latest status of rc6
requested by user.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_debugfs.c | 101 ++
drivers/gpu/drm/i915/intel_display.c |2 +-
drivers/gpu/drm/i915/intel_drv.h |3 +
3 files changed
On Tue, Nov 29, 2011 at 12:12, Daniel Vetter wrote:
> On Tue, Nov 29, 2011 at 01:05:03PM +, Chris Wilson wrote:
> > On Tue, 29 Nov 2011 10:55:05 -0200, Eugeni Dodonov <
> eugeni.dodo...@intel.com> wrote:
> > > This allows to enable/disable rps and rc6 from userspa
lable_schedulers
none *fair batch
or list them in the output of i915_scheduler, like:
# cat /sys/kernel/debug/dri/0/i915_scheduler
...
Scheduler: batch
Available schedulers: none fair batch
To see what schedulers are available for usage.
But besides this, fo
t;
> Signed-off-by: Chris Wilson
> Cc: sta...@kernel.org
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Mon, Nov 28, 2011 at 19:12, Daniel Vetter wrote:
> Name the function accordingly. Suggested by Chris Wilson.
>
> Signed-Off-by: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
Such patches are so hard to review. The only ones harder than those are the
white space-related ones :).
t accessed with _NOTRACE.
>
s/FROCEWAKE/FORCEWAKE/g :)
Besides this,
Reviewed-by: Eugeni Dodonov
for the series.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.or
angs.
> >
> > Signed-Off-by: Daniel Vetter
> Reviewed-and-tested-by: Chris Wilson
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
otentially happen anyway as a result of a pathological error.
>
> Signed-off-by: Chris Wilson
> Cc: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists
On Tue, Nov 29, 2011 at 19:42, Ben Widawsky wrote:
> On Tue, Nov 29, 2011 at 01:01:31PM +, Chris Wilson wrote:
> > On Tue, 29 Nov 2011 10:55:04 -0200, Eugeni Dodonov <
> eugeni.dodo...@intel.com> wrote:
> > > This should allow even more energy saving when GPU is
>
> Comments, test reports, reviews and flames highly welcome.
>
For the series (with the latest patches on your fd.o repo)
Tested-by: Eugeni Dodonov
I left it running on 2 SNBs and 1 IVB for couple of hours under different
set of workloads, and haven't seen any issues so far
On Mon, Nov 28, 2011 at 18:35, Daniel Vetter wrote:
> +#define ERROR 0x40a0
>
You do not seem to use this, and it is the same value as ERROR_GEN6. Do we
need it? Or I misread something?
--
Eugeni Dodonov
<http://eugeni.do
el Vetter
>
Yep, I think that error_state does the job just fine now.
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
hanks,
Eugeni Dodonov
Intel Open Source Technology Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
2011/12/6 Alan W. Irwin
> On 2011-12-05 23:03-0200 Eugeni Dodonov wrote:
>
> Hi,
>>
>> We’d like to announce Intel 2011Q4 graphics package, focused on
>> performance and stability improvements in the Intel Linux Graphics
>> stack.
>>
>> Please che
ktop.org/show_bug.cgi?id=39730#c48 for this problem?
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
: https://bugs.freedesktop.org/show_bug.cgi?id=41353
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38862
Reviewed-by: Chris Wilson
Reviewed-by: Daniel Vetter
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c|4 ++--
drivers/gpu/drm/i915/i915_drv.h
LLC is not SNB-specific, so we should check for it in a more generic way.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |2 ++
drivers/gpu/drm/i915/i915_drv.h |2 ++
drivers/gpu/drm/i915/i915_gem.c |4 ++--
3 files changed, 6 insertions(+), 2 deletions(-)
diff
From: Eugeni Dodonov
LLC is not SNB-specific, so we should check for it in a more generic way.
v2: export LLC support status via debugfs and DRM GETPARAM.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_debugfs.c |1 +
drivers/gpu/drm/i915/i915_dma.c |3 +++
drivers
From: Eugeni Dodonov
LLC is not SNB-specific, so we should check for it in a more generic way.
v2: export LLC support status via debugfs and DRM GETPARAM.
v3: rebase on newer kernel version which says that IVB supports LLC as
well.
Reviewed-by: Chris Wilson
Reviewed-by: Daniel Vetter
Signed
With base on comments and feedback on LLC handling from Daniel Vetter, Chris
Wilson and Ben Widawsky, I reworked my patch into a more useful approach for
detecting presence of different cache levels from userspace.
So, in this context, PATCH 1 adds the very same .has_llc flag and debugfs
entry for
From: Eugeni Dodonov
LLC is not SNB-specific, so we should check for it in a more generic way.
v2: export LLC support status via debugfs.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_debugfs.c |1 +
drivers/gpu/drm/i915/i915_drv.c |4
drivers/gpu/drm/i915
From: Eugeni Dodonov
Simplify the code and cache-related parameters handling, and prepare to
export them to userspace in subsequent patches.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.h | 10 ++
drivers/gpu/drm/i915/i915_gem.c |2 +-
drivers/gpu/drm
From: Eugeni Dodonov
This allows to query available cache levels from libdrm and check for
presence of LLC from userspace.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_dma.c |6 ++
include/drm/i915_drm.h |1 +
2 files changed, 7 insertions(+), 0 deletions
From: Eugeni Dodonov
Allow userspace to discover current cache level for a bo and set a
specific cache level if necessary.
The patch is mostly based on the original patch from Ben Widawsky.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_dma.c |2 +
drivers/gpu/drm/i915
understanding what is going on with your machine,
could you setup some thermal sensors and check what do they say about
temperature? And also check with powertop about what is using the most
power out there?
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_
On Wed, Dec 14, 2011 at 10:57, Daniel Vetter wrote:
> Only forcewake has an open with special semantics, the other r/w
> debugfs only assign the file private pointer.
>
> Signed-Off-by: Daniel Vetter
> Reviewed-by: Ben Widawsky
> Reviewed-by: Kenneth Graunke
>
Review
the error_state to protect from concurent freeeing.
> This will be only really used in the next patch.
>
> Signed-Off-by: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing
prevent
> resume bugs from totally hanging the machine.
>
> Signed-off-by: Daniel Vetter
> Reviewed-by: Chris Wilson
> Acked-by: Ben Widawsky
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
around because I'll reuse ring_str in the next
> patch.
>
> Signed-off-by: Daniel Vetter
> Reviewed-by: Chris Wilson
> Reviewed-by: Ben Widawsky
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_
on.
>
> Signed-off-by: Daniel Vetter
> Reviewed-by: Chris Wilson
> Signed-off-by: Ben Widawsky
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedeskto
te where just guessing these is not good enough.
>
> v2: Also dump INSTPM for every ring.
>
> v3: Fix a few really silly goof-ups spotted by Chris Wilson.
>
> Signed-off-by: Daniel Vetter
> Reviewed-by: Chris Wilson
> Signed-off-by: Ben Wida
eth Graunke
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Wed, Dec 14, 2011 at 10:57, Daniel Vetter wrote:
> This way we can simulate a bunch of gpu hangs and run the error_state
> capture code every time (without the need to reload the module).
>
> Signed-Off-by: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
--
Eugen
lazy GTT binding,
> but could potentially happen anyway as a result of a pathological error.
>
> Signed-off-by: Chris Wilson
> Reviewed-by: Daniel Vetter
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_
e semantic patch that makes this change is available
> in https://lkml.org/lkml/2011/11/25/107
>
> Signed-off-by: Thomas Meyer
> Signed-Off-by: Daniel Vetter
>
It looks nicer indeed, so:
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_
On Wed, Dec 14, 2011 at 10:57, Daniel Vetter wrote:
> Signed-off-by: Daniel Vetter
> Reviewed-by: Ben Widawsky
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.
y
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
915_ring_stop automatically
> resets after a gpu hang is detected to avoid hanging the gpu to fast
> and declaring it wedged.
>
> v2: Incorporate feedback from Chris Wilson.
>
> v3: Add the missing cleanup.
>
> Signed-Off-by: Da
On Wed, Dec 14, 2011 at 16:58, Eugeni Dodonov wrote:
> On Wed, Dec 14, 2011 at 10:57, Daniel Vetter wrote:
>
>> Signed-off-by: Daniel Vetter
>> Reviewed-by: Ben Widawsky
>>
>
> Reviewed-by: Eugeni Dodonov
>
Just clarifying - the R-b is for the ppgtt series
On Tue, Dec 13, 2011 at 01:21, Ben Widawsky wrote:
> This matches the modern specs more accurately.
>
> This will be used by the following patch to fix the way we display RC
> status.
>
> Signed-off-by: Ben Widawsky
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<h
naturally idle state on the
> > GPU.
> >
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42578
> > Signed-off-by: Ben Widawsky
> > ---
>
> Aside from the merge error:
>
> Reviewed-by: Jesse Barnes
>
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<
on IVB at least. Anyone else wanna give it a try and confirm?
> I've only tested with Eric's blit-and-wait.c test so far.
>
I am still hitting missed IRQs with gem_dummy_reloc_loop even with this one
on IVB mobile :(.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
On Thu, Dec 1, 2011 at 08:57, Yi Sun wrote:
> If test all modes rather than preferred mode, remove framebuffer and set
> CRTC to zero after each connector mode setting.
>
> Signed-off-by: Sun Yi
>
I forgot to comment on this before, but better late than never.
Reviewed-by:
and has been sitting on my todo list for way to long).
>
> Very-Much-Acked-by: Daniel Vetter
>
Yeah, I agree with Daniel - I'll be very happy to have this in libdrm.
Thanks a lot!
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
From: Eugeni Dodonov
Per report from Udo Steinberg, enablement of semaphores causes issues on
his Sandy Bridge. So disabling them again for this platform.
Note that they are still enabled by default for Ivy Bridge.
Reported-by: Udo Steinberg
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm
On Dec 27, 2011 6:41 PM, "Eric Anholt" wrote:
>
> On Fri, 23 Dec 2011 12:11:10 -0800, Chad Versace <
chad.vers...@linux.intel.com> wrote:
> > The function uses libpciaccess, which is not present on Android. So,
make
> > it a no-op that returns 0.
>
> It's a bug that we're not using this function
d check what
happens during the hangs? The last dying words of the machine could be very
helpful.
Thanks!
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
lue = IS_GEN7(dev);
as it is gen7+-specific item. This way, userspace could check for this
support early, and avoid setting the flag on the batchbuffer in vain on
pre-gen7 architectures.
Either way, it will work, so:
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/&g
ould enable polling, and '0' would use the usual
gen6_ring_get_irq. This would default to '1' on gen7, and '0' on older
architectures.
Other than that,
Tested-by: Eugeni Dodonov
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
d with Eric's glCopyPixels loop which without this patch scores a
> missed irq every few seconds.
>
> This patch needs my forcewake rework to use a spinlock instead of
> dev->struct_mutex.
>
> Cc: sta...@kernel.org
> Cc: Eric Anholt
> Cc: Kenneth Graunke
> Cc: Eug
ms to work even without HWSTAM, I tried this too with base on
Ben's HWSTAM disabling patches.
So I think we have a winner :).
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
27; command:
Machine 4: from 3.210s to 1.060s
Reviewed-by: Chris Wilson
Reviewed-by: Keith Packard
Tested-by: Sean Finney
Tested-by: Soren Hansen
Tested-by: Hernando Torque
Tested-by: Mike Lothian
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41059
Signed-off-by: Eugeni Dodonov
After checking the specs and discussing with Jesse, turns out CxSR is not
available on Ironlake and gen5, and its advertisement on the device
description is misleading.
Acked-by: Jesse Barnes
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_drv.c |2 +-
1 files changed, 1
On Thu, Jan 5, 2012 at 12:43, Daniel Vetter wrote:
> On Thu, Jan 05, 2012 at 09:34:28AM -0200, Eugeni Dodonov wrote:
> > This allows to avoid talking to a non-responding bus repeatedly until we
> > finally timeout after 15 attempts. We can do this by catching the -ENXIO
> &g
/bugs.freedesktop.org/show_bug.cgi?id=41059 apparently) shortly.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
y ducttape!
Reviewed-by: Eugeni Dodonov
Tested-by: Eugeni Dodonov
I wonder if we could drop the definitions of GEN6_*_HWSTAM from i915_reg.h
as well now...
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx
the GPU goes idle before the counters are reset and
> sampling ceases.
>
> Sample output for running aa10text under gnome-shell:
>
> Total: 55.29 57.54 61.14
> render ring: 54.80 57.17 60.76
> gen6 bsd ring: 0.00 0.00 0.00
> blt ring: 0.48 0.37 0.37
>
> Signed-off-by
This is also handled by i915_reg.h, so just reuse this trick to reduce
universe enthropy.
Signed-off-by: Eugeni Dodonov
Reviewed-by: Jesse Barnes
---
drivers/gpu/drm/i915/i915_suspend.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i915
We don't need to check 3rd pipe specifically, as it shares PLL with some
other one.
Signed-off-by: Eugeni Dodonov
Reviewed-by: Jesse Barnes
---
drivers/gpu/drm/i915/i915_suspend.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_suspend
nge_avi_infoframe and parse_infoframe_option_s - you could
just pass all the values as parameters instead.
Other than that,
Reviewed-by: Eugeni Dodonov
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://li
r statistics ...
>
Or we could start the worker when someone 'echo 1 > load_avg'. When it is
not started, cat load_avg would say:
"load_avg counter is stopped, echo 1 to enable it"
for example.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
__
e-i915-for-stable-kernels/.
Cheers,
Eugeni
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
I wonder if it's an eDP config; my
> t420 here works ok with current bits, but it uses LVDS.
>
If you are using eDP, you might be facing
https://bugs.freedesktop.org/show_bug.cgi?id=42263 or
https://bugs.freedesktop.org/show_bug.cgi?id=42278.
Could you try with patch mentioned at
http
On Jan 12, 2012 8:45 PM, "Jesse Barnes" wrote:
>
> On Thu, 12 Jan 2012 23:42:35 +0100
> Vincent Vanackere wrote:
>
> > On Thu, Jan 12, 2012 at 23:24, Jesse Barnes wrote:
> >
> > > On Thu, 12 Jan 2012 23:21:10 +0100
> > > Vincent Vanackere wrote:
> > > > I'm happy to report that this patch indeed
familiar with this area, but both patches look correct to me
and are according to the specs as far as I can see.
So, for both of them:
Reviewed-by: Eugeni Dodonov
Thanks!
P.S.: it would be interested to gather some Tested-by's for this though..
--
Eugeni Dodonov
<http://eugeni.dodono
eaks, we'll be able to notice it pretty shortly and fix it.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
you running stable kernels
who want to test the latest breakages^w features :).
[1] http://comments.gmane.org/gmane.comp.freedesktop.xorg.drivers.intel/7961
--
Eugeni Dodonov
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Mon, Jan 16, 2012 at 17:50, Jesse Barnes wrote:
> Do we get the ILK+ side right here?
>
>From the specs, looks like ILK and SNB+ share same logic for this part.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing
n power saving figures for snb?
>
The FBC is good in some cases and bad in others. Let me confirm with our
windows drivers friends first, but I'd say that we could disable it as it
brings no significant improvements..
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
;
> >> /me would like to kill this cruft
>
> All sounds good to me, kill it with fire. I've never known this code
> to be used in anything.
>
I think we can kill it, it does not seems to be used by anything.
Just to be sure - Eric, you was the last one to touch it
LLC is not SNB/IVB-specific, so we should check for it in a more generic
way.
Reviewed-by: Chris Wilson
Reviewed-by: Daniel Vetter
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/i915_debugfs.c |1 +
drivers/gpu/drm/i915/i915_dma.c |3 +++
drivers/gpu/drm/i915/i915_drv.c
This adds support for querying the kernel about the LLC support in the
hardware.
In case the ioctl fails, we assume that it is present on GEN6 and GEN7.
Signed-off-by: Eugeni Dodonov
---
include/drm/i915_drm.h |1 +
intel/intel_bufmgr_gem.c | 12
2 files changed, 13
This adds support for querying the kernel about the LLC support in the
hardware.
In case the ioctl fails, we assume that it is present on GEN6 and GEN7.
Signed-off-by: Eugeni Dodonov
---
include/drm/i915_drm.h |1 +
intel/intel_bufmgr_gem.c | 12
2 files changed, 13
On Tue, Jan 17, 2012 at 14:44, Eugeni Dodonov wrote:
> This adds support for querying the kernel about the LLC support in the
> hardware.
>
> In case the ioctl fails, we assume that it is present on GEN6 and GEN7.
>
> Signed-off-by: Eugeni Dodonov
>
Please ignore this and
Otherwise, we are left with pretty bogus message saying that the pixel
format is not supported while leaving the details to the telepatic powers.
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/intel_display.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a
This adds support for querying the kernel about the LLC support in the
hardware.
In case the ioctl fails, we assume that it is present on GEN6 and GEN7.
v2: fix the return code checking
Signed-off-by: Eugeni Dodonov
---
include/drm/i915_drm.h |1 +
intel/intel_bufmgr_gem.c | 12
Otherwise, we are left with pretty bogus message saying that the pixel
format is not supported while leaving the details to the telepatic powers.
v2: use DRM_DEBUG_KMS instead of DRM_ERROR
Signed-off-by: Eugeni Dodonov
---
drivers/gpu/drm/i915/intel_display.c |3 ++-
1 files changed, 2
Instead of checking for CPU generation, use the libdrm-provided
I915_PARAM_HAS_LLC instead.
Signed-off-by: Eugeni Dodonov
---
src/sna/kgem.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/sna/kgem.c b/src/sna/kgem.c
index 970e462..96c5ddd 100644
--- a/src/sna
This is requried for I915_PARAM_HAS_LLC support.
Signed-off-by: Eugeni Dodonov
---
configure.ac |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure.ac b/configure.ac
index 63beb64..604ea6d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -132,6 +132,7
Instead of checking for CPU generation, use the libdrm-provided
I915_PARAM_HAS_LLC instead.
v2: use a define check to verify if we have I915_PARAM_HAS_LLC.
Signed-off-by: Eugeni Dodonov
---
src/sna/kgem.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/src
org/gmane.comp.freedesktop.xorg.drivers.intel/8006
Don't know if they will fix the problem, but they could help perhaps.
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
401 - 500 of 636 matches
Mail list logo
