netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Graf <tgraf@infradead.org>
To: Vladislav Yasevich <vladislav.yasevich@hp.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net,
	Wei Yongjun <yjwei@cn.fujitsu.com>,
	Sridhar Samudrala <sri@us.ibm.com>,
	linux-sctp@vger.kernel.org
Subject: Re: [PATCH] sctp: Enforce maximum retransmissions during shutdown
Date: Thu, 30 Jun 2011 04:49:33 -0400	[thread overview]
Message-ID: <20110630084933.GA24074@canuck.infradead.org> (raw)
In-Reply-To: <4E0B4F71.4020108@hp.com>

On Wed, Jun 29, 2011 at 12:14:41PM -0400, Vladislav Yasevich wrote:
> Right.  The lack of ABORT from the receive of data is a bug.  I was trying to point out
> that instead of modified the sender of data to send the ABORT, you modify the receiver
> to send the ABORT when it is being closed while having data queued.

Agreed. This makes a good procedure if there is data is on
sk_receive_queue and gets us in line with TCP although I don't see this
in the spec at all :-)

> But we don't even get to sending the SHUTDOWN, so from the wire protocol, we
> do not violated it.  We have bad behavior in that when both sender and receiver
> are dead, the association is hung.

So how do we get out if ...

1) there is nothing queued on sk_receive_queue but the window still
   remains 0 forver?
   
2) the receiver is an older Linux without the above fix or another stack
   that does not ABORT?

I agree that using ABORT on the receiver is the ideal way whenver
possible but we still need to fix this if the receiver does not do so.

What sideeffects are you worried about resulting from my proposal?

  reply	other threads:[~2011-06-30  8:49 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-29 13:57 [PATCH] sctp: Enforce maximum retransmissions during shutdown Thomas Graf
2011-06-29 14:20 ` Vladislav Yasevich
2011-06-29 14:36   ` Thomas Graf
2011-06-29 14:58     ` Vladislav Yasevich
2011-06-29 15:48       ` Thomas Graf
2011-06-29 16:14         ` Vladislav Yasevich
2011-06-30  8:49           ` Thomas Graf [this message]
2011-06-30 14:08             ` Vladislav Yasevich
2011-06-30 16:17               ` Thomas Graf
2011-07-04 13:50               ` [PATCHv2] sctp: Enforce retransmission limit " Thomas Graf
2011-07-06  7:24                 ` David Miller
2011-07-06 12:15                 ` Neil Horman
2011-07-06 13:16                   ` Thomas Graf
2011-07-06 14:19                     ` Neil Horman
2011-07-06 13:42                 ` Vladislav Yasevich
2011-07-06 14:18                   ` Thomas Graf
2011-07-06 14:31                     ` Vladislav Yasevich
2011-07-06 15:49                       ` Thomas Graf
2011-07-06 16:23                         ` Vladislav Yasevich
2011-07-06 21:58                           ` Thomas Graf
2011-07-07 10:28                           ` [PATCHv3] " Thomas Graf
2011-07-07 13:36                             ` Vladislav Yasevich
2011-07-07 21:09                               ` David Miller
2011-06-30 13:31           ` [PATCH] sctp: ABORT if receive queue is not empty while closing socket Thomas Graf
2011-06-30 14:11             ` Vladislav Yasevich
2011-06-30 16:19               ` Thomas Graf
2011-06-30 16:27                 ` Vladislav Yasevich
2011-07-08 10:57               ` [PATCHv2] " Thomas Graf
2011-07-08 13:49                 ` Vladislav Yasevich
2011-07-08 14:29                   ` Thomas Graf
2011-07-08 14:37                   ` [PATCHv3] sctp: ABORT if receive, reassmbly, or reodering " Thomas Graf
2011-07-08 16:37                     ` David Miller
2011-07-08 16:43                     ` Vladislav Yasevich
2011-07-08 16:53                       ` David 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=20110630084933.GA24074@canuck.infradead.org \
    --to=tgraf@infradead.org \
    --cc=davem@davemloft.net \
    --cc=linux-sctp@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=sri@us.ibm.com \
    --cc=vladislav.yasevich@hp.com \
    --cc=yjwei@cn.fujitsu.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).