netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Graf <tgraf@suug.ch>
To: Andi Kleen <ak@suse.de>
Cc: netdev@oss.sgi.com
Subject: Re: [RFC] Extend netlink error codes
Date: Sat, 11 Sep 2004 19:57:52 +0200	[thread overview]
Message-ID: <20040911175752.GE21181@postel.suug.ch> (raw)
In-Reply-To: <20040911165056.GQ4431@wotan.suse.de>

* Andi Kleen <20040911165056.GQ4431@wotan.suse.de> 2004-09-11 18:50
> I was thinking about a variable length netlink packet
> that contains it and that is sent back to the application.  

All solutions discussed so far are based on sending a netlink
message back to the application. Changing the existing
nlmsgerr.error or adding a TLV with an error message doesn't
make much difference from an implementors view.

> No need to store it anywhere else.

Not storing it would mean to call netlink_ack from everywhere
which requires knowledge of the original netlink skb and the
original netlink message in order to get the pid and sequence
number right. (We can't use the pid assigned to the peer's
sock.) It would also mean to set a bit to avoid sending out
the ack again. Wrong?

> On the other hand there are not 
> that many sched/cls modules and some minor changes to them
> are probably ok.

True, but also think of xfrm, tcp_diag, netfilter, dnrmg. I haven't
checked how much work it would actually be to improve better
error messages for them but I plan to at least extend all
error codes related to the data passed via a netlink socket.

Will wait for some comments. I guess I would be swamped with
something like discussed above.

  reply	other threads:[~2004-09-11 17:57 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-09-10 22:51 [RFC] Extend netlink error codes Thomas Graf
2004-09-10 23:07 ` David S. Miller
2004-09-11  1:38   ` jamal
2004-09-11 15:49     ` Thomas Graf
2004-09-11 15:58 ` Andi Kleen
2004-09-11 16:24   ` Thomas Graf
2004-09-11 16:50     ` Andi Kleen
2004-09-11 17:57       ` Thomas Graf [this message]
2004-09-11 18:48     ` Sam Leffler
2004-09-11 21:10       ` jamal
2004-09-13 20:36         ` Thomas Graf
2004-09-13 23:28           ` David S. Miller
2004-09-14  2:00           ` jamal
2004-09-12 23:37     ` David S. Miller
2004-09-13  6:26       ` Andi Kleen
2004-09-13 19:33         ` David S. Miller

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=20040911175752.GE21181@postel.suug.ch \
    --to=tgraf@suug.ch \
    --cc=ak@suse.de \
    --cc=netdev@oss.sgi.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).