[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v11 07/13] block: include supported_read_flags into BDS struc
From: |
Max Reitz |
Subject: |
Re: [PATCH v11 07/13] block: include supported_read_flags into BDS structure |
Date: |
Wed, 14 Oct 2020 14:31:00 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 12.10.20 19:43, Andrey Shinkevich wrote:
> Add the new member supported_read_flags to BlockDriverState structure.
> It will control the BDRV_REQ_PREFETCH flag set for copy-on-read
> operations.
>
> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com>
> ---
> include/block/block_int.h | 4 ++++
> 1 file changed, 4 insertions(+)
Not sure what the problem with passing BDRV_REQ_PREFETCH to drivers that
aren’t the COR filter are would be, but:
Reviewed-by: Max Reitz <mreitz@redhat.com>
(I mean, outside of the context of COR the flag is undefined, so it can
be anything, but intuitively I’d assume it means “no need to read the
data to memory” then, too. So if a driver doesn’t support it, nothing
bad should happen. Hm. Well, unless the driver passes the flag on
(unsuspectingly) and a metadata-bearing child doesn’t return any data
that the drviver needs. Yeah, better to explicitly disallow the flag
for all unsupporting drivers then.)
signature.asc
Description: OpenPGP digital signature
- Re: [PATCH v11 05/13] copy-on-read: limit COR operations to base in COR driver, (continued)
[PATCH v11 06/13] block: modify the comment for BDRV_REQ_PREFETCH flag, Andrey Shinkevich, 2020/10/12
[PATCH v11 07/13] block: include supported_read_flags into BDS structure, Andrey Shinkevich, 2020/10/12
- Re: [PATCH v11 07/13] block: include supported_read_flags into BDS structure,
Max Reitz <=
[PATCH v11 08/13] copy-on-read: add support for BDRV_REQ_PREFETCH to COR-filter, Andrey Shinkevich, 2020/10/12
[PATCH v11 09/13] copy-on-read: skip non-guest reads if no copy needed, Andrey Shinkevich, 2020/10/12
Re: [PATCH v11 09/13] copy-on-read: skip non-guest reads if no copy needed, Andrey Shinkevich, 2020/10/21
Re: [PATCH v11 09/13] copy-on-read: skip non-guest reads if no copy needed, Andrey Shinkevich, 2020/10/22