Re: [PATCH 1/1] xen/blkfront: fix comment for need_copy
From: Roger Pau Monné
Date: Mon Mar 21 2022 - 06:06:59 EST
On Thu, Mar 17, 2022 at 03:09:30PM -0700, Dongli Zhang wrote:
> The 'need_copy' is set when rq_data_dir(req) returns WRITE, in order to
> copy the written data to persistent page.
>
> ".need_copy = rq_data_dir(req) && info->feature_persistent,"
I would also add:
Fixes: c004a6fe0c40 ('block/xen-blkfront: Make it running on 64KB page granularity')
> Signed-off-by: Dongli Zhang <dongli.zhang@xxxxxxxxxx>
Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
Albeit I have one nit since you are already changing the line.
> ---
> drivers/block/xen-blkfront.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c
> index 03b5fb341e58..dbc32d0a4b1a 100644
> --- a/drivers/block/xen-blkfront.c
> +++ b/drivers/block/xen-blkfront.c
> @@ -576,7 +576,7 @@ struct setup_rw_req {
> struct blkif_request *ring_req;
> grant_ref_t gref_head;
> unsigned int id;
> - /* Only used when persistent grant is used and it's a read request */
> + /* Only used when persistent grant is used and it's a write request */
While there you might want to adjust the comment to:
"... persistent grants are used ..."
Thanks, Roger.