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:   Niklas Cassel <Niklas.Cassel@wdc.com>
To:     Johannes Thumshirn <Johannes.Thumshirn@wdc.com>
CC:     Jens Axboe <axboe@kernel.dk>,
        Damien Le Moal <Damien.LeMoal@wdc.com>,
        Hannes Reinecke <hare@suse.com>,
        Shaun Tancheff <shaun@tancheff.com>,
        "Martin K. Petersen" <martin.petersen@oracle.com>,
        Damien Le Moal <damien.lemoal@dc.com>,
        Aravind Ramesh <Aravind.Ramesh@wdc.com>,
        Adam Manzanares <a.manzanares@samsung.com>,
        Himanshu Madhani <himanshu.madhani@oracle.com>,
        "stable@vger.kernel.org" <stable@vger.kernel.org>,
        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 v4 2/2] blk-zoned: allow BLKREPORTZONE without
 CAP_SYS_ADMIN
Thread-Topic: [PATCH v4 2/2] blk-zoned: allow BLKREPORTZONE without
 CAP_SYS_ADMIN
Thread-Index: AQHXjQP4aNCn2fw5YUWhEgczWRLRa6tq8fmA
Date:   Mon, 9 Aug 2021 10:04:56 +0000
Message-ID: <YRD9x3+DOTFK82pr@x1-carbon>
References: <20210809094855.6226-1-Niklas.Cassel@wdc.com>
 <20210809094855.6226-3-Niklas.Cassel@wdc.com>
 <PH0PR04MB7416400F3559116FA3192A5F9BF69@PH0PR04MB7416.namprd04.prod.outlook.com>
In-Reply-To: <PH0PR04MB7416400F3559116FA3192A5F9BF69@PH0PR04MB7416.namprd04.prod.outlook.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?psQxS2FTxQWQMyGFWA5Vzmilmq27lHuBELZJKiSAO9zOh6W0ZVshZpF7j205?=
 =?us-ascii?Q?F8VxiVAjw8LUjKajGqh1uJ3hZlKjD7BJOI+Zd5YuXpyOlzY3ZsaE5hvvA3E6?=
 =?us-ascii?Q?LeQz/MFC6g/dJE8u03K1ETVFuujURhuDLqLGpEoABxzDgcwNDKyXYq2Sp+mz?=
 =?us-ascii?Q?H2xL2EgqrUfBKbtqkTvnCDMbe1/ZCW3uZk0eiKlHOuM31ke1GpIeHHVUDsvV?=
 =?us-ascii?Q?xm4UUZwpTXuyk/EmMG7zUne806IJ7ix6vGNQJgCAZiqFM67FLy75VU9c8IBO?=
 =?us-ascii?Q?tmNfUZE3ZryggbZv8iIRmwP+vG1HC9MIH/pDNQOsf+pQH8G07ssfRJvRcHG5?=
 =?us-ascii?Q?O0L5bHsP8jzoQT4kQQL/c2xVuE8vneTuOAaVfDnUgdr83PBberAV2exUXCox?=
 =?us-ascii?Q?z5c/W5sNX7/GHoUqVE5d7u0hUWZd1GnMD7AhocgXR7vsOxLJBOE6Ri94EP97?=
 =?us-ascii?Q?GsCkqzGKVRu+kxm+LiIClDjZj3urwAEueX6raGGlPXIwabUo6Vks0ogXL6NK?=
 =?us-ascii?Q?90LdqYRc1/hrT3gHUnGEOxuzxruX7+KEIA0hkKhnDfkJ+9EmWm/i22RWoV3J?=
 =?us-ascii?Q?NJwJL6FygO7M1ypIv38fomEmxm+Jsxq/2lNeZw+s2rbjHFdBSvuDi6xb78WU?=
 =?us-ascii?Q?eUiPQYZEn7tVTXKC3cB3pxejPazz5ZfN3wR1ytxgv+6P9IqPsUUtdo6gPEtP?=
 =?us-ascii?Q?7GdK0E1/Bo8reEZz8qVqS5ulyQpzlOSmVsEpayLJ6P94rx24BlpoTgRBBi22?=
 =?us-ascii?Q?zqkDP1Uune1l//+2Y1ak7/Aai+XHQSK7B+cO7cfjR2vXVeH8Q0YaU1mq9kwx?=
 =?us-ascii?Q?Ynd9N86LlgREQHGt71nEOZ6Fzcc85Jy+4dCfSnkYI1DAdUqxba/r5meNGTM1?=
 =?us-ascii?Q?t86Fn6iWXcb89lev1P2Tmh4PQw364PmSJ54yD7PliLPS0Iv4BjwRDQlHqJ2u?=
 =?us-ascii?Q?HqgJ/vy7aqK1StKzzDeT8qTQXxaF0oPWCfiM/kymrj+/H+g+4TRChplKsg8c?=
 =?us-ascii?Q?tucO5O01q8tzRIxCS5ixNJcPZc71nDI/RiGN9iR0hvG6OQ51quI6FsvOtTHh?=
 =?us-ascii?Q?c7Vunvh8BuLjvgo0WKskcMplUWaw0aO2yJT4SfyePRGlaQrZe1I8ueWKMkpG?=
 =?us-ascii?Q?v/1Nyc8l0barGDzX5L+LXnVhZ7HLIMtC57Bozj4JHlueH8DVoPIPo1wwcjmw?=
 =?us-ascii?Q?Vf8okMPg2Y7B/h1hwF/WHnHGd3o0ZikpsCgs6LkPvqB/nCy4bkLYso7knSMI?=
 =?us-ascii?Q?49xrV33j1xRYWX8zu5oHXc9E5Ivn4pXDtLgTiFBQaqzCmdZzLUa8Z/Rj3RtN?=
 =?us-ascii?Q?zRc4WOr6Gafre3haVg8dd+m8?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <A524B31B99C0D54482FCCA8F37E498E3@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: f55cd8d5-bb1d-4984-d446-08d95b1d23f5
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Aug 2021 10:04:56.3814
 (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: 5NV1EaKABB8/VPE/8KXPW6KO8q0Jzpwr754ObBqjAieYgRSw50Cbez1PV8Z8q1Ro2Ckv0cy7fuBamWam3njZSw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR04MB7494
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Aug 09, 2021 at 09:58:17AM +0000, Johannes Thumshirn wrote:
> On 09/08/2021 11:50, Niklas Cassel wrote:
> > From: Niklas Cassel <niklas.cassel@wdc.com>
> >=20
> > A user space process should not need the CAP_SYS_ADMIN capability set
> > in order to perform a BLKREPORTZONE ioctl.
> >=20
> > Getting the zone report is required in order to get the write pointer.
> > Neither read() nor write() requires CAP_SYS_ADMIN, so it is reasonable
> > that a user space process that can read/write from/to the device, also
> > can get the write pointer. (Since e.g. writes have to be at the write
> > pointer.)
> >=20
> > Fixes: 3ed05a987e0f ("blk-zoned: implement ioctls")
> > Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
> > Reviewed-by: Damien Le Moal <damien.lemoal@dc.com>
> Nit: Missing 'w'

Seems like Damien's email had a typo in his original reply:

https://patchwork.kernel.org/project/linux-block/patch/20210614122303.15437=
8-3-Niklas.Cassel@wdc.com/#24252237

Surprised that he doesn't have a shortcut to paste his Reviewed-by tag,
considering the amount of patches that he reviews :)

> > Reviewed-by: Aravind Ramesh <aravind.ramesh@wdc.com>
> > Reviewed-by: Adam Manzanares <a.manzanares@samsung.com>
> > Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>
> > Cc: stable@vger.kernel.org # v4.10+
> > ---
> Looks good,
> Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
>=20
> =