From: Matthew Fioravante <matthew.fioravante@jhuapl.edu>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: "samuel.thibault@ens-lyon.org" <samuel.thibault@ens-lyon.org>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH 08/11] add tpmfront, tpm_tis, and tpmback drivers to mini-os
Date: Tue, 02 Oct 2012 10:17:08 -0400 [thread overview]
Message-ID: <506AF764.5040701@jhuapl.edu> (raw)
In-Reply-To: <1349175465.650.39.camel@zakaz.uk.xensource.com>
[-- Attachment #1.1: Type: text/plain, Size: 2939 bytes --]
On 10/02/2012 06:57 AM, Ian Campbell wrote:
>> diff --git a/extras/mini-os/include/tpm_tis.h b/extras/mini-os/include/tpm_tis.h
>> new file mode 100644
>> index 0000000..b463cea
>> --- /dev/null
>> +++ b/extras/mini-os/include/tpm_tis.h
>> @@ -0,0 +1,74 @@
>> +/*
>> + * Copyright (c) 2010-2012 United States Government, as represented by
>> + * the Secretary of Defense. All rights reserved.
>> + *
>> + * This program is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License version 2
>> + * as published by the Free Software Foundation; or, when distributed
>> + * separately from the Linux kernel or incorporated into other
>> + * software packages, subject to the following license:
>> + *
>> + * Permission is hereby granted, free of charge, to any person obtaining a copy
>> + * of this source file (the "Software"), to deal in the Software without
>> + * restriction, including without limitation the rights to use, copy, modify,
>> + * merge, publish, distribute, sublicense, and/or sell copies of the Software,
>> + * and to permit persons to whom the Software is furnished to do so, subject to
>> + * the following conditions:
>> + *
>> + * The above copyright notice and this permission notice shall be included in
>> + * all copies or substantial portions of the Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
>> + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
>> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
>> + * IN THE SOFTWARE.
>> + *
>> + * Based upon the files:
>> + * drivers/char/tpm/tpm_tis.c
>> + * drivers/char/tpm/tpm.c
>> + * from the Linux kernel, which are Copyright (C) 2006 IBM Corporation
> Given that then I don't think you have the authority to dual license
> code owned by IBM as GPL or the MIT style thing above -- these files are
> subject to the licensing terms of those original files (which looks to
> be GPL v2 only).
>
> It might be acceptable to have a GPL header as the primary followed by a
> "modifications by Matthew Fioravante/US Gov/Secretary Defense are also
> licensed under... .MIT thing ...." but I'd want to see a second opinion
> on that I think.
>
> I didn't check all the other licensing thoroughly but I expect similar
> comments will apply elsewhere too.
As far as I understand (I'm in no way a copyright lawyer), there is a
difference between copyright and license. What this is trying to convey
is that the changes are copyrighted by the US Govt and the license being
offered is GPLv2.
> Ian.
>
>
[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 1459 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2012-10-02 14:17 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-27 17:09 [PATCH 01/11] Add ioread/iowrite functions to mini-os Matthew Fioravante
2012-09-27 17:09 ` [PATCH 02/11] add posix io for blkfront Matthew Fioravante
2012-09-28 10:36 ` George Dunlap
2012-09-27 17:09 ` [PATCH 03/11] Add endian, byteswap, and wordsize macros to mini-os Matthew Fioravante
2012-09-28 10:48 ` George Dunlap
2012-09-27 17:09 ` [PATCH 04/11] Disable the mfn_is_ram() check, it doesn't work correctly on all systems Matthew Fioravante
2012-10-02 10:49 ` Ian Campbell
2012-10-02 14:20 ` Matthew Fioravante
2012-09-27 17:09 ` [PATCH 05/11] add CONFIG_XC conditional Matthew Fioravante
2012-09-28 11:18 ` George Dunlap
2012-09-28 13:59 ` Matthew Fioravante
2012-09-28 15:24 ` Matthew Fioravante
2012-09-28 15:26 ` Matthew Fioravante
2012-09-27 17:10 ` [PATCH 06/11] add select definition to sys/time.h when HAVE_LIBC is defined Matthew Fioravante
2012-09-27 17:10 ` [PATCH 07/11] setup fpu and sse in mini-os Matthew Fioravante
2012-09-27 17:10 ` [PATCH 08/11] add tpmfront, tpm_tis, and tpmback drivers to mini-os Matthew Fioravante
2012-10-02 10:57 ` Ian Campbell
2012-10-02 11:08 ` Ian Jackson
2012-10-02 14:17 ` Matthew Fioravante [this message]
2012-10-02 14:44 ` Ian Campbell
2012-09-28 10:16 ` [PATCH 01/11] Add ioread/iowrite functions " George Dunlap
2012-09-28 13:57 ` Matthew Fioravante
2012-09-28 15:39 ` George Dunlap
2012-10-02 9:10 ` Ian Campbell
2012-10-02 14:23 ` Matthew Fioravante
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=506AF764.5040701@jhuapl.edu \
--to=matthew.fioravante@jhuapl.edu \
--cc=Ian.Campbell@citrix.com \
--cc=samuel.thibault@ens-lyon.org \
--cc=xen-devel@lists.xen.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.