linux-kernel - Re: [PATCH v3 2/2] zonefs: Add documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <BYAPR04MB58167BE64D952F385E3F7146E7280@BYAPR04MB5816.namprd04.prod.outlook.com>
Date:   Wed, 25 Dec 2019 06:40:29 +0000
From:   Damien Le Moal <Damien.LeMoal@....com>
To:     Randy Dunlap <rdunlap@...radead.org>,
        "linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
        "linux-xfs@...r.kernel.org" <linux-xfs@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>
CC:     Johannes Thumshirn <jth@...nel.org>,
        Naohiro Aota <Naohiro.Aota@....com>,
        "Darrick J . Wong" <darrick.wong@...cle.com>,
        Hannes Reinecke <hare@...e.de>
Subject: Re: [PATCH v3 2/2] zonefs: Add documentation

On 2019/12/25 10:33, Randy Dunlap wrote:
[...]
>> +For sequential write zone files, the file size changes as data is appended at
>> +the end of the file, similarly to any regular file system.
>> +
>> +# dd if=/dev/zero of=/mnt/seq/0 bs=4096 count=1 conv=notrunc oflag=direct
>> +1+0 records in
>> +1+0 records out
>> +4096 bytes (4.1 kB, 4.0 KiB) copied, 1.05112 s, 3.9 kB/s
> 
> why so slow?
> 
>> +
>> +# ls -l /mnt/seq/0
>> +-rw-r----- 1 root root 4096 Nov 25 13:23 /mnt/sdh/seq/0
> 
> I don't understand the "sdh/" here. Please explain for me (not necessarily
> in the doc file).

The drive I used for generating the example was /dev/sdh and it was in
fact mounted under /mnt/sdh/ for the test, but I removed "sdh" from the
pasted commands and results to make things simpler. I forgot to remove
the drive name on this line. Fixed now.

Thank you for the review.

-- 
Damien Le Moal
Western Digital Research