Messages in this thread | | | Date | Fri, 20 Apr 2012 14:06:41 -0500 | From | Seth Jennings <> | Subject | Re: linux-next: Tree for Apr 19 (zcache) |
| |
On 04/20/2012 11:37 AM, Konrad Rzeszutek Wilk wrote: > On Thu, Apr 19, 2012 at 11:36:26PM -0500, Seth Jennings wrote: >> One fix is this: >> >> diff --git a/drivers/staging/zcache/Kconfig b/drivers/staging/zcache/Kconfig >> index 3ed2c8f..7048e01 100644 >> --- a/drivers/staging/zcache/Kconfig >> +++ b/drivers/staging/zcache/Kconfig >> @@ -2,7 +2,7 @@ config ZCACHE >> bool "Dynamic compression of swap pages and clean pagecache pages" >> # X86 dependency is because zsmalloc uses non-portable pte/tlb >> # functions >> - depends on (CLEANCACHE || FRONTSWAP) && CRYPTO && X86 >> + depends on (CLEANCACHE || FRONTSWAP) && CRYPTO=y && X86 >> select ZSMALLOC >> select CRYPTO_LZO >> default n >> >> I think this is the best way since ZCACHE is a bool and CRYPTO is the only >> one of the dependencies that is a tristate. This forces both ZSMALLOC and >> CRYPTO_LZO to be builtin if ZCACHE is selected. >> >> Any other suggestions? > > Could you do: > > default y if (CRYPTO=y || ZSMALLOC=m) > default m if (CRYPTO=m || ZSMALLOC=y)
ZCACHE is a bool and can't be built as a module.
The requirement is the if ZCACHE=y, then ZSMALLOC and CRYPTO must also =y.
Thanks, Seth
| |