On Wed, Nov 20, 2013 at 7:42 PM, Bob Liu <bob.liu@xxxxxxxxxx> wrote: > Hi Dan, > > On 11/21/2013 03:49 AM, Dan Streetman wrote: >> Currently, zswap is writeback cache; stored pages are not sent >> to swap disk, and when zswap wants to evict old pages it must >> first write them back to swap cache/disk manually. This avoids >> swap out disk I/O up front, but only moves that disk I/O to >> the writeback case (for pages that are evicted), and adds the >> overhead of having to uncompress the evicted pages, and adds the >> need for an additional free page (to store the uncompressed page) >> at a time of likely high memory pressure. Additionally, being >> writeback adds complexity to zswap by having to perform the >> writeback on page eviction. >> > > Good work! > >> This changes zswap to writethrough cache by enabling >> frontswap_writethrough() before registering, so that any >> successful page store will also be written to swap disk. All the >> writeback code is removed since it is no longer needed, and the >> only operation during a page eviction is now to remove the entry >> from the tree and free it. >> > > Could you do some testing using eg. SPECjbb? And compare the result with > original zswap. Sure, I have a small test program that I used for performance comparisions, which I'll send and include some results, and I'll also try to find a copy of SPECjbb to get results with. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>