Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1214764

Re: [PATCH v3] selftests/zram: Adding zram tests

From Shuah Khan <shuahkh@osg.samsung.com>
Newsgroups linux.kernel
Subject Re: [PATCH v3] selftests/zram: Adding zram tests
Date 2015-08-27 20:20 +0200
Message-ID <q29Wh-3Qj-11@gated-at.bofh.it> (permalink)
References <pYJbY-7oT-37@gated-at.bofh.it>
Organization Samsung Open Source Group

Show all headers | View raw


On 08/18/2015 01:01 AM, naresh.kamboju@linaro.org wrote:
> From: Naresh Kamboju <naresh.kamboju@linaro.org>
> 
> zram: Compressed RAM based block devices
> ----------------------------------------
> The zram module creates RAM based block devices named /dev/zram<id>
> (<id> = 0, 1, ...). Pages written to these disks are compressed and stored
> in memory itself. These disks allow very fast I/O and compression provides
> good amounts of memory savings. Some of the usecases include /tmp storage,
> use as swap disks, various caches under /var and maybe many more :)
> 
> Statistics for individual zram devices are exported through sysfs nodes at
> /sys/block/zram<id>/
> 
> This patch is to validate the zram functionality. Test interacts with block
> device /dev/zram<id> and sysfs nodes /sys/block/zram<id>/
> 

Hi Naresh,

This doesn't fail gracefully when a regular user runs it. Please add
a check for root and make the test fail gracefully when a non-root
user runs it.

Please refer to tools/testing/selftests/efivarfs/efivarfs.sh
for an example on how to check for root.

I was getting ready to apply this to linux-kselftest next for 4.3
and ran some sanity tests. If you get me new revision quickly
I can get it into 4.3-rc1.

thanks,
-- Shuah

-- 
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH v3] selftests/zram: Adding zram tests Shuah Khan <shuahkh@osg.samsung.com> - 2015-08-27 20:20 +0200
  Re: [PATCH v3] selftests/zram: Adding zram tests Shuah Khan <shuahkh@osg.samsung.com> - 2015-08-28 00:20 +0200
    Re: [PATCH v3] selftests/zram: Adding zram tests Naresh Kamboju <naresh.kamboju@linaro.org> - 2015-08-31 13:50 +0200

csiph-web