Re: [PATCH] mm: cleanup swap unused warning

[Christoph Lameter]

On Tue, 16 May 2006, Con Kolivas wrote:

> On Thursday 11 May 2006 09:04, Christoph Lameter wrote:
> > On Wed, 10 May 2006, Con Kolivas wrote:
> > > Are there any users of swp_entry_t when CONFIG_SWAP is not defined?
> >
> > Yes, a migration entry is a form of swap entry.
> 
> mm/vmscan.c: In function ÿÿremove_mappingÿÿ:
> mm/vmscan.c:387: warning: unused variable ÿÿswapÿÿ
> 
> Ok so if we fix it by making swp_entry_t __attribute__((__unused__) we break 
> swap migration code?

This will generally break page migration in mm.
 
> If we make swap_free() an empty static inline function then gcc compiles in 
> the variable needlessly and we won't know it.

PageSwapCache() returns false if CONFIG_SWAP is not set and therefore no 
code is generated.