[Intel-gfx] [PATCH 5/6] drm/i915: Handle framebuffer offsets[]

2012-05-24 Thread Ville Syrjälä
On Thu, May 24, 2012 at 11:31:32AM -0700, Jesse Barnes wrote: > On Thu, 24 May 2012 21:08:58 +0300 > ville.syrjala at linux.intel.com wrote: > > > From: Ville Syrj?l? > > > > Take fb->offset[0] into account when calculating the linear and tile x/y > > offsets. > > > > For non-tiled surfaces fb-

[PATCH 2/3] drm: Be more paranoid with integer overflows

2012-05-24 Thread Ville Syrjälä
On Thu, May 24, 2012 at 08:30:23PM +0200, Daniel Vetter wrote: > On Thu, May 24, 2012 at 08:53:59PM +0300, ville.syrjala at linux.intel.com > wrote: > > From: Ville Syrj?l? > > > > Make sure 'width * cpp' and 'height * pitch + offset' don't exceed > > UINT_MAX. > > > > Signed-off-by: Ville Syrj

[Intel-gfx] [PATCH 5/6] drm/i915: Handle framebuffer offsets[]

2012-05-24 Thread Daniel Vetter
On Thu, May 24, 2012 at 09:49:15PM +0300, Ville Syrj?l? wrote: > On Thu, May 24, 2012 at 11:31:32AM -0700, Jesse Barnes wrote: > > On Thu, 24 May 2012 21:08:58 +0300 > > ville.syrjala at linux.intel.com wrote: > > > > > From: Ville Syrj?l? > > > > > > Take fb->offset[0] into account when calcula

[PATCH 2/3] drm: Be more paranoid with integer overflows

2012-05-24 Thread Daniel Vetter
On Thu, May 24, 2012 at 09:54:37PM +0300, Ville Syrj?l? wrote: > On Thu, May 24, 2012 at 08:30:23PM +0200, Daniel Vetter wrote: > > On Thu, May 24, 2012 at 08:53:59PM +0300, ville.syrjala at linux.intel.com > > wrote: > > > From: Ville Syrj?l? > > > > > > Make sure 'width * cpp' and 'height * pi

[git pull] drm merge window pull

2012-05-24 Thread Dave Airlie
Hi Linus, this is the main merge window request for the drm, Its big, but jam packed will lots of features and of course 0 regressions. (okay maybe there'll be one). Highlights: new KMS drivers for server GPU chipsets: ast, mgag200 and cirrus (qemu only). These drivers use the generic modeset

[Bug 43207] radeon driver on HD6570 shows pixel noise

2012-05-24 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=43207 --- Comment #8 from J?r?me Glisse 2012-05-24 19:13:23 --- Created an attachment (id=73382) --> (https://bugzilla.kernel.org/attachment.cgi?id=73382) Fix backend map This patch should fix your issue -- Configure bugmail: https://bugzilla.

[PATCH 0/4] drm/i915: Make video sprites survive a modeset

2012-05-24 Thread Ville Syrjälä
On Thu, May 24, 2012 at 08:49:53PM +0200, Daniel Vetter wrote: > On Thu, May 24, 2012 at 11:35:35AM -0700, Jesse Barnes wrote: > > On Thu, 24 May 2012 21:29:46 +0300 > > ville.syrjala at linux.intel.com wrote: > > > > > Currently the video sprites appear to get disabled on modeset more by > > > ac

[RFC][PATCH 0/6] WIP: drm: Atomic mode setting idea

2012-05-24 Thread ville.syrj...@linux.intel.com
This is some very early demo code for the atomic modesetting feature. According to my current plan there would be just one ioctl. You simply feed it an arbitrary list of object properties and the implementation will decice how it can apply those (for example, whether it can complete the operation

[RFC][PATCH 1/6] drm: Export drm_property_create_blob() and drm_property_destroy_blob()

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? Signed-off-by: Ville Syrj?l? --- drivers/gpu/drm/drm_crtc.c |8 +--- include/drm/drm_crtc.h |4 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index 6dafb99..ce0f446 100644 --- a/dri

[RFC][PATCH 2/6] drm: Allow signed values for range properties

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? Treat a range property as signed when the unsigned minimum value is larger than the unsigned maximum value. Signed-off-by: Ville Syrj?l? --- drivers/gpu/drm/drm_crtc.c | 17 ++--- 1 files changed, 14 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/

[RFC][PATCH 4/6] drm: Refactor object property check code

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? Refactor the code to check whether an object has a specific property to a new function. Signed-off-by: Ville Syrj?l? --- drivers/gpu/drm/drm_crtc.c | 20 ++-- 1 files changed, 14 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/driv

[RFC][PATCH 3/6] drm: Allow drm_mode_object_find() to look up an object of any type

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? To avoid having to pass object types from userspace for atomic mode setting ioctl, allow drm_mode_object_find() to look up an object of any type. This will only work as long as the all object types share the ID space. Signed-off-by: Ville Syrj?l? --- drivers/gpu/drm/drm_crt

[RFC][PATCH 5/6] WIP: drm: Atomic modeset ioctl

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? First draft. The ioctl simply takes a list of object IDs and property IDs and their values. For setting values to blob properties, the property value indicates the length of the data, and the actual data is passed via another blob pointer. Detailed error reporting is missing

[RFC][PATCH 6/6] WIP drm/i915: "Atomic" modeset test implementation

2012-05-24 Thread ville.syrj...@linux.intel.com
From: Ville Syrj?l? Just a quick prototype to test the atomic modeset API. Implemented via already existing non-atomic mechanisms internally. --- drivers/gpu/drm/i915/Makefile|1 + drivers/gpu/drm/i915/intel_atomic.c | 788 ++ drivers/gpu/drm/i915/in

[Bug 50230] offset mapping in nexuiz results in bad texturing with the llvm compiler

2012-05-24 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50230 --- Comment #10 from almos 2012-05-24 13:25:32 PDT --- (In reply to comment #9) > Created attachment 62057 [details] [review] > [PATCH] radeon/llvm: fix sampler index in llvm_emit_tex > > Does this patch help? Yes, the patch fixes it. I also tri

[Bug 50232] screen redraw is wrong in sauerbraten with the llvm compiler

2012-05-24 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50232 --- Comment #2 from almos 2012-05-24 13:25:38 PDT --- (In reply to comment #1) > Please test with the first patch from bug 50230. The revert patch fixes the ghost image problem. The warning is still printed, but the visuals seem to be good now.

[RFC] [PATCH 00/14] HPD/connector-polling rework

2012-05-24 Thread Daniel Vetter
Hi all, I've got fed up with our sorry state of connector detection and rampant edid re and rere-reading. This patch series lays the groundwork in the drm helpers so that drivers can avoid all this madness (at least on working hw) and properly cache the edid. With the additional changes for drm/

[PATCH 01/14] drm: extract drm_kms_helper_hotplug_event

2012-05-24 Thread Daniel Vetter
Useful if drivers want to be slightly more clever about hotplug handling. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc_helper.c | 17 +++-- include/drm/drm_crtc_helper.h |1 + 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_c

[PATCH 02/14] drm: handle HDP and polled connectors separately

2012-05-24 Thread Daniel Vetter
Instead of reusing the polling code for hdp handling, split them up. This has a few consequences: - Don't touch HDP capable connectors in the poll loop. - Only touch HDP capable connectors in drm_helper_hpd_irq_event. - Run the HDP handling directly instead of going through a work item - all call

[PATCH 03/14] drm: introduce DRM_CONNECTOR_POLL_FORCE

2012-05-24 Thread Daniel Vetter
Useful for ->detect functions that have different behaviour if force is set. This way probe_single_connector can avoid to do the expensive edid dance on connectors where this is not needed. I've checked through all drivers and set this flag everywhere where the connector->detect function has diffe

[PATCH 04/14] drm/i915: set POLL_FORCE for sdvo outputs

2012-05-24 Thread Daniel Vetter
The detection function is simply too unreliable - it doesn't properly pick up changes right away. For now, just set the compat flag and ignore this, because on a quick look fixing this properly is a very big fish to fry. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_sdvo.c |2

[PATCH 05/14] drm: properly init/reset connector status

2012-05-24 Thread Daniel Vetter
We need this because otherwise the improved connector code has no idea when it needs to run the ->detect callback after boot/resume on all connectors. Because drm/i915 is the only driver that properly calls mode_config_reset at resume time, this will horribly blow up everywhere else. Signed-Off-b

[PATCH 06/14] drm: kill unnecessary calls to connector->detect

2012-05-24 Thread Daniel Vetter
Only call that function if something has actually changed (i.e. in the output polling or hdp handling functions) or when userspace asks for the information and DRM_CONNECTOR_POLL_FORCE is set. Let's see how many bugs this uncovers. v2: Run ->detect if the current connector status is 'unknown' - o

[PATCH 07/14] drm: don't start the poll engine in probe_single_connector

2012-05-24 Thread Daniel Vetter
Actually there's a reason this stuff is there, and it's called commit e58f637bb96d5a0ae0919b9998b891d1ba7e47c9 Author: Chris Wilson Date: Fri Aug 20 09:13:36 2010 +0100 drm/kms: Add a module parameter to disable polling The idea has been that users can enable/disable polling at runtime. S

[PATCH 08/14] drm: don't unnecessarily enable the polling work

2012-05-24 Thread Daniel Vetter
... by properly checking connector->polled. This doesn't matter too much because the polling work itself gets this slightly more right and doesn't set repoll if there's nothing to do. But we can do better. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc_helper.c |6 ++ 1 files

[PATCH 09/14] drm: don't poll forced connectors

2012-05-24 Thread Daniel Vetter
Otherwise if the detect callback reports a different state than what the user forced (rather likely), we continously annoy userspace about a hotplug uevent. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/drm_crtc_helper.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --gi

[PATCH 10/14] drm/i915: cache crt edid

2012-05-24 Thread Daniel Vetter
Let's put all this new&neat output detection infrastructure and rework to some good use and cache the crt edid. Given that the drm helpers now only call ->detect when actually required, we only need to reset the edid there and can keep it otherwise. Slashes xrandr time on systems that are hotplug

[PATCH 11/14] drm/i915: cache dp edid

2012-05-24 Thread Daniel Vetter
Again, let's be slightly more clever here. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_dp.c | 47 ++ 1 files changed, 22 insertions(+), 25 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 3bbd

[PATCH 12/14] drm/i915: cache hdmi edid

2012-05-24 Thread Daniel Vetter
Like the previous patches. While at it also kill a stale comment - we've moved hdmi audio detection from ->get_modes to ->detect and the audio property handling functions. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_drv.h |1 + drivers/gpu/drm/i915/intel_hdmi.c | 48 +

[PATCH 13/14] drm/i915/sdvo: implement correct return value for ->get_modes

2012-05-24 Thread Daniel Vetter
We should return the number of added modes. Luckily no one really cares, but it kinda sticked out compared to the other ->get_modes functions I've looked at recently. Signed-Off-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_sdvo.c | 41 ++-- 1 files changed,

[PATCH 14/14] drm/i915: s/mdelay/msleep/ in the sdvo detect function

2012-05-24 Thread Daniel Vetter
A 30 ms delay is simply way too big to waste cpu cycles on. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/i915/intel_sdvo.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c index 6056603..efa0d17 1

[Bug 50325] New: Glyphy bad render on r600g (software render is fine)

2012-05-24 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50325 Bug #: 50325 Summary: Glyphy bad render on r600g (software render is fine) Classification: Unclassified Product: Mesa Version: 8.0 Platform: x86-64 (AMD64) OS/Version: Linux (All)

[Bug 50325] Glyphy bad render on r600g (software render is fine)

2012-05-24 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50325 --- Comment #1 from T?r?k Edwin 2012-05-24 14:57:08 PDT --- Created attachment 62079 --> https://bugs.freedesktop.org/attachment.cgi?id=62079 r600g-bad.png -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- Yo

[Bug 50325] Glyphy bad render on r600g (software render is fine)

2012-05-24 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50325 --- Comment #2 from T?r?k Edwin 2012-05-24 14:57:24 PDT --- Created attachment 62080 --> https://bugs.freedesktop.org/attachment.cgi?id=62080 sw-good.png -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You

drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-24 Thread Linus Torvalds
These guys seem to be recently introduced: [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy: GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206 [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy: GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700 This is on my

[PATCH] drm/radeon: fix typo in trinity tiling setup

2012-05-24 Thread alexdeuc...@gmail.com
From: Alex Deucher Using the wrong union. Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/ni.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/radeon/ni.c b/drivers/gpu/drm/radeon/ni.c index b01c2dd..ce4e7cc 100644

[PATCH] omap2+: add drm device

2012-05-24 Thread Clark, Rob
On Thu, May 24, 2012 at 12:01 AM, Tomi Valkeinen wrote: > Hi, > > On Wed, 2012-05-23 at 15:08 -0500, Andy Gross wrote: >> Register OMAP DRM/KMS platform device. ?DMM is split into a >> separate device using hwmod. >> >> Signed-off-by: Andy Gross > > > >> +static int __init omap_init_drm(void) >

Oops with Radeon/Uninorth on Maple

2012-05-24 Thread Dmitry Eremin-Solenikov
Hello, colleagues, I'm trying to enable an AGP slot (again) on my Maple board (dual PPC970FX board, with CPC925 (U3H) north bridge). For now I'm stuck with a problem: I use radeon card, drm-radeon driver with KMS. If I force drm-radeon to think about a card as about PCI card (by commenting

[patch] drm/udl: unlock before returning in udl_gem_mmap()

2012-05-24 Thread Dan Carpenter
On Thu, Mar 22, 2012 at 09:30:56AM +0300, Dan Carpenter wrote: > If we hit an error here, then we should unlock and unreference obj > before returning. > This is still present in linux-next. regards, dan carpenter > Signed-off-by: Dan Carpenter > > diff --git a/drivers/gpu/drm/udl/udl_gem.c b

[PATCH 08/10] drm/radeon: replace pflip and sw_int counters with atomics

2012-05-24 Thread Koenig, Christian
> -Original Message- > From: Sylvain BERTRAND [mailto:sylware at legeek.net] > Sent: Thursday, May 24, 2012 1:59 PM > To: Christian K?nig > Cc: j.glisse at gmail.com; Koenig, Christian; dri-devel at > lists.freedesktop.org > Subject: Re: [PATCH 08/10] drm/radeon: replace pflip and sw_int c

<    1   2   3