[siemens/jailhouse] dc2ea3: x86: fix erroneous sizeof() usage

3 views
Skip to first unread message

GitHub

unread,
Nov 6, 2015, 1:29:45 AM11/6/15
to jailho...@googlegroups.com
Branch: refs/heads/next
Home: https://github.com/siemens/jailhouse
Commit: dc2ea3108c6be82d09bd81a7330735a074733de0
https://github.com/siemens/jailhouse/commit/dc2ea3108c6be82d09bd81a7330735a074733de0
Author: Valentine Sinitsyn <valentine...@gmail.com>
Date: 2015-11-04 (Wed, 04 Nov 2015)

Changed paths:
M hypervisor/arch/x86/svm.c
M hypervisor/arch/x86/vmx.c

Log Message:
-----------
x86: fix erroneous sizeof() usage

vcpu functions were using sizeof() to determine the size of dynamically
allocated I/O bitmap, which won't work. Assign this value statically per
sub-architecture (Intel or AMD).

Reported-by: Ralf Ramsauer <ra...@ramses-pyramidenbau.de>
Signed-off-by: Valentine Sinitsyn <valentine...@gmail.com>
Signed-off-by: Jan Kiszka <jan.k...@siemens.com>


Commit: 592b2b3a9dfeee54b8e163cbb6c3f90158a784b4
https://github.com/siemens/jailhouse/commit/592b2b3a9dfeee54b8e163cbb6c3f90158a784b4
Author: Ralf Ramsauer <ra...@ramses-pyramidenbau.de>
Date: 2015-11-04 (Wed, 04 Nov 2015)

Changed paths:
M hypervisor/arch/x86/vcpu.c

Log Message:
-----------
Respect size of io bitmap in vcpu_cell_init()

Previous code copied the IO bitmap without respect to its actual size.
This patch simplifies the copying process and respects the size of the
destination.

Signed-off-by: Ralf Ramsauer <ra...@ramses-pyramidenbau.de>
[Jan: fine-tuned comments]
Signed-off-by: Jan Kiszka <jan.k...@siemens.com>


Compare: https://github.com/siemens/jailhouse/compare/aabfa1e34c01...592b2b3a9dfe
Reply all
Reply to author
Forward
0 new messages