Re: [PATCH 5/7] zram: use crypto api to check alg availability
From: Sergey Senozhatsky
Date: Fri May 27 2016 - 03:51:08 EST
On (05/27/16 13:43), Minchan Kim wrote:
[..]
> > modprobe zram
> > cat /proc/crypto | grep -i lz4
> > modprobe lz4
> > cat /proc/crypto | grep -i lz4
> > name : lz4
> > driver : lz4-generic
> > module : lz4
> >
> > So the user can't tell exactly if the lz4 is really supported
> > from /proc/crypto output, unless someone or something has loaded
> > it.
> >
> > This patch also adds crypto_has_comp() to zcomp_available_show().
>
> crypto_has_comp works regardless of that whether module is loading or not?
> IOW, currently, if lz4 modules is not loading, but crypto_has_comp return
> true about lz4 module.
> Right?
correct. crypto_has_comp() regardless the module being loaded.
# modprobe zram
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
# echo lzo > /sys/block/zram0/comp_algorithm
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
name : lzo
driver : lzo-generic
module : lzo
# echo lz4 > /sys/block/zram0/comp_algorithm
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
name : lz4
driver : lz4-generic
module : lz4
name : lzo
driver : lzo-generic
module : lzo
# echo deflate > /sys/block/zram0/comp_algorithm
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
name : deflate
driver : deflate-generic
module : deflate
name : lz4
driver : lz4-generic
module : lz4
name : lzo
driver : lzo-generic
module : lzo
what is does, tho, it modprobs() the module upon the first
request:
crypto_has_comp(...)
crypto_has_alg()
crypto_alg_mod_lookup()
crypto_larval_lookup()
request_module("crypto-%s", name)
__request_module()
call_modprobe()
and this is when /proc/crypto is getting updated. otherwise user has
no information (well, unless modules were loaded by something/someome
else, or crypto compressors were built-in into the kernel). I'm not
aware of any other way to achieve this functionality for zram.
> > We store all the compression algorithms names in zcomp's `backends'
> > array, regardless the CONFIG_CRYPTO_FOO configuration, but show
>
> Then, you mean we should add new string into backend array whenever
> adding new crypto compatible compression algorithm?
yes. which looks quite trivial: adding or removing a string to/from
the array.
> > cat /proc/crypto | grep -i lz4
> >
> > cat /sys/block/zram0/comp_algorithm
> > [lzo] lz4 deflate lz4hc 842
>
> So, when lzo module is loading?
when we execute crypto_has_comp("lzo") for the first time.
that's why doing just
# modprobe zram
will not cause /proc/crypto update
# modprobe zram
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
crypto_has_comp() updates it -- when we read or write
from/to comp_algorithm:
# cat /sys/block/zram0/comp_algorithm
[lzo] lz4 deflate lz4hc 842
# cat /proc/crypto | egrep -e "lzo|lz4|deflate"
name : lzo
driver : lzo-generic
module : lzo
but I didn't want zram to depend on this, or to depend on
/proc/crypto content; that's why I did it the way it is.
-ss