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.
next prev parent 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).