---
domain.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/domain.go b/domain.go
index 2bd9852..7276ef8 100644
--- a/domain.go
+++ b/domain.go
@@ -1768,7 +1768,7 @@ func (d *Domain) PinVcpuFlags(vcpu uint, cpuMap []bool,
flags DomainModification
ccpumap :=
In PinEmulator() and PinIOThread() there is an identical code that
converts []bool into a bitmask. It calculates the location in the
bitmask and then sets it always to 1, instead of looking at the
actual bool value.
---
domain.go | 16 ++--
1 file changed, 10 insertions(+), 6
Please ignore this patch, I failed to see that in 2 out of 4 locations we
already are looking at the value. I'm sending a better patch instead.
On Tue, Feb 14, 2017 at 4:48 PM, Leonid Podolny <leo...@podolny.net> wrote:
> In PinVcpu(), PinVcpuFlags(), PinEmulator() and Pi
In PinVcpu(), PinVcpuFlags(), PinEmulator() and PinIOThread() there is
an almost identical code that converts []bool into a bitmask. It
calculates the location in the bitmask and then sets it always to 1,
instead of looking at the actual bool value.
---
domain.go | 16
1 file
The pointers calculation included a typo which caused segfaults.
Signed-off-by: Leonid Podolny <leo...@podolny.net>
---
domain.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/domain.go b/domain.go
index 515138a..74061c2 100644
--- a/domain.go
+++ b/domain.go
@@ -
0001-Fix-segfault-in-GetIOThreadInfo.patch
Description: Binary data
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list