linux-kernel - Re: [PATCH v2] sd: Consider max_xfer_blocks if opt_xfer

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 for Android: free password hash cracker in your pocket

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

Date:   Tue, 28 Mar 2017 15:14:04 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Fam Zheng <famz@...hat.com>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org,
        "Martin K. Petersen" <martin.petersen@...cle.com>, famz@...hat.com,
        linux-scsi@...r.kernel.org, Laurence Oberman <loberman@...hat.com>,
        "James E.J. Bottomley" <jejb@...ux.vnet.ibm.com>
Subject: Re: [PATCH v2] sd: Consider max_xfer_blocks if opt_xfer_blocks is
 unusable

Hi Fam,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.11-rc4 next-20170327]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Fam-Zheng/sd-Consider-max_xfer_blocks-if-opt_xfer_blocks-is-unusable/20170328-141853
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: x86_64-randconfig-x016-201713 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   In file included from include/linux/list.h:8:0,
                    from include/linux/module.h:9,
                    from drivers//scsi/sd.c:35:
   drivers//scsi/sd.c: In function 'sd_revalidate_disk':
   include/linux/kernel.h:755:16: warning: comparison of distinct pointer types lacks a cast
     (void) (&min1 == &min2);   \
                   ^
   include/linux/kernel.h:758:2: note: in expansion of macro '__min'
     __min(typeof(x), typeof(y),   \
     ^~~~~
>> include/linux/kernel.h:783:39: note: in expansion of macro 'min'
     __x == 0 ? __y : ((__y == 0) ? __x : min(__x, __y)); })
                                          ^~~
>> drivers//scsi/sd.c:2960:11: note: in expansion of macro 'min_not_zero'
     rw_max = min_not_zero(rw_max, logical_to_sectors(sdp, dev_max));
              ^~~~~~~~~~~~
--
   In file included from include/linux/list.h:8:0,
                    from include/linux/module.h:9,
                    from drivers/scsi/sd.c:35:
   drivers/scsi/sd.c: In function 'sd_revalidate_disk':
   include/linux/kernel.h:755:16: warning: comparison of distinct pointer types lacks a cast
     (void) (&min1 == &min2);   \
                   ^
   include/linux/kernel.h:758:2: note: in expansion of macro '__min'
     __min(typeof(x), typeof(y),   \
     ^~~~~
>> include/linux/kernel.h:783:39: note: in expansion of macro 'min'
     __x == 0 ? __y : ((__y == 0) ? __x : min(__x, __y)); })
                                          ^~~
   drivers/scsi/sd.c:2960:11: note: in expansion of macro 'min_not_zero'
     rw_max = min_not_zero(rw_max, logical_to_sectors(sdp, dev_max));
              ^~~~~~~~~~~~

vim +/min_not_zero +2960 drivers//scsi/sd.c

  2944		dev_max = min_not_zero(dev_max, sdkp->max_xfer_blocks);
  2945		q->limits.max_dev_sectors = logical_to_sectors(sdp, dev_max);
  2946	
  2947		/*
  2948		 * Use the device's preferred I/O size for reads and writes
  2949		 * unless the reported value is unreasonably small, large, or
  2950		 * garbage.
  2951		 */
  2952		if (sdkp->opt_xfer_blocks &&
  2953		    sdkp->opt_xfer_blocks <= dev_max &&
  2954		    sdkp->opt_xfer_blocks <= SD_DEF_XFER_BLOCKS &&
  2955		    logical_to_bytes(sdp, sdkp->opt_xfer_blocks) >= PAGE_SIZE) {
  2956			q->limits.io_opt = logical_to_bytes(sdp, sdkp->opt_xfer_blocks);
  2957			rw_max = logical_to_sectors(sdp, sdkp->opt_xfer_blocks);
  2958		} else
  2959			rw_max = BLK_DEF_MAX_SECTORS;
> 2960		rw_max = min_not_zero(rw_max, logical_to_sectors(sdp, dev_max));
  2961	
  2962		/* Combine with controller limits */
  2963		q->limits.max_sectors = min(rw_max, queue_max_hw_sectors(q));
  2964	
  2965		set_capacity(disk, logical_to_sectors(sdp, sdkp->capacity));
  2966		sd_config_write_same(sdkp);
  2967		kfree(buffer);
  2968	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (28375 bytes)