From: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
To: Xin Long <lucien.xin@gmail.com>
Cc: network dev <netdev@vger.kernel.org>,
linux-sctp@vger.kernel.org, Neil Horman <nhorman@tuxdriver.com>,
Vlad Yasevich <vyasevich@gmail.com>,
davem@davemloft.net
Subject: Re: [PATCHv5 net-next 0/5] sctp: add sender-side procedures for stream reconf asoc reset and add str
Date: Tue, 24 Jan 2017 13:11:18 +0000 [thread overview]
Message-ID: <20170124131118.GI3781@localhost.localdomain> (raw)
In-Reply-To: <20170123184217.GB3781@localhost.localdomain>
On Mon, Jan 23, 2017 at 04:42:17PM -0200, Marcelo Ricardo Leitner wrote:
> On Sat, Jan 21, 2017 at 06:24:21PM +0800, Xin Long wrote:
> > Patch 3/5 is to implement sender-side procedures for the SSN/TSN Reset
> > Request Parameter described in rfc6525 section 5.1.4, patch 2/5 is
> > ahead of it to define a function to make the request chunk for it.
> >
> > Patch 5/5 is to implement sender-side procedures for the Add Incoming
> > and Outgoing Streams Request Parameter Request Parameter described in
> > rfc6525 section 5.1.5 and 5.1.6, patch 4/5 is ahead of it to define a
> > function to make the request chunk for it.
> >
> > Patch 1/5 is a fix to recover streams states when it fails to send
> > request.
> >
> > v1->v2:
> > - put these into a smaller group.
> > - rename some temporary variables in the codes.
> > - rename the titles of the commits and improve some changelogs.
> > v2->v3:
> > - re-split the patchset and make sure it has no dead codes for review.
> > - move some codes into stream.c from socket.c.
> > v3->v4:
> > - add one more patch to fix a send reset stream request issue.
> > - doing actual work only when request is sent successfully.
> > - reduce some indents in sctp_send_add_streams.
> > v4->v5:
> > - close streams before sending request and recover them when sending
> > fails in patch 1/5 and patch 3/5
>
> To make it clear, the notes on v3 patchset also apply to this one, which
> are:
> - usage of krealloc()
> - the removal of __packed attribute
- the int change too.
>
> So there should be a v6 of it soon.
>
> Thanks
>
> >
> > Xin Long (5):
> > sctp: streams should be recovered when it fails to send request.
> > sctp: add support for generating stream reconf ssn/tsn reset request
> > chunk
> > sctp: implement sender-side procedures for SSN/TSN Reset Request
> > Parameter
> > sctp: add support for generating stream reconf add incoming/outgoing
> > streams request chunk
> > sctp: implement sender-side procedures for Add Incoming/Outgoing
> > Streams Request Parameter
> >
> > include/linux/sctp.h | 12 ++++
> > include/net/sctp/sctp.h | 3 +
> > include/net/sctp/sm.h | 5 ++
> > include/uapi/linux/sctp.h | 8 +++
> > net/sctp/sm_make_chunk.c | 75 +++++++++++++++++++++++++
> > net/sctp/socket.c | 58 +++++++++++++++++++
> > net/sctp/stream.c | 138 +++++++++++++++++++++++++++++++++++++++++++++-
> > 7 files changed, 298 insertions(+), 1 deletion(-)
> >
> > --
> > 2.1.0
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-sctp" 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 linux-sctp" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
WARNING: multiple messages have this Message-ID (diff)
From: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
To: Xin Long <lucien.xin@gmail.com>
Cc: network dev <netdev@vger.kernel.org>,
linux-sctp@vger.kernel.org, Neil Horman <nhorman@tuxdriver.com>,
Vlad Yasevich <vyasevich@gmail.com>,
davem@davemloft.net
Subject: Re: [PATCHv5 net-next 0/5] sctp: add sender-side procedures for stream reconf asoc reset and add streams
Date: Tue, 24 Jan 2017 11:11:18 -0200 [thread overview]
Message-ID: <20170124131118.GI3781@localhost.localdomain> (raw)
In-Reply-To: <20170123184217.GB3781@localhost.localdomain>
On Mon, Jan 23, 2017 at 04:42:17PM -0200, Marcelo Ricardo Leitner wrote:
> On Sat, Jan 21, 2017 at 06:24:21PM +0800, Xin Long wrote:
> > Patch 3/5 is to implement sender-side procedures for the SSN/TSN Reset
> > Request Parameter described in rfc6525 section 5.1.4, patch 2/5 is
> > ahead of it to define a function to make the request chunk for it.
> >
> > Patch 5/5 is to implement sender-side procedures for the Add Incoming
> > and Outgoing Streams Request Parameter Request Parameter described in
> > rfc6525 section 5.1.5 and 5.1.6, patch 4/5 is ahead of it to define a
> > function to make the request chunk for it.
> >
> > Patch 1/5 is a fix to recover streams states when it fails to send
> > request.
> >
> > v1->v2:
> > - put these into a smaller group.
> > - rename some temporary variables in the codes.
> > - rename the titles of the commits and improve some changelogs.
> > v2->v3:
> > - re-split the patchset and make sure it has no dead codes for review.
> > - move some codes into stream.c from socket.c.
> > v3->v4:
> > - add one more patch to fix a send reset stream request issue.
> > - doing actual work only when request is sent successfully.
> > - reduce some indents in sctp_send_add_streams.
> > v4->v5:
> > - close streams before sending request and recover them when sending
> > fails in patch 1/5 and patch 3/5
>
> To make it clear, the notes on v3 patchset also apply to this one, which
> are:
> - usage of krealloc()
> - the removal of __packed attribute
- the int change too.
>
> So there should be a v6 of it soon.
>
> Thanks
>
> >
> > Xin Long (5):
> > sctp: streams should be recovered when it fails to send request.
> > sctp: add support for generating stream reconf ssn/tsn reset request
> > chunk
> > sctp: implement sender-side procedures for SSN/TSN Reset Request
> > Parameter
> > sctp: add support for generating stream reconf add incoming/outgoing
> > streams request chunk
> > sctp: implement sender-side procedures for Add Incoming/Outgoing
> > Streams Request Parameter
> >
> > include/linux/sctp.h | 12 ++++
> > include/net/sctp/sctp.h | 3 +
> > include/net/sctp/sm.h | 5 ++
> > include/uapi/linux/sctp.h | 8 +++
> > net/sctp/sm_make_chunk.c | 75 +++++++++++++++++++++++++
> > net/sctp/socket.c | 58 +++++++++++++++++++
> > net/sctp/stream.c | 138 +++++++++++++++++++++++++++++++++++++++++++++-
> > 7 files changed, 298 insertions(+), 1 deletion(-)
> >
> > --
> > 2.1.0
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-sctp" 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 linux-sctp" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2017-01-24 13:11 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-21 10:24 [PATCHv5 net-next 0/5] sctp: add sender-side procedures for stream reconf asoc reset and add streams Xin Long
2017-01-21 10:24 ` Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 1/5] sctp: streams should be recovered when it fails to send request Xin Long
2017-01-21 10:24 ` Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 2/5] sctp: add support for generating stream reconf ssn/tsn reset request chunk Xin Long
2017-01-21 10:24 ` Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 3/5] sctp: implement sender-side procedures for SSN/TSN Reset Request Parameter Xin Long
2017-01-21 10:24 ` Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 4/5] sctp: add support for generating stream reconf add incoming/outgoing streams Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 4/5] sctp: add support for generating stream reconf add incoming/outgoing streams request chunk Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 5/5] sctp: implement sender-side procedures for Add Incoming/Outgoing Streams Requ Xin Long
2017-01-21 10:24 ` [PATCHv5 net-next 5/5] sctp: implement sender-side procedures for Add Incoming/Outgoing Streams Request Parameter Xin Long
2017-01-23 18:42 ` [PATCHv5 net-next 0/5] sctp: add sender-side procedures for stream reconf asoc reset and add str Marcelo Ricardo Leitner
2017-01-23 18:42 ` [PATCHv5 net-next 0/5] sctp: add sender-side procedures for stream reconf asoc reset and add streams Marcelo Ricardo Leitner
2017-01-24 13:11 ` Marcelo Ricardo Leitner [this message]
2017-01-24 13:11 ` Marcelo Ricardo Leitner
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=20170124131118.GI3781@localhost.localdomain \
--to=marcelo.leitner@gmail.com \
--cc=davem@davemloft.net \
--cc=linux-sctp@vger.kernel.org \
--cc=lucien.xin@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=nhorman@tuxdriver.com \
--cc=vyasevich@gmail.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 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.