On Wed, Jan 13, 2016 at 05:51:34PM +0100, Martin Kletzander wrote: > On Wed, Jan 13, 2016 at 07:29:46AM -0500, John Ferlan wrote: > >Reposting my cgroup fixes series: > > > >http://www.redhat.com/archives/libvir-list/2016-January/msg00236.html > > > >partially because I originally forgot to CC the author (Henning Schild) > >of the original series for which these patch fix a couple of issues > >discovered during regression testing (virt-test memtune failures in > >Red Hat regression environment), but also to bring them up to date > >with the top of libvirt git. > > > >NB: I did send Henning the changes after the fact, but my resend using > >the same message-id skills so that replies are left in the onlist series > >are lacking. Henning has looked at the first patch - with a response here: > > > >http://www.redhat.com/archives/libvir-list/2016-January/msg00443.html > > > >Finally, I think these changes should go into 1.3.1 since that's when the > >regression was introduced. > > > > It would be nice to have them in, I really tried reviewing them, but I > can't wrap my head around last two of them. Maybe because I'm already > late for an appointment I have. > > So unfortunately I have to leave you without the review for those two as > I really need to go, but anyone else feel free to continue. And even > re-check my reviews for 1 and 2 if you want. It would be a pity not to > fix a regression when we could. I agree we need to get these into the next release, but please hold off from merging them. I want to re-examine Henning's original patches in more detail, as I have a bad feeling we might need to simply revert all of them and start again. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list