[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>

[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

[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] 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

[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/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.

[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

[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: 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(-)

[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,

[[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

[[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][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 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

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

[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 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

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

[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 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 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 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

[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(-)

[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] 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

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

[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

[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 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

[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] 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

[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|

[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

[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 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/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] 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 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 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

[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 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 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 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 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

[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 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 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

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 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 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>

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 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>

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] 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

[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 +

[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] 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 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] 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

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

[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

[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

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

[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

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

[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.

[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

[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|

[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 +++

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

[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

[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 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

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 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|

[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 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|

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

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] 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: [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 > +

[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] 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

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

[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

[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 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 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 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 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

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

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

[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: [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