On 2/3/22 18:56, Vasily Ulyanov wrote:
> Hi Michal,
>
> On 03/02/2022 18:09, Michal Prívozník wrote:
>>
>> Apart from patch 2/3 this looks good. So here's what we are going to do.
>> I've uploaded fixup commits onto my gitlab:
>>
>>
Hi Michal,
On 03/02/2022 18:09, Michal Prívozník wrote:
>
> Apart from patch 2/3 this looks good. So here's what we are going to do.
> I've uploaded fixup commits onto my gitlab:
>
> https://gitlab.com/MichalPrivoznik/libvirt/-/commits/virpidfile/
>
> Please take a look and if you agree I
On 2/2/22 17:28, Vasiliy Ulyanov wrote:
> [v1]
> https://listman.redhat.com/archives/libvir-list/2022-January/msg8.html
> [v2]
> https://listman.redhat.com/archives/libvir-list/2022-January/msg00582.html
>
> As suggesed in the review comments:
> - dropped virFileGetLockOwner;
> - simplified
[v1] https://listman.redhat.com/archives/libvir-list/2022-January/msg8.html
[v2] https://listman.redhat.com/archives/libvir-list/2022-January/msg00582.html
As suggesed in the review comments:
- dropped virFileGetLockOwner;
- simplified lock validation by using VIR_AUTOCLOSE and just trying to