#13 [2/3] Signal patch

closed-accepted
nobody
None
5
2005-01-02
2004-06-24
No

This patch adds SIGUSR1 and SIGUSR2 for sending only one signal
to the first bandwidthd process (which will send SIGTERM or SIGHUP
to all bandwidthd processes) to get rid of the requirement to use
"killall".

Please read the top of the patch-file for more information.

Regards,
Andreas Henriksson
andreas at fjortis.info

Discussion

  • Andreas Henriksson

    add signals patch

     
    Attachments
  • David R Hinkle

    David R Hinkle - 2004-08-02

    Logged In: YES
    user_id=863707

    I don't like the use of sigusr1 and sigusr2. I'd rather
    bandwidthd's signaling be consistant than backwards
    compatible. Please modify this so that you kill bandwidthd
    and all children with a SIGTERM to the main process, and
    that you rotate all the logs with a SIGHUP. Feel free to
    just stub in the log rotatation for the children, you don't
    need to actually code it unless you feel like it.

     
  • Andreas Henriksson

    Logged In: YES
    user_id=645255

    I've made a new signal patch....

    As usual some comments at the top of the diff, most
    importantly: do NOT use "killall bandwithd" after applying this
    patch.
    ("killall bandwithd" should also be removed from the
    documentation.)

    Regards,
    Andreas Henriksson
    andreas at fjortis.info

     
  • David R Hinkle

    David R Hinkle - 2005-01-02

    Logged In: YES
    user_id=863707

    This patch has been merged with some minor changes,

    Thankyou again Andreas,
    David

     
  • David R Hinkle

    David R Hinkle - 2005-01-02
    • status: open --> closed-accepted
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks