Re: [UnifiedV4 00/16] The Unified slab allocator (V4)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Mel Gorman
Date: Tuesday, October 12, 2010 - 11:25 am

On Wed, Oct 06, 2010 at 11:01:35AM +0300, Pekka Enberg wrote:

I have not come even close to reviewing this yet but I made a start on
putting it through a series of tests. It fails to build on ppc64

  CC      mm/slub.o
mm/slub.c:1477: warning: 'drain_alien_caches' declared inline after being called
mm/slub.c:1477: warning: previous declaration of 'drain_alien_caches' was here
mm/slub.c: In function `alloc_shared_caches':
mm/slub.c:1748: error: `cpu_info' undeclared (first use in this function)
mm/slub.c:1748: error: (Each undeclared identifier is reported only once
mm/slub.c:1748: error: for each function it appears in.)
mm/slub.c:1748: warning: type defaults to `int' in declaration of `type name'
mm/slub.c:1748: warning: type defaults to `int' in declaration of `type name'
mm/slub.c:1748: warning: type defaults to `int' in declaration of `type name'
mm/slub.c:1748: warning: type defaults to `int' in declaration of `type name'
mm/slub.c:1748: error: invalid type argument of `unary *'
make[1]: *** [mm/slub.o] Error 1
make: *** [mm] Error 2

I didn't look closely yet but cpu_info is an arch-specific variable.
Checking to see if there is a known fix yet before setting aside time to
dig deeper.

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Tue Oct 5, 11:57 am)
[UnifiedV4 06/16] slub: Drop allocator announcement, Christoph Lameter, (Tue Oct 5, 11:57 am)
[UnifiedV4 08/16] slub: Get rid of page lock and rely on p ..., Christoph Lameter, (Tue Oct 5, 11:57 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Pekka Enberg, (Wed Oct 6, 1:01 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Richard Kennedy, (Wed Oct 6, 4:03 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Pekka Enberg, (Wed Oct 6, 4:19 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Wu Fengguang, (Wed Oct 6, 5:37 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Richard Kennedy, (Wed Oct 6, 8:46 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Wed Oct 6, 8:56 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Wed Oct 6, 9:00 am)
[UnifiedV4 slabinfo 2/2] slub: update slabinfo.c for queuing, Christoph Lameter, (Wed Oct 6, 9:21 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Wed Oct 6, 1:56 pm)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Mel Gorman, (Tue Oct 12, 11:25 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Pekka Enberg, (Wed Oct 13, 12:16 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Wed Oct 13, 9:10 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Mon Oct 18, 11:00 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Mon Oct 18, 11:13 am)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), David Rientjes, (Tue Oct 19, 1:39 pm)
Re: [UnifiedV4 00/16] The Unified slab allocator (V4), Christoph Lameter, (Wed Oct 20, 6:47 am)