linux-kernel - Re: [PATCH 4/6] drm/msm/a6xx: Improve GMU force shutdown sequence

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 18 May 2023 06:19:34 +0800
From:   kernel test robot <lkp@...el.com>
To:     Konrad Dybcio <konrad.dybcio@...aro.org>,
        Rob Clark <robdclark@...il.com>,
        Abhinav Kumar <quic_abhinavk@...cinc.com>,
        Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Sean Paul <sean@...rly.run>, David Airlie <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>
Cc:     oe-kbuild-all@...ts.linux.dev, linux-arm-msm@...r.kernel.org,
        linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Marijn Suijten <marijn.suijten@...ainline.org>,
        freedreno@...ts.freedesktop.org
Subject: Re: [PATCH 4/6] drm/msm/a6xx: Improve GMU force shutdown sequence

Hi Konrad,

kernel test robot noticed the following build errors:

[auto build test ERROR on 065efa589871e93b6610c70c1e9de274ef1f1ba2]

url:    https://github.com/intel-lab-lkp/linux/commits/Konrad-Dybcio/drm-msm-a6xx-Explain-CP_PROTECT_CNTL-writes-in-a6xx_set_cp_protect/20230518-010130
base:   065efa589871e93b6610c70c1e9de274ef1f1ba2
patch link:    https://lore.kernel.org/r/20230517-topic-a7xx_prep-v1-4-7a964f2e99c2%40linaro.org
patch subject: [PATCH 4/6] drm/msm/a6xx: Improve GMU force shutdown sequence
config: mips-allyesconfig
compiler: mips-linux-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/c8d08c2d02fc0a1ef0cdcc93d041e3802e01904e
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Konrad-Dybcio/drm-msm-a6xx-Explain-CP_PROTECT_CNTL-writes-in-a6xx_set_cp_protect/20230518-010130
        git checkout c8d08c2d02fc0a1ef0cdcc93d041e3802e01904e
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=mips olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash drivers/gpu/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202305180505.bG7RPhZo-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/gpu/drm/msm/adreno/a6xx_gmu.c: In function 'a6xx_gmu_force_off':
>> drivers/gpu/drm/msm/adreno/a6xx_gmu.c:934:24: error: 'REG_A6XX_GMU_AHB_FENCE_STATUS_CLR' undeclared (first use in this function); did you mean 'REG_A6XX_GMU_AHB_FENCE_STATUS'?
     934 |         gmu_write(gmu, REG_A6XX_GMU_AHB_FENCE_STATUS_CLR, 0x7);
         |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
         |                        REG_A6XX_GMU_AHB_FENCE_STATUS
   drivers/gpu/drm/msm/adreno/a6xx_gmu.c:934:24: note: each undeclared identifier is reported only once for each function it appears in


vim +934 drivers/gpu/drm/msm/adreno/a6xx_gmu.c

   913	
   914	/* Force the GMU off in case it isn't responsive */
   915	static void a6xx_gmu_force_off(struct a6xx_gmu *gmu)
   916	{
   917		struct a6xx_gpu *a6xx_gpu = container_of(gmu, struct a6xx_gpu, gmu);
   918		struct adreno_gpu *adreno_gpu = &a6xx_gpu->base;
   919		struct msm_gpu *gpu = &adreno_gpu->base;
   920	
   921		/* Flush all the queues */
   922		a6xx_hfi_stop(gmu);
   923	
   924		/* Stop the interrupts */
   925		a6xx_gmu_irq_disable(gmu);
   926	
   927		/* Force off SPTP in case the GMU is managing it */
   928		a6xx_sptprac_disable(gmu);
   929	
   930		/* Make sure there are no outstanding RPMh votes */
   931		a6xx_gmu_rpmh_off(gmu);
   932	
   933		/* Clear the WRITEDROPPED fields and put fence into allow mode */
 > 934		gmu_write(gmu, REG_A6XX_GMU_AHB_FENCE_STATUS_CLR, 0x7);
   935		gmu_write(gmu, REG_A6XX_GMU_AO_AHB_FENCE_CTRL, 0);
   936	
   937		/* Make sure the above writes go through */
   938		wmb();
   939	
   940		/* Halt the gmu cm3 core */
   941		gmu_write(gmu, REG_A6XX_GMU_CM3_SYSRESET, 1);
   942	
   943		a6xx_bus_clear_pending_transactions(adreno_gpu, true);
   944	
   945		/* Reset GPU core blocks */
   946		gpu_write(gpu, REG_A6XX_RBBM_SW_RESET_CMD, 1);
   947		udelay(100);
   948	}
   949	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

View attachment "config" of type "text/plain" (335864 bytes)