Linux Advanced Routing and Traffic Control list
 help / color / mirror / Atom feed
From: "Michael T. Babcock" <mbabcock@fibrespeed.net>
To: lartc@vger.kernel.org
Subject: Re: [LARTC] RTNETLINK (long reply)
Date: Tue, 27 Nov 2001 02:02:33 +0000	[thread overview]
Message-ID: <marc-lartc-100694039003018@msgid-missing> (raw)

On Mon, Nov 26, 2001 at 08:12:44PM -0500, Whit Blauvelt wrote:
> > RTNETLINK answers: File exists
> 
> Let me be clear about this: anything that can be deleted, should be able to
> be added. The "File exists" message is quite wrong, since no file creation

The message text is wrong, but it you need to first understand that 
most things in Unix are treated like files, whether you realise it or 
not.  Shared memory is now treated like a set of files even and when 
you fail to create an 'object' (using a file descriptor), you get an 
error number 17 (from errno.h) which is defined as EEXIST or "File 
exists".  If the author of the code simply requested the text for the 
error message in question, "File exists" is what would be returned (do 
a man strerror or perror).

In this particular case, and probably others, the code could be edited 
to specifically handle the EEXIST condition and instead return 
"RTNETLINK answers: Route exists".  Also, the code returning this error 
should only return the error if in fact the route already exists.  If 
it can't be created for another reason, a different error should be 
returned.

> Has Linus really allowed such nonsense to be integrated with the kernel?

Do you read C code?  If not, Linus' decisions are probably quite far 
from your grasp.  I appreciate your problem understanding the error in 
question, but its quite obvious to people who work with this code all 
the time, even though it should probably be revised to be friendly to 
others as well.

> if the implementation is this lame, it shouldn't have been integrated at
> all.

That's your call.  I understand for the most part it so I'm much 
happier having it integrated than not.
-- 
Michael T. Babcock
CTO, FibreSpeed Ltd.
http://www.fibrespeed.net/~mbabcock/

_______________________________________________
LARTC mailing list / LARTC@mailman.ds9a.nl
http://mailman.ds9a.nl/mailman/listinfo/lartc HOWTO: http://ds9a.nl/2.4Routing/

                 reply	other threads:[~2001-11-27  2:02 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=marc-lartc-100694039003018@msgid-missing \
    --to=mbabcock@fibrespeed.net \
    --cc=lartc@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox