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

Re: [Xen-devel] [PATCH] common/vm_event: Initialize vm_event lists on domain creation



>>> Andrew Cooper <andrew.cooper3@xxxxxxxxxx> 06/26/17 5:11 PM >>>
>There is also a difference in timing; vm_event_init_domain() is called
>when vm_event is started on the domain, not when the domain is
>constructed.  IMO, the two should happen at the same time to be
>consistent.  I'm not fussed at which point, as it would be fine (in
>principle) for d->vm_event to be NULL in most cases.

I very much support that last aspect - there's shouldn't be any memory
allocated here if the domain isn't going to make use of VM events.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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