Re: [PATCH 1/1] xen/blkfront: fix comment for need_copy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dongli Zhang <dongli.zhang@xxxxxxxxxx>, "linux-block@xxxxxxxxxxxxxxx" <linux-block@xxxxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH 1/1] xen/blkfront: fix comment for need_copy
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Fri, 18 Mar 2022 00:27:28 +0000
Accept-language: en-US
Cc: "boris.ostrovsky@xxxxxxxxxx" <boris.ostrovsky@xxxxxxxxxx>, "jgross@xxxxxxxx" <jgross@xxxxxxxx>, "sstabellini@xxxxxxxxxx" <sstabellini@xxxxxxxxxx>, "roger.pau@xxxxxxxxxx" <roger.pau@xxxxxxxxxx>, "axboe@xxxxxxxxx" <axboe@xxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <20220317220930.5698-1-dongli.zhang@oracle.com>
References: <20220317220930.5698-1-dongli.zhang@oracle.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0

On 3/17/22 3:09 PM, 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,"
> 
> Signed-off-by: Dongli Zhang <dongli.zhang@xxxxxxxxxx>
> ---

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>

-ck

References:
- [PATCH 1/1] xen/blkfront: fix comment for need_copy
  - From: Dongli Zhang

Prev by Date: Re: [PATCH] xen-blkback: remove redundant assignment to variable i
Next by Date: Re: [PATCH 1/3] block: don't show disk stats if io accounting is disabled
Previous by thread: [PATCH 1/1] xen/blkfront: fix comment for need_copy
Next by thread: Re: [PATCH 1/1] xen/blkfront: fix comment for need_copy
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]