Re: [PATCH guest-common v2 1/11] mapping: dir: add 'live-migration-method' parameter

["Daniel Kral" <d.kral@proxmox.com>]

On Mon Feb 23, 2026 at 4:39 PM CET, Markus Frank wrote:
> If a live migration method is selected, qemu-server starts virtiofsd
> with the flags to enable live migration using the chosen method.
>
> Signed-off-by: Markus Frank <m.frank@proxmox.com>
> ---
>  src/PVE/Mapping/Dir.pm | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/src/PVE/Mapping/Dir.pm b/src/PVE/Mapping/Dir.pm
> index 5ff02d1..a7d4c05 100644
> --- a/src/PVE/Mapping/Dir.pm
> +++ b/src/PVE/Mapping/Dir.pm
> @@ -74,6 +74,15 @@ my $defaultData = {
>              description => "The ID of the directory mapping",
>              format => 'pve-configid',
>          },
> +        'live-migration-method' => {
> +            description => "Allow live-migration when using the directory with a virtiofs device."
> +                ." Ensure you are using the same shared directory on all hosts."
> +                ." Available migration methods are 'file-handles' and 'find-path'.",
> +            type => 'string',
> +            optional => 1,
> +            default => 'file-handles',

The default is off here, as an unset 'live-migration-method' disables
the live migration for VMs that use that virtiofsd mapping.

This should either be clarified as-is in the {verbose_,}description or a
third enum such as 'none' could be introduced to expose that state
explicitly.

> +            enum => ['file-handles', 'find-paths'],
> +        },
>          description => {
>              type => 'string',
>              description => "Description of the directory mapping",
> @@ -99,6 +108,7 @@ sub private {
>  sub options {
>      return {
>          description => { optional => 1 },
> +        'live-migration-method' => { optional => 1 },
>          map => {},
>      };
>  }