Re: [Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-19 Thread Julien Grall
Hi Ian, On 18/05/17 19:11, Ian Jackson wrote: Julien Grall writes ("Re: [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes"): On 17/05/17 15:02, Julien Grall wrote: For the last patch, at this stage of the release I would prefer to defer it for Xen 4.10. After reviewing these, I'd like to make a ca

Re: [Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-18 Thread Ian Jackson
Julien Grall writes ("Re: [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes"): > On 17/05/17 15:02, Julien Grall wrote: > > For the last patch, at this stage of the release I would prefer to defer > > it for Xen 4.10. After reviewing these, I'd like to make a case for the third patch for 4.9: I haven't

Re: [Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-18 Thread Wei Liu
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 Rog

Re: [Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-18 Thread Julien Grall
(CC Ian and Wei) 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 subcomman

Re: [Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-17 Thread Julien Grall
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

[Xen-devel] [PATCH v3 for-4.9 0/3] libxl/devd: bugfixes

2017-05-16 Thread Roger Pau Monne
Hello, 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