Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-24 Thread Tamas K Lengyel
On Wed, Jun 24, 2020 at 6:40 AM Andrew Cooper wrote: > > On 24/06/2020 11:03, Jan Beulich wrote: > > On 23.06.2020 19:24, Andrew Cooper wrote: > >> On 23/06/2020 09:51, Jan Beulich wrote: > >>> On 23.06.2020 03:04, Michał Leszczyński wrote: > - 22 cze 2020 o 18:16, Jan Beulich jbeul...@su

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-24 Thread Andrew Cooper
On 24/06/2020 11:03, Jan Beulich wrote: > On 23.06.2020 19:24, Andrew Cooper wrote: >> On 23/06/2020 09:51, Jan Beulich wrote: >>> On 23.06.2020 03:04, Michał Leszczyński wrote: - 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): > On 22.06.2020 18:02, Michał Leszczyń

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-24 Thread Michał Leszczyński
- 23 cze 2020 o 19:24, Andrew Cooper andrew.coop...@citrix.com napisał(a): > On 23/06/2020 09:51, Jan Beulich wrote: >> I'd still like to see an explicit confirmation by him that this >> use of memory is indeed what he has intended. There are much smaller >> amounts of memory which we allocate

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-24 Thread Jan Beulich
On 23.06.2020 19:24, Andrew Cooper wrote: > On 23/06/2020 09:51, Jan Beulich wrote: >> On 23.06.2020 03:04, Michał Leszczyński wrote: >>> - 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): >>> On 22.06.2020 18:02, Michał Leszczyński wrote: > - 22 cze 2020 o 17:22, Jan

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-23 Thread Andrew Cooper
On 23/06/2020 09:51, Jan Beulich wrote: > On 23.06.2020 03:04, Michał Leszczyński wrote: >> - 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): >> >>> On 22.06.2020 18:02, Michał Leszczyński wrote: - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): > On 2

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-23 Thread Jan Beulich
On 23.06.2020 03:04, Michał Leszczyński wrote: > - 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): > >> On 22.06.2020 18:02, Michał Leszczyński wrote: >>> - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): On 22.06.2020 16:35, Michał Leszczyński wrote: >>>

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-23 Thread Jan Beulich
On 22.06.2020 19:05, Michał Leszczyński wrote: >> +struct xen_hvm_vmtrace_op { >> +/* IN variable */ >> +uint32_t version; /* HVMOP_VMTRACE_INTERFACE_VERSION */ >> +uint32_t cmd; >> +/* Enable/disable external vmtrace for given domain */ >> +#define HVMOP_vmtra

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
- 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): > On 22.06.2020 18:02, Michał Leszczyński wrote: >> - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): >>> On 22.06.2020 16:35, Michał Leszczyński wrote: - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
> +struct xen_hvm_vmtrace_op { > +/* IN variable */ > +uint32_t version; /* HVMOP_VMTRACE_INTERFACE_VERSION */ > +uint32_t cmd; > +/* Enable/disable external vmtrace for given domain */ > +#define HVMOP_vmtrace_ipt_enable 1 > +#define HVMOP_vmtrace_ipt

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
- 22 cze 2020 o 18:25, Roger Pau Monné roger@citrix.com napisał(a): > On Mon, Jun 22, 2020 at 06:16:57PM +0200, Jan Beulich wrote: >> On 22.06.2020 18:02, Michał Leszczyński wrote: >> > - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): >> >> On 22.06.2020 16:35, Michał L

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Roger Pau Monné
On Mon, Jun 22, 2020 at 06:16:57PM +0200, Jan Beulich wrote: > On 22.06.2020 18:02, Michał Leszczyński wrote: > > - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): > >> On 22.06.2020 16:35, Michał Leszczyński wrote: > >>> - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse.com n

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
- 22 cze 2020 o 18:16, Jan Beulich jbeul...@suse.com napisał(a): > On 22.06.2020 18:02, Michał Leszczyński wrote: >> - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): >>> On 22.06.2020 16:35, Michał Leszczyński wrote: - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Jan Beulich
On 22.06.2020 18:02, Michał Leszczyński wrote: > - 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): >> On 22.06.2020 16:35, Michał Leszczyński wrote: >>> - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse.com napisał(a): Is any of what you do in this switch() actually legit

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
- 22 cze 2020 o 17:22, Jan Beulich jbeul...@suse.com napisał(a): > On 22.06.2020 16:35, Michał Leszczyński wrote: >> - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse.com napisał(a): >>> On 19.06.2020 01:41, Michał Leszczyński wrote: + +domain_pause(d); >>> >>> Who's the inten

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Jan Beulich
On 22.06.2020 16:35, Michał Leszczyński wrote: > - 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse.com napisał(a): >> On 19.06.2020 01:41, Michał Leszczyński wrote: >>> + >>> +domain_pause(d); >> >> Who's the intended caller of this interface? You making it a hvm-op >> suggests the guest may

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Michał Leszczyński
- 22 cze 2020 o 15:25, Jan Beulich jbeul...@suse.com napisał(a): > On 19.06.2020 01:41, Michał Leszczyński wrote: >> + >> +domain_pause(d); > > Who's the intended caller of this interface? You making it a hvm-op > suggests the guest may itself call this. But of course a guest > can't paus

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Jan Beulich
On 19.06.2020 01:41, Michał Leszczyński wrote: > @@ -1631,6 +1649,8 @@ void hvm_vcpu_destroy(struct vcpu *v) > vlapic_destroy(v); > > hvm_vcpu_cacheattr_destroy(v); > + > +hvm_vmtrace_destroy(v); > } Whenever possible resource cleanup should occur from hvm_domain_relinquish_resour

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-22 Thread Jan Beulich
On 22.06.2020 04:56, Michał Leszczyński wrote: > - 19 cze 2020 o 1:41, Michał Leszczyński michal.leszczyn...@cert.pl > napisał(a): >> --- a/xen/arch/x86/hvm/vmx/vmx.c >> +++ b/xen/arch/x86/hvm/vmx/vmx.c >> @@ -508,11 +508,25 @@ static void vmx_restore_host_msrs(void) >> >> static void vmx_save

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-21 Thread Michał Leszczyński
- 19 cze 2020 o 1:41, Michał Leszczyński michal.leszczyn...@cert.pl napisał(a): > Provide an interface for privileged domains to manage > external IPT monitoring. Guest IPT state will be preserved > across vmentry/vmexit using ipt_state structure. > > Signed-off-by: Michal Leszczynski > ---

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-21 Thread Michał Leszczyński
- 19 cze 2020 o 17:50, Jan Beulich jbeul...@suse.com napisał(a): > On 19.06.2020 17:30, Roger Pau Monné wrote: >> On Fri, Jun 19, 2020 at 01:41:03AM +0200, Michał Leszczyński wrote: >>> --- a/xen/arch/x86/hvm/hvm.c >>> +++ b/xen/arch/x86/hvm/hvm.c >>> @@ -1612,6 +1612,24 @@ int hvm_vcpu_initia

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-19 Thread Jan Beulich
On 19.06.2020 17:30, Roger Pau Monné wrote: > On Fri, Jun 19, 2020 at 01:41:03AM +0200, Michał Leszczyński wrote: >> --- a/xen/arch/x86/hvm/hvm.c >> +++ b/xen/arch/x86/hvm/hvm.c >> @@ -1612,6 +1612,24 @@ int hvm_vcpu_initialise(struct vcpu *v) >> return rc; >> } >> >> +void hvm_vmtrace_dest

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-19 Thread Roger Pau Monné
On Fri, Jun 19, 2020 at 01:41:03AM +0200, Michał Leszczyński wrote: > Provide an interface for privileged domains to manage > external IPT monitoring. Guest IPT state will be preserved > across vmentry/vmexit using ipt_state structure. Thanks! I have some comments below, some of them are cosmetic

Re: [PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-18 Thread Michał Leszczyński
- 19 cze 2020 o 1:41, Michał Leszczyński michal.leszczyn...@cert.pl napisał(a): > Provide an interface for privileged domains to manage > external IPT monitoring. Guest IPT state will be preserved > across vmentry/vmexit using ipt_state structure. > > Signed-off-by: Michal Leszczynski > ---

[PATCH v2 4/7] x86/vmx: add do_vmtrace_op

2020-06-18 Thread Michał Leszczyński
Provide an interface for privileged domains to manage external IPT monitoring. Guest IPT state will be preserved across vmentry/vmexit using ipt_state structure. Signed-off-by: Michal Leszczynski --- xen/arch/x86/hvm/hvm.c | 167 + xen/arch/x86/hvm/vmx/vmx