[linux-next:master 1761/14297] drivers/scsi/sd.c:4493:2-17: WARNING: NULL check before some freeing functions is not needed.

[linux-next:master 1761/14297] drivers/scsi/sd.c:4493:2-17: WARNING: NULL check before some freeing functions is not needed.

[kernel test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   97e797263a5e963da3d1e66e743fd518567dfe37
commit: 7179e626b76eb42f2529c6f6dd6ba88ea2445372 [1761/14297] scsi: sd: Enable sector size > PAGE_SIZE in SCSI sd driver
config: powerpc-randconfig-r051-20260421 (https://download.01.org/0day-ci/archive/20260421/202604212233.FA0funYe-lkp@intel.com/config)
compiler: powerpc-linux-gcc (GCC) 8.5.0

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202604212233.FA0funYe-lkp@intel.com/

cocci warnings: (new ones prefixed by >>)
>> drivers/scsi/sd.c:4493:2-17: WARNING: NULL check before some freeing functions is not needed.

vim +4493 drivers/scsi/sd.c

  4478	
  4479	/**
  4480	 *	exit_sd - exit point for this driver (when it is a module).
  4481	 *
  4482	 *	Note: this function unregisters this driver from the scsi mid-level.
  4483	 **/
  4484	static void __exit exit_sd(void)
  4485	{
  4486		int i;
  4487	
  4488		SCSI_LOG_HLQUEUE(3, printk("exit_sd: exiting sd driver\n"));
  4489	
  4490		scsi_unregister_driver(&sd_template);
  4491		mempool_destroy(sd_page_pool);
  4492		if (sd_large_page_pool)
> 4493			mempool_destroy(sd_large_page_pool);
  4494	
  4495		class_unregister(&sd_disk_class);
  4496	
  4497		for (i = 0; i < SD_MAJORS; i++)
  4498			unregister_blkdev(sd_major(i), "sd");
  4499	}
  4500	

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