golang/xenlight: add missing arguments to libxl_domain_shutdown/reboot

Commit Message

Nick Rosbrook Nov. 14, 2019, 10:44 p.m. UTC

From: Nick Rosbrook <[e-mail]>

These functions now have a third parameter of type const *libxl_asyncop_how.

Pass nil for this argument to fix compilation and maintain the
synchronous behavior.

Signed-off-by: Nick Rosbrook <[e-mail]>
---
 tools/golang/xenlight/xenlight.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

George Dunlap Nov. 15, 2019, 9:26 a.m. UTC | #1

On 11/14/19 10:44 PM, Nick Rosbrook wrote:
> From: Nick Rosbrook <[e-mail]>
> 
> These functions now have a third parameter of type const *libxl_asyncop_how.
> 
> Pass nil for this argument to fix compilation and maintain the
> synchronous behavior.
> 
> Signed-off-by: Nick Rosbrook <[e-mail]>

Actually this has already been submitted and Release-acked here:

https://patchew.org/Xen/[e-mail]/

It was just done during a commit moratorium, and then forgotten
afterwards.  I'll check it in now.

 -George

Nick Rosbrook Nov. 15, 2019, 2:41 p.m. UTC | #2

> Actually this has already been submitted and Release-acked here:
>
> https://patchew.org/Xen/[e-mail]/

Ah thanks, I remember seeing that now but I confused it with the
*very* similar patch for libxl_domain_pause/unpause.

-NR

Patch

diff --git a/tools/golang/xenlight/xenlight.go b/tools/golang/xenlight/xenlight.go
index 59b8186a64..902cbb839e 100644
--- a/tools/golang/xenlight/xenlight.go
+++ b/tools/golang/xenlight/xenlight.go
@@ -1041,7 +1041,7 @@  func (Ctx *Context) DomainShutdown(id Domid) (err error) {
 		return
 	}
 
-	ret := C.libxl_domain_shutdown(Ctx.ctx, C.uint32_t(id))
+	ret := C.libxl_domain_shutdown(Ctx.ctx, C.uint32_t(id), nil)
 
 	if ret != 0 {
 		err = Error(-ret)
@@ -1056,7 +1056,7 @@  func (Ctx *Context) DomainReboot(id Domid) (err error) {
 		return
 	}
 
-	ret := C.libxl_domain_reboot(Ctx.ctx, C.uint32_t(id))
+	ret := C.libxl_domain_reboot(Ctx.ctx, C.uint32_t(id), nil)
 
 	if ret != 0 {
 		err = Error(-ret)