Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > Quoting Mika Kuoppala (2017-10-31 15:27:33) >> +static inline struct execlist_port * >> +execlists_port_next(struct intel_engine_execlists * const execlists, >> + const struct execlist_port * const port) >> +{ >> + const unsigned int n = __port_add(port_index(port, execlists), >> + 1, >> + execlists->port_mask); > > How does this compare to > > if (port++ == execlists->port + execlists->port_mask) > port = execlists->port; > > return port; > ? add/remove: 0/0 grow/shrink: 1/1 up/down: 29/-29 (0) function old new delta i915_guc_irq_handler 2584 2613 +29 intel_lrc_irq_handler 2963 2934 -29 Total: Before=1123627, After=1123627, chg +0.00% :) -Mika _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx