From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: systemd-devel-bounces@lists.freedesktop.org Date: Tue, 24 Jun 2014 07:49:58 +0000 From: Minchan Kim To: "Alexander E. Patrakov" Message-ID: <20140624074958.GB11030@gmail.com> References: <53953D5C.1080004@gmail.com> <5395BE60.70309@gmail.com> <5395FF3A.9050407@gmail.com> <20140624062209.GA10661@gmail.com> <53A91C0A.2020807@gmail.com> <20140624065816.GA11030@gmail.com> MIME-Version: 1.0 In-Reply-To: <20140624065816.GA11030@gmail.com> Cc: util-linux@vger.kernel.org, Kai Krakow , systemd-devel@lists.freedesktop.org, kay@vrfy.org, Nitin Gupta Subject: Re: [systemd-devel] systemd-213: regression with zram List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Errors-To: systemd-devel-bounces@lists.freedesktop.org Sender: "systemd-devel" List-ID: On Tue, Jun 24, 2014 at 07:21:12AM +0000, Minchan Kim wrote: > On Tue, Jun 24, 2014 at 12:34:50PM +0600, Alexander E. Patrakov wrote: > > 24.06.2014 12:22, Minchan Kim wrote: > > > > > >Hello, > > > > > >Sorry for the late response. > > > > Better late than never. > > > > >If I parse your problem correctly, you meant that changing disksize is > > >failed while someone opens /dev/zram0? > > > > > >I tried test which opens "/dev/zram0" with O_RDWR and sleep forever > > >and then echo $((512<<20)) > /sys/block/zram0/disksize is successful. > > >IOW, it's okay with me. > > > > No, this is only a part of the problem. A valid test would be: > > > > 0. Reset the unused zram device. > > 1. Use a program that opens /dev/zram0 with O_RDWR and sleeps until killed. > > 2. While that program sleeps, echo the correct value to > > /sys/block/zram0/disksize. > > 3. Verify (e.g. in /proc/partitions) that the disk size is applied > > correctly. It is. > > 4. While that program still sleeps, attempt to mkswap /dev/zram0. > > This fails: > > > > mkswap: error: swap area needs to be at least 40 KiB > > > > There is nothing in dmesg. > > Thanks. I could reproduce. > When I read strace result, fstat access closed fd which seem to be > a culprit. > > Cced util-linux. When I read disk-utils/mkswap.c, it's not a problem of mkswap. The problem is ioctl(fd, BLKGETSIZE64, bytes) is not work in your scenario. IOW, kernel fault. Hmm,, will investgate it. Thanks for the report! _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel