Re: [PATCH 02/27] drm/i915: Stop storing the ring size in the ring pointer

2021-05-14 Thread Jason Ekstrand
On Tue, May 4, 2021 at 3:47 AM Daniel Vetter wrote: > > On Mon, May 03, 2021 at 10:57:23AM -0500, Jason Ekstrand wrote: > > Previously, we were storing the ring size in the ring pointer before it > > was actually allocated. We would then guard setting the ring size on > > checking for CONTEXT_ALL

Re: [PATCH 02/27] drm/i915: Stop storing the ring size in the ring pointer

2021-05-04 Thread Daniel Vetter
On Mon, May 03, 2021 at 10:57:23AM -0500, Jason Ekstrand wrote: > Previously, we were storing the ring size in the ring pointer before it > was actually allocated. We would then guard setting the ring size on > checking for CONTEXT_ALLOC_BIT. This is error-prone at best and really > only saves us

[PATCH 02/27] drm/i915: Stop storing the ring size in the ring pointer

2021-05-03 Thread Jason Ekstrand
Previously, we were storing the ring size in the ring pointer before it was actually allocated. We would then guard setting the ring size on checking for CONTEXT_ALLOC_BIT. This is error-prone at best and really only saves us a few bytes on something that already burns at least 4K. Instead, this