Re: [PATCH 3/5] Swap Migration V5: migrate_pages() function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Monday 31 October 2005 21:12, Christoph Lameter wrote:
> Page migration support in vmscan.c

This has no #ifdef SWAP:

> + if (PageSwapCache(page)) {
> +  swp_entry_t swap = { .val = page_private(page) };
> +  add_to_swapped_list(swap.val);
> +  __delete_from_swap_cache(page);
> +  write_unlock_irq(&mapping->tree_lock);
> +  swap_free(swap);
> +  __put_page(page); /* The pagecache ref */
> +  return 1;
> + }

But what you removed did:

> -#ifdef CONFIG_SWAP
> -  if (PageSwapCache(page)) {
> -   swp_entry_t swap = { .val = page_private(page) };
> -   add_to_swapped_list(swap.val);
> -   __delete_from_swap_cache(page);
> -   write_unlock_irq(&mapping->tree_lock);
> -   swap_free(swap);
> -   __put_page(page); /* The pagecache ref */
> -   goto free_it;
> -  }
> -#endif /* CONFIG_SWAP */

What happens if you build without swap?

Rob
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[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]
  Powered by Linux