Ilya Dryomov <idryomov@xxxxxxxxx> writes: > On Thu, Oct 18, 2018 at 1:28 PM Luis Henriques <lhenriques@xxxxxxxx> wrote: >> >> Luis Henriques <lhenriques@xxxxxxxx> writes: >> >> > kbuild test robot <lkp@xxxxxxxxx> writes: >> > >> >> tree: https://github.com/ceph/ceph-client.git testing >> >> head: 6fb0eaf13dcddf369aee4f72d00bd041111cc606 >> >> commit: cd574cfa473f997b8124a97a0eac2c24671a414c [3/4] ceph: support copy_file_range file operation >> >> config: sh-allmodconfig (attached as .config) >> >> compiler: sh4-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 >> >> reproduce: >> >> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross >> >> chmod +x ~/bin/make.cross >> >> git checkout cd574cfa473f997b8124a97a0eac2c24671a414c >> >> # save the attached .config to linux build tree >> >> GCC_VERSION=7.2.0 make.cross ARCH=sh >> >> >> >> All warnings (new ones prefixed by >>): >> >> >> >> fs//ceph/file.c: In function 'is_file_size_ok': >> >>>> <command-line>:0:16: warning: format '%ld' expects argument of type 'long int', but argument 8 has type 'size_t {aka unsigned int}' [-Wformat=] >> > >> > Ugh! I believe the patch below should fix this build warning (although >> > I didn't actually tried to reproduce it). >> > >> > Anyway, sorry for the noise and feel free to fold this patch into the >> > copy_file_range series. >> > >> >> Actually.. I just found another similar warning. Please use the patch >> bellow instead. > > Done, and I also patched it to work with preallocated message data > items change, which came in a bit earlier. Awesome, thanks! I haven't took a look at those patches yet, but they're on my todo list ;-) Cheers, -- Luis
