[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4BD16D09.2030803@redhat.com>
Date: Fri, 23 Apr 2010 12:48:57 +0300
From: Avi Kivity <avi@...hat.com>
To: Dan Magenheimer <dan.magenheimer@...cle.com>
CC: linux-kernel@...r.kernel.org, linux-mm@...ck.org, jeremy@...p.org,
hugh.dickins@...cali.co.uk, ngupta@...are.org, JBeulich@...ell.com,
chris.mason@...cle.com, kurt.hackel@...cle.com,
dave.mccracken@...cle.com, npiggin@...e.de,
akpm@...ux-foundation.org, riel@...hat.com
Subject: Re: Frontswap [PATCH 0/4] (was Transcendent Memory): overview
On 04/22/2010 11:15 PM, Dan Magenheimer wrote:
>>
>> Much easier to simulate an asynchronous API with a synchronous backend.
>>
> Indeed. But an asynchronous API is not appropriate for frontswap
> (or cleancache). The reason the hooks are so simple is because they
> are assumed to be synchronous so that the page can be immediately
> freed/reused.
>
Swapping is inherently asynchronous, so we'll have to wait for that to
complete anyway (as frontswap does not guarantee swap-in will succeed).
I don't doubt it makes things simpler, but also less flexible and useful.
Something else that bothers me is the double swapping. Sure we're
making swapin faster, but we we're still loading the io subsystem with
writes. Much better to make swap-to-ram authoritative (and have the
hypervisor swap it to disk if it needs the memory).
>> Well, copying memory so you can use a zero-copy dma engine is
>> counterproductive.
>>
> Yes, but for something like an SSD where copying can be used to
> build up a full 64K write, the cost of copying memory may not be
> counterproductive.
>
I don't understand. Please clarify.
--
Do not meddle in the internals of kernels, for they are subtle and quick to panic.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists