Change in libosmocore[master]: vty: add "shutdown"

osmith gerrit-no-reply at
Thu Jul 15 07:55:14 UTC 2021

osmith has posted comments on this change. ( )

Change subject: vty: add "shutdown"

Patch Set 3:

> Patch Set 2:
> > Patch Set 2:
> > 
> > > Patch Set 1:
> > > 
> > > Why not simply sending some signal() to the process itself and let usual SIGINT or alike take the usual exit path?
> > 
> > I've tried it, but found that it doesn't nicely service the pending writes before quitting, at least with the programs I've tested. So I did a combination of both that doesn't need setting the variable in every loop.
> I would really avoid having a separate shutdown procedure/steps done when triggering by VTY. If the app doesn't support proper shutdown in usual case, I think it's dangerous doing so through VTY. So I'd vote for going the way I mentioned (sending signal SIGTERM or similar), and if specific apps don't behave as expected, then fix the usual shutdown path (SIGTERM or alike, usually sent by user or systemd).

good point, updated.

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I3596a35b0f4a24e49484ec3f24a98e4d4f583e1e
Gerrit-Change-Number: 24926
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at>
Gerrit-CC: pespin <pespin at>
Gerrit-Comment-Date: Thu, 15 Jul 2021 07:55:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list