From: Alex Pankratov <ap@swapped.cc>
To: hadi@cyberus.ca
Cc: netdev@oss.sgi.com
Subject: Re: ENOBUFS and dev_queue_xmit()
Date: Tue, 15 Jun 2004 08:39:27 -0700 [thread overview]
Message-ID: <40CF182F.4060401@swapped.cc> (raw)
In-Reply-To: <1087304060.1043.72.camel@jzny.localdomain>
jamal wrote:
> On Tue, 2004-06-15 at 00:56, Alex Pankratov wrote:
>
>>I've been poking around rather weird problem today where send()
[snip]
>>meaning that the device's queue was stopped. The comment there
>>implies that only a broken virtual device may end up $here,
>
>
> How did you end up there with a real phy device?? Are you trying to
> circumvent the qdisc subsystem? If yes, you are responsible for how
> all this gets handled.
Now that I looked at dev.c code again I ask myself the very same
questions :) I am not circumventing qdisc, at least intentionally,
and I don't do anything fancy with dev->qdisc. It must be a bug
due to some of my changes, ignore the original question. Thanks
for your help.
>>
>>Is this a known (pseudo?) issue ? ENOBUFS makes much more sense
>>in this context. I can certainly check interface status (IFF_UP)
>>on every ENETDOWN to see what's the real cause, but that's kind
>>of ugly.
>
> Did you mean when no space is left in the ring? Thats different
> from ENOBUFF. If not, not sure i see how a driver xmit path gets
> involved in kmallocing.
> Look at the return code the driver returns. In case of a full ring, it
> should return a busy signal and the top layer will retry later.
> You dont have to worry about any of that if you are running the
> std linux semantics, of course. I have a feeling you have attempted to
> bypass it otherwise the question becomes: how you even ended in this
> situation.
>
I'm pretty sure you're right about breaking the semantics. I'll check
it out, and re-complain if it's not my problem. Thanks again.
prev parent reply other threads:[~2004-06-15 15:39 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-06-15 4:56 ENOBUFS and dev_queue_xmit() Alex Pankratov
2004-06-15 12:54 ` jamal
2004-06-15 15:39 ` Alex Pankratov [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=40CF182F.4060401@swapped.cc \
--to=ap@swapped.cc \
--cc=hadi@cyberus.ca \
--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).