[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] tools/xendomains: Only save/restore/migrate if supported by xenlight
On 04/04/2023 17:28, Anthony PERARD wrote: Yea the default is the issue. We are building for embedded, using Yocto, so there isn't really an admin.On Wed, Mar 22, 2023 at 01:58:00PM +0000, Peter Hoyes wrote:From: Peter Hoyes <Peter.Hoyes@xxxxxxx> Saving, restoring and migrating domains are not currently supported on arm and arm64 platforms, so xendomains prints the warning: An error occurred while saving domain: command not implemented when attempting to run `xendomains stop`. It otherwise continues to shut down the domains cleanly, with the unsupported steps skipped.The patch looks kind of ok, but shouldn't $XENDOMAINS_SAVE be set to an empty string in the config by the admin instead? Or is the issue that $XENDOMAINS_SAVE is set by default, even on arm* ? Maybe it's easier to check that the command is implemented at run time rather than trying to have a good default value for XENDOMAINS_SAVE at install/package time. It would be cleaner to do this at build time for sure, but I'm not sure the autotools config file approach for sysconfig.xendomains.in can handle the logic for this? Thanks, Peter
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |