[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Christian König
Am 19.09.2014 um 15:40 schrieb Jerome Glisse: > On Fri, Sep 19, 2014 at 10:03:43AM +0200, Christian K?nig wrote: >> Hi Jerome, >> >> this is exactly the approach which we wanted to avoid. And according to Alex >> it was actually you guys (I think Dave) who noted that this is probably not >> such a

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Jerome Glisse
On Fri, Sep 19, 2014 at 04:29:35PM +0200, Christian K?nig wrote: > Am 19.09.2014 um 15:40 schrieb Jerome Glisse: > >On Fri, Sep 19, 2014 at 10:03:43AM +0200, Christian K?nig wrote: > >>Hi Jerome, > >> > >>this is exactly the approach which we wanted to avoid. And according to Alex > >>it was

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Christian König
Hi Jerome, this is exactly the approach which we wanted to avoid. And according to Alex it was actually you guys (I think Dave) who noted that this is probably not such a good idea. The issue is that by telling userspace the ring specific sequence number we nail down the order of execution.

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Jerome Glisse
On Fri, Sep 19, 2014 at 10:03:43AM +0200, Christian K?nig wrote: > Hi Jerome, > > this is exactly the approach which we wanted to avoid. And according to Alex > it was actually you guys (I think Dave) who noted that this is probably not > such a good idea. > > The issue is that by telling

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Jerome Glisse
On Thu, Sep 18, 2014 at 11:11:57PM -0400, Jerome Glisse wrote: > On Thu, Sep 18, 2014 at 08:42:16PM -0400, Jerome Glisse wrote: > > On Thu, Sep 18, 2014 at 05:30:12PM +0200, Christian K?nig wrote: > > > From: Christian K?nig > > > > > > This patch adds a new 64bit ID as a result to each command

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-19 Thread Jerome Glisse
On Thu, Sep 18, 2014 at 08:42:16PM -0400, Jerome Glisse wrote: > On Thu, Sep 18, 2014 at 05:30:12PM +0200, Christian K?nig wrote: > > From: Christian K?nig > > > > This patch adds a new 64bit ID as a result to each command submission. > > NAK for design reasons. > > First and foremost we DO

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-18 Thread Jerome Glisse
On Thu, Sep 18, 2014 at 05:30:12PM +0200, Christian K?nig wrote: > From: Christian K?nig > > This patch adds a new 64bit ID as a result to each command submission. NAK for design reasons. First and foremost we DO NOT WANT TO ALLOCATE ANYTHING for submission id. I know that hooking up to fence

[PATCH 2/3] drm/radeon: add command submission IDs

2014-09-18 Thread Christian König
From: Christian K?nig This patch adds a new 64bit ID as a result to each command submission. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/Makefile | 2 +- drivers/gpu/drm/radeon/radeon.h | 13 +- drivers/gpu/drm/radeon/radeon_cs.c | 13 ++