linux-kernel - Re: [PATCH v2 1/2] drm/msm: Allocate msm_drm_private early and pass it as driver data

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]

Message-ID: <202112011405.Wn79h7q3-lkp@intel.com>
Date:   Wed, 1 Dec 2021 14:04:32 +0800
From:   kernel test robot <lkp@...el.com>
To:     AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@...labora.com>, robdclark@...il.com
Cc:     kbuild-all@...ts.01.org, dmitry.baryshkov@...aro.org,
        sean@...rly.run, airlied@...ux.ie, daniel@...ll.ch,
        maxime@...no.tech, linux-arm-msm@...r.kernel.org,
        dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] drm/msm: Allocate msm_drm_private early and pass
 it as driver data

Hi AngeloGioacchino,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on next-20211130]
[cannot apply to v5.16-rc3]
[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/AngeloGioacchino-Del-Regno/drm-msm-Fix-dsi-bridge-probe/20211130-221304
base:   git://anongit.freedesktop.org/drm/drm drm-next
config: csky-randconfig-c004-20211128 (https://download.01.org/0day-ci/archive/20211201/202112011405.Wn79h7q3-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>


cocci warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/msm/msm_drv.c:412:1-6: WARNING: invalid free of devm_ allocated data

vim +412 drivers/gpu/drm/msm/msm_drv.c

2aa31767259722 Sean Paul           2019-05-24  384  
98659487b845c0 Abhinav Kumar       2021-04-16  385  	msm_disp_snapshot_destroy(ddev);
98659487b845c0 Abhinav Kumar       2021-04-16  386  
2b669875332fbd Archit Taneja       2016-05-02  387  	drm_mode_config_cleanup(ddev);
c8afe684c95cd1 Rob Clark           2013-06-26  388  
2b669875332fbd Archit Taneja       2016-05-02  389  	pm_runtime_get_sync(dev);
f026e431cf8611 Thomas Zimmermann   2021-08-03  390  	msm_irq_uninstall(ddev);
2b669875332fbd Archit Taneja       2016-05-02  391  	pm_runtime_put_sync(dev);
c8afe684c95cd1 Rob Clark           2013-06-26  392  
16976085a114ae Archit Taneja       2016-11-03  393  	if (kms && kms->funcs)
c8afe684c95cd1 Rob Clark           2013-06-26  394  		kms->funcs->destroy(kms);
c8afe684c95cd1 Rob Clark           2013-06-26  395  
871d812aa43e63 Rob Clark           2013-11-16  396  	if (priv->vram.paddr) {
00085f1efa387a Krzysztof Kozlowski 2016-08-03  397  		unsigned long attrs = DMA_ATTR_NO_KERNEL_MAPPING;
871d812aa43e63 Rob Clark           2013-11-16  398  		drm_mm_takedown(&priv->vram.mm);
2b669875332fbd Archit Taneja       2016-05-02  399  		dma_free_attrs(dev, priv->vram.size, NULL,
00085f1efa387a Krzysztof Kozlowski 2016-08-03  400  			       priv->vram.paddr, attrs);
871d812aa43e63 Rob Clark           2013-11-16  401  	}
871d812aa43e63 Rob Clark           2013-11-16  402  
2b669875332fbd Archit Taneja       2016-05-02  403  	component_unbind_all(dev, ddev);
060530f1ea6740 Rob Clark           2014-03-03  404  
bc3220be22577e Rajesh Yadav        2018-06-21  405  	if (mdss && mdss->funcs)
bc3220be22577e Rajesh Yadav        2018-06-21  406  		mdss->funcs->destroy(ddev);
0a6030d224d3a4 Archit Taneja       2016-05-08  407  
2b669875332fbd Archit Taneja       2016-05-02  408  	ddev->dev_private = NULL;
4d8dc2dfae2c48 Thomas Zimmermann   2018-09-26  409  	drm_dev_put(ddev);
c8afe684c95cd1 Rob Clark           2013-06-26  410  
2aa31767259722 Sean Paul           2019-05-24  411  	destroy_workqueue(priv->wq);
c8afe684c95cd1 Rob Clark           2013-06-26 @412  	kfree(priv);
c8afe684c95cd1 Rob Clark           2013-06-26  413  
c8afe684c95cd1 Rob Clark           2013-06-26  414  	return 0;
c8afe684c95cd1 Rob Clark           2013-06-26  415  }
c8afe684c95cd1 Rob Clark           2013-06-26  416  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org