[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V5] vm_event: Allow subscribing to write events for specific MSR-s

> @@ -55,6 +61,9 @@ void vm_event_cleanup_domain(struct domain *d)
>          v->arch.vm_event = NULL;
>      }
> +    xfree(d->arch.monitor_msr_bitmap);
> +    d->arch.monitor_msr_bitmap = NULL;
> +
>      d->arch.mem_access_emulate_each_rep = 0;
>      memset(&d->arch.monitor, 0, sizeof(d->arch.monitor));
>      memset(&d->monitor, 0, sizeof(d->monitor));

Aside from accidentally duplicating the alloc() / free() code here, I
should have probably moved the following monitor-related memset()s in
arch_monitor_init_domain() as well. I'll do that in V6.

Yes, +1 for moving the memsets to the new init function as well. It may be cleaner if you break this patch into two, one where you introduce the new monitor init and move these memsets there, and then do the msr patch.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.