Re: [libvirt] [PATCH 1/1] vz: fix raise in vzDomainBlock

2017-05-24 Thread Michal Privoznik
On 05/03/2017 04:10 PM, Nikolay Shirokovskiy wrote: > > > On 03.05.2017 13:44, Konstantin Neumoin wrote: >> Need begin job before lookup disk in config, >> because it can be edited at this moment. > > I would slightly change commit message to something like: > > Put domain access after

Re: [libvirt] [PATCH 1/1] vz: fix raise in vzDomainBlock

2017-05-03 Thread Konstantin Neumoin
On 05/03/2017 05:10 PM, Nikolay Shirokovskiy wrote: On 03.05.2017 13:44, Konstantin Neumoin wrote: Need begin job before lookup disk in config, because it can be edited at this moment. I would slightly change commit message to something like: OK -- libvir-list mailing list

Re: [libvirt] [PATCH 1/1] vz: fix raise in vzDomainBlock

2017-05-03 Thread Nikolay Shirokovskiy
On 03.05.2017 13:44, Konstantin Neumoin wrote: > Need begin job before lookup disk in config, > because it can be edited at this moment. I would slightly change commit message to something like: Put domain access after acquiring job condition, otherwise another job can change it meanwhile.

[libvirt] [PATCH 1/1] vz: fix raise in vzDomainBlock

2017-05-03 Thread Konstantin Neumoin
Need begin job before lookup disk in config, because it can be edited at this moment. Signed-off-by: Konstantin Neumoin --- src/vz/vz_driver.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c index