From: Wang Weidong <wangweidong1@xxxxxxxxxx>
On 2013/12/10 22:38, Vlad Yasevich wrote:
On 12/10/2013 09:34 AM, Wang Weidong wrote:
From: Wang Weidong <wangweidong1@xxxxxxxxxx>
On 2013/12/10 20:51, Daniel Borkmann wrote:
On 12/10/2013 03:46 AM, Wang Weidong wrote:
v5 -> v6
-patch1: do rto_min/max socket option handling in its own patch, and
fix the check of rto_min/max.
-patch2: do rto_min/max sysctl handling in its own patch.
-patch3: add Suggested-by Daniel.
Fyi, for future submission, please keep the full changelog.
Yeah, Thanks! I will do this in future submission.
I would have much rather liked seeing you with finishing the last
2 patches first, and then approach the newly introduced 1st one
in this series for now. You still haven't fully fixed whitespace
issues in your second patch in the function itself which I hoped
you would address, but fair enough ...
As Vlad's suggestion, I split it to 2 patches. Should I combine these
2 patch to one?
No, they deserve a separate patch. What Daniel meant was that you
should have addressed the sysctl control issues first. It doesn't
really matter all that much here. In fact the last whitespace patch
is the one that doesn't really fit in this series as it fixes something
quite unrelated to the reset of the series, but we are talking semantics.
I am fine with the series as long as you address the spacing issue
in the new proc handler functions.
Nice, I will fix them in v7.
Regards.
Wang
The second patch, do you mean that?
I should do it from
+static int proc_sctp_do_rto_min(struct ctl_table *ctl, int write,
+ void __user *buffer, size_t *lenp,
+ loff_t *ppos);
+static int proc_sctp_do_rto_max(struct ctl_table *ctl, int write,
+ void __user *buffer, size_t *lenp,
+ loff_t *ppos);
to
+static int proc_sctp_do_rto_min(struct ctl_table *ctl, int write,
+ void __user *buffer, size_t *lenp,
+ loff_t *ppos);
+static int proc_sctp_do_rto_max(struct ctl_table *ctl, int write,
+ void __user *buffer, size_t *lenp,
+ loff_t *ppos);
Just line up to first line? And I should change the third patch as well.
No, the prototypes are fine. I sent a separate comment pointing out
the spacing issue.
-vlad
Regards.
Wang
Wang Weidong (3):
sctp: check the rto_min and rto_max in setsockopt
sctp: add check rto_min and rto_max in sysctl
sctp: fix up a spacing
net/sctp/socket.c | 32 +++++++++++++++--------
net/sctp/sysctl.c | 76
++++++++++++++++++++++++++++++++++++++++++++++++-------
2 files changed, 89 insertions(+), 19 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
