|
Hello,
On Mon, 5 Mar 2018, Inju Song wrote:
> > Makefile and Kconfig should be a last patch, otherwise
> > on bisecting we will fail to compile.
> >
>
> Ok. I will move Makefile and Kconfig patch to last of the
> patchsets in the next v3 patchsets. Then do I include a patch about
> last_weight again?
I would say that you make the review from other people
difficult because they have to keep all previous submissions.
You should submit the 3 patches every time, with new PATCHv<N>
number. And these patches should not be relative to previous
versions but to the net-next kernel, i.e. the patchset should
be easy to include upstream. When all issues are resolved we
will ack the latest patchset version and the process ends,
patchset is submitted to other maintainers.
> > > --- a/net/netfilter/ipvs/ip_vs_mh.c
> > > +++ b/net/netfilter/ipvs/ip_vs_mh.c
> >
> > There must be license info at the beginning.
> >
>
> May be I will use GPLv1 license. Should I write the license info
> like other source(ie. the beginning of ip_vs_sh.c)?
Yes, at the beginning. According to
Documentation/process/license-rules.rst the GPLv1 is placed in
'other' dir (LICENSES/other/GPL-1.0) and so is not "recommended"
for new code. Also, instead of license text you can put such
line as first in ip_vs_mh.c:
// SPDX-License-Identifier: GPL-2.0
And that should be enough for the license. After that
line you can put more text which includes your name as Author,
info about the algorithm, links to used resources, etc.
For example:
/* The algorithm is detailed in:
* [3.4 Consistent Hasing]
*
https://static.googleusercontent.com/media/research.google.com/ko//pubs/archive/44824.pdf
*/
You can 'grep -r SPDX net/' for references.
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
|
