linux-kernel - Re: [PATCH v3 1/1] DM: inplace compressed DM target

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] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 31 Jan 2017 18:32:44 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Ram Pai <linuxram@...ibm.com>
Cc:     kbuild-all@...org, dm-devel@...hat.com, linux-doc@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-raid@...r.kernel.org,
        agk@...hat.com, snitzer@...hat.com, corbet@....net,
        shli@...nel.org, hbabu@...ibm.com
Subject: Re: [PATCH v3 1/1] DM: inplace compressed DM target

Hi Ram,

[auto build test WARNING on dm/for-next]
[also build test WARNING on v4.10-rc6 next-20170130]
[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/Ram-Pai/DM-inplace-compressed-DM-target/20170131-154811
base:   https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git for-next
config: cris-allyesconfig (attached as .config)
compiler: cris-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=cris 

All warnings (new ones prefixed by >>):

   drivers/md/dm-inplace-compress.c: In function 'dm_icomp_handle_read_decomp':
>> drivers/md/dm-inplace-compress.c:1580:41: warning: passing argument 3 of 'dm_icomp_bio_copy' makes pointer from integer without a cast [-Wint-conversion]
       dm_icomp_bio_copy(req->bio, bio_off, empty_zero_page,
                                            ^~~~~~~~~~~~~~~
   drivers/md/dm-inplace-compress.c:1133:13: note: expected 'void *' but argument is of type 'long unsigned int'
    static void dm_icomp_bio_copy(struct bio *bio, off_t bio_off, void *buf,
                ^~~~~~~~~~~~~~~~~
   drivers/md/dm-inplace-compress.c:1593:40: warning: passing argument 3 of 'dm_icomp_bio_copy' makes pointer from integer without a cast [-Wint-conversion]
      dm_icomp_bio_copy(req->bio, bio_off, empty_zero_page,
                                           ^~~~~~~~~~~~~~~
   drivers/md/dm-inplace-compress.c:1133:13: note: expected 'void *' but argument is of type 'long unsigned int'
    static void dm_icomp_bio_copy(struct bio *bio, off_t bio_off, void *buf,
                ^~~~~~~~~~~~~~~~~
   In file included from drivers/md/dm-inplace-compress.c:12:0:
   drivers/md/dm-inplace-compress.c: In function 'dm_icomp_status':
   drivers/md/dm-inplace-compress.c:2090:10: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'long long int' [-Wformat=]
      DMEMIT("%lu %lu %lu",
             ^
   include/linux/device-mapper.h:577:46: note: in definition of macro 'DMEMIT'
         0 : scnprintf(result + sz, maxlen - sz, x))
                                                 ^
   drivers/md/dm-inplace-compress.c:2090:10: warning: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'long long int' [-Wformat=]
      DMEMIT("%lu %lu %lu",
             ^
   include/linux/device-mapper.h:577:46: note: in definition of macro 'DMEMIT'
         0 : scnprintf(result + sz, maxlen - sz, x))
                                                 ^
   drivers/md/dm-inplace-compress.c:2090:10: warning: format '%lu' expects argument of type 'long unsigned int', but argument 6 has type 'long long int' [-Wformat=]
      DMEMIT("%lu %lu %lu",
             ^
   include/linux/device-mapper.h:577:46: note: in definition of macro 'DMEMIT'
         0 : scnprintf(result + sz, maxlen - sz, x))
                                                 ^

vim +/dm_icomp_bio_copy +1580 drivers/md/dm-inplace-compress.c

  1564				dm_icomp_release_comp_buffer(io);
  1565				req->result = -EIO;
  1566				return;
  1567			}
  1568	
  1569			len = min_t(ssize_t,
  1570				max_t(ssize_t, decomp_len - src_off, 0),
  1571				max_t(ssize_t, bio_len - dst_off, 0));
  1572	
  1573			dm_icomp_bio_copy(req->bio, dst_off,
  1574				   io->decomp_data + src_off, len, false);
  1575	
  1576			/* io range in all_io list is ordered for read IO */
  1577			while (bio_off < dst_off) {
  1578				ssize_t size = min_t(ssize_t, PAGE_SIZE,
  1579						dst_off - bio_off);
> 1580				dm_icomp_bio_copy(req->bio, bio_off, empty_zero_page,
  1581						size, false);
  1582				bio_off += size;
  1583			}
  1584	
  1585			bio_off = dst_off + len;
  1586			dm_icomp_release_decomp_buffer(io);
  1587			dm_icomp_release_comp_buffer(io);
  1588		}

---
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" (40207 bytes)