12 Feb
2013
12 Feb
'13
1:06 a.m.
On 02/11/2013 07:07 PM, Eric Blake wrote:
Commit 8b55992f added some Coverity comments to silence what was a real bug in the code. Since then, we've had a miserable run of trying to fix the underlying problem (commits c059cde and ba5193c), and still have a problem on 32-bit machines.
This fixes the problem for once and for all, by realizing that on older xen, cpumap_t is identical to uint64_t, and using the new virendian.h to do the transformation from the API (documented to be little-endian) to the host structure.
* src/xen/xen_hypervisor.c (virXen_setvcpumap): Do the conversion correctly. Finally. --- src/xen/xen_hypervisor.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-)
ACK