drivers/scsi/bfa/bfad_attr.c:984:18: sparse: sparse: symbol 'bfad_im_vport_attrs' was not declared. Should it be static?
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ee1703cda8dc777e937dec172da55beaf1a74919
commit: e73af234a1a26a034dbf92539b54aedc5014dbb2 scsi: bfa: Switch to attribute groups
date: 5 weeks ago
config: i386-randconfig-s001-20211116 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.4-dirty
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout e73af234a1a26a034dbf92539b54aedc5014dbb2
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash drivers/scsi/bfa/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
drivers/scsi/bfa/bfad_attr.c:66:39: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned long long [usertype] node_name @@ got restricted __be64 [usertype] @@
drivers/scsi/bfa/bfad_attr.c:66:39: sparse: expected unsigned long long [usertype] node_name
drivers/scsi/bfa/bfad_attr.c:66:39: sparse: got restricted __be64 [usertype]
drivers/scsi/bfa/bfad_attr.c:92:39: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned long long [usertype] port_name @@ got restricted __be64 [usertype] @@
drivers/scsi/bfa/bfad_attr.c:92:39: sparse: expected unsigned long long [usertype] port_name
drivers/scsi/bfa/bfad_attr.c:92:39: sparse: got restricted __be64 [usertype]
drivers/scsi/bfa/bfad_attr.c:248:36: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned long long [usertype] fabric_name @@ got restricted __be64 [usertype] @@
drivers/scsi/bfa/bfad_attr.c:248:36: sparse: expected unsigned long long [usertype] fabric_name
drivers/scsi/bfa/bfad_attr.c:248:36: sparse: got restricted __be64 [usertype]
>> drivers/scsi/bfa/bfad_attr.c:984:18: sparse: sparse: symbol 'bfad_im_vport_attrs' was not declared. Should it be static?
vim +/bfad_im_vport_attrs +984 drivers/scsi/bfa/bfad_attr.c
983
> 984 struct attribute *bfad_im_vport_attrs[] = {
985 &dev_attr_serial_number.attr,
986 &dev_attr_model.attr,
987 &dev_attr_model_description.attr,
988 &dev_attr_node_name.attr,
989 &dev_attr_symbolic_name.attr,
990 &dev_attr_hardware_version.attr,
991 &dev_attr_driver_version.attr,
992 &dev_attr_option_rom_version.attr,
993 &dev_attr_firmware_version.attr,
994 &dev_attr_number_of_ports.attr,
995 &dev_attr_driver_name.attr,
996 &dev_attr_number_of_discovered_ports.attr,
997 NULL,
998 };
999
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
Re: [PATCHv7 09/11] powerpc: avoid discarding flags in system_call_exception()
by kernel test robot
Hi Mark,
I love your patch! Yet something to improve:
[auto build test ERROR on arm64/for-next/core]
[also build test ERROR on powerpc/next linus/master v5.16-rc1 next-20211117]
[cannot apply to tip/core/entry]
[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/Mark-Rutland/thread_info-use-hel...
base: https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/core
config: powerpc-allnoconfig (attached as .config)
compiler: powerpc-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/e1afc9f363490d24e2c29f28f27ba8673...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Mark-Rutland/thread_info-use-helpers-to-snapshot-thread-flags/20211118-004322
git checkout e1afc9f363490d24e2c29f28f27ba8673d9c8612
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
arch/powerpc/kernel/interrupt.c: In function 'system_call_exception':
>> arch/powerpc/kernel/interrupt.c:151:64: error: passing argument 2 of 'set_bits' makes pointer from integer without a cast [-Werror=int-conversion]
151 | set_bits(_TIF_RESTOREALL, current_thread_info()->flags);
In file included from include/linux/bitops.h:33,
from include/linux/kernel.h:12,
from include/linux/list.h:9,
from include/linux/rculist.h:10,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from include/linux/context_tracking.h:5,
from arch/powerpc/kernel/interrupt.c:3:
arch/powerpc/include/asm/bitops.h:67:41: note: expected 'volatile long unsigned int *' but argument is of type 'long unsigned int'
67 | volatile unsigned long *_p) \
| ~~~~~~~~~~~~~~~~~~~~~~~~^~
arch/powerpc/include/asm/bitops.h:82:1: note: in expansion of macro 'DEFINE_BITOP'
82 | DEFINE_BITOP(set_bits, or, "")
| ^~~~~~~~~~~~
cc1: all warnings being treated as errors
vim +/set_bits +151 arch/powerpc/kernel/interrupt.c
76
77 /* Has to run notrace because it is entered not completely "reconciled" */
78 notrace long system_call_exception(long r3, long r4, long r5,
79 long r6, long r7, long r8,
80 unsigned long r0, struct pt_regs *regs)
81 {
82 syscall_fn f;
83
84 kuep_lock();
85
86 regs->orig_gpr3 = r3;
87
88 if (IS_ENABLED(CONFIG_PPC_IRQ_SOFT_MASK_DEBUG))
89 BUG_ON(irq_soft_mask_return() != IRQS_ALL_DISABLED);
90
91 trace_hardirqs_off(); /* finish reconciling */
92
93 CT_WARN_ON(ct_state() == CONTEXT_KERNEL);
94 user_exit_irqoff();
95
96 BUG_ON(regs_is_unrecoverable(regs));
97 BUG_ON(!(regs->msr & MSR_PR));
98 BUG_ON(arch_irq_disabled_regs(regs));
99
100 #ifdef CONFIG_PPC_PKEY
101 if (mmu_has_feature(MMU_FTR_PKEY)) {
102 unsigned long amr, iamr;
103 bool flush_needed = false;
104 /*
105 * When entering from userspace we mostly have the AMR/IAMR
106 * different from kernel default values. Hence don't compare.
107 */
108 amr = mfspr(SPRN_AMR);
109 iamr = mfspr(SPRN_IAMR);
110 regs->amr = amr;
111 regs->iamr = iamr;
112 if (mmu_has_feature(MMU_FTR_BOOK3S_KUAP)) {
113 mtspr(SPRN_AMR, AMR_KUAP_BLOCKED);
114 flush_needed = true;
115 }
116 if (mmu_has_feature(MMU_FTR_BOOK3S_KUEP)) {
117 mtspr(SPRN_IAMR, AMR_KUEP_BLOCKED);
118 flush_needed = true;
119 }
120 if (flush_needed)
121 isync();
122 } else
123 #endif
124 kuap_assert_locked();
125
126 booke_restore_dbcr0();
127
128 account_cpu_user_entry();
129
130 account_stolen_time();
131
132 /*
133 * This is not required for the syscall exit path, but makes the
134 * stack frame look nicer. If this was initialised in the first stack
135 * frame, or if the unwinder was taught the first stack frame always
136 * returns to user with IRQS_ENABLED, this store could be avoided!
137 */
138 irq_soft_mask_regs_set_state(regs, IRQS_ENABLED);
139
140 /*
141 * If system call is called with TM active, set _TIF_RESTOREALL to
142 * prevent RFSCV being used to return to userspace, because POWER9
143 * TM implementation has problems with this instruction returning to
144 * transactional state. Final register values are not relevant because
145 * the transaction will be aborted upon return anyway. Or in the case
146 * of unsupported_scv SIGILL fault, the return state does not much
147 * matter because it's an edge case.
148 */
149 if (IS_ENABLED(CONFIG_PPC_TRANSACTIONAL_MEM) &&
150 unlikely(MSR_TM_TRANSACTIONAL(regs->msr)))
> 151 set_bits(_TIF_RESTOREALL, current_thread_info()->flags);
152
153 /*
154 * If the system call was made with a transaction active, doom it and
155 * return without performing the system call. Unless it was an
156 * unsupported scv vector, in which case it's treated like an illegal
157 * instruction.
158 */
159 #ifdef CONFIG_PPC_TRANSACTIONAL_MEM
160 if (unlikely(MSR_TM_TRANSACTIONAL(regs->msr)) &&
161 !trap_is_unsupported_scv(regs)) {
162 /* Enable TM in the kernel, and disable EE (for scv) */
163 hard_irq_disable();
164 mtmsr(mfmsr() | MSR_TM);
165
166 /* tabort, this dooms the transaction, nothing else */
167 asm volatile(".long 0x7c00071d | ((%0) << 16)"
168 :: "r"(TM_CAUSE_SYSCALL|TM_CAUSE_PERSISTENT));
169
170 /*
171 * Userspace will never see the return value. Execution will
172 * resume after the tbegin. of the aborted transaction with the
173 * checkpointed register state. A context switch could occur
174 * or signal delivered to the process before resuming the
175 * doomed transaction context, but that should all be handled
176 * as expected.
177 */
178 return -ENOSYS;
179 }
180 #endif // CONFIG_PPC_TRANSACTIONAL_MEM
181
182 local_irq_enable();
183
184 if (unlikely(current_thread_info()->flags & _TIF_SYSCALL_DOTRACE)) {
185 if (unlikely(trap_is_unsupported_scv(regs))) {
186 /* Unsupported scv vector */
187 _exception(SIGILL, regs, ILL_ILLOPC, regs->nip);
188 return regs->gpr[3];
189 }
190 /*
191 * We use the return value of do_syscall_trace_enter() as the
192 * syscall number. If the syscall was rejected for any reason
193 * do_syscall_trace_enter() returns an invalid syscall number
194 * and the test against NR_syscalls will fail and the return
195 * value to be used is in regs->gpr[3].
196 */
197 r0 = do_syscall_trace_enter(regs);
198 if (unlikely(r0 >= NR_syscalls))
199 return regs->gpr[3];
200 r3 = regs->gpr[3];
201 r4 = regs->gpr[4];
202 r5 = regs->gpr[5];
203 r6 = regs->gpr[6];
204 r7 = regs->gpr[7];
205 r8 = regs->gpr[8];
206
207 } else if (unlikely(r0 >= NR_syscalls)) {
208 if (unlikely(trap_is_unsupported_scv(regs))) {
209 /* Unsupported scv vector */
210 _exception(SIGILL, regs, ILL_ILLOPC, regs->nip);
211 return regs->gpr[3];
212 }
213 return -ENOSYS;
214 }
215
216 /* May be faster to do array_index_nospec? */
217 barrier_nospec();
218
219 if (unlikely(is_compat_task())) {
220 f = (void *)compat_sys_call_table[r0];
221
222 r3 &= 0x00000000ffffffffULL;
223 r4 &= 0x00000000ffffffffULL;
224 r5 &= 0x00000000ffffffffULL;
225 r6 &= 0x00000000ffffffffULL;
226 r7 &= 0x00000000ffffffffULL;
227 r8 &= 0x00000000ffffffffULL;
228
229 } else {
230 f = (void *)sys_call_table[r0];
231 }
232
233 return f(r3, r4, r5, r6, r7, r8);
234 }
235
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
drivers/gpu/drm/i915/gem/i915_gem_object.c:102: warning: expecting prototype for i915_gem_object_fini(). Prototype was for __i915_gem_object_fini() instead
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ee1703cda8dc777e937dec172da55beaf1a74919
commit: 068396bb21c8aa3b2f797c58eb9e623d7cf271bb drm/i915/ttm: Rework object initialization slightly
date: 7 weeks ago
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 068396bb21c8aa3b2f797c58eb9e623d7cf271bb
# save the attached .config to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/drm/i915/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/i915/gem/i915_gem_object.c:102: warning: expecting prototype for i915_gem_object_fini(). Prototype was for __i915_gem_object_fini() instead
drivers/gpu/drm/i915/gem/i915_gem_object.c:109: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* Mark up the object's coherency levels for a given cache_level
vim +102 drivers/gpu/drm/i915/gem/i915_gem_object.c
91
92 /**
93 * i915_gem_object_fini - Clean up a GEM object initialization
94 * @obj: The gem object to cleanup
95 *
96 * This function cleans up gem object fields that are set up by
97 * drm_gem_private_object_init() and i915_gem_object_init().
98 * It's primarily intended as a helper for backends that need to
99 * clean up the gem object in separate steps.
100 */
101 void __i915_gem_object_fini(struct drm_i915_gem_object *obj)
> 102 {
103 mutex_destroy(&obj->mm.get_page.lock);
104 mutex_destroy(&obj->mm.get_dma_page.lock);
105 dma_resv_fini(&obj->base._resv);
106 }
107
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
[xilinx-xlnx:master 11230/12535] drivers/remoteproc/ingenic_rproc.c:144:21: error: initialization of 'void * (*)(struct rproc *, u64, int)' {aka 'void * (*)(struct rproc *, long long unsigned int, int)'} from incompatible pointer type 'void * (*)(struct rproc *, u64, size_t)' {aka 'vo...
by kernel test robot
tree: https://github.com/Xilinx/linux-xlnx master
head: 0a88ef03d3015782318b4bc94ceb20dca375a01b
commit: a324889a971b8e827c53deaffe630b420bde4067 [11230/12535] remoteproc: Add support for peek from remote and acking kick from remote
config: microblaze-randconfig-r025-20211031 (attached as .config)
compiler: microblaze-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/Xilinx/linux-xlnx/commit/a324889a971b8e827c53deaffe630...
git remote add xilinx-xlnx https://github.com/Xilinx/linux-xlnx
git fetch --no-tags xilinx-xlnx master
git checkout a324889a971b8e827c53deaffe630b420bde4067
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=microblaze
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> drivers/remoteproc/ingenic_rproc.c:144:21: error: initialization of 'void * (*)(struct rproc *, u64, int)' {aka 'void * (*)(struct rproc *, long long unsigned int, int)'} from incompatible pointer type 'void * (*)(struct rproc *, u64, size_t)' {aka 'void * (*)(struct rproc *, long long unsigned int, unsigned int)'} [-Werror=incompatible-pointer-types]
144 | .da_to_va = ingenic_rproc_da_to_va,
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/remoteproc/ingenic_rproc.c:144:21: note: (near initialization for 'ingenic_rproc_ops.da_to_va')
cc1: some warnings being treated as errors
--
>> drivers/remoteproc/mtk_scp.c:463:27: error: initialization of 'void * (*)(struct rproc *, u64, int)' {aka 'void * (*)(struct rproc *, long long unsigned int, int)'} from incompatible pointer type 'void * (*)(struct rproc *, u64, size_t)' {aka 'void * (*)(struct rproc *, long long unsigned int, unsigned int)'} [-Werror=incompatible-pointer-types]
463 | .da_to_va = scp_da_to_va,
| ^~~~~~~~~~~~
drivers/remoteproc/mtk_scp.c:463:27: note: (near initialization for 'scp_ops.da_to_va')
cc1: some warnings being treated as errors
vim +144 drivers/remoteproc/ingenic_rproc.c
48f0a1bbb7586c Paul Cercueil 2020-05-15 137
48f0a1bbb7586c Paul Cercueil 2020-05-15 138 static struct rproc_ops ingenic_rproc_ops = {
4605ad8f458198 Mathieu Poirier 2020-06-30 139 .prepare = ingenic_rproc_prepare,
4605ad8f458198 Mathieu Poirier 2020-06-30 140 .unprepare = ingenic_rproc_unprepare,
48f0a1bbb7586c Paul Cercueil 2020-05-15 141 .start = ingenic_rproc_start,
48f0a1bbb7586c Paul Cercueil 2020-05-15 142 .stop = ingenic_rproc_stop,
48f0a1bbb7586c Paul Cercueil 2020-05-15 143 .kick = ingenic_rproc_kick,
48f0a1bbb7586c Paul Cercueil 2020-05-15 @144 .da_to_va = ingenic_rproc_da_to_va,
48f0a1bbb7586c Paul Cercueil 2020-05-15 145 };
48f0a1bbb7586c Paul Cercueil 2020-05-15 146
:::::: The code at line 144 was first introduced by commit
:::::: 48f0a1bbb7586c94e0f15116b06f8179df2fd60f remoteproc: ingenic: Added remoteproc driver
:::::: TO: Paul Cercueil <paul(a)crapouillou.net>
:::::: CC: Bjorn Andersson <bjorn.andersson(a)linaro.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
undefined reference to `drm_gem_fb_get_obj'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ee1703cda8dc777e937dec172da55beaf1a74919
commit: 57f6190a60ecc19f1ddddac0d7ea6524229271a9 drm/panel: ws2401: Add driver for WideChips WS2401
date: 4 months ago
config: m68k-randconfig-r026-20211018 (attached as .config)
compiler: m68k-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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 57f6190a60ecc19f1ddddac0d7ea6524229271a9
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=m68k SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
m68k-linux-ld: section .rodata VMA [0000000000002000,00000000000f513f] overlaps section .text VMA [0000000000000400,00000000005769af]
m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o: in function `mipi_dbi_buf_copy':
>> (.text+0x10e): undefined reference to `drm_gem_fb_get_obj'
>> m68k-linux-ld: (.text+0x16a): undefined reference to `drm_fb_swab'
>> m68k-linux-ld: (.text+0x17e): undefined reference to `drm_fb_memcpy'
>> m68k-linux-ld: (.text+0x19a): undefined reference to `drm_fb_xrgb8888_to_rgb565'
m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o: in function `mipi_dbi_dev_init_with_formats':
>> (.text+0x2ee): undefined reference to `drm_simple_display_pipe_init'
m68k-linux-ld: (.text+0x306): undefined reference to `drm_plane_enable_fb_damage_clips'
m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o: in function `mipi_dbi_fb_dirty':
>> drm_mipi_dbi.c:(.text+0x1508): undefined reference to `drm_gem_fb_get_obj'
m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o: in function `mipi_dbi_pipe_update':
>> (.text+0x1662): undefined reference to `drm_atomic_helper_damage_merged'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0xdc): undefined reference to `drm_gem_fb_create_with_dirty'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0xec): undefined reference to `drm_atomic_helper_check'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0xf0): undefined reference to `drm_atomic_helper_commit'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0x104): undefined reference to `drm_atomic_helper_connector_reset'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0x110): undefined reference to `drm_helper_probe_single_connector_modes'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0x124): undefined reference to `drm_atomic_helper_connector_duplicate_state'
>> m68k-linux-ld: drivers/gpu/drm/drm_mipi_dbi.o:(.rodata+0x128): undefined reference to `drm_atomic_helper_connector_destroy_state'
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
Re: [PATCH v3 1/6] drm: move the buddy allocator from i915 into common drm
by kernel test robot
Hi Arunpravin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm/drm-next]
[also build test ERROR on drm-intel/for-linux-next v5.16-rc1]
[cannot apply to drm-tip/drm-tip next-20211117]
[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/Arunpravin/drm-move-the-buddy-al...
base: git://anongit.freedesktop.org/drm/drm drm-next
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/47fb1aeae75661971f4526efddf4ae5b5...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Arunpravin/drm-move-the-buddy-allocator-from-i915-into-common-drm/20211117-041920
git checkout 47fb1aeae75661971f4526efddf4ae5b5738977f
# save the attached .config to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/intel_memory_region.c:242:
>> drivers/gpu/drm/i915/selftests/intel_memory_region.c:23:10: fatal error: i915_buddy.h: No such file or directory
23 | #include "i915_buddy.h"
| ^~~~~~~~~~~~~~
compilation terminated.
vim +23 drivers/gpu/drm/i915/selftests/intel_memory_region.c
232a6ebae419193 Matthew Auld 2019-10-08 14
340be48f2c5a3c0 Matthew Auld 2019-10-25 15 #include "gem/i915_gem_context.h"
b908be543e44414 Matthew Auld 2019-10-25 16 #include "gem/i915_gem_lmem.h"
232a6ebae419193 Matthew Auld 2019-10-08 17 #include "gem/i915_gem_region.h"
340be48f2c5a3c0 Matthew Auld 2019-10-25 18 #include "gem/selftests/igt_gem_utils.h"
232a6ebae419193 Matthew Auld 2019-10-08 19 #include "gem/selftests/mock_context.h"
99919be74aa3753 Thomas Hellström 2021-06-17 20 #include "gt/intel_engine_pm.h"
6804da20bb549e3 Chris Wilson 2019-10-27 21 #include "gt/intel_engine_user.h"
b908be543e44414 Matthew Auld 2019-10-25 22 #include "gt/intel_gt.h"
d53ec322dc7de32 Matthew Auld 2021-06-16 @23 #include "i915_buddy.h"
99919be74aa3753 Thomas Hellström 2021-06-17 24 #include "gt/intel_migrate.h"
ba12993c5228015 Matthew Auld 2020-01-29 25 #include "i915_memcpy.h"
d53ec322dc7de32 Matthew Auld 2021-06-16 26 #include "i915_ttm_buddy_manager.h"
01377a0d7e6648b Abdiel Janulgue 2019-10-25 27 #include "selftests/igt_flush_test.h"
2f0b97ca0211863 Matthew Auld 2019-10-08 28 #include "selftests/i915_random.h"
232a6ebae419193 Matthew Auld 2019-10-08 29
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
Re: [PATCH 2/2] xfs: make sure link path does not go away at access
by kernel test robot
Hi Ian,
I love your patch! Perhaps something to improve:
[auto build test WARNING on xfs-linux/for-next]
[also build test WARNING on mszeredi-vfs/overlayfs-next linus/master v5.16-rc1 next-20211117]
[cannot apply to djwong-xfs/djwong-devel]
[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/Ian-Kent/vfs-check-dentry-is-sti...
base: https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git for-next
config: i386-randconfig-s001-20211115 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.4-dirty
# https://github.com/0day-ci/linux/commit/1208c3b6210fbb49718cdf4fa5f7db35b...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Ian-Kent/vfs-check-dentry-is-still-valid-in-get_link/20211111-114013
git checkout 1208c3b6210fbb49718cdf4fa5f7db35bea008f6
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash fs/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> fs/xfs/xfs_inode.c:2648:17: sparse: sparse: incompatible types in comparison expression (different address spaces):
>> fs/xfs/xfs_inode.c:2648:17: sparse: char [noderef] __rcu *
>> fs/xfs/xfs_inode.c:2648:17: sparse: char *
--
>> fs/xfs/xfs_iops.c:531:16: sparse: sparse: incompatible types in comparison expression (different address spaces):
>> fs/xfs/xfs_iops.c:531:16: sparse: char [noderef] __rcu *
>> fs/xfs/xfs_iops.c:531:16: sparse: char *
vim +2648 fs/xfs/xfs_inode.c
2600
2601 /*
2602 * This is called to return an inode to the inode free list.
2603 * The inode should already be truncated to 0 length and have
2604 * no pages associated with it. This routine also assumes that
2605 * the inode is already a part of the transaction.
2606 *
2607 * The on-disk copy of the inode will have been added to the list
2608 * of unlinked inodes in the AGI. We need to remove the inode from
2609 * that list atomically with respect to freeing it here.
2610 */
2611 int
2612 xfs_ifree(
2613 struct xfs_trans *tp,
2614 struct xfs_inode *ip)
2615 {
2616 struct xfs_mount *mp = ip->i_mount;
2617 struct xfs_perag *pag;
2618 struct xfs_icluster xic = { 0 };
2619 struct xfs_inode_log_item *iip = ip->i_itemp;
2620 int error;
2621
2622 ASSERT(xfs_isilocked(ip, XFS_ILOCK_EXCL));
2623 ASSERT(VFS_I(ip)->i_nlink == 0);
2624 ASSERT(ip->i_df.if_nextents == 0);
2625 ASSERT(ip->i_disk_size == 0 || !S_ISREG(VFS_I(ip)->i_mode));
2626 ASSERT(ip->i_nblocks == 0);
2627
2628 pag = xfs_perag_get(mp, XFS_INO_TO_AGNO(mp, ip->i_ino));
2629
2630 /*
2631 * Pull the on-disk inode from the AGI unlinked list.
2632 */
2633 error = xfs_iunlink_remove(tp, pag, ip);
2634 if (error)
2635 goto out;
2636
2637 error = xfs_difree(tp, pag, ip->i_ino, &xic);
2638 if (error)
2639 goto out;
2640
2641 /*
2642 * Free any local-format data sitting around before we reset the
2643 * data fork to extents format. Note that the attr fork data has
2644 * already been freed by xfs_attr_inactive.
2645 */
2646 if (ip->i_df.if_format == XFS_DINODE_FMT_LOCAL) {
2647 kmem_free_rcu(ip->i_df.if_u1.if_data);
> 2648 RCU_INIT_POINTER(ip->i_df.if_u1.if_data, NULL);
2649 ip->i_df.if_bytes = 0;
2650 }
2651
2652 VFS_I(ip)->i_mode = 0; /* mark incore inode as free */
2653 ip->i_diflags = 0;
2654 ip->i_diflags2 = mp->m_ino_geo.new_diflags2;
2655 ip->i_forkoff = 0; /* mark the attr fork not in use */
2656 ip->i_df.if_format = XFS_DINODE_FMT_EXTENTS;
2657 if (xfs_iflags_test(ip, XFS_IPRESERVE_DM_FIELDS))
2658 xfs_iflags_clear(ip, XFS_IPRESERVE_DM_FIELDS);
2659
2660 /* Don't attempt to replay owner changes for a deleted inode */
2661 spin_lock(&iip->ili_lock);
2662 iip->ili_fields &= ~(XFS_ILOG_AOWNER | XFS_ILOG_DOWNER);
2663 spin_unlock(&iip->ili_lock);
2664
2665 /*
2666 * Bump the generation count so no one will be confused
2667 * by reincarnations of this inode.
2668 */
2669 VFS_I(ip)->i_generation++;
2670 xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
2671
2672 if (xic.deleted)
2673 error = xfs_ifree_cluster(tp, pag, ip, &xic);
2674 out:
2675 xfs_perag_put(pag);
2676 return error;
2677 }
2678
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1148:33: warning: variable 'priv' set but not used
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ee1703cda8dc777e937dec172da55beaf1a74919
commit: de99e6479885dfa3f64a9511a6477c2b7899e53f Merge tag 'drm-msm-next-2021-10-26' of https://gitlab.freedesktop.org/drm/msm into drm-next
date: 3 weeks ago
config: arm-randconfig-r005-20211116 (attached as .config)
compiler: arm-linux-gnueabi-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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout de99e6479885dfa3f64a9511a6477c2b7899e53f
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: In function 'dpu_encoder_virt_enable':
>> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1148:33: warning: variable 'priv' set but not used [-Wunused-but-set-variable]
1148 | struct msm_drm_private *priv;
| ^~~~
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: In function 'dpu_encoder_virt_disable':
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1197:33: warning: variable 'priv' set but not used [-Wunused-but-set-variable]
1197 | struct msm_drm_private *priv;
| ^~~~
vim +/priv +1148 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1143
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1144 static void dpu_encoder_virt_enable(struct drm_encoder *drm_enc)
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1145 {
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1146 struct dpu_encoder_virt *dpu_enc = NULL;
86b89080368b469 Jeykumar Sankaran 2018-09-05 1147 int ret = 0;
c943b4948b5848f Chandan Uddaraju 2020-08-27 @1148 struct msm_drm_private *priv;
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1149 struct drm_display_mode *cur_mode = NULL;
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1150
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1151 if (!drm_enc) {
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1152 DPU_ERROR("invalid encoder\n");
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1153 return;
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1154 }
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1155 dpu_enc = to_dpu_encoder_virt(drm_enc);
fba7427eb594964 Sean Paul 2018-11-16 1156
fba7427eb594964 Sean Paul 2018-11-16 1157 mutex_lock(&dpu_enc->enc_lock);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1158 cur_mode = &dpu_enc->base.crtc->state->adjusted_mode;
c943b4948b5848f Chandan Uddaraju 2020-08-27 1159 priv = drm_enc->dev->dev_private;
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1160
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1161 trace_dpu_enc_enable(DRMID(drm_enc), cur_mode->hdisplay,
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1162 cur_mode->vdisplay);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1163
86b89080368b469 Jeykumar Sankaran 2018-09-05 1164 /* always enable slave encoder before master */
86b89080368b469 Jeykumar Sankaran 2018-09-05 1165 if (dpu_enc->cur_slave && dpu_enc->cur_slave->ops.enable)
86b89080368b469 Jeykumar Sankaran 2018-09-05 1166 dpu_enc->cur_slave->ops.enable(dpu_enc->cur_slave);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1167
86b89080368b469 Jeykumar Sankaran 2018-09-05 1168 if (dpu_enc->cur_master && dpu_enc->cur_master->ops.enable)
86b89080368b469 Jeykumar Sankaran 2018-09-05 1169 dpu_enc->cur_master->ops.enable(dpu_enc->cur_master);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1170
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1171 ret = dpu_encoder_resource_control(drm_enc, DPU_ENC_RC_EVENT_KICKOFF);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1172 if (ret) {
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1173 DPU_ERROR_ENC(dpu_enc, "dpu resource control failed: %d\n",
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1174 ret);
fba7427eb594964 Sean Paul 2018-11-16 1175 goto out;
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1176 }
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1177
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1178 _dpu_encoder_virt_enable_helper(drm_enc);
fba7427eb594964 Sean Paul 2018-11-16 1179
167dac97eb46c7b Bjorn Andersson 2021-10-16 1180 if (drm_enc->encoder_type == DRM_MODE_ENCODER_TMDS) {
167dac97eb46c7b Bjorn Andersson 2021-10-16 1181 ret = msm_dp_display_enable(dpu_enc->dp, drm_enc);
c943b4948b5848f Chandan Uddaraju 2020-08-27 1182 if (ret) {
c943b4948b5848f Chandan Uddaraju 2020-08-27 1183 DPU_ERROR_ENC(dpu_enc, "dp display enable failed: %d\n",
c943b4948b5848f Chandan Uddaraju 2020-08-27 1184 ret);
c943b4948b5848f Chandan Uddaraju 2020-08-27 1185 goto out;
c943b4948b5848f Chandan Uddaraju 2020-08-27 1186 }
c943b4948b5848f Chandan Uddaraju 2020-08-27 1187 }
fba7427eb594964 Sean Paul 2018-11-16 1188 dpu_enc->enabled = true;
fba7427eb594964 Sean Paul 2018-11-16 1189
fba7427eb594964 Sean Paul 2018-11-16 1190 out:
fba7427eb594964 Sean Paul 2018-11-16 1191 mutex_unlock(&dpu_enc->enc_lock);
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1192 }
25fdd5933e4c0f5 Jeykumar Sankaran 2018-06-27 1193
:::::: The code at line 1148 was first introduced by commit
:::::: c943b4948b5848fc0e07f875edbd35a973879e22 drm/msm/dp: add displayPort driver support
:::::: TO: Chandan Uddaraju <chandanu(a)codeaurora.org>
:::::: CC: Rob Clark <robdclark(a)chromium.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week