* [Intel-gfx] [PATCH v3 0/3] Move #define wbvind_on_all_cpus
@ 2022-02-22 17:26 ` Michael Cheng
0 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx; +Cc: michael.cheng, lucas.demarchi, dri-devel
This series moves the logic for wbvind_on_all_cpus to drm_cache. The logic
changes a little here, if platform is not x86 then we throw out a warning
for when wbvind_on_all_cpus is being called.
v2(Michael Cheng): Move and redo logic for wbvind_on_all_cpus. Also
add drm_cache.h where the function is being called and
remove uneeded header files.
v3(Michael Cheng): Updated commit messages.
Michael Cheng (3):
drm_cache: Add logic for wbvind_on_all_cpus
drm/i915/gem: Remove logic for wbinvd_on_all_cpus
drm/i915/: Add drm_cache.h
drivers/gpu/drm/drm_cache.c | 2 --
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 2 +-
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 7 +------
drivers/gpu/drm/i915/gt/intel_ggtt.c | 2 +-
include/drm/drm_cache.h | 6 ++++++
5 files changed, 9 insertions(+), 10 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
@ 2022-02-22 17:26 ` Michael Cheng
-1 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx
Cc: tvrtko.ursulin, michael.cheng, balasubramani.vivekanandan,
wayne.boyer, casey.g.bowman, lucas.demarchi, dri-devel
Add logic for wbvind_on_all_cpus for non-x86 platforms.
v2(Michael Cheng): Change logic to if platform is not x86, then
we add pr_warn for calling wbvind_on_all_cpus.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/drm_cache.c | 2 --
include/drm/drm_cache.h | 6 ++++++
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_cache.c b/drivers/gpu/drm/drm_cache.c
index 66597e411764..722e3931d68a 100644
--- a/drivers/gpu/drm/drm_cache.c
+++ b/drivers/gpu/drm/drm_cache.c
@@ -40,8 +40,6 @@
#define MEMCPY_BOUNCE_SIZE 128
#if defined(CONFIG_X86)
-#include <asm/smp.h>
-
/*
* clflushopt is an unordered instruction which needs fencing with mfence or
* sfence to avoid ordering issues. For drm_clflush_page this fencing happens
diff --git a/include/drm/drm_cache.h b/include/drm/drm_cache.h
index 22deb216b59c..24fcf6be1419 100644
--- a/include/drm/drm_cache.h
+++ b/include/drm/drm_cache.h
@@ -34,6 +34,12 @@
#define _DRM_CACHE_H_
#include <linux/scatterlist.h>
+#include <asm/smp.h>
+
+#if !defined(CONFIG_x86)
+#define wbinvd_on_all_cpus() \
+ pr_warn("Missing cache flush in %s\n", __func__)
+#endif
struct iosys_map;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [Intel-gfx] [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
@ 2022-02-22 17:26 ` Michael Cheng
0 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx; +Cc: michael.cheng, lucas.demarchi, dri-devel
Add logic for wbvind_on_all_cpus for non-x86 platforms.
v2(Michael Cheng): Change logic to if platform is not x86, then
we add pr_warn for calling wbvind_on_all_cpus.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/drm_cache.c | 2 --
include/drm/drm_cache.h | 6 ++++++
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_cache.c b/drivers/gpu/drm/drm_cache.c
index 66597e411764..722e3931d68a 100644
--- a/drivers/gpu/drm/drm_cache.c
+++ b/drivers/gpu/drm/drm_cache.c
@@ -40,8 +40,6 @@
#define MEMCPY_BOUNCE_SIZE 128
#if defined(CONFIG_X86)
-#include <asm/smp.h>
-
/*
* clflushopt is an unordered instruction which needs fencing with mfence or
* sfence to avoid ordering issues. For drm_clflush_page this fencing happens
diff --git a/include/drm/drm_cache.h b/include/drm/drm_cache.h
index 22deb216b59c..24fcf6be1419 100644
--- a/include/drm/drm_cache.h
+++ b/include/drm/drm_cache.h
@@ -34,6 +34,12 @@
#define _DRM_CACHE_H_
#include <linux/scatterlist.h>
+#include <asm/smp.h>
+
+#if !defined(CONFIG_x86)
+#define wbinvd_on_all_cpus() \
+ pr_warn("Missing cache flush in %s\n", __func__)
+#endif
struct iosys_map;
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
@ 2022-02-22 21:07 ` kernel test robot
-1 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 21:07 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: tvrtko.ursulin, kbuild-all, balasubramani.vivekanandan,
wayne.boyer, casey.g.bowman, lucas.demarchi, dri-devel,
michael.cheng
Hi Michael,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: nios2-randconfig-r003-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230507.uLh3qqTV-lkp@intel.com/config)
compiler: nios2-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=nios2 SHELL=/bin/bash drivers/gpu/drm/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/drm_cache.c:37:
>> include/drm/drm_cache.h:37:10: fatal error: asm/smp.h: No such file or directory
37 | #include <asm/smp.h>
| ^~~~~~~~~~~
compilation terminated.
vim +37 include/drm/drm_cache.h
35
36 #include <linux/scatterlist.h>
> 37 #include <asm/smp.h>
38
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
@ 2022-02-22 21:07 ` kernel test robot
0 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 21:07 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: kbuild-all, lucas.demarchi, dri-devel, michael.cheng
Hi Michael,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-intel/for-linux-next]
[also build test ERROR on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: nios2-randconfig-r003-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230507.uLh3qqTV-lkp@intel.com/config)
compiler: nios2-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=nios2 SHELL=/bin/bash drivers/gpu/drm/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/drm_cache.c:37:
>> include/drm/drm_cache.h:37:10: fatal error: asm/smp.h: No such file or directory
37 | #include <asm/smp.h>
| ^~~~~~~~~~~
compilation terminated.
vim +37 include/drm/drm_cache.h
35
36 #include <linux/scatterlist.h>
> 37 #include <asm/smp.h>
38
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
(?)
@ 2022-02-22 23:31 ` kernel test robot
-1 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 23:31 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: tvrtko.ursulin, casey.g.bowman, kbuild-all,
balasubramani.vivekanandan, wayne.boyer, llvm, lucas.demarchi,
dri-devel, michael.cheng
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: hexagon-randconfig-r041-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230702.Ya7PkY7G-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/drm_cache.c:37:
In file included from include/drm/drm_cache.h:37:
>> arch/hexagon/include/asm/smp.h:13:9: warning: 'raw_smp_processor_id' macro redefined [-Wmacro-redefined]
#define raw_smp_processor_id() (current_thread_info()->cpu)
^
include/linux/smp.h:191:9: note: previous definition is here
#define raw_smp_processor_id() 0
^
1 warning generated.
vim +/raw_smp_processor_id +13 arch/hexagon/include/asm/smp.h
43afdf50838634 Richard Kuo 2011-10-31 12
43afdf50838634 Richard Kuo 2011-10-31 @13 #define raw_smp_processor_id() (current_thread_info()->cpu)
43afdf50838634 Richard Kuo 2011-10-31 14
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
@ 2022-02-22 23:31 ` kernel test robot
0 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 23:31 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: llvm, kbuild-all, tvrtko.ursulin, michael.cheng,
balasubramani.vivekanandan, wayne.boyer, casey.g.bowman,
lucas.demarchi, dri-devel
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: hexagon-randconfig-r041-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230702.Ya7PkY7G-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/drm_cache.c:37:
In file included from include/drm/drm_cache.h:37:
>> arch/hexagon/include/asm/smp.h:13:9: warning: 'raw_smp_processor_id' macro redefined [-Wmacro-redefined]
#define raw_smp_processor_id() (current_thread_info()->cpu)
^
include/linux/smp.h:191:9: note: previous definition is here
#define raw_smp_processor_id() 0
^
1 warning generated.
vim +/raw_smp_processor_id +13 arch/hexagon/include/asm/smp.h
43afdf50838634 Richard Kuo 2011-10-31 12
43afdf50838634 Richard Kuo 2011-10-31 @13 #define raw_smp_processor_id() (current_thread_info()->cpu)
43afdf50838634 Richard Kuo 2011-10-31 14
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
@ 2022-02-22 23:31 ` kernel test robot
0 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 23:31 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: kbuild-all, llvm, lucas.demarchi, dri-devel, michael.cheng
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: hexagon-randconfig-r041-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230702.Ya7PkY7G-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/gpu/drm/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/drm_cache.c:37:
In file included from include/drm/drm_cache.h:37:
>> arch/hexagon/include/asm/smp.h:13:9: warning: 'raw_smp_processor_id' macro redefined [-Wmacro-redefined]
#define raw_smp_processor_id() (current_thread_info()->cpu)
^
include/linux/smp.h:191:9: note: previous definition is here
#define raw_smp_processor_id() 0
^
1 warning generated.
vim +/raw_smp_processor_id +13 arch/hexagon/include/asm/smp.h
43afdf50838634 Richard Kuo 2011-10-31 12
43afdf50838634 Richard Kuo 2011-10-31 @13 #define raw_smp_processor_id() (current_thread_info()->cpu)
43afdf50838634 Richard Kuo 2011-10-31 14
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
@ 2022-02-22 23:31 ` kernel test robot
-1 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 23:31 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: tvrtko.ursulin, kbuild-all, balasubramani.vivekanandan,
wayne.boyer, casey.g.bowman, lucas.demarchi, dri-devel,
michael.cheng
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: openrisc-randconfig-r025-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230701.l0i1vncs-lkp@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=openrisc SHELL=/bin/bash drivers/gpu/drm/vgem/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/drm/drm_cache.h:37,
from drivers/gpu/drm/vgem/vgem_drv.h:33,
from drivers/gpu/drm/vgem/vgem_fence.c:28:
>> arch/openrisc/include/asm/smp.h:15: warning: "raw_smp_processor_id" redefined
15 | #define raw_smp_processor_id() (current_thread_info()->cpu)
|
In file included from include/linux/lockdep.h:14,
from include/linux/spinlock.h:62,
from include/linux/vmalloc.h:5,
from include/asm-generic/io.h:911,
from arch/openrisc/include/asm/io.h:36,
from include/linux/io.h:13,
from include/linux/iosys-map.h:9,
from include/linux/dma-buf.h:16,
from drivers/gpu/drm/vgem/vgem_fence.c:23:
include/linux/smp.h:191: note: this is the location of the previous definition
191 | #define raw_smp_processor_id() 0
|
vim +/raw_smp_processor_id +15 arch/openrisc/include/asm/smp.h
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 14
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 @15 #define raw_smp_processor_id() (current_thread_info()->cpu)
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 16 #define hard_smp_processor_id() mfspr(SPR_COREID)
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 17
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 1/3] drm_cache: Add logic for wbvind_on_all_cpus
@ 2022-02-22 23:31 ` kernel test robot
0 siblings, 0 replies; 27+ messages in thread
From: kernel test robot @ 2022-02-22 23:31 UTC (permalink / raw)
To: Michael Cheng, intel-gfx
Cc: kbuild-all, lucas.demarchi, dri-devel, michael.cheng
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm/drm-next drm-tip/drm-tip v5.17-rc5 next-20220217]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
base: git://anongit.freedesktop.org/drm-intel for-linux-next
config: openrisc-randconfig-r025-20220221 (https://download.01.org/0day-ci/archive/20220223/202202230701.l0i1vncs-lkp@intel.com/config)
compiler: or1k-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/3aaa40c95b16a78c9059a77536de70bb08ce05e9
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Michael-Cheng/Move-define-wbvind_on_all_cpus/20220223-012853
git checkout 3aaa40c95b16a78c9059a77536de70bb08ce05e9
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=openrisc SHELL=/bin/bash drivers/gpu/drm/vgem/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/drm/drm_cache.h:37,
from drivers/gpu/drm/vgem/vgem_drv.h:33,
from drivers/gpu/drm/vgem/vgem_fence.c:28:
>> arch/openrisc/include/asm/smp.h:15: warning: "raw_smp_processor_id" redefined
15 | #define raw_smp_processor_id() (current_thread_info()->cpu)
|
In file included from include/linux/lockdep.h:14,
from include/linux/spinlock.h:62,
from include/linux/vmalloc.h:5,
from include/asm-generic/io.h:911,
from arch/openrisc/include/asm/io.h:36,
from include/linux/io.h:13,
from include/linux/iosys-map.h:9,
from include/linux/dma-buf.h:16,
from drivers/gpu/drm/vgem/vgem_fence.c:23:
include/linux/smp.h:191: note: this is the location of the previous definition
191 | #define raw_smp_processor_id() 0
|
vim +/raw_smp_processor_id +15 arch/openrisc/include/asm/smp.h
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 14
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 @15 #define raw_smp_processor_id() (current_thread_info()->cpu)
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 16 #define hard_smp_processor_id() mfspr(SPR_COREID)
8e6d08e0a15e7d4 Stefan Kristiansson 2014-05-11 17
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
@ 2022-02-22 17:26 ` Michael Cheng
-1 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx
Cc: tvrtko.ursulin, michael.cheng, balasubramani.vivekanandan,
wayne.boyer, casey.g.bowman, lucas.demarchi, dri-devel
This patch removes logic for wbinvd_on_all_cpus and brings in
drm_cache.h. This header has the logic that outputs a warning
when wbinvd_on_all_cpus when its being used on a non-x86 platform.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_pm.c b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
index 00359ec9d58b..ee4783e4d135 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_pm.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
@@ -13,12 +13,7 @@
#include "i915_driver.h"
#include "i915_drv.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
+#include <drm/drm_cache.h>
void i915_gem_suspend(struct drm_i915_private *i915)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
@ 2022-02-22 17:26 ` Michael Cheng
0 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx; +Cc: michael.cheng, lucas.demarchi, dri-devel
This patch removes logic for wbinvd_on_all_cpus and brings in
drm_cache.h. This header has the logic that outputs a warning
when wbinvd_on_all_cpus when its being used on a non-x86 platform.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_pm.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_pm.c b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
index 00359ec9d58b..ee4783e4d135 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_pm.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
@@ -13,12 +13,7 @@
#include "i915_driver.h"
#include "i915_drv.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
+#include <drm/drm_cache.h>
void i915_gem_suspend(struct drm_i915_private *i915)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
(?)
@ 2022-02-22 19:24 ` Thomas Hellström (Intel)
2022-02-23 1:13 ` Michael Cheng
2022-03-08 17:58 ` Lucas De Marchi
-1 siblings, 2 replies; 27+ messages in thread
From: Thomas Hellström (Intel) @ 2022-02-22 19:24 UTC (permalink / raw)
To: Michael Cheng, intel-gfx; +Cc: lucas.demarchi, dri-devel
Hi, Michael,
On 2/22/22 18:26, Michael Cheng wrote:
> This patch removes logic for wbinvd_on_all_cpus and brings in
> drm_cache.h. This header has the logic that outputs a warning
> when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>
> Signed-off-by: Michael Cheng <michael.cheng@intel.com>
Linus has been pretty clear that he won't accept patches that add macros
that works on one arch and warns on others anymore in i915 and I figure
even less so in drm code.
So we shouldn't try to move this out to drm. Instead we should restrict
the wbinvd() inside our driver to integrated and X86 only. For discrete
on all architectures we should be coherent and hence not be needing
wbinvd().
Thanks,
/Thomas
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-02-22 19:24 ` Thomas Hellström (Intel)
@ 2022-02-23 1:13 ` Michael Cheng
2022-03-08 17:58 ` Lucas De Marchi
1 sibling, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-23 1:13 UTC (permalink / raw)
To: Thomas Hellström (Intel), intel-gfx; +Cc: lucas.demarchi, dri-devel
Thanks for letting me know! I will try for a different solution.
On 2022-02-22 11:24 a.m., Thomas Hellström (Intel) wrote:
> Hi, Michael,
>
> On 2/22/22 18:26, Michael Cheng wrote:
>> This patch removes logic for wbinvd_on_all_cpus and brings in
>> drm_cache.h. This header has the logic that outputs a warning
>> when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>>
>> Signed-off-by: Michael Cheng <michael.cheng@intel.com>
>
> Linus has been pretty clear that he won't accept patches that add
> macros that works on one arch and warns on others anymore in i915 and
> I figure even less so in drm code.
>
> So we shouldn't try to move this out to drm. Instead we should
> restrict the wbinvd() inside our driver to integrated and X86 only.
> For discrete on all architectures we should be coherent and hence not
> be needing wbinvd().
>
> Thanks,
>
> /Thomas
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-02-22 19:24 ` Thomas Hellström (Intel)
2022-02-23 1:13 ` Michael Cheng
@ 2022-03-08 17:58 ` Lucas De Marchi
2022-03-14 23:06 ` Michael Cheng
1 sibling, 1 reply; 27+ messages in thread
From: Lucas De Marchi @ 2022-03-08 17:58 UTC (permalink / raw)
To: Thomas Hellström (Intel); +Cc: intel-gfx, Michael Cheng, dri-devel
On Tue, Feb 22, 2022 at 08:24:31PM +0100, Thomas Hellström (Intel) wrote:
>Hi, Michael,
>
>On 2/22/22 18:26, Michael Cheng wrote:
>>This patch removes logic for wbinvd_on_all_cpus and brings in
>>drm_cache.h. This header has the logic that outputs a warning
>>when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>>
>>Signed-off-by: Michael Cheng <michael.cheng@intel.com>
>
>Linus has been pretty clear that he won't accept patches that add
>macros that works on one arch and warns on others anymore in i915 and
>I figure even less so in drm code.
>
>So we shouldn't try to move this out to drm. Instead we should
>restrict the wbinvd() inside our driver to integrated and X86 only.
>For discrete on all architectures we should be coherent and hence not
>be needing wbinvd().
the warn is there to guarantee we don't forget a code path. However
simply adding the warning is the real issue: we should rather guarantee
we can't take that code path. I.e., as you said refactor the code to
guarantee it works on discrete without that logic.
$ git grep wbinvd_on_all_cpus -- drivers/gpu/drm/
drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: * Currently we just do a heavy handed wbinvd_on_all_cpus() here since
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: wbinvd_on_all_cpus();
It looks like we actually go through this on other discrete graphics. Is
this missing an update like s/IS_DG1/IS_DGFX/? Or should we be doing
something else?
drivers/gpu/drm/i915/gem/i915_gem_pm.c:#define wbinvd_on_all_cpus() \
drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
Those are for suspend. Revert ac05a22cd07a ("drm/i915/gem: Almagamate clflushes on suspend")
or extract that part to a helper function and implement it differently
for arches != x86?
drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
Probably take a similar approach to the suspend case?
drivers/gpu/drm/i915/gt/intel_ggtt.c: wbinvd_on_all_cpus();
This one comes from 64b95df91f44 ("drm/i915: Assume exclusive access to objects inside resume")
Shouldn't that be doing the invalidate if the write domain is I915_GEM_DOMAIN_CPU
In the end I think the warning would be ok if it was the cherry on top,
to guarantee we don't take those paths. We should probably have a
warn_once() to avoid spamming the console. But we also have to rework
the code to guarantee we are the only ones who may eventually get that
warning, and not the end user.
Lucas De Marchi
>
>Thanks,
>
>/Thomas
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-03-08 17:58 ` Lucas De Marchi
@ 2022-03-14 23:06 ` Michael Cheng
2022-03-15 16:59 ` Michael Cheng
0 siblings, 1 reply; 27+ messages in thread
From: Michael Cheng @ 2022-03-14 23:06 UTC (permalink / raw)
To: Lucas De Marchi, Thomas Hellström (Intel); +Cc: intel-gfx, dri-devel
On 2022-03-08 10:58 a.m., Lucas De Marchi wrote:
> On Tue, Feb 22, 2022 at 08:24:31PM +0100, Thomas Hellström (Intel) wrote:
>> Hi, Michael,
>>
>> On 2/22/22 18:26, Michael Cheng wrote:
>>> This patch removes logic for wbinvd_on_all_cpus and brings in
>>> drm_cache.h. This header has the logic that outputs a warning
>>> when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>>>
>>> Signed-off-by: Michael Cheng <michael.cheng@intel.com>
>>
>> Linus has been pretty clear that he won't accept patches that add
>> macros that works on one arch and warns on others anymore in i915 and
>> I figure even less so in drm code.
>>
>> So we shouldn't try to move this out to drm. Instead we should
>> restrict the wbinvd() inside our driver to integrated and X86 only.
>> For discrete on all architectures we should be coherent and hence not
>> be needing wbinvd().
>
> the warn is there to guarantee we don't forget a code path. However
> simply adding the warning is the real issue: we should rather guarantee
> we can't take that code path. I.e., as you said refactor the code to
> guarantee it works on discrete without that logic.
>
> $ git grep wbinvd_on_all_cpus -- drivers/gpu/drm/
> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>
> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: * Currently we
> just do a heavy handed wbinvd_on_all_cpus() here since
> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: wbinvd_on_all_cpus();
>
> It looks like we actually go through this on other discrete graphics. Is
> this missing an update like s/IS_DG1/IS_DGFX/? Or should we be doing
> something else?
>
> drivers/gpu/drm/i915/gem/i915_gem_pm.c:#define wbinvd_on_all_cpus() \
> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>
> Those are for suspend. Revert ac05a22cd07a ("drm/i915/gem: Almagamate
> clflushes on suspend")
> or extract that part to a helper function and implement it differently
> for arches != x86?
>
> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>
> Probably take a similar approach to the suspend case?
>
> drivers/gpu/drm/i915/gt/intel_ggtt.c: wbinvd_on_all_cpus();
For a helper function, I have a #define for all non x86 architecture
that gives a warn on [1] within drm_cache.h Or would it be better to
implement a helper function instead?
[1]. https://patchwork.freedesktop.org/patch/475750/?series=99991&rev=5
>
> This one comes from 64b95df91f44 ("drm/i915: Assume exclusive access
> to objects inside resume")
> Shouldn't that be doing the invalidate if the write domain is
> I915_GEM_DOMAIN_CPU
>
> In the end I think the warning would be ok if it was the cherry on top,
> to guarantee we don't take those paths. We should probably have a
> warn_once() to avoid spamming the console. But we also have to rework
> the code to guarantee we are the only ones who may eventually get that
> warning, and not the end user.
Could we first add the helper function/#define for now, and rework the
code in a different patch series?
>
> Lucas De Marchi
>
>>
>> Thanks,
>>
>> /Thomas
>>
>>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-03-14 23:06 ` Michael Cheng
@ 2022-03-15 16:59 ` Michael Cheng
2022-03-16 8:48 ` Thomas Hellström (Intel)
0 siblings, 1 reply; 27+ messages in thread
From: Michael Cheng @ 2022-03-15 16:59 UTC (permalink / raw)
To: Lucas De Marchi, Thomas Hellström (Intel), Daniel Vetter
Cc: intel-gfx, dri-devel
+Daniel for additional feedback!
On 2022-03-14 4:06 p.m., Michael Cheng wrote:
> On 2022-03-08 10:58 a.m., Lucas De Marchi wrote:
>
>> On Tue, Feb 22, 2022 at 08:24:31PM +0100, Thomas Hellström (Intel)
>> wrote:
>>> Hi, Michael,
>>>
>>> On 2/22/22 18:26, Michael Cheng wrote:
>>>> This patch removes logic for wbinvd_on_all_cpus and brings in
>>>> drm_cache.h. This header has the logic that outputs a warning
>>>> when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>>>>
>>>> Signed-off-by: Michael Cheng <michael.cheng@intel.com>
>>>
>>> Linus has been pretty clear that he won't accept patches that add
>>> macros that works on one arch and warns on others anymore in i915
>>> and I figure even less so in drm code.
>>>
>>> So we shouldn't try to move this out to drm. Instead we should
>>> restrict the wbinvd() inside our driver to integrated and X86 only.
>>> For discrete on all architectures we should be coherent and hence
>>> not be needing wbinvd().
>>
>> the warn is there to guarantee we don't forget a code path. However
>> simply adding the warning is the real issue: we should rather guarantee
>> we can't take that code path. I.e., as you said refactor the code to
>> guarantee it works on discrete without that logic.
>>
>> $ git grep wbinvd_on_all_cpus -- drivers/gpu/drm/
>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>>
>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: * Currently we
>> just do a heavy handed wbinvd_on_all_cpus() here since
>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: wbinvd_on_all_cpus();
>>
>> It looks like we actually go through this on other discrete graphics. Is
>> this missing an update like s/IS_DG1/IS_DGFX/? Or should we be doing
>> something else?
>>
>> drivers/gpu/drm/i915/gem/i915_gem_pm.c:#define
>> wbinvd_on_all_cpus() \
>> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>>
>> Those are for suspend. Revert ac05a22cd07a ("drm/i915/gem: Almagamate
>> clflushes on suspend")
>> or extract that part to a helper function and implement it differently
>> for arches != x86?
>>
>> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>>
>> Probably take a similar approach to the suspend case?
>>
>> drivers/gpu/drm/i915/gt/intel_ggtt.c: wbinvd_on_all_cpus();
>
> For a helper function, I have a #define for all non x86 architecture
> that gives a warn on [1] within drm_cache.h Or would it be better to
> implement a helper function instead?
>
> [1]. https://patchwork.freedesktop.org/patch/475750/?series=99991&rev=5
>
>>
>> This one comes from 64b95df91f44 ("drm/i915: Assume exclusive access
>> to objects inside resume")
>> Shouldn't that be doing the invalidate if the write domain is
>> I915_GEM_DOMAIN_CPU
>>
>> In the end I think the warning would be ok if it was the cherry on top,
>> to guarantee we don't take those paths. We should probably have a
>> warn_once() to avoid spamming the console. But we also have to rework
>> the code to guarantee we are the only ones who may eventually get that
>> warning, and not the end user.
> Could we first add the helper function/#define for now, and rework the
> code in a different patch series?
>>
>> Lucas De Marchi
>>
>>>
>>> Thanks,
>>>
>>> /Thomas
>>>
>>>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [Intel-gfx] [PATCH v3 2/3] drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2022-03-15 16:59 ` Michael Cheng
@ 2022-03-16 8:48 ` Thomas Hellström (Intel)
0 siblings, 0 replies; 27+ messages in thread
From: Thomas Hellström (Intel) @ 2022-03-16 8:48 UTC (permalink / raw)
To: Michael Cheng, Lucas De Marchi, Daniel Vetter, Dave Airlie
Cc: intel-gfx, dri-devel
Hi, Michael, others.
This is the response from Linus last time we copied that already
pre-existing wbinvd_on_all_cpus() macro to another place in the driver:
https://lists.freedesktop.org/archives/dri-devel/2021-November/330928.html
My first interpretation of this is that even if there currently are
similar patterns in drm_cache.c, we shouldn't introduce more,
encouraging other drivers to use incoherent IO.
Other than that I think we should move whatever wbinvds we can over to
the ranged versions, unless that is proven to be a performance drop.
Finally for any wbinvds left in our driver, ensure that they are never
executed for any gpu where we provide full coherency. That is all
discrete gpus (and to be discussed integrated gpus moving forward).
Might be that drm maintainers want to chime in here with other views.
Thanks,
Thomas
On 3/15/22 17:59, Michael Cheng wrote:
> +Daniel for additional feedback!
>
> On 2022-03-14 4:06 p.m., Michael Cheng wrote:
>
>> On 2022-03-08 10:58 a.m., Lucas De Marchi wrote:
>>
>>> On Tue, Feb 22, 2022 at 08:24:31PM +0100, Thomas Hellström (Intel)
>>> wrote:
>>>> Hi, Michael,
>>>>
>>>> On 2/22/22 18:26, Michael Cheng wrote:
>>>>> This patch removes logic for wbinvd_on_all_cpus and brings in
>>>>> drm_cache.h. This header has the logic that outputs a warning
>>>>> when wbinvd_on_all_cpus when its being used on a non-x86 platform.
>>>>>
>>>>> Signed-off-by: Michael Cheng <michael.cheng@intel.com>
>>>>
>>>> Linus has been pretty clear that he won't accept patches that add
>>>> macros that works on one arch and warns on others anymore in i915
>>>> and I figure even less so in drm code.
>>>>
>>>> So we shouldn't try to move this out to drm. Instead we should
>>>> restrict the wbinvd() inside our driver to integrated and X86 only.
>>>> For discrete on all architectures we should be coherent and hence
>>>> not be needing wbinvd().
>>>
>>> the warn is there to guarantee we don't forget a code path. However
>>> simply adding the warning is the real issue: we should rather guarantee
>>> we can't take that code path. I.e., as you said refactor the code to
>>> guarantee it works on discrete without that logic.
>>>
>>> $ git grep wbinvd_on_all_cpus -- drivers/gpu/drm/
>>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>>> drivers/gpu/drm/drm_cache.c: if (wbinvd_on_all_cpus())
>>>
>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: * Currently we
>>> just do a heavy handed wbinvd_on_all_cpus() here since
>>> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c: wbinvd_on_all_cpus();
>>>
>>> It looks like we actually go through this on other discrete
>>> graphics. Is
>>> this missing an update like s/IS_DG1/IS_DGFX/? Or should we be doing
>>> something else?
>>>
>>> drivers/gpu/drm/i915/gem/i915_gem_pm.c:#define
>>> wbinvd_on_all_cpus() \
>>> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>>>
>>> Those are for suspend. Revert ac05a22cd07a ("drm/i915/gem:
>>> Almagamate clflushes on suspend")
>>> or extract that part to a helper function and implement it differently
>>> for arches != x86?
>>>
>>> drivers/gpu/drm/i915/gem/i915_gem_pm.c: wbinvd_on_all_cpus();
>>>
>>> Probably take a similar approach to the suspend case?
>>>
>>> drivers/gpu/drm/i915/gt/intel_ggtt.c: wbinvd_on_all_cpus();
>>
>> For a helper function, I have a #define for all non x86 architecture
>> that gives a warn on [1] within drm_cache.h Or would it be better to
>> implement a helper function instead?
>>
>> [1]. https://patchwork.freedesktop.org/patch/475750/?series=99991&rev=5
>>
>>>
>>> This one comes from 64b95df91f44 ("drm/i915: Assume exclusive access
>>> to objects inside resume")
>>> Shouldn't that be doing the invalidate if the write domain is
>>> I915_GEM_DOMAIN_CPU
>>>
>>> In the end I think the warning would be ok if it was the cherry on top,
>>> to guarantee we don't take those paths. We should probably have a
>>> warn_once() to avoid spamming the console. But we also have to rework
>>> the code to guarantee we are the only ones who may eventually get that
>>> warning, and not the end user.
>> Could we first add the helper function/#define for now, and rework
>> the code in a different patch series?
>>>
>>> Lucas De Marchi
>>>
>>>>
>>>> Thanks,
>>>>
>>>> /Thomas
>>>>
>>>>
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH v3 3/3] drm/i915/: Add drm_cache.h
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
@ 2022-02-22 17:26 ` Michael Cheng
-1 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx
Cc: tvrtko.ursulin, michael.cheng, balasubramani.vivekanandan,
wayne.boyer, casey.g.bowman, lucas.demarchi, dri-devel
Add drm_cache.h to additionals files that calls wbinvd_on_all_cpus and
remove un-needed header files. This will prevent compile errors when
building for non-x86 platforms, as well as output a warning when
wbinvd_on_all_cpus is being called incorrectly.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ggtt.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
index 13917231ae81..edb0ebbb089c 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
@@ -9,7 +9,7 @@
#include <linux/dma-resv.h>
#include <linux/module.h>
-#include <asm/smp.h>
+#include <drm/drm_cache.h>
#include "gem/i915_gem_dmabuf.h"
#include "i915_drv.h"
diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
index 8850d4e0f9cc..dac62e3ba142 100644
--- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
@@ -7,10 +7,10 @@
#include <linux/stop_machine.h>
#include <asm/set_memory.h>
-#include <asm/smp.h>
#include <drm/i915_drm.h>
#include <drm/intel-gtt.h>
+#include <drm/drm_cache.h>
#include "gem/i915_gem_lmem.h"
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [Intel-gfx] [PATCH v3 3/3] drm/i915/: Add drm_cache.h
@ 2022-02-22 17:26 ` Michael Cheng
0 siblings, 0 replies; 27+ messages in thread
From: Michael Cheng @ 2022-02-22 17:26 UTC (permalink / raw)
To: intel-gfx; +Cc: michael.cheng, lucas.demarchi, dri-devel
Add drm_cache.h to additionals files that calls wbinvd_on_all_cpus and
remove un-needed header files. This will prevent compile errors when
building for non-x86 platforms, as well as output a warning when
wbinvd_on_all_cpus is being called incorrectly.
Signed-off-by: Michael Cheng <michael.cheng@intel.com>
---
drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 2 +-
drivers/gpu/drm/i915/gt/intel_ggtt.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
index 13917231ae81..edb0ebbb089c 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
@@ -9,7 +9,7 @@
#include <linux/dma-resv.h>
#include <linux/module.h>
-#include <asm/smp.h>
+#include <drm/drm_cache.h>
#include "gem/i915_gem_dmabuf.h"
#include "i915_drv.h"
diff --git a/drivers/gpu/drm/i915/gt/intel_ggtt.c b/drivers/gpu/drm/i915/gt/intel_ggtt.c
index 8850d4e0f9cc..dac62e3ba142 100644
--- a/drivers/gpu/drm/i915/gt/intel_ggtt.c
+++ b/drivers/gpu/drm/i915/gt/intel_ggtt.c
@@ -7,10 +7,10 @@
#include <linux/stop_machine.h>
#include <asm/set_memory.h>
-#include <asm/smp.h>
#include <drm/i915_drm.h>
#include <drm/intel-gtt.h>
+#include <drm/drm_cache.h>
#include "gem/i915_gem_lmem.h"
--
2.25.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Move #define wbvind_on_all_cpus (rev4)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (3 preceding siblings ...)
(?)
@ 2022-02-23 2:33 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-23 2:33 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev4)
URL : https://patchwork.freedesktop.org/series/99991/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
6e4fd482bfcd drm_cache: Add logic for wbvind_on_all_cpus
-:13: WARNING:BAD_SIGN_OFF: Duplicate signature
#13:
Reported-by: kernel test robot <lkp@intel.com>
-:14: WARNING:BAD_SIGN_OFF: Duplicate signature
#14:
Reported-by: kernel test robot <lkp@intel.com>
-:37: WARNING:INCLUDE_LINUX: Use #include <linux/smp.h> instead of <asm/smp.h>
#37: FILE: include/drm/drm_cache.h:37:
+#include <asm/smp.h>
total: 0 errors, 3 warnings, 0 checks, 20 lines checked
919d613ba305 drm/i915/gem: Remove logic for wbinvd_on_all_cpus
3de274fe25b2 drm/i915/: Add drm_cache.h
^ permalink raw reply [flat|nested] 27+ messages in thread* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Move #define wbvind_on_all_cpus (rev4)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (4 preceding siblings ...)
(?)
@ 2022-02-23 2:33 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-23 2:33 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev4)
URL : https://patchwork.freedesktop.org/series/99991/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
^ permalink raw reply [flat|nested] 27+ messages in thread* [Intel-gfx] ✗ Fi.CI.BAT: failure for Move #define wbvind_on_all_cpus (rev4)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (5 preceding siblings ...)
(?)
@ 2022-02-23 3:05 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-23 3:05 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 8774 bytes --]
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev4)
URL : https://patchwork.freedesktop.org/series/99991/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11268 -> Patchwork_22360
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_22360 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_22360, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/index.html
Participating hosts (20 -> 16)
------------------------------
Missing (4): fi-icl-u2 shard-rkl shard-dg1 shard-tglu
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_22360:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_fence@basic-busy@rcs0:
- fi-blb-e6850: [PASS][1] -> [DMESG-WARN][2] +13 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-blb-e6850/igt@gem_exec_fence@basic-busy@rcs0.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-blb-e6850/igt@gem_exec_fence@basic-busy@rcs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- fi-glk-j4005: [PASS][3] -> [DMESG-WARN][4] +15 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-glk-j4005/igt@gem_exec_suspend@basic-s0@smem.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-glk-j4005/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_exec_suspend@basic-s3@smem:
- bat-dg1-6: [PASS][5] -> [DMESG-WARN][6] +4 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-dg1-6/igt@gem_exec_suspend@basic-s3@smem.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-dg1-6/igt@gem_exec_suspend@basic-s3@smem.html
- fi-rkl-guc: [PASS][7] -> [DMESG-WARN][8] +3 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-rkl-guc/igt@gem_exec_suspend@basic-s3@smem.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-rkl-guc/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@gem:
- fi-tgl-1115g4: [PASS][9] -> [DMESG-WARN][10] +3 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-tgl-1115g4/igt@i915_selftest@live@gem.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-tgl-1115g4/igt@i915_selftest@live@gem.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- bat-dg1-5: [PASS][11] -> [DMESG-WARN][12] +5 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-dg1-5/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-dg1-5/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
* igt@prime_vgem@basic-read:
- fi-bwr-2160: [PASS][13] -> [DMESG-WARN][14] +9 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-bwr-2160/igt@prime_vgem@basic-read.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-bwr-2160/igt@prime_vgem@basic-read.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_exec_fence@basic-busy@vecs0:
- {fi-jsl-1}: [PASS][15] -> [DMESG-WARN][16] +15 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-jsl-1/igt@gem_exec_fence@basic-busy@vecs0.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-jsl-1/igt@gem_exec_fence@basic-busy@vecs0.html
* igt@gem_exec_fence@basic-wait@vecs0:
- {fi-ehl-2}: [PASS][17] -> [DMESG-WARN][18] +15 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-ehl-2/igt@gem_exec_fence@basic-wait@vecs0.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-ehl-2/igt@gem_exec_fence@basic-wait@vecs0.html
- {bat-jsl-1}: [PASS][19] -> [DMESG-WARN][20] +15 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-jsl-1/igt@gem_exec_fence@basic-wait@vecs0.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-jsl-1/igt@gem_exec_fence@basic-wait@vecs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- {bat-rpls-2}: [PASS][21] -> [DMESG-WARN][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-rpls-2/igt@gem_exec_suspend@basic-s0@smem.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-rpls-2/igt@gem_exec_suspend@basic-s0@smem.html
- {fi-rkl-11600}: [PASS][23] -> [DMESG-WARN][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-rkl-11600/igt@gem_exec_suspend@basic-s0@smem.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-rkl-11600/igt@gem_exec_suspend@basic-s0@smem.html
* igt@i915_selftest@live@gem:
- {fi-tgl-dsi}: [PASS][25] -> [DMESG-WARN][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-tgl-dsi/igt@i915_selftest@live@gem.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-tgl-dsi/igt@i915_selftest@live@gem.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- {bat-adlp-6}: [PASS][27] -> [DMESG-WARN][28] +3 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
Known issues
------------
Here are the changes found in Patchwork_22360 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@requests:
- fi-blb-e6850: [PASS][29] -> [DMESG-FAIL][30] ([i915#4528] / [i915#5026])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/fi-blb-e6850/igt@i915_selftest@live@requests.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-blb-e6850/igt@i915_selftest@live@requests.html
* igt@runner@aborted:
- fi-blb-e6850: NOTRUN -> [FAIL][31] ([fdo#109271] / [i915#2403] / [i915#2426] / [i915#4312])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/fi-blb-e6850/igt@runner@aborted.html
#### Possible fixes ####
* igt@i915_selftest@live@hangcheck:
- bat-dg1-5: [DMESG-FAIL][32] ([i915#4494] / [i915#4957]) -> [PASS][33]
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-dg1-5/igt@i915_selftest@live@hangcheck.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-dg1-5/igt@i915_selftest@live@hangcheck.html
- bat-dg1-6: [DMESG-FAIL][34] ([i915#4957]) -> [PASS][35]
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11268/bat-dg1-6/igt@i915_selftest@live@hangcheck.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/bat-dg1-6/igt@i915_selftest@live@hangcheck.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#2403]: https://gitlab.freedesktop.org/drm/intel/issues/2403
[i915#2411]: https://gitlab.freedesktop.org/drm/intel/issues/2411
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4494]: https://gitlab.freedesktop.org/drm/intel/issues/4494
[i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
[i915#4897]: https://gitlab.freedesktop.org/drm/intel/issues/4897
[i915#4957]: https://gitlab.freedesktop.org/drm/intel/issues/4957
[i915#5026]: https://gitlab.freedesktop.org/drm/intel/issues/5026
Build changes
-------------
* Linux: CI_DRM_11268 -> Patchwork_22360
CI-20190529: 20190529
CI_DRM_11268: 26326bf05392ab3da8cba36642a0efec97f00da9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6352: 11b4d227d8f0efad522519c8fd9525774247c8d8 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_22360: 3de274fe25b29ecdf03cd48d9cb9b31b0b795745 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
3de274fe25b2 drm/i915/: Add drm_cache.h
919d613ba305 drm/i915/gem: Remove logic for wbinvd_on_all_cpus
6e4fd482bfcd drm_cache: Add logic for wbvind_on_all_cpus
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22360/index.html
[-- Attachment #2: Type: text/html, Size: 9812 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Move #define wbvind_on_all_cpus (rev5)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (6 preceding siblings ...)
(?)
@ 2022-02-24 1:26 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-24 1:26 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev5)
URL : https://patchwork.freedesktop.org/series/99991/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
2faaf6c21ed1 drm_cache: Add logic for wbvind_on_all_cpus
-:13: WARNING:BAD_SIGN_OFF: Duplicate signature
#13:
Reported-by: kernel test robot <lkp@intel.com>
-:14: WARNING:BAD_SIGN_OFF: Duplicate signature
#14:
Reported-by: kernel test robot <lkp@intel.com>
-:37: WARNING:INCLUDE_LINUX: Use #include <linux/smp.h> instead of <asm/smp.h>
#37: FILE: include/drm/drm_cache.h:37:
+#include <asm/smp.h>
total: 0 errors, 3 warnings, 0 checks, 20 lines checked
2c3691719f58 drm/i915/gem: Remove logic for wbinvd_on_all_cpus
fdc855c609b3 drm/i915/: Add drm_cache.h
^ permalink raw reply [flat|nested] 27+ messages in thread* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Move #define wbvind_on_all_cpus (rev5)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (7 preceding siblings ...)
(?)
@ 2022-02-24 1:27 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-24 1:27 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev5)
URL : https://patchwork.freedesktop.org/series/99991/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
^ permalink raw reply [flat|nested] 27+ messages in thread* [Intel-gfx] ✗ Fi.CI.BAT: failure for Move #define wbvind_on_all_cpus (rev5)
2022-02-22 17:26 ` [Intel-gfx] " Michael Cheng
` (8 preceding siblings ...)
(?)
@ 2022-02-24 2:00 ` Patchwork
-1 siblings, 0 replies; 27+ messages in thread
From: Patchwork @ 2022-02-24 2:00 UTC (permalink / raw)
To: Michael Cheng; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 24540 bytes --]
== Series Details ==
Series: Move #define wbvind_on_all_cpus (rev5)
URL : https://patchwork.freedesktop.org/series/99991/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_11276 -> Patchwork_22375
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_22375 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_22375, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/index.html
Participating hosts (46 -> 43)
------------------------------
Additional (2): fi-kbl-soraka fi-icl-u2
Missing (5): fi-bdw-5557u fi-hsw-4200u fi-ctg-p8600 fi-hsw-4770 fi-bdw-samus
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_22375:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_fence@basic-busy@rcs0:
- fi-blb-e6850: [PASS][1] -> [DMESG-WARN][2] +13 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-blb-e6850/igt@gem_exec_fence@basic-busy@rcs0.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-blb-e6850/igt@gem_exec_fence@basic-busy@rcs0.html
* igt@gem_exec_fence@basic-busy@vecs0:
- fi-bxt-dsi: [PASS][3] -> [DMESG-WARN][4] +15 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bxt-dsi/igt@gem_exec_fence@basic-busy@vecs0.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bxt-dsi/igt@gem_exec_fence@basic-busy@vecs0.html
* igt@gem_exec_fence@nb-await@vecs0:
- fi-glk-dsi: [PASS][5] -> [DMESG-WARN][6] +15 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-glk-dsi/igt@gem_exec_fence@nb-await@vecs0.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-glk-dsi/igt@gem_exec_fence@nb-await@vecs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- fi-kbl-guc: [PASS][7] -> [DMESG-WARN][8] +1 similar issue
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-kbl-guc/igt@gem_exec_suspend@basic-s0@smem.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-guc/igt@gem_exec_suspend@basic-s0@smem.html
- fi-bsw-nick: [PASS][9] -> [DMESG-WARN][10] +13 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bsw-nick/igt@gem_exec_suspend@basic-s0@smem.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bsw-nick/igt@gem_exec_suspend@basic-s0@smem.html
- bat-adlp-4: [PASS][11] -> [DMESG-WARN][12] +3 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-adlp-4/igt@gem_exec_suspend@basic-s0@smem.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-adlp-4/igt@gem_exec_suspend@basic-s0@smem.html
- fi-glk-j4005: [PASS][13] -> [DMESG-WARN][14] +15 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-glk-j4005/igt@gem_exec_suspend@basic-s0@smem.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-glk-j4005/igt@gem_exec_suspend@basic-s0@smem.html
- fi-kbl-x1275: [PASS][15] -> [DMESG-WARN][16] +3 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-kbl-x1275/igt@gem_exec_suspend@basic-s0@smem.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-x1275/igt@gem_exec_suspend@basic-s0@smem.html
- fi-kbl-7567u: [PASS][17] -> [DMESG-WARN][18] +2 similar issues
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-kbl-7567u/igt@gem_exec_suspend@basic-s0@smem.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-7567u/igt@gem_exec_suspend@basic-s0@smem.html
- fi-skl-6600u: [PASS][19] -> [DMESG-WARN][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-skl-6600u/igt@gem_exec_suspend@basic-s0@smem.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_exec_suspend@basic-s3@smem:
- fi-elk-e7500: [PASS][21] -> [DMESG-WARN][22] +15 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-elk-e7500/igt@gem_exec_suspend@basic-s3@smem.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-elk-e7500/igt@gem_exec_suspend@basic-s3@smem.html
- bat-dg1-6: [PASS][23] -> [DMESG-WARN][24] +4 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-dg1-6/igt@gem_exec_suspend@basic-s3@smem.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-dg1-6/igt@gem_exec_suspend@basic-s3@smem.html
- fi-skl-6700k2: [PASS][25] -> [DMESG-WARN][26] +3 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
- fi-skl-guc: [PASS][27] -> [DMESG-WARN][28] +3 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-skl-guc/igt@gem_exec_suspend@basic-s3@smem.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-guc/igt@gem_exec_suspend@basic-s3@smem.html
- fi-cfl-guc: [PASS][29] -> [DMESG-WARN][30] +3 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-cfl-guc/igt@gem_exec_suspend@basic-s3@smem.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-cfl-guc/igt@gem_exec_suspend@basic-s3@smem.html
- fi-ivb-3770: [PASS][31] -> [DMESG-WARN][32] +3 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-ivb-3770/igt@gem_exec_suspend@basic-s3@smem.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-ivb-3770/igt@gem_exec_suspend@basic-s3@smem.html
- fi-rkl-guc: [PASS][33] -> [DMESG-WARN][34] +3 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-rkl-guc/igt@gem_exec_suspend@basic-s3@smem.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-rkl-guc/igt@gem_exec_suspend@basic-s3@smem.html
- fi-cml-u2: [PASS][35] -> [DMESG-WARN][36] +3 similar issues
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-cml-u2/igt@gem_exec_suspend@basic-s3@smem.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-cml-u2/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@dmabuf:
- fi-blb-e6850: NOTRUN -> [DMESG-WARN][37] +1 similar issue
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-blb-e6850/igt@i915_selftest@live@dmabuf.html
* igt@i915_selftest@live@gem:
- fi-snb-2600: [PASS][38] -> [DMESG-WARN][39] +3 similar issues
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-snb-2600/igt@i915_selftest@live@gem.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-snb-2600/igt@i915_selftest@live@gem.html
- fi-tgl-1115g4: [PASS][40] -> [DMESG-WARN][41] +3 similar issues
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-tgl-1115g4/igt@i915_selftest@live@gem.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-tgl-1115g4/igt@i915_selftest@live@gem.html
- fi-kbl-7500u: [PASS][42] -> [DMESG-WARN][43] +3 similar issues
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-kbl-7500u/igt@i915_selftest@live@gem.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-7500u/igt@i915_selftest@live@gem.html
- fi-cfl-8109u: [PASS][44] -> [DMESG-WARN][45] +3 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-cfl-8109u/igt@i915_selftest@live@gem.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-cfl-8109u/igt@i915_selftest@live@gem.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-kbl-soraka: NOTRUN -> [DMESG-WARN][46] +3 similar issues
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
- bat-dg1-5: [PASS][47] -> [DMESG-WARN][48] +5 similar issues
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-dg1-5/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-dg1-5/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
- fi-skl-6600u: NOTRUN -> [DMESG-WARN][49] +2 similar issues
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
- fi-snb-2520m: [PASS][50] -> [DMESG-WARN][51] +3 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
- fi-cfl-8700k: [PASS][52] -> [DMESG-WARN][53] +3 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
- fi-icl-u2: NOTRUN -> [DMESG-WARN][54] +3 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
* igt@prime_vgem@basic-fence-read:
- fi-bsw-kefka: [PASS][55] -> [DMESG-WARN][56] +14 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bsw-kefka/igt@prime_vgem@basic-fence-read.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bsw-kefka/igt@prime_vgem@basic-fence-read.html
* igt@prime_vgem@basic-gtt:
- fi-ilk-650: [PASS][57] -> [DMESG-WARN][58] +15 similar issues
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-ilk-650/igt@prime_vgem@basic-gtt.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-ilk-650/igt@prime_vgem@basic-gtt.html
* igt@prime_vgem@basic-read:
- fi-bwr-2160: [PASS][59] -> [DMESG-WARN][60] +9 similar issues
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bwr-2160/igt@prime_vgem@basic-read.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bwr-2160/igt@prime_vgem@basic-read.html
* igt@prime_vgem@basic-write:
- fi-bsw-n3050: [PASS][61] -> [DMESG-WARN][62] +13 similar issues
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bsw-n3050/igt@prime_vgem@basic-write.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bsw-n3050/igt@prime_vgem@basic-write.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_exec_fence@basic-busy@vecs0:
- {fi-jsl-1}: [PASS][63] -> [DMESG-WARN][64] +15 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-jsl-1/igt@gem_exec_fence@basic-busy@vecs0.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-jsl-1/igt@gem_exec_fence@basic-busy@vecs0.html
* igt@gem_exec_fence@basic-wait@vecs0:
- {fi-ehl-2}: [PASS][65] -> [DMESG-WARN][66] +15 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-ehl-2/igt@gem_exec_fence@basic-wait@vecs0.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-ehl-2/igt@gem_exec_fence@basic-wait@vecs0.html
- {bat-jsl-1}: [PASS][67] -> [DMESG-WARN][68] +15 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-jsl-1/igt@gem_exec_fence@basic-wait@vecs0.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-jsl-1/igt@gem_exec_fence@basic-wait@vecs0.html
* igt@gem_exec_suspend@basic-s0@smem:
- {bat-rpls-2}: [PASS][69] -> [DMESG-WARN][70]
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-rpls-2/igt@gem_exec_suspend@basic-s0@smem.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-rpls-2/igt@gem_exec_suspend@basic-s0@smem.html
- {fi-rkl-11600}: [PASS][71] -> [DMESG-WARN][72]
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-rkl-11600/igt@gem_exec_suspend@basic-s0@smem.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-rkl-11600/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_exec_suspend@basic-s3@smem:
- {bat-rpls-2}: [INCOMPLETE][73] ([i915#4898]) -> [DMESG-WARN][74]
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-rpls-2/igt@gem_exec_suspend@basic-s3@smem.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-rpls-2/igt@gem_exec_suspend@basic-s3@smem.html
* igt@i915_selftest@live@gem:
- {fi-tgl-dsi}: [PASS][75] -> [DMESG-WARN][76]
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-tgl-dsi/igt@i915_selftest@live@gem.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-tgl-dsi/igt@i915_selftest@live@gem.html
- {bat-rpls-2}: NOTRUN -> [DMESG-WARN][77]
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-rpls-2/igt@i915_selftest@live@gem.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- {bat-adlp-6}: [PASS][78] -> [DMESG-WARN][79] +3 similar issues
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
* igt@prime_vgem@basic-fence-read:
- {bat-jsl-2}: [PASS][80] -> [DMESG-WARN][81] +15 similar issues
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-jsl-2/igt@prime_vgem@basic-fence-read.html
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-jsl-2/igt@prime_vgem@basic-fence-read.html
Known issues
------------
Here are the changes found in Patchwork_22375 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@cs-multi-fence:
- fi-blb-e6850: NOTRUN -> [SKIP][82] ([fdo#109271]) +17 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-blb-e6850/igt@amdgpu/amd_basic@cs-multi-fence.html
* igt@amdgpu/amd_cs_nop@fork-gfx0:
- fi-icl-u2: NOTRUN -> [SKIP][83] ([fdo#109315]) +17 similar issues
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@amdgpu/amd_cs_nop@fork-gfx0.html
* igt@amdgpu/amd_cs_nop@sync-fork-compute0:
- fi-snb-2600: NOTRUN -> [SKIP][84] ([fdo#109271]) +17 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html
* igt@amdgpu/amd_cs_nop@sync-fork-gfx0:
- fi-skl-6600u: NOTRUN -> [SKIP][85] ([fdo#109271]) +21 similar issues
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@amdgpu/amd_cs_nop@sync-fork-gfx0.html
* igt@gem_exec_fence@basic-busy@bcs0:
- fi-kbl-soraka: NOTRUN -> [SKIP][86] ([fdo#109271]) +8 similar issues
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@gem_exec_fence@basic-busy@bcs0.html
* igt@gem_huc_copy@huc-copy:
- fi-skl-6600u: NOTRUN -> [SKIP][87] ([fdo#109271] / [i915#2190])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@gem_huc_copy@huc-copy.html
- fi-kbl-soraka: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#2190])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html
- fi-icl-u2: NOTRUN -> [SKIP][89] ([i915#2190])
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@basic:
- fi-kbl-soraka: NOTRUN -> [SKIP][90] ([fdo#109271] / [i915#4613]) +3 similar issues
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-icl-u2: NOTRUN -> [SKIP][91] ([i915#4613]) +3 similar issues
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@gem_lmem_swapping@verify-random:
- fi-skl-6600u: NOTRUN -> [SKIP][92] ([fdo#109271] / [i915#4613]) +3 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@gem_lmem_swapping@verify-random.html
* igt@i915_selftest@live@gem:
- fi-kbl-8809g: [PASS][93] -> [DMESG-WARN][94] ([i915#1888])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-kbl-8809g/igt@i915_selftest@live@gem.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-8809g/igt@i915_selftest@live@gem.html
* igt@i915_selftest@live@gt_pm:
- fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][95] ([i915#1886] / [i915#2291])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html
* igt@kms_busy@basic@flip:
- bat-adlp-4: [PASS][96] -> [DMESG-WARN][97] ([i915#3576])
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-adlp-4/igt@kms_busy@basic@flip.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-adlp-4/igt@kms_busy@basic@flip.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-kbl-soraka: NOTRUN -> [SKIP][98] ([fdo#109271] / [fdo#111827]) +8 similar issues
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_chamelium@hdmi-hpd-fast:
- fi-icl-u2: NOTRUN -> [SKIP][99] ([fdo#111827]) +8 similar issues
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_chamelium@vga-edid-read:
- fi-skl-6600u: NOTRUN -> [SKIP][100] ([fdo#109271] / [fdo#111827]) +8 similar issues
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@kms_chamelium@vga-edid-read.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- fi-icl-u2: NOTRUN -> [SKIP][101] ([fdo#109278]) +2 similar issues
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_force_connector_basic@force-load-detect:
- fi-icl-u2: NOTRUN -> [SKIP][102] ([fdo#109285])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- fi-skl-6600u: NOTRUN -> [SKIP][103] ([fdo#109271] / [i915#533])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-skl-6600u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
- fi-kbl-soraka: NOTRUN -> [SKIP][104] ([fdo#109271] / [i915#533])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-kbl-soraka/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@prime_vgem@basic-userptr:
- fi-icl-u2: NOTRUN -> [SKIP][105] ([i915#3301])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-icl-u2/igt@prime_vgem@basic-userptr.html
#### Possible fixes ####
* igt@i915_selftest@live@hangcheck:
- fi-snb-2600: [INCOMPLETE][106] ([i915#3921]) -> [PASS][107]
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
* igt@i915_selftest@live@requests:
- fi-blb-e6850: [DMESG-FAIL][108] ([i915#5026]) -> [PASS][109]
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-blb-e6850/igt@i915_selftest@live@requests.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-blb-e6850/igt@i915_selftest@live@requests.html
* igt@kms_busy@basic@modeset:
- bat-adlp-4: [DMESG-WARN][110] ([i915#3576]) -> [PASS][111] +1 similar issue
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-adlp-4/igt@kms_busy@basic@modeset.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-adlp-4/igt@kms_busy@basic@modeset.html
- {bat-adlp-6}: [DMESG-WARN][112] ([i915#3576]) -> [PASS][113]
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/bat-adlp-6/igt@kms_busy@basic@modeset.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/bat-adlp-6/igt@kms_busy@basic@modeset.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size:
- fi-bsw-n3050: [FAIL][114] ([i915#2346]) -> [PASS][115]
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11276/fi-bsw-n3050/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/fi-bsw-n3050/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.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
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308
[fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
[i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
[i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
[i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
[i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2411]: https://gitlab.freedesktop.org/drm/intel/issues/2411
[i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
[i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
[i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
[i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
[i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4898]: https://gitlab.freedesktop.org/drm/intel/issues/4898
[i915#5026]: https://gitlab.freedesktop.org/drm/intel/issues/5026
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
Build changes
-------------
* Linux: CI_DRM_11276 -> Patchwork_22375
CI-20190529: 20190529
CI_DRM_11276: 9f1f2bb5b108286547a5bb3e7b89d41b6c1300e4 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_6355: 83ec34916bd8268bc331105cf77c4d3d3cd352be @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_22375: fdc855c609b3edffaea14d27fa1341652db519e4 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
fdc855c609b3 drm/i915/: Add drm_cache.h
2c3691719f58 drm/i915/gem: Remove logic for wbinvd_on_all_cpus
2faaf6c21ed1 drm_cache: Add logic for wbvind_on_all_cpus
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22375/index.html
[-- Attachment #2: Type: text/html, Size: 27305 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread