mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Martin Steigerwald <martin@lichtvoll.de>
To: debian-legal@lists.debian.org
Cc: Jens Axboe <axboe@kernel.dk>, Ben Finney <bignose@debian.org>,
	Martin Steigerwald <Martin.Steigerwald@proact.de>,
	Domenico Andreoli <cavok@debian.org>,
	919356@bugs.debian.org, Kristian Fiskerstrand <k_f@gentoo.org>,
	Nadia Yvette Chambers <nyc@holomorphy.com>,
	Arnaldo Carvalho de Melo <acme@redhat.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Jens Axboe <jens.axboe@oracle.com>
Subject: Re: Bug#919356: Licensing of include/linux/hash.h
Date: Tue, 12 Feb 2019 21:34:49 +0100	[thread overview]
Message-ID: <35245979.F6S9gYasto@merkaba> (raw)
In-Reply-To: <8a87da85-3aef-a113-f93f-c96ab465c96f@kernel.dk>

Jens Axboe - 12.02.19, 17:16:
> On 2/11/19 11:27 PM, Ben Finney wrote:
> > Martin Steigerwald <Martin.Steigerwald@proact.de> writes:
> >> Well the file has in its header:
> >> 
> >> /* Fast hashing routine for a long.
> >> 
> >>    (C) 2002 William Lee Irwin III, IBM */
> >> 
> >> /*
> >> 
> >>  * Knuth recommends primes in approximately golden ratio to the
> >>  maximum * integer representable by a machine word for
> >>  multiplicative hashing. * Chuck Lever verified the effectiveness
> >>  of this technique:
> >>  * http://www.citi.umich.edu/techreports/reports/citi-tr-00-1.pdf
> >>  *
> >>  * These primes are chosen to be bit-sparse, that is operations on
> >>  * them can use shifts and additions instead of multiplications for
> >>  * machines where multiplications are slow.
> >>  */
> >> 
> >> It has been quite a while ago. I bet back then I did not regard
> >> this
> >> as license information since it does not specify a license. Thus I
> >> assumed it to be GPL-2 as the other files which have no license
> >> boiler plate. I.e.: Check file is it has different license, if
> >> not, then assume it has license as specified in COPYING.
> >> 
> >> Not specifying a license can however also mean in this context that
> >> it has no license as the file contains copyright information from
> >> another author.
> > 
> > If a work (even one file) “has no license”, that means no special
> > permissions are granted and normal copyright applies: All rights
> > reserved, i.e. not redistributable. So, no license is grounds to
> > consider a work non-free and non-redistributable.
> > 
> > If, on the other hand, the file is to be free software, there would
> > need to be a clear grant of some free software license to that
> > work.
> > 
> > Given the confusion over this file, I would consider it a
> > significant
> > risk to just assume we have GPLv2 permissions without being told
> > that
> > explicitly by the copyright holder. Rather, the reason we are
> > seeking a clearly-granted free license for this one file, is
> > because we are trying to replace a probably non-free file with the
> > same code in it.
> > 
> > It seems we need to keep looking, and in the meantime assume we have
> > no free license in this file.
> 
> FWIW, fio.c includes the following mention:
> 
>  * The license below covers all files distributed with fio unless
> otherwise * noted in the file itself.
> 
> followed by the GPL v2 license. I'll go through and add SPDX headers
> to everything to avoid wasting anymore time on this nonsense.

Thank you, Jens, for settling this. I did not remember that one. It may 
very well be that I have seen this note as I initially packaged fio as my 
first package for Debian about 10 years ago.

I forwarded your mail and the one from Domenico with the SPDX patch to 
Debian bug

#922112 fio: hash.h is not DFSG compliant
https://bugs.debian.org/922112

which I closed before as you told already that hash.c is GPL-2.

Thanks,
-- 
Martin



  parent reply	other threads:[~2019-02-12 20:34 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-15 17:00 Domenico Andreoli
2019-01-23  8:50 ` Domenico Andreoli
2019-02-10 23:08   ` Kristian Fiskerstrand
2019-02-11  7:22     ` Domenico Andreoli
2019-02-11 12:43       ` Ulrich Mueller
2019-02-11 14:51         ` Ulrich Mueller
     [not found]       ` <2977287.v5rPGuGa1n@merkaba>
2019-02-11 22:55         ` Jens Axboe
2019-02-12  6:26         ` Bug#919356: " Ben Finney
2019-02-12  6:27         ` Ben Finney
2019-02-12  6:27         ` Ben Finney
2019-02-12 16:16           ` Jens Axboe
2019-02-12 17:16             ` [PATCH] Add the SPDX header to include/linux/hash.h Domenico Andreoli
2019-02-12 20:34             ` Martin Steigerwald [this message]
2019-02-12 22:08             ` Bug#919356: Licensing of include/linux/hash.h Ben Finney
2019-02-12 20:36           ` Martin Steigerwald

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=35245979.F6S9gYasto@merkaba \
    --to=martin@lichtvoll.de \
    --cc=919356@bugs.debian.org \
    --cc=Martin.Steigerwald@proact.de \
    --cc=acme@redhat.com \
    --cc=axboe@kernel.dk \
    --cc=bignose@debian.org \
    --cc=cavok@debian.org \
    --cc=debian-legal@lists.debian.org \
    --cc=jens.axboe@oracle.com \
    --cc=k_f@gentoo.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nyc@holomorphy.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®