From: "Bjørn Mork" <bjorn@mork•no>
To: Cong Wang <amwang@redhat•com>
Cc: David Miller <davem@davemloft•net>, netdev@vger•kernel.org
Subject: Re: [Patch net-next v9 00/11] vxlan: add ipv6 support
Date: Tue, 28 May 2013 09:28:53 +0200 [thread overview]
Message-ID: <87vc63a7gq.fsf@nemi.mork.no> (raw)
In-Reply-To: <1369708978.8181.39.camel@cr0> (Cong Wang's message of "Tue, 28 May 2013 10:42:58 +0800")
Cong Wang <amwang@redhat•com> writes:
> On Mon, 2013-05-27 at 15:07 +0200, Bjørn Mork wrote:
>
>> Give me one reason why I should bother even looking at your next
>> submission. Just to make it clear: No comments from me the next time
>> you post this set only means that I didn't test it. Which seems
>> likely given your ability to take any feedback at all.
>
> You said "I see that you don't bother listen to advice", this is
> OBVIOUSLY not true, I really did compile with three different configs,
> just due a stupid mistake I took my IPV6=n config as IPV6=m config.
I do understand that mistakes can happen, but this error does not change
the fact that the patchset was posted without possibly being tested with
either IPV6=m or IPV6=y. I do find that extremely odd, and really
unexpected. The advice you got was
Doing different combinations of buildtime and runtime IPv6
enable/disable is an obvious requirement when you add code which has
some sort of "if IPv6 is enabled" conditionals.
ref http://permalink.gmane.org/gmane.linux.network/268130
If you never successfully built your patchset with IPv6 support, then
you obviously never attempted any runtime testing at all. It doesn't
matter whether the missing builds were unintentional or not. If you
followed the advice you got, then you would have noticed. So, sorry,
it is obvious that you did not follow this advice. But that doesn't say
anything about whether you intended to follow it.
But I do take your work that you intended to, and tried to, follow the
advice, and only missed part of it.
Please take some addtional advice: Write a test plan for yourself.
Your explanations show that you missed a number of interesting test
configurations without noticing. Such things can of course happen, but
it can also be mostly avoided by better structuring of the work. Having
a plan and following that is one way. But there are other ways that
will work as well. Adding more eyes is another one. These can of course
be combined.
The point is that you need to do something actively to avoid repeating
this bummer.
> Your conclusion like this one is really offending, and I can't see any
> respect for my work from your reply.
Oh, I do absolutely respect your work. And I respect you, if that
wasn't clear either.
Please understand that I am testing your code only out of interest in
it. I am not after you in any way. In fact, I am very interested in
seeing how this goes. And I do appreciate all the work you've put into
coding this. That only makes the surprise bigger when you take these
short cuts wrt testing. I'd really expect that you by revision 9 would
have learned that there is no time to save doing that. On the
contrary...
Please accept my apologies for the lack of respect for your work in my
reply. I know I am often too harsh in email, and I should work on that.
It has never been my intention to do anything more than helping you get
this code ready and merged. I should have added a simple appreciation in
the initial bug report to make that clearer.
Please continue your good work! But please, please try to do more
testing yourself before reposting it another time. You don't want me or
anyone else to find bugs like this one.
> I am sorry for my mistake, and I always welcome for any feedbacks, but
> just please be with respect. If next time I still see offending reply
> like this, I will just ignore you. Sorry, but this is my right. :(
Of course. I appreciate that you let me know, and hope my explanation
above is enough to make you understand that the lack of respect in that
reply doesn't reflect my respect for you or your work.
Bjørn
prev parent reply other threads:[~2013-05-28 7:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-27 4:16 [Patch net-next v9 00/11] vxlan: add ipv6 support Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 01/11] vxlan: defer vxlan init as late as possible Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 02/11] vxlan: use unsigned int instead of unsigned Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 03/11] ipv6: make ip6_dst_hoplimit() static inline Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 04/11] ipv6: move ip6_local_out into core kernel Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 05/11] ipv6: export a stub for IPv6 symbols used by vxlan Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 06/11] ipv6: export in6addr_loopback to modules Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 07/11] vxlan: add ipv6 support Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 08/11] vxlan: add ipv6 route short circuit support Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 09/11] vxlan: add ipv6 proxy support Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 10/11] vxlan: respect scope_id for ll addr Cong Wang
2013-05-27 4:16 ` [Patch net-next v9 11/11] ipv6: Add generic UDP Tunnel segmentation Cong Wang
2013-05-27 8:39 ` [Patch net-next v9 00/11] vxlan: add ipv6 support Bjørn Mork
2013-05-27 8:57 ` Bjørn Mork
2013-05-27 9:30 ` David Miller
2013-05-27 10:54 ` Cong Wang
2013-05-27 11:12 ` Cong Wang
2013-05-27 11:00 ` Cong Wang
2013-05-27 13:07 ` Bjørn Mork
2013-05-28 2:42 ` Cong Wang
2013-05-28 7:28 ` Bjørn Mork [this message]
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=87vc63a7gq.fsf@nemi.mork.no \
--to=bjorn@mork$(echo .)no \
--cc=amwang@redhat$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=netdev@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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