All of lore.kernel.org
 help / color / mirror / Atom feed
From: Oliver Hartkopp <oliver@hartkopp.net>
To: Neil Horman <nhorman@tuxdriver.com>
Cc: netdev@vger.kernel.org
Subject: Re: dropwatch for network drop monitor
Date: Fri, 17 Apr 2009 20:13:58 +0200	[thread overview]
Message-ID: <49E8C6E6.9020601@hartkopp.net> (raw)
In-Reply-To: <20090417172239.GC2723@hmsreliant.think-freely.org>

Neil Horman wrote:

> 
> Quick update for you, I've reproduced the problem.  The inability to restart
> after the first few errors is actually by design.  Since you had to kill the
> user space process, it left the kernel bits in a state where they were still
> monitoring, so a second start command (when you restarted the user space daemon)
> caused an error, and the program shutdown (which I probably shouldn't do,
> returning instead to the command console).  You can work around this by issuing
> a stop command, which successfully stops the kernel bits and lets you restart
> them.

Hard terminations are mostly critical to handle. I'll try the workaround.

> 
> This also tells me that the kernel netlink warning is not fatal (as our netlink
> message commands are getting processed properly).  I still need to figure out
> why thats happening though.
> 

Ok.

> As for the ctrl-c issue, I've got no idea whats going on there, its like the
> recv operation is restarting after we catch the SIGINT.  I think I may just
> switch that to a poll operation on the netlink socket and stdin input.  That
> should fix that up quick.  Expect a git commit on that sometime this weekend,
> and some subseqent netdev posts to fix the kernel bits. 

You can also send me an off-list mail when you have something to test this
weekend.

I can produce a lot of drops on my machine ;-)

Regards,
Oliver


      reply	other threads:[~2009-04-17 18:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-17 12:57 dropwatch for network drop monitor Oliver Hartkopp
2009-04-17 13:28 ` Neil Horman
2009-04-17 14:16   ` Oliver Hartkopp
2009-04-17 14:30     ` Neil Horman
2009-04-17 17:22     ` Neil Horman
2009-04-17 18:13       ` Oliver Hartkopp [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=49E8C6E6.9020601@hartkopp.net \
    --to=oliver@hartkopp.net \
    --cc=netdev@vger.kernel.org \
    --cc=nhorman@tuxdriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.