On 08/03/18 17:33, Chris Wilson wrote:
Give the compiler a helping hand in mapping (bank,bit) to our struct
intel_engine_cs by trading object code size for data cache:
add/remove: 2/0 grow/shrink: 0/1 up/down: 48/-135 (-87)
Function old new delta
Quoting Tvrtko Ursulin (2018-03-09 10:06:48)
>
> On 09/03/2018 01:38, Chris Wilson wrote:
> And in general I think too many bike-sheds on this area of code before
> we are even running it on real hw. :(
Come on now, it's not a proper bikeshed if the discussion is meaningful!
-Chris
On 09/03/2018 01:38, Chris Wilson wrote:
Quoting Chris Wilson (2018-03-09 01:33:08)
gen11_gt_engine_intr(struct drm_i915_private * const i915,
const unsigned int bank, const unsigned int bit)
@@ -2836,10 +2798,23 @@ static void
gen11_gt_irq_handler(struct
Quoting Chris Wilson (2018-03-09 01:33:08)
> gen11_gt_engine_intr(struct drm_i915_private * const i915,
> const unsigned int bank, const unsigned int bit)
> @@ -2836,10 +2798,23 @@ static void
> gen11_gt_irq_handler(struct drm_i915_private * const i915,
>
Give the compiler a helping hand in mapping (bank,bit) to our struct
intel_engine_cs by trading object code size for data cache:
add/remove: 2/0 grow/shrink: 0/1 up/down: 48/-135 (-87)
Function old new delta
bank1_map
