Quoting Mika Kuoppala (2018-12-19 12:27:22) > Make csb entry count variable in preparation for larger > CSB status FIFO size found on gen11+ hardware. > > v2: adapt to hwsp access only (Chris) > non continuous mmio (Daniele) > v3: entries (Chris), fix macro for checkpatch > > Cc: Daniele Ceraolo Spurio <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Signed-off-by: Mika Kuoppala <[email protected]>
> @@ -867,6 +867,7 @@ static void process_csb(struct intel_engine_cs *engine) > struct intel_engine_execlists * const execlists = &engine->execlists; > struct execlist_port *port = execlists->port; > const u32 * const buf = execlists->csb_status; > + const u8 entries = execlists->csb_size; > u8 head, tail; > > /* > @@ -900,7 +901,7 @@ static void process_csb(struct intel_engine_cs *engine) > unsigned int status; > unsigned int count; > > - if (++head == GEN8_CSB_ENTRIES) > + if (++head == entries) Fwiw, I would go with s/entries/num_entries/ Reviewed-by: Chris Wilson <[email protected]> -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
