[PATCHv10 21/26] v4l: vb2-dma-contig: add reference counting for a device from allocator context

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, Thanks for the patch. On Wednesday 10 October 2012 16:46:40 Tomasz Stanislawski wrote: > This patch adds taking reference to the device for MMAP buffers. > > Such buffers, may be exported using DMABUF mechanism. If the driver that > created a queue is unloaded then the queue is releas

[PATCHv10 20/26] v4l: vb2-dma-contig: add support for DMABUF exporting

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, On Wednesday 10 October 2012 16:46:39 Tomasz Stanislawski wrote: > This patch adds support for exporting a dma-contig buffer using > DMABUF interface. > > Signed-off-by: Tomasz Stanislawski > Signed-off-by: Kyungmin Park > Acked-by: Hans Verkuil Acked-by: Laurent Pinchart > --- >

[PATCHv10 22/26] v4l: vb2-dma-contig: fail if user ptr buffer is not correctly aligned

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, Thanks for the patch. On Wednesday 10 October 2012 16:46:41 Tomasz Stanislawski wrote: > From: Marek Szyprowski > > The DMA transfer must be aligned to a specific value. If userptr is not > aligned to DMA requirements then unexpected corruptions of the memory may > occur before or af

[PATCHv10 23/26] v4l: vb2-dma-contig: align buffer size to PAGE_SIZE

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, On Wednesday 10 October 2012 16:46:42 Tomasz Stanislawski wrote: > Most operations on DMA and DMABUF framework need page > aligned buffers. The comment is a bit misleading, the buffer is already page-aligned (unless I'm mistaken dma_alloc_coherent() returns a page-aligned buffer) but

Re: [PATCHv10 21/26] v4l: vb2-dma-contig: add reference counting for a device from allocator context

2012-10-11 Thread Tomasz Stanislawski
Hi Laurent, Thank your your review. On 10/11/2012 11:49 PM, Laurent Pinchart wrote: > Hi Tomasz, > > Thanks for the patch. > > On Wednesday 10 October 2012 16:46:40 Tomasz Stanislawski wrote: >> This patch adds taking reference to the device for MMAP buffers. >> >> Such buffers, may be exported

[Bug 26891] Radeon KMS fails with inaccessible AtomBIOS on systems with (U)EFI boot

2012-10-11 Thread bugzilla-dae...@freedesktop.org
- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/7bac5fbc/attachment.html>

[Bug 30620] [RADEON:KMS:RS100:MODESET] no display on VGA

2012-10-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=30620 --- Comment #12 from aux...@gmail.com --- Created attachment 68470 --> https://bugs.freedesktop.org/attachment.cgi?id=68470&action=edit diff -u regs-fail regs-working I just tried installing openSUSE 12.1 on an old compaq presario 900 with a br

[PATCH] drm/omap: Fix release of refill engine

2012-10-11 Thread Andy Gross
During asynchronous refills, we don't wait for the refill to finish. However, we cannot release the engine back to the idle list until it has actually completed the refill operation. The engine release will now be done in the IRQ handler, but only for asynchronous refill operations. Synchronous

[PATCH] drm/omap: Use writecombine for descriptors

2012-10-11 Thread Andy Gross
Use writecombine for descriptor and PAT programming memory. Signed-off-by: Andy Gross --- drivers/staging/omapdrm/omap_dmm_tiler.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/omapdrm/omap_dmm_tiler.c b/drivers/staging/omapdrm/omap_dmm_tiler.c inde

[PATCH] drm/omap: Remove shadow lut usage

2012-10-11 Thread Andy Gross
Removing extraneous shadow lut maintenance. There is no need for this to be in place until power management is added to the driver, and this extra copy degrades performance for no gain. Signed-off-by: Andy Gross --- drivers/staging/omapdrm/omap_dmm_priv.h |6 -- drivers/staging/omapdrm

Re: buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 10:55 PM, Maarten Lankhorst wrote: Op 11-10-12 21:26, Thomas Hellstrom schreef: On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: Anyway, if you plan to remove the fence lock and protect it with reserve, you must make sure that a waiting reserve is never done in a destruction pat

buggy/weird behavior in ttm

2012-10-11 Thread Maarten Lankhorst
Op 11-10-12 21:26, Thomas Hellstrom schreef: > On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: > >> >>> Anyway, if you plan to remove the fence lock and protect it with reserve, >>> you must >>> make sure that a waiting reserve is never done in a destruction path. I >>> think this >>> mostly con

Re: pipe B assertion failure (expected off, current on)

2012-10-11 Thread Bjørn Mork
Daniel Vetter writes: > On Thu, Oct 11, 2012 at 4:13 PM, Bjørn Mork wrote: >> Let me know if you want ot have more details about these warnings. >> Otherwise I'll just keep ignoring them ;-) > > The drm-intel-fixes branch from > http://cgit.freedesktop.org/~danvet/drm-intel has a patch to fix som

Re: [PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Mauro Carvalho Chehab
Em Thu, 11 Oct 2012 08:47:15 -0500 Rob Clark escreveu: > On Thu, Oct 11, 2012 at 6:13 AM, Mauro Carvalho Chehab > wrote: > > Em Thu, 11 Oct 2012 09:20:12 +0200 > > Hans Verkuil escreveu: > > > >> > my understaning is > >> > that the drivers/media/ authors should also ack with this licensing > >

pipe B assertion failure (expected off, current on)

2012-10-11 Thread Bjørn Mork
Hello, from time to time, I see the following warning on resume: [ 7022.245347] PM: Syncing filesystems ... done. [ 7022.271923] PM: Preparing system for mem sleep [ 7022.324107] [ cut here ] [ 7022.324154] WARNING: at drivers/gpu/drm/i915/intel_display.c:1225 intel_crtc_

buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 09:26 PM, Thomas Hellstrom wrote: > On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: > I can't see how the waiting reserve in ttm_bo_cleanup_refs would cause > a deadlock, > because the buffer about to be reserved is always *last* in a > reservation sequence, and the > reservation

[PATCH 1/2 v6] of: add helper to parse display timings

2012-10-11 Thread Thierry Reding
was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/155b6671/attachment-0001.pgp>

[RFC 01/11] drm: add atomic fxns

2012-10-11 Thread Laurent Pinchart
Hi Rob, On Wednesday 12 September 2012 22:49:47 Rob Clark wrote: > From: Rob Clark > > The 'atomic' mechanism allows for multiple properties to be updated, > checked, and commited atomically. This will be the basis of atomic- > modeset and nuclear-pageflip. > > The basic flow is: > >state

buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: > >> Anyway, if you plan to remove the fence lock and protect it with reserve, >> you must >> make sure that a waiting reserve is never done in a destruction path. I >> think this >> mostly concerns the nvidia driver. > Well I don't think any lock

[PATCH] drm/omap: Fix release of refill engine

2012-10-11 Thread Andy Gross
During asynchronous refills, we don't wait for the refill to finish. However, we cannot release the engine back to the idle list until it has actually completed the refill operation. The engine release will now be done in the IRQ handler, but only for asynchronous refill operations. Synchronous

[PATCH] drm/omap: Remove shadow lut usage

2012-10-11 Thread Andy Gross
Removing extraneous shadow lut maintenance. There is no need for this to be in place until power management is added to the driver, and this extra copy degrades performance for no gain. Signed-off-by: Andy Gross --- drivers/staging/omapdrm/omap_dmm_priv.h |6 -- drivers/staging/omapdrm

[PATCH] drm/omap: Use writecombine for descriptors

2012-10-11 Thread Andy Gross
Use writecombine for descriptor and PAT programming memory. Signed-off-by: Andy Gross --- drivers/staging/omapdrm/omap_dmm_tiler.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/omapdrm/omap_dmm_tiler.c b/drivers/staging/omapdrm/omap_dmm_tiler.c inde

[PATCH 9/9] drm: remove legacy drm_connector_property fxns

2012-10-11 Thread Rob Clark
From: Rob Clark Replace references to and remove the connector property fxns, which have been superseded with the more general object property fxns: + drm_connector_attach_property -> drm_object_attach_property + drm_connector_property_set_value -> drm_object_property_set_value + drm_conne

[PATCH 8/9] drm/i2c: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/i2c/ch7006_drv.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/i2c/ch7006_drv.c b/drivers/gpu/drm/i2c/ch7006_drv.c index 599099f..b865d07 100644 --- a/drivers/gpu/drm/i2c/

[PATCH 7/9] drm/vmwgfx: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c |2 +- drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c index 070fb23..7

[PATCH 6/9] drm/udl: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/udl/udl_connector.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/udl/udl_connector.c b/drivers/gpu/drm/udl/udl_connector.c index d568aa4..4faf39c 100644 --- a/drivers/gpu/drm/udl/udl_connecto

[PATCH 5/9] drm/shmob: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c b/drivers/gpu/drm/shmobile/shmob_drm_crtc.c index 3ecb702..698b0cd 100644 --- a/drivers/gpu/dr

[PATCH 4/9] drm/radeon: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/radeon/radeon_connectors.c | 62 ++-- 1 file changed, 31 insertions(+), 31 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c index 895e628..ef

[PATCH 3/9] drm/nouveau: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/nouveau/nouveau_connector.c | 22 +++--- drivers/gpu/drm/nouveau/nv17_tv.c | 16 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_connector

[PATCH 2/9] drm/gma500: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/gma500/cdv_device.c |4 ++-- drivers/gpu/drm/gma500/cdv_intel_dp.c|2 +- drivers/gpu/drm/gma500/cdv_intel_hdmi.c |6 +++--- drivers/gpu/drm/gma500/cdv_intel_lvds.c | 10 +- drivers/gpu/

[PATCH 1/9] drm/i915: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/i915/intel_display.c |2 +- drivers/gpu/drm/i915/intel_dp.c |2 +- drivers/gpu/drm/i915/intel_hdmi.c|2 +- drivers/gpu/drm/i915/intel_lvds.c|2 +- drivers/gpu/drm/i915/intel_modes.c |4 ++-- drivers/

[PATCH 0/9] cleanup: remove legacy connector property functions

2012-10-11 Thread Rob Clark
From: Rob Clark It was suggested by Laurent Pinchart to clean up the legacy connector property functions, as they are no longer needed. This patchset replaces references to and remove the connector property fxns, which have been superseded with the more general object property fxns: + drm_con

buggy/weird behavior in ttm

2012-10-11 Thread Maarten Lankhorst
Op 11-10-12 18:57, Thomas Hellstrom schreef: > On 10/11/2012 04:50 PM, Maarten Lankhorst wrote: >> I was trying to clean ttm up a little so my changes would be less invasive, >> and simplify >> the code for debuggability. During testing I noticed the following >> weirdnesses: >> - ttm_mem_evict_f

[PATCH 01/11] drm: add drm_send_vblank_event() helper (v3)

2012-10-11 Thread Rob Clark
From: Rob Clark A helper that drivers can use to send vblank event after a pageflip. If the driver doesn't support proper vblank irq based time/seqn then just pass -1 for the pipe # to get do_gettimestamp() behavior (since there are a lot of drivers that don't use drm_vblank_count_and_time()) Al

[PATCH 9/9] drm: remove legacy drm_connector_property fxns

2012-10-11 Thread Rob Clark
From: Rob Clark Replace references to and remove the connector property fxns, which have been superseded with the more general object property fxns: + drm_connector_attach_property -> drm_object_attach_property + drm_connector_property_set_value -> drm_object_property_set_value + drm_conne

[PATCH 8/9] drm/i2c: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/i2c/ch7006_drv.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/i2c/ch7006_drv.c b/drivers/gpu/drm/i2c/ch7006_drv.c index 599099f..b865d07 100644 --- a/drivers/gpu/drm/i2c/

[PATCH 7/9] drm/vmwgfx: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c |2 +- drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c b/drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c index 070fb23..7

[PATCH 6/9] drm/udl: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/udl/udl_connector.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/udl/udl_connector.c b/drivers/gpu/drm/udl/udl_connector.c index d568aa4..4faf39c 100644 --- a/drivers/gpu/drm/udl/udl_connecto

[PATCH 5/9] drm/shmob: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c b/drivers/gpu/drm/shmobile/shmob_drm_crtc.c index 3ecb702..698b0cd 100644 --- a/drivers/gpu/dr

[PATCH 4/9] drm/radeon: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/radeon/radeon_connectors.c | 62 ++-- 1 file changed, 31 insertions(+), 31 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c index 895e628..ef

[PATCH 3/9] drm/nouveau: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/nouveau/nouveau_connector.c | 22 +++--- drivers/gpu/drm/nouveau/nv17_tv.c | 16 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_connector

[PATCH 2/9] drm/gma500: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/gma500/cdv_device.c |4 ++-- drivers/gpu/drm/gma500/cdv_intel_dp.c|2 +- drivers/gpu/drm/gma500/cdv_intel_hdmi.c |6 +++--- drivers/gpu/drm/gma500/cdv_intel_lvds.c | 10 +- drivers/gpu/

[PATCH 1/9] drm/i915: drm_connector_property -> drm_object_property

2012-10-11 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/i915/intel_display.c |2 +- drivers/gpu/drm/i915/intel_dp.c |2 +- drivers/gpu/drm/i915/intel_hdmi.c|2 +- drivers/gpu/drm/i915/intel_lvds.c|2 +- drivers/gpu/drm/i915/intel_modes.c |4 ++-- drivers/

[PATCH 0/9] cleanup: remove legacy connector property functions

2012-10-11 Thread Rob Clark
From: Rob Clark It was suggested by Laurent Pinchart to clean up the legacy connector property functions, as they are no longer needed. This patchset replaces references to and remove the connector property fxns, which have been superseded with the more general object property fxns: + drm_con

buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 04:50 PM, Maarten Lankhorst wrote: > I was trying to clean ttm up a little so my changes would be less invasive, > and simplify > the code for debuggability. During testing I noticed the following > weirdnesses: > - ttm_mem_evict_first ignores no_wait_gpu if the buffer is on the dde

Re: [PATCH 2/2] drm: exynos: moved exynos drm hdmi device registration to drm driver

2012-10-11 Thread Inki Dae
2012년 10월 12일 금요일에 Rahul Sharma님이 작성: > This patch moved the exynos-drm-hdmi platform device registration to the > drm > driver. When DT is enabled, platform devices needs to be registered within > the > driver code. This patch fits the requirement of both DT and Non DT based > drm > drivers. > >

pipe B assertion failure (expected off, current on)

2012-10-11 Thread Bjørn Mork
Daniel Vetter writes: > On Thu, Oct 11, 2012 at 4:13 PM, Bj?rn Mork wrote: >> Let me know if you want ot have more details about these warnings. >> Otherwise I'll just keep ignoring them ;-) > > The drm-intel-fixes branch from > http://cgit.freedesktop.org/~danvet/drm-intel has a patch to fix som

[Bug 40211] texture probleme in wolfenstein enemy territory rv770

2012-10-11 Thread bugzilla-dae...@freedesktop.org
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/01eec3e1/attachment.html>

[PATCH 01/11] drm: add drm_send_vblank_event() helper (v3)

2012-10-11 Thread Rob Clark
From: Rob Clark A helper that drivers can use to send vblank event after a pageflip. If the driver doesn't support proper vblank irq based time/seqn then just pass -1 for the pipe # to get do_gettimestamp() behavior (since there are a lot of drivers that don't use drm_vblank_count_and_time()) Al

pipe B assertion failure (expected off, current on)

2012-10-11 Thread Daniel Vetter
On Thu, Oct 11, 2012 at 4:13 PM, Bj?rn Mork wrote: > Let me know if you want ot have more details about these warnings. > Otherwise I'll just keep ignoring them ;-) The drm-intel-fixes branch from http://cgit.freedesktop.org/~danvet/drm-intel has a patch to fix some of the already reported WARNs.

[Patch 0/1]drm_irq: Introducing the irq_thread support

2012-10-11 Thread Daniel Vetter
On Thu, Oct 11, 2012 at 3:19 PM, Rob Clark wrote: >> Should the documenation be updated to mark those functions as deprecated for >> new drivers and explain how to handle IRQ (un)registration manually ? They're not deprecated, since for most drivers they're good enough. Maybe just make it clear t

buggy/weird behavior in ttm

2012-10-11 Thread Maarten Lankhorst
I was trying to clean ttm up a little so my changes would be less invasive, and simplify the code for debuggability. During testing I noticed the following weirdnesses: - ttm_mem_evict_first ignores no_wait_gpu if the buffer is on the ddestroy list. If you follow the code, it will effectively sp

[Bug 26891] Radeon KMS fails with inaccessible AtomBIOS on systems with (U)EFI boot

2012-10-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=26891 --- Comment #43 from Austin Lund --- Seems to be my bad. I've got some extra patches that aren't in 3.6 to make this EFI stub booting thing work (not that the actual graphics switching works for me at the moment). They can be downloaded here:

[PATCH 01/11] drm: add drm_send_vblank_event() helper

2012-10-11 Thread Laurent Pinchart
Hi Rob, Thanks for the patch. On Monday 08 October 2012 14:50:39 Rob Clark wrote: > From: Rob Clark > > A helper that drivers can use to send vblank event after a pageflip. > If the driver doesn't support proper vblank irq based time/seqn then > just pass -1 for the pipe # to get do_gettimestam

pipe B assertion failure (expected off, current on)

2012-10-11 Thread Bjørn Mork
Hello, from time to time, I see the following warning on resume: [ 7022.245347] PM: Syncing filesystems ... done. [ 7022.271923] PM: Preparing system for mem sleep [ 7022.324107] [ cut here ] [ 7022.324154] WARNING: at drivers/gpu/drm/i915/intel_display.c:1225 intel_crtc_

[Bug 55047] [855gm] Boots noisely with the lid close (but works!)

2012-10-11 Thread bugzilla-dae...@freedesktop.org
vel/attachments/20121011/3dea0994/attachment.html>

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Laurent Pinchart
Hi Hans, On Thursday 11 October 2012 13:36:45 Hans Verkuil wrote: > On Thu 11 October 2012 13:34:07 Alan Cox wrote: > > > The whole purpose of this API is to let DRM and V4L drivers share > > > buffers for zero-copy pipelines. Unfortunately it is a fact that > > > several popular DRM drivers are c

Re: [PATCHv10 21/26] v4l: vb2-dma-contig: add reference counting for a device from allocator context

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, Thanks for the patch. On Wednesday 10 October 2012 16:46:40 Tomasz Stanislawski wrote: > This patch adds taking reference to the device for MMAP buffers. > > Such buffers, may be exported using DMABUF mechanism. If the driver that > created a queue is unloaded then the queue is releas

Re: [PATCHv10 20/26] v4l: vb2-dma-contig: add support for DMABUF exporting

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, On Wednesday 10 October 2012 16:46:39 Tomasz Stanislawski wrote: > This patch adds support for exporting a dma-contig buffer using > DMABUF interface. > > Signed-off-by: Tomasz Stanislawski > Signed-off-by: Kyungmin Park > Acked-by: Hans Verkuil Acked-by: Laurent Pinchart > --- >

Re: [PATCHv10 22/26] v4l: vb2-dma-contig: fail if user ptr buffer is not correctly aligned

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, Thanks for the patch. On Wednesday 10 October 2012 16:46:41 Tomasz Stanislawski wrote: > From: Marek Szyprowski > > The DMA transfer must be aligned to a specific value. If userptr is not > aligned to DMA requirements then unexpected corruptions of the memory may > occur before or af

Re: [PATCHv10 23/26] v4l: vb2-dma-contig: align buffer size to PAGE_SIZE

2012-10-11 Thread Laurent Pinchart
Hi Tomasz, On Wednesday 10 October 2012 16:46:42 Tomasz Stanislawski wrote: > Most operations on DMA and DMABUF framework need page > aligned buffers. The comment is a bit misleading, the buffer is already page-aligned (unless I'm mistaken dma_alloc_coherent() returns a page-aligned buffer) but

[RFC 0/4] drm: add raw monotonic timestamp support

2012-10-11 Thread Imre Deak
On Thu, 2012-10-11 at 12:32 +0200, Laurent Pinchart wrote: > Hi Imre, > > On Friday 05 October 2012 16:36:58 Imre Deak wrote: > > This is needed to make applications depending on vblank/page flip > > timestamps independent of time ajdustments. > > > > I've tested these with an updated intel-gpu-t

[RFC 0/4] drm: add raw monotonic timestamp support

2012-10-11 Thread Imre Deak
Hi, On Thu, 2012-10-11 at 12:29 +0200, Laurent Pinchart wrote: > (CC'ing linux-media) > > On Friday 05 October 2012 16:36:58 Imre Deak wrote: > > This is needed to make applications depending on vblank/page flip > > timestamps independent of time ajdustments. > > We're in the process to switchin

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Hans Verkuil
On Thu 11 October 2012 13:36:45 Hans Verkuil wrote: > On Thu 11 October 2012 13:34:07 Alan Cox wrote: > > > The whole purpose of this API is to let DRM and V4L drivers share buffers > > > for > > > zero-copy pipelines. Unfortunately it is a fact that several popular DRM > > > drivers > > > are cl

[Patch 0/1]drm_irq: Introducing the irq_thread support

2012-10-11 Thread Laurent Pinchart
On Wednesday 05 September 2012 15:27:24 Daniel Vetter wrote: > On Wed, Sep 05, 2012 at 01:53:44AM +, Liu, Chuansheng wrote: > > This patch is for introducing the irq thread support in drm_irq. > > > > Why we need irq thread in drm_irq code? > > In our GPU system, the gpu interrupt handler need

Re: buggy/weird behavior in ttm

2012-10-11 Thread Maarten Lankhorst
Op 11-10-12 21:26, Thomas Hellstrom schreef: > On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: > >> >>> Anyway, if you plan to remove the fence lock and protect it with reserve, >>> you must >>> make sure that a waiting reserve is never done in a destruction path. I >>> think this >>> mostly con

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Hans Verkuil
On Thu 11 October 2012 13:34:07 Alan Cox wrote: > > The whole purpose of this API is to let DRM and V4L drivers share buffers > > for > > zero-copy pipelines. Unfortunately it is a fact that several popular DRM > > drivers > > are closed source. So we have a choice between keeping the export symb

[PATCH] drm: fix returning -EINVAL on setmaster if another master is active

2012-10-11 Thread Laurent Pinchart
Hi David, On Thursday 11 October 2012 12:41:43 David Herrmann wrote: > On Thu, Oct 11, 2012 at 12:35 PM, Laurent Pinchart wrote: > > Hi David, > > > > Would you have time to document the master_set operation in > > Documentation/DocBook/drm.tmpl ? :-) > > I have actually some drafts for "drmSetM

[PATCH 0/3] [RFC] DRM Render Nodes

2012-10-11 Thread Laurent Pinchart
Hi Kristian, Could you please update Documentation/DocBook/drm.tmpl with render nodes documentation ? On Friday 28 September 2012 12:35:56 Kristian H?gsberg wrote: > Here's the patch to implement render nodes as discussed in the "DRM2" > talk at XDC: > > http://wiki.x.org/wiki/Events/XDC2012/

Re: [Linaro-mm-sig] [PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Dave Airlie
On Thu, Oct 11, 2012 at 9:34 PM, Alan Cox wrote: >> The whole purpose of this API is to let DRM and V4L drivers share buffers for >> zero-copy pipelines. Unfortunately it is a fact that several popular DRM >> drivers >> are closed source. So we have a choice between keeping the export symbols GPL

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Dave Airlie
>> On Thu, Oct 11, 2012 at 4:17 AM, Alan Cox >> wrote: >> > On Wed, 10 Oct 2012 08:56:32 -0700 >> > Robert Morell wrote: >> > >> >> EXPORT_SYMBOL_GPL is intended to be used for "an internal implementation >> >> issue, and not really an interface". The dma-buf infrastructure is >> >> explicitly

Re: [PATCH 1/2 v6] of: add helper to parse display timings

2012-10-11 Thread Thierry Reding
On Mon, Oct 08, 2012 at 09:49:21AM +0200, Steffen Trumtrar wrote: > On Mon, Oct 08, 2012 at 10:07:45AM +0300, Tomi Valkeinen wrote: > > On Thu, 2012-10-04 at 19:59 +0200, Steffen Trumtrar wrote: [...] > > > + > > > + disp->num_timings = 0; > > > + > > > + for_each_child_of_node(timings_np, entry) {

[PATCH] drm: fix returning -EINVAL on setmaster if another master is active

2012-10-11 Thread David Herrmann
Hi Laurent On Thu, Oct 11, 2012 at 12:35 PM, Laurent Pinchart wrote: > Hi David, > > Would you have time to document the master_set operation in > Documentation/DocBook/drm.tmpl ? :-) I have actually some drafts for "drmSetMaster/drmDropMaster" man-pages for libdrm on my machine. However, I am s

[PATCH 0/3] Android support

2012-10-11 Thread Tapani Pälli
convinced we would like to start using it. > -Chad > -- // Tapani -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 551 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/50da2ddf/attachment-0001.pgp>

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Alan Cox
> > So, developers implicitly or explicitly copied in this thread that might be > > considering the usage of dmabuf on proprietary drivers should consider > > this email as a formal notification of my viewpoint: e. g. that I consider > > any attempt of using DMABUF or media core/drivers together wi

Re: buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 09:26 PM, Thomas Hellstrom wrote: On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: I can't see how the waiting reserve in ttm_bo_cleanup_refs would cause a deadlock, because the buffer about to be reserved is always *last* in a reservation sequence, and the reservation is always

[PATCH] drm: fix returning -EINVAL on setmaster if another master is active

2012-10-11 Thread Laurent Pinchart
Hi David, Would you have time to document the master_set operation in Documentation/DocBook/drm.tmpl ? :-) On Sunday 07 October 2012 19:53:26 David Herrmann wrote: > We link every DRM "file_priv" to a "drm_master" structure. Currently, the > drmSetMaster() call returns 0 when there is _any_ acti

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Alan Cox
> The whole purpose of this API is to let DRM and V4L drivers share buffers for > zero-copy pipelines. Unfortunately it is a fact that several popular DRM > drivers > are closed source. So we have a choice between keeping the export symbols GPL > and forcing those closed-source drivers to make the

[RFC 0/4] drm: add raw monotonic timestamp support

2012-10-11 Thread Laurent Pinchart
Hi Imre, On Friday 05 October 2012 16:36:58 Imre Deak wrote: > This is needed to make applications depending on vblank/page flip > timestamps independent of time ajdustments. > > I've tested these with an updated intel-gpu-test/flip_test and will send > the update for that once there's no objecti

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Alan Cox
> As long as dmabuf uses EXPORT_SYMBOL_GPL that is definitely correct. Does your > statement also hold if dmabuf would use EXPORT_SYMBOL? (Just asking) Yes. The GPL talks about derivative works (as does copyright law). Alan

[RFC 0/4] drm: add raw monotonic timestamp support

2012-10-11 Thread Laurent Pinchart
(CC'ing linux-media) On Friday 05 October 2012 16:36:58 Imre Deak wrote: > This is needed to make applications depending on vblank/page flip > timestamps independent of time ajdustments. We're in the process to switching to CLOCK_MONOTONIC timestamps in V4L2. The reason why we're using CLOCK_MON

Re: buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 08:42 PM, Maarten Lankhorst wrote: Anyway, if you plan to remove the fence lock and protect it with reserve, you must make sure that a waiting reserve is never done in a destruction path. I think this mostly concerns the nvidia driver. Well I don't think any lock should ever b

Re: [RFC 01/11] drm: add atomic fxns

2012-10-11 Thread Laurent Pinchart
Hi Rob, On Wednesday 12 September 2012 22:49:47 Rob Clark wrote: > From: Rob Clark > > The 'atomic' mechanism allows for multiple properties to be updated, > checked, and commited atomically. This will be the basis of atomic- > modeset and nuclear-pageflip. > > The basic flow is: > >state

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Mauro Carvalho Chehab
Em Thu, 11 Oct 2012 08:47:15 -0500 Rob Clark escreveu: > On Thu, Oct 11, 2012 at 6:13 AM, Mauro Carvalho Chehab > wrote: > > Em Thu, 11 Oct 2012 09:20:12 +0200 > > Hans Verkuil escreveu: > > > >> > my understaning is > >> > that the drivers/media/ authors should also ack with this licensing > >

[PATCH] Android.mk: use LOCAL_COPY_HEADERS to export headers. v2

2012-10-11 Thread Tapani Pälli
Y) >> >> include $(LOCAL_PATH)/intel/Android.mk >> > -- // Tapani -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 551 bytes Desc: OpenPGP digital signature URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/9eebd7e2/attachment.pgp>

Re: buggy/weird behavior in ttm

2012-10-11 Thread Maarten Lankhorst
Op 11-10-12 18:57, Thomas Hellstrom schreef: > On 10/11/2012 04:50 PM, Maarten Lankhorst wrote: >> I was trying to clean ttm up a little so my changes would be less invasive, >> and simplify >> the code for debuggability. During testing I noticed the following >> weirdnesses: >> - ttm_mem_evict_f

[Bug 55047] [855gm] Boots noisely with the lid close (but works!)

2012-10-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/20121011/a919d792/attachment.html>

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Maarten Lankhorst
Op 11-10-12 09:51, Hans Verkuil schreef: >>> my understaning is >>> that the drivers/media/ authors should also ack with this licensing >>> (possible) change. I am one of the main contributors there. Alan also has >>> copyrights there, and at other parts of the Linux Kernel, including the >>> dri

[Bug 40211] texture probleme in wolfenstein enemy territory rv770

2012-10-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40211 --- Comment #13 from pitam...@free.fr --- yes it's installed -- You are receiving this mail because: You are the assignee for the bug. ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://

Re: buggy/weird behavior in ttm

2012-10-11 Thread Thomas Hellstrom
On 10/11/2012 04:50 PM, Maarten Lankhorst wrote: I was trying to clean ttm up a little so my changes would be less invasive, and simplify the code for debuggability. During testing I noticed the following weirdnesses: - ttm_mem_evict_first ignores no_wait_gpu if the buffer is on the ddestroy lis

drm/nouveau: port all engines to new engine module format

2012-10-11 Thread Dan Carpenter
Hello Ben Skeggs, The patch ebb945a94bba: "drm/nouveau: port all engines to new engine module format" from Jul 20, 2012, leads to the following warning: drivers/gpu/drm/nouveau/nouveau_abi16.c:264 nouveau_abi16_ioctl_channel_alloc() warn: should '1 << init->channel' be a 64 bit type

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Hans Verkuil
On Thu 11 October 2012 09:20:12 Hans Verkuil wrote: > On Thu October 11 2012 03:11:19 Mauro Carvalho Chehab wrote: > > Em Thu, 11 Oct 2012 09:22:34 +1000 > > Dave Airlie escreveu: > > > > > On Thu, Oct 11, 2012 at 4:17 AM, Alan Cox > > > wrote: > > > > On Wed, 10 Oct 2012 08:56:32 -0700 > > > >

[PATCH 01/11] drm: add drm_send_vblank_event() helper

2012-10-11 Thread Rob Clark
On Thu, Oct 11, 2012 at 9:19 AM, Laurent Pinchart wrote: > Hi Rob, > > Thanks for the patch. > > On Monday 08 October 2012 14:50:39 Rob Clark wrote: >> From: Rob Clark >> >> A helper that drivers can use to send vblank event after a pageflip. >> If the driver doesn't support proper vblank irq bas

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Dave Airlie
On Thu, Oct 11, 2012 at 4:17 AM, Alan Cox wrote: > On Wed, 10 Oct 2012 08:56:32 -0700 > Robert Morell wrote: > >> EXPORT_SYMBOL_GPL is intended to be used for "an internal implementation >> issue, and not really an interface". The dma-buf infrastructure is >> explicitly intended as an interface

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Hans Verkuil
On Thu October 11 2012 03:11:19 Mauro Carvalho Chehab wrote: > Em Thu, 11 Oct 2012 09:22:34 +1000 > Dave Airlie escreveu: > > > On Thu, Oct 11, 2012 at 4:17 AM, Alan Cox > > wrote: > > > On Wed, 10 Oct 2012 08:56:32 -0700 > > > Robert Morell wrote: > > > > > >> EXPORT_SYMBOL_GPL is intended to

[Bug 55047] [855gm] Boots noisely with the lid close (but works!)

2012-10-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/20121011/5b367a0c/attachment.html>

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Hans Verkuil
On Wed October 10 2012 23:02:06 Rob Clark wrote: > On Wed, Oct 10, 2012 at 1:17 PM, Alan Cox wrote: > > On Wed, 10 Oct 2012 08:56:32 -0700 > > Robert Morell wrote: > > > >> EXPORT_SYMBOL_GPL is intended to be used for "an internal implementation > >> issue, and not really an interface". The dma-

[PATCH] dma-buf: Use EXPORT_SYMBOL

2012-10-11 Thread Rob Clark
On Thu, Oct 11, 2012 at 6:13 AM, Mauro Carvalho Chehab wrote: > Em Thu, 11 Oct 2012 09:20:12 +0200 > Hans Verkuil escreveu: > >> > my understaning is >> > that the drivers/media/ authors should also ack with this licensing >> > (possible) change. I am one of the main contributors there. Alan also

[Bug 55047] [855gm] Boots noisely with the lid close (but works!)

2012-10-11 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20121011/dd447ae2/attachment.html>

Re: pipe B assertion failure (expected off, current on)

2012-10-11 Thread Daniel Vetter
On Thu, Oct 11, 2012 at 4:13 PM, Bjørn Mork wrote: > Let me know if you want ot have more details about these warnings. > Otherwise I'll just keep ignoring them ;-) The drm-intel-fixes branch from http://cgit.freedesktop.org/~danvet/drm-intel has a patch to fix some of the already reported WARNs.

  1   2   >