Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-08 Thread Michel Dänzer
On 07/09/17 07:24 PM, Christian König wrote: > Am 07.09.2017 um 12:14 schrieb Marek Olšák: >> On Sep 7, 2017 12:08 PM, "Christian König" > > wrote: >> Am 07.09.2017 um 11:23 schrieb Michel Dänzer: >> On 01/09/17 07:40 PM,

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-07 Thread Christian König
Am 07.09.2017 um 12:14 schrieb Marek Olšák: On Sep 7, 2017 12:08 PM, "Christian König" > wrote: Am 07.09.2017 um 11:23 schrieb Michel Dänzer: On 01/09/17 07:40 PM, Christian König wrote: Am 01.09.2017 um

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-07 Thread Marek Olšák
On Sep 7, 2017 12:08 PM, "Christian König" wrote: Am 07.09.2017 um 11:23 schrieb Michel Dänzer: > On 01/09/17 07:40 PM, Christian König wrote: > >> Am 01.09.2017 um 12:28 schrieb Michel Dänzer: >> >>> On 01/09/17 07:23 PM, Nicolai Hähnle wrote: >>> On 01.09.2017

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-07 Thread Christian König
Am 07.09.2017 um 11:23 schrieb Michel Dänzer: On 01/09/17 07:40 PM, Christian König wrote: Am 01.09.2017 um 12:28 schrieb Michel Dänzer: On 01/09/17 07:23 PM, Nicolai Hähnle wrote: On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-07 Thread Michel Dänzer
On 01/09/17 07:40 PM, Christian König wrote: > Am 01.09.2017 um 12:28 schrieb Michel Dänzer: >> On 01/09/17 07:23 PM, Nicolai Hähnle wrote: >>> On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: > From: Marek Olšák > > For

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-02 Thread Michel Dänzer
On 01/09/17 09:18 PM, Marek Olšák wrote: > Well, this patch shouldn't change behavior. Are you sure it's not some > random thing? Pretty sure. I never saw this GPUVM fault before this commit, but with it or later commits it's reproducible reliably. > I know, for example, that SI isn't very

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Marek Olšák
Well, this patch shouldn't change behavior. Are you sure it's not some random thing? I know, for example, that SI isn't very stable with piglit on radeon either. Marek On Sep 1, 2017 12:28 PM, "Michel Dänzer" wrote: > On 01/09/17 07:23 PM, Nicolai Hähnle wrote: > > On

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Christian König
Am 01.09.2017 um 12:28 schrieb Michel Dänzer: On 01/09/17 07:23 PM, Nicolai Hähnle wrote: On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Michel Dänzer
On 01/09/17 07:23 PM, Nicolai Hähnle wrote: > On 01.09.2017 11:58, Michel Dänzer wrote: >> On 29/08/17 11:47 PM, Christian König wrote: >>> From: Marek Olšák >>> >>> For lower overhead in the CS ioctl. >>> Winsys allocators are not used with interprocess-sharable resources.

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Nicolai Hähnle
On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable resources. v2: It shouldn't crash anymore, but the kernel will reject

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Michel Dänzer
On 29/08/17 11:47 PM, Christian König wrote: > From: Marek Olšák > > For lower overhead in the CS ioctl. > Winsys allocators are not used with interprocess-sharable resources. > > v2: It shouldn't crash anymore, but the kernel will reject the new flag. > v3 (christian):

[Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-08-29 Thread Christian König
From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable resources. v2: It shouldn't crash anymore, but the kernel will reject the new flag. v3 (christian): Rename the flag, avoid sending those buffers in the BO list.