]> code.ossystems Code Review - openembedded-core.git/commit
linux-yocto/4.12: fix aufs compile warning
authorBruce Ashfield <bruce.ashfield@windriver.com>
Tue, 6 Mar 2018 18:11:13 +0000 (13:11 -0500)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 26 Mar 2018 14:06:24 +0000 (15:06 +0100)
commit20a7e72a1fa97f673de12856b3bf53c48a05a834
tree0f805e4092ec66cbe45bbbd65f6b68f5b284daad
parentafb9c2b28190e8f9ca8f2366a9129a5ccdd06356
linux-yocto/4.12: fix aufs compile warning

   From: "Kexin(Casey) Chen" <Casey.Chen@windriver.com>

   Forward port linux-yocto-4.8's patch to fix the build warning.

       http://git.yoctoproject.org/cgit/cgit.cgi/linux-yocto-4.8/commit/?h=standard/base&id=7e0dd2f2b0971f0e3191e1ddc088e09eb9855567

   fs/aufs/debug.h:95:19: warning: comparison of constant '0'
   with boolean expression is always false [-Wbool-compare]
      if (unlikely((e) < 0)) \
                    ^

   fs/aufs/vdir.c:852:2: note: in expansion of macro 'AuTraceErr'
      AuTraceErr(!valid);
      ^~~~~~~~~~

   In expansion of AuTraceErr(!valid), comparison of (!valid)
   and constant '0' always passes unlikely(x) false. function
   'static int seek_vdir(struct file *file, struct dir_context *ctx)'
   is to find whether there is a valid vd_deblk following ctx->pos.
   return 1 means valid, 0 for not. Change to AuTraceErr(valid - 1)
   makes more sense.

Signed-off-by: Kexin(Casey) Chen <Casey.Chen@windriver.com>
Signed-off-by: Dengke Du <dengke.du@windriver.com>
(From OE-Core rev: 1ba624ec410b13da9645efe7bc72908195c08259)

Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
meta/recipes-kernel/linux/linux-yocto-rt_4.12.bb
meta/recipes-kernel/linux/linux-yocto-tiny_4.12.bb
meta/recipes-kernel/linux/linux-yocto_4.12.bb