Re: [Mesa-dev] [PATCH 25/53] i965/drm: Drop intel_chipset.h in favor of using gen_device_info.

2017-04-05 Thread Kenneth Graunke
On Wednesday, April 5, 2017 2:50:01 PM PDT Chris Wilson wrote: > On Wed, Apr 05, 2017 at 02:36:38PM -0700, Kenneth Graunke wrote: > > On Wednesday, April 5, 2017 3:33:39 AM PDT Chris Wilson wrote: > > > On Tue, Apr 04, 2017 at 05:10:15PM -0700, Kenneth Graunke wrote: > > > > This moves the PCI ID d

Re: [Mesa-dev] [PATCH 25/53] i965/drm: Drop intel_chipset.h in favor of using gen_device_info.

2017-04-05 Thread Chris Wilson
On Wed, Apr 05, 2017 at 02:36:38PM -0700, Kenneth Graunke wrote: > On Wednesday, April 5, 2017 3:33:39 AM PDT Chris Wilson wrote: > > On Tue, Apr 04, 2017 at 05:10:15PM -0700, Kenneth Graunke wrote: > > > This moves the PCI ID detection to intel_screen.c and makes > > > drm_bacon_bufmgr_gem_init()

Re: [Mesa-dev] [PATCH 25/53] i965/drm: Drop intel_chipset.h in favor of using gen_device_info.

2017-04-05 Thread Kenneth Graunke
On Wednesday, April 5, 2017 3:33:39 AM PDT Chris Wilson wrote: > On Tue, Apr 04, 2017 at 05:10:15PM -0700, Kenneth Graunke wrote: > > This moves the PCI ID detection to intel_screen.c and makes > > drm_bacon_bufmgr_gem_init() take a devinfo pointer. > > > > We also drop the HAS_LLC query stuff - d

Re: [Mesa-dev] [PATCH 25/53] i965/drm: Drop intel_chipset.h in favor of using gen_device_info.

2017-04-05 Thread Chris Wilson
On Tue, Apr 04, 2017 at 05:10:15PM -0700, Kenneth Graunke wrote: > This moves the PCI ID detection to intel_screen.c and makes > drm_bacon_bufmgr_gem_init() take a devinfo pointer. > > We also drop the HAS_LLC query stuff - devinfo has that info already, > without kernel queries, and it makes no s

[Mesa-dev] [PATCH 25/53] i965/drm: Drop intel_chipset.h in favor of using gen_device_info.

2017-04-04 Thread Kenneth Graunke
This moves the PCI ID detection to intel_screen.c and makes drm_bacon_bufmgr_gem_init() take a devinfo pointer. We also drop the HAS_LLC query stuff - devinfo has that info already, without kernel queries, and it makes no sense to have two has_llc flags set by different mechanisms. --- src/mesa/d