Re: [PATCH 1/2] tools/hotplug: fix invalid frontend path for set_mtu

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Fri, Aug 14, 2026 at 07:03:50AM +0100, James Dingwall wrote:
> From: James Dingwall <james@xxxxxxxxxxxxxx>
> 
> The set_mtu() function of xen-network-common.sh currently has this code:
> 
>         if [ ${type_if} = vif ]
>         then
>             local dev_=${dev#vif}
>             local domid=${dev_%.*}
>             local devid=${dev_#*.}
> 
>             local FRONTEND_PATH="/local/domain/$domid/device/vif/$devid"
> 
>             xenstore_write "$FRONTEND_PATH/mtu" ${mtu}
>         fi
> 
> This works fine if the device has its default name but if the xen config
> defines the vifname parameter the FRONTEND_PATH is incorrectly constructed.
> Learn the frontend path by reading the appropriate value from the backend.
> 
> Also change use of `...` to $(...) for a consistent style in the script.
> 
> Signed-off-by: James Dingwall <james@xxxxxxxxxxxxxx>

Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Thanks,


--
Anthony Perard | Vates XCP-ng Developer

XCP-ng & Xen Orchestra - Vates solutions

web: https://vates.tech