Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-17 Thread Koenig, Christian
Hi Emil, Sorry for the delay, finally coming back to this after my vacation. Am 03.07.19 um 17:14 schrieb Emil Velikov: > On Wed, 3 Jul 2019 at 15:58, Koenig, Christian > wrote: >> Am 03.07.2019 16:51 schrieb Emil Velikov : >> >> On Wed, 3 Jul 2019 at 15:33, Koenig, Christian >> wrote: >>>

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-06 Thread Koenig, Christian
Am 03.07.2019 17:14 schrieb Emil Velikov : On Wed, 3 Jul 2019 at 15:58, Koenig, Christian wrote: > Am 03.07.2019 16:51 schrieb Emil Velikov : > > On Wed, 3 Jul 2019 at 15:33, Koenig, Christian > wrote: > > Am 03.07.2019 16:00 schrieb Emil Velikov : > > > > On Wed, 3 Jul 2019 at 14:48, Koenig,

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Emil Velikov
On Wed, 3 Jul 2019 at 15:58, Koenig, Christian wrote: > Am 03.07.2019 16:51 schrieb Emil Velikov : > > On Wed, 3 Jul 2019 at 15:33, Koenig, Christian > wrote: > > Am 03.07.2019 16:00 schrieb Emil Velikov : > > > > On Wed, 3 Jul 2019 at 14:48, Koenig, Christian > > wrote: > > > > > > Well this

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Koenig, Christian
Am 03.07.2019 16:51 schrieb Emil Velikov : On Wed, 3 Jul 2019 at 15:33, Koenig, Christian wrote: > Am 03.07.2019 16:00 schrieb Emil Velikov : > > On Wed, 3 Jul 2019 at 14:48, Koenig, Christian > wrote: > > > > Well this is still a NAK. > > > > As stated previously please just don't remove

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Emil Velikov
On Wed, 3 Jul 2019 at 15:33, Koenig, Christian wrote: > Am 03.07.2019 16:00 schrieb Emil Velikov : > > On Wed, 3 Jul 2019 at 14:48, Koenig, Christian > wrote: > > > > Well this is still a NAK. > > > > As stated previously please just don't remove DRM_AUTH and keep the > > functionality as it

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Koenig, Christian
Am 03.07.2019 16:00 schrieb Emil Velikov : On Wed, 3 Jul 2019 at 14:48, Koenig, Christian wrote: > > Well this is still a NAK. > > As stated previously please just don't remove DRM_AUTH and keep the > functionality as it is. > AFAICT nobody was in favour of your suggestion to remove DRM_AUTH

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Emil Velikov
On Wed, 3 Jul 2019 at 14:48, Koenig, Christian wrote: > > Well this is still a NAK. > > As stated previously please just don't remove DRM_AUTH and keep the > functionality as it is. > AFAICT nobody was in favour of your suggestion to remove DRM_AUTH from the handle to/from fd ioclts. Thus this

Re: [PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Koenig, Christian
Well this is still a NAK. As stated previously please just don't remove DRM_AUTH and keep the functionality as it is. I absolutely don't see the point to add a new flag to remove the same functionality a different flag provides. Christian. Am 03.07.2019 15:30 schrieb Emil Velikov : From:

[PATCH 2/3] drm: introduce DRIVER_FORCE_AUTH

2019-07-03 Thread Emil Velikov
From: Emil Velikov With earlier commits we've removed DRM_AUTH for driver ioctls annotated with DRM_AUTH | DRM_RENDER_ALLOW, as the protection it introduces is effectively not existent. With next commit, we'll effectively do the same for DRM core. Yet the AMD developers have voiced concerns