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.