Re: [PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Semwal, Sumit
On Fri, Jul 20, 2012 at 10:09 PM, Rob Clark wrote: > Fyi, Daniel Vetter had suggested on IRC that it would be cleaner to > have a single helper fxn that most-restrictive union of all attached > device's dma_parms. Really this should include dma_mask and > coherent_dma_mask, I think. But that tou

[PATCH] gpu/mfd/usb: Fix USB randconfig problems

2012-08-06 Thread Guenter Roeck
Fix config warning: warning: ( ... && DRM_USB) selects USB which has unmet direct dependencies (USB_SUPPORT && USB_ARCH_HAS_HCD) by adding the missing dependency on USB_ARCH_HAS_HCD to DRM_UDL and DRM_USB. This exposes: drivers/video/Kconfig:36:error: recursive dependency detected! drivers/video

Re: [RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-06 Thread Seth Forshee
On Sun, Aug 05, 2012 at 07:20:31PM -0400, Alex Deucher wrote: > On Sun, Aug 5, 2012 at 5:44 PM, Dave Airlie wrote: > > On Mon, Aug 6, 2012 at 7:40 AM, Daniel Vetter wrote: > >> On Sun, Aug 05, 2012 at 10:18:38PM +0100, Matthew Garrett wrote: > >>> On Sun, Aug 05, 2012 at 11:14:12PM +0200, Daniel

Re: [RFC PATCH 1/3] dma-fence: dma-buf synchronization (v5)

2012-08-06 Thread Sumit Semwal
Hi Maarten, On 27 July 2012 19:09, Maarten Lankhorst wrote: > A dma-fence can be attached to a buffer which is being filled or consumed > by hw, to allow userspace to pass the buffer without waiting to another > device. For example, userspace can call page_flip ioctl to display the > next frame o

[PATCH 0/5] drm/exynos: Use devm_* functions and cleanup

2012-08-06 Thread Sachin Kamat
This series adds devm_* functionality in some drivers and cleans up some code that remained when the same was added to other drivers earlier. This series is compile tested against the latest (6 Aug 2012) linux-next code. Sachin Kamat (5): drm/exynos: Remove redundant check in exynos_hdmi.c file

[PATCH 1/5] drm/exynos: Remove redundant check in exynos_hdmi.c file

2012-08-06 Thread Sachin Kamat
devm_request_and_ioremap function checks the validity of the pointer returned by platform_get_resource. Hence an additional check in the probe function is not necessary. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_hdmi.c |5 - 1 files changed, 0 insertions(+), 5 deletio

[PATCH 2/5] drm/exynos: Remove redundant check in exynos_drm_fimd.c file

2012-08-06 Thread Sachin Kamat
devm_request_and_ioremap function checks the validity of the pointer returned by platform_get_resource. Hence an additional check in the probe function is not necessary. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_fimd.c |5 - 1 files changed, 0 insertions(+), 5 del

[PATCH 3/5] drm/exynos: Use devm_kzalloc in exynos_drm_vidi.c file

2012-08-06 Thread Sachin Kamat
devm_kzalloc is a device managed function and makes freeing and error handling simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_vidi.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c b/drivers/gpu/drm

[PATCH 4/5] drm/exynos: Use devm_kzalloc in exynos_drm_hdmi.c file

2012-08-06 Thread Sachin Kamat
devm_kzalloc is a device managed function and makes freeing and error handling simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_hdmi.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_hdmi.c b/drivers/gpu/drm/

[PATCH 5/5] drm/exynos: Use devm_* functions in exynos_drm_g2d.c file

2012-08-06 Thread Sachin Kamat
devm_* functions are device managed functions and make error handling and cleanup cleaner and simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_g2d.c | 50 ++ 1 files changed, 10 insertions(+), 40 deletions(-) diff --git a/drivers/gpu/drm/e

Re: [RFC PATCH 1/3] dma-fence: dma-buf synchronization (v5)

2012-08-06 Thread Maarten Lankhorst
Hey Sumit, Op 06-08-12 08:41, Sumit Semwal schreef: > Hi Maarten, > On 27 July 2012 19:09, Maarten Lankhorst > wrote: >> A dma-fence can be attached to a buffer which is being filled or consumed >> by hw, to allow userspace to pass the buffer without waiting to another >> device. For example, us

Re: [PATCH] vmwgfx: add missing mutex_unlocks

2012-08-06 Thread Thomas Hellstrom
NAK, The mutex unlock is done in vmw_fifo_commit. The mutex is protecting the fifo memory against multiple simultaneous reservers. /Thomas On 08/04/2012 07:33 AM, Devendra Naga wrote: we have done a proper mutex_unlock in the error cases of the vmw_fifo_reserve, but there are missing mute

[Bug 53122] X lockups /

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53122 --- Comment #4 from #Paul <201208bugzil...@moo.uklinux.net> 2012-08-06 09:07:01 UTC --- Created attachment 65154 --> https://bugs.freedesktop.org/attachment.cgi?id=65154 /var/log/messages (dmesg) -- Configure bugmail: https://bugs.freedesktop

Re: [PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Tomasz Stanislawski
Hi Rob, On 07/19/2012 06:23 PM, Rob Clark wrote: > From: Rob Clark > > Add some helpers to iterate through all attachers and get the most > restrictive segment size/count/boundary. > > Signed-off-by: Rob Clark > --- > drivers/base/dma-buf.c | 63 > +

Re: [PATCH] vmwgfx: add missing mutex_unlocks

2012-08-06 Thread Devendra Naga
Hi Thomas, On Mon, Aug 6, 2012 at 1:52 PM, Thomas Hellstrom wrote: > NAK, > > The mutex unlock is done in vmw_fifo_commit. The mutex is protecting the > fifo memory against multiple simultaneous reservers. > Thanks for the answer :-). sorry for your time... i will definitely try more reading and

Re: [RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-06 Thread Matthew Garrett
On Sun, Aug 05, 2012 at 11:40:16PM +0200, Daniel Vetter wrote: > As long as it's only apple shipping multi-gpu machines with > broken/non-existing vbt, I'll happily stomach the quirk list entries. > They're bad, but imo the lesser evil. Doing this via quirks means that we'll always be broken on t

Re: [PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Tomasz Stanislawski
On 08/06/2012 01:58 PM, Michal Nazarewicz wrote: > > Tomasz Stanislawski writes: >> I recommend to change the semantics for unlimited number of segments >> from 'value 0' to: >> >> #define DMA_SEGMENTS_COUNT_UNLIMITED ((unsigned long)INT_MAX) Sorry. It should be: #define DMA_SEGMENTS_COUNT_UNLIM

[Bug 53109] egl_gallium fails to link if both r600 and radeonsi are enabled

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53109 Michel Dänzer changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Bug 50389] [radeonsi] Unknown type of argument: ../../../../i386-linux-gnu/egl/egl_gallium.so

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50389 Michel Dänzer changed: What|Removed |Added CC||freedesk...@blino.org --- Comment #2 fro

[Bug 52997] evergreen_cs_track_validate_cb:477 cb[0] bo too small when launching ds2 in wine

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=52997 Michel Dänzer changed: What|Removed |Added Attachment #65141|text/x-log |text/plain mime type|

Re: [PATCH v2 7/7] drm: Renesas SH Mobile DRM driver

2012-08-06 Thread Laurent Pinchart
Hi Sascha, Thanks for the review. > On Fri, Jul 20, 2012 at 03:04:22PM +0200, Laurent Pinchart wrote: > > The SH Mobile LCD controller (LCDC) DRM driver supports the main > > graphics plane in RGB and YUV formats, as well as the overlay planes (in > > alpha-blending mode only). > > > > Only flat

[Bug 53111] [bisected] lockups since added support for virtual address space on cayman v11

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53111 --- Comment #3 from Michel Dänzer 2012-08-06 13:27:48 UTC --- FWIW, r600.tests should no longer be used in favour of quick-driver.tests. I assume it still happens with the latter though, so here's some debugging tips: For isolating a single pig

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Michel Dänzer changed: What|Removed |Added AssignedTo|dri-devel@lists.freedesktop |dan...@ffwll.ch |.org

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Michel Dänzer changed: What|Removed |Added AssignedTo|dan...@ffwll.ch |dri-devel@lists.freedesktop

Re: [PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Michal Nazarewicz
Tomasz Stanislawski writes: > I recommend to change the semantics for unlimited number of segments > from 'value 0' to: > > #define DMA_SEGMENTS_COUNT_UNLIMITED ((unsigned long)INT_MAX) > > Using INT_MAX will allow using safe conversions between signed and > unsigned integers. LONG_MAX seems cle

[PATCH] drm/radeon: add some new SI pci ids

2012-08-06 Thread alexdeucher
From: Alex Deucher Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- include/drm/drm_pciids.h |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 7ff5c99..c78bb99 100644 --- a/include/drm/drm_pciids.h +++

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Bug 53111] [bisected] lockups since added support for virtual address space on cayman v11

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53111 --- Comment #4 from Alexandre Demers 2012-08-06 14:42:39 UTC --- (In reply to comment #3) > FWIW, r600.tests should no longer be used in favour of quick-driver.tests. I > assume it still happens with the latter though, so here's some debugging t

Re: [PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Christian König
On 04.08.2012 11:07, Christian König wrote: On 03.08.2012 23:26, j.gli...@gmail.com wrote: From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. Kernel 3.5/3.4 need a simila

Re: [PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Jerome Glisse
On Mon, Aug 6, 2012 at 12:06 PM, Christian König wrote: > On 04.08.2012 11:07, Christian König wrote: >> >> On 03.08.2012 23:26, j.gli...@gmail.com wrote: >>> >>> From: Jerome Glisse >>> >>> Virtual address need to be fenced to know when we can safely remove it. >>> This patch also properly clear

[PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v3

2012-08-06 Thread j . glisse
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. v2: For to update pagetable when unbinding bo (don't bailout if bo_va->valid is true). v3: Fix compilation warnings T

[PATCH] drm/radeon: fence virtual address and free it once idle v4

2012-08-06 Thread j . glisse
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. Kernel 3.5/3.4 need a similar patch but adapted for difference in mutex locking. v2: For to update pagetable when unbindi

Re: [PATCH] drm/radeon: fence virtual address and free it once idle v4

2012-08-06 Thread Alex Deucher
On Mon, Aug 6, 2012 at 12:32 PM, wrote: > From: Jerome Glisse > > Virtual address need to be fenced to know when we can safely remove it. > This patch also properly clear the pagetable. Previously it was > serouisly broken. > > Kernel 3.5/3.4 need a similar patch but adapted for difference in mu

Re: [PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Christian König
On 06.08.2012 18:30, Jerome Glisse wrote: On Mon, Aug 6, 2012 at 12:06 PM, Christian König wrote: [SNIP] Additional to that patch we still need a minor fix to mesa (just move freeing the VM range after closing the handle). Going to send that in the next minute to the mesa-dev list. Otherwise t

[PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v4

2012-08-06 Thread j . glisse
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. v2: For to update pagetable when unbinding bo (don't bailout if bo_va->valid is true). v3: Fix compilation warnings v4

Re: [PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v3

2012-08-06 Thread Greg KH
On Mon, Aug 06, 2012 at 12:30:22PM -0400, j.gli...@gmail.com wrote: > From: Jerome Glisse > > Virtual address need to be fenced to know when we can safely remove it. > This patch also properly clear the pagetable. Previously it was > serouisly broken. > > v2: For to update pagetable when unbindi

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Steven Fuerst
Replace i2f() in r600_blit.c with an optimized version. We use __fls() to find the most significant bit. Using that, the loop can be avoided. A second trick is to use the mod(32) behaviour of the rotate instructions on x86 to expand the range of the unsigned int to float conversion to the full 3

PATCH 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

2012-08-06 Thread Steven Fuerst
Replace i2f() in r600_blit_kms.c with an optimized version. We use __fls() to find the most significant bit. Using that, the loop can be avoided. A second trick is to use the mod(32) behaviour of the rotate instructions on x86 to expand the range of the unsigned int to float conversion to the fu

[PATCH 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

2012-08-06 Thread Steven Fuerst
Rename i2f() to int2float(), and make it global so one copy can be removed. Remove the copy of i2f() in r600_blit_kms.c We rename the function to something longer now that it is a global symbol. This reduces the likelyhood of unintended clashes later. This might be a candidate for inclusion insi

Re: [bisected] nouveau: "Failed to idle channel x" after resume

2012-08-06 Thread Maxim Levitsky
On Sat, 2012-08-04 at 17:41 +0300, Maxim Levitsky wrote: > On Mon, 2012-07-23 at 18:25 +0300, Aioanei Rares wrote: > > On Thu, Jul 5, 2012 at 11:24 PM, Martin Nyhus wrote: > > > > > > On Mon, 11 Jun 2012 23:18:42 +0200 Martin Nyhus wrote: > > > > after resuming from suspend nouveau starts writin

[PATCH] drm/radeon: fix ordering in pll picking on dce4+

2012-08-06 Thread alexdeucher
From: Alex Deucher No functional change, but re-order the cases so they evaluate properly due to the way the DCE macros work. Noticed by kallisti5 on IRC. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/atombios_crtc.c |8 1 files changed, 4 insertions(+), 4 deletions(-)

[Bug 51383] 'make clean' broken after switching to automake

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51383 --- Comment #2 from Tobias Droste 2012-08-06 21:16:56 UTC --- I'm not sure if this is realy true. Looking at the "problematic" .o files there is "src/mesa/drivers/dri/common/utils.o" metioned, which is completly handled by automake IIRC. I test

Re: [PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Paul Menzel
Dear Steven, thank you for your patches. Am Montag, den 06.08.2012, 11:14 -0700 schrieb Steven Fuerst: > Replace i2f() in r600_blit.c with an optimized version. > > We use __fls() to find the most significant bit. Using that, the > loop can be avoided. A second trick is to use the mod(32) > b

Re: [PATCH 1/3] drm/i915: implement dma buf begin_cpu_access

2012-08-06 Thread Dave Airlie
On Mon, Aug 6, 2012 at 6:52 AM, Daniel Vetter wrote: > On Mon, Jul 30, 2012 at 02:06:54PM +1000, Dave Airlie wrote: >> From: Dave Airlie >> >> In order for udl vmap to work properly, we need to push the object >> into the CPU domain before we start copying the data to the USB device. >> >> questi

[Bug 53111] [bisected] lockups since added support for virtual address space on cayman v11

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53111 --- Comment #5 from Alexandre Demers 2012-08-07 01:21:27 UTC --- Tested running one piglit test at a time (thanks Michel) and it always locks on "texturing/depthstencil-render-miplevels 146 s=z24_s8_d=z32f_s8". It locks hard, resets, stays locke

[Bug 53130] 99c65ba breaks rendering (flickery, eventual fail)

2012-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53130 Darren Salt changed: What|Removed |Added Keywords||regression -- Configure bugmail: https://

[PATCH] drm/i915: make forcewake work again

2012-08-06 Thread Ben Widawsky
While trying to track down the power regression, I noticed that on my SNB I had more severe problems, ie. forcewake seemed to never happen once i915 was loaded. After a bit of bisection, I tracked the bad commit to: commit 7b0cfee1a24efdfe0235bac62e53f686fe8a8e24 Merge: 9756fe3 6b16351 Author: Dan

[RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-06 Thread Dave Airlie
On Mon, Aug 6, 2012 at 7:40 AM, Daniel Vetter wrote: > On Sun, Aug 05, 2012 at 10:18:38PM +0100, Matthew Garrett wrote: >> On Sun, Aug 05, 2012 at 11:14:12PM +0200, Daniel Vetter wrote: >> >> > I like this approach more - the only other solution I see is to ask the >> > currently active driver (i.

[PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Semwal, Sumit
On Fri, Jul 20, 2012 at 10:09 PM, Rob Clark wrote: > Fyi, Daniel Vetter had suggested on IRC that it would be cleaner to > have a single helper fxn that most-restrictive union of all attached > device's dma_parms. Really this should include dma_mask and > coherent_dma_mask, I think. But that tou

[RFC PATCH 1/3] dma-fence: dma-buf synchronization (v5)

2012-08-06 Thread Sumit Semwal
Hi Maarten, On 27 July 2012 19:09, Maarten Lankhorst wrote: > A dma-fence can be attached to a buffer which is being filled or consumed > by hw, to allow userspace to pass the buffer without waiting to another > device. For example, userspace can call page_flip ioctl to display the > next frame o

[PATCH 0/5] drm/exynos: Use devm_* functions and cleanup

2012-08-06 Thread Sachin Kamat
This series adds devm_* functionality in some drivers and cleans up some code that remained when the same was added to other drivers earlier. This series is compile tested against the latest (6 Aug 2012) linux-next code. Sachin Kamat (5): drm/exynos: Remove redundant check in exynos_hdmi.c file

[PATCH 1/5] drm/exynos: Remove redundant check in exynos_hdmi.c file

2012-08-06 Thread Sachin Kamat
devm_request_and_ioremap function checks the validity of the pointer returned by platform_get_resource. Hence an additional check in the probe function is not necessary. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_hdmi.c |5 - 1 files changed, 0 insertions(+), 5 deletio

[PATCH 2/5] drm/exynos: Remove redundant check in exynos_drm_fimd.c file

2012-08-06 Thread Sachin Kamat
devm_request_and_ioremap function checks the validity of the pointer returned by platform_get_resource. Hence an additional check in the probe function is not necessary. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_fimd.c |5 - 1 files changed, 0 insertions(+), 5 del

[PATCH 3/5] drm/exynos: Use devm_kzalloc in exynos_drm_vidi.c file

2012-08-06 Thread Sachin Kamat
devm_kzalloc is a device managed function and makes freeing and error handling simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_vidi.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c b/drivers/gpu/drm

[PATCH 4/5] drm/exynos: Use devm_kzalloc in exynos_drm_hdmi.c file

2012-08-06 Thread Sachin Kamat
devm_kzalloc is a device managed function and makes freeing and error handling simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_hdmi.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_hdmi.c b/drivers/gpu/drm/

[PATCH 5/5] drm/exynos: Use devm_* functions in exynos_drm_g2d.c file

2012-08-06 Thread Sachin Kamat
devm_* functions are device managed functions and make error handling and cleanup cleaner and simpler. Signed-off-by: Sachin Kamat --- drivers/gpu/drm/exynos/exynos_drm_g2d.c | 50 ++ 1 files changed, 10 insertions(+), 40 deletions(-) diff --git a/drivers/gpu/drm/e

[RFC PATCH 1/3] dma-fence: dma-buf synchronization (v5)

2012-08-06 Thread Maarten Lankhorst
Hey Sumit, Op 06-08-12 08:41, Sumit Semwal schreef: > Hi Maarten, > On 27 July 2012 19:09, Maarten Lankhorst > wrote: >> A dma-fence can be attached to a buffer which is being filled or consumed >> by hw, to allow userspace to pass the buffer without waiting to another >> device. For example, us

[PATCH] vmwgfx: add missing mutex_unlocks

2012-08-06 Thread Thomas Hellstrom
NAK, The mutex unlock is done in vmw_fifo_commit. The mutex is protecting the fifo memory against multiple simultaneous reservers. /Thomas On 08/04/2012 07:33 AM, Devendra Naga wrote: > we have done a proper mutex_unlock in the error cases of the > vmw_fifo_reserve, but > there are missing m

[Bug 53122] X lockups /

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53122 --- Comment #4 from #Paul <201208bugzillaz at moo.uklinux.net> 2012-08-06 09:07:01 UTC --- Created attachment 65154 --> https://bugs.freedesktop.org/attachment.cgi?id=65154 /var/log/messages (dmesg) -- Configure bugmail: https://bugs.freedesk

[PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Tomasz Stanislawski
Hi Rob, On 07/19/2012 06:23 PM, Rob Clark wrote: > From: Rob Clark > > Add some helpers to iterate through all attachers and get the most > restrictive segment size/count/boundary. > > Signed-off-by: Rob Clark > --- > drivers/base/dma-buf.c | 63 > +

[PATCH] vmwgfx: add missing mutex_unlocks

2012-08-06 Thread Devendra Naga
Hi Thomas, On Mon, Aug 6, 2012 at 1:52 PM, Thomas Hellstrom wrote: > NAK, > > The mutex unlock is done in vmw_fifo_commit. The mutex is protecting the > fifo memory against multiple simultaneous reservers. > Thanks for the answer :-). sorry for your time... i will definitely try more reading an

[RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-06 Thread Matthew Garrett
On Sun, Aug 05, 2012 at 11:40:16PM +0200, Daniel Vetter wrote: > As long as it's only apple shipping multi-gpu machines with > broken/non-existing vbt, I'll happily stomach the quirk list entries. > They're bad, but imo the lesser evil. Doing this via quirks means that we'll always be broken on t

[PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Tomasz Stanislawski
On 08/06/2012 01:58 PM, Michal Nazarewicz wrote: > > Tomasz Stanislawski writes: >> I recommend to change the semantics for unlimited number of segments >> from 'value 0' to: >> >> #define DMA_SEGMENTS_COUNT_UNLIMITED ((unsigned long)INT_MAX) Sorry. It should be: #define DMA_SEGMENTS_COUNT_UNLIM

[Bug 53109] egl_gallium fails to link if both r600 and radeonsi are enabled

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53109 Michel D?nzer changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Bug 50389] [radeonsi] Unknown type of argument: ../../../../i386-linux-gnu/egl/egl_gallium.so

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=50389 Michel D?nzer changed: What|Removed |Added CC||freedesktop at blino.org --- Comment #2

[Bug 52997] evergreen_cs_track_validate_cb:477 cb[0] bo too small when launching ds2 in wine

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=52997 Michel D?nzer changed: What|Removed |Added Attachment #65141|text/x-log |text/plain mime type|

[PATCH v2 7/7] drm: Renesas SH Mobile DRM driver

2012-08-06 Thread Laurent Pinchart
Hi Sascha, Thanks for the review. > On Fri, Jul 20, 2012 at 03:04:22PM +0200, Laurent Pinchart wrote: > > The SH Mobile LCD controller (LCDC) DRM driver supports the main > > graphics plane in RGB and YUV formats, as well as the overlay planes (in > > alpha-blending mode only). > > > > Only flat

[Bug 53111] [bisected] lockups since added support for virtual address space on cayman v11

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53111 --- Comment #3 from Michel D?nzer 2012-08-06 13:27:48 UTC --- FWIW, r600.tests should no longer be used in favour of quick-driver.tests. I assume it still happens with the latter though, so here's some debugging tips: For isolating a single pig

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Michel D?nzer changed: What|Removed |Added AssignedTo|dri-devel at lists.freedesktop |daniel at ffwll.ch

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Michel D?nzer changed: What|Removed |Added AssignedTo|daniel at ffwll.ch |dri-devel at lists.freedesktop

[PATCH 2/2] dma-buf: add helpers for attacher dma-parms

2012-08-06 Thread Michal Nazarewicz
next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120806/977107ce/attachment.pgp>

[PATCH] drm/radeon: add some new SI pci ids

2012-08-06 Thread alexdeuc...@gmail.com
From: Alex Deucher Signed-off-by: Alex Deucher Cc: stable at vger.kernel.org --- include/drm/drm_pciids.h |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/include/drm/drm_pciids.h b/include/drm/drm_pciids.h index 7ff5c99..c78bb99 100644 --- a/include/drm/drm_pciids.h +

[Bug 53124] Missing Radeon R600 PCI IDs

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53124 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Bug 53111] [bisected] lockups since added support for virtual address space on cayman v11

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=53111 --- Comment #4 from Alexandre Demers 2012-08-06 14:42:39 UTC --- (In reply to comment #3) > FWIW, r600.tests should no longer be used in favour of quick-driver.tests. I > assume it still happens with the latter though, so here's some debugging t

[PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Christian König
On 04.08.2012 11:07, Christian K?nig wrote: > On 03.08.2012 23:26, j.glisse at gmail.com wrote: >> From: Jerome Glisse >> >> Virtual address need to be fenced to know when we can safely remove it. >> This patch also properly clear the pagetable. Previously it was >> serouisly broken. >> >> Kernel

[PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Jerome Glisse
On Mon, Aug 6, 2012 at 12:06 PM, Christian K?nig wrote: > On 04.08.2012 11:07, Christian K?nig wrote: >> >> On 03.08.2012 23:26, j.glisse at gmail.com wrote: >>> >>> From: Jerome Glisse >>> >>> Virtual address need to be fenced to know when we can safely remove it. >>> This patch also properly cl

[PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v3

2012-08-06 Thread j.gli...@gmail.com
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. v2: For to update pagetable when unbinding bo (don't bailout if bo_va->valid is true). v3: Fix compilation warnings T

[PATCH] drm/radeon: fence virtual address and free it once idle v4

2012-08-06 Thread j.gli...@gmail.com
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. Kernel 3.5/3.4 need a similar patch but adapted for difference in mutex locking. v2: For to update pagetable when unbindi

[PATCH] drm/radeon: fence virtual address and free it once idle v4

2012-08-06 Thread Alex Deucher
On Mon, Aug 6, 2012 at 12:32 PM, wrote: > From: Jerome Glisse > > Virtual address need to be fenced to know when we can safely remove it. > This patch also properly clear the pagetable. Previously it was > serouisly broken. > > Kernel 3.5/3.4 need a similar patch but adapted for difference in mu

[PATCH] drm/radeon: fence virtual address and free it once idle v3

2012-08-06 Thread Christian König
On 06.08.2012 18:30, Jerome Glisse wrote: > On Mon, Aug 6, 2012 at 12:06 PM, Christian K?nig > wrote: >> [SNIP] >> Additional to that patch we still need a minor fix to mesa (just move >> freeing the VM range after closing the handle). Going to send that in the >> next minute to the mesa-dev list.

[PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v4

2012-08-06 Thread j.gli...@gmail.com
From: Jerome Glisse Virtual address need to be fenced to know when we can safely remove it. This patch also properly clear the pagetable. Previously it was serouisly broken. v2: For to update pagetable when unbinding bo (don't bailout if bo_va->valid is true). v3: Fix compilation warnings v4

[PATCH] drm/radeon: fence virtual address and free it once idle [3.5] v3

2012-08-06 Thread Greg KH
On Mon, Aug 06, 2012 at 12:30:22PM -0400, j.glisse at gmail.com wrote: > From: Jerome Glisse > > Virtual address need to be fenced to know when we can safely remove it. > This patch also properly clear the pagetable. Previously it was > serouisly broken. > > v2: For to update pagetable when unbi

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Steven Fuerst
. + */ + fraction = ror32(x, msb - I2F_FRAC_BITS) & I2F_MASK; + exponent = (127 + msb) << I2F_FRAC_BITS; + + return fraction + exponent; } -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120806/2251f723/attachment-0001.html>

PATCH 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

2012-08-06 Thread Steven Fuerst
*/ + fraction = ror32(x, msb - I2F_FRAC_BITS) & I2F_MASK; + exponent = (127 + msb) << I2F_FRAC_BITS; + + return fraction + exponent; } int r600_blit_init(struct radeon_device *rdev) -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedeskto

[PATCH 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

2012-08-06 Thread Steven Fuerst
_size, r7xx_default_size; +uint32_t int2float(uint32_t x); #endif -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120806/059d2927/attachment.html>

[bisected] nouveau: "Failed to idle channel x" after resume

2012-08-06 Thread Maxim Levitsky
On Sat, 2012-08-04 at 17:41 +0300, Maxim Levitsky wrote: > On Mon, 2012-07-23 at 18:25 +0300, Aioanei Rares wrote: > > On Thu, Jul 5, 2012 at 11:24 PM, Martin Nyhus > > wrote: > > > > > > On Mon, 11 Jun 2012 23:18:42 +0200 Martin Nyhus wrote: > > > > after resuming from suspend nouveau starts w

[PATCH] drm/radeon: fix ordering in pll picking on dce4+

2012-08-06 Thread alexdeuc...@gmail.com
From: Alex Deucher No functional change, but re-order the cases so they evaluate properly due to the way the DCE macros work. Noticed by kallisti5 on IRC. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/atombios_crtc.c |8 1 files changed, 4 insertions(+), 4 deletions(-)

[Bug 51383] 'make clean' broken after switching to automake

2012-08-06 Thread bugzilla-dae...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=51383 --- Comment #2 from Tobias Droste 2012-08-06 21:16:56 UTC --- I'm not sure if this is realy true. Looking at the "problematic" .o files there is "src/mesa/drivers/dri/common/utils.o" metioned, which is completly handled by automake IIRC. I test

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Paul Menzel
Thanks, Paul [1] http://en.opensuse.org/openSUSE:Mailing_list_netiquette -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120806/3997abee/attachment.pgp>

[PATCH] drm/i915: make forcewake work again

2012-08-06 Thread Ben Widawsky
While trying to track down the power regression, I noticed that on my SNB I had more severe problems, ie. forcewake seemed to never happen once i915 was loaded. After a bit of bisection, I tracked the bad commit to: commit 7b0cfee1a24efdfe0235bac62e53f686fe8a8e24 Merge: 9756fe3 6b16351 Author: Dan

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Steven Fuerst
> Unfortunately you sent your message not just as plain text [1] and the > Google Mail mailer automatically wrapped the lines and there mangled the > patch. Oops, sorry about that. New patches incoming via git-send-email. No changes other than the unwanted line-wrapping is hopefully avoided. St

[[PATCH][RESENT] 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

2012-08-06 Thread Steven Fuerst
Remove the copy of i2f() in r600_blit_kms.c We rename the function to something longer now that it is a global symbol. This reduces the likelyhood of unintended clashes later. This might be a candidate for inclusion inside general drm infrastructure. However, at the moment only the radeon driver

[RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-06 Thread Seth Forshee
On Mon, Aug 06, 2012 at 01:23:17PM +0100, Matthew Garrett wrote: > On Sun, Aug 05, 2012 at 11:40:16PM +0200, Daniel Vetter wrote: > > > As long as it's only apple shipping multi-gpu machines with > > broken/non-existing vbt, I'll happily stomach the quirk list entries. > > They're bad, but imo the

[[PATCH][RESENT] 1/3] Replace i2f() in r600_blit.c with an optimized version.

2012-08-06 Thread Steven Fuerst
We use __fls() to find the most significant bit. Using that, the loop can be avoided. A second trick is to use the mod(32) behaviour of the rotate instructions on x86 to expand the range of the unsigned int to float conversion to the full 32 bits. The routine is now exact up to 2^24. Above that

[[PATCH][RESENT] 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

2012-08-06 Thread Steven Fuerst
We use __fls() to find the most significant bit. Using that, the loop can be avoided. A second trick is to use the mod(32) behaviour of the rotate instructions on x86 to expand the range of the unsigned int to float conversion to the full 32 bits. The routine is now exact up to 2^24. Above that

[PATCH 4/3] Annotate int2float() as being a pure function.

2012-08-06 Thread Steven Fuerst
This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +- 2 files changed,