Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: 8o4X2MhmNedNwatHp0CQBp96N9BP5anpSoIb71Y+2BeRy/Jfj3e2Q3XG/uQeFv88Jx4YcJupKd
 oGpM9x0IZ+ClTcDrF/NehHbrnoEsxAm/yHPykt0VfQ+b1oIap2JfSNJvH2sGNZBa6HHVHuuJJb
 Z5P/jPNjkzWs0Jxl+D7fiLrtNXl9L1KhBuZeR2gNOpwAOVPGjgp55cGo2CGsLpFIvmUdPwrWSL
 4KINk1ixctJ/LzK+peAYFE9bRQHP1QBoYLp/qUBHDixx8LBP80q3tOY7YmF2v9b/PBIAVAaWo3
 +Hc=
From:   Johannes Thumshirn <Johannes.Thumshirn@wdc.com>
To:     Niklas Cassel <nks@flawful.org>, Jens Axboe <axboe@kernel.dk>
CC:     Damien Le Moal <Damien.LeMoal@wdc.com>,
        "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Niklas Cassel <Niklas.Cassel@wdc.com>
Subject: Re: [PATCH v3] null_blk: add support for max open/active zone limit
 for zoned devices
Thread-Topic: [PATCH v3] null_blk: add support for max open/active zone limit
 for zoned devices
Thread-Index: AQHWfSmaV4Z1Tg4VL0+Dee3d19pWQg==
Date:   Wed, 23 Sep 2020 07:46:48 +0000
Message-ID: <SN4PR0401MB359871B467B7F5FDB3115ADE9B380@SN4PR0401MB3598.namprd04.prod.outlook.com>
References: <20200828105400.80893-1-niklas.cassel@wdc.com>
 <20200907081824.GA260080@localhost.localdomain>
 <20200917075647.GA20845@flawful.org>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: d1cf75d3-ed2b-465b-bdf7-08d85f94d3a5
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Sep 2020 07:46:48.2439
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: L1Lqjx6gyJEvJbMW3XPFygNlJeZ9N9GYIzqZBXJWWRBxZ9HXNzoMiw7gTlbSqiX77VOncRan7agsnLIe3RBH3wl2dwiVRJjf/C9d4hrccaQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR04MB5245
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 17/09/2020 09:57, Niklas Cassel wrote:=0A=
> On Mon, Sep 07, 2020 at 08:18:26AM +0000, Niklas Cassel wrote:=0A=
>> On Fri, Aug 28, 2020 at 12:54:00PM +0200, Niklas Cassel wrote:=0A=
>>> Add support for user space to set a max open zone and a max active zone=
=0A=
>>> limit via configfs. By default, the default values are 0 =3D=3D no limi=
t.=0A=
>>>=0A=
>>> Call the block layer API functions used for exposing the configured=0A=
>>> limits to sysfs.=0A=
>>>=0A=
>>> Add accounting in null_blk_zoned so that these new limits are respected=
.=0A=
>>> Performing an operation that would exceed these limits results in a=0A=
>>> standard I/O error.=0A=
>>>=0A=
>>> A max open zone limit exists in the ZBC standard.=0A=
>>> While null_blk_zoned is used to test the Zoned Block Device model in=0A=
>>> Linux, when it comes to differences between ZBC and ZNS, null_blk_zoned=
=0A=
>>> mostly follows ZBC.=0A=
>>>=0A=
>>> Therefore, implement the manage open zone resources function from ZBC,=
=0A=
>>> but additionally add support for max active zones.=0A=
>>> This enables user space not only to test against a device with an open=
=0A=
>>> zone limit, but also to test against a device with an active zone limit=
.=0A=
>>>=0A=
>>> Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>=0A=
>>> Reviewed-by: Damien Le Moal <damien.lemoal@wdc.com>=0A=
>>> ---=0A=
>>> Changes since v2:=0A=
>>> -Picked up Damien's Reviewed-by tag.=0A=
>>> -Fixed a typo in the commit message.=0A=
>>> -Renamed null_manage_zone_resources() to null_has_zone_resources().=0A=
>>>=0A=
>>>  drivers/block/null_blk.h       |   5 +=0A=
>>>  drivers/block/null_blk_main.c  |  16 +-=0A=
>>>  drivers/block/null_blk_zoned.c | 319 +++++++++++++++++++++++++++------=
=0A=
>>>  3 files changed, 282 insertions(+), 58 deletions(-)=0A=
>>=0A=
>> Hello Jens,=0A=
>>=0A=
>> A gentle ping on this.=0A=
>>=0A=
>> As far as I can tell, there are no outstanding review comments.=0A=
> =0A=
> =0A=
> Hello Jens,=0A=
> =0A=
> Pinging you from another address, in case my corporate email is getting=
=0A=
> stuck in your spam filter.=0A=
> =0A=
> Kind regards,=0A=
> Niklas=0A=
> =0A=
=0A=
=0A=
Jens,=0A=
=0A=
Any chance we can get this queued up for 5.10? This is really helpful for e=
.g. =0A=
the zonefs test suite or xfstests when btrfs HMZONED support lands.=0A=
=0A=
Thanks,=0A=
	Johannes=0A=