Messages in this thread | | | Date | Wed, 28 Apr 2010 13:45:07 -0400 | From | Mathieu Desnoyers <> | Subject | Re: [171/197] module: fix __module_ref_addr() |
| |
* Greg KH (greg@kroah.com) wrote: > On Wed, Apr 28, 2010 at 07:24:13PM +0200, Tejun Heo wrote: > > On 04/28/2010 06:55 PM, Jiri Benc wrote: > > > On Wed, 28 Apr 2010 12:20:56 -0400, Mathieu Desnoyers wrote: > > >> Can you try reverting commit c8d52465f95c4187871f8e65666c07806ca06d41 and see if > > >> it helps ? > > > > > > It doesn't. The produced code is identical. > > > > > >> If you have other compiler versions handy, that would also be helpful to see if > > >> the problem is specific to the gcc version you are using. > > > > > > Tried 4.5.0, the same problem (at least looking at the produced assembler > > > code, I haven't booted the kernel, but it looks very similar to 4.3.3). > > > > I wrote on the bugzilla but this is not a compiler bug but the -stable > > patch shouldn't have been applied only to 2.6.33. Not 2.6.32. This > > is because till 2.6.32, ia64 hadn't been converted to dynamic percpu > > allocator, so its static and dynamic percpu areas were separate and > > the per_cpu_ptr() wouldn't do the offsetting the module code expects > > there. So, please revert the patch from 2.6.32. > > Ah crap, I missed that. I'll go revert it from .32 now, sorry about > that. > > greg k-h
My bad, I indicated that it should be applied to 2.6.29.x through 2.6.33.x in the changelog.
Sorry,
Mathieu
-- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com
| |