On Thu, May 18, 2017 at 04:06:33PM +0100, Julien Grall wrote: > (CC Ian and Wei) >
I did see this. Ian said he wanted to review this in detail. That's why I haven't committed them. > On 17/05/17 15:02, Julien Grall wrote: > > On 16/05/17 08:59, Roger Pau Monne wrote: > > > Hello, > > > > Hi Roger, > > > > > > > > The first two patches in the series fix a race with concurrent device > > > addition/removal and two bugs related to manipulation of the list of > > > active > > > domains in the devd subcommand. The last patch is not a bugfix itself, > > > but > > > it makes the code easier to understand. > > > > > > IMHO they should be part of 4.9 because they are confined to devd > > > code, and > > > without them devd is unusable (it's trivial to segfault it), so the > > > risk is > > > low. Worse thing that could happen is that devd crashes, which is > > > already the > > > case without them. > > > > For the first 2 patches: > > > > Release-acked-by: Julien Grall <julien.gr...@arm.com> > > > > For the last patch, at this stage of the release I would prefer to defer > > it for Xen 4.10. > > > > Cheers, > > > > -- > Julien Grall > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > https://lists.xen.org/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
