linux-kernel - Re: [PATCH v4 0/4] ublk: add zoned storage support

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-next>] [day] [month] [year] [list]

Date:   Fri, 30 Jun 2023 16:03:37 +0530
From:   aravind.ramesh@...nsource.wdc.com
To:     Hans Holmberg <Hans.Holmberg@....com>,
        Jens Axboe <axboe@...nel.dk>,
        "open list:BLOCK LAYER" <linux-block@...r.kernel.org>,
        hch@...radead.org,
        Matias Bjørling <Matias.Bjorling@....com>,
        Andreas Hindborg <a.hindborg@...sung.com>,
        open list <linux-kernel@...r.kernel.org>,
        Damien Le Moal <dlemoal@...nel.org>, gost.dev@...sung.com,
        Minwoo Im <minwoo.im.dev@...il.com>,
        Ming Lei <ming.lei@...hat.com>
Subject: Re: [PATCH v4 0/4] ublk: add zoned storage support

> On 29/06/23, 12:37 AM, "Andreas Hindborg" <nmi@...aspace.dk
> <mailto:nmi@...aspace.dk>> wrote:
> 
> 
> From: Andreas Hindborg <a.hindborg@...sung.com 
> <mailto:a.hindborg@...sung.com>>
> 
> 
> Hi All,
> 
> 
> This patch set adds zoned storage support to `ublk`. The first two 
> patches does
> some house cleaning in preparation for the last two patches. The third 
> patch
> adds support for report_zones and the following operations:
> 

Just to clarify, we do need you ublk user space patches
to create a ublk device node (with these patches in kernel), right ?

> 
> - REQ_OP_ZONE_OPEN
> - REQ_OP_ZONE_CLOSE
> - REQ_OP_ZONE_FINISH
> - REQ_OP_ZONE_RES

REQ_OP_ZONE_RESET

> 
> 
> The last patch adds support for REQ_OP_ZONE_APPEND.
> 
> 
> v3 [2] -> v4 changes:
> - Split up v3 patches
> - Add zone append support
> - Change order of variables in `ublk_report_zones`
> 
> 
> Read/write and zone operations are tested with zenfs [3].
> 
> 
> The zone append path is tested with fio -> zonefs -> ublk -> null_blk.
> 
> 
> The implementation of zone append requires ublk user copy feature, and 
> therefore
> the series is based on branch for-next (6afa337a3789) of [4].
> 
> 
> [1]
> https://github.com/metaspace/ubdsrv/commit/7de0d901c329fde7dc5a2e998952dd88bf5e668b
> <https://github.com/metaspace/ubdsrv/commit/7de0d901c329fde7dc5a2e998952dd88bf5e668b>
> [2]
> https://lore.kernel.org/linux-block/20230316145539.300523-1-nmi@metaspace.dk
> <mailto:20230316145539.300523-1-nmi@...aspace.dk>/
> [3] https://github.com/westerndigitalcorporation/zenfs
> <https://github.com/westerndigitalcorporation/zenfs>
> [4] https://git.kernel.dk/linux.git <https://git.kernel.dk/linux.git>
> 
> 
> Andreas Hindborg (4):
> ublk: change ublk IO command defines to enum
> ublk: move types to shared header file
> ublk: enable zoned storage support
> ublk: add zone append
> 
> 
> MAINTAINERS | 2 +
> drivers/block/Kconfig | 4 +
> drivers/block/Makefile | 4 +-
> drivers/block/ublk_drv-zoned.c | 155 +++++++++++++++++++++++++++++++++
> drivers/block/ublk_drv.c | 150 +++++++++++++++++++------------
> drivers/block/ublk_drv.h | 71 +++++++++++++++
> include/uapi/linux/ublk_cmd.h | 38 ++++++--
> 7 files changed, 363 insertions(+), 61 deletions(-)
> create mode 100644 drivers/block/ublk_drv-zoned.c
> create mode 100644 drivers/block/ublk_drv.h
> 
> 
> 
> 
> base-commit: 3261ea42710e9665c9151006049411bd23b5411f

Regards,
Aravind