> From: Seth Jennings [mailto:sjenning@xxxxxxxxxxxxxxxxxx] > Sent: Wednesday, May 15, 2013 2:10 PM > To: Konrad Rzeszutek Wilk > Cc: Dan Magenheimer; Andrew Morton; Greg Kroah-Hartman; Nitin Gupta; Minchan Kim; Robert Jennings; > Jenifer Hopper; Mel Gorman; Johannes Weiner; Rik van Riel; Larry Woodman; Benjamin Herrenschmidt; Dave > Hansen; Joe Perches; Joonsoo Kim; Cody P Schafer; Hugh Dickens; Paul Mackerras; linux-mm@xxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCHv11 3/4] zswap: add to mm/ > > On Wed, May 15, 2013 at 02:55:06PM -0400, Konrad Rzeszutek Wilk wrote: > > > Sorry, but I don't think that's appropriate for a patch in the MM subsystem. > > > > I am heading to the airport shortly so this email is a bit hastily typed. > > > > Perhaps a compromise can be reached where this code is merged as a driver > > not a core mm component. There is a high bar to be in the MM - it has to > > work with many many different configurations. > > > > And drivers don't have such a high bar. They just need to work on a specific > > issue and that is it. If zswap ended up in say, drivers/mm that would make > > it more palpable I think. > > > > Thoughts? > > zswap, the writeback code particularly, depends on a number of non-exported > kernel symbols, namely: > > swapcache_free > __swap_writepage > __add_to_swap_cache > swapcache_prepare > swapper_spaces > > So it can't currently be built as a module and I'm not sure what the MM > folks would think about exporting them and making them part of the KABI. It can be built as a module if writeback is disabled (or ifdef'd by a CONFIG_ZSWAP_WRITEBACK which depends on CONFIG_ZSWAP=y). The folks at LSFMM who were planning to use zswap will be turning off writeback anyway so an alternate is to pull writeback out of zswap completely for now, since you don't really have a good policy to manage it yet anyway. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
