Re: [PATCH] init/FreeBSD: fix incorrect usage of $rc_pids in xendriverdomain

[Wei Liu <wei.liu2@citrix.com>]

On Wed, Dec 21, 2016 at 04:47:26PM +0000, Roger Pau Monne wrote:
> It should be rc_pid.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
> Reported-by: Nathan Friess <nathan.friess@gmail.com>

Acked + applied.

I suspect all the patches to fix FreeBSD init script should be
backported?

> ---
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> Cc: Wei Liu <wei.liu2@citrix.com>
> ---
>  tools/hotplug/FreeBSD/rc.d/xendriverdomain.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/hotplug/FreeBSD/rc.d/xendriverdomain.in b/tools/hotplug/FreeBSD/rc.d/xendriverdomain.in
> index b01a470..a032822 100644
> --- a/tools/hotplug/FreeBSD/rc.d/xendriverdomain.in
> +++ b/tools/hotplug/FreeBSD/rc.d/xendriverdomain.in
> @@ -38,8 +38,8 @@ xendriverdomain_stop()
>  
>  	rc_pid=$(check_pidfile ${XLDEVD_PIDFILE} ${sbindir}/xl)
>  
> -	kill -${sig_stop:-TERM} $rc_pids
> -	wait_for_pids $rc_pids
> +	kill -${sig_stop:-TERM} $rc_pid
> +	wait_for_pids $rc_pid
>  }
>  
>  run_rc_command "$1"
> -- 
> 2.10.1 (Apple Git-78)
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel