search for: qemu_stop_r

...quot;$QEMU" --test > /dev/null \ > > + || return 1 > > + start-stop-daemon --start --quiet --pidfile "$QEMU_PIDFILE" --exec "$QEMU" -- \ > > + $QEMU_ARGS -pidfile "$QEMU_PIDFILE" \ > > + || return 2 > > +} > > + > > +qemu_stop_real() > > +{ > > + start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile "$QEMU_PIDFILE" --name xenconsoled > > + RETVAL="$?" > > + [ "$RETVAL" = 2 ] && return 2 > > + start-stop-daemon --stop --quiet --oknodo --retry=0/...

...2 ] && return 2 - rm -f $PIDFILE + rm -f $XENCONSOLED_PIDFILE return "$RETVAL" } +qemu_start() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_start_real + return $? +} + +qemu_stop() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_stop_real + return $? +} + +qemu_restart() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_stop_real + case "$?" in + 0|1) + qemu_start_real + case "$?" in + 0) ;; + *) return 2 ;; + esac + ;; + *) return 2 ;; + esac + return 0 +} + +qemu_start_real()...

..."$QEMU_PIDFILE" --exec "$QEMU" --test > /dev/null \ > + || return 1 > + start-stop-daemon --start --quiet --pidfile "$QEMU_PIDFILE" --exec "$QEMU" -- \ > + $QEMU_ARGS -pidfile "$QEMU_PIDFILE" \ > + || return 2 > +} > + > +qemu_stop_real() > +{ > + start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile "$QEMU_PIDFILE" --name xenconsoled > + RETVAL="$?" > + [ "$RETVAL" = 2 ] && return 2 > + start-stop-daemon --stop --quiet --oknodo --retry=0/30/KILL/5 --exec "$Q...

...Once I have a bug number for this I will add it to debian/changelog and >>> push the result to feature/bugNNNN as usual. >>> >>> Thanks, >>> Ian. >>> >> <old path removed> >> Not sure this already was handled but the --name argument of qemu_stop_real >> seems a copy-and-paste bug. > > Yes it is, whoops! > >> Playing with it right now, --exec instead of --name >> also works out better since qemu-system-i386 is just about too long. > > So it is, so this is probably a good idea. > > Will you send an u...

...it to debian/changelog and > >>> push the result to feature/bugNNNN as usual. > >>> > >>> Thanks, > >>> Ian. > >>> > >> > <old path removed> > > >> Not sure this already was handled but the --name argument of qemu_stop_real > >> seems a copy-and-paste bug. > > > > Yes it is, whoops! > > > >> Playing with it right now, --exec instead of --name > >> also works out better since qemu-system-i386 is just about too long. > > > > So it is, so this is probably a...

...was > > expecting a change to qemu_start -- did you find that code was OK in the > > end? Or did you end up switching to --exec? > > Errm, that part was the addition I inlined. The updated change was the full diff > between current init script and your changes with the updated qemu_stop_real. > And that might have gone missing in the bug tracker at least. It hopefully has > survived in the cc that went directly... hopefully... It was even in the copy which went via the BTS, I just missed it, sorry! Ian.

...2 ] && return 2 - rm -f $PIDFILE + rm -f $XENCONSOLED_PIDFILE return "$RETVAL" } +qemu_start() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_start_real + return $? +} + +qemu_stop() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_stop_real + return $? +} + +qemu_restart() +{ + [ -x $QEMU ] || return 0 + log_progress_msg "qemu" + qemu_stop_real + case "$?" in + 0|1) + qemu_start_real + case "$?" in + 0) ;; + *) return 2 ;; + esac + ;; + *) return 2 ;; + esac + return 0 +} + +qemu_start_real()...

The contents are the wip.testme branch currently on salsa. I combined the wip.initscript and wip.oxenstored into this and added more things today. I think this is pretty gtg and it's smoke tested (in several cases by scping files around instead of doing package build), so it needs a final extra review and test round before putting it in master branch (which I don't want to force push). I