All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Graf <tgraf@suug.ch>
To: Dan Williams <dcbw@redhat.com>
Cc: "Denis V. Lunev" <den@sw.ru>,
	netdev@vger.kernel.org, "Denis V. Lunev" <den@openvz.org>,
	"David S. Miller" <davem@davemloft.net>,
	Alexey Kuznetsov <kuznet@ms2.inr.ac.ru>
Subject: Re: Regression in current git - Network Manager fails (bisected)
Date: Tue, 23 Oct 2007 14:11:23 +0200	[thread overview]
Message-ID: <20071023121123.GA8125@postel.suug.ch> (raw)
In-Reply-To: <1193068645.30205.22.camel@localhost.localdomain>

* Dan Williams <dcbw@redhat.com> 2007-10-22 11:57
> On Mon, 2007-10-22 at 13:22 +0400, Denis V. Lunev wrote:
> > We have spent some time with the problem with Alexey and there are no 
> > guesses for now.
> > 
> > Is it possible to name exact version of Network Manager and all 
> > libraries related + provide us an output of strace with full buffers 
> > send/received from netlink. Something like
> >      strace -v -x -s 32768 <nm>
> 
> NM uses netlink in two places; libnl (from Thomas Graf) and some custom
> code for listening for interface up/down events and wireless events.
> 
> It looks like that code comes from libnl's lib/handlers.c where it
> thinks the received message is invalid.
> 
> I'm pretty sure the code that checks carrier status of the device isn't
> libnl code; so maybe the error message (which should get fixed of
> course) isn't in the same path as the link detection.
> 
> The link detection comes from src/nm-netlink-monitor.c, so maybe we
> should look at debugging there.

The patch introduced a change in semantics because it removed the
special ACK handling after a dump was started.

I will look into this.

  reply	other threads:[~2007-10-23 12:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-22  0:58 Regression in current git - Network Manager fails (bisected) Joseph Fannin
2007-10-22  9:22 ` Denis V. Lunev
2007-10-22 15:57   ` Dan Williams
2007-10-23 12:11     ` Thomas Graf [this message]
2007-10-23 13:09       ` Denis V. Lunev
2007-10-23 13:38         ` Thomas Graf
2007-10-23 14:10           ` Dan Williams
2007-10-25  9:06             ` Thomas Graf

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=20071023121123.GA8125@postel.suug.ch \
    --to=tgraf@suug.ch \
    --cc=davem@davemloft.net \
    --cc=dcbw@redhat.com \
    --cc=den@openvz.org \
    --cc=den@sw.ru \
    --cc=kuznet@ms2.inr.ac.ru \
    --cc=netdev@vger.kernel.org \
    /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.