[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 08/14] migration/multifd: Add new migration option for mul
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v4 08/14] migration/multifd: Add new migration option for multifd DSA offloading. |
Date: |
Thu, 25 Apr 2024 15:17:05 +0100 |
User-agent: |
Mutt/2.2.12 (2023-09-09) |
On Thu, Apr 25, 2024 at 02:21:11AM +0000, Hao Xiang wrote:
> Intel DSA offloading is an optional feature that turns on if
> proper hardware and software stack is available. To turn on
> DSA offloading in multifd live migration:
>
> multifd-dsa-accel="[dsa_dev_path1] [dsa_dev_path2] ... [dsa_dev_pathX]"
>
> This feature is turned off by default.
>
> Signed-off-by: Hao Xiang <hao.xiang@linux.dev>
> ---
> migration/migration-hmp-cmds.c | 8 ++++++++
> migration/options.c | 30 ++++++++++++++++++++++++++++++
> migration/options.h | 1 +
> qapi/migration.json | 26 +++++++++++++++++++++++---
> 4 files changed, 62 insertions(+), 3 deletions(-)
> diff --git a/qapi/migration.json b/qapi/migration.json
> index 8c65b90328..934fa8839e 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -914,6 +914,12 @@
> # See description in @ZeroPageDetection. Default is 'multifd'.
> # (since 9.0)
> #
> +# @multifd-dsa-accel: If enabled, use DSA accelerator offloading for
> +# certain memory operations. Enable DSA accelerator offloading by
> +# setting this string to a list of DSA device path separated by space
> +# characters. Setting this string to an empty string means disabling
> +# DSA accelerator offloading. Defaults to an empty string. (since 9.2)
Passing a list of paths as a single space separate string is a
design anti-pattern. This needs to use a list type at the QAPI
level.
Also I don't think we need add 'multifd' on the name - all
new features are for multifd.
Overall it should be called 'dsa-accel-path' I thjink
> @@ -1122,6 +1128,12 @@
> # See description in @ZeroPageDetection. Default is 'multifd'.
> # (since 9.0)
> #
> +# @multifd-dsa-accel: If enabled, use DSA accelerator offloading for
> +# certain memory operations. Enable DSA accelerator offloading by
> +# setting this string to a list of DSA device path separated by space
> +# characters. Setting this string to an empty string means disabling
> +# DSA accelerator offloading. Defaults to an empty string. (since 9.2)
> +#
> # Features:
> #
> # @deprecated: Member @block-incremental is deprecated. Use
> @@ -1176,7 +1188,8 @@
> 'features': [ 'unstable' ] },
> '*vcpu-dirty-limit': 'uint64',
> '*mode': 'MigMode',
> - '*zero-page-detection': 'ZeroPageDetection'} }
> + '*zero-page-detection': 'ZeroPageDetection',
> + '*multifd-dsa-accel': 'StrOrNull'} }
This needs to be
['str'] not 'StrOrNull'
>
> ##
> # @migrate-set-parameters:
> @@ -1354,6 +1367,12 @@
> # See description in @ZeroPageDetection. Default is 'multifd'.
> # (since 9.0)
> #
> +# @multifd-dsa-accel: If enabled, use DSA accelerator offloading for
> +# certain memory operations. Enable DSA accelerator offloading by
> +# setting this string to a list of DSA device path separated by space
> +# characters. Setting this string to an empty string means disabling
> +# DSA accelerator offloading. Defaults to an empty string. (since 9.2)
> +#
> # Features:
> #
> # @deprecated: Member @block-incremental is deprecated. Use
> @@ -1405,7 +1424,8 @@
> 'features': [ 'unstable' ] },
> '*vcpu-dirty-limit': 'uint64',
> '*mode': 'MigMode',
> - '*zero-page-detection': 'ZeroPageDetection'} }
> + '*zero-page-detection': 'ZeroPageDetection',
> + '*multifd-dsa-accel': 'str'} }
Liekewise needs to be
['str']
Having mgmt apps pass in the path every time though, feels like
overkill. Surely there's a standard path that QEMU should use
by default, and should only require flag to turn on its usage.
IOW, why not extend the ZeroPageDetection enum, to have a further
entry for 'dsa' to request ue of dsa accel. Passing paths could
be optional.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PATCH v4 03/14] util/dsa: Implement DSA device start and stop logic., (continued)
- [PATCH v4 04/14] util/dsa: Implement DSA task enqueue and dequeue., Hao Xiang, 2024/04/24
- [PATCH v4 05/14] util/dsa: Implement DSA task asynchronous completion thread model., Hao Xiang, 2024/04/24
- [PATCH v4 07/14] util/dsa: Implement DSA task asynchronous submission and wait for completion., Hao Xiang, 2024/04/24
- [PATCH v4 08/14] migration/multifd: Add new migration option for multifd DSA offloading., Hao Xiang, 2024/04/24
- Re: [PATCH v4 08/14] migration/multifd: Add new migration option for multifd DSA offloading.,
Daniel P . Berrangé <=
- [PATCH v4 06/14] util/dsa: Implement zero page checking in DSA task., Hao Xiang, 2024/04/24
- [PATCH v4 09/14] migration/multifd: Prepare to introduce DSA acceleration on the multifd path., Hao Xiang, 2024/04/24
- [PATCH v4 10/14] migration/multifd: Enable DSA offloading in multifd sender path., Hao Xiang, 2024/04/24
- [PATCH v4 11/14] migration/multifd: Add migration option set packet size., Hao Xiang, 2024/04/24
- [PATCH v4 12/14] migration/multifd: Enable set packet size migration option., Hao Xiang, 2024/04/24
- [PATCH v4 13/14] util/dsa: Add unit test coverage for Intel DSA task submission and completion., Hao Xiang, 2024/04/24
- [PATCH v4 14/14] migration/multifd: Add integration tests for multifd with Intel DSA offloading., Hao Xiang, 2024/04/24