GEM - radeon cs ioctl deadlock

2010-10-18 Thread Thomas Hellstrom
On 10/14/2010 03:47 AM, Ben Skeggs wrote: > On Thu, 2010-10-14 at 08:14 +1000, Dave Airlie wrote: > >> On Wed, 2010-10-13 at 17:57 -0400, Jerome Glisse wrote: >> >>> So we are facing a deadlock with the radeon cs ioctl. When a buffer is given >>> a name (with flink) we could endup with 2

Re: GEM - radeon cs ioctl deadlock

2010-10-18 Thread Thomas Hellstrom
On 10/14/2010 03:47 AM, Ben Skeggs wrote: On Thu, 2010-10-14 at 08:14 +1000, Dave Airlie wrote: On Wed, 2010-10-13 at 17:57 -0400, Jerome Glisse wrote: So we are facing a deadlock with the radeon cs ioctl. When a buffer is given a name (with flink) we could endup with 2 handle

GEM - radeon cs ioctl deadlock

2010-10-14 Thread Ben Skeggs
On Thu, 2010-10-14 at 08:14 +1000, Dave Airlie wrote: > On Wed, 2010-10-13 at 17:57 -0400, Jerome Glisse wrote: > > So we are facing a deadlock with the radeon cs ioctl. When a buffer is given > > a name (with flink) we could endup with 2 handle pointing to the same > > object (flink object and

GEM - radeon cs ioctl deadlock

2010-10-14 Thread Dave Airlie
On Wed, 2010-10-13 at 17:57 -0400, Jerome Glisse wrote: > So we are facing a deadlock with the radeon cs ioctl. When a buffer is given > a name (with flink) we could endup with 2 handle pointing to the same > object (flink object and open it from same file descriptor). Would it be ok > if i change

GEM - radeon cs ioctl deadlock

2010-10-13 Thread Jerome Glisse
So we are facing a deadlock with the radeon cs ioctl. When a buffer is given a name (with flink) we could endup with 2 handle pointing to the same object (flink object and open it from same file descriptor). Would it be ok if i change gem open to first look if we already have an handle for the

GEM - radeon cs ioctl deadlock

2010-10-13 Thread Jerome Glisse
So we are facing a deadlock with the radeon cs ioctl. When a buffer is given a name (with flink) we could endup with 2 handle pointing to the same object (flink object and open it from same file descriptor). Would it be ok if i change gem open to first look if we already have an handle for the

Re: GEM - radeon cs ioctl deadlock

2010-10-13 Thread Ben Skeggs
On Thu, 2010-10-14 at 08:14 +1000, Dave Airlie wrote: On Wed, 2010-10-13 at 17:57 -0400, Jerome Glisse wrote: So we are facing a deadlock with the radeon cs ioctl. When a buffer is given a name (with flink) we could endup with 2 handle pointing to the same object (flink object and open it