On (05/27/15 13:23), a...@linux-foundation.org wrote:
> This patch should soon appear at
>
> http://ozlabs.org/~akpm/mmots/broken-out/zram-check-comp-algorithm-availability-earlier-v2.patch
> and later at
>
> http://ozlabs.org/~akpm/mmotm/broken-out/zram-check-comp-algorithm-availability-earlier-v2.patch
[..]
Sir,
just for note, v2 has an updated commit message (no dmesg output)
8<---
Improvement idea by Marcin Jabrzyk.
comp_algorithm_store() silently accepts any supplied algorithm
name, because zram performs algorithm availability check later,
during the device configuration phase in disksize_store() and
emits the following error:
"zram: Cannot initialise %s compressing backend"
this error line is somewhat generic and, besides, can indicate
a failed attempt to allocate compression backend's working
buffers.
add algorithm availability check to comp_algorithm_store():
echo lzz > /sys/block/zram0/comp_algorithm
-bash: echo: write error: Invalid argument
8<---
thanks,
-ss
> ------------------------------------------------------
> From: Sergey Senozhatsky <sergey.senozhat...@gmail.com>
> Subject: zram-check-comp-algorithm-availability-earlier-v2
>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhat...@gmail.com>
> Cc: Marcin Jabrzyk <m.jabr...@samsung.com>
> Cc: Minchan Kim <minc...@kernel.org>
> Cc: Nitin Gupta <ngu...@vflare.org>
> Signed-off-by: Andrew Morton <a...@linux-foundation.org>
> ---
>
> drivers/block/zram/zcomp.c | 10 +++++-----
> drivers/block/zram/zram_drv.c | 5 +----
> 2 files changed, 6 insertions(+), 9 deletions(-)
>
> diff -puN
> drivers/block/zram/zcomp.c~zram-check-comp-algorithm-availability-earlier-v2
> drivers/block/zram/zcomp.c
> ---
> a/drivers/block/zram/zcomp.c~zram-check-comp-algorithm-availability-earlier-v2
> +++ a/drivers/block/zram/zcomp.c
> @@ -286,6 +286,11 @@ ssize_t zcomp_available_show(const char
> return sz;
> }
>
> +bool zcomp_available_algorithm(const char *comp)
> +{
> + return find_backend(comp) != NULL;
> +}
> +
> bool zcomp_set_max_streams(struct zcomp *comp, int num_strm)
> {
> return comp->set_max_streams(comp, num_strm);
> @@ -320,11 +325,6 @@ void zcomp_destroy(struct zcomp *comp)
> kfree(comp);
> }
>
> -bool zcomp_available_algorithm(const char *comp)
> -{
> - return find_backend(comp) != NULL;
> -}
> -
> /*
> * search available compressors for requested algorithm.
> * allocate new zcomp and initialize it. return compressing
> diff -puN
> drivers/block/zram/zram_drv.c~zram-check-comp-algorithm-availability-earlier-v2
> drivers/block/zram/zram_drv.c
> ---
> a/drivers/block/zram/zram_drv.c~zram-check-comp-algorithm-availability-earlier-v2
> +++ a/drivers/block/zram/zram_drv.c
> @@ -378,11 +378,8 @@ static ssize_t comp_algorithm_store(stru
> if (sz > 0 && zram->compressor[sz - 1] == '\n')
> zram->compressor[sz - 1] = 0x00;
>
> - if (!zcomp_available_algorithm(zram->compressor)) {
> - pr_err("Error: unavailable compression algorithm: %s\n",
> - zram->compressor);
> + if (!zcomp_available_algorithm(zram->compressor))
> len = -EINVAL;
> - }
>
> up_write(&zram->init_lock);
> return len;
> _
>
> Patches currently in -mm which might be from sergey.senozhat...@gmail.com are
>
> zram-add-compact-sysfs-entry-to-documentation.patch
> zram-cosmetic-zram_attr_ro-code-formatting-tweak.patch
> zram-use-idr-instead-of-zram_devices-array.patch
> zram-reorganize-code-layout.patch
> zram-remove-max_num_devices-limitation.patch
> zram-report-every-added-and-removed-device.patch
> zram-trivial-correct-flag-operations-comment.patch
> zram-return-zram-device_id-from-zram_add.patch
> zram-close-race-by-open-overriding.patch
> zram-add-dynamic-device-add-remove-functionality.patch
> zram-cosmetic-zram_bvec_write-cleanup.patch
> zram-cut-trailing-newline-in-algorithm-name.patch
> zram-check-comp-algorithm-availability-earlier.patch
> zram-check-comp-algorithm-availability-earlier-v2.patch
> linux-next.patch
>
> --
> To unsubscribe from this list: send the line "unsubscribe mm-commits" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
