Re: [Mesa-dev] [PATCH 4/7] i965: Record the presence of the kernel scheduler

[Kenneth Graunke]

On Monday, October 2, 2017 1:52:12 AM PDT Joonas Lahtinen wrote:
> On Fri, 2017-09-29 at 12:52 -0700, Kenneth Graunke wrote:
> > On Friday, September 29, 2017 3:25:07 AM PDT Chris Wilson wrote:
> > > Mention to the debug log if the kernel scheduler is enabled; and in
> > > particular if it has preemption enabled.
> > > 
> > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > > Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
> > > Cc: Ben Widawsky <b...@bwidawsk.net>
> > > Reviewed-by: Joonas Lahtinen <joonas.lahti...@linux.intel.com>
> > > ---
> > >  src/mesa/drivers/dri/i965/intel_screen.c | 11 +++++++++++
> > >  1 file changed, 11 insertions(+)
> > > 
> > > diff --git a/src/mesa/drivers/dri/i965/intel_screen.c 
> > > b/src/mesa/drivers/dri/i965/intel_screen.c
> > > index bd1365f232..22d9f19298 100644
> > > --- a/src/mesa/drivers/dri/i965/intel_screen.c
> > > +++ b/src/mesa/drivers/dri/i965/intel_screen.c
> > > @@ -2516,6 +2516,17 @@ __DRIconfig **intelInitScreen2(__DRIscreen 
> > > *dri_screen)
> > >  
> > >     intel_screen_init_surface_formats(screen);
> > >  
> > > +   if (INTEL_DEBUG & DEBUG_SUBMIT) {
> > 
> > How about:
> > 
> >    if (INTEL_DEBUG & (DEBUG_SUBMIT | DEBUG_BAT)) {
> 
> Assuming that's DEBUG_BATCH (not some new Basic Acceptance Testing
> variable), that works too :)
> 
> Regards, Joonas

Ah yes, sorry, INTEL_DEBUG="bat" turns into INTEL_DEBUG & DEBUG_BATCH :)

--Ken

signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev