Quoting Daniele Ceraolo Spurio (2019-07-31 01:49:01) > @@ -1401,7 +1453,7 @@ static void process_csb(struct intel_engine_cs *engine) > engine->name, head, > buf[2 * head + 0], buf[2 * head + 1]); > > - switch (csb_parse(execlists, buf + 2 * head)) { > + switch (execlists->csb_parse(execlists, buf + 2 * head)) { So I worry about the cost of a retpoline here (tucked away inside an irqs-off loop), and whether a local func avoids the retpoline or if we just have to use an if-ladder. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx