* [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-21 12:53 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-21 12:53 UTC (permalink / raw) To: intel-gfx; +Cc: dri-devel, kernel test robot, Thomas Hellström wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to include asm/smp.h here. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> --- drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c index 1adcd8e02d29..a45d0ec2c5b6 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c @@ -12,6 +12,13 @@ #include "i915_gem_object.h" #include "i915_scatterlist.h" +#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) -- 2.26.3 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-21 12:53 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-21 12:53 UTC (permalink / raw) To: intel-gfx; +Cc: dri-devel, kernel test robot, Thomas Hellström wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to include asm/smp.h here. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> --- drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c index 1adcd8e02d29..a45d0ec2c5b6 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c @@ -12,6 +12,13 @@ #include "i915_gem_object.h" #include "i915_scatterlist.h" +#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) -- 2.26.3 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld @ 2021-10-21 12:53 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-21 12:53 UTC (permalink / raw) To: intel-gfx; +Cc: dri-devel, Thomas Hellström We were overzealous here; even though discrete is non-LLC, it should still be always coherent. Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> --- drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c index a45d0ec2c5b6..848e81368043 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct drm_i915_gem_object *obj) return PTR_ERR(pages); /* XXX: consider doing a vmap flush or something */ - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || + i915_gem_object_can_bypass_llc(obj)) wbinvd_on_all_cpus(); sg_page_sizes = i915_sg_dma_sizes(pages->sgl); -- 2.26.3 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* [Intel-gfx] [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete @ 2021-10-21 12:53 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-21 12:53 UTC (permalink / raw) To: intel-gfx; +Cc: dri-devel, Thomas Hellström We were overzealous here; even though discrete is non-LLC, it should still be always coherent. Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> --- drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c index a45d0ec2c5b6..848e81368043 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct drm_i915_gem_object *obj) return PTR_ERR(pages); /* XXX: consider doing a vmap flush or something */ - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || + i915_gem_object_can_bypass_llc(obj)) wbinvd_on_all_cpus(); sg_page_sizes = i915_sg_dma_sizes(pages->sgl); -- 2.26.3 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld @ 2021-10-22 9:26 ` Thomas Hellström -1 siblings, 0 replies; 38+ messages in thread From: Thomas Hellström @ 2021-10-22 9:26 UTC (permalink / raw) To: Matthew Auld, intel-gfx; +Cc: dri-devel Hi, Matt On 10/21/21 14:53, Matthew Auld wrote: > We were overzealous here; even though discrete is non-LLC, it should > still be always coherent. > > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index a45d0ec2c5b6..848e81368043 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct drm_i915_gem_object *obj) > return PTR_ERR(pages); > > /* XXX: consider doing a vmap flush or something */ > - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) > + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || Q: I notice that DG1 at least has HAS_SNOOP. Would it be incorrect to use that in this case? /Thomas > + i915_gem_object_can_bypass_llc(obj)) > wbinvd_on_all_cpus(); > > sg_page_sizes = i915_sg_dma_sizes(pages->sgl); ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete @ 2021-10-22 9:26 ` Thomas Hellström 0 siblings, 0 replies; 38+ messages in thread From: Thomas Hellström @ 2021-10-22 9:26 UTC (permalink / raw) To: Matthew Auld, intel-gfx; +Cc: dri-devel Hi, Matt On 10/21/21 14:53, Matthew Auld wrote: > We were overzealous here; even though discrete is non-LLC, it should > still be always coherent. > > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index a45d0ec2c5b6..848e81368043 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct drm_i915_gem_object *obj) > return PTR_ERR(pages); > > /* XXX: consider doing a vmap flush or something */ > - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) > + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || Q: I notice that DG1 at least has HAS_SNOOP. Would it be incorrect to use that in this case? /Thomas > + i915_gem_object_can_bypass_llc(obj)) > wbinvd_on_all_cpus(); > > sg_page_sizes = i915_sg_dma_sizes(pages->sgl); ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete 2021-10-22 9:26 ` [Intel-gfx] " Thomas Hellström @ 2021-10-22 9:47 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-22 9:47 UTC (permalink / raw) To: Thomas Hellström, intel-gfx; +Cc: dri-devel On 22/10/2021 10:26, Thomas Hellström wrote: > Hi, Matt > > On 10/21/21 14:53, Matthew Auld wrote: >> We were overzealous here; even though discrete is non-LLC, it should >> still be always coherent. >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index a45d0ec2c5b6..848e81368043 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct >> drm_i915_gem_object *obj) >> return PTR_ERR(pages); >> /* XXX: consider doing a vmap flush or something */ >> - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) >> + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || > > Q: I notice that DG1 at least has HAS_SNOOP. Would it be incorrect to > use that in this case? AFAIK DG1 is special in that CACHE_NONE will still snoop transactions, which is not the case for other HAS_SNOOP platforms. AFAIK that is part of the reason why we also just force CACHE_LLC everywhere on DG1. Could maybe do s/IS_DGFX/IS_DG1/ here? In case that changes on other discrete platforms. And then add a comment. > > /Thomas > > > >> + i915_gem_object_can_bypass_llc(obj)) >> wbinvd_on_all_cpus(); >> sg_page_sizes = i915_sg_dma_sizes(pages->sgl); ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete @ 2021-10-22 9:47 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-22 9:47 UTC (permalink / raw) To: Thomas Hellström, intel-gfx; +Cc: dri-devel On 22/10/2021 10:26, Thomas Hellström wrote: > Hi, Matt > > On 10/21/21 14:53, Matthew Auld wrote: >> We were overzealous here; even though discrete is non-LLC, it should >> still be always coherent. >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index a45d0ec2c5b6..848e81368043 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct >> drm_i915_gem_object *obj) >> return PTR_ERR(pages); >> /* XXX: consider doing a vmap flush or something */ >> - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) >> + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || > > Q: I notice that DG1 at least has HAS_SNOOP. Would it be incorrect to > use that in this case? AFAIK DG1 is special in that CACHE_NONE will still snoop transactions, which is not the case for other HAS_SNOOP platforms. AFAIK that is part of the reason why we also just force CACHE_LLC everywhere on DG1. Could maybe do s/IS_DGFX/IS_DG1/ here? In case that changes on other discrete platforms. And then add a comment. > > /Thomas > > > >> + i915_gem_object_can_bypass_llc(obj)) >> wbinvd_on_all_cpus(); >> sg_page_sizes = i915_sg_dma_sizes(pages->sgl); ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld (?) (?) @ 2021-10-28 15:31 ` Daniel Vetter -1 siblings, 0 replies; 38+ messages in thread From: Daniel Vetter @ 2021-10-28 15:31 UTC (permalink / raw) To: Matthew Auld; +Cc: intel-gfx, dri-devel, Thomas Hellström On Thu, Oct 21, 2021 at 01:53:32PM +0100, Matthew Auld wrote: > We were overzealous here; even though discrete is non-LLC, it should > still be always coherent. > > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index a45d0ec2c5b6..848e81368043 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -251,7 +251,8 @@ static int i915_gem_object_get_pages_dmabuf(struct drm_i915_gem_object *obj) > return PTR_ERR(pages); > > /* XXX: consider doing a vmap flush or something */ > - if (!HAS_LLC(i915) || i915_gem_object_can_bypass_llc(obj)) > + if ((!HAS_LLC(i915) && !IS_DGFX(i915)) || > + i915_gem_object_can_bypass_llc(obj)) > wbinvd_on_all_cpus(); I think a comment here why we do wbinvd and not clflush like everywhere else here would be good. There's the XXX, but it's kinda not explaining why vmap isn't a good idea. Just an aside. -Daneil > > sg_page_sizes = i915_sg_dma_sizes(pages->sgl); > -- > 2.26.3 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 38+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915/dmabuf: fix broken build 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld (?) (?) @ 2021-10-21 14:52 ` Patchwork -1 siblings, 0 replies; 38+ messages in thread From: Patchwork @ 2021-10-21 14:52 UTC (permalink / raw) To: Matthew Auld; +Cc: intel-gfx == Series Details == Series: series starting with [1/2] drm/i915/dmabuf: fix broken build URL : https://patchwork.freedesktop.org/series/96125/ State : warning == Summary == $ dim checkpatch origin/drm-tip 68c88f65e7ba drm/i915/dmabuf: fix broken build -:25: WARNING:INCLUDE_LINUX: Use #include <linux/smp.h> instead of <asm/smp.h> #25: FILE: drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c:16: +#include <asm/smp.h> total: 0 errors, 1 warnings, 0 checks, 13 lines checked 57f4f8cece5b drm/i915/dmabuf: drop the flush on discrete ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld @ 2021-10-21 15:18 ` Dixit, Ashutosh -1 siblings, 0 replies; 38+ messages in thread From: Dixit, Ashutosh @ 2021-10-21 15:18 UTC (permalink / raw) To: Matthew Auld Cc: intel-gfx, dri-devel, kernel test robot, Thomas Hellström On Thu, 21 Oct 2021 05:53:31 -0700, Matthew Auld wrote: > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > include asm/smp.h here. Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index 1adcd8e02d29..a45d0ec2c5b6 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -12,6 +12,13 @@ > #include "i915_gem_object.h" > #include "i915_scatterlist.h" > > +#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) > -- > 2.26.3 > ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-21 15:18 ` Dixit, Ashutosh 0 siblings, 0 replies; 38+ messages in thread From: Dixit, Ashutosh @ 2021-10-21 15:18 UTC (permalink / raw) To: Matthew Auld Cc: intel-gfx, dri-devel, kernel test robot, Thomas Hellström On Thu, 21 Oct 2021 05:53:31 -0700, Matthew Auld wrote: > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > include asm/smp.h here. Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index 1adcd8e02d29..a45d0ec2c5b6 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -12,6 +12,13 @@ > #include "i915_gem_object.h" > #include "i915_scatterlist.h" > > +#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) > -- > 2.26.3 > ^ permalink raw reply [flat|nested] 38+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/dmabuf: fix broken build 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld ` (3 preceding siblings ...) (?) @ 2021-10-21 15:23 ` Patchwork -1 siblings, 0 replies; 38+ messages in thread From: Patchwork @ 2021-10-21 15:23 UTC (permalink / raw) To: Matthew Auld; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 4722 bytes --] == Series Details == Series: series starting with [1/2] drm/i915/dmabuf: fix broken build URL : https://patchwork.freedesktop.org/series/96125/ State : success == Summary == CI Bug Log - changes from CI_DRM_10770 -> Patchwork_21405 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_21405: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_frontbuffer_tracking@basic: - {fi-hsw-gt1}: [PASS][1] -> [DMESG-WARN][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-hsw-gt1/igt@kms_frontbuffer_tracking@basic.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-hsw-gt1/igt@kms_frontbuffer_tracking@basic.html Known issues ------------ Here are the changes found in Patchwork_21405 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@fbdev@write: - fi-bdw-gvtdvm: NOTRUN -> [SKIP][3] ([fdo#109271]) +5 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-bdw-gvtdvm/igt@fbdev@write.html * igt@gem_exec_suspend@basic-s0: - fi-bdw-gvtdvm: NOTRUN -> [INCOMPLETE][4] ([i915#146]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-bdw-gvtdvm/igt@gem_exec_suspend@basic-s0.html #### Possible fixes #### * igt@i915_module_load@reload: - {fi-tgl-dsi}: [DMESG-WARN][5] ([i915#1982]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-tgl-dsi/igt@i915_module_load@reload.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-tgl-dsi/igt@i915_module_load@reload.html * igt@i915_selftest@live@hangcheck: - {fi-hsw-gt1}: [DMESG-WARN][7] ([i915#3303]) -> [PASS][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html * igt@kms_flip@basic-flip-vs-modeset@c-dp1: - fi-cfl-8109u: [FAIL][9] ([i915#4165]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-cfl-8109u/igt@kms_flip@basic-flip-vs-modeset@c-dp1.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-cfl-8109u/igt@kms_flip@basic-flip-vs-modeset@c-dp1.html * igt@kms_frontbuffer_tracking@basic: - fi-cml-u2: [DMESG-WARN][11] ([i915#4269]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html - fi-cfl-8109u: [FAIL][13] ([i915#2546]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/fi-cfl-8109u/igt@kms_frontbuffer_tracking@basic.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/fi-cfl-8109u/igt@kms_frontbuffer_tracking@basic.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#2546]: https://gitlab.freedesktop.org/drm/intel/issues/2546 [i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303 [i915#4165]: https://gitlab.freedesktop.org/drm/intel/issues/4165 [i915#4269]: https://gitlab.freedesktop.org/drm/intel/issues/4269 Participating hosts (38 -> 35) ------------------------------ Additional (1): fi-bdw-gvtdvm Missing (4): fi-ctg-p8600 fi-bsw-cyan fi-ilk-650 fi-hsw-4200u Build changes ------------- * Linux: CI_DRM_10770 -> Patchwork_21405 CI-20190529: 20190529 CI_DRM_10770: 214e8b46143416c4a130cbaeea8430ad9fa19f63 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6258: 4c80c71d7dec29b6376846ae96bd04dc0b6e34d9 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_21405: 57f4f8cece5bb249f150b19f5e32cc88289b66fa @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 57f4f8cece5b drm/i915/dmabuf: drop the flush on discrete 68c88f65e7ba drm/i915/dmabuf: fix broken build == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/index.html [-- Attachment #2: Type: text/html, Size: 5538 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/dmabuf: fix broken build 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld ` (4 preceding siblings ...) (?) @ 2021-10-21 18:16 ` Patchwork -1 siblings, 0 replies; 38+ messages in thread From: Patchwork @ 2021-10-21 18:16 UTC (permalink / raw) To: Matthew Auld; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 30283 bytes --] == Series Details == Series: series starting with [1/2] drm/i915/dmabuf: fix broken build URL : https://patchwork.freedesktop.org/series/96125/ State : success == Summary == CI Bug Log - changes from CI_DRM_10770_full -> Patchwork_21405_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_21405_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@preservation-s3@rcs0: - shard-apl: [PASS][1] -> [DMESG-WARN][2] ([i915#180]) +1 similar issue [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-apl2/igt@gem_ctx_isolation@preservation-s3@rcs0.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl1/igt@gem_ctx_isolation@preservation-s3@rcs0.html * igt@gem_ctx_persistence@legacy-engines-queued: - shard-snb: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099]) +7 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-snb7/igt@gem_ctx_persistence@legacy-engines-queued.html * igt@gem_eio@in-flight-contexts-10ms: - shard-tglb: [PASS][4] -> [TIMEOUT][5] ([i915#3063]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb5/igt@gem_eio@in-flight-contexts-10ms.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb8/igt@gem_eio@in-flight-contexts-10ms.html * igt@gem_eio@unwedge-stress: - shard-tglb: [PASS][6] -> [TIMEOUT][7] ([i915#2369] / [i915#3063] / [i915#3648]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb5/igt@gem_eio@unwedge-stress.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb8/igt@gem_eio@unwedge-stress.html * igt@gem_exec_fair@basic-deadline: - shard-skl: NOTRUN -> [FAIL][8] ([i915#2846]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-tglb: [PASS][9] -> [FAIL][10] ([i915#2842]) +2 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb7/igt@gem_exec_fair@basic-pace-share@rcs0.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-iclb: [PASS][11] -> [FAIL][12] ([i915#2842]) +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb1/igt@gem_exec_fair@basic-pace-solo@rcs0.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb1/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@gem_exec_fair@basic-pace@rcs0: - shard-glk: [PASS][13] -> [FAIL][14] ([i915#2842]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-glk3/igt@gem_exec_fair@basic-pace@rcs0.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-glk9/igt@gem_exec_fair@basic-pace@rcs0.html * igt@gem_exec_params@secure-non-root: - shard-tglb: NOTRUN -> [SKIP][15] ([fdo#112283]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@gem_exec_params@secure-non-root.html * igt@gem_userptr_blits@dmabuf-unsync: - shard-tglb: NOTRUN -> [SKIP][16] ([i915#3297]) +1 similar issue [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@gem_userptr_blits@dmabuf-unsync.html * igt@gem_userptr_blits@vma-merge: - shard-kbl: NOTRUN -> [FAIL][17] ([i915#3318]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl6/igt@gem_userptr_blits@vma-merge.html * igt@gen9_exec_parse@basic-rejected: - shard-tglb: NOTRUN -> [SKIP][18] ([i915#2856]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@gen9_exec_parse@basic-rejected.html * igt@gen9_exec_parse@batch-invalid-length: - shard-snb: NOTRUN -> [SKIP][19] ([fdo#109271]) +455 similar issues [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-snb7/igt@gen9_exec_parse@batch-invalid-length.html * igt@i915_pm_rpm@dpms-mode-unset-non-lpsp: - shard-iclb: NOTRUN -> [SKIP][20] ([fdo#110892]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html * igt@i915_pm_rpm@dpms-non-lpsp: - shard-tglb: NOTRUN -> [SKIP][21] ([fdo#111644] / [i915#1397] / [i915#2411]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@i915_pm_rpm@dpms-non-lpsp.html * igt@i915_pm_rpm@system-suspend-modeset: - shard-tglb: [PASS][22] -> [INCOMPLETE][23] ([i915#2411] / [i915#456]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb6/igt@i915_pm_rpm@system-suspend-modeset.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb7/igt@i915_pm_rpm@system-suspend-modeset.html * igt@i915_pm_sseu@full-enable: - shard-tglb: NOTRUN -> [SKIP][24] ([fdo#109288]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@i915_pm_sseu@full-enable.html * igt@kms_async_flips@alternate-sync-async-flip: - shard-skl: [PASS][25] -> [FAIL][26] ([i915#2521]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl8/igt@kms_async_flips@alternate-sync-async-flip.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl5/igt@kms_async_flips@alternate-sync-async-flip.html * igt@kms_big_fb@x-tiled-16bpp-rotate-90: - shard-tglb: NOTRUN -> [SKIP][27] ([fdo#111614]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_big_fb@x-tiled-16bpp-rotate-90.html * igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-0-hflip: - shard-apl: NOTRUN -> [SKIP][28] ([fdo#109271] / [i915#3777]) +1 similar issue [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-0-hflip.html * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip: - shard-skl: NOTRUN -> [SKIP][29] ([fdo#109271] / [i915#3777]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip.html * igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-async-flip: - shard-skl: NOTRUN -> [FAIL][30] ([i915#3743]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-async-flip.html * igt@kms_ccs@pipe-a-bad-pixel-format-y_tiled_gen12_mc_ccs: - shard-skl: NOTRUN -> [SKIP][31] ([fdo#109271] / [i915#3886]) +1 similar issue [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_ccs@pipe-a-bad-pixel-format-y_tiled_gen12_mc_ccs.html * igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_mc_ccs: - shard-apl: NOTRUN -> [SKIP][32] ([fdo#109271] / [i915#3886]) +11 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl1/igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_mc_ccs.html * igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_rc_ccs: - shard-iclb: NOTRUN -> [SKIP][33] ([fdo#109278]) +3 similar issues [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_rc_ccs.html * igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_mc_ccs: - shard-kbl: NOTRUN -> [SKIP][34] ([fdo#109271] / [i915#3886]) +1 similar issue [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl6/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_mc_ccs.html * igt@kms_ccs@pipe-a-missing-ccs-buffer-yf_tiled_ccs: - shard-tglb: NOTRUN -> [SKIP][35] ([i915#3689]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_ccs@pipe-a-missing-ccs-buffer-yf_tiled_ccs.html * igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_mc_ccs: - shard-iclb: NOTRUN -> [SKIP][36] ([fdo#109278] / [i915#3886]) [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_mc_ccs.html * igt@kms_ccs@pipe-d-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc: - shard-kbl: NOTRUN -> [SKIP][37] ([fdo#109271]) +79 similar issues [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl6/igt@kms_ccs@pipe-d-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html * igt@kms_cdclk@mode-transition: - shard-apl: NOTRUN -> [SKIP][38] ([fdo#109271]) +210 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_cdclk@mode-transition.html * igt@kms_chamelium@dp-hpd-enable-disable-mode: - shard-iclb: NOTRUN -> [SKIP][39] ([fdo#109284] / [fdo#111827]) +1 similar issue [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_chamelium@dp-hpd-enable-disable-mode.html * igt@kms_chamelium@dp-mode-timings: - shard-apl: NOTRUN -> [SKIP][40] ([fdo#109271] / [fdo#111827]) +16 similar issues [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl8/igt@kms_chamelium@dp-mode-timings.html * igt@kms_chamelium@hdmi-mode-timings: - shard-snb: NOTRUN -> [SKIP][41] ([fdo#109271] / [fdo#111827]) +24 similar issues [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-snb7/igt@kms_chamelium@hdmi-mode-timings.html * igt@kms_color@pipe-a-ctm-0-5: - shard-skl: [PASS][42] -> [DMESG-WARN][43] ([i915#1982]) +1 similar issue [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl9/igt@kms_color@pipe-a-ctm-0-5.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl5/igt@kms_color@pipe-a-ctm-0-5.html * igt@kms_color_chamelium@pipe-b-ctm-max: - shard-skl: NOTRUN -> [SKIP][44] ([fdo#109271] / [fdo#111827]) +4 similar issues [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_color_chamelium@pipe-b-ctm-max.html * igt@kms_color_chamelium@pipe-c-ctm-0-25: - shard-tglb: NOTRUN -> [SKIP][45] ([fdo#109284] / [fdo#111827]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_color_chamelium@pipe-c-ctm-0-25.html * igt@kms_color_chamelium@pipe-c-ctm-limited-range: - shard-kbl: NOTRUN -> [SKIP][46] ([fdo#109271] / [fdo#111827]) +3 similar issues [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl6/igt@kms_color_chamelium@pipe-c-ctm-limited-range.html * igt@kms_color_chamelium@pipe-d-degamma: - shard-iclb: NOTRUN -> [SKIP][47] ([fdo#109278] / [fdo#109284] / [fdo#111827]) [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_color_chamelium@pipe-d-degamma.html * igt@kms_content_protection@srm: - shard-apl: NOTRUN -> [TIMEOUT][48] ([i915#1319]) [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_content_protection@srm.html * igt@kms_cursor_crc@pipe-b-cursor-32x32-random: - shard-tglb: NOTRUN -> [SKIP][49] ([i915#3319]) [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_cursor_crc@pipe-b-cursor-32x32-random.html * igt@kms_cursor_crc@pipe-b-cursor-512x170-random: - shard-tglb: NOTRUN -> [SKIP][50] ([fdo#109279] / [i915#3359]) [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@kms_cursor_crc@pipe-b-cursor-512x170-random.html * igt@kms_cursor_crc@pipe-c-cursor-32x10-rapid-movement: - shard-tglb: NOTRUN -> [SKIP][51] ([i915#3359]) [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_cursor_crc@pipe-c-cursor-32x10-rapid-movement.html * igt@kms_cursor_crc@pipe-d-cursor-suspend: - shard-tglb: [PASS][52] -> [INCOMPLETE][53] ([i915#2411] / [i915#4211]) [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb2/igt@kms_cursor_crc@pipe-d-cursor-suspend.html [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb7/igt@kms_cursor_crc@pipe-d-cursor-suspend.html * igt@kms_cursor_legacy@2x-long-cursor-vs-flip-legacy: - shard-tglb: NOTRUN -> [SKIP][54] ([fdo#111825]) +2 similar issues [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-legacy.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions: - shard-skl: [PASS][55] -> [FAIL][56] ([i915#2346]) [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-kbl: [PASS][57] -> [INCOMPLETE][58] ([i915#180] / [i915#636]) [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl6/igt@kms_fbcon_fbt@fbc-suspend.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@kms_fbcon_fbt@fbc-suspend.html * igt@kms_flip@2x-plain-flip-ts-check-interruptible: - shard-iclb: NOTRUN -> [SKIP][59] ([fdo#109274]) +1 similar issue [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_flip@2x-plain-flip-ts-check-interruptible.html * igt@kms_flip@flip-vs-suspend-interruptible@a-dp1: - shard-kbl: [PASS][60] -> [DMESG-WARN][61] ([i915#180]) +7 similar issues [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl4/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html * igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs: - shard-iclb: [PASS][62] -> [SKIP][63] ([i915#3701]) [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb1/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs.html [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb2/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs.html * igt@kms_frontbuffer_tracking@fbc-farfromfence-mmap-gtt: - shard-skl: NOTRUN -> [SKIP][64] ([fdo#109271]) +38 similar issues [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_frontbuffer_tracking@fbc-farfromfence-mmap-gtt.html * igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-gtt: - shard-iclb: NOTRUN -> [SKIP][65] ([fdo#109280]) [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-gtt.html * igt@kms_hdr@bpc-switch-suspend: - shard-skl: [PASS][66] -> [FAIL][67] ([i915#1188]) [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl1/igt@kms_hdr@bpc-switch-suspend.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl2/igt@kms_hdr@bpc-switch-suspend.html * igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d: - shard-kbl: NOTRUN -> [SKIP][68] ([fdo#109271] / [i915#533]) [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl2/igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d.html * igt@kms_plane_alpha_blend@pipe-a-alpha-7efc: - shard-kbl: NOTRUN -> [FAIL][69] ([fdo#108145] / [i915#265]) +1 similar issue [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl7/igt@kms_plane_alpha_blend@pipe-a-alpha-7efc.html * igt@kms_plane_alpha_blend@pipe-a-alpha-basic: - shard-apl: NOTRUN -> [FAIL][70] ([fdo#108145] / [i915#265]) +1 similar issue [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_plane_alpha_blend@pipe-a-alpha-basic.html * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc: - shard-skl: NOTRUN -> [FAIL][71] ([fdo#108145] / [i915#265]) [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [PASS][72] -> [FAIL][73] ([fdo#108145] / [i915#265]) +1 similar issue [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_plane_lowres@pipe-b-tiling-yf: - shard-iclb: NOTRUN -> [SKIP][74] ([i915#3536]) [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@kms_plane_lowres@pipe-b-tiling-yf.html * igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping: - shard-apl: NOTRUN -> [SKIP][75] ([fdo#109271] / [i915#2733]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping.html * igt@kms_psr2_sf@cursor-plane-update-sf: - shard-skl: NOTRUN -> [SKIP][76] ([fdo#109271] / [i915#658]) [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl6/igt@kms_psr2_sf@cursor-plane-update-sf.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-apl: NOTRUN -> [SKIP][77] ([fdo#109271] / [i915#658]) +2 similar issues [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2: - shard-kbl: NOTRUN -> [SKIP][78] ([fdo#109271] / [i915#658]) [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2.html * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5: - shard-tglb: NOTRUN -> [SKIP][79] ([i915#2920]) [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html * igt@kms_psr@psr2_cursor_plane_move: - shard-iclb: [PASS][80] -> [SKIP][81] ([fdo#109441]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb2/igt@kms_psr@psr2_cursor_plane_move.html [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb1/igt@kms_psr@psr2_cursor_plane_move.html * igt@kms_sysfs_edid_timing: - shard-apl: NOTRUN -> [FAIL][82] ([IGT#2]) [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl8/igt@kms_sysfs_edid_timing.html * igt@kms_vblank@pipe-a-ts-continuation-suspend: - shard-tglb: [PASS][83] -> [INCOMPLETE][84] ([i915#456]) [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb1/igt@kms_vblank@pipe-a-ts-continuation-suspend.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb7/igt@kms_vblank@pipe-a-ts-continuation-suspend.html * igt@kms_vblank@pipe-d-wait-idle: - shard-apl: NOTRUN -> [SKIP][85] ([fdo#109271] / [i915#533]) +2 similar issues [85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_vblank@pipe-d-wait-idle.html * igt@kms_writeback@writeback-check-output: - shard-apl: NOTRUN -> [SKIP][86] ([fdo#109271] / [i915#2437]) [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_writeback@writeback-check-output.html * igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame: - shard-tglb: NOTRUN -> [SKIP][87] ([i915#2530]) [87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame.html * igt@prime_vgem@fence-write-hang: - shard-tglb: NOTRUN -> [SKIP][88] ([fdo#109295]) [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb6/igt@prime_vgem@fence-write-hang.html * igt@sysfs_clients@fair-1: - shard-apl: NOTRUN -> [SKIP][89] ([fdo#109271] / [i915#2994]) +3 similar issues [89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@sysfs_clients@fair-1.html * igt@sysfs_clients@fair-3: - shard-iclb: NOTRUN -> [SKIP][90] ([i915#2994]) [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@sysfs_clients@fair-3.html * igt@sysfs_clients@sema-10: - shard-kbl: NOTRUN -> [SKIP][91] ([fdo#109271] / [i915#2994]) +1 similar issue [91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl7/igt@sysfs_clients@sema-10.html #### Possible fixes #### * igt@drm_mm@all@evict_range: - shard-skl: [INCOMPLETE][92] ([i915#2485]) -> [PASS][93] [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl8/igt@drm_mm@all@evict_range.html [93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl8/igt@drm_mm@all@evict_range.html * igt@gem_ctx_isolation@preservation-s3@vcs0: - shard-kbl: [DMESG-WARN][94] ([i915#180]) -> [PASS][95] +6 similar issues [94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl4/igt@gem_ctx_isolation@preservation-s3@vcs0.html [95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl6/igt@gem_ctx_isolation@preservation-s3@vcs0.html * igt@gem_exec_fair@basic-deadline: - shard-glk: [FAIL][96] ([i915#2846]) -> [PASS][97] [96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-glk3/igt@gem_exec_fair@basic-deadline.html [97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-glk6/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-pace@vecs0: - shard-tglb: [FAIL][98] ([i915#2842]) -> [PASS][99] [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb3/igt@gem_exec_fair@basic-pace@vecs0.html [99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb5/igt@gem_exec_fair@basic-pace@vecs0.html * igt@gem_exec_nop@basic-sequential: - shard-glk: [DMESG-WARN][100] ([i915#118]) -> [PASS][101] [100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-glk2/igt@gem_exec_nop@basic-sequential.html [101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-glk1/igt@gem_exec_nop@basic-sequential.html * igt@gem_exec_whisper@basic-queues-priority: - shard-iclb: [INCOMPLETE][102] ([i915#1895]) -> [PASS][103] [102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb7/igt@gem_exec_whisper@basic-queues-priority.html [103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@gem_exec_whisper@basic-queues-priority.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-tglb: [INCOMPLETE][104] ([i915#456]) -> [PASS][105] [104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb7/igt@kms_fbcon_fbt@fbc-suspend.html [105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_fbcon_fbt@fbc-suspend.html * igt@kms_flip@flip-vs-suspend-interruptible@a-dp1: - shard-apl: [DMESG-WARN][106] ([i915#180]) -> [PASS][107] [106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-apl1/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html [107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html * igt@kms_flip@plain-flip-fb-recreate@a-edp1: - shard-skl: [FAIL][108] ([i915#2122]) -> [PASS][109] [108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl10/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html [109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl4/igt@kms_flip@plain-flip-fb-recreate@a-edp1.html * igt@kms_frontbuffer_tracking@fbc-suspend: - shard-tglb: [INCOMPLETE][110] ([i915#2828] / [i915#456]) -> [PASS][111] [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-tglb7/igt@kms_frontbuffer_tracking@fbc-suspend.html [111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-tglb1/igt@kms_frontbuffer_tracking@fbc-suspend.html * igt@kms_hdr@bpc-switch: - shard-skl: [FAIL][112] ([i915#1188]) -> [PASS][113] [112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl1/igt@kms_hdr@bpc-switch.html [113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl2/igt@kms_hdr@bpc-switch.html * igt@kms_properties@connector-properties-legacy: - shard-kbl: [DMESG-WARN][114] ([i915#165] / [i915#180]) -> [PASS][115] [114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl2/igt@kms_properties@connector-properties-legacy.html [115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@kms_properties@connector-properties-legacy.html * igt@kms_psr@psr2_cursor_mmap_cpu: - shard-iclb: [SKIP][116] ([fdo#109441]) -> [PASS][117] [116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb4/igt@kms_psr@psr2_cursor_mmap_cpu.html [117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html * igt@perf@polling-parameterized: - shard-skl: [FAIL][118] ([i915#1542]) -> [PASS][119] [118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-skl10/igt@perf@polling-parameterized.html [119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-skl4/igt@perf@polling-parameterized.html #### Warnings #### * igt@gem_exec_fair@basic-none-rrul@rcs0: - shard-iclb: [FAIL][120] ([i915#2842]) -> [FAIL][121] ([i915#2852]) [120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb8/igt@gem_exec_fair@basic-none-rrul@rcs0.html [121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb7/igt@gem_exec_fair@basic-none-rrul@rcs0.html * igt@i915_pm_rc6_residency@rc6-fence: - shard-iclb: [WARN][122] ([i915#2684]) -> [WARN][123] ([i915#1804] / [i915#2684]) [122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb5/igt@i915_pm_rc6_residency@rc6-fence.html [123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb4/igt@i915_pm_rc6_residency@rc6-fence.html * igt@kms_flip@flip-vs-suspend@c-dp1: - shard-kbl: [INCOMPLETE][124] ([i915#636]) -> [DMESG-WARN][125] ([i915#180]) [124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl2/igt@kms_flip@flip-vs-suspend@c-dp1.html [125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@kms_flip@flip-vs-suspend@c-dp1.html * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1: - shard-iclb: [SKIP][126] ([i915#658]) -> [SKIP][127] ([i915#2920]) +4 similar issues [126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb4/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1.html [127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-iclb: [SKIP][128] ([i915#2920]) -> [SKIP][129] ([i915#658]) [128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-iclb2/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html [129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-iclb3/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@runner@aborted: - shard-kbl: ([FAIL][130], [FAIL][131], [FAIL][132], [FAIL][133], [FAIL][134], [FAIL][135]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#4312]) -> ([FAIL][136], [FAIL][137], [FAIL][138], [FAIL][139], [FAIL][140], [FAIL][141], [FAIL][142], [FAIL][143]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#4312] / [i915#92]) [130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl4/igt@runner@aborted.html [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl4/igt@runner@aborted.html [132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl7/igt@runner@aborted.html [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl1/igt@runner@aborted.html [134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl1/igt@runner@aborted.html [135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10770/shard-kbl4/igt@runner@aborted.html [136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl4/igt@runner@aborted.html [137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl4/igt@runner@aborted.html [138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl4/igt@runner@aborted.html [139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl4/igt@runner@aborted.html [140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl7/igt@runner@aborted.html [141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@runner@aborted.html [142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@runner@aborted.html [143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/shard-kbl1/igt@runner@aborted.html - shard-skl: ([FAIL][144], [FAIL][145]) ([i915#3002] / [i915#3363] / [i915#4312]) -> ([FAIL][146], [FAIL][147], [FAIL][148]) ([i915#1814] / [i915#2029] / [i915#3002] / [i915#3363] / [i915#4312]) [144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_ == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_21405/index.html [-- Attachment #2: Type: text/html, Size: 34040 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld @ 2021-10-27 7:57 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 7:57 UTC (permalink / raw) To: Matthew Auld, Jani Nikula Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > include asm/smp.h here. > > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> Jani, would it make sense to cherry-pick this to -fixes? The offending commit is in drm-next, and there have been a few reports around this. Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index 1adcd8e02d29..a45d0ec2c5b6 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -12,6 +12,13 @@ > #include "i915_gem_object.h" > #include "i915_scatterlist.h" > > +#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) > -- > 2.26.3 > ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 7:57 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 7:57 UTC (permalink / raw) To: Matthew Auld, Jani Nikula Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > include asm/smp.h here. > > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> Jani, would it make sense to cherry-pick this to -fixes? The offending commit is in drm-next, and there have been a few reports around this. Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > --- > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > index 1adcd8e02d29..a45d0ec2c5b6 100644 > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > @@ -12,6 +12,13 @@ > #include "i915_gem_object.h" > #include "i915_scatterlist.h" > > +#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) > -- > 2.26.3 > ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 7:57 ` [Intel-gfx] " Matthew Auld @ 2021-10-27 8:58 ` Jani Nikula -1 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 8:58 UTC (permalink / raw) To: Matthew Auld, Matthew Auld Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> include asm/smp.h here. >> >> Reported-by: kernel test robot <lkp@intel.com> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > Jani, would it make sense to cherry-pick this to -fixes? The offending > commit is in drm-next, and there have been a few reports around this. > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") If the Fixes: tag is in place, our tooling will cherry-pick it where it belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) Cc: Rodrigo who covers drm-intel-next-fixes atm. BR, Jani. > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >> >> +#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) >> -- >> 2.26.3 >> -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 8:58 ` Jani Nikula 0 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 8:58 UTC (permalink / raw) To: Matthew Auld, Matthew Auld Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> include asm/smp.h here. >> >> Reported-by: kernel test robot <lkp@intel.com> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > Jani, would it make sense to cherry-pick this to -fixes? The offending > commit is in drm-next, and there have been a few reports around this. > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") If the Fixes: tag is in place, our tooling will cherry-pick it where it belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) Cc: Rodrigo who covers drm-intel-next-fixes atm. BR, Jani. > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >> >> +#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) >> -- >> 2.26.3 >> -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 8:58 ` [Intel-gfx] " Jani Nikula @ 2021-10-27 9:00 ` Jani Nikula -1 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 9:00 UTC (permalink / raw) To: Matthew Auld, Matthew Auld Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: >> On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >>> >>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >>> include asm/smp.h here. >>> >>> Reported-by: kernel test robot <lkp@intel.com> >>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> >> Jani, would it make sense to cherry-pick this to -fixes? The offending >> commit is in drm-next, and there have been a few reports around this. >> >> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > If the Fixes: tag is in place, our tooling will cherry-pick it where it > belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > Cc: Rodrigo who covers drm-intel-next-fixes atm. PS. 'dim tc a035154da45d' tells you where that commit is: drm/drm-next drm-intel/drm-intel-gt-next drm-misc/drm-misc-next drm-misc/topic/amdgpu-dp2.0-mst So we see it's not in Linus' tree. > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >>> >>> +#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) >>> -- >>> 2.26.3 >>> -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 9:00 ` Jani Nikula 0 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 9:00 UTC (permalink / raw) To: Matthew Auld, Matthew Auld Cc: Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: >> On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >>> >>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >>> include asm/smp.h here. >>> >>> Reported-by: kernel test robot <lkp@intel.com> >>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> >> Jani, would it make sense to cherry-pick this to -fixes? The offending >> commit is in drm-next, and there have been a few reports around this. >> >> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > If the Fixes: tag is in place, our tooling will cherry-pick it where it > belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > Cc: Rodrigo who covers drm-intel-next-fixes atm. PS. 'dim tc a035154da45d' tells you where that commit is: drm/drm-next drm-intel/drm-intel-gt-next drm-misc/drm-misc-next drm-misc/topic/amdgpu-dp2.0-mst So we see it's not in Linus' tree. > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >>> >>> +#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) >>> -- >>> 2.26.3 >>> -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 8:58 ` [Intel-gfx] " Jani Nikula @ 2021-10-27 9:03 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 9:03 UTC (permalink / raw) To: Jani Nikula Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> > >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> include asm/smp.h here. > >> > >> Reported-by: kernel test robot <lkp@intel.com> > >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > > Jani, would it make sense to cherry-pick this to -fixes? The offending > > commit is in drm-next, and there have been a few reports around this. > > > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > If the Fixes: tag is in place, our tooling will cherry-pick it where it > belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) Yeah, I forgot to add the fixes tag here unfortunately. > > Cc: Rodrigo who covers drm-intel-next-fixes atm. > > BR, > Jani. > > > > > > > > >> --- > >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> @@ -12,6 +12,13 @@ > >> #include "i915_gem_object.h" > >> #include "i915_scatterlist.h" > >> > >> +#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) > >> -- > >> 2.26.3 > >> > > -- > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 9:03 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 9:03 UTC (permalink / raw) To: Jani Nikula Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> > >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> include asm/smp.h here. > >> > >> Reported-by: kernel test robot <lkp@intel.com> > >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > > Jani, would it make sense to cherry-pick this to -fixes? The offending > > commit is in drm-next, and there have been a few reports around this. > > > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > If the Fixes: tag is in place, our tooling will cherry-pick it where it > belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) Yeah, I forgot to add the fixes tag here unfortunately. > > Cc: Rodrigo who covers drm-intel-next-fixes atm. > > BR, > Jani. > > > > > > > > >> --- > >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> @@ -12,6 +12,13 @@ > >> #include "i915_gem_object.h" > >> #include "i915_scatterlist.h" > >> > >> +#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) > >> -- > >> 2.26.3 > >> > > -- > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 9:03 ` [Intel-gfx] " Matthew Auld @ 2021-10-27 9:44 ` Jani Nikula -1 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 9:44 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: >> >> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: >> > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> >> include asm/smp.h here. >> >> >> >> Reported-by: kernel test robot <lkp@intel.com> >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> > >> > Jani, would it make sense to cherry-pick this to -fixes? The offending >> > commit is in drm-next, and there have been a few reports around this. >> > >> > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") >> >> If the Fixes: tag is in place, our tooling will cherry-pick it where it >> belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > Yeah, I forgot to add the fixes tag here unfortunately. Already merged? What's the commit id to be cherry-picked? Rodrigo can do it manually. Note to self, we should set up some way to check which maintainer is responsible for which branches and when. BR, Jani. > >> >> Cc: Rodrigo who covers drm-intel-next-fixes atm. >> >> BR, >> Jani. >> >> >> >> > >> > >> >> --- >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> >> 1 file changed, 7 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> @@ -12,6 +12,13 @@ >> >> #include "i915_gem_object.h" >> >> #include "i915_scatterlist.h" >> >> >> >> +#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) >> >> -- >> >> 2.26.3 >> >> >> >> -- >> Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 9:44 ` Jani Nikula 0 siblings, 0 replies; 38+ messages in thread From: Jani Nikula @ 2021-10-27 9:44 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: >> >> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: >> > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> >> include asm/smp.h here. >> >> >> >> Reported-by: kernel test robot <lkp@intel.com> >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> > >> > Jani, would it make sense to cherry-pick this to -fixes? The offending >> > commit is in drm-next, and there have been a few reports around this. >> > >> > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") >> >> If the Fixes: tag is in place, our tooling will cherry-pick it where it >> belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > Yeah, I forgot to add the fixes tag here unfortunately. Already merged? What's the commit id to be cherry-picked? Rodrigo can do it manually. Note to self, we should set up some way to check which maintainer is responsible for which branches and when. BR, Jani. > >> >> Cc: Rodrigo who covers drm-intel-next-fixes atm. >> >> BR, >> Jani. >> >> >> >> > >> > >> >> --- >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> >> 1 file changed, 7 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> @@ -12,6 +12,13 @@ >> >> #include "i915_gem_object.h" >> >> #include "i915_scatterlist.h" >> >> >> >> +#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) >> >> -- >> >> 2.26.3 >> >> >> >> -- >> Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 9:44 ` [Intel-gfx] " Jani Nikula @ 2021-10-27 9:48 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 9:48 UTC (permalink / raw) To: Jani Nikula Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021 at 10:44, Jani Nikula <jani.nikula@linux.intel.com> wrote: > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > > On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: > >> > >> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > >> > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> >> > >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> >> include asm/smp.h here. > >> >> > >> >> Reported-by: kernel test robot <lkp@intel.com> > >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > >> > > >> > Jani, would it make sense to cherry-pick this to -fixes? The offending > >> > commit is in drm-next, and there have been a few reports around this. > >> > > >> > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > >> > >> If the Fixes: tag is in place, our tooling will cherry-pick it where it > >> belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > > > Yeah, I forgot to add the fixes tag here unfortunately. > > Already merged? What's the commit id to be cherry-picked? Rodrigo can do > it manually. Yeah, it was merged to gt-next: 777226dac058 ("drm/i915/dmabuf: fix broken build") > > Note to self, we should set up some way to check which maintainer is > responsible for which branches and when. > > BR, > Jani. > > > > >> > >> Cc: Rodrigo who covers drm-intel-next-fixes atm. > >> > >> BR, > >> Jani. > >> > >> > >> > >> > > >> > > >> >> --- > >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> >> 1 file changed, 7 insertions(+) > >> >> > >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> @@ -12,6 +12,13 @@ > >> >> #include "i915_gem_object.h" > >> >> #include "i915_scatterlist.h" > >> >> > >> >> +#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) > >> >> -- > >> >> 2.26.3 > >> >> > >> > >> -- > >> Jani Nikula, Intel Open Source Graphics Center > > -- > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 9:48 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 9:48 UTC (permalink / raw) To: Jani Nikula Cc: Matthew Auld, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström, Rodrigo Vivi On Wed, 27 Oct 2021 at 10:44, Jani Nikula <jani.nikula@linux.intel.com> wrote: > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > > On Wed, 27 Oct 2021 at 09:58, Jani Nikula <jani.nikula@linux.intel.com> wrote: > >> > >> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> wrote: > >> > On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> >> > >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> >> include asm/smp.h here. > >> >> > >> >> Reported-by: kernel test robot <lkp@intel.com> > >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > >> > > >> > Jani, would it make sense to cherry-pick this to -fixes? The offending > >> > commit is in drm-next, and there have been a few reports around this. > >> > > >> > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > >> > >> If the Fixes: tag is in place, our tooling will cherry-pick it where it > >> belongs. (In this case, drm-intel-next-fixes, not drm-intel-fixes.) > > > > Yeah, I forgot to add the fixes tag here unfortunately. > > Already merged? What's the commit id to be cherry-picked? Rodrigo can do > it manually. Yeah, it was merged to gt-next: 777226dac058 ("drm/i915/dmabuf: fix broken build") > > Note to self, we should set up some way to check which maintainer is > responsible for which branches and when. > > BR, > Jani. > > > > >> > >> Cc: Rodrigo who covers drm-intel-next-fixes atm. > >> > >> BR, > >> Jani. > >> > >> > >> > >> > > >> > > >> >> --- > >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> >> 1 file changed, 7 insertions(+) > >> >> > >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> >> @@ -12,6 +12,13 @@ > >> >> #include "i915_gem_object.h" > >> >> #include "i915_scatterlist.h" > >> >> > >> >> +#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) > >> >> -- > >> >> 2.26.3 > >> >> > >> > >> -- > >> Jani Nikula, Intel Open Source Graphics Center > > -- > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 9:48 ` [Intel-gfx] " Matthew Auld @ 2021-10-28 1:43 ` Vivi, Rodrigo -1 siblings, 0 replies; 38+ messages in thread From: Vivi, Rodrigo @ 2021-10-28 1:43 UTC (permalink / raw) To: jani.nikula, matthew.william.auld Cc: dri-devel, intel-gfx, Auld, Matthew, lkp, thomas.hellstrom On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: > On Wed, 27 Oct 2021 at 10:44, Jani Nikula > <jani.nikula@linux.intel.com> wrote: > > > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> > > wrote: > > > On Wed, 27 Oct 2021 at 09:58, Jani Nikula > > > <jani.nikula@linux.intel.com> wrote: > > > > > > > > On Wed, 27 Oct 2021, Matthew Auld > > > > <matthew.william.auld@gmail.com> wrote: > > > > > On Thu, 21 Oct 2021 at 13:54, Matthew Auld > > > > > <matthew.auld@intel.com> wrote: > > > > > > > > > > > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus > > > > > > we need to > > > > > > include asm/smp.h here. > > > > > > > > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > > > > > > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > > > > > > > > Jani, would it make sense to cherry-pick this to -fixes? The > > > > > offending > > > > > commit is in drm-next, and there have been a few reports > > > > > around this. > > > > > > > > > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- > > > > > acquire") > > > > > > > > If the Fixes: tag is in place, our tooling will cherry-pick it > > > > where it > > > > belongs. (In this case, drm-intel-next-fixes, not drm-intel- > > > > fixes.) > > > > > > Yeah, I forgot to add the fixes tag here unfortunately. > > > > Already merged? What's the commit id to be cherry-picked? Rodrigo > > can do > > it manually. > > Yeah, it was merged to gt-next: > > 777226dac058 ("drm/i915/dmabuf: fix broken build") picked up to drm-intel-next-fixes thanks, Rodrigo. > > > > > Note to self, we should set up some way to check which maintainer > > is > > responsible for which branches and when. > > > > BR, > > Jani. > > > > > > > > > > > > > Cc: Rodrigo who covers drm-intel-next-fixes atm. > > > > > > > > BR, > > > > Jani. > > > > > > > > > > > > > > > > > > > > > > > > > > > > --- > > > > > > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > > > > > > 1 file changed, 7 insertions(+) > > > > > > > > > > > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > index 1adcd8e02d29..a45d0ec2c5b6 100644 > > > > > > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > @@ -12,6 +12,13 @@ > > > > > > #include "i915_gem_object.h" > > > > > > #include "i915_scatterlist.h" > > > > > > > > > > > > +#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) > > > > > > -- > > > > > > 2.26.3 > > > > > > > > > > > > > > -- > > > > Jani Nikula, Intel Open Source Graphics Center > > > > -- > > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-28 1:43 ` Vivi, Rodrigo 0 siblings, 0 replies; 38+ messages in thread From: Vivi, Rodrigo @ 2021-10-28 1:43 UTC (permalink / raw) To: jani.nikula, matthew.william.auld Cc: dri-devel, intel-gfx, Auld, Matthew, lkp, thomas.hellstrom On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: > On Wed, 27 Oct 2021 at 10:44, Jani Nikula > <jani.nikula@linux.intel.com> wrote: > > > > On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> > > wrote: > > > On Wed, 27 Oct 2021 at 09:58, Jani Nikula > > > <jani.nikula@linux.intel.com> wrote: > > > > > > > > On Wed, 27 Oct 2021, Matthew Auld > > > > <matthew.william.auld@gmail.com> wrote: > > > > > On Thu, 21 Oct 2021 at 13:54, Matthew Auld > > > > > <matthew.auld@intel.com> wrote: > > > > > > > > > > > > wbinvd_on_all_cpus() is only defined on x86 it seems, plus > > > > > > we need to > > > > > > include asm/smp.h here. > > > > > > > > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > > > > Signed-off-by: Matthew Auld <matthew.auld@intel.com> > > > > > > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > > > > > > > > Jani, would it make sense to cherry-pick this to -fixes? The > > > > > offending > > > > > commit is in drm-next, and there have been a few reports > > > > > around this. > > > > > > > > > > Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- > > > > > acquire") > > > > > > > > If the Fixes: tag is in place, our tooling will cherry-pick it > > > > where it > > > > belongs. (In this case, drm-intel-next-fixes, not drm-intel- > > > > fixes.) > > > > > > Yeah, I forgot to add the fixes tag here unfortunately. > > > > Already merged? What's the commit id to be cherry-picked? Rodrigo > > can do > > it manually. > > Yeah, it was merged to gt-next: > > 777226dac058 ("drm/i915/dmabuf: fix broken build") picked up to drm-intel-next-fixes thanks, Rodrigo. > > > > > Note to self, we should set up some way to check which maintainer > > is > > responsible for which branches and when. > > > > BR, > > Jani. > > > > > > > > > > > > > Cc: Rodrigo who covers drm-intel-next-fixes atm. > > > > > > > > BR, > > > > Jani. > > > > > > > > > > > > > > > > > > > > > > > > > > > > --- > > > > > > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > > > > > > 1 file changed, 7 insertions(+) > > > > > > > > > > > > diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > index 1adcd8e02d29..a45d0ec2c5b6 100644 > > > > > > --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > > > > > @@ -12,6 +12,13 @@ > > > > > > #include "i915_gem_object.h" > > > > > > #include "i915_scatterlist.h" > > > > > > > > > > > > +#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) > > > > > > -- > > > > > > 2.26.3 > > > > > > > > > > > > > > -- > > > > Jani Nikula, Intel Open Source Graphics Center > > > > -- > > Jani Nikula, Intel Open Source Graphics Center ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-28 1:43 ` [Intel-gfx] " Vivi, Rodrigo @ 2021-10-28 8:48 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-28 8:48 UTC (permalink / raw) To: Vivi, Rodrigo, jani.nikula, matthew.william.auld Cc: dri-devel, intel-gfx, lkp, thomas.hellstrom On 28/10/2021 02:43, Vivi, Rodrigo wrote: > On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: >> On Wed, 27 Oct 2021 at 10:44, Jani Nikula >> <jani.nikula@linux.intel.com> wrote: >>> >>> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> >>> wrote: >>>> On Wed, 27 Oct 2021 at 09:58, Jani Nikula >>>> <jani.nikula@linux.intel.com> wrote: >>>>> >>>>> On Wed, 27 Oct 2021, Matthew Auld >>>>> <matthew.william.auld@gmail.com> wrote: >>>>>> On Thu, 21 Oct 2021 at 13:54, Matthew Auld >>>>>> <matthew.auld@intel.com> wrote: >>>>>>> >>>>>>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus >>>>>>> we need to >>>>>>> include asm/smp.h here. >>>>>>> >>>>>>> Reported-by: kernel test robot <lkp@intel.com> >>>>>>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>>>>>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >>>>>> >>>>>> Jani, would it make sense to cherry-pick this to -fixes? The >>>>>> offending >>>>>> commit is in drm-next, and there have been a few reports >>>>>> around this. >>>>>> >>>>>> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- >>>>>> acquire") >>>>> >>>>> If the Fixes: tag is in place, our tooling will cherry-pick it >>>>> where it >>>>> belongs. (In this case, drm-intel-next-fixes, not drm-intel- >>>>> fixes.) >>>> >>>> Yeah, I forgot to add the fixes tag here unfortunately. >>> >>> Already merged? What's the commit id to be cherry-picked? Rodrigo >>> can do >>> it manually. >> >> Yeah, it was merged to gt-next: >> >> 777226dac058 ("drm/i915/dmabuf: fix broken build") > > picked up to drm-intel-next-fixes Thanks. > > thanks, > Rodrigo. > >> >>> >>> Note to self, we should set up some way to check which maintainer >>> is >>> responsible for which branches and when. >>> >>> BR, >>> Jani. >>> >>>> >>>>> >>>>> Cc: Rodrigo who covers drm-intel-next-fixes atm. >>>>> >>>>> BR, >>>>> Jani. >>>>> >>>>> >>>>> >>>>>> >>>>>> >>>>>>> --- >>>>>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >>>>>>> 1 file changed, 7 insertions(+) >>>>>>> >>>>>>> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> index 1adcd8e02d29..a45d0ec2c5b6 100644 >>>>>>> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> @@ -12,6 +12,13 @@ >>>>>>> #include "i915_gem_object.h" >>>>>>> #include "i915_scatterlist.h" >>>>>>> >>>>>>> +#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) >>>>>>> -- >>>>>>> 2.26.3 >>>>>>> >>>>> >>>>> -- >>>>> Jani Nikula, Intel Open Source Graphics Center >>> >>> -- >>> Jani Nikula, Intel Open Source Graphics Center > ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-28 8:48 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-28 8:48 UTC (permalink / raw) To: Vivi, Rodrigo, jani.nikula, matthew.william.auld Cc: dri-devel, intel-gfx, lkp, thomas.hellstrom On 28/10/2021 02:43, Vivi, Rodrigo wrote: > On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: >> On Wed, 27 Oct 2021 at 10:44, Jani Nikula >> <jani.nikula@linux.intel.com> wrote: >>> >>> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> >>> wrote: >>>> On Wed, 27 Oct 2021 at 09:58, Jani Nikula >>>> <jani.nikula@linux.intel.com> wrote: >>>>> >>>>> On Wed, 27 Oct 2021, Matthew Auld >>>>> <matthew.william.auld@gmail.com> wrote: >>>>>> On Thu, 21 Oct 2021 at 13:54, Matthew Auld >>>>>> <matthew.auld@intel.com> wrote: >>>>>>> >>>>>>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus >>>>>>> we need to >>>>>>> include asm/smp.h here. >>>>>>> >>>>>>> Reported-by: kernel test robot <lkp@intel.com> >>>>>>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>>>>>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >>>>>> >>>>>> Jani, would it make sense to cherry-pick this to -fixes? The >>>>>> offending >>>>>> commit is in drm-next, and there have been a few reports >>>>>> around this. >>>>>> >>>>>> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- >>>>>> acquire") >>>>> >>>>> If the Fixes: tag is in place, our tooling will cherry-pick it >>>>> where it >>>>> belongs. (In this case, drm-intel-next-fixes, not drm-intel- >>>>> fixes.) >>>> >>>> Yeah, I forgot to add the fixes tag here unfortunately. >>> >>> Already merged? What's the commit id to be cherry-picked? Rodrigo >>> can do >>> it manually. >> >> Yeah, it was merged to gt-next: >> >> 777226dac058 ("drm/i915/dmabuf: fix broken build") > > picked up to drm-intel-next-fixes Thanks. > > thanks, > Rodrigo. > >> >>> >>> Note to self, we should set up some way to check which maintainer >>> is >>> responsible for which branches and when. >>> >>> BR, >>> Jani. >>> >>>> >>>>> >>>>> Cc: Rodrigo who covers drm-intel-next-fixes atm. >>>>> >>>>> BR, >>>>> Jani. >>>>> >>>>> >>>>> >>>>>> >>>>>> >>>>>>> --- >>>>>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >>>>>>> 1 file changed, 7 insertions(+) >>>>>>> >>>>>>> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> index 1adcd8e02d29..a45d0ec2c5b6 100644 >>>>>>> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>> @@ -12,6 +12,13 @@ >>>>>>> #include "i915_gem_object.h" >>>>>>> #include "i915_scatterlist.h" >>>>>>> >>>>>>> +#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) >>>>>>> -- >>>>>>> 2.26.3 >>>>>>> >>>>> >>>>> -- >>>>> Jani Nikula, Intel Open Source Graphics Center >>> >>> -- >>> Jani Nikula, Intel Open Source Graphics Center > ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-28 8:48 ` [Intel-gfx] " Matthew Auld @ 2021-11-22 13:34 ` Tvrtko Ursulin -1 siblings, 0 replies; 38+ messages in thread From: Tvrtko Ursulin @ 2021-11-22 13:34 UTC (permalink / raw) To: Matthew Auld, Vivi, Rodrigo, jani.nikula, matthew.william.auld Cc: thomas.hellstrom, lkp, intel-gfx, dri-devel On 28/10/2021 09:48, Matthew Auld wrote: > On 28/10/2021 02:43, Vivi, Rodrigo wrote: >> On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: >>> On Wed, 27 Oct 2021 at 10:44, Jani Nikula >>> <jani.nikula@linux.intel.com> wrote: >>>> >>>> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> >>>> wrote: >>>>> On Wed, 27 Oct 2021 at 09:58, Jani Nikula >>>>> <jani.nikula@linux.intel.com> wrote: >>>>>> >>>>>> On Wed, 27 Oct 2021, Matthew Auld >>>>>> <matthew.william.auld@gmail.com> wrote: >>>>>>> On Thu, 21 Oct 2021 at 13:54, Matthew Auld >>>>>>> <matthew.auld@intel.com> wrote: >>>>>>>> >>>>>>>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus >>>>>>>> we need to >>>>>>>> include asm/smp.h here. >>>>>>>> >>>>>>>> Reported-by: kernel test robot <lkp@intel.com> >>>>>>>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>>>>>>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >>>>>>> >>>>>>> Jani, would it make sense to cherry-pick this to -fixes? The >>>>>>> offending >>>>>>> commit is in drm-next, and there have been a few reports >>>>>>> around this. >>>>>>> >>>>>>> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- >>>>>>> acquire") >>>>>> >>>>>> If the Fixes: tag is in place, our tooling will cherry-pick it >>>>>> where it >>>>>> belongs. (In this case, drm-intel-next-fixes, not drm-intel- >>>>>> fixes.) >>>>> >>>>> Yeah, I forgot to add the fixes tag here unfortunately. >>>> >>>> Already merged? What's the commit id to be cherry-picked? Rodrigo >>>> can do >>>> it manually. >>> >>> Yeah, it was merged to gt-next: >>> >>> 777226dac058 ("drm/i915/dmabuf: fix broken build") >> >> picked up to drm-intel-next-fixes > > Thanks. Hmm it looks like we have this in -fixes (and in -gt) while Linus has explicitly thrown it out (see https://lists.freedesktop.org/archives/dri-devel/2021-November/330928.html). End result is that now we have a different Kernel Test Robot warning ("asm/smp.h is included more than once."). So we need to fish out "drm/i915/dmabuf: fix broken build" out from fixes with a revert or what? Regards, Tvrtko > >> >> thanks, >> Rodrigo. >> >>> >>>> >>>> Note to self, we should set up some way to check which maintainer >>>> is >>>> responsible for which branches and when. >>>> >>>> BR, >>>> Jani. >>>> >>>>> >>>>>> >>>>>> Cc: Rodrigo who covers drm-intel-next-fixes atm. >>>>>> >>>>>> BR, >>>>>> Jani. >>>>>> >>>>>> >>>>>> >>>>>>> >>>>>>> >>>>>>>> --- >>>>>>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >>>>>>>> 1 file changed, 7 insertions(+) >>>>>>>> >>>>>>>> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> index 1adcd8e02d29..a45d0ec2c5b6 100644 >>>>>>>> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> @@ -12,6 +12,13 @@ >>>>>>>> #include "i915_gem_object.h" >>>>>>>> #include "i915_scatterlist.h" >>>>>>>> >>>>>>>> +#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) >>>>>>>> -- >>>>>>>> 2.26.3 >>>>>>>> >>>>>> >>>>>> -- >>>>>> Jani Nikula, Intel Open Source Graphics Center >>>> >>>> -- >>>> Jani Nikula, Intel Open Source Graphics Center >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-11-22 13:34 ` Tvrtko Ursulin 0 siblings, 0 replies; 38+ messages in thread From: Tvrtko Ursulin @ 2021-11-22 13:34 UTC (permalink / raw) To: Matthew Auld, Vivi, Rodrigo, jani.nikula, matthew.william.auld Cc: thomas.hellstrom, intel-gfx, dri-devel On 28/10/2021 09:48, Matthew Auld wrote: > On 28/10/2021 02:43, Vivi, Rodrigo wrote: >> On Wed, 2021-10-27 at 10:48 +0100, Matthew Auld wrote: >>> On Wed, 27 Oct 2021 at 10:44, Jani Nikula >>> <jani.nikula@linux.intel.com> wrote: >>>> >>>> On Wed, 27 Oct 2021, Matthew Auld <matthew.william.auld@gmail.com> >>>> wrote: >>>>> On Wed, 27 Oct 2021 at 09:58, Jani Nikula >>>>> <jani.nikula@linux.intel.com> wrote: >>>>>> >>>>>> On Wed, 27 Oct 2021, Matthew Auld >>>>>> <matthew.william.auld@gmail.com> wrote: >>>>>>> On Thu, 21 Oct 2021 at 13:54, Matthew Auld >>>>>>> <matthew.auld@intel.com> wrote: >>>>>>>> >>>>>>>> wbinvd_on_all_cpus() is only defined on x86 it seems, plus >>>>>>>> we need to >>>>>>>> include asm/smp.h here. >>>>>>>> >>>>>>>> Reported-by: kernel test robot <lkp@intel.com> >>>>>>>> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >>>>>>>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >>>>>>> >>>>>>> Jani, would it make sense to cherry-pick this to -fixes? The >>>>>>> offending >>>>>>> commit is in drm-next, and there have been a few reports >>>>>>> around this. >>>>>>> >>>>>>> Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on- >>>>>>> acquire") >>>>>> >>>>>> If the Fixes: tag is in place, our tooling will cherry-pick it >>>>>> where it >>>>>> belongs. (In this case, drm-intel-next-fixes, not drm-intel- >>>>>> fixes.) >>>>> >>>>> Yeah, I forgot to add the fixes tag here unfortunately. >>>> >>>> Already merged? What's the commit id to be cherry-picked? Rodrigo >>>> can do >>>> it manually. >>> >>> Yeah, it was merged to gt-next: >>> >>> 777226dac058 ("drm/i915/dmabuf: fix broken build") >> >> picked up to drm-intel-next-fixes > > Thanks. Hmm it looks like we have this in -fixes (and in -gt) while Linus has explicitly thrown it out (see https://lists.freedesktop.org/archives/dri-devel/2021-November/330928.html). End result is that now we have a different Kernel Test Robot warning ("asm/smp.h is included more than once."). So we need to fish out "drm/i915/dmabuf: fix broken build" out from fixes with a revert or what? Regards, Tvrtko > >> >> thanks, >> Rodrigo. >> >>> >>>> >>>> Note to self, we should set up some way to check which maintainer >>>> is >>>> responsible for which branches and when. >>>> >>>> BR, >>>> Jani. >>>> >>>>> >>>>>> >>>>>> Cc: Rodrigo who covers drm-intel-next-fixes atm. >>>>>> >>>>>> BR, >>>>>> Jani. >>>>>> >>>>>> >>>>>> >>>>>>> >>>>>>> >>>>>>>> --- >>>>>>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >>>>>>>> 1 file changed, 7 insertions(+) >>>>>>>> >>>>>>>> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> index 1adcd8e02d29..a45d0ec2c5b6 100644 >>>>>>>> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >>>>>>>> @@ -12,6 +12,13 @@ >>>>>>>> #include "i915_gem_object.h" >>>>>>>> #include "i915_scatterlist.h" >>>>>>>> >>>>>>>> +#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) >>>>>>>> -- >>>>>>>> 2.26.3 >>>>>>>> >>>>>> >>>>>> -- >>>>>> Jani Nikula, Intel Open Source Graphics Center >>>> >>>> -- >>>> Jani Nikula, Intel Open Source Graphics Center >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 7:57 ` [Intel-gfx] " Matthew Auld @ 2021-10-27 14:54 ` Lucas De Marchi -1 siblings, 0 replies; 38+ messages in thread From: Lucas De Marchi @ 2021-10-27 14:54 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> include asm/smp.h here. >> >> Reported-by: kernel test robot <lkp@intel.com> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > >Jani, would it make sense to cherry-pick this to -fixes? The offending >commit is in drm-next, and there have been a few reports around this. > >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >> >> +#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__) not sure I understand why this is a fix. Sure, it's true, but right now this file can't be built on any other arch. For clflush, wbind, etc, I'd rather change the code to use things from drm_cache rather than ifdef it out locally. This is "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? Lucas De Marchi >> +#endif >> + >> I915_SELFTEST_DECLARE(static bool force_different_devices;) >> >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >> -- >> 2.26.3 >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 14:54 ` Lucas De Marchi 0 siblings, 0 replies; 38+ messages in thread From: Lucas De Marchi @ 2021-10-27 14:54 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> include asm/smp.h here. >> >> Reported-by: kernel test robot <lkp@intel.com> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > >Jani, would it make sense to cherry-pick this to -fixes? The offending >commit is in drm-next, and there have been a few reports around this. > >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > >> --- >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> @@ -12,6 +12,13 @@ >> #include "i915_gem_object.h" >> #include "i915_scatterlist.h" >> >> +#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__) not sure I understand why this is a fix. Sure, it's true, but right now this file can't be built on any other arch. For clflush, wbind, etc, I'd rather change the code to use things from drm_cache rather than ifdef it out locally. This is "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? Lucas De Marchi >> +#endif >> + >> I915_SELFTEST_DECLARE(static bool force_different_devices;) >> >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >> -- >> 2.26.3 >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 14:54 ` Lucas De Marchi @ 2021-10-27 16:23 ` Matthew Auld -1 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 16:23 UTC (permalink / raw) To: Lucas De Marchi Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, 27 Oct 2021 at 15:54, Lucas De Marchi <lucas.demarchi@intel.com> wrote: > > On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: > >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> > >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> include asm/smp.h here. > >> > >> Reported-by: kernel test robot <lkp@intel.com> > >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > >Jani, would it make sense to cherry-pick this to -fixes? The offending > >commit is in drm-next, and there have been a few reports around this. > > > >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > > > > >> --- > >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> @@ -12,6 +12,13 @@ > >> #include "i915_gem_object.h" > >> #include "i915_scatterlist.h" > >> > >> +#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__) > > > not sure I understand why this is a fix. Sure, it's true, but right now > this file can't be built on any other arch. > > For clflush, wbind, etc, I'd rather change the code to use things from > drm_cache rather than ifdef it out locally. This is > "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? I guess it depends on the kernel config, but it reported: error: implicit declaration of function 'wbinvd_on_all_cpus' AFAIK it's the missing <asm/smp.h>, the CONFIG_X86 thing is just for good measure. I didn't spot anything in drm_cache which just offers a simple wrapper for wbinvd? > > Lucas De Marchi > > >> +#endif > >> + > >> I915_SELFTEST_DECLARE(static bool force_different_devices;) > >> > >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) > >> -- > >> 2.26.3 > >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 16:23 ` Matthew Auld 0 siblings, 0 replies; 38+ messages in thread From: Matthew Auld @ 2021-10-27 16:23 UTC (permalink / raw) To: Lucas De Marchi Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, 27 Oct 2021 at 15:54, Lucas De Marchi <lucas.demarchi@intel.com> wrote: > > On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: > >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: > >> > >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to > >> include asm/smp.h here. > >> > >> Reported-by: kernel test robot <lkp@intel.com> > >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> > >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> > > > >Jani, would it make sense to cherry-pick this to -fixes? The offending > >commit is in drm-next, and there have been a few reports around this. > > > >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") > > > > > >> --- > >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> index 1adcd8e02d29..a45d0ec2c5b6 100644 > >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > >> @@ -12,6 +12,13 @@ > >> #include "i915_gem_object.h" > >> #include "i915_scatterlist.h" > >> > >> +#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__) > > > not sure I understand why this is a fix. Sure, it's true, but right now > this file can't be built on any other arch. > > For clflush, wbind, etc, I'd rather change the code to use things from > drm_cache rather than ifdef it out locally. This is > "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? I guess it depends on the kernel config, but it reported: error: implicit declaration of function 'wbinvd_on_all_cpus' AFAIK it's the missing <asm/smp.h>, the CONFIG_X86 thing is just for good measure. I didn't spot anything in drm_cache which just offers a simple wrapper for wbinvd? > > Lucas De Marchi > > >> +#endif > >> + > >> I915_SELFTEST_DECLARE(static bool force_different_devices;) > >> > >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) > >> -- > >> 2.26.3 > >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] drm/i915/dmabuf: fix broken build 2021-10-27 16:23 ` [Intel-gfx] " Matthew Auld @ 2021-10-27 17:16 ` Lucas De Marchi -1 siblings, 0 replies; 38+ messages in thread From: Lucas De Marchi @ 2021-10-27 17:16 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, Oct 27, 2021 at 05:23:59PM +0100, Matthew Auld wrote: >On Wed, 27 Oct 2021 at 15:54, Lucas De Marchi <lucas.demarchi@intel.com> wrote: >> >> On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: >> >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> >> include asm/smp.h here. >> >> >> >> Reported-by: kernel test robot <lkp@intel.com> >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> > >> >Jani, would it make sense to cherry-pick this to -fixes? The offending >> >commit is in drm-next, and there have been a few reports around this. >> > >> >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") >> > >> > >> >> --- >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> >> 1 file changed, 7 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> @@ -12,6 +12,13 @@ >> >> #include "i915_gem_object.h" >> >> #include "i915_scatterlist.h" >> >> >> >> +#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__) >> >> >> not sure I understand why this is a fix. Sure, it's true, but right now >> this file can't be built on any other arch. >> >> For clflush, wbind, etc, I'd rather change the code to use things from >> drm_cache rather than ifdef it out locally. This is >> "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? > >I guess it depends on the kernel config, but it reported: >error: implicit declaration of function 'wbinvd_on_all_cpus' oh, ok. The commit message was not very clear. Just checked and this also matches what is done in gem/i915_gem_pm.c (although gt/intel_ggtt.c just includes it unconditionally) > >AFAIK it's the missing <asm/smp.h>, the CONFIG_X86 thing is just for >good measure. > >I didn't spot anything in drm_cache which just offers a simple wrapper >for wbinvd? there isn't yet, but it's something we are adding as we add suppport for other archs. thanks Lucas De Marchi > >> >> Lucas De Marchi >> >> >> +#endif >> >> + >> >> I915_SELFTEST_DECLARE(static bool force_different_devices;) >> >> >> >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >> >> -- >> >> 2.26.3 >> >> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [Intel-gfx] [PATCH 1/2] drm/i915/dmabuf: fix broken build @ 2021-10-27 17:16 ` Lucas De Marchi 0 siblings, 0 replies; 38+ messages in thread From: Lucas De Marchi @ 2021-10-27 17:16 UTC (permalink / raw) To: Matthew Auld Cc: Matthew Auld, Jani Nikula, Intel Graphics Development, ML dri-devel, kernel test robot, Thomas Hellström On Wed, Oct 27, 2021 at 05:23:59PM +0100, Matthew Auld wrote: >On Wed, 27 Oct 2021 at 15:54, Lucas De Marchi <lucas.demarchi@intel.com> wrote: >> >> On Wed, Oct 27, 2021 at 08:57:48AM +0100, Matthew Auld wrote: >> >On Thu, 21 Oct 2021 at 13:54, Matthew Auld <matthew.auld@intel.com> wrote: >> >> >> >> wbinvd_on_all_cpus() is only defined on x86 it seems, plus we need to >> >> include asm/smp.h here. >> >> >> >> Reported-by: kernel test robot <lkp@intel.com> >> >> Signed-off-by: Matthew Auld <matthew.auld@intel.com> >> >> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com> >> > >> >Jani, would it make sense to cherry-pick this to -fixes? The offending >> >commit is in drm-next, and there have been a few reports around this. >> > >> >Fixes: a035154da45d ("drm/i915/dmabuf: add paranoid flush-on-acquire") >> > >> > >> >> --- >> >> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 7 +++++++ >> >> 1 file changed, 7 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> index 1adcd8e02d29..a45d0ec2c5b6 100644 >> >> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c >> >> @@ -12,6 +12,13 @@ >> >> #include "i915_gem_object.h" >> >> #include "i915_scatterlist.h" >> >> >> >> +#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__) >> >> >> not sure I understand why this is a fix. Sure, it's true, but right now >> this file can't be built on any other arch. >> >> For clflush, wbind, etc, I'd rather change the code to use things from >> drm_cache rather than ifdef it out locally. This is >> "Reported-by: kernel test robot <lkp@intel.com>", but what's the error? > >I guess it depends on the kernel config, but it reported: >error: implicit declaration of function 'wbinvd_on_all_cpus' oh, ok. The commit message was not very clear. Just checked and this also matches what is done in gem/i915_gem_pm.c (although gt/intel_ggtt.c just includes it unconditionally) > >AFAIK it's the missing <asm/smp.h>, the CONFIG_X86 thing is just for >good measure. > >I didn't spot anything in drm_cache which just offers a simple wrapper >for wbinvd? there isn't yet, but it's something we are adding as we add suppport for other archs. thanks Lucas De Marchi > >> >> Lucas De Marchi >> >> >> +#endif >> >> + >> >> I915_SELFTEST_DECLARE(static bool force_different_devices;) >> >> >> >> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf) >> >> -- >> >> 2.26.3 >> >> ^ permalink raw reply [flat|nested] 38+ messages in thread
end of thread, other threads:[~2021-11-22 13:34 UTC | newest] Thread overview: 38+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-10-21 12:53 [PATCH 1/2] drm/i915/dmabuf: fix broken build Matthew Auld 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld 2021-10-21 12:53 ` [PATCH 2/2] drm/i915/dmabuf: drop the flush on discrete Matthew Auld 2021-10-21 12:53 ` [Intel-gfx] " Matthew Auld 2021-10-22 9:26 ` Thomas Hellström 2021-10-22 9:26 ` [Intel-gfx] " Thomas Hellström 2021-10-22 9:47 ` Matthew Auld 2021-10-22 9:47 ` [Intel-gfx] " Matthew Auld 2021-10-28 15:31 ` Daniel Vetter 2021-10-21 14:52 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] drm/i915/dmabuf: fix broken build Patchwork 2021-10-21 15:18 ` [PATCH 1/2] " Dixit, Ashutosh 2021-10-21 15:18 ` [Intel-gfx] " Dixit, Ashutosh 2021-10-21 15:23 ` [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] " Patchwork 2021-10-21 18:16 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 2021-10-27 7:57 ` [PATCH 1/2] " Matthew Auld 2021-10-27 7:57 ` [Intel-gfx] " Matthew Auld 2021-10-27 8:58 ` Jani Nikula 2021-10-27 8:58 ` [Intel-gfx] " Jani Nikula 2021-10-27 9:00 ` Jani Nikula 2021-10-27 9:00 ` [Intel-gfx] " Jani Nikula 2021-10-27 9:03 ` Matthew Auld 2021-10-27 9:03 ` [Intel-gfx] " Matthew Auld 2021-10-27 9:44 ` Jani Nikula 2021-10-27 9:44 ` [Intel-gfx] " Jani Nikula 2021-10-27 9:48 ` Matthew Auld 2021-10-27 9:48 ` [Intel-gfx] " Matthew Auld 2021-10-28 1:43 ` Vivi, Rodrigo 2021-10-28 1:43 ` [Intel-gfx] " Vivi, Rodrigo 2021-10-28 8:48 ` Matthew Auld 2021-10-28 8:48 ` [Intel-gfx] " Matthew Auld 2021-11-22 13:34 ` Tvrtko Ursulin 2021-11-22 13:34 ` [Intel-gfx] " Tvrtko Ursulin 2021-10-27 14:54 ` Lucas De Marchi 2021-10-27 14:54 ` Lucas De Marchi 2021-10-27 16:23 ` Matthew Auld 2021-10-27 16:23 ` [Intel-gfx] " Matthew Auld 2021-10-27 17:16 ` Lucas De Marchi 2021-10-27 17:16 ` [Intel-gfx] " Lucas De Marchi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.