From: Matt Domsch <Matt_Domsch@dell.com>
To: Matt Mackall <mpm@selenic.com>
Cc: Clay Haapala <chaapala@cisco.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2.6.1 -- take two] Add CRC32C chksums to crypto and lib routines
Date: Wed, 18 Feb 2004 15:24:03 -0600 [thread overview]
Message-ID: <20040218152403.A30333@lists.us.dell.com> (raw)
In-Reply-To: <20040204172116.GF31138@waste.org>; from mpm@selenic.com on Wed, Feb 04, 2004 at 11:21:16AM -0600
On Wed, Feb 04, 2004 at 11:21:16AM -0600, Matt Mackall wrote:
> On Wed, Feb 04, 2004 at 10:14:13AM -0600, Clay Haapala wrote:
> > On Tue, 3 Feb 2004, Matt Mackall said:
> > > On Tue, Feb 03, 2004 at 05:25:08PM -0600, Matt Domsch wrote:
> > >> > >> +MODULE_LICENSE("GPL and additional rights");
> > >> > >
> > >> > > "additional rights?"
> > >> > >
> > >> > Take it up with Matt_Domsch@dell.com -- it's his code that I
> > >> > cribbed, so that's the license line I used.
> > >>
> > >> The crc32 code came from linux@horizon.com with the following
> > >> copyright abandonment disclaimer, which is still in lib/crc32.c:
> > >>
> > >> /*
> > >> * This code is in the public domain; copyright abandoned.
> > >> * Liability for non-performance of this code is limited to the
> > >> * amount you paid for it. Since it is distributed for free, your
> > >> * refund will be very very small. If it breaks, you get to keep
> > >> * both pieces. */
> > >>
> > >> Thus GPL plus additional rights is appropriate.
> > >
> > > Ok, makes sense for CRC32 stuff which can be easily lifted from the
> > > kernel or 50 other places, but not for stuff that's depends on
> > > cryptoapi.
> >
> > Matt is correct about crypto/crc32c.c -- that should be simply "GPL".
> >
> > As for the derived-from-public-domain-CRC32 stuff, should one even
> > claim "GPL" on it? That would be, in effect, licensing public-domain
> > code and placing restrictions on it, something only a copyright holder
> > should be able to do, and not the intent of the author, in this case.
>
> Hard to guess what the author's intent was as he left no license, but
> perhaps you're right. However, public domain works are routinely
> relicensed by publishers when they make a trivially derived work, see
> any reprint of Shakespeare or Bach.
>
> As has been pointed out, _not_ putting some sort of license on it
> potentially opens people who ship it up to liability. Arguably, by
> compiling it into the kernel, you're accepting the GPL liability terms
> for that use. But that doesn't stop someone from taking crc32.c,
> incorporating it into something else, having it blow up disastrously,
> and then suing whoever sold them the kernel tarball. Sounds
> outlandish, but crazier things have happened.
>
> As "dual GPL/public domain license" is an oxymoron, the best thing to
> do is probably to slap a dual GPL/2-clause BSD license on it to
> disclaim liability while minimally limiting all other rights. Matt,
> since you're the last one to touch this, I'll let you make the call,
> but here's what I would suggest (still needs an actual copyright
> notice):
After seeking advice from Dell's lawyers, they recommend simply adding
the GPL license text to the top of the file and be done with it.
It's public domain, we're free to include (and relicense) it as we
wish. If someone else wants to use it in a non-GPL fashion, they'll
need to start from the original public domain submission, not this one
which clearly has been modified somewhat since we first received it,
with faster algorithms, creation of the table at compile time, etc.
Thanks,
Matt
--
Matt Domsch
Sr. Software Engineer, Lead Engineer
Dell Linux Solutions linux.dell.com & www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com
next prev parent reply other threads:[~2004-02-18 21:25 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-01-14 21:31 [PATCH] Add CRC32C chksums to crypto routines Clay Haapala
2004-01-14 21:45 ` James Morris
2004-01-14 22:12 ` Clay Haapala
2004-01-16 1:40 ` Matt Mackall
2004-01-19 20:13 ` James Morris
2004-01-19 21:15 ` Clay Haapala
2004-01-19 21:33 ` James Morris
2004-02-03 16:58 ` [PATCH 2.6.1] Add CRC32C chksums to crypto and lib routines Clay Haapala
2004-02-03 17:09 ` James Morris
2004-02-04 17:07 ` Clay Haapala
2004-02-03 17:13 ` James Morris
2004-02-03 17:50 ` [PATCH 2.6.1 -- take two] " Clay Haapala
2004-02-03 18:51 ` Matt Mackall
2004-02-03 19:13 ` Clay Haapala
2004-02-03 19:27 ` Matt Mackall
2004-02-03 20:17 ` James Morris
2004-02-03 23:25 ` Matt Domsch
2004-02-03 23:37 ` Matt Mackall
2004-02-04 16:14 ` Clay Haapala
2004-02-04 17:21 ` Matt Mackall
2004-02-07 15:20 ` Matt Domsch
2004-02-18 21:24 ` Matt Domsch [this message]
2004-02-18 23:18 ` Matt Domsch
2004-02-06 23:54 ` Jamie Lokier
2004-02-04 2:18 ` James Morris
2004-02-04 3:11 ` Valdis.Kletnieks
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=20040218152403.A30333@lists.us.dell.com \
--to=matt_domsch@dell.com \
--cc=chaapala@cisco.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mpm@selenic.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