Re: [PATCH] mm: cleanup swap unused warning

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.

---

• References:
  • [PATCH] mm: cleanup swap unused warning
    • From: Con Kolivas <[email protected]>
  • Re: [PATCH] mm: cleanup swap unused warning
    • From: Christoph Lameter <[email protected]>
  • Re: [PATCH] mm: cleanup swap unused warning
    • From: Con Kolivas <[email protected]>

• Prev by Date: Re: PATCH: Fix broken PIO with libata
• Next by Date: Re: [PATCH] mm: cleanup swap unused warning
• Previous by thread: [PATCH][respin] mm: cleanup swap unused warning
• Next by thread: Re: [PATCH] mm: cleanup swap unused warning
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]