[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <55FAB985.9060705@suse.cz>
Date: Thu, 17 Sep 2015 15:00:53 +0200
From: Vlastimil Babka <vbabka@...e.cz>
To: Vitaly Wool <vitalywool@...il.com>, ddstreet@...e.org,
akpm@...ux-foundation.org, minchan@...nel.org,
sergey.senozhatsky@...il.com
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org
Subject: Re: [PATCH 1/2] zbud: allow PAGE_SIZE allocations
On 09/16/2015 01:50 PM, Vitaly Wool wrote:
> For zram to be able to use zbud via the common zpool API,
> allocations of size PAGE_SIZE should be allowed by zpool.
> zbud uses the beginning of an allocated page for its internal
> structure but it is not a problem as long as we keep track of
> such special pages using a newly introduced page flag.
> To be able to keep track of zbud pages in any case, struct page's
> lru pointer will be used for zbud page lists instead of the one
> that used to be part of the aforementioned internal structure.
I don't know how zsmalloc handles uncompressible PAGE_SIZE allocations,
but I wouldn't expect it to be any more clever than this? So why
duplicate the functionality in zswap and zbud? This could be handled
e.g. at the zpool level? Or maybe just in zram, as IIRC in zswap
(frontswap) it's valid just to reject a page and it goes to physical swap.
--
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