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:   Johannes Thumshirn <Johannes.Thumshirn@wdc.com>
To:     Niklas Cassel <Niklas.Cassel@wdc.com>,
        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>
CC:     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: AQHXjQQDlOGMT4pK+Eu3d6/OWUdW7A==
Date:   Mon, 9 Aug 2021 09:58:17 +0000
Message-ID: <PH0PR04MB7416400F3559116FA3192A5F9BF69@PH0PR04MB7416.namprd04.prod.outlook.com>
References: <20210809094855.6226-1-Niklas.Cassel@wdc.com>
 <20210809094855.6226-3-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?30rvB+ASGdiVUkFvgdedf7jmamjkcn2PmSJ7P2CC83J+Qeotbu1Ql7rfzd5b?=
 =?us-ascii?Q?5PeoQskGjXs4a3T+HedfQmQNlYM9m4h5r647+u7hJCittEgznCsKsPZ+NYG1?=
 =?us-ascii?Q?R+uEVXDyQhGB6Ig7Z3RxbhDea3vFXBF+Htv9ExblQ1cPIhhmGTnm1zStKt2N?=
 =?us-ascii?Q?7QGJMZBw6cHnya0Qf/wnQRS5An/1m+bGA+AcTXr0Zbkrhp0mxCMg1dGhn7jN?=
 =?us-ascii?Q?LlU9uPx/qcsEpsHgTAh11OiWjT3J0nFGm75wIsF5NfSF5i1wf/sZgf24VM6N?=
 =?us-ascii?Q?jEVg+uV2h2NYV0z17xwxyUz0HUK13N8yluAmGtUDpycLqjTRetUZOj8/IJT9?=
 =?us-ascii?Q?ZRDLb7J4+7qRx2gPbqY0+Ht2eZ6miV3Zpj4VkNhOi5JvaKQyi1ZLV7tFeEoi?=
 =?us-ascii?Q?PdppTfKDe4FrjcETqOUzIydrj0/xYQcJorQlyE+5tqnpyLZWt2UM8+PiIBKS?=
 =?us-ascii?Q?6ihymnd4ukAZgv2A32R8FgnBw0mHjGvWdCoBZbF/waZwVQZQSO1qkLgHo8Sh?=
 =?us-ascii?Q?6YpCXBW7bOb8T/YWAwUyY9ZJgB9jVO/PODo8YMgI5emD/bV0nSRJeyNOulwq?=
 =?us-ascii?Q?HH7QvE2nsFSu541YCtUjPoYExVoOcIMzhfV1PQBnQ1+jd9m49iqK16621Sr3?=
 =?us-ascii?Q?jt7wrWzLHu+8NfxMgaZZJ0rtIVKRdygiXaBK+cvSWpQ0RUp3OM0eKze3/ssB?=
 =?us-ascii?Q?C33NRUdrxoXzkD+JO98hoDvaHRWubodNIoWroV3ZlGF2+SN5ufJgfaW679i6?=
 =?us-ascii?Q?pmUVtvBwiYeNkjdjveUprnRuhz7/DKhjq+nFPnEE9ninLMQOVC7yBujv6FwP?=
 =?us-ascii?Q?baPJVOXiqg4DuqIuBvG45S5O9J7B/PlCh5BOKBmhaR4zN/ukc7Yf6OA4RalD?=
 =?us-ascii?Q?WgjWkSzNZW2XB4vWUwNlEza6iu9wxxqj63+vQ2z80IJd941Thvlpj5CPnvX2?=
 =?us-ascii?Q?fceyLsIol4+HR6+0eXPH2Z0U7sP29Xlya2meReBEdqzzHosEkWY8+rKBJ8lu?=
 =?us-ascii?Q?LvrRTeaop58rkK+oOnApBwVl+LtI2t8pooaCRwDyu20A2LZezNBaBngZVMij?=
 =?us-ascii?Q?YYkl7WSRcmc64Txlely79dEDYIR427XRoAQSkkImQyNZPG/NngPvFlP5qQan?=
 =?us-ascii?Q?eojT3GluOr2DsRc0t9N2tZeXnIQwPhcEWu9q+rxYuaAGLitrXN08LGoOv3Hp?=
 =?us-ascii?Q?0qvGJCgvmSAM+MhGzu12J2Bf6Njn7aKnQjJxLKQKkq6+mj359uaP3tmF6teh?=
 =?us-ascii?Q?qB/LxTKOyZj43u7Sfgt8G1drMHqZF6DCNt1TTVRJeKJFaaiTYQxMEr4Yge95?=
 =?us-ascii?Q?PRMb9qCli2z4MCEshSdX2vnxkQKSqsRCXsDQW5EWJJWIlotRzdqk0VNtpLPL?=
 =?us-ascii?Q?mdPOTBBNPvfLrk7rbUMcL+RbgUb90N4p5QmkpwasPIZT20O1Tg=3D=3D?=
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: PH0PR04MB7416.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8ec174e5-6cb2-4c85-cf23-08d95b1c360a
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Aug 2021 09:58:17.2012
 (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: 9g/C+TurVBYKiOAp+FHRbC1mKQ5uaUN6kBkbKRdXCZLhR5TpCGNu/bqJ/3Dg/CfVjofF96cqoSc7eNENCZs85a+PYmA8XK/WicUJxJ1+uR0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR04MB7671
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 09/08/2021 11:50, Niklas Cassel wrote:=0A=
> From: Niklas Cassel <niklas.cassel@wdc.com>=0A=
> =0A=
> A user space process should not need the CAP_SYS_ADMIN capability set=0A=
> in order to perform a BLKREPORTZONE ioctl.=0A=
> =0A=
> Getting the zone report is required in order to get the write pointer.=0A=
> Neither read() nor write() requires CAP_SYS_ADMIN, so it is reasonable=0A=
> that a user space process that can read/write from/to the device, also=0A=
> can get the write pointer. (Since e.g. writes have to be at the write=0A=
> pointer.)=0A=
> =0A=
> Fixes: 3ed05a987e0f ("blk-zoned: implement ioctls")=0A=
> Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>=0A=
> Reviewed-by: Damien Le Moal <damien.lemoal@dc.com>=0A=
Nit: Missing 'w'=0A=
> Reviewed-by: Aravind Ramesh <aravind.ramesh@wdc.com>=0A=
> Reviewed-by: Adam Manzanares <a.manzanares@samsung.com>=0A=
> Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>=0A=
> Cc: stable@vger.kernel.org # v4.10+=0A=
> ---=0A=
Looks good,=0A=
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>=0A=
=0A=