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;
From:   Aravind Ramesh <Aravind.Ramesh@wdc.com>
To:     Jens Axboe <axboe@kernel.dk>, Jens Axboe <axboe@fb.com>
CC:     "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 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+YyFqspGeKAgAtFQACAGAKAgA==
Date:   Wed, 21 Jul 2021 05:04:36 +0000
Message-ID: <BYAPR04MB4056026E85897DA145B45BB08CE39@BYAPR04MB4056.namprd04.prod.outlook.com>
References: <20210614122303.154378-1-Niklas.Cassel@wdc.com>
 <YNl4LXUKOftl15M9@x1-carbon.lan> <YOLsgFzIBlg/H/ba@x1-carbon.lan>
In-Reply-To: <YOLsgFzIBlg/H/ba@x1-carbon.lan>
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?vjZikROkv44DHYABtkYHwVugW2VZEkb21PiebylrG5CH3kRPB4yQ6+N9ihDE?=
 =?us-ascii?Q?VILwHjdyAMv3yZKLa/8HZSB1Ht2QRtDk6EFnHTARlA7FFNwRQRcxQgWgScYE?=
 =?us-ascii?Q?YHkelcPcVt0U2lEh8upXKLmJpEsbdBIP8XdAsmxYqTA6VAi5S/kegg3ipk6z?=
 =?us-ascii?Q?sYC0MngnPvTcRvBSPMpytQoIhsINIRgQyuT6eeZcCB82BBI0/LpwRCpHjJBQ?=
 =?us-ascii?Q?z6trGCTiRiFeZQDybOL25tWPgwSMfJJQ8Z/06YLosPgtawwaV2WuikDuWOpI?=
 =?us-ascii?Q?5yB5TVCsQ+UO226x37Ms1K8pM8tApshgodZ0A/Yp+SkS80BPjXKpD967a0CZ?=
 =?us-ascii?Q?NJNa3qYyTnkrIBH8D/B8n5om6jKKxA1jik7MZbxy07ZOLVz8bA1SFuycI+pD?=
 =?us-ascii?Q?VjDdi0DcSVBvrEoWcujkttBUNRDO+epaaizBaKHV4Bpvtkx0DIkjPbL/PGvL?=
 =?us-ascii?Q?t4ix+i/hGwHQ3zm/BjSapQ5/H4SN2DPUSktexLovQveZ9OubFQ9weP2TUrl1?=
 =?us-ascii?Q?EoSD/xgRP9t340EoMh+GDz+ujs2oQ1AgYZMtOFGmSYGyelN4P13dzQg6qcce?=
 =?us-ascii?Q?2/iIw74fTjOOlwfFLOn4mlHpSw9q2xt/OvLELvyLpcMxFWn/cnGJCLjtaPur?=
 =?us-ascii?Q?G84s69IK3kYUPO9cAZskd92XFR1jDtEpo8s1MuCrgJjEQtksQKuoftU8iwMe?=
 =?us-ascii?Q?P2+vi1Tatb+xkrYRtj3FhR+aoguCb5v+3EiduBDjEUfMxJSv6KhmzJBp6Omr?=
 =?us-ascii?Q?CgZDB7lNXHbL4DAZqouGbgmkfhH2oGRXdj4eg1IbSXNJRo2aOVuN01QwkJ2c?=
 =?us-ascii?Q?J25SUo0YnCtYo2hPBKNT7qdNX/vH/n/448OCwuAXgvL2qCpo/lxyoVTtFyRr?=
 =?us-ascii?Q?8WNJvUtRaMIAhRMtf89kCKVpytPvxMpVAYPFDAJeyEoGM/EF77yM7wq34Un7?=
 =?us-ascii?Q?i/TRY+J/I8aPxBW2YQNKcnNiICu7gUhP884fwc3xQ80ZJG+PSxiWnq+VmG0Y?=
 =?us-ascii?Q?oF4JVf2mpXD+CDuMEmDJaf0vP1/o+06rImAZyk2lWLZ+JW9PLFndbpnRl/PP?=
 =?us-ascii?Q?SVuDRDJAv0Set7xCBncwX8W4L0ecY/cQJOw+7krXw5mcrCwWv37KPMkCHQaT?=
 =?us-ascii?Q?f94MPYJ6u/oluEhFSrEYqS7obb36vmQdjSI3Y34KCSOixIx5iMxHvPgFKkqL?=
 =?us-ascii?Q?FrlArxE20+BeGGBHLo7BSJrrxczbgH9Zq0hy/6b6ou7U6W0xXn+ETdqXIP5t?=
 =?us-ascii?Q?Ne+HvIWgiVn8KE5JY/t4wv2oBN513cECTGu4zX+imGqbhdz0+4BqndVl6BFM?=
 =?us-ascii?Q?A0QFjKwUKF9hJoz/tlag60Sb?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BYAPR04MB4056.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: acb74c69-de42-4583-2ce3-08d94c05092c
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jul 2021 05:04:36.0734
 (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: TndyF6cpCl3dZpcABTs8oj0goM8JE2XcLXIs4uHEqfp1MCMqtmoMWFMC8gXG9bHwMbzCCaUHuo2N572QkahPYg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR04MB6963
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Niklas Cassel <Niklas.Cassel@wdc.com>
> Sent: Monday, July 5, 2021 4:57 PM
> To: Jens Axboe <axboe@kernel.dk>; Jens Axboe <axboe@fb.com>
> Cc: linux-block@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH v3 0/2] allow blk-zoned ioctls without CAP_SYS_ADMIN
>=20
> On Mon, Jun 28, 2021 at 09:20:15AM +0200, Niklas Cassel wrote:
> > On Mon, Jun 14, 2021 at 12:23:19PM +0000, Niklas Cassel wrote:
> > > From: Niklas Cassel <niklas.cassel@wdc.com>
> > >
> > > Allow the following blk-zoned ioctls: BLKREPORTZONE, BLKRESETZONE,
> > > BLKOPENZONE, BLKCLOSEZONE, and BLKFINISHZONE to be performed without
> > > CAP_SYS_ADMIN.
> > >
> > > 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.
> > >
> > > 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.
> > >
> > >
> > > Niklas Cassel (2):
> > >   blk-zoned: allow zone management send operations without CAP_SYS_AD=
MIN
> > >   blk-zoned: allow BLKREPORTZONE without CAP_SYS_ADMIN
> > >
> > >  block/blk-zoned.c | 6 ------
> > >  1 file changed, 6 deletions(-)
> > >
> > > --
> > > 2.31.1
> >
> > Hello Jens,
> >
> >
> > A gentle ping on this series.
> >
> > I think it has sufficient Reviewed-by tags by now.
> >
> >
> > Kind regards,
> > Niklas
>=20
> Hello again Jens,
>=20
>=20
> any chance of this series being picked up?
>=20
Hello Jens,

Gentle ping.
Could you please take a look at this series ?

Thanks,
Aravind

>=20
> Kind regards,
> Niklas