On 10/12/2018 10:27 PM, John Ferlan wrote:
[...]
402 virResctrlMonitorDispose(void *obj) 403 { 404 virResctrlMonitorPtr monitor = obj; 405 406 virObjectUnref(monitor->alloc); 407 VIR_FREE(monitor->id); 408 VIR_FREE(monitor->path); 409 }
The one "thing" about the order of patches here is that it forces me to look forward to ensure decisions made in previous patches will be handled in the future. Maybe combine virResctrlMonitorDispose and virResctrlMonitorCreate in one patch? Will that make you look better for understanding how @monitor->alloc is used.
That usually is best. I think the order has me off a bit too, but there is no perfect solution for order. I find myself looking forward a lot and trying to keep track.
I'll change the order of the series to fix this.
John
Thanks for review. Huaqiang