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: NvIr1/MdJHZUKJH/dD1ajw/qa0Gl8cfaPqE5brTYwsVFRO9gdOHit/p7w7AUknKymGiiEmRj5Q
 40hFUskRQRp4hP/5RZu0tR8z8Qt/hranp3OVqkoQ1kd06epNoOYiSa7My8SRlnOO9xHdax4lWV
 RBzJYo60P1NdzrBO2S4wFtSrO+qm0X4ADDQEhKQZK6vYNzCADdgL8ad173CwQsfQgu/NgKrdkr
 4sRaybBv4Iq2m2LbNfm3XnxAq8e3vGEKql1VjzzXxAfPxeSFToZmUa6sncXyAUkgyTPDME1eU4
 ygI=
From:   Niklas Cassel <Niklas.Cassel@wdc.com>
To:     Jens Axboe <axboe@kernel.dk>
CC:     Jens Axboe <axboe@fb.com>,
        "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 0/2] allow blk-zoned ioctls without CAP_SYS_ADMIN
Thread-Topic: [PATCH v3 0/2] allow blk-zoned ioctls without CAP_SYS_ADMIN
Thread-Index: AQHXYRgPtP8+uBs1gU68gsGoV+YyFqspGeKA
Date:   Mon, 28 Jun 2021 07:20:15 +0000
Message-ID: <YNl4LXUKOftl15M9@x1-carbon.lan>
References: <20210614122303.154378-1-Niklas.Cassel@wdc.com>
In-Reply-To: <20210614122303.154378-1-Niklas.Cassel@wdc.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?PZ/kCvBdi5puj1ntmmKMeDjAuWPC4Sj7XZ0INC2843sXoSXayuGuU8szFIIG?=
 =?us-ascii?Q?RVpCuN6KM5fGrLUezcCSwoC5407lZoP9htLjP0uvP2Y0XPTMBuDVljZryKKt?=
 =?us-ascii?Q?Jk2g11P5Oy8+/MjU4xOq3wEZCeQa6DpLXBDBj5xcnF1pF//nIBvf3dCRy9LX?=
 =?us-ascii?Q?DnKEIpy2woLkPeDv/HZ2tytd9rY0sM3NpdolxdEn4O0fga3pMTUaSvo81tLe?=
 =?us-ascii?Q?2PHBCWdmPPZ3fdMRAV0aeqNzZVos/84YgyKePNE5g7bO3ineUoItL5FSRlgH?=
 =?us-ascii?Q?gvvS/8hx6JstW+J21rAiWQXqZBEBe8tjfF9bAANhta3VAZp0bYyUBjMjeRj6?=
 =?us-ascii?Q?NN2+01/RRODYGk0J3AzVICPgQkV2OTxPSTN0dL76O1f2tnTsIUm9Y7wGEB6F?=
 =?us-ascii?Q?HDJmb9DBJ7l88gmsKmkcVoQOvjMNUvo9KGVNvQ4KQ6EGgZ6YpgJMJ+rkqmQa?=
 =?us-ascii?Q?GnC8Kj8/Qb1aPnO34BnXzU/MwA/6WfiGzIxTB2LbihPNBbb1ypgd2sQLrZeh?=
 =?us-ascii?Q?lV1ZWUqgWkL7UblQRbdntoLGs5+mpZykgbSBTQ5wY9dWF7hyF/tyqNTyOYJw?=
 =?us-ascii?Q?RVdEdL9uaBqmfKJs2mRcr2+BlUk5MCQ1tvPx+HAXnJ3D2Ahl1O39aIotpyK8?=
 =?us-ascii?Q?y+QE/HcQeWiQ17lkKInNpqVy5Dw4/HvloyF/3VutQulr4nd8RqQthZ65yQaR?=
 =?us-ascii?Q?X1x8GHfDmoMq6YPIdwZnlmPkYFPq1A5agni0ckpvl4UBW09orqdENENO294c?=
 =?us-ascii?Q?TVefwd99T6FMRbpzL35dgCgkxxABdpssk7aZIBLsb+vo3oBBsvEX02IjkFGK?=
 =?us-ascii?Q?b3cLEom2IGebpmTL0QVI4CKvfEVDyQSOm1D0X39X3xP/AG7GuQlJpascKudZ?=
 =?us-ascii?Q?1MGDb2rcR6HHeh4y9O7hrzESW77Q1pG0Y97GRwtP5a2fgLgMLdztKxOnmQbl?=
 =?us-ascii?Q?hxuK+RwJB0ty2TEPyoOQvTbk3oGBKhMix7W+mXcs9Xid2yD6ybGR/Ce181mG?=
 =?us-ascii?Q?rzmsjJQe75fQgK+K7ew6UumNX30DBFRNByhv2YaV2CEXDoC04gE0G/Qh67Iq?=
 =?us-ascii?Q?yOftaI0VnUAX5M0IWde03E6OExZQXhKw0eyEERTDKUxPtIt+Z/qWEUU+WMPE?=
 =?us-ascii?Q?aa37CrwvyPKrLpauZ1sx6szQIsWuY6Wjl8OXuEOoSSTzlJSe4lMBitLJ75uS?=
 =?us-ascii?Q?X6waRxaOHCIvhuucrTbG8shOzoxgEo4iLFuhZ3qs3dk3uZXppmJW6iNb98Hv?=
 =?us-ascii?Q?LvQ08+A89sSWQodfejgPRRUDslWhJ+sDV2i4Vj10+YKjoRe6eiTNA0jI3NFj?=
 =?us-ascii?Q?13gZ2aQqxmhWNzB/K7f9S296?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-ID: <204BA4122C434345BA2FC49F47EB916F@namprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR04MB7158.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d64cbe0c-3c91-414a-1a3a-08d93a052cec
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Jun 2021 07:20:15.1588
 (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: vemUxLY1IXsN9WwzjDrp/6hRitduhMBhKifoR2GTfkWCYAzmDuZtU6QDFHQp+D1pd0RycbnqJxw1T5cVIcXwbg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR04MB7256
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Jun 14, 2021 at 12:23:19PM +0000, Niklas Cassel wrote:
> From: Niklas Cassel <niklas.cassel@wdc.com>
>=20
> Allow the following blk-zoned ioctls: BLKREPORTZONE, BLKRESETZONE,
> BLKOPENZONE, BLKCLOSEZONE, and BLKFINISHZONE to be performed without
> CAP_SYS_ADMIN.
>=20
> Neither read() nor write() requires CAP_SYS_ADMIN, and considering
> the close relationship between read()/write() and these ioctls, there
> is no reason to require CAP_SYS_ADMIN for these ioctls either.
>=20
> Changes since v2:
> -Drop the FMODE_READ check from patch 2/2.
> Right now it is possible to open() the device with O_WRONLY
> and get the zone report from that fd. Therefore adding a
> FMODE_READ check on BLKREPORTZONE would break existing applications.
> Instead, just remove the existing CAP_SYS_ADMIN check.
>=20
>=20
> Niklas Cassel (2):
>   blk-zoned: allow zone management send operations without CAP_SYS_ADMIN
>   blk-zoned: allow BLKREPORTZONE without CAP_SYS_ADMIN
>=20
>  block/blk-zoned.c | 6 ------
>  1 file changed, 6 deletions(-)
>=20
> --=20
> 2.31.1

Hello Jens,


A gentle ping on this series.

I think it has sufficient Reviewed-by tags by now.


Kind regards,
Niklas=