2010-06-06 22:35:52

by Michael Cree

[permalink] [raw]
Subject: 2.6.35-rc2 compilation error on Alpha

When compiling 2.6.35-rc2 for Alpha architecture the following compiler
error is seen:

CC arch/alpha/kernel/asm-offsets.s
In file included from include/linux/kernel.h:17,
from include/linux/sched.h:54,
from arch/alpha/kernel/asm-offsets.c:9:
include/linux/bitops.h: In function ?hweight_long?:
include/linux/bitops.h:49: error: implicit declaration of function
?__arch_hweight32?

Cheers
Michael.


2010-06-07 00:08:56

by Matt Turner

[permalink] [raw]
Subject: Re: 2.6.35-rc2 compilation error on Alpha

On Sun, Jun 6, 2010 at 6:35 PM, Michael Cree <[email protected]> wrote:
> When compiling 2.6.35-rc2 for Alpha architecture the following compiler
> error is seen:
>
> ?CC ? ? ?arch/alpha/kernel/asm-offsets.s
> In file included from include/linux/kernel.h:17,
> ? ? ? ? ? ? ? ? from include/linux/sched.h:54,
> ? ? ? ? ? ? ? ? from arch/alpha/kernel/asm-offsets.c:9:
> include/linux/bitops.h: In function ?hweight_long?:
> include/linux/bitops.h:49: error: implicit declaration of function
> ?__arch_hweight32?
>
> Cheers
> Michael.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>

Yeah, it was a stupid typo in
1527bc8b928dd1399c3d3467dd47d9ede210978a. I've fixed it and put the
patch in my tree for Linus.

Unfortunately, there's another compile error--PCI breakage. I'll do this too...

Thanks,
Matt

2010-06-07 01:13:18

by Michael Cree

[permalink] [raw]
Subject: Re: 2.6.35-rc2 compilation error on Alpha

On 07/06/10 12:08, Matt Turner wrote:
> On Sun, Jun 6, 2010 at 6:35 PM, Michael Cree<[email protected]> wrote:
>> When compiling 2.6.35-rc2 for Alpha architecture the following compiler
>> error is seen:
>>
>> CC arch/alpha/kernel/asm-offsets.s
>> In file included from include/linux/kernel.h:17,
>> from include/linux/sched.h:54,
>> from arch/alpha/kernel/asm-offsets.c:9:
>> include/linux/bitops.h: In function ?hweight_long?:
>> include/linux/bitops.h:49: error: implicit declaration of function
>> ?__arch_hweight32?
>
> Yeah, it was a stupid typo in
> 1527bc8b928dd1399c3d3467dd47d9ede210978a. I've fixed it and put the
> patch in my tree for Linus.

Ah, now the offending commit is pointed out to me I can see the trivial
fix. With that, as you say, compilation now bombs out in the pci-sysfs
stuff. Since you claim to have that in hand I'll await your solution.

Cheers
Michael.

2010-06-07 01:24:09

by Matt Turner

[permalink] [raw]
Subject: Re: 2.6.35-rc2 compilation error on Alpha

On Sun, Jun 6, 2010 at 9:13 PM, Michael Cree <[email protected]> wrote:
> On 07/06/10 12:08, Matt Turner wrote:
>>
>> On Sun, Jun 6, 2010 at 6:35 PM, Michael Cree<[email protected]> ?wrote:
>>>
>>> When compiling 2.6.35-rc2 for Alpha architecture the following compiler
>>> error is seen:
>>>
>>> ?CC ? ? ?arch/alpha/kernel/asm-offsets.s
>>> In file included from include/linux/kernel.h:17,
>>> ? ? ? ? ? ? ? ? from include/linux/sched.h:54,
>>> ? ? ? ? ? ? ? ? from arch/alpha/kernel/asm-offsets.c:9:
>>> include/linux/bitops.h: In function ?hweight_long?:
>>> include/linux/bitops.h:49: error: implicit declaration of function
>>> ?__arch_hweight32?
>>
>> Yeah, it was a stupid typo in
>> 1527bc8b928dd1399c3d3467dd47d9ede210978a. I've fixed it and put the
>> patch in my tree for Linus.
>
> Ah, now the offending commit is pointed out to me I can see the trivial fix.
> ?With that, as you say, compilation now bombs out in the pci-sysfs stuff.
> ?Since you claim to have that in hand I'll await your solution.
>
> Cheers
> Michael.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>

Fixes for both breaks are in:
git://git.kernel.org/pub/scm/linux/kernel/git/mattst88/alpha-2.6.git
for-linus

I know alpha hardware isn't easy to get your hands on to compile test
these changes, but guys, please just send linux-alpha@ the patch and
_I'll_ do it. Both these breaks were not even compile tested.

pci break: 2c3c8bea608866d8bd9dcf92657d57fdcac011c5
hweight break: 1527bc8b928dd1399c3d3467dd47d9ede210978a

Greg, I haven't checked, but if either of these commits are in .34 or
older, then my fixes need to go into -stable.

Thanks,
Matt

2010-06-07 03:07:42

by Greg KH

[permalink] [raw]
Subject: Re: 2.6.35-rc2 compilation error on Alpha

On Sun, Jun 06, 2010 at 09:24:03PM -0400, Matt Turner wrote:
> On Sun, Jun 6, 2010 at 9:13 PM, Michael Cree <[email protected]> wrote:
> > On 07/06/10 12:08, Matt Turner wrote:
> >>
> >> On Sun, Jun 6, 2010 at 6:35 PM, Michael Cree<[email protected]> ?wrote:
> >>>
> >>> When compiling 2.6.35-rc2 for Alpha architecture the following compiler
> >>> error is seen:
> >>>
> >>> ?CC ? ? ?arch/alpha/kernel/asm-offsets.s
> >>> In file included from include/linux/kernel.h:17,
> >>> ? ? ? ? ? ? ? ? from include/linux/sched.h:54,
> >>> ? ? ? ? ? ? ? ? from arch/alpha/kernel/asm-offsets.c:9:
> >>> include/linux/bitops.h: In function ?hweight_long?:
> >>> include/linux/bitops.h:49: error: implicit declaration of function
> >>> ?__arch_hweight32?
> >>
> >> Yeah, it was a stupid typo in
> >> 1527bc8b928dd1399c3d3467dd47d9ede210978a. I've fixed it and put the
> >> patch in my tree for Linus.
> >
> > Ah, now the offending commit is pointed out to me I can see the trivial fix.
> > ?With that, as you say, compilation now bombs out in the pci-sysfs stuff.
> > ?Since you claim to have that in hand I'll await your solution.
> >
> > Cheers
> > Michael.
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
> > the body of a message to [email protected]
> > More majordomo info at ?http://vger.kernel.org/majordomo-info.html
> >
>
> Fixes for both breaks are in:
> git://git.kernel.org/pub/scm/linux/kernel/git/mattst88/alpha-2.6.git
> for-linus
>
> I know alpha hardware isn't easy to get your hands on to compile test
> these changes, but guys, please just send linux-alpha@ the patch and
> _I'll_ do it. Both these breaks were not even compile tested.
>
> pci break: 2c3c8bea608866d8bd9dcf92657d57fdcac011c5
> hweight break: 1527bc8b928dd1399c3d3467dd47d9ede210978a
>
> Greg, I haven't checked, but if either of these commits are in .34 or
> older, then my fixes need to go into -stable.

Could you check please?

thanks,

greg k-h

2010-06-08 17:25:14

by Chris Wright

[permalink] [raw]
Subject: Re: 2.6.35-rc2 compilation error on Alpha

* Matt Turner ([email protected]) wrote:
> Fixes for both breaks are in:
> git://git.kernel.org/pub/scm/linux/kernel/git/mattst88/alpha-2.6.git
> for-linus

Your 83926ce2dd37e4337ecf53b1e435d08e70b5871b will work, but you can
also simply drop the filp from pci_mmap_resource(). That's an internal
helper (local to alpha/kernel/pci-sysfs.c) and doesn't require the filp.
Sorry, that was my fault, and should be localized to 2.6.35-rc.

> I know alpha hardware isn't easy to get your hands on to compile test
> these changes, but guys, please just send linux-alpha@ the patch and
> _I'll_ do it. Both these breaks were not even compile tested.
>
> pci break: 2c3c8bea608866d8bd9dcf92657d57fdcac011c5

$ git describe --contains 2c3c8bea608866d8bd9dcf92657d57fdcac011c5
v2.6.35-rc1~459^2~9

> hweight break: 1527bc8b928dd1399c3d3467dd47d9ede210978a
>
> Greg, I haven't checked, but if either of these commits are in .34 or
> older, then my fixes need to go into -stable.