LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Kujau <lists@nerdbynature.de>
To: linuxppc-dev@lists.ozlabs.org
Cc: linux-fsdevel@vger.kernel.org, geert@linux-m68k.org
Subject: fs/hfsplus/xattr.c: comparison of distinct pointer types lacks a cast
Date: Sat, 24 May 2014 18:42:05 -0700 (PDT)	[thread overview]
Message-ID: <alpine.DEB.2.19.4.1405241829250.7552@trent.utfs.org> (raw)

Hi,

while compiling 3.15-rc6 for powerpc (gcc-4.8.1 crosscompiling on x86_64), 
this happens:

---------------------------------------------------------------
  CC [M]  fs/hfsplus/xattr_user.o
  CC [M]  fs/hfsplus/xattr_security.o
  CC [M]  fs/hfsplus/xattr.o
  CC [M]  fs/hfsplus/xattr_trusted.o
In file included from 
/usr/local/src/linux-git/arch/powerpc/include/asm/div64.h:1:0,
                 from /usr/local/src/linux-git/include/linux/kernel.h:124,
                 from /usr/local/src/linux-git/include/asm-generic/bug.h:13,
                 from /usr/local/src/linux-git/arch/powerpc/include/asm/bug.h:127,
                 from /usr/local/src/linux-git/include/linux/bug.h:4,
                 from /usr/local/src/linux-git/include/linux/thread_info.h:11,
                 from /usr/local/src/linux-git/include/asm-generic/preempt.h:4,
                 from arch/powerpc/include/generated/asm/preempt.h:1,
                 from /usr/local/src/linux-git/include/linux/preempt.h:18,
                 from /usr/local/src/linux-git/include/linux/spinlock.h:50,
                 from /usr/local/src/linux-git/include/linux/wait.h:8,
                 from /usr/local/src/linux-git/include/linux/fs.h:6,
                 from /usr/local/src/linux-git/fs/hfsplus/hfsplus_fs.h:19,
                 from /usr/local/src/linux-git/fs/hfsplus/xattr.c:9:
/usr/local/src/linux-git/fs/hfsplus/xattr.c: In function 
'hfsplus_init_header_node':
/usr/local/src/linux-git/include/asm-generic/div64.h:43:28: warning: 
comparison of distinct pointer types lacks a cast [enabled by default]
  (void)(((typeof((n)) *)0) == ((uint64_t *)0)); \
                            ^
/usr/local/src/linux-git/fs/hfsplus/xattr.c:86:2: note: in expansion of 
macro 'do_div'
  do_div(tmp, node_size);
  ^
  LD [M]  fs/hfsplus/hfsplus.o
---------------------------------------------------------------

I don't fully understand the issue here, but xattr.c:86 reads:

           do_div(tmp, node_size);

Now, "git log -S do_div --source fs/hfsplus/xattr.c" brought up the 
following commit from Geert, introducing do_div() to xattr.c

  commit a99b7069aab8fc3fb4f26d15795dc280b52e38b1 HEAD
  Author: Geert Uytterhoeven <geert@linux-m68k.org>
  Date:   Thu Nov 14 14:32:18 2013 -0800
  hfsplus: Fix undefined __divdi3 in hfsplus_init_header_node()

Reverting this commit makes the warning above go away, but I doubt this 
would be the correct solution here :-)

Note: the warning has been reported[0] by Geert for v3.13-rc7 back in 
January 2014 already.

Thanks,
Christian.

[0] https://lkml.org/lkml/2014/1/6/107
-- 
BOFH excuse #133:

It's not plugged in.

             reply	other threads:[~2014-05-25  1:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-25  1:42 Christian Kujau [this message]
2014-05-25  8:48 ` fs/hfsplus/xattr.c: comparison of distinct pointer types lacks a cast Geert Uytterhoeven
2014-05-25 19:07   ` Christian Kujau
2014-05-25 21:19     ` Sergei Antonov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.DEB.2.19.4.1405241829250.7552@trent.utfs.org \
    --to=lists@nerdbynature.de \
    --cc=geert@linux-m68k.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox