Re: [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2

Trond Myklebust <[email protected]> wrote:

> -	if (PagePrivate(page) && !try_to_release_page(page, 0))
> +	if (PagePrivate(page) && !try_to_release_page(page, GFP_KERNEL))

This can't be the right way to fix things.  try_to_release_page() may fail
whatever GFP flags you give it.  If the page *must* be invalidated at this
point then you _must_ call the invalidatepage() op, not the releasepage() op.

What is currently there allows an instikill to take place on a page at this
point, but defers the kill on pages that are busy until all the pages have
been scanned once, thus giving I/O time to happen on busy pages whilst reaping
pages that can be made immediately available.

David
-
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/

---

• Follow-Ups:
  • Re: [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
    • From: David Howells <[email protected]>
  • Re: [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
    • From: Trond Myklebust <[email protected]>

• References:
  • [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
    • From: Trond Myklebust <[email protected]>

• Prev by Date: Re: [PATCH] SCSI osst: add error handling to module init, sysfs
• Next by Date: Re: [PATCH 1/4] LOG2: Implement a general integer log2 facility in the kernel [try #4]
• Previous by thread: Re: [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
• Next by thread: Re: [PATCH] VM: Fix the gfp_mask in invalidate_complete_page2
• 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]