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. Thanks, Ilya
