[Bug 72369] glitches in serious sam 3 with the sb shader backend

2013-12-11 Thread bugzilla-dae...@freedesktop.org
vel/attachments/20131211/a0b737fa/attachment.html>

[PATCH] radeon: avoid possible divide by 0 in surface manager

2013-12-11 Thread Michel Dänzer
On Die, 2013-12-10 at 12:42 -0500, Alex Deucher wrote: > Some users report hitting a divide by 0 with the tile split in > certain apps. Tile_split shouldn't ever be 0 unless the surface > structure was not properly initialized. I think there may be some > cases where mesa uses an improperly

[Bug 66871] New: Radeon multi-screen intermittent display failure

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=66871 Bug ID: 66871 Summary: Radeon multi-screen intermittent display failure Product: Drivers Version: 2.5 Kernel Version: 3.13.0-rc3 Hardware: x86-64 OS: Linux

TTM's role in score-based eviction

2013-12-11 Thread Michel Dänzer
On Die, 2013-12-10 at 12:03 +0100, Maarten Lankhorst wrote: > op 10-12-13 01:49, Michel D?nzer schreef: > > On Mon, 2013-12-09 at 23:45 +0100, Marek Ol??k wrote: > >> On Mon, Dec 9, 2013 at 9:30 PM, Lauri Kasanen wrote: > >>> Note that the hotness calculation will be in userspace, as only there >

Fixing nouveau for >4k PAGE_SIZE

2013-12-11 Thread Ben Skeggs
On Fri, Nov 29, 2013 at 4:01 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-08-29 at 16:49 +1000, Ben Skeggs wrote: > >> > Additionally the current code is broken in that the upper layer in >> > vm/base.c doesn't increment "pte" by the right amount. >> > >> > Now, if those two assertions can be

[Bug 69723] GPU lockups with kernel 3.11.0 / 3.12-rc1 when dpm=1 on r600g (Cayman)

2013-12-11 Thread bugzilla-dae...@freedesktop.org
<http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/18aac1b5/attachment.html>

[Bug 69723] GPU lockups with kernel 3.11.0 / 3.12-rc1 when dpm=1 on r600g (Cayman)

2013-12-11 Thread bugzilla-dae...@freedesktop.org
ving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/b6ed2064/attachment.html>

[Bug 39612] radeon blocks with new style fencing

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=39612 --- Comment #6 from Michel D?nzer --- Is this still an issue with current kernels and userspace stack? -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 66871] Radeon multi-screen intermittent display failure

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=66871 --- Comment #1 from Michel D?nzer --- Can you bisect? -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 72387] Tearing at one specific part of the screen on CAYMAN

2013-12-11 Thread bugzilla-dae...@freedesktop.org
eedesktop.org/archives/dri-devel/attachments/20131211/5f469160/attachment.html>

[Bug 72283] [] dce6_afmt_write_speaker_allocation+0xdb/0x140 [radeon]

2013-12-11 Thread bugzilla-dae...@freedesktop.org
part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/dbea7e3f/attachment.html>

Can no longer shutdown after drm/radeon: Implement radeon_pci_shutdown

2013-12-11 Thread Peter Chubb
On my HP Elitebook 8740w qith a Mobility Radeon HD 5870 commit 846ae41ae99d314bf2a02784152208a6ddf7eddc breaks shutdown. The machine hangs when trying to shutdown, kexec or hibernate, before seeing the usual `machine halted' (or whatever) message. If I comment out thus: index 9f5ff28..40bff3c

Can no longer shutdown after drm/radeon: Implement radeon_pci_shutdown

2013-12-11 Thread Markus Trippelsdorf
On 2013.12.11 at 11:37 +1100, Peter Chubb wrote: > On my HP Elitebook 8740w qith a Mobility Radeon HD 5870 > commit 846ae41ae99d314bf2a02784152208a6ddf7eddc > breaks shutdown. The machine hangs when trying to shutdown, kexec or > hibernate, before seeing the usual `machine halted' (or whatever)

[PATCH] gpu: host1x: clk_round_rate() can return a zero upon error

2013-12-11 Thread Sascha Hauer
On Mon, Dec 09, 2013 at 06:00:12PM -0800, Paul Walmsley wrote: > > Treat both negative and zero return values from clk_round_rate() as > errors. This is needed since subsequent patches will convert > clk_round_rate()'s return value to be an unsigned type, rather than a > signed type, since some

TTM's role in score-based eviction

2013-12-11 Thread Maarten Lankhorst
op 11-12-13 04:04, Michel D?nzer schreef: > On Die, 2013-12-10 at 12:03 +0100, Maarten Lankhorst wrote: >> op 10-12-13 01:49, Michel D?nzer schreef: >>> On Mon, 2013-12-09 at 23:45 +0100, Marek Ol??k wrote: On Mon, Dec 9, 2013 at 9:30 PM, Lauri Kasanen wrote: > Note that the hotness

TTM's role in score-based eviction

2013-12-11 Thread Thomas Hellstrom
On 12/11/2013 08:57 AM, Maarten Lankhorst wrote: > op 11-12-13 04:04, Michel D?nzer schreef: >> On Die, 2013-12-10 at 12:03 +0100, Maarten Lankhorst wrote: >>> op 10-12-13 01:49, Michel D?nzer schreef: On Mon, 2013-12-09 at 23:45 +0100, Marek Ol??k wrote: > On Mon, Dec 9, 2013 at 9:30 PM,

drm: update VIA driver to 2.7.2

2013-12-11 Thread Dan Carpenter
Hello Dave Airlie, The patch 925142431bd6: "drm: update VIA driver to 2.7.2" from Nov 12, 2005, leads to the following static checker warning: drivers/gpu/drm/via/via_irq.c:242 via_driver_irq_wait() error: buffer overflow 'masks' 4 <= 5 drivers/gpu/drm/via/via_irq.c 225

[PULL] drm-intel-next

2013-12-11 Thread Daniel Vetter
Hi Dave, So first pull request for 3.14. A bit later than usual, but it's also been a really quiet month for feature stuff somehow. Much fewer patches than usual. Highlights: - some more ppgtt prep patches from Ben - a few fbc fixes from Ville - power well rework from Imre - vlv forcewake

[PATCH 01/50] drm/rcar: call drm_put_dev directly in the ->remove hook

2013-12-11 Thread Daniel Vetter
The magic dance drm_platform_exit does is actually a remnant of the old legacy shadow attach support for platform devices. Modern modesetting drm drivers shouldn't do this any more (and usb/pci devices actually don't do this). Cc: Laurent Pinchart

[PATCH 00/50] more drm de-midlayering

2013-12-11 Thread Daniel Vetter
Hi all, This series almost removes drm_bus, the last thing remaining is the ->setversion callback. Unfortunately we can't kill that completely since we need the backwards compat cruft for pci domain bonghits on alpha/ppc. I've also shot at a few easy marks on the road while at it. My plan is to

[PATCH 02/50] drm/exynos: call drm_put_dev directly from ->remove

2013-12-11 Thread Daniel Vetter
I didn't find any user of the driver data yet, so store the drm_device pointer in there. Cc: Inki Dae Acked-by: Inki Dae Signed-off-by: Daniel Vetter --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 04/50] drm/tilcdc: call drm_put_dev directly from ->remove

2013-12-11 Thread Daniel Vetter
tilcdc already stores the drm_device in the driver data pointer. So use that. Cc: Rob Clark Signed-off-by: Daniel Vetter --- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c

[PATCH 09/50] drm: rip out drm_platform_exit

2013-12-11 Thread Daniel Vetter
This very much looks like a remnant of the old legady ums shadow attach days. Now with the last users gone we can rip it out since we won't ever support an ums drm driver again. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_platform.c | 11 --- include/drm/drmP.h | 1

[PATCH 05/50] drm/omap: call drm_put_dev directly in ->remove

2013-12-11 Thread Daniel Vetter
Again omap already sets the driver data pointer to the drm_device. Also drop the driver unregister call, that should be (and already is) done in the module unload hook. Cc: Rob Clark Signed-off-by: Daniel Vetter --- drivers/gpu/drm/omapdrm/omap_drv.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH 06/50] drm/shmob: call drm_put_dev directly from ->remove hook

2013-12-11 Thread Daniel Vetter
We need to chase one pointer here. Cc: Laurent Pinchart Signed-off-by: Daniel Vetter --- drivers/gpu/drm/shmobile/shmob_drm_drv.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_drv.c

[PATCH 07/50] drm/armada: directly call drm_put_dev in ->remove

2013-12-11 Thread Daniel Vetter
Again no apparent user of the driver data field. Cc: Russell King Signed-off-by: Daniel Vetter --- drivers/gpu/drm/armada/armada_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/armada/armada_drv.c

[PATCH 08/50] drm/msm: call drm_put_dev directly in ->remove

2013-12-11 Thread Daniel Vetter
The drvdata pointer is already assigned to something useful. Cc: Rob Clark Signed-off-by: Daniel Vetter --- drivers/gpu/drm/msm/msm_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c index

[PATCH 10/50] drm: restrict the device list for shadow attached drivers

2013-12-11 Thread Daniel Vetter
There's really no need for the drm core to keep a list of all devices of a given driver - the linux device model keeps perfect track of this already for us. The exception is old legacy ums drivers using pci shadow attaching. So rename the lists to make the use case clearer and rip out everything

[PATCH 13/50] drm: ->agp_init can't fail

2013-12-11 Thread Daniel Vetter
Thanks to the removal of REQUIRE_AGP we can use a void return value and shed a bit of complexity. Reviewed-by: David Herrmann Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_pci.c | 3 +-- drivers/gpu/drm/drm_stub.c | 7 ++- include/drm/drmP.h | 2 +- 3 files changed, 4

[PATCH 14/50] drm: rip out drm_core_has_AGP

2013-12-11 Thread Daniel Vetter
Most place actually want to just check for dev->agp (most do, but a few don't so this fixes a few potential NULL derefs). The only exception is the agp init code which should check for the AGP driver feature flag. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_agpsupport.c| 2 +-

[PATCH 16/50] drm: inline drm_agp_destroy

2013-12-11 Thread Daniel Vetter
Wrapping a kfree is pointless. v2: Add a comment to the kerneldoc for drm_agp_init to explain where the kfree happens as requested by David. Note that for modeset drivers agp cleanup is fairly complicated anyway: The drm_agp_clear is a noop and drivers must call drm_agp_release on their own.

[PATCH 19/50] drm: rip out DRM_AGP_MEM and DRM_AGP_KERN

2013-12-11 Thread Daniel Vetter
The header provides dummy functions and fallbacks, so no need for screaming macros. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_agpsupport.c | 8 drivers/gpu/drm/drm_memory.c | 6 +++--- include/drm/drmP.h | 5 +++-- include/drm/drm_agpsupport.h | 16

[PATCH 20/50] drm: Kill DRM_HZ

2013-12-11 Thread Daniel Vetter
We don't have any userspace interfaces that use HZ as a time unit, so having our own DRM define is useless. Remove this remnant from the shared drm core days. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c | 4 ++-- drivers/gpu/drm/exynos/exynos_mixer.c | 2 +-

[PATCH 21/50] drm: Kill DRM_IRQ_ARGS

2013-12-11 Thread Daniel Vetter
I've killed them a long time ago in drm/i915, let's get rid of this remnant of shared drm core days for good. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/cirrus/cirrus_drv.h | 2 +- drivers/gpu/drm/gma500/psb_drv.h| 2 +- drivers/gpu/drm/gma500/psb_irq.c| 2 +-

[PATCH 34/50] drm: kill drm_bus->bus_type

2013-12-11 Thread Daniel Vetter
Completely unused. Hooray, midlayer mistakes that didn't cause work to undo! v2: Rebase on top of the recent tegra changes which added a host1x drm bus. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_pci.c | 1 - drivers/gpu/drm/drm_platform.c | 1 - drivers/gpu/drm/drm_usb.c |

[PATCH 35/50] drm: Rip out totally bogus vga_switcheroo->can_switch locking

2013-12-11 Thread Daniel Vetter
So I just wanted to add a new field to struct drm_device and accidentally stumbled over something. According to comments dev->open_count is protected by dev->count_lock, but that's totally not the case. It's protected by drm_global_mutex. Unfortunately the vga switcheroo callbacks took this

[PATCH 36/50] drm: rename dev->count_lock to dev->buf_lock

2013-12-11 Thread Daniel Vetter
Since really that's all it protects - legacy horror stories in drm_bufs.c. Since I don't want to waste any more time on this I didn't bother to actually look at what it protects in there, but it's at least contained now. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_bufs.c | 32

[PATCH 42/50] drm: rip out dev->devname

2013-12-11 Thread Daniel Vetter
This was only ever used to pretty-print the irq driver name. And on kms systems due to set_version bonghits we never set up the prettier name, ever. Which make this a bit pointless. Also, we can always dig out the driver-instance/irq relationship through other means, so this isn't that useful. So

[PATCH 11/50] drm/bufs: remove handling of _DRM_GEM mappings

2013-12-11 Thread Daniel Vetter
Gone with the new gem vma offset manager from David. We can also ditch the uapi header definition from the enum since userspace never used this. It ended up in there purely for historical reasons (for reusing the old drm mmap code essentially), not because userspace ever needed it. Cc: David

[PATCH 15/50] drm: remove agp_init() bus callback

2013-12-11 Thread Daniel Vetter
The PCI bus helper is the only user of it. Call it directly before device-registration to get rid of the callback. Note that all drm_agp_*() calls are locked with the drm-global-mutex so we need to explicitly lock it during initialization. It's not really clear why it's needed, but lets be safe.

[PATCH 18/50] drm: remove global_mutex locking around agp_init

2013-12-11 Thread Daniel Vetter
David Herrmann dutifully moved this locking along when moving the agp_init call out of the generic drm_dev_register into the pci specific load helpers. But afaict there's no need and the reason for that locking has been purely a historical accident - we need the lock around the driver dev node

[PATCH 22/50] drm: Kill DRM_WAKUP and DRM_INIT_WAITQUEUE

2013-12-11 Thread Daniel Vetter
Less yelling ftw! Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c| 6 +++--- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 2 +- drivers/gpu/drm/exynos/exynos_mixer.c| 4 ++-- drivers/gpu/drm/mga/mga_irq.c| 4 ++-- drivers/gpu/drm/omapdrm/omap_irq.c

[PATCH 23/50] drm: Kill DRM_COPY_(TO|FROM)_USER

2013-12-11 Thread Daniel Vetter
Less yelling ftw! Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_buffer.c | 2 +- drivers/gpu/drm/i915/i915_dma.c | 6 +++--- drivers/gpu/drm/mga/mga_dma.c | 4 ++-- drivers/gpu/drm/mga/mga_state.c | 2 +- drivers/gpu/drm/nouveau/nouveau_gem.c | 4 ++--

[PATCH 30/50] drm/pci: fold in irq_by_busid support

2013-12-11 Thread Daniel Vetter
This is a ums-only ioctl, and we've only ever supported ums (at least in upstream) on pci devices. So no point in keeping that piece of legacy logic abstracted within the drm bus driver. To keep things work without CONFIG_PCI also add a dummy ioctl. v2: Block the irq_by_busid ioctl for modeset

[PATCH 31/50] drm/irq: drm_control is a legacy ioctl, so pci devices only

2013-12-11 Thread Daniel Vetter
This just adds a correspdonding check, follow-up patches will exploit this. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index

[PATCH 32/50] drm/irq: remove cargo-culted locking from irq_install/unistall

2013-12-11 Thread Daniel Vetter
The dev->struct_mutex locking in drm_irq.c only protects dev->irq_enabled. Which isn't really much at all and only prevents especially nasty ums userspace from concurrently installing the interrupt handling a few times. Or at least trying. There are tons of unlocked readers of dev->irqs_enabled

[PATCH 33/50] drm: remove drm_dev_to_irq from drivers

2013-12-11 Thread Daniel Vetter
Only used in some legacy pci drivers, and dereferncing the pci irq is actually shorter ... Since this removes all users for drm_dev_to_irq from the tree except in drm_irq.c, move the inline helper in there. It'll disappear soon, too. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c

[PATCH 37/50] drm/irq: track the irq installed in drm_irq_install in dev->irq

2013-12-11 Thread Daniel Vetter
To get rid of the dev->bus->get_irq callback we need to pass in the desired irq explicitly into drm_irq_install. To avoid having to do the same for drm_irq_unistall just track it internally. That leaves drivers with less room to botch things up. Signed-off-by: Daniel Vetter ---

[PATCH 39/50] drm: pass the irq explicitly to drm_irq_install

2013-12-11 Thread Daniel Vetter
Unfortunately this requires a drm-wide change, and I didn't see a sane way around that. Luckily it's fairly simple, we just need to inline the respective get_irq implementation from either drm_pci.c or drm_platform.c. With that we can now also remove drm_dev_to_irq from drm_irq.c. Signed-off-by:

[PATCH 41/50] drm: inline drm_pci_set_unique

2013-12-11 Thread Daniel Vetter
This is only used for drm versions 1.0, and kms drivers have never been there. So we can appropriately restrict this to legacy and hence pci devices and inline everything. v2: Make the dummy function actually return something, caught by Wu Fengguang's 0-day tester. Signed-off-by: Daniel Vetter

[PATCH 43/50] drm: remove drm_bus->get_name

2013-12-11 Thread Daniel Vetter
The only user is the info debugfs file, so we only need something human readable. Now for both pci and platform devices we've used the name of the underlying device driver, which matches the name of the drm driver in all cases. So we can just use that instead. The exception is usb, which used a

[PATCH 44/50] drm: Remove dev->kdriver

2013-12-11 Thread Daniel Vetter
With the last patch to ditch the ->get_name callbacks the last user is now gone. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_pci.c | 1 - drivers/gpu/drm/drm_platform.c | 1 - drivers/gpu/drm/drm_usb.c | 1 - include/drm/drmP.h | 5 - 4 files changed, 8

[PATCH 47/50] drm: rip out dev->ioctl_count tracking

2013-12-11 Thread Daniel Vetter
Now dev->ioctl_count tries to prevent the device from disappearing if it's still in use. And if we'd actually need this code it would be hopelessly racy and broken. But luckily the vfs already takes care of this. So we can just rip it out. Signed-off-by: Daniel Vetter ---

[PATCH 48/50] drm: Kill file_priv->ioctl_count tracking

2013-12-11 Thread Daniel Vetter
It's racy, and it's only used in debugfs. There are simpler ways to know whether something is going on (like looking at dmesg with full debugging enabled). And they're all much more useful. So let's just rip this out. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_drv.c | 2 --

[PATCH 49/50] drm: remove dev->vma_count

2013-12-11 Thread Daniel Vetter
This is just used for a debugfs file, and we can easily reconstruct this number by just walking the list twice. Which isn't really bad for a debugfs file anyway. So let's rip this out. There's the other issue that the dev->vmalist itself is a bit useless, since that can be reconstructed with all

[PATCH 12/50] drm: kill DRIVER_REQUIRE_AGP

2013-12-11 Thread Daniel Vetter
Only the two intel drivers need this and they can easily check for working agp support in their driver ->load callbacks. This is the only reason why agp initialization could fail, so allows us to rip out a bit of error handling code in the next patch. Signed-off-by: Daniel Vetter ---

[PATCH 17/50] drm: kill the ->agp_destroy callback

2013-12-11 Thread Daniel Vetter
Call drm_pci_agp_destroy directly, there's no point in the indirection. Long term we want to shuffle this into each driver's unload logic, but that needs cleared-up drm lifetime rules first. v2: Add a dummy function for !CONFIG_PCI, spotted my David Herrmann. v3: Fixup for the coding style

[PATCH 24/50] drm: Kill DRM_*MEMORYBARRIER

2013-12-11 Thread Daniel Vetter
The real linux interfaces are s much easier on the eyes ... Signed-off-by: Daniel Vetter --- drivers/gpu/drm/gma500/psb_irq.c | 2 +- drivers/gpu/drm/mga/mga_drv.h | 2 +- drivers/gpu/drm/nouveau/nouveau_dma.c | 2 +- drivers/gpu/drm/nouveau/nouveau_dma.h | 2 +-

[PATCH 25/50] drm: Kill DRM_SUSER

2013-12-11 Thread Daniel Vetter
Checking directly for the right capability is simpler. Also this rids us of a few places that use DRM_CURRENTPID. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +- drivers/gpu/drm/via/via_dma.c | 4 ++-- include/drm/drm_os_linux.h| 1 - 3 files changed, 3

[PATCH 26/50] drm/gma500: Remove dead code

2013-12-11 Thread Daniel Vetter
This has the nice advantage that we'll get rid of a DRM_WAIT_ON user for free. Cc: Patrik Jakobsson Cc: Alan Cox Signed-off-by: Daniel Vetter --- drivers/gpu/drm/gma500/psb_irq.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/gpu/drm/gma500/psb_irq.c

[PATCH 27/50] drm/irq: Replace DRM_WAIT_ON with wait_event

2013-12-11 Thread Daniel Vetter
Note that I've dropped the timeout - we don't expect the display hardware to ever hang, and the current code isn't really up to handle this correctly anyway. The only risk I see is that old user modesetting drivers wreak havoc with themselves by disabling the interrupt support at the wrong time.

[PATCH 28/50] drm: Remove DRM_WAIT_ON from all drivers

2013-12-11 Thread Daniel Vetter
Since this is all old ums stuff (including the one in the radeon driver) I've just tried to perfectly replicate the existing semantics. Reinventing wait queue code with semantics that differ from all the standard linux wait functions is just hairy, so now we can get rid of this and so make sure

[PATCH 29/50] drm/irq: simplify irq checks in drm_wait_vblank

2013-12-11 Thread Daniel Vetter
Checking for both an irq number _and_ whether it's enabled is redundant. Also this will breakd drivers which do their own irq management and just set dev->irq_enabled once the irq stuff is all set up. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c | 2 +- 1 file changed, 1

[PATCH 38/50] drm/irq: Look up the pci irq directly in the drm_control ioctl

2013-12-11 Thread Daniel Vetter
It's only ever called for legacy drivers, which are all pci. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index 1348e041c2af..24c6590bb33e 100644 ---

[PATCH 40/50] drm: remove bus->get_irq implementations

2013-12-11 Thread Daniel Vetter
Now that they're all unused we can get rid of them, including the dummy version in drm_usb.c. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_pci.c | 6 -- drivers/gpu/drm/drm_platform.c | 6 -- drivers/gpu/drm/drm_usb.c | 6 -- include/drm/drmP.h | 1 - 4

[PATCH 45/50] drm/: don't set driver->dev_priv_size to 0

2013-12-11 Thread Daniel Vetter
Especially not on modesetting drivers - this is used to size the driver private structure for legacy drm buffers. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/ast/ast_drv.c | 1 - drivers/gpu/drm/qxl/qxl_drv.c | 1 - drivers/gpu/drm/radeon/radeon_drv.c | 1 - 3 files changed, 3

[PATCH 46/50] drm: store the gem vma offset manager in a typed pointer

2013-12-11 Thread Daniel Vetter
This was hidden in a generic void * dev->mm_private. But only ever used for gem. But thanks to this fake generic pretension no one noticed that Rob's drm drivers are now all broken. So just give the offset manager a type pointer and fix up msm, omapdrm and tilcdc. v2: Fixup compile fail. Cc:

[PATCH 50/50] drm: use memdup_user() as a cleanup

2013-12-11 Thread Daniel Vetter
From: Dan Carpenter drivers/gpu/drm/r128/r128_state.c:1014:10-17: WARNING opportunity for memdup_user /c/kernel-tests/src/cocci/drivers/gpu/drm/r128/r128_state.c:1029:9-16: WARNING opportunity for memdup_user

[PATCH 03/50] drm/imx: directly call drm_put_dev in ->remove

2013-12-11 Thread Daniel Vetter
Again no apparent user of the driver data field. Cc: Sascha Hauer Cc: Greg Kroah-Hartman Acked-by: Sascha Hauer Signed-off-by: Daniel Vetter --- drivers/staging/imx-drm/imx-drm-core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 26/50] drm/gma500: Remove dead code

2013-12-11 Thread Patrik Jakobsson
On Wed, Dec 11, 2013 at 11:34 AM, Daniel Vetter wrote: > This has the nice advantage that we'll get rid of a DRM_WAIT_ON user > for free. > > Cc: Patrik Jakobsson > Cc: Alan Cox > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/gma500/psb_irq.c | 15 --- > 1 file changed, 15

[PATCH 46/50] drm: store the gem vma offset manager in a typed pointer

2013-12-11 Thread David Herrmann
Hi Daniel On Wed, Dec 11, 2013 at 11:35 AM, Daniel Vetter wrote: > This was hidden in a generic void * dev->mm_private. But only ever > used for gem. But thanks to this fake generic pretension no one > noticed that Rob's drm drivers are now all broken. > > So just give the offset manager a type

[Bug 22312] radeon 0000:01:00.0: Invalid ROM contents

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=22312 --- Comment #4 from Cesko Voeten --- Hi Alex, I am no longer experiencing the problem, but since this bug still gets new people CC'd to it from time to time I presume that the problem still occurs for some configurations (though not mine). If

TTM's role in score-based eviction

2013-12-11 Thread Lauri Kasanen
On Wed, 11 Dec 2013 12:04:05 +0900 Michel D?nzer wrote: > > Of all the worries that exist, this is a non-issue. Userspace can > > simply queue a lot of draw calls that take 1 second each through the > > normal command submission methods, why would it need to tweak some > > obscure number to

[Bug 26442] [RADEON:KMS:RV620M:SUSPEND] s2ram fails (HD4225) when HDMI connected

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=26442 Alan changed: What|Removed |Added Resolution|--- |OBSOLETE Status|NEW

[Bug 27192] [BISECTED]boot fail on kernel drm-radeon-testing/add NI pci ids

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=27192 Alan changed: What|Removed |Added Component|Video(Other)|Video(DRI - non Intel)

[PATCH 05/50] drm/omap: call drm_put_dev directly in ->remove

2013-12-11 Thread Rob Clark
On Wed, Dec 11, 2013 at 5:34 AM, Daniel Vetter wrote: > Again omap already sets the driver data pointer to the drm_device. > > Also drop the driver unregister call, that should be (and already is) > done in the module unload hook. umm.. there are two devices+drivers at play in there. The

[PATCH 06/50] drm/shmob: call drm_put_dev directly from ->remove hook

2013-12-11 Thread Laurent Pinchart
Hi Daniel, Thank you for the patch. On Wednesday 11 December 2013 11:34:27 Daniel Vetter wrote: > We need to chase one pointer here. > > Cc: Laurent Pinchart > Signed-off-by: Daniel Vetter Acked-by: Laurent Pinchart

[Bug 28902] activated KMS leads to complete freeze when viewing BIG images in Iceweasel

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=28902 Alan changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 29512] [PATCH]radeon rejects EDID with RV350 when hw_i2c=1

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=29512 Alan changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|---

[PATCH] drm/omap: call drm_put_dev directly in ->remove

2013-12-11 Thread Daniel Vetter
Again omap already sets the driver data pointer to the drm_device. v2: Don't rip out the platform_driver_unregister call for omap_dmm_driver. Pesky difference between drm and dmm ... Cc: Rob Clark Signed-off-by: Daniel Vetter --- drivers/gpu/drm/omapdrm/omap_drv.c | 2 +- 1 file changed, 1

[PATCH 1/2] drm: store the gem vma offset manager in a typed pointer

2013-12-11 Thread Daniel Vetter
This was hidden in a generic void * dev->mm_private. But only ever used for gem. But thanks to this fake generic pretension no one noticed that Rob's drm drivers are now all broken. So just give the offset manager a type pointer and fix up msm, omapdrm and tilcdc. v2: Fixup compile fail. v3:

[PATCH 2/2] drm/gma500: Remove unused function declaration

2013-12-11 Thread Daniel Vetter
Cc: Patrik Jakobsson Signed-off-by: Daniel Vetter --- drivers/gpu/drm/gma500/psb_intel_drv.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/gma500/psb_intel_drv.h b/drivers/gpu/drm/gma500/psb_intel_drv.h index bde27fdb41bf..dc2c8eb030fa 100644 ---

[Bug 66871] Radeon multi-screen intermittent display failure

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=66871 Alex Deucher changed: What|Removed |Added CC||alexdeucher at gmail.com --- Comment #2

TTM's role in score-based eviction

2013-12-11 Thread Thomas Hellstrom
On 12/11/2013 12:35 PM, Lauri Kasanen wrote: > On Wed, 11 Dec 2013 12:04:05 +0900 > Michel D?nzer wrote: > >>> Of all the worries that exist, this is a non-issue. Userspace can >>> simply queue a lot of draw calls that take 1 second each through the >>> normal command submission methods, why

[PATCH] gpu: host1x: clk_round_rate() can return a zero upon error

2013-12-11 Thread Thierry Reding
vers as well to make sure the signed variables don't overflow. Perhaps unsigned long long would be a better choice for future compatibility? Thierry -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/5717e8c8/attachment.pgp>

[PATCH 2/2] drm/gma500: Remove unused function declaration

2013-12-11 Thread Patrik Jakobsson
On Wed, Dec 11, 2013 at 2:24 PM, Daniel Vetter wrote: > Cc: Patrik Jakobsson > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/gma500/psb_intel_drv.h | 5 - > 1 file changed, 5 deletions(-) > > diff --git a/drivers/gpu/drm/gma500/psb_intel_drv.h >

[Bug 22312] radeon 0000:01:00.0: Invalid ROM contents

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=22312 --- Comment #5 from Dnitry --- Created attachment 118051 --> https://bugzilla.kernel.org/attachment.cgi?id=118051=edit uname -a, lspci -v -s 01:00.0, dmesg Yes, I still have this issue. It doesn't manifest itself. Everything works fine except

[Bug 22312] radeon 0000:01:00.0: Invalid ROM contents

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=22312 Dnitry changed: What|Removed |Added Attachment #118051|application/octet-stream|text/plain mime type|

[Bug 22312] radeon 0000:01:00.0: Invalid ROM contents

2013-12-11 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=22312 --- Comment #6 from Alex Deucher --- (In reply to Dnitry from comment #5) > Created attachment 118051 [details] > uname -a, lspci -v -s 01:00.0, dmesg > > Yes, I still have this issue. It doesn't manifest itself. Everything works > fine except

[PULL] drm-intel-fixes

2013-12-11 Thread Daniel Vetter
Hi Dave, Just a bunch of regression fixes plus a few patches for long-standing issues in gem corner-cases that we've hunted down in the past weeks. Since apparently people hit those in the wild (and we also have nice igts for them) I've opted for -fixes and cc: stable. There's 1-2 things

TTM's role in score-based eviction

2013-12-11 Thread Lauri Kasanen
On Wed, 11 Dec 2013 15:46:53 +0100 Thomas Hellstrom wrote: > > I think the kernel just has to trust userspace on this. I can't think > > of any way of not involving userspace, so if somebody really wants to > > hack mesa to gain some fps advantage on a multiseat system, let them ;) > > > >

[Bug 72602] New: [REGRESSION] r600 cayman: forced performance level gets reset to auto when playing a video with vdpau

2013-12-11 Thread bugzilla-dae...@freedesktop.org
gnee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/6bb2ad8b/attachment.html>

[Bug 72602] [REGRESSION] r600 cayman: forced performance level gets reset to auto when playing a video with vdpau

2013-12-11 Thread bugzilla-dae...@freedesktop.org
was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/f3a14d1e/attachment.html>

[PATCH] drm/radeon: add missing display tiling setup for oland

2013-12-11 Thread Alex Deucher
Fixes improperly set up display params for 2D tiling on oland. Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- drivers/gpu/drm/radeon/atombios_crtc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/atombios_crtc.c

[Bug 35457] [rs690m] Graphics corruption with ati x1200

2013-12-11 Thread bugzilla-dae...@freedesktop.org
am. -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/90b2544e/attachment.html>

[Bug 72602] [REGRESSION] r600 cayman: forced performance level gets reset to auto when playing a video with vdpau

2013-12-11 Thread bugzilla-dae...@freedesktop.org
because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/3b7d4d24/attachment-0001.html>

[Nouveau] Fixing nouveau for >4k PAGE_SIZE

2013-12-11 Thread Patrick Baggett
> > *size = roundup(*size, PAGE_SIZE); > > } > > > > @@ -221,7 +221,7 @@ nouveau_bo_new(struct drm_device *dev, int size, int > align, > > nvbo->page_shift = 12; > > if (drm->client.base.vm) { > > if (!(flags & TTM_PL_FLAG_TT) && size > 256 * 1024) > > - nvbo->page_shift = > drm->client.base.vm->vmm->lpg_shift; > > + nvbo->page_shift = lpg_shift; > > } > Ack both hunks. > > > > > nouveau_bo_fixup_align(nvbo, flags, , ); > > diff --git a/drivers/gpu/drm/nouveau/nouveau_sgdma.c > b/drivers/gpu/drm/nouveau/nouveau_sgdma.c > > index 0843ebc..f255ff8 100644 > > --- a/drivers/gpu/drm/nouveau/nouveau_sgdma.c > > +++ b/drivers/gpu/drm/nouveau/nouveau_sgdma.c > > @@ -31,7 +31,7 @@ nv04_sgdma_bind(struct ttm_tt *ttm, struct ttm_mem_reg > *mem) > > { > > struct nouveau_sgdma_be *nvbe = (struct nouveau_sgdma_be *)ttm; > > struct nouveau_mem *node = mem->mm_node; > > - u64 size = mem->num_pages << 12; > > + u64 size = mem->num_pages << PAGE_SHIFT; > Ack. However, a patch doing this already exists in the Nouveau kernel > tree ( > http://cgit.freedesktop.org/~darktama/nouveau/commit/?id=19ab15062b8fde70ff04784bd49abc330e92310d > ). > > > > > > if (ttm->sg) { > > node->sg = ttm->sg; > > diff --git a/drivers/gpu/drm/nouveau/nouveau_ttm.c > b/drivers/gpu/drm/nouveau/nouveau_ttm.c > > index 19e3757..b7fc456 100644 > > --- a/drivers/gpu/drm/nouveau/nouveau_ttm.c > > +++ b/drivers/gpu/drm/nouveau/nouveau_ttm.c > > @@ -252,8 +252,9 @@ nv04_gart_manager_new(struct ttm_mem_type_manager > *man, > > > > node->page_shift = 12; > > > > - ret = nouveau_vm_get(man->priv, mem->num_pages << 12, > node->page_shift, > > -NV_MEM_ACCESS_RW, >vma[0]); > > + ret = nouveau_vm_get(man->priv, mem->num_pages << PAGE_SHIFT, > > +node->page_shift, NV_MEM_ACCESS_RW, > > +>vma[0]); > Ack. > > > if (ret) { > > kfree(node); > > return ret; > > > > > ___ > Nouveau mailing list > Nouveau at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/nouveau > -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/a9ca2d87/attachment-0001.html>

[PATCH] of: Add simple panel device tree binding

2013-12-11 Thread Tomi Valkeinen
so on, so it would be easy to see if a particular panel fits into our simple panel spec. Tomi -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 901 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/9ce0e72b/attachment.pgp>

[Bug 72602] [REGRESSION] r600 cayman: forced performance level gets reset to auto when playing a video with vdpau

2013-12-11 Thread bugzilla-dae...@freedesktop.org
archives/dri-devel/attachments/20131211/19466503/attachment.html>

[Bug 72602] [REGRESSION] r600 cayman: forced performance level gets reset to auto when playing a video with vdpau

2013-12-11 Thread bugzilla-dae...@freedesktop.org
this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131211/61489062/attachment.html>

  1   2   >