Re: [PATCH] drm/lease: look at ->universal_planes only once

2018-11-06 Thread Daniel Vetter
On Mon, Nov 05, 2018 at 11:12:11AM +0100, Daniel Vetter wrote: > It's lockless, and userspace might chance it underneath us. That's not > really a problem, all userspace gets is a slightly dysfunctional > lease with the current code. But this might change, and gcc might > decide to reload a few

[PATCH] drm/lease: look at ->universal_planes only once

2018-11-05 Thread Daniel Vetter
It's lockless, and userspace might chance it underneath us. That's not really a problem, all userspace gets is a slightly dysfunctional lease with the current code. But this might change, and gcc might decide to reload a few too many times, and then boom. So better safe than sorry. v2: Remove the

[PATCH] drm/lease: look at ->universal_planes only once

2018-11-02 Thread Daniel Vetter
It's lockless, and userspace might chance it underneath us. That's not really a problem, all userspace gets is a slightly dysfunctional lease with the current code. But this might change, and gcc might decide to reload a few too many times, and then boom. So better safe than sorry. v2: Remove the