> On Oct 13, 2020, at 3:40 AM, Pino Toscano <ptoscano@xxxxxxxxxx> wrote: > > What about making a common helper function that calls > hypervMsvmComputerSystemFromDomain + > hypervInvokeMsvmComputerSystemRequestStateChange? I’ll include that in the next version of this patchset. > Note that virDomainReboot() can take various flags, however none is > used ATM. Would it be okay to stick with what I’ve got, now, and look at adding support for the flags in a future commit? -- Matt