Hi Julian, Thanks a lot for the comments. I will send a v2 patch. Regards Hangbin 2017-02-19 22:04 GMT+08:00 Julian Anastasov <ja@xxxxxx>: > > Hello, > > On Fri, 17 Feb 2017, Hangbin Liu wrote: > >> Fix and add some new ipvs sysctl options. Also use tab instead of whitespaces >> to indent the descriptions. > > You change only docs, so the patchset should be for > the net-next tree. > >> Hangbin Liu (4): >> ipvs: fix sync_threshold description and add sync_refresh_period, >> sync_retries >> ipvs: Document sysctl sync_qlen_max and sync_sock_size > > - we can add "are " here but then also check if lines become too long: > > "Hard limit for queued sync messages that [are ]not sent yet." > > You can also move the "commit" word to next line > to make checkpatch happy: > > scripts/checkpatch.pl --strict /tmp/file2.patch > >> ipvs: Document sysctl sync_ports >> ipvs: Document sysctl pmtu_disc > > - we can make it like this: > > "By default, reject with FRAG_NEEDED all DF packets that exceed > the PMTU, irrespective of the forwarding method. For TUN method > the flag can be disabled to fragment such packets." > >> Documentation/networking/ipvs-sysctl.txt | 68 ++++++++++++++++++++++++++++---- >> 1 file changed, 60 insertions(+), 8 deletions(-) > > Regards > > -- > Julian Anastasov <ja@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
