* [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-03 4:20 ` Wei Yongjun
0 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-03 4:20 UTC (permalink / raw)
To: netdev@vger.kernel.org, lksctp, David Miller, Vlad Yasevich
Chunk is marked abandoned if the chunk is expires, and it not be
retransmited even if the peer has no PRSCTP capable, but the peer
will still wait for retransmit it to update CTSN.
This patch disable mark chunk abandoned if peer has no PRSCTP
capable.
Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
---
net/sctp/chunk.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
index 6c85564..0d4832d 100644
--- a/net/sctp/chunk.c
+++ b/net/sctp/chunk.c
@@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
{
struct sctp_datamsg *msg = chunk->msg;
+ if (!chunk->asoc->peer.prsctp_capable)
+ return 0;
+
if (!msg->can_abandon)
return 0;
--
1.6.5.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-03 4:20 ` Wei Yongjun
0 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-03 4:20 UTC (permalink / raw)
To: netdev@vger.kernel.org, lksctp, David Miller, Vlad Yasevich
Chunk is marked abandoned if the chunk is expires, and it not be
retransmited even if the peer has no PRSCTP capable, but the peer
will still wait for retransmit it to update CTSN.
This patch disable mark chunk abandoned if peer has no PRSCTP
capable.
Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
---
net/sctp/chunk.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
index 6c85564..0d4832d 100644
--- a/net/sctp/chunk.c
+++ b/net/sctp/chunk.c
@@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
{
struct sctp_datamsg *msg = chunk->msg;
+ if (!chunk->asoc->peer.prsctp_capable)
+ return 0;
+
if (!msg->can_abandon)
return 0;
--
1.6.5.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP
2011-03-03 4:20 ` Wei Yongjun
@ 2011-03-03 14:14 ` Vladislav Yasevich
-1 siblings, 0 replies; 10+ messages in thread
From: Vladislav Yasevich @ 2011-03-03 14:14 UTC (permalink / raw)
To: Wei Yongjun; +Cc: netdev@vger.kernel.org, lksctp, David Miller
On 03/02/2011 11:20 PM, Wei Yongjun wrote:
> Chunk is marked abandoned if the chunk is expires, and it not be
> retransmited even if the peer has no PRSCTP capable, but the peer
> will still wait for retransmit it to update CTSN.
> This patch disable mark chunk abandoned if peer has no PRSCTP
> capable.
>
> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
> ---
> net/sctp/chunk.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
> index 6c85564..0d4832d 100644
> --- a/net/sctp/chunk.c
> +++ b/net/sctp/chunk.c
> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
> {
> struct sctp_datamsg *msg = chunk->msg;
>
> + if (!chunk->asoc->peer.prsctp_capable)
> + return 0;
> +
> if (!msg->can_abandon)
> return 0;
>
The trouble is that timetolive can be set on a message independent of Partial Reliability.
The difference in behavior is that when PR can't be used, a chunk can only be abandoned
if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
So, you can't blindly disallow abandonment.
-vlad
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-03 14:14 ` Vladislav Yasevich
0 siblings, 0 replies; 10+ messages in thread
From: Vladislav Yasevich @ 2011-03-03 14:14 UTC (permalink / raw)
To: Wei Yongjun; +Cc: netdev@vger.kernel.org, lksctp, David Miller
On 03/02/2011 11:20 PM, Wei Yongjun wrote:
> Chunk is marked abandoned if the chunk is expires, and it not be
> retransmited even if the peer has no PRSCTP capable, but the peer
> will still wait for retransmit it to update CTSN.
> This patch disable mark chunk abandoned if peer has no PRSCTP
> capable.
>
> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
> ---
> net/sctp/chunk.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
> index 6c85564..0d4832d 100644
> --- a/net/sctp/chunk.c
> +++ b/net/sctp/chunk.c
> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
> {
> struct sctp_datamsg *msg = chunk->msg;
>
> + if (!chunk->asoc->peer.prsctp_capable)
> + return 0;
> +
> if (!msg->can_abandon)
> return 0;
>
The trouble is that timetolive can be set on a message independent of Partial Reliability.
The difference in behavior is that when PR can't be used, a chunk can only be abandoned
if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
So, you can't blindly disallow abandonment.
-vlad
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP
2011-03-03 14:14 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Vladislav Yasevich
@ 2011-03-04 5:10 ` Wei Yongjun
-1 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-04 5:10 UTC (permalink / raw)
To: Vladislav Yasevich; +Cc: netdev@vger.kernel.org, lksctp, David Miller
> On 03/02/2011 11:20 PM, Wei Yongjun wrote:
>> Chunk is marked abandoned if the chunk is expires, and it not be
>> retransmited even if the peer has no PRSCTP capable, but the peer
>> will still wait for retransmit it to update CTSN.
>> This patch disable mark chunk abandoned if peer has no PRSCTP
>> capable.
>>
>> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
>> ---
>> net/sctp/chunk.c | 3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
>> index 6c85564..0d4832d 100644
>> --- a/net/sctp/chunk.c
>> +++ b/net/sctp/chunk.c
>> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
>> {
>> struct sctp_datamsg *msg = chunk->msg;
>>
>> + if (!chunk->asoc->peer.prsctp_capable)
>> + return 0;
>> +
>> if (!msg->can_abandon)
>> return 0;
>>
> The trouble is that timetolive can be set on a message independent of Partial Reliability.
> The difference in behavior is that when PR can't be used, a chunk can only be abandoned
> if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
>
> So, you can't blindly disallow abandonment.
But, how can we do PR if peer has no PRSCTP capable?
Return error to application in sendmsg()? If so, how we check this if there is no asoc?
> -vlad
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-04 5:10 ` Wei Yongjun
0 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-04 5:10 UTC (permalink / raw)
To: Vladislav Yasevich; +Cc: netdev@vger.kernel.org, lksctp, David Miller
> On 03/02/2011 11:20 PM, Wei Yongjun wrote:
>> Chunk is marked abandoned if the chunk is expires, and it not be
>> retransmited even if the peer has no PRSCTP capable, but the peer
>> will still wait for retransmit it to update CTSN.
>> This patch disable mark chunk abandoned if peer has no PRSCTP
>> capable.
>>
>> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
>> ---
>> net/sctp/chunk.c | 3 +++
>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>
>> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
>> index 6c85564..0d4832d 100644
>> --- a/net/sctp/chunk.c
>> +++ b/net/sctp/chunk.c
>> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
>> {
>> struct sctp_datamsg *msg = chunk->msg;
>>
>> + if (!chunk->asoc->peer.prsctp_capable)
>> + return 0;
>> +
>> if (!msg->can_abandon)
>> return 0;
>>
> The trouble is that timetolive can be set on a message independent of Partial Reliability.
> The difference in behavior is that when PR can't be used, a chunk can only be abandoned
> if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
>
> So, you can't blindly disallow abandonment.
But, how can we do PR if peer has no PRSCTP capable?
Return error to application in sendmsg()? If so, how we check this if there is no asoc?
> -vlad
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no
2011-03-04 5:10 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Wei Yongjun
@ 2011-03-04 13:08 ` Vlad Yasevich
-1 siblings, 0 replies; 10+ messages in thread
From: Vlad Yasevich @ 2011-03-04 13:08 UTC (permalink / raw)
To: Wei Yongjun; +Cc: netdev@vger.kernel.org, lksctp, David Miller
On Fri, 2011-03-04 at 13:10 +0800, Wei Yongjun wrote:
> > On 03/02/2011 11:20 PM, Wei Yongjun wrote:
> >> Chunk is marked abandoned if the chunk is expires, and it not be
> >> retransmited even if the peer has no PRSCTP capable, but the peer
> >> will still wait for retransmit it to update CTSN.
> >> This patch disable mark chunk abandoned if peer has no PRSCTP
> >> capable.
> >>
> >> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
> >> ---
> >> net/sctp/chunk.c | 3 +++
> >> 1 files changed, 3 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
> >> index 6c85564..0d4832d 100644
> >> --- a/net/sctp/chunk.c
> >> +++ b/net/sctp/chunk.c
> >> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
> >> {
> >> struct sctp_datamsg *msg = chunk->msg;
> >>
> >> + if (!chunk->asoc->peer.prsctp_capable)
> >> + return 0;
> >> +
> >> if (!msg->can_abandon)
> >> return 0;
> >>
> > The trouble is that timetolive can be set on a message independent of Partial Reliability.
> > The difference in behavior is that when PR can't be used, a chunk can only be abandoned
> > if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
> >
> > So, you can't blindly disallow abandonment.
>
> But, how can we do PR if peer has no PRSCTP capable?
That's already taken care of. We report the message as unsent. When
PRSCTP is disabled, you may only abandon messages/chunks that have not
been transmitted (assigned a TSN).
> Return error to application in sendmsg()? If so, how we check this if there is no asoc?
How would you have data without an association?
-vlad
>
>
> > -vlad
> > --
> > To unsubscribe from this list: send the line "unsubscribe netdev" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-04 13:08 ` Vlad Yasevich
0 siblings, 0 replies; 10+ messages in thread
From: Vlad Yasevich @ 2011-03-04 13:08 UTC (permalink / raw)
To: Wei Yongjun; +Cc: netdev@vger.kernel.org, lksctp, David Miller
On Fri, 2011-03-04 at 13:10 +0800, Wei Yongjun wrote:
> > On 03/02/2011 11:20 PM, Wei Yongjun wrote:
> >> Chunk is marked abandoned if the chunk is expires, and it not be
> >> retransmited even if the peer has no PRSCTP capable, but the peer
> >> will still wait for retransmit it to update CTSN.
> >> This patch disable mark chunk abandoned if peer has no PRSCTP
> >> capable.
> >>
> >> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
> >> ---
> >> net/sctp/chunk.c | 3 +++
> >> 1 files changed, 3 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
> >> index 6c85564..0d4832d 100644
> >> --- a/net/sctp/chunk.c
> >> +++ b/net/sctp/chunk.c
> >> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
> >> {
> >> struct sctp_datamsg *msg = chunk->msg;
> >>
> >> + if (!chunk->asoc->peer.prsctp_capable)
> >> + return 0;
> >> +
> >> if (!msg->can_abandon)
> >> return 0;
> >>
> > The trouble is that timetolive can be set on a message independent of Partial Reliability.
> > The difference in behavior is that when PR can't be used, a chunk can only be abandoned
> > if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
> >
> > So, you can't blindly disallow abandonment.
>
> But, how can we do PR if peer has no PRSCTP capable?
That's already taken care of. We report the message as unsent. When
PRSCTP is disabled, you may only abandon messages/chunks that have not
been transmitted (assigned a TSN).
> Return error to application in sendmsg()? If so, how we check this if there is no asoc?
How would you have data without an association?
-vlad
>
>
> > -vlad
> > --
> > To unsubscribe from this list: send the line "unsubscribe netdev" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP
2011-03-04 13:08 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Vlad Yasevich
@ 2011-03-07 7:08 ` Wei Yongjun
-1 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-07 7:08 UTC (permalink / raw)
To: Vlad Yasevich; +Cc: netdev@vger.kernel.org, lksctp, David Miller
> On Fri, 2011-03-04 at 13:10 +0800, Wei Yongjun wrote:
>>> On 03/02/2011 11:20 PM, Wei Yongjun wrote:
>>>> Chunk is marked abandoned if the chunk is expires, and it not be
>>>> retransmited even if the peer has no PRSCTP capable, but the peer
>>>> will still wait for retransmit it to update CTSN.
>>>> This patch disable mark chunk abandoned if peer has no PRSCTP
>>>> capable.
>>>>
>>>> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
>>>> ---
>>>> net/sctp/chunk.c | 3 +++
>>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
>>>> index 6c85564..0d4832d 100644
>>>> --- a/net/sctp/chunk.c
>>>> +++ b/net/sctp/chunk.c
>>>> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
>>>> {
>>>> struct sctp_datamsg *msg = chunk->msg;
>>>>
>>>> + if (!chunk->asoc->peer.prsctp_capable)
>>>> + return 0;
>>>> +
>>>> if (!msg->can_abandon)
>>>> return 0;
>>>>
>>> The trouble is that timetolive can be set on a message independent of Partial Reliability.
>>> The difference in behavior is that when PR can't be used, a chunk can only be abandoned
>>> if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
>>>
>>> So, you can't blindly disallow abandonment.
>> But, how can we do PR if peer has no PRSCTP capable?
> That's already taken care of. We report the message as unsent. When
> PRSCTP is disabled, you may only abandon messages/chunks that have not
> been transmitted (assigned a TSN).
I got it, thanks
>> Return error to application in sendmsg()? If so, how we check this if there is no asoc?
> How would you have data without an association?
>
> -vlad
>
>>
>>> -vlad
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe netdev" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable
@ 2011-03-07 7:08 ` Wei Yongjun
0 siblings, 0 replies; 10+ messages in thread
From: Wei Yongjun @ 2011-03-07 7:08 UTC (permalink / raw)
To: Vlad Yasevich; +Cc: netdev@vger.kernel.org, lksctp, David Miller
> On Fri, 2011-03-04 at 13:10 +0800, Wei Yongjun wrote:
>>> On 03/02/2011 11:20 PM, Wei Yongjun wrote:
>>>> Chunk is marked abandoned if the chunk is expires, and it not be
>>>> retransmited even if the peer has no PRSCTP capable, but the peer
>>>> will still wait for retransmit it to update CTSN.
>>>> This patch disable mark chunk abandoned if peer has no PRSCTP
>>>> capable.
>>>>
>>>> Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
>>>> ---
>>>> net/sctp/chunk.c | 3 +++
>>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>>
>>>> diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
>>>> index 6c85564..0d4832d 100644
>>>> --- a/net/sctp/chunk.c
>>>> +++ b/net/sctp/chunk.c
>>>> @@ -347,6 +347,9 @@ int sctp_chunk_abandoned(struct sctp_chunk *chunk)
>>>> {
>>>> struct sctp_datamsg *msg = chunk->msg;
>>>>
>>>> + if (!chunk->asoc->peer.prsctp_capable)
>>>> + return 0;
>>>> +
>>>> if (!msg->can_abandon)
>>>> return 0;
>>>>
>>> The trouble is that timetolive can be set on a message independent of Partial Reliability.
>>> The difference in behavior is that when PR can't be used, a chunk can only be abandoned
>>> if it has not yet been transmitted. With PR enabled, the chunk can be abandoned at any time.
>>>
>>> So, you can't blindly disallow abandonment.
>> But, how can we do PR if peer has no PRSCTP capable?
> That's already taken care of. We report the message as unsent. When
> PRSCTP is disabled, you may only abandon messages/chunks that have not
> been transmitted (assigned a TSN).
I got it, thanks
>> Return error to application in sendmsg()? If so, how we check this if there is no asoc?
> How would you have data without an association?
>
> -vlad
>
>>
>>> -vlad
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe netdev" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2011-03-07 7:08 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-03 4:20 [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Wei Yongjun
2011-03-03 4:20 ` Wei Yongjun
2011-03-03 14:14 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP Vladislav Yasevich
2011-03-03 14:14 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Vladislav Yasevich
2011-03-04 5:10 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP Wei Yongjun
2011-03-04 5:10 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Wei Yongjun
2011-03-04 13:08 ` [PATCH] sctp: do not mark chunk abandoned if peer has no Vlad Yasevich
2011-03-04 13:08 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Vlad Yasevich
2011-03-07 7:08 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP Wei Yongjun
2011-03-07 7:08 ` [PATCH] sctp: do not mark chunk abandoned if peer has no PRSCTP capable Wei Yongjun
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.