On Wed, 26 Jan 2022, Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: > On 1/26/22 21:04, akpm@xxxxxxxxxxxxxxxxxxxx wrote: >> The mm-of-the-moment snapshot 2022-01-26-21-04 has been uploaded to >> >> https://www.ozlabs.org/~akpm/mmotm/ >> >> mmotm-readme.txt says >> >> README for mm-of-the-moment: >> >> https://www.ozlabs.org/~akpm/mmotm/ >> >> This is a snapshot of my -mm patch queue. Uploaded at random hopefully >> more than once a week. >> >> You will need quilt to apply these patches to the latest Linus release (5.x >> or 5.x-rcY). The series file is in broken-out.tar.gz and is duplicated in >> https://ozlabs.org/~akpm/mmotm/series >> >> The file broken-out.tar.gz contains two datestamp files: .DATE and >> .DATE-yyyy-mm-dd-hh-mm-ss. Both contain the string yyyy-mm-dd-hh-mm-ss, >> followed by the base kernel version against which this patch series is to >> be applied. > > on x86_64: > (from linux-next.patch) > > > HDRTEST drivers/gpu/drm/i915/i915_gem_evict.h > In file included from <command-line>:0:0: > ./../drivers/gpu/drm/i915/i915_gem_evict.h:15:15: error: ‘struct i915_gem_ww_ctx’ declared inside parameter list will not be visible outside of this definition or declaration [-Werror] > struct i915_gem_ww_ctx *ww, > ^~~~~~~~~~~~~~~ > ./../drivers/gpu/drm/i915/i915_gem_evict.h:21:14: error: ‘struct i915_gem_ww_ctx’ declared inside parameter list will not be visible outside of this definition or declaration [-Werror] > struct i915_gem_ww_ctx *ww, > ^~~~~~~~~~~~~~~ > ./../drivers/gpu/drm/i915/i915_gem_evict.h:25:16: error: ‘struct i915_gem_ww_ctx’ declared inside parameter list will not be visible outside of this definition or declaration [-Werror] > struct i915_gem_ww_ctx *ww); > ^~~~~~~~~~~~~~~ > cc1: all warnings being treated as errors Thanks for the report. This is only visible with CONFIG_DRM_I915_WERROR=y which depends on COMPILE_TEST=n. We use the "HDRTEST" and -Werror for development and CI to keep the driver clean, but it's not really intended for general consumption. Usually when something like this even hits the tree it's because of a merge mishap somewhere down the line. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center