Message ID | 20211122135758.85444-1-tvrtko.ursulin@linux.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Revert "drm/i915/dmabuf: fix broken build" | expand |
On 22/11/2021 13:57, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > > This reverts commit 777226dac058d119286b4081953cb5aa2cb7394b. > > Approach taken in the patch was rejected by Linus and the upstream tree > now already contains the required include directive via 304ac8032d3f > ("Merge tag 'drm-next-2021-11-12' of git://anongit.freedesktop.org/drm/drm"). > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > Fixes: 777226dac058 ("drm/i915/dmabuf: fix broken build") > Cc: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: Jani Nikula <jani.nikula@intel.com> Acked-by: Matthew Auld <matthew.auld@intel.com> This was copy-paste from gem/i915_gem_pm.c, does that need a similar patch? > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index f291cf4c3886..1b526039a60d 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -17,13 +17,6 @@ > > MODULE_IMPORT_NS(DMA_BUF); > > -#if defined(CONFIG_X86) > -#include <asm/smp.h> > -#else > -#define wbinvd_on_all_cpus() \ > - pr_warn(DRIVER_NAME ": Missing cache flush in %s\n", __func__) > -#endif > - > I915_SELFTEST_DECLARE(static bool force_different_devices;) > > static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >
On 22/11/2021 14:04, Matthew Auld wrote: > On 22/11/2021 13:57, Tvrtko Ursulin wrote: >> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> >> This reverts commit 777226dac058d119286b4081953cb5aa2cb7394b. >> >> Approach taken in the patch was rejected by Linus and the upstream tree >> now already contains the required include directive via 304ac8032d3f >> ("Merge tag 'drm-next-2021-11-12' of >> git://anongit.freedesktop.org/drm/drm"). >> >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> Fixes: 777226dac058 ("drm/i915/dmabuf: fix broken build") >> Cc: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> >> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> >> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> >> Cc: Jani Nikula <jani.nikula@intel.com> > > Acked-by: Matthew Auld <matthew.auld@intel.com> > > This was copy-paste from gem/i915_gem_pm.c, does that need a similar patch? Personally I don't think we need to act immediately and can wait until the !x86 build efforts crystalize things for us a bit. At least the commentary and commit in i915_gem_pm.c give rationale on pros and cons of clflush vs wbinvd. And commentary in i915_gem_dmabuf.c makes it sound we cannot simply go to clflush there in all cases. Regards, Tvrtko > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 ------- >> 1 file changed, 7 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index f291cf4c3886..1b526039a60d 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -17,13 +17,6 @@ >> MODULE_IMPORT_NS(DMA_BUF); >> -#if defined(CONFIG_X86) >> -#include <asm/smp.h> >> -#else >> -#define wbinvd_on_all_cpus() \ >> - pr_warn(DRIVER_NAME ": Missing cache flush in %s\n", __func__) >> -#endif >> - >> I915_SELFTEST_DECLARE(static bool force_different_devices;) >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >>
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c index f291cf4c3886..1b526039a60d 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c @@ -17,13 +17,6 @@ MODULE_IMPORT_NS(DMA_BUF); -#if defined(CONFIG_X86) -#include <asm/smp.h> -#else -#define wbinvd_on_all_cpus() \ - pr_warn(DRIVER_NAME ": Missing cache flush in %s\n", __func__) -#endif - I915_SELFTEST_DECLARE(static bool force_different_devices;) static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf)