From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 137FF2C80 for ; Tue, 7 Dec 2021 10:51:56 +0000 (UTC) X-IronPort-AV: E=McAfee;i="6200,9189,10190"; a="224420891" X-IronPort-AV: E=Sophos;i="5.87,293,1631602800"; d="scan'208";a="224420891" Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Dec 2021 02:51:55 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,293,1631602800"; d="scan'208";a="611630779" Received: from lkp-server02.sh.intel.com (HELO 9e1e9f9b3bcb) ([10.239.97.151]) by orsmga004.jf.intel.com with ESMTP; 07 Dec 2021 02:51:53 -0800 Received: from kbuild by 9e1e9f9b3bcb with local (Exim 4.92) (envelope-from ) id 1muY4e-000MTV-L4; Tue, 07 Dec 2021 10:51:52 +0000 Date: Tue, 7 Dec 2021 18:51:27 +0800 From: kernel test robot To: Dan Carpenter , Antonino Daplas Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH] fbdev: savagefb: make a variable local Message-ID: <202112071822.Ry2XAYAN-lkp@intel.com> References: <20211203095715.GD2480@kili> Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In-Reply-To: <20211203095715.GD2480@kili> User-Agent: Mutt/1.10.1 (2018-07-13) Hi Dan, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v5.16-rc4 next-20211207] [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/Dan-Carpenter/fbdev-savage= fb-make-a-variable-local/20211203-175849 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git = 5f58da2befa58edf3a70b91ed87ed9bf77f1e70e config: mips-buildonly-randconfig-r004-20211207 (https://download.01.org/0d= ay-ci/archive/20211207/202112071822.Ry2XAYAN-lkp@intel.com/config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project f50be8= eb0a12a61d23db6cda452c693001d76898) reproduce (this is a W=3D1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/= make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install mips cross compiling tool for clang build # apt-get install binutils-mips-linux-gnu # https://github.com/0day-ci/linux/commit/9ea7012b220fc1bd8aa2f0a65= b97403cea046343 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Dan-Carpenter/fbdev-savagefb-make-= a-variable-local/20211203-175849 git checkout 9ea7012b220fc1bd8aa2f0a65b97403cea046343 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dclang make.cross W=3D= 1 O=3Dbuild_dir ARCH=3Dmips SHELL=3D/bin/bash drivers/video/fbdev/savage/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All warnings (new ones prefixed by >>): >> drivers/video/fbdev/savage/savagefb_driver.c:2173:17: warning: unused va= riable 'edid' unsigned char ^ fatal error: error in backend: Nested variants found in inline asm strin= g: '.if ( 0x00 ) !=3D -1)) 0x00 ) !=3D -1)) : ($( static struct ftrace_bran= ch_data __attribute__((__aligned__(4))) __attribute__((__section__("_ftrace= _branch"))) __if_trace =3D $( .func =3D __func__, .file =3D "arch/mips/incl= ude/asm/barrier.h", .line =3D 16, $); 0x00 ) !=3D -1)) : $))) ) && ( (1 << = 0) ); .set push; .set mips64r2; .rept 1; sync 0x00; .endr; .set pop; .else;= ; .endif' PLEASE submit a bug report to https://bugs.llvm.org/ and include the cra= sh backtrace, preprocessed source, and associated run script. Stack dump: 0. Program arguments: clang -Wp,-MMD,drivers/video/fbdev/savage/.savagef= b_driver.o.d -nostdinc -Iarch/mips/include -I./arch/mips/include/generated = -Iinclude -I./include -Iarch/mips/include/uapi -I./arch/mips/include/genera= ted/uapi -Iinclude/uapi -I./include/generated/uapi -include include/linux/c= ompiler-version.h -include include/linux/kconfig.h -include include/linux/c= ompiler_types.h -D__KERNEL__ -DVMLINUX_LOAD_ADDRESS=3D0xffffffff80100000 -D= LINKER_LOAD_ADDRESS=3D0xffffffff80100000 -DDATAOFFSET=3D0 -Qunused-argument= s -fmacro-prefix-map=3D=3D -DKBUILD_EXTRA_WARN1 -Wall -Wundef -Werror=3Dstr= ict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common -fshort-wcha= r -fno-PIE -Werror=3Dimplicit-function-declaration -Werror=3Dimplicit-int -= Werror=3Dreturn-type -Wno-format-security -std=3Dgnu89 --target=3Dmips64el-= linux -fintegrated-as -Werror=3Dunknown-warning-option -Werror=3Dignored-op= timization-argument -mabi=3D64 -G 0 -mno-abicalls -fno-pic -pipe -msoft-flo= at -DGAS_HAS_SET_HARDFLOAT -Wa,-msoft-float -ffreestanding -EL -fno-stack-c= heck -march=3Dmips64 -Wa,--trap -DTOOLCHAIN_SUPPORTS_VIRT -Iarch/mips/inclu= de/asm/mach-malta -Iarch/mips/include/asm/mach-generic -fno-asynchronous-un= wind-tables -fno-delete-null-pointer-checks -Wno-frame-address -Wno-address= -of-packed-member -O2 -Wframe-larger-than=3D2048 -fstack-protector -Wimplic= it-fallthrough -Wno-gnu -mno-global-merge -Wno-unused-but-set-variable -Wno= -unused-const-variable -ftrivial-auto-var-init=3Dzero -enable-trivial-auto-= var-init-zero-knowing-it-will-be-removed-from-clang -fno-stack-clash-protec= tion -pg -falign-functions=3D64 -Wdeclaration-after-statement -Wvla -Wno-po= inter-sign -Wno-array-bounds -fno-strict-overflow -fno-stack-check -Werror= =3Ddate-time -Werror=3Dincompatible-pointer-types -Wextra -Wunused -Wno-unu= sed-parameter -Wmissing-declarations -Wmissing-format-attribute -Wmissing-p= rototypes -Wold-style-definition -Wmissing-include-dirs -Wunused-but-set-va= riable -Wunused-const-variable -Wno-missing-field-initializers -Wno-sign-co= mpare -Wno-type-limits -I drivers/video/fbdev/savage -I ./drivers/video/fbd= ev/savage -ffunction-sections -fdata-sections -DKBUILD_MODFILE=3D"drivers/v= ideo/fbdev/savage/savagefb" -DKBUILD_BASENAME=3D"savagefb_driver" -DKBUILD_= MODNAME=3D"savagefb" -D__KBUILD_MODNAME=3Dkmod_savagefb -c -o drivers/video= /fbdev/savage/savagefb_driver.o drivers/video/fbdev/savage/savagefb_driver.c 1. parser at end of file 2. Code generation 3. Running pass 'Function Pass Manager' on module 'drivers/video/fbdev/s= avage/savagefb_driver.c'. 4. Running pass 'Mips Assembly Printer' on function '@savagefb_probe' #0 0x000055c047cb37df Signals.cpp:0:0 #1 0x000055c047cb16fc llvm::sys::CleanupOnSignal(unsigned long) (/opt/cr= oss/clang-097a1cb1d5/bin/clang-14+0x339a6fc) #2 0x000055c047bfef87 llvm::CrashRecoveryContext::HandleExit(int) (/opt/= cross/clang-097a1cb1d5/bin/clang-14+0x32e7f87) #3 0x000055c047ca9dae llvm::sys::Process::Exit(int, bool) (/opt/cross/cl= ang-097a1cb1d5/bin/clang-14+0x3392dae) #4 0x000055c04597378b (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x105c78= b) #5 0x000055c047c05a2c llvm::report_fatal_error(llvm::Twine const&, bool)= (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x32eea2c) #6 0x000055c0488b15e8 llvm::AsmPrinter::emitInlineAsm(llvm::MachineInstr= const (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x3f9a5e8) #7 0x000055c0488ad3e9 llvm::AsmPrinter::emitFunctionBody() (/opt/cross/c= lang-097a1cb1d5/bin/clang-14+0x3f963e9) #8 0x000055c0463c0afe llvm::MipsAsmPrinter::runOnMachineFunction(llvm::M= achineFunction&) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x1aa9afe) #9 0x000055c047045a1d llvm::MachineFunctionPass::runOnFunction(llvm::Fun= ction&) (.part.53) MachineFunctionPass.cpp:0:0 #10 0x000055c04746dfc7 llvm::FPPassManager::runOnFunction(llvm::Function= &) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x2b56fc7) #11 0x000055c04746e141 llvm::FPPassManager::runOnModule(llvm::Module&) (= /opt/cross/clang-097a1cb1d5/bin/clang-14+0x2b57141) #12 0x000055c04746f41f llvm::legacy::PassManagerImpl::run(llvm::Module&)= (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x2b5841f) #13 0x000055c047fbab9a clang::EmitBackendOutput(clang::DiagnosticsEngine= &, clang::HeaderSearchOptions const&, clang::CodeGenOptions const&, clang::= TargetOptions const&, clang::LangOptions const&, llvm::StringRef, clang::Ba= ckendAction, std::unique_ptr >) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x36a3= b9a) #14 0x000055c048c39443 clang::BackendConsumer::HandleTranslationUnit(cla= ng::ASTContext&) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x4322443) #15 0x000055c049738d99 clang::ParseAST(clang::Sema&, bool, bool) (/opt/c= ross/clang-097a1cb1d5/bin/clang-14+0x4e21d99) #16 0x000055c048c3829f clang::CodeGenAction::ExecuteAction() (/opt/cross= /clang-097a1cb1d5/bin/clang-14+0x432129f) #17 0x000055c0485c1401 clang::FrontendAction::Execute() (/opt/cross/clan= g-097a1cb1d5/bin/clang-14+0x3caa401) #18 0x000055c0485594ea clang::CompilerInstance::ExecuteAction(clang::Fro= ntendAction&) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x3c424ea) #19 0x000055c04868a5ab (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x3d735= ab) #20 0x000055c0459744d4 cc1_main(llvm::ArrayRef::callback_fn >, std::__cxx11::basic_string, std::alloca= tor const::'lambda'()>(long) Job.cpp:0:0 #23 0x000055c047bfee43 llvm::CrashRecoveryContext::RunSafely(llvm::funct= ion_ref) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x32e7e43) #24 0x000055c0483fcdb7 clang::driver::CC1Command::Execute(llvm::ArrayRef= >, std::__cxx11::basic_string, std::allocator const (.part.216) Job.cpp:0:0 #25 0x000055c0483d40c7 clang::driver::Compilation::ExecuteCommand(clang:= :driver::Command const&, clang::driver::Command const (/opt/cross/clang-097= a1cb1d5/bin/clang-14+0x3abd0c7) #26 0x000055c0483d4aa7 clang::driver::Compilation::ExecuteJobs(clang::dr= iver::JobList const&, llvm::SmallVectorImpl&) const (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x3abdaa7) #27 0x000055c0483ddde9 clang::driver::Driver::ExecuteCompilation(clang::= driver::Compilation&, llvm::SmallVectorImpl&) (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x3ac6de9) #28 0x000055c04589f17f main (/opt/cross/clang-097a1cb1d5/bin/clang-14+0x= f8817f) #29 0x00007f312c6b8d0a __libc_start_main (/lib/x86_64-linux-gnu/libc.so.= 6+0x26d0a) #30 0x000055c04597153a _start (/opt/cross/clang-097a1cb1d5/bin/clang-14+= 0x105a53a) clang-14: error: clang frontend command failed with exit code 70 (use -v= to see invocation) clang version 14.0.0 (git://gitmirror/llvm_project 097a1cb1d5ebb3a0ec4bc= aed8ba3ff6a8e33c00a) Target: mips64el-unknown-linux Thread model: posix InstalledDir: /opt/cross/clang-097a1cb1d5/bin clang-14: note: diagnostic msg: Makefile arch drivers include kernel nr_bisected scripts source usr vim +/edid +2173 drivers/video/fbdev/savage/savagefb_driver.c 2167=09 2168 static int savagefb_probe(struct pci_dev *dev, const struct pci_devi= ce_id *id) 2169 { 2170 struct fb_info *info; 2171 struct savagefb_par *par; 2172 u_int h_sync, v_sync; > 2173 unsigned char *edid; 2174 int err, lpitch; 2175 int video_len; 2176=09 2177 DBG("savagefb_probe"); 2178=09 2179 info =3D framebuffer_alloc(sizeof(struct savagefb_par), &dev->dev); 2180 if (!info) 2181 return -ENOMEM; 2182 par =3D info->par; 2183 mutex_init(&par->open_lock); 2184 err =3D pci_enable_device(dev); 2185 if (err) 2186 goto failed_enable; 2187=09 2188 if ((err =3D pci_request_regions(dev, "savagefb"))) { 2189 printk(KERN_ERR "cannot request PCI regions\n"); 2190 goto failed_enable; 2191 } 2192=09 2193 err =3D -ENOMEM; 2194=09 2195 if ((err =3D savage_init_fb_info(info, dev, id))) 2196 goto failed_init; 2197=09 2198 err =3D savage_map_mmio(info); 2199 if (err) 2200 goto failed_mmio; 2201=09 2202 video_len =3D savage_init_hw(par); 2203 /* FIXME: can't be negative */ 2204 if (video_len < 0) { 2205 err =3D video_len; 2206 goto failed_mmio; 2207 } 2208=09 2209 err =3D savage_map_video(info, video_len); 2210 if (err) 2211 goto failed_video; 2212=09 2213 INIT_LIST_HEAD(&info->modelist); 2214 #if defined(CONFIG_FB_SAVAGE_I2C) 2215 savagefb_create_i2c_busses(info); 2216 savagefb_probe_i2c_connector(info, &edid); 2217 fb_edid_to_monspecs(edid, &info->monspecs); 2218 kfree(edid); 2219 fb_videomode_to_modelist(info->monspecs.modedb, 2220 info->monspecs.modedb_len, 2221 &info->modelist); 2222 #endif 2223 info->var =3D savagefb_var800x600x8; 2224 /* if a panel was detected, default to a CVT mode instead */ 2225 if (par->SavagePanelWidth) { 2226 struct fb_videomode cvt_mode; 2227=09 2228 memset(&cvt_mode, 0, sizeof(cvt_mode)); 2229 cvt_mode.xres =3D par->SavagePanelWidth; 2230 cvt_mode.yres =3D par->SavagePanelHeight; 2231 cvt_mode.refresh =3D 60; 2232 /* FIXME: if we know there is only the panel 2233 * we can enable reduced blanking as well */ 2234 if (fb_find_mode_cvt(&cvt_mode, 0, 0)) 2235 printk(KERN_WARNING "No CVT mode found for panel\n"); 2236 else if (fb_find_mode(&info->var, info, NULL, NULL, 0, 2237 &cvt_mode, 0) !=3D 3) 2238 info->var =3D savagefb_var800x600x8; 2239 } 2240=09 2241 if (mode_option) { 2242 fb_find_mode(&info->var, info, mode_option, 2243 info->monspecs.modedb, info->monspecs.modedb_len, 2244 NULL, 8); 2245 } else if (info->monspecs.modedb !=3D NULL) { 2246 const struct fb_videomode *mode; 2247=09 2248 mode =3D fb_find_best_display(&info->monspecs, &info->modelist); 2249 savage_update_var(&info->var, mode); 2250 } 2251=09 2252 /* maximize virtual vertical length */ 2253 lpitch =3D info->var.xres_virtual*((info->var.bits_per_pixel + 7) >= > 3); 2254 info->var.yres_virtual =3D info->fix.smem_len/lpitch; 2255=09 2256 if (info->var.yres_virtual < info->var.yres) { 2257 err =3D -ENOMEM; 2258 goto failed; 2259 } 2260=09 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org