Quoting Oscar Mateo (2017-09-12 23:50:08) > > > On 09/12/2017 02:49 PM, Chris Wilson wrote: > > The context descriptor is stored inside the per-engine context state, as > > we only need to compute it once and access it frequently. However, > > currently only intel_lrc.c has easy access, but i915_guc_submission.c > > would like to frequently read it as well, and more so only ever needs > > the lower 32bits. Make it an inline as the compiler should be able to > > retrieve the value in less instructions than it takes to do the function > > call: > > > > add/remove: 0/1 grow/shrink: 1/0 up/down: 8/-45 (-37) > > function old new delta > > i915_guc_submit 621 629 +8 > > intel_lr_context_descriptor 45 - -45 > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > --- > > drivers/gpu/drm/i915/intel_lrc.c | 6 ------ > > drivers/gpu/drm/i915/intel_lrc.h | 11 +++++++++-- > > 2 files changed, 9 insertions(+), 8 deletions(-) > > Reviewed-by: Oscar Mateo <oscar.mateo@xxxxxxxxx> Thanks for the review, pushed along with your series. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
