Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-17 Thread Michel Dänzer
On 17/04/17 02:05 PM, Christopher James Halse Rogers wrote: > On 10 April 2017 6:51:21 pm AEST, "Michel Dänzer" wrote: >> On 06/04/17 04:47 PM, Christopher James Halse Rogers wrote: >>> On Wed, 5 Apr 2017 at 20:14 Lucas Stach >> > wrote: >>> Drivers with separat

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-16 Thread Christopher James Halse Rogers
On 10 April 2017 6:51:21 pm AEST, "Michel Dänzer" wrote: >On 06/04/17 04:47 PM, Christopher James Halse Rogers wrote: >> On Wed, 5 Apr 2017 at 20:14 Lucas Stach > > wrote: >> >> Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: >> > On Wed, Apr

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-10 Thread Alex Deucher
On Mon, Apr 10, 2017 at 5:03 AM, Christian König wrote: > Am 10.04.2017 um 10:52 schrieb Michel Dänzer: >> >> On 05/04/17 08:21 PM, Christian König wrote: >>> >>> Am 05.04.2017 um 13:13 schrieb Christopher James Halse Rogers: On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach >>>

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-10 Thread Christian König
Am 10.04.2017 um 10:52 schrieb Michel Dänzer: On 05/04/17 08:21 PM, Christian König wrote: Am 05.04.2017 um 13:13 schrieb Christopher James Halse Rogers: On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach mailto:l.st...@pengutronix.de>> wrote: Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Dani

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-10 Thread Michel Dänzer
On 05/04/17 08:21 PM, Christian König wrote: > Am 05.04.2017 um 13:13 schrieb Christopher James Halse Rogers: >> On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach > > wrote: >> >> Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: >> > On Wed, Apr 05, 20

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-10 Thread Michel Dänzer
On 06/04/17 04:47 PM, Christopher James Halse Rogers wrote: > On Wed, 5 Apr 2017 at 20:14 Lucas Stach > wrote: > > Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > > Am M

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-06 Thread Christopher James Halse Rogers
On Wed, 5 Apr 2017 at 20:14 Lucas Stach wrote: > Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James > Halse > > > Rogers: > > > > > > > > > > > >

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Christian König
Am 05.04.2017 um 13:13 schrieb Christopher James Halse Rogers: On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach > wrote: Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > Am Mitt

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Christopher James Halse Rogers
On Wed, Apr 5, 2017 at 8:14 PM Lucas Stach wrote: > Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James > Halse > > > Rogers: > > > > > > > > > > >

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Lucas Stach
Am Mittwoch, den 05.04.2017, 11:59 +0200 schrieb Daniel Vetter: > On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse > > Rogers: > > > > > > > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > > > >

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 10:15:44AM +0200, Lucas Stach wrote: > Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse > Rogers: > > > > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > > > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > > wrote: > >

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-05 Thread Lucas Stach
Am Mittwoch, den 05.04.2017, 00:20 + schrieb Christopher James Halse Rogers: > > > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > wrote: > >> If I could guarantee that I'd only ever run on > 4.13-or-lat

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Christopher James Halse Rogers
On Wed, Apr 5, 2017 at 4:27 PM, Daniel Vetter wrote: On Wed, Apr 05, 2017 at 12:20:46AM +, Christopher James Halse Rogers wrote: On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > wrote: > >> If I could guarantee that I'd only ev

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Vetter
On Wed, Apr 05, 2017 at 12:20:46AM +, Christopher James Halse Rogers wrote: > On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > > > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > > wrote: > > >> If I could guarantee that I'd only ever run on 4.13-or-later kernels > > >> (I think that's w

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Christopher James Halse Rogers
On Tue, Apr 4, 2017 at 9:53 PM Daniel Vetter wrote: > On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach > wrote: > >> If I could guarantee that I'd only ever run on 4.13-or-later kernels > >> (I think that's when the previous patches will land?), then this would > >> indeed be mostly unnecessary. It

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Vetter
On Tue, Apr 4, 2017 at 12:43 PM, Lucas Stach wrote: >> If I could guarantee that I'd only ever run on 4.13-or-later kernels >> (I think that's when the previous patches will land?), then this would >> indeed be mostly unnecessary. It would save me a bunch of addfb calls >> that would predictably f

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Christian König
Am 04.04.2017 um 13:32 schrieb Daniel Stone: Hi, On 4 April 2017 at 12:15, Christian König wrote: Am 04.04.2017 um 12:43 schrieb Daniel Stone: Yeah, the ABI is that AddFB2 should fail hard on something which can never be used as a framebuffer. The fact it didn't is a bug rather than a behavio

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Stone
Hi, On 4 April 2017 at 12:15, Christian König wrote: > Am 04.04.2017 um 12:43 schrieb Daniel Stone: >> Yeah, the ABI is that AddFB2 should fail hard on something which can >> never be used as a framebuffer. The fact it didn't is a bug rather >> than a behavioural change per se ... > > I agree on

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Christian König
Am 04.04.2017 um 12:43 schrieb Daniel Stone: Hi, On 4 April 2017 at 11:27, Christopher James Halse Rogers wrote: On 4 April 2017 6:31:12 pm AEST, Daniel Vetter wrote: This seems like an awful specific special case. Why can't you do the entire dance upfront, i.e. import buffer, addfb2? None o

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Stone
Hi, On 4 April 2017 at 11:27, Christopher James Halse Rogers wrote: > On 4 April 2017 6:31:12 pm AEST, Daniel Vetter wrote: >>This seems like an awful specific special case. Why can't you do the >>entire dance upfront, i.e. import buffer, addfb2? None of that has any >>visible effect, and it wil

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Lucas Stach
Am Dienstag, den 04.04.2017, 20:27 +1000 schrieb Christopher James Halse Rogers: > On 4 April 2017 6:31:12 pm AEST, Daniel Vetter wrote: > >On Tue, Apr 04, 2017 at 06:13:20PM +1000, r...@ubuntu.com wrote: > >> From: Christopher James Halse Rogers > >> > >> Until recently, on (at least) nouveau,

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Christopher James Halse Rogers
On 4 April 2017 6:31:12 pm AEST, Daniel Vetter wrote: >On Tue, Apr 04, 2017 at 06:13:20PM +1000, r...@ubuntu.com wrote: >> From: Christopher James Halse Rogers >> >> Until recently, on (at least) nouveau, radeon, and amdgpu attempting >to scanout of an >> imported dma-buf would silently result i

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Stone
Hi, On 4 April 2017 at 09:31, Daniel Vetter wrote: > On Tue, Apr 04, 2017 at 06:13:20PM +1000, r...@ubuntu.com wrote: >> Until recently, on (at least) nouveau, radeon, and amdgpu attempting to >> scanout of an >> imported dma-buf would silently result in the dma-buf sharing being broken. >> >> W

Re: [PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread Daniel Vetter
On Tue, Apr 04, 2017 at 06:13:20PM +1000, r...@ubuntu.com wrote: > From: Christopher James Halse Rogers > > Until recently, on (at least) nouveau, radeon, and amdgpu attempting to > scanout of an > imported dma-buf would silently result in the dma-buf sharing being broken. > > While the hardwar

[PATCH 1/2] drm: Add DRM_CAP_PRIME_SCANOUT.

2017-04-04 Thread raof
From: Christopher James Halse Rogers Until recently, on (at least) nouveau, radeon, and amdgpu attempting to scanout of an imported dma-buf would silently result in the dma-buf sharing being broken. While the hardware is capable of scanning out of imported dma-bufs (at least in some circumstan