Re: [PATCH V4 0/5] Introduce Advanced Watch Dog module

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

 




On 2/12/2020 10:56 AM, Jason Wang wrote:
On 2020/2/11 下午4:58, Zhang, Chen wrote:
-----Original Message-----
From: Jason Wang<jasowang@xxxxxxxxxx>
Sent: Monday, January 20, 2020 10:57 AM
To: Zhang, Chen<chen.zhang@xxxxxxxxx>; Paolo Bonzini
<pbonzini@xxxxxxxxxx>; Philippe Mathieu-Daudé<philmd@xxxxxxxxxx>;
qemu-dev<qemu-devel@xxxxxxxxxx>
Cc: Zhang Chen<zhangckid@xxxxxxxxx>
Subject: Re: [PATCH V4 0/5] Introduce Advanced Watch Dog module


On 2020/1/19 下午5:10, Zhang, Chen wrote:
Hi~

Anyone have comments about this module?
Hi Chen:

I will take a look at this series.
Sorry for slow reply due to CNY and extend leave.
OK, waiting your comments~ Thanks~

Two general questions:

- if it can detect more than network stall, it should not belong to /net
This module use network connection status to detect all the issue(Host to Guest/Host to Host/Host to Admin...).
The target is more than network but all use network way. So it is looks a tricky problem.

Ok.


- need to convince libvirt guys for this proposal, since usually it's the duty of
upper layer instead of qemu itself

Yes, It looks a upper layer responsibility, but In the cover latter I have explained the reason why we need this in Qemu.
   try to make this module as simple as possible. This module give upper layer software a new way to connect/monitoring Qemu.
And due to all the COLO code implement in Qemu side, Many customer want to use this FT solution without other dependencies,
it is very easy to integrated to real product.

Thanks
Zhang Chen

I would like to hear from libvirt about such design.


Hi Jason,

OK. I add the libvirt mailing list in this thread.

The full mail discussion and patches:

https://lists.nongnu.org/archive/html/qemu-devel/2020-02/msg02611.html


By the way, I noticed Eric is libvirt maintianer.

Hi Eric and Paolo, Can you give some comments about this series?


Thanks

Zhang Chen



Thanks







[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