Re: [PATCH] qemu_agent: report VIR_ERR_AGENT_UNSYNCED if guest_sync failed.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 30.11.2017 13:37, Chen Hanxiao wrote:
> 
> At 2017-11-30 14:11:36, "Nikolay Shirokovskiy" <nshirokovskiy@xxxxxxxxxxxxx> wrote:
>>
>>
>> On 30.11.2017 06:08, Chen Hanxiao wrote:
>>> From: Chen Hanxiao <chenhanxiao@xxxxxxxxx>
>>>
>>> Commit 5e5019bf removed this kind of error.
>>>
>>> But we've already protected race condition by qemuDomainObjEnterAgent,
>>> it was very unlikely to revieve a sync from other calls.
>>> For call from virDomain*, it's better to show this error
>>> if we really got a mismatch sync.
>>
>> Hi, Chen. 
>>
>> We've observed such condition on practice. So if you report error in this
>> place then agent monitor will enter error state and will become unusable
>> and there are no means to leave this state other then restarting libvirtd
>> AFAIK. Even if there is a option to reset monitor then client will reset
>> it by itself on receiving VIR_ERR_AGENT_UNSYNCED error. But then why enter
>> error state in this case?
>>
> 
> Hi, Nikolay 
>  
> My concern is that the mismatch of sync id informs qga is malfunctioning.

Not neccessarily. This can happen due to the way guest channel works.

> If in that state, we SHOULD set agent monitor to error state,
> we'll get VIR_ERR_AGENT_UNRESPONSIVE> 
> The sysadmin should notice this, and login to VMs to do something.
> 
> Restarting qga inside guest will reset the monitor's agentError,
> maybe malfunction of qga will be reset at the same time.
> 
> Regards,
> - Chen
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux