[Intel-gfx] [PATCH 27/38] drm/i915: Split struct intel_context definition to its own header
Chris Wilson
chris at chris-wilson.co.uk
Tue Mar 5 16:35:14 UTC 2019
Quoting Tvrtko Ursulin (2019-03-05 16:19:12)
>
> On 01/03/2019 14:03, Chris Wilson wrote:
> > This complex struct pulling in half the driver deserves its own
> > isolation in preparation for intel_context becoming an outright
> > complicated class of its own.
> >
> > In order to split this beast into its own header also requests splitting
> > several of its dependent types and their dependencies into their own
> > headers as well.
>
> I don't feel like I need to read this one in detail. If it compiles it
> should be good.
Hmm, one thing I started doing for the next series is
# Test the headers are compilable as standalone units
i915-$(CONFIG_DRM_I915_WERROR) += \
gem/test_i915_gemfs_standalone.o \
gem/test_i915_gem_clflush_standalone.o \
gem/test_i915_gem_context_standalone.o \
gem/test_i915_gem_context_types_standalone.o \
gem/test_i915_gem_ioctls_standalone.o \
gem/test_i915_gem_object_standalone.o \
gem/test_i915_gem_object_types_standalone.o
which would be good here as it helps with those fuzzy feelings that the
compiler is doing all the checking.
-Chris
More information about the Intel-gfx
mailing list