Re: [patch v2] nouveau: unwind on load errors

2010-08-04 Thread Francisco Jerez
Marcin Slusarz writes: > On Fri, Jul 30, 2010 at 05:04:32PM +0200, Dan Carpenter wrote: >> nuveau_load() just returned directly if there was an error instead of >> > ^^ typo > >> releasing resources.

Re: 2.6.35-rc6+: i915: Bisected regression

2010-08-04 Thread Shuang He
On 2010-8-1 18:51, Thomas Meyer wrote: Am 01.08.2010 um 13:42 schrieb Stefan Richter: Thomas Meyer wrote: This commit makes suspend to ram unreliable for me: commit d1b851fc0d105caa6b6e3e7c92d2987dfb52cbe0 Author: Zou Nan hai Date: Fri May 21 09:08:57 2010 +0800 drm/i915: implement BS

[PATCH RESEND block#for-2.6.36] block_dev: always serialize exclusive open attempts

2010-08-04 Thread Tejun Heo
bd_prepare_to_claim() incorrectly allowed multiple attempts for exclusive open to progress in parallel if the attempting holders are identical. This triggered BUG_ON() as reported in the following bug. https://bugzilla.kernel.org/show_bug.cgi?id=16393 __bd_abort_claiming() is used to finish cl

[PATCH] light weight drm fbdev panning

2010-08-04 Thread James Simmons
> > This patch uses the much lighter mode_set_base instead of calling a full > > mode set. Tested on a i915 netbook. Patch should be against drm-core-next. > > The kms code should always fallback if appropriate, I've definitely > seen some modesets I thought were triggered by the pan_display > co

[PATCH block#for-2.6.36] block_dev: always serialize exclusive open attempts

2010-08-04 Thread Tejun Heo
bd_prepare_to_claim() incorrectly allowed multiple attempts for exclusive open to progress in parallel if the attempting holders are identical. This triggered BUG_ON() as reported in the following bug. https://bugzilla.kernel.org/show_bug.cgi?id=16393 __bd_abort_claiming() is used to finish cl

[PATCH 1/4] drm: Remove drm_resource wrappers

2010-08-04 Thread Matt Turner
On Wed, Aug 4, 2010 at 4:48 PM, Jordan Crouse wrote: > Remove the drm_resource wrappers and directly use the > actual PCI and/or platform functions in their place. > > Signed-off-by: Jordan Crouse > --- > ?drivers/gpu/drm/drm_bufs.c ? ? ? ? ? ? ? ?| ? 13 - > ?drivers/gpu/drm/i915/i915

Re: [PATCH] drm: Use ENOENT consistently for the error return for an unmatched handle.

2010-08-04 Thread Dave Airlie
On Wed, 2010-08-04 at 14:19 +0100, Chris Wilson wrote: > This is consistent with trying to access a filename that not exist > within a directory which is a good analogy here. The main reason for the > change is that it is easy to confuse the error code of EBADF as an > performing an ioctl on an inv

[Bug 29400] R300G : Regression, Transparency in KDE effects is corrupted

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=29400 Marek Olšák changed: What|Removed |Added Component|Other |Drivers/DRI/r300 AssignedTo|mesa-.

[Bug 29400] R300G : Regression, Transparency in KDE effects is corrupted

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=29400 Marek Ol??k changed: What|Removed |Added Component|Other |Drivers/DRI/r300 AssignedTo|mesa-d

[PATCH 4/4] drm: Make sure the DRM offset matches the CPU

2010-08-04 Thread Jordan Crouse
The pgoff option in mmap() is defined as an unsigned long so the offset generated by DRM needs to fit into BITS_PER_LONG for the CPU in question. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_gem.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/

[PATCH 3/4] drm: Add __arm defines to DRM

2010-08-04 Thread Jordan Crouse
Add __arm defines to specify behavior specific for an ARM processor. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_bufs.c |2 +- drivers/gpu/drm/drm_vm.c | 14 +- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/d

[PATCH 2/4] drm: Add support for platform devices to register as DRM devices

2010-08-04 Thread Jordan Crouse
Allow platform devices without PCI resources to be DRM devices. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/Kconfig |4 +- drivers/gpu/drm/Makefile |2 +- drivers/gpu/drm/drm_drv.c | 37 +--- drivers/gpu/drm/drm_edid.c|4 +

[PATCH 1/4] drm: Remove drm_resource wrappers

2010-08-04 Thread Jordan Crouse
Remove the drm_resource wrappers and directly use the actual PCI and/or platform functions in their place. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_bufs.c| 13 - drivers/gpu/drm/i915/i915_dma.c |6 +++--- drivers/gpu/drm/mga/mga_dma.c

[PATCH v3] DRM platform device support

2010-08-04 Thread Jordan Crouse
DRM platform device support freshly rebased for 2.6.35. Nothing changed in the code over the set I sent on 5/27. Jordan

[PATCH 2/2] drm/i915: Record error batch buffers using iomem

2010-08-04 Thread Chris Wilson
Directly read the GTT mapping for the contents of the batch buffers rather than relying on possibly stale CPU caches. Also for completeness scan the flushing/inactive lists for the current buffers - we are collecting error state after all. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i91

[PATCH 1/2] drm,io-mapping: Specify slot to use for atomic mappings

2010-08-04 Thread Chris Wilson
This is required should we ever attempt to use an io-mapping where KM_USER0 is verboten, such as inside an IRQ context. Signed-off-by: Chris Wilson Cc: Eric Anholt --- drivers/gpu/drm/i915/i915_gem.c|9 + drivers/gpu/drm/i915/intel_overlay.c |5 +++-- drivers/gpu/drm/n

[PATCH 2/2] drm/radeon/kms: enable underscan option for digital connectors

2010-08-04 Thread Marius Gröger
Am 04.08.2010 01:59, schrieb Alex Deucher: > This connector attribute allows you to enable or disable underscan > on a digital output to compensate for panels that automatically > overscan (e.g., many HDMI TVs). Valid values for the attribute are: > > off - forces underscan off > on - forces under

[PATCH] drm: Use ENOENT consistently for the error return for an unmatched handle.

2010-08-04 Thread Chris Wilson
This is consistent with trying to access a filename that not exist within a directory which is a good analogy here. The main reason for the change is that it is easy to confuse the error code of EBADF as an performing an ioctl on an invalid file descriptor (rather than an unknown object). Signed-o

Re: [PATCH 1/4] drm: Remove drm_resource wrappers

2010-08-04 Thread Matt Turner
On Wed, Aug 4, 2010 at 4:48 PM, Jordan Crouse wrote: > Remove the drm_resource wrappers and directly use the > actual PCI and/or platform functions in their place. > > Signed-off-by: Jordan Crouse > --- >  drivers/gpu/drm/drm_bufs.c                |   13 - >  drivers/gpu/drm/i915/i915

[PATCH 4/4] drm: Make sure the DRM offset matches the CPU

2010-08-04 Thread Jordan Crouse
The pgoff option in mmap() is defined as an unsigned long so the offset generated by DRM needs to fit into BITS_PER_LONG for the CPU in question. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_gem.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/

[PATCH 3/4] drm: Add __arm defines to DRM

2010-08-04 Thread Jordan Crouse
Add __arm defines to specify behavior specific for an ARM processor. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_bufs.c |2 +- drivers/gpu/drm/drm_vm.c | 14 +- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/d

[PATCH 2/4] drm: Add support for platform devices to register as DRM devices

2010-08-04 Thread Jordan Crouse
Allow platform devices without PCI resources to be DRM devices. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/Kconfig |4 +- drivers/gpu/drm/Makefile |2 +- drivers/gpu/drm/drm_drv.c | 37 +--- drivers/gpu/drm/drm_edid.c|4 +

[PATCH 1/4] drm: Remove drm_resource wrappers

2010-08-04 Thread Jordan Crouse
Remove the drm_resource wrappers and directly use the actual PCI and/or platform functions in their place. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_bufs.c| 13 - drivers/gpu/drm/i915/i915_dma.c |6 +++--- drivers/gpu/drm/mga/mga_dma.c

[PATCH v3] DRM platform device support

2010-08-04 Thread Jordan Crouse
DRM platform device support freshly rebased for 2.6.35. Nothing changed in the code over the set I sent on 5/27. Jordan ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH RESEND block#for-2.6.36] block_dev: always serialize exclusive open attempts

2010-08-04 Thread Tejun Heo
bd_prepare_to_claim() incorrectly allowed multiple attempts for exclusive open to progress in parallel if the attempting holders are identical. This triggered BUG_ON() as reported in the following bug. https://bugzilla.kernel.org/show_bug.cgi?id=16393 __bd_abort_claiming() is used to finish cl

[PATCH block#for-2.6.36] block_dev: always serialize exclusive open attempts

2010-08-04 Thread Tejun Heo
bd_prepare_to_claim() incorrectly allowed multiple attempts for exclusive open to progress in parallel if the attempting holders are identical. This triggered BUG_ON() as reported in the following bug. https://bugzilla.kernel.org/show_bug.cgi?id=16393 __bd_abort_claiming() is used to finish cl

[PATCH] drm/radeon: add new pci ids

2010-08-04 Thread Alex Deucher
New evergreen and r7xx ids. Signed-off-by: Alex Deucher Cc: stable at kernel.org --- include/drm/drm_pciids.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 2d428b0..3a9940e 100644 --- a/include/drm/drm_pci

[PATCH] drm/radeon/kms: mark 3D power states as performance

2010-08-04 Thread Alex Deucher
Fixes lack of power saving with multiple heads on some desktop cards. Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=16474 Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_atombios.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/ra

[PATCH] drm: Cleanup after failing to create master->unique and dev->name

2010-08-04 Thread Chris Wilson
v2: Userspace (notably xf86-video-{intel,ati}) became confused when drmSetInterfaceVersion() started returning -EBUSY as they used a second call (the first done in drmOpen()) to check their master credentials. Since userspace wants to be able to repeatedly call drmSetInterfaceVersion() allow them t

[PATCH 2/2] drm/radeon/kms: enable underscan option for digital connectors

2010-08-04 Thread Alex Deucher
2010/8/4 Marius Gr?ger : > Am 04.08.2010 01:59, schrieb Alex Deucher: >> >> This connector attribute allows you to enable or disable underscan >> on a digital output to compensate for panels that automatically >> overscan (e.g., many HDMI TVs). ?Valid values for the attribute are: >> >> off - force

Re: [PATCH] light weight drm fbdev panning

2010-08-04 Thread James Simmons
> > This patch uses the much lighter mode_set_base instead of calling a full > > mode set. Tested on a i915 netbook. Patch should be against drm-core-next. > > The kms code should always fallback if appropriate, I've definitely > seen some modesets I thought were triggered by the pan_display > co

[Bug 29355] M92 : Xserver fails to start

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=29355 --- Comment #6 from Alex Deucher 2010-08-04 08:55:25 PDT --- Created an attachment (id=37571) View: https://bugs.freedesktop.org/attachment.cgi?id=37571 Review: https://bugs.freedesktop.org/review?bug=29355&attachment=37571 add missign pci ids

[Bug 29355] M92 : Xserver fails to start

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=29355 --- Comment #6 from Alex Deucher 2010-08-04 08:55:25 PDT --- Created an attachment (id=37571) View: https://bugs.freedesktop.org/attachment.cgi?id=37571 Review: https://bugs.freedesktop.org/review?bug=29355&attachment=37571 add missign pci id

[Bug 28474] [r300g] lugaru/etc locks up laptop

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=28474 Giacomo Perale changed: What|Removed |Added CC||ghe...@virgilio.it --- Comment #11 from

[Bug 28474] [r300g] lugaru/etc locks up laptop

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=28474 Giacomo Perale changed: What|Removed |Added CC||ghepeu at virgilio.it --- Comment #11 f

[PATCH] drm/radeon: add new pci ids

2010-08-04 Thread Alex Deucher
New evergreen and r7xx ids. Signed-off-by: Alex Deucher Cc: sta...@kernel.org --- include/drm/drm_pciids.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 2d428b0..3a9940e 100644 --- a/include/drm/drm_pciids

[PATCH] drm/radeon/kms: mark 3D power states as performance

2010-08-04 Thread Alex Deucher
Fixes lack of power saving with multiple heads on some desktop cards. Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=16474 Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_atombios.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/ra

Re: [PATCH 2/2] drm/radeon/kms: enable underscan option for digital connectors

2010-08-04 Thread Alex Deucher
2010/8/4 Marius Gröger : > Am 04.08.2010 01:59, schrieb Alex Deucher: >> >> This connector attribute allows you to enable or disable underscan >> on a digital output to compensate for panels that automatically >> overscan (e.g., many HDMI TVs).  Valid values for the attribute are: >> >> off - force

[PATCH 1/2] drm,io-mapping: Specify slot to use for atomic mappings

2010-08-04 Thread Chris Wilson
This is required should we ever attempt to use an io-mapping where KM_USER0 is verboten, such as inside an IRQ context. Signed-off-by: Chris Wilson Cc: Eric Anholt --- drivers/gpu/drm/i915/i915_gem.c|9 + drivers/gpu/drm/i915/intel_overlay.c |5 +++-- drivers/gpu/drm/n

[PATCH 2/2] drm/i915: Record error batch buffers using iomem

2010-08-04 Thread Chris Wilson
Directly read the GTT mapping for the contents of the batch buffers rather than relying on possibly stale CPU caches. Also for completeness scan the flushing/inactive lists for the current buffers - we are collecting error state after all. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i91

[PATCH] drm: Use ENOENT consistently for the error return for an unmatched handle.

2010-08-04 Thread Chris Wilson
This is consistent with trying to access a filename that not exist within a directory which is a good analogy here. The main reason for the change is that it is easy to confuse the error code of EBADF as an performing an ioctl on an invalid file descriptor (rather than an unknown object). Signed-o

[Bug 28294] [r300g] Unigine Sanctuary v2.2: black glitches

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=28294 --- Comment #25 from Sven Arvidsson 2010-08-04 06:11:23 PDT --- Terminal output contains a lot of these error messages: OpenGL error: invalid value OpenGL error: invalid framebuffer operation GLFrameBuffer::check_status(): incomplete attachme

[Bug 28294] [r300g] Unigine Sanctuary v2.2: black glitches

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=28294 --- Comment #25 from Sven Arvidsson 2010-08-04 06:11:23 PDT --- Terminal output contains a lot of these error messages: OpenGL error: invalid value OpenGL error: invalid framebuffer operation GLFrameBuffer::check_status(): incomplete attachme

[Bug 29355] M92 : Xserver fails to start

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=29355 --- Comment #5 from samit vats 2010-08-04 06:01:06 PDT --- Created an attachment (id=37570) --> (https://bugs.freedesktop.org/attachment.cgi?id=37570) lspci.txt -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ---

[Bug 29355] M92 : Xserver fails to start

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=29355 --- Comment #5 from samit vats 2010-08-04 06:01:06 PDT --- Created an attachment (id=37570) --> (https://bugs.freedesktop.org/attachment.cgi?id=37570) lspci.txt -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --

Re: [PATCH 2/2] drm/radeon/kms: enable underscan option for digital connectors

2010-08-04 Thread Marius Gröger
Am 04.08.2010 01:59, schrieb Alex Deucher: This connector attribute allows you to enable or disable underscan on a digital output to compensate for panels that automatically overscan (e.g., many HDMI TVs). Valid values for the attribute are: off - forces underscan off on - forces underscan on a

[PATCH] drm/radeon/kms: only expose underscan on avivo chips

2010-08-04 Thread Alex Deucher
R4xx also uses the atom add connector function, but underscan is only supported on avivo chips. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_connectors.c | 21 - drivers/gpu/drm/radeon/radeon_encoders.c |6 -- 2 files changed, 16 insertions(+), 11

[PATCH] drm: Cleanup after failing to create master->unique and dev->name

2010-08-04 Thread Chris Wilson
v2: Userspace (notably xf86-video-{intel,ati}) became confused when drmSetInterfaceVersion() started returning -EBUSY as they used a second call (the first done in drmOpen()) to check their master credentials. Since userspace wants to be able to repeatedly call drmSetInterfaceVersion() allow them t

[Bug 28294] [r300g] Unigine Sanctuary v2.2: black glitches

2010-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=28294 Pavel Ondračka changed: What|Removed |Added Attachment #35899|0 |1 is obsolete|

[Bug 28294] [r300g] Unigine Sanctuary v2.2: black glitches

2010-08-04 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=28294 Pavel Ondra?ka changed: What|Removed |Added Attachment #35899|0 |1 is obsolete|

[PATCH] drm/radeon/kms: only expose underscan on avivo chips

2010-08-04 Thread Alex Deucher
R4xx also uses the atom add connector function, but underscan is only supported on avivo chips. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_connectors.c | 21 - drivers/gpu/drm/radeon/radeon_encoders.c |6 -- 2 files changed, 16 insertions(+), 11