[ovs-dev] [PATCH] ovs-lib: Add timeout at ovs-check-dead-ifs.
Guru Shetty
guru at ovn.org
Tue Aug 27 17:21:09 UTC 2019
On Fri, 23 Aug 2019 at 11:29, William Tu <u9012063 at gmail.com> wrote:
> At SUSE12 SP3, we hit a case where ovs-check-dead-ifs tries to read
> an entry in /proc/<pid>/fd/<some fd> but hangs forever. The pid is
> a qemu-system-x86_64 process and we suspect it's an issue related to
> qemu, not ovs. As a result, force-reload-kmod hangs and OVS bridge
> never gets restarted. This patch adds a timeout of 5-seconds to
> ovs-check-dead-ifs.
>
> VMware-BZ: #2408059
> Signed-off-by: William Tu <u9012063 at gmail.com>
> Cc: Ashish Varma <ashishvarma.ovs at gmail.com>
> Cc: Gurucharan Shetty <guru at ovn.org>
> ---
> utilities/ovs-lib.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in
> index fa840ec637f5..4cdc2a6d5d22 100644
> --- a/utilities/ovs-lib.in
> +++ b/utilities/ovs-lib.in
> @@ -653,7 +653,7 @@ force_reload_kmod () {
>
> restore_interfaces
>
> - "$datadir/scripts/ovs-check-dead-ifs"
> + timeout 5 "$datadir/scripts/ovs-check-dead-ifs"
>
If there is a timeout, the exit status of the "force-reload-kmod" command
will be non-zero. That may fail some installations.
How about:
action "Finding processes on dead interfaces" timeout 5
"$datadir/scripts/ovs-check-dead-ifs" || true
}
>
> ## ------- ##
> --
> 2.7.4
>
>
More information about the dev
mailing list