Re: [kbuild-all] [PATCH V3 4/4] dma: add Qualcomm Technologies HIDMA channel driver
From: Arnd Bergmann
Date: Thu Nov 12 2015 - 08:49:56 EST
On Thursday 12 November 2015 16:20:15 Fengguang Wu wrote:
> Hi Arnd,
> On Wed, Nov 11, 2015 at 09:42:00AM +0100, Arnd Bergmann wrote:
> > On Wednesday 11 November 2015 10:21:03 Fengguang Wu wrote:
> > > Hi Sinan,
> > >
> > > Sorry please ignore this warning -- it's actually a problem specific
> > > to the mn10300 arch. I'll disable such warning in mn10300 in future.
> > I just tried to find what happened here. mn10300 appears to define
> > the type based on the gcc version:
> > #if __GNUC__ == 4
> > typedef unsigned int __kernel_size_t;
> > typedef signed int __kernel_ssize_t;
> > #else
> > typedef unsigned long __kernel_size_t;
> > typedef signed long __kernel_ssize_t;
> > #endif
> > while gcc defines it based on whether you are using a Linux targetted
> > gcc or a bare-metal one:
> > gcc/config/mn10300/linux.h:#undef SIZE_TYPE
> > gcc/config/mn10300/mn10300.h:#undef SIZE_TYPE
> > gcc/config/mn10300/mn10300.h:#define SIZE_TYPE "unsigned int"
> > I can think of two reasons why it went wrong here:
> > a) You are using gcc-5.x, and the check in the kernel should be ">="
> > rather than "==". We should probably fix that regardless
> > b) You are using a bare-metal gcc rather than a Linux version.
> > I couldn't find an mn10300 gcc on kernel.org, which one do you use?
> I used this mn10300 compiler:
Ok, so this is not gcc-5.x (i.e. we are not hitting the first problem), but it uses
./lib/gcc/am33_2.0-linux/4.9.0/include/stddef.h:#define __SIZE_TYPE__ long unsigned int
which does not match what the kernel expects. I see I have the same thing in
my locally built am33_2.0-linux-gcc-4.9.3.
I have just tried this again with a newly built am33_2.0-linux-gcc-5.2.1, and that
indeed avoids almost all warnings for the mn10300 kernel. I suspect this is
really a combination of two bugs that cancel each other out, but if you do the
same update on your system, you will get the results you want and will no longer
see the bogus warning.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/