[Xen-devel] [PATCH v5] x86/mm: Suppresses vm_events caused by page-walks

2018-04-23 Thread Alexandru Isaila
. In p2m_mem_access_check() we emulate so no event will get sent. Signed-off-by: Alexandru Isaila --- Changes since V4: - Added the hvm_emulate_one_vm_event() call in p2m_mem_access_check() - Removed p2m_set_ad_bits(). --- tools/libxc/include/xenctrl.h | 2 ++ tools

[Xen-devel] [PATCH v4 2/2] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-05-07 Thread Alexandru Isaila
This patch adds a vcpu param for all the *save() fucntions. The *save() functions now handle only one instance. The for_each loop is transferred to the caller. Signed-off-by: Alexandru Isaila --- Changes since V3: - Rb to the lateste staging version - Moved the for_each() loop

[Xen-devel] [PATCH v4 1/2] x86/hvm: Introduce *save_one() functions

2018-05-07 Thread Alexandru Isaila
This patch introduces save_one() functions. They will be called in the *save() so we can extract data for a single instance. Signed-off-by: Alexandru Isaila --- Changes since V3: - Rb to the lateste staging version - Split the patch into 2 patches. --- xen/arch/x86/cpu/mcheck

[Xen-devel] [PATCH v6] x86/mm: Suppresses vm_events caused by page-walks

2018-05-07 Thread Alexandru Isaila
. In p2m_mem_access_check() we emulate so no event will get sent. Signed-off-by: Alexandru Isaila --- Changes since V5: - Add comment for the xc_monitor_inguest_pagefault() func. - Add altp2m_write_no_gpt test in xen_access --- tools/libxc/include/xenctrl.h | 7

[Xen-devel] [PATCH v1 2/2] hvm/svm: Enable EMUL_UNIMPLEMENTED events on svm

2018-05-11 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/include/asm-x86/monitor.h | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/xen/include/asm-x86/monitor.h b/xen/include/asm-x86/monitor.h index c5a86d1..7ef2aa2 100644 --- a/xen/include/asm-x86/monitor.h +++ b/xen/include

[Xen-devel] [PATCH v1 1/2] x86/mm: Add mem access rights to NPT

2018-05-11 Thread Alexandru Isaila
: Alexandru Isaila --- xen/arch/x86/mm/mem_access.c | 4 +- xen/arch/x86/mm/p2m-pt.c | 85 --- xen/include/asm-x86/mem_access.h | 2 +- xen/include/asm-x86/x86_64/page.h | 9 + 4 files changed, 84 insertions(+), 16 deletions(-) diff --git a/xen

[Xen-devel] [PATCH v1] hvm/svm: Implement Debug events

2018-03-19 Thread Alexandru Isaila
tor 1 exception generated by the single byte INT1 instruction (also known as ICEBP) does not trigger the #DB intercept. Software should use the dedicated ICEBP intercept to intercept ICEBP" Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/svm/svm

[Xen-devel] [PATCH v2] hvm/svm: Implement Debug events

2018-03-20 Thread Alexandru Isaila
- Updated __get_instruction_length() for the INSTR_ICEBP instruction Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/svm/emulate.c| 1 + xen/arch/x86/hvm/svm/svm.c| 37 +-- xen/arch/x86/hvm/svm/vmcb.c | 2 +- xen/include/asm-x86/hvm/svm/emul

[Xen-devel] [PATCH v3] hvm/svm: Implement Debug events

2018-03-21 Thread Alexandru Isaila
able icebp interception - Changed trap_type to unsigned int - Replaced svm_propagate_intr with hvm_inject_exception Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/svm/emulate.c| 1 + xen/arch/x86/hvm/svm/svm.c| 71 +++ xe

[Xen-devel] [PATCH v4] hvm/svm: Implement Debug events

2018-03-22 Thread Alexandru Isaila
tor 1 exception generated by the single byte INT1 instruction (also known as ICEBP) does not trigger the #DB intercept. Software should use the dedicated ICEBP intercept to intercept ICEBP" Signed-off-by: Alexandru Isaila --- Changes since V3: - Merge disable/enable

[Xen-devel] [PATCH v5] hvm/svm: Implement Debug events

2018-03-23 Thread Alexandru Isaila
tor 1 exception generated by the single byte INT1 instruction (also known as ICEBP) does not trigger the #DB intercept. Software should use the dedicated ICEBP intercept to intercept ICEBP" Signed-off-by: Alexandru Isaila --- Changes since V4: - Add const to struct vcpu *v

[Xen-devel] [PATCH v6] hvm/svm: Implement Debug events

2018-03-26 Thread Alexandru Isaila
tor 1 exception generated by the single byte INT1 instruction (also known as ICEBP) does not trigger the #DB intercept. Software should use the dedicated ICEBP intercept to intercept ICEBP" Signed-off-by: Alexandru Isaila --- Changes since V5: -Reformulate if in case VMEXIT_ICEBP --- xe

[Xen-devel] [PATCH v3] x86/mm: Add mem access rights to NPT

2018-07-02 Thread Alexandru Isaila
xen-access write Signed-off-by: Alexandru Isaila --- Changes since V2: - Delete blak line - Add return if p2m_access_rwx = a - Delete the comment from p2m_pt_get_entry() - Moved radix_tree_init() to arch_monitor_init_domain(). --- xen/arch/x86/mm/mem_access.c

[Xen-devel] [PATCH v10 08/11] x86/hvm: Add handler for save_one funcs

2018-07-04 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V8: - Add comment for the handler return values. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c

[Xen-devel] [PATCH v10 07/11] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/viridian.c | 24 +++- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c b/xen/arch/x86/hvm/viridian.c index 694eae6..1e87cd6 100644

[Xen-devel] [PATCH v10 04/11] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Move continue out of save_one func. --- xen/arch/x86/hvm/hvm.c | 34 +- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/xen/arch/x86

[Xen-devel] [PATCH v10 01/11] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff

[Xen-devel] [PATCH v10 10/11] x86/hvm: Remove redundant save functions

2018-07-04 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V9: - Add enum return type for save funcs --- xen/arch/x86/cpu/mcheck/vmce.c | 24

[Xen-devel] [PATCH v10 02/11] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. --- xen/arch/x86/hvm/hvm.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v10 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-07-04 Thread Alexandru Isaila
This patch is focused on moving the for loop to the caller so now we can save info for a single vcpu instance with the save_one handlers. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/save.c | 141 +--- 1 file changed, 111 insertions(+), 30

[Xen-devel] [PATCH v10 00/11] x86/domctl: Save info for one vcpu instance

2018-07-04 Thread Alexandru Isaila
. The first one removes the save* funcs and renames the save_one* to save. The last patch removes the save_one* handler that is no longer used. Cheers, Alexandru Isaila (11): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce

[Xen-devel] [PATCH v10 05/11] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V7: - Moved the init of ctxt->count to hvm_save_cpu_msrs_one() --- xen/arch/x86/hvm/hvm.c | 101 +++-- 1 file changed, 55 insertions(+),

[Xen-devel] [PATCH v10 06/11] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c

[Xen-devel] [PATCH v10 03/11] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-07-04 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V8: - Change return of the save_one func to return hvm_save_entry - Move continue out of on func - Remove #define CONTINUE. --- xen/arch/x86/hvm/hvm.c | 211

[Xen-devel] [PATCH v10 11/11] x86/hvm: Remove save_one handler

2018-07-04 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/arch/x86/cpu/mcheck/vmce.c | 1 - xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 5 + xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c | 6 +++--- xen/arch/x86/hvm/mtrr.c| 2 +- xen/arch/x86

[Xen-devel] [PATCH v11 00/11] x86/domctl: Save info for one vcpu instance

2018-07-13 Thread Alexandru Isaila
. The first one removes the save* funcs and renames the save_one* to save. The last patch removes the save_one* handler that is no longer used. Cheers, Alexandru Isaila (11): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce

[Xen-devel] [PATCH v11 04/11] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V10: - Add ASSERT to save_one func. --- xen/arch/x86/hvm/hvm.c | 36 +++- 1 file changed, 23 insertions(+), 13 deletions(-) diff --git a/xen/arch/x86/hvm

[Xen-devel] [PATCH v11 02/11] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. --- xen/arch/x86/hvm/hvm.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions

[Xen-devel] [PATCH v11 01/11] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V10: - Add memset to 0 for ctxt. --- xen/arch/x86/cpu/mcheck/vmce.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/xen/arch/x86/cpu/mcheck

[Xen-devel] [PATCH v11 06/11] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v10: - Add blank line before memset. Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c | 76 ++--- 1

[Xen-devel] [PATCH v11 08/11] x86/hvm: Add handler for save_one funcs

2018-07-13 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V8: - Add comment for the handler return values. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c

[Xen-devel] [PATCH v11 05/11] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V7: - Moved the init of ctxt->count to hvm_save_cpu_msrs_one() --- xen/arch/x86/hvm/hvm.c | 101 +++-- 1 f

[Xen-devel] [PATCH v11 03/11] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V8: - Change return of the save_one func to return hvm_save_entry - Move continue out of on func - Remove #define CONTINUE. --- xen/arch/x86

[Xen-devel] [PATCH v11 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-07-13 Thread Alexandru Isaila
This patch is focused on moving the for loop to the caller so now we can save info for a single vcpu instance with the save_one handlers. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/save.c | 141 +--- 1 file changed, 111 insertions(+), 30

[Xen-devel] [PATCH v11 07/11] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-07-13 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- xen/arch/x86/hvm/viridian.c | 24 +++- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c b/xen/arch/x86/hvm/viridian.c

[Xen-devel] [PATCH v11 10/11] x86/hvm: Remove redundant save functions

2018-07-13 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V9: - Add enum return type for save funcs --- xen/arch/x86/cpu/mcheck/vmce.c | 24

[Xen-devel] [PATCH v11 11/11] x86/hvm: Remove save_one handler

2018-07-13 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/arch/x86/cpu/mcheck/vmce.c | 1 - xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 5 + xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c | 6 +++--- xen/arch/x86/hvm/mtrr.c| 2 +- xen/arch/x86

[Xen-devel] [PATCH v12 04/11] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - hvm_save_cpu_xsave_states_one() returns the err from _hvm_init_entry(). - hvm_save_cpu_xsave_states() returns err from hvm_save_cpu_xsave_states_one

[Xen-devel] [PATCH v12 10/11] x86/hvm: Remove redundant save functions

2018-07-16 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V11: - Remove enum return type for save funcs. --- xen/arch/x86/cpu/mcheck/vmce.c | 19

[Xen-devel] [PATCH v12 02/11] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. --- xen/arch/x86/hvm/hvm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v12 11/11] x86/hvm: Remove save_one handler

2018-07-16 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/arch/x86/cpu/mcheck/vmce.c | 1 - xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 5 + xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c | 6 +++--- xen/arch/x86/hvm/mtrr.c| 2 +- xen/arch/x86

[Xen-devel] [PATCH v12 07/11] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/viridian.c | 29 +++-- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c b/xen/arch/x86/hvm/viridian.c index 694eae6..f3430bb

[Xen-devel] [PATCH v12 01/11] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - Removed the memset and added init with {}. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v12 08/11] x86/hvm: Add handler for save_one funcs

2018-07-16 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V8: - Add comment for the handler return values. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c

[Xen-devel] [PATCH v12 06/11] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v11: - hvm_save_mtrr_msr() now returns err from hvm_save_mtrr_msr_one(). Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c

[Xen-devel] [PATCH v12 00/11] x86/domctl: Save info for one vcpu instance

2018-07-16 Thread Alexandru Isaila
. The first one removes the save* funcs and renames the save_one* to save. The last patch removes the save_one* handler that is no longer used. Cheers, Alexandru Isaila (11): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce

[Xen-devel] [PATCH v12 05/11] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V11: - hvm_save_cpu_msrs() returns err from hvm_save_cpu_msrs_one(). --- xen/arch/x86/hvm/hvm.c | 105 +++-- 1 file

[Xen-devel] [PATCH v12 03/11] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-07-16 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - hvm_save_cpu_ctxt() now returns err from hvm_save_cpu_ctxt_one(). --- xen/arch/x86/hvm/hvm.c | 216 ++--- 1 file changed, 113

[Xen-devel] [PATCH v12 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-07-16 Thread Alexandru Isaila
This patch is focused on moving the for loop to the caller so now we can save info for a single vcpu instance with the save_one handlers. Signed-off-by: Alexandru Isaila --- Changes since V11: - Changed the CONTINUE return to return 0. --- xen/arch/x86/hvm/hvm.c | 18 +++ xen

[Xen-devel] [PATCH v13 08/11] x86/hvm: Add handler for save_one funcs

2018-07-19 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V8: - Add comment for the handler return values. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c

[Xen-devel] [PATCH v13 07/11] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V12: - Remove blank line - Apply coding style to for_each. --- xen/arch/x86/hvm/viridian.c | 30 +++--- 1 file changed, 19

[Xen-devel] [PATCH v13 00/11] x86/domctl: Save info for one vcpu instance

2018-07-19 Thread Alexandru Isaila
. The first one removes the save* funcs and renames the save_one* to save. The last patch removes the save_one* handler that is no longer used. Cheers, Alexandru Isaila (11): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce

[Xen-devel] [PATCH v13 04/11] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - hvm_save_cpu_xsave_states_one() returns the err from _hvm_init_entry(). - hvm_save_cpu_xsave_states() returns err from hvm_save_cpu_xsave_states_one

[Xen-devel] [PATCH v13 11/11] x86/hvm: Remove save_one handler

2018-07-19 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/arch/x86/cpu/mcheck/vmce.c | 1 - xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 5 + xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c | 6 +++--- xen/arch/x86/hvm/mtrr.c| 2 +- xen/arch/x86

[Xen-devel] [PATCH v13 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-07-19 Thread Alexandru Isaila
This patch is focused on moving the for loop to the caller so now we can save info for a single vcpu instance with the save_one handlers. Signed-off-by: Alexandru Isaila --- Changes since V11: - Changed the CONTINUE return to return 0. --- xen/arch/x86/hvm/hvm.c | 19 --- xen

[Xen-devel] [PATCH v13 02/11] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V9: - Change return of the save_one func to return hvm_save_entry. --- xen/arch/x86/hvm/hvm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v13 10/11] x86/hvm: Remove redundant save functions

2018-07-19 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V11: - Remove enum return type for save funcs. --- xen/arch/x86/cpu/mcheck/vmce.c | 19

[Xen-devel] [PATCH v13 03/11] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V12: - Changed memset to {} init. --- xen/arch/x86/hvm/hvm.c | 214 + 1 file changed, 111 insertions(+), 103 deletions(-) diff --git a

[Xen-devel] [PATCH v13 05/11] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V11: - hvm_save_cpu_msrs() returns err from hvm_save_cpu_msrs_one(). --- xen/arch/x86/hvm/hvm.c | 105 +++-- 1 file

[Xen-devel] [PATCH v13 06/11] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v11: - hvm_save_mtrr_msr() now returns err from hvm_save_mtrr_msr_one(). Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c

[Xen-devel] [PATCH v13 01/11] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-07-19 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - Removed the memset and added init with {}. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v4] x86/mm: Add mem access rights to NPT

2018-07-23 Thread Alexandru Isaila
them. Modify p2m-pt.c:p2m_type_to_flags() to mirror the ept version: taking an access, and removing / adding RW or NX flags as appropriate. Note: It was tested with xen-access write Signed-off-by: Alexandru Isaila --- Changes since V3: - Add p2m_pt_check_access() to filter n, w, wx, n2rwx

[Xen-devel] [PATCH v14 02/11] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V13: - Moved tsc_adjust to the initializer. --- xen/arch/x86/hvm/hvm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/xen/arch/x86/hvm/hvm.c b/xen

[Xen-devel] [PATCH v14 04/11] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - hvm_save_cpu_xsave_states_one() returns the err from _hvm_init_entry(). - hvm_save_cpu_xsave_states() returns err from hvm_save_cpu_xsave_states_one

[Xen-devel] [PATCH v14 05/11] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V11: - hvm_save_cpu_msrs() returns err from hvm_save_cpu_msrs_one(). --- xen/arch/x86/hvm/hvm.c | 105 +++-- 1 file

[Xen-devel] [PATCH v14 01/11] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - Removed the memset and added init with {}. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v14 07/11] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V13: - Fixed style. --- xen/arch/x86/hvm/viridian.c | 30 +++--- 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/xen

[Xen-devel] [PATCH v14 00/11] x86/domctl: Save info for one vcpu instance

2018-07-25 Thread Alexandru Isaila
. The first one removes the save* funcs and renames the save_one* to save. The last patch removes the save_one* handler that is no longer used. Cheers, Alexandru Isaila (11): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce

[Xen-devel] [PATCH v14 08/11] x86/hvm: Add handler for save_one funcs

2018-07-25 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V8: - Add comment for the handler return values. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c

[Xen-devel] [PATCH v14 03/11] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V12: - Changed memset to {} init. --- xen/arch/x86/hvm/hvm.c | 214 + 1 file changed, 111 insertions(+), 103 deletions(-) diff --git a

[Xen-devel] [PATCH v14 06/11] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-07-25 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v11: - hvm_save_mtrr_msr() now returns err from hvm_save_mtrr_msr_one(). Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c

[Xen-devel] [PATCH v14 09/11] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-07-25 Thread Alexandru Isaila
This patch is focused on moving the for loop to the caller so now we can save info for a single vcpu instance with the save_one handlers. Signed-off-by: Alexandru Isaila --- Changes since V11: - Changed the CONTINUE return to return 0. --- xen/arch/x86/hvm/hvm.c | 19 --- xen

[Xen-devel] [PATCH v14 10/11] x86/hvm: Remove redundant save functions

2018-07-25 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V11: - Remove enum return type for save funcs. --- xen/arch/x86/cpu/mcheck/vmce.c | 19

[Xen-devel] [PATCH v14 11/11] x86/hvm: Remove save_one handler

2018-07-25 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- xen/arch/x86/cpu/mcheck/vmce.c | 1 - xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 5 + xen/arch/x86/hvm/i8254.c | 2 +- xen/arch/x86/hvm/irq.c | 6 +++--- xen/arch/x86/hvm/mtrr.c| 2 +- xen/arch/x86

[Xen-devel] [PATCH v1] x86/hvm: Drop hvm_sr_handlers initializer

2018-08-03 Thread Alexandru Isaila
This initializer is flawed and only sets .name of array entry 0 to a non-NULL string. Signed-off-by: Alexandru Isaila Suggested-by: Jan Beulich --- xen/arch/x86/hvm/save.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/x86/hvm/save.c b/xen/arch/x86/hvm/save.c

[Xen-devel] [PATCH v15 07/14] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V14: - Moved all the operations in the initializer. --- xen/arch/x86/hvm/viridian.c | 30 +++--- 1 file changed, 19 insertions(+), 11

[Xen-devel] [PATCH v15 08/14] x86/hvm: Introduce lapic_save_hidden_one

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/vlapic.c | 22 ++ 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/xen/arch/x86/hvm/vlapic.c b/xen/arch/x86/hvm/vlapic.c index 1b9f00a..0795161 100644 --- a

[Xen-devel] [PATCH v15 12/14] x86/hvm: Drop the use of save functions

2018-08-03 Thread Alexandru Isaila
This patch drops the use of save functions in hvm_save. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/save.c | 25 - 1 file changed, 4 insertions(+), 21 deletions(-) diff --git a/xen/arch/x86/hvm/save.c b/xen/arch/x86/hvm/save.c index 61565fe..363695c 100644

[Xen-devel] [PATCH v15 03/14] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V14: - Move all free fields to the initializer - Add blank line to before the return - Move v->pause_flags check to the save_one function. --- xen/arch/x86/hvm/hv

[Xen-devel] [PATCH v15 14/14] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-08-03 Thread Alexandru Isaila
This patch is focused on moving changing hvm_save_one() to save one typecode from one vcpu and now that the save functions get data from a single vcpu we can pause the specific vcpu instead of the domain. Signed-off-by: Alexandru Isaila --- xen/arch/x86/domctl.c | 4 ++-- xen/arch/x86/hvm

[Xen-devel] [PATCH v15 06/14] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v14: - Fix style violations - Use structure fields over cast - Use memcpy for fixed_ranges. Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x8

[Xen-devel] [PATCH v15 11/14] x86/domctl: Use hvm_save_vcpu_handler

2018-08-03 Thread Alexandru Isaila
This patch is aimed on using the new save_one fuctions in the hvm_save Signed-off-by: Alexandru Isaila --- Changes since V14: - Removed the modification from the hvm_save_one - Removed vcpu init - Declared rc as int - Add vcpu id to the log print. --- xen/arch

[Xen-devel] [PATCH v15 00/14] x86/domctl: Save info for one vcpu instance

2018-08-03 Thread Alexandru Isaila
and change the hvm_save_one() func while changing domain_pause to vcpu_pause. Cheers, Alexandru Isaila (14): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce hvm_save_cpu_ctxt_one func x86/hvm: Introduce

[Xen-devel] [PATCH v15 04/14] x86/hvm: Introduce hvm_save_cpu_xsave_states_one

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V14: - Remove err init - Add blank line ahead of return - Move xsave_enabled() check to the save_one func. --- xen/arch/x86/hvm/hvm.c | 47

[Xen-devel] [PATCH v15 02/14] x86/hvm: Introduce hvm_save_tsc_adjust_one() func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V13: - Moved tsc_adjust to the initializer. --- xen/arch/x86/hvm/hvm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/xen/arch/x86/hvm/hvm.c b/xen

[Xen-devel] [PATCH v15 13/14] x86/hvm: Remove redundant save functions

2018-08-03 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V14: - Change vcpu to v - Remove extra space - Rename save_one handler to save

[Xen-devel] [PATCH v15 10/14] x86/hvm: Add handler for save_one funcs

2018-08-03 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila --- Changes since V14: - Change handler name from hvm_save_one_handler to hvm_save_vcpu_handler. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6 +- xen/arch/x86/hvm/i8254.c

[Xen-devel] [PATCH v15 05/14] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant --- Changes since V14: - Remove err init - Add blank line ahead of return. --- xen/arch/x86/hvm/hvm.c | 106 +++-- 1 file

[Xen-devel] [PATCH v15 01/14] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since V11: - Removed the memset and added init with {}. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/xen/arch

[Xen-devel] [PATCH v15 09/14] x86/hvm: Introduce lapic_save_regs_one func

2018-08-03 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- xen/arch/x86/hvm/vlapic.c | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/xen/arch/x86/hvm/vlapic.c b/xen/arch/x86/hvm/vlapic.c index 0795161..d35810e 100644

[Xen-devel] [PATCH v16 12/13] x86/hvm: Remove redundant save functions

2018-08-09 Thread Alexandru Isaila
This patch removes the redundant save functions and renames the save_one* to save. It then changes the domain param to vcpu in the save funcs. Signed-off-by: Alexandru Isaila --- Changes since V15: - Add if for hvm_sr_handlers[i].kind to separate HVMSR_PER_VCPU from

[Xen-devel] [PATCH v16 03/13] x86/hvm: Introduce hvm_save_cpu_ctxt_one func

2018-08-09 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Jan Beulich --- Changes since V14: - Move all free fields to the initializer - Add blank line to before the return - Move v->pause_flags check to the save_one funct

[Xen-devel] [PATCH v16 10/13] x86/hvm: Add handler for save_one funcs

2018-08-09 Thread Alexandru Isaila
Signed-off-by: Alexandru Isaila Reviewed-by: Jan Beulich --- Changes since V14: - Change handler name from hvm_save_one_handler to hvm_save_vcpu_handler. --- xen/arch/x86/cpu/mcheck/vmce.c | 1 + xen/arch/x86/hvm/hpet.c| 2 +- xen/arch/x86/hvm/hvm.c | 6

[Xen-devel] [PATCH v16 08/13] x86/hvm: Introduce lapic_save_hidden_one

2018-08-09 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v15: - Drop struct vlapic *s. --- xen/arch/x86/hvm/vlapic.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/xen/arch/x86/hvm/vlapic.c b/xen/arch

[Xen-devel] [PATCH v16 13/13] x86/domctl: Don't pause the whole domain if only getting vcpu state

2018-08-09 Thread Alexandru Isaila
This patch is focused on moving changing hvm_save_one() to save one typecode from one vcpu and now that the save functions get data from a single vcpu we can pause the specific vcpu instead of the domain. Signed-off-by: Alexandru Isaila --- Changes since V15: - Moved pause/unpause calls

[Xen-devel] [PATCH v16 01/13] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func

2018-08-09 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Jan Beulich --- Changes since V11: - Removed the memset and added init with {}. --- xen/arch/x86/cpu/mcheck/vmce.c | 21 + 1 file changed, 13 insertions(+), 8 deletions

[Xen-devel] [PATCH v16 00/13] x86/domctl: Save info for one vcpu instance

2018-08-09 Thread Alexandru Isaila
and change the hvm_save_one() func while changing domain_pause to vcpu_pause. Cheers, Alexandru Isaila (13): x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func x86/hvm: Introduce hvm_save_tsc_adjust_one() func x86/hvm: Introduce hvm_save_cpu_ctxt_one func x86/hvm: Introduce

[Xen-devel] [PATCH v16 05/13] x86/hvm: Introduce hvm_save_cpu_msrs_one func

2018-08-09 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila Reviewed-by: Paul Durrant Reviewed-by: Jan Beulich --- Changes since V14: - Remove err init - Add blank line ahead of return. --- xen/arch/x86/hvm/hvm.c | 106

[Xen-devel] [PATCH v16 11/13] x86/domctl: Use hvm_save_vcpu_handler

2018-08-09 Thread Alexandru Isaila
This patch is aimed on using the new save_one fuctions in the hvm_save Signed-off-by: Alexandru Isaila --- Changes since V15: - Moved declarations into their scopes - Remove redundant NULL check - Remove rc variable - Change fault return to -ENODATA. --- xen

[Xen-devel] [PATCH v16 06/13] x86/hvm: Introduce hvm_save_mtrr_msr_one func

2018-08-09 Thread Alexandru Isaila
This is used to save data from a single instance. Signed-off-by: Alexandru Isaila --- Changes since v15: - Drop comments - Add BUILD_BUG_ON - memcpy for sizeof(). Note: This patch is based on Roger Pau Monne's series[1] --- xen/arch/x86/hvm/mtrr.c

  1   2   3   >