Re: [PATCH 3/4] drm/i915/tgl: Gen12 csb support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux